Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 6, 2025

  • Explore repository structure and understand codebase
  • Build and test current implementation (all tests pass)
  • Refactor resume mode to use standard selector mechanism
  • Update -r flag to add -s resume=true selector and skip rules
  • Update tests to use -s resume=false for normal mode
  • Update documentation to reflect new approach
  • Fix indentation in integration_test.go using gofmt
  • Fix matchesIncludes to handle boolean frontmatter values
  • All tests pass (15 integration tests, 12 unit tests)

Summary

Fixed tests after @alexec's changes to support boolean values in frontmatter:

  • Updated matchesIncludes to use fmt.Sprint(fmValue) to convert any frontmatter value type to string before comparison
  • This allows proper comparison when YAML parses resume: true/false as boolean values
  • All tests now pass including the new boolean value test
Original prompt

I want the tool to be "resume" aware. when a "-r" flag is passed, then we're resuming. We don't need to restate the rules. Thes should not be output. Which task prompt? The one labelled with "resume: true".


✨ 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 AI and others added 2 commits November 6, 2025 00:09
Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>
Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>
Copilot AI changed the title [WIP] Add resume awareness with -r flag support Add resume mode (-r flag) to skip rules and select resume-specific task prompts Nov 6, 2025
Copy link
Contributor

@alexec alexec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot please make the changes.

main.go Outdated
}

// Filter based on resume mode
resumeValue, hasResume := frontmatter["resume"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot "resume" can be a standard front-matter filter. We should be able to have two prompts for the same task.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Refactored to use standard frontmatter filtering. The resume field is now a regular selector field, and -r is shorthand for -s resume=true + skip rules. This allows multiple task prompts for the same task name. See commit 07213fe.

Initial task (`.agents/tasks/fix-bug-initial.md`):
```markdown
---
task_name: fix-bug
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot to exclude from resume, "resume: false".

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated documentation and examples to show resume: false for initial tasks. Users can now use -s resume=false to explicitly select non-resume tasks. See commit 07213fe.

Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 6, 2025

@copilot please make the changes.

Changes completed. Refactored resume mode to use standard frontmatter filtering mechanism as requested. The -r flag now adds -s resume=true selector and skips rules, allowing multiple task prompts for the same task name. See commit 07213fe.

Copilot AI changed the title Add resume mode (-r flag) to skip rules and select resume-specific task prompts Add resume mode (-r flag) using standard frontmatter filtering Nov 6, 2025
Copilot AI and others added 2 commits November 6, 2025 00:24
Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>
Copilot AI requested a review from alexec November 6, 2025 00:25
@alexec
Copy link
Contributor

alexec commented Nov 6, 2025

@copilot I have made changes. Please fix the tests.

Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 6, 2025

@copilot I have made changes. Please fix the tests.

Fixed. Updated matchesIncludes to use fmt.Sprint(fmValue) to properly compare frontmatter values (which can be booleans from YAML) with selector strings. All tests now pass. See commit b6276cc.

Copilot AI requested a review from alexec November 6, 2025 00:43
@alexec alexec marked this pull request as ready for review November 6, 2025 00:43
@alexec alexec enabled auto-merge (squash) November 6, 2025 00:44
@alexec alexec merged commit 1f57b56 into main Nov 6, 2025
2 checks passed
@alexec alexec deleted the copilot/add-resume-functionality branch November 6, 2025 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants