docs: fix stale repo structure in copilot-instructions (follow-up to #17)#22
Merged
Conversation
Follow-up to #17. The repository-structure diagram still listed the deleted prompt_loader.py and the per-category {name}-prompt.md files. Replace with the consolidated layout: categories.py (single source of truth for VALID_CATEGORIES) and the single shared prompts/prompt.md. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The 'Prompt Files' prose still described 8 identical per-category files and listed the consolidation as 'planned'. #17 completed it: update the location to the single prompt.md and reword the design/version notes. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR is a documentation-only follow-up that updates the repository structure diagram in .github/copilot-instructions.md to reflect the post-#17 layout (canonical categories.py and consolidated prompts/prompt.md).
Changes:
- Remove the now-deleted
prompt_loader.pyfrom the structure diagram. - Update the prompt layout depiction to show
prompts/prompt.mdas the shared rule-agnostic prompt andprompts/v0.6.1/as the archived per-category prompts. - Add
categories.pyto the diagram as the canonical home ofVALID_CATEGORIES.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Small documentation follow-up to #17. Brings
.github/copilot-instructions.mdin line with the consolidated layout #17 introduced. Two stale spots, both describing the pre-#17 structure:1. Repository Structure diagram
prompt_loader.py— deleted in Consolidate VALID_CATEGORIES; delete dead prompt_loader (TECHNICAL-REVIEW §4.1, §4.2) #17 (was dead production code){name}-prompt.mdfiles (writing-prompt.md,math-prompt.md, …) — deleted in Consolidate VALID_CATEGORIES; delete dead prompt_loader (TECHNICAL-REVIEW §4.1, §4.2) #17, consolidated into a single shared prompt2. "Prompt Files" section
What changed
categories.py(single source of truth forVALID_CATEGORIES) and the singleprompts/prompt.md;prompts/v0.6.1/clarified as the archived previous (per-category) prompts.prompts/prompt.md; design reworded to "a single rule-agnostic prompt shared across all 8 categories (consolidated in Consolidate VALID_CATEGORIES; delete dead prompt_loader (TECHNICAL-REVIEW §4.1, §4.2) #17)".Docs-only; no code or test changes.
🤖 Generated with Claude Code