Repository navigation
Generate one prompts/ file per Session prompt, checked by the golden-file test - #184
Merged
Merged
Conversation
…le test The prompts_page test now also renders each prompt as Markdown in prompts/ and fails, with a diff, on a missing, differing or stale file. UPDATE_PROMPTS replaces UPDATE_PROMPTS_PAGE and regenerates both. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e note The prompts/ note and AGENTS.md now say the titles and when sentences come from src/prompts_page.rs. Regenerating also deletes a stale directory in prompts/ rather than panicking on it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
9 tasks
This was referenced Sep 30, 2026
Merged
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
Every Session prompt is now checked in as Markdown under
prompts/. The same golden-file test that keeps the Prompts and skills page in sync renders these files.Each file contains a generated-from note,
# Title, the when sentence, and the text inside a fence longer than any backtick run in it.UPDATE_PROMPTS_PAGEis renamed toUPDATE_PROMPTS, with no alias. AGENTS.md and README.md are updated to match.prompts/is left out of the crate, and the test skips whenprompts/is absent.Evidence
prompts/directory is empty (a resume.md removed and a stray.md added gives the same result):UPDATE_PROMPTS=1 cargo test prompts_pagewrote all nine files and deletedstray.md. Running it a second time leftgit statusunchanged. The fullcargo testpasses andcargo clippy --all-targetsis clean.a_fence_is_longer_than_any_run_of_backticks_in_the_texta_prompt_file_carries_the_title_the_when_and_the_fenced_textchecking_prompts_names_each_missing_differing_and_stale_file: a missing file, a changed file, a stray file and a stray directory are all reported; regenerating clears every one.Merge Danger
Door: two-way
Only test code, docs and generated files change. The code that builds and sends Session prompts is untouched.
Blast Radius: contributors
Anyone with
UPDATE_PROMPTS_PAGEin a script or in muscle memory must switch toUPDATE_PROMPTS.Unaddressed findings
Standards
prompt-prefix: the spec defines file names by the page's entry ids, and the ids are also the page anchors, so a separate name field would duplicate them.fence,markdown,check_prompt_files) directly: the codebase-design skill permits internal seams used by the module's own tests.markdownis a generic name: it is private to a small test-only module, where it reads clearly.Spec
cargo test prompts_pageruns both. Keeping them apart lets the page andprompts/skip independently when either is absent.Closes #177
🤖 Generated with Claude Code