[WIP] Remove skills from resume mode output

[Copilot]

Plan: Skip Skills in Resume Mode

• Understand the codebase and how resume mode works
• Add check in discoverSkills() to skip skill discovery when resume mode is enabled (similar to findExecuteRuleFiles)
• Write unit tests to verify skills are not discovered in resume mode
• Run existing tests to ensure no regressions
• Manual verification of the fix
• Run full test suite to ensure all tests pass

Original prompt

When used in resume mode, skills (like tasks) should not be returned.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

This PR implements the feature to skip skills in resume mode, ensuring that when the -r (resume) flag is used, skill discovery is disabled, consistent with how rule files are already handled in resume mode.

• Adds early return in discoverSkills() method to skip skill discovery when resume mode is enabled
• Implements comprehensive unit test to verify skills are not discovered in resume mode

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
pkg/codingcontext/context.go	Adds resume mode check at the beginning of discoverSkills() to skip skill discovery when resume flag is enabled, mirroring the existing pattern used in findExecuteRuleFiles()
pkg/codingcontext/context_test.go	Adds table-driven test case that creates a skill file and verifies it is not included in the result when resume mode is enabled

[Copilot]

The nil check for cc.includes is unnecessary here. The includes field is always initialized to a non-nil value in the New() function (line 45: includes: make(selectors.Selectors)), so it can never be nil. The check cc.resume alone is sufficient, as the selector check is only needed for backward compatibility with external callers that might set the selector directly.

Since this matches the pattern in findExecuteRuleFiles() at line 501, you may want to remove the nil check in both locations for consistency and simplicity.

Suggested change

	if cc.resume || (cc.includes != nil && cc.includes.GetValue("resume", "true")) {
	if cc.resume || cc.includes.GetValue("resume", "true") {