Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 4, 2025

  • Explore repository structure and understand current task selection mechanism
  • Run existing tests to verify baseline functionality
  • Modify task selection logic to use front-matter task_name instead of filename
  • Add validation: error if multiple files have the same task_name (without selectors)
  • Add validation: error if task file is missing task_name in front-matter
  • Implement selector-based filtering for tasks (allowing multiple prompts for one task)
  • Update all existing tests to include task_name in front-matter
  • Add new tests to verify task selection behavior
  • Update documentation
  • Final testing and verification
  • Address code review feedback:
    • Changed from array to single string variable for matching task file
    • Error immediately on duplicate match instead of collecting all matches
    • Removed taskFileErrors array, return first error immediately
    • Simplified selector matching by using includes directly (task_name already in includes)
Original prompt

Modify the code so that we can select tasks in the same way as we select rule. This will allow mulitple prompts for one task. Use front-matter. Have a front-matter named "task_name" and select on that. If more than one file matches, that is an error. If the taskfile is missing the task_name, that's also an error. Finally, the name of the file does matter anymore.


✨ 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 3 commits November 4, 2025 20:36
Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>
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] Modify code for task selection using front-matter Select tasks by task_name frontmatter field instead of filename Nov 4, 2025
Copilot AI requested a review from alexec November 4, 2025 20:44
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 make the requested changes.

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

Copilot AI commented Nov 4, 2025

@copilot make the requested changes.

All requested changes have been made in commit abc03a9:

  • Using single string variable instead of array for matching task file
  • Returning first error immediately instead of collecting errors
  • Simplified selector matching by using includes directly

All tests pass and manual verification confirms the behavior works correctly.

Copilot AI requested a review from alexec November 4, 2025 20:54
@alexec alexec marked this pull request as ready for review November 4, 2025 20:56
@alexec alexec merged commit c7165df into main Nov 5, 2025
2 checks passed
@alexec alexec deleted the copilot/modify-task-selection-logic branch November 5, 2025 02:54
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