Skip to content

Remove redundant WithEmitTaskFrontmatter option - always emit frontmatter#121

Merged
alexec merged 3 commits into
mainfrom
copilot/remove-withemit-taskfrontmatter
Nov 21, 2025
Merged

Remove redundant WithEmitTaskFrontmatter option - always emit frontmatter#121
alexec merged 3 commits into
mainfrom
copilot/remove-withemit-taskfrontmatter

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 21, 2025

The WithEmitTaskFrontmatter option and -t flag were redundant. Task frontmatter contains metadata needed by AI agents and downstream tools, so there's no use case for omitting it.

Changes

  • Removed -t CLI flag - frontmatter now always emitted when present
  • Removed WithEmitTaskFrontmatter option from Context configuration API
  • Removed emitTaskFrontmatter field from Context struct
  • Updated integration test TestPrintTaskFrontmatter to verify default behavior
  • Updated documentation to reflect automatic frontmatter emission

Before/After

# Before: required -t flag
coding-context -t implement-feature

# After: always included
coding-context implement-feature

Both produce the same output with frontmatter at the beginning.

Original prompt

WithEmitTaskFrontmatter seems redundant. Remove it. Default to 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.

…task frontmatter

Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove WithEmitTaskFrontmatter default value Remove redundant WithEmitTaskFrontmatter option - always emit frontmatter Nov 21, 2025
Copilot AI requested a review from alexec November 21, 2025 05:14
@alexec alexec marked this pull request as ready for review November 21, 2025 05:16
@alexec alexec enabled auto-merge (squash) November 21, 2025 05:16
@alexec alexec merged commit 668c728 into main Nov 21, 2025
1 check passed
@alexec alexec deleted the copilot/remove-withemit-taskfrontmatter branch November 21, 2025 05:16
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