Skip to content

docs(extensions): clarify agent-context README and add config examples#3389

Open
yanukadeneth99 wants to merge 1 commit into
github:mainfrom
yanukadeneth99:docs/agent-context-readme
Open

docs(extensions): clarify agent-context README and add config examples#3389
yanukadeneth99 wants to merge 1 commit into
github:mainfrom
yanukadeneth99:docs/agent-context-readme

Conversation

@yanukadeneth99

Copy link
Copy Markdown

Description

Edited the agent-context README.md and agent-context-config.yml for clarity. Added the missing install command.

Testing

Ran the commands on a project to ensure it's correct. No other testing are required for this.

AI Disclosure

  • I did not use AI assistance for this contribution
  • I did use AI assistance

AI was used to check whether there were any active PRs/issues related to this, and then to create a suggested fix for the README. However, I only used that file as a reference and made the changes to the README.md myself.

Please do let me know if there are changes/suggestions to be done.

Rewrite the agent-context extension README to read as plain prose
instead of a bullet dump, and add the missing install/disable
commands (specify extension add/disable/enable agent-context).
Add inline example comments to agent-context-config.yml for
context_file/context_files.
@yanukadeneth99 yanukadeneth99 requested a review from mnriem as a code owner July 7, 2026 05:10
# managed by this extension (e.g. CLAUDE.md, AGENTS.md,
# .github/copilot-instructions.md). Set automatically from the active
# integration and regenerated during `specify init` or integration switches.
# EXAMPLE: context_file: CLAUDE.md

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The # EXAMPLE: context_file: CLAUDE.md here reads as if context_file is something you're expected to fill in. But empty is the intended default — that's what triggers the self-seed from agent-context-defaults.json based on the active integration. If someone copies the example and pins CLAUDE.md, they've overridden the auto-resolution, so switching agents later (e.g. to Copilot) will keep writing the managed block to CLAUDE.md instead of .github/copilot-instructions.md.

Could we reframe it so it's clear this is an override, not a required value? Something like:

# Leave empty to auto-resolve from the active integration (recommended).
# Only set this to override the auto-detected file, e.g.:
#   context_file: CLAUDE.md
context_file: ""

The context_files example below is fine as-is, since multi-anchor sync genuinely requires the user to list paths explicitly.

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