Tighten the handout editor and finalizer skills - #386
Merged
Conversation
Two rules the handouts already follow but neither skill stated. An \eqno earns its number only when the text points back at it from further than the paragraph below, so a display cited once in the next sentence stays untagged. And hideSolutionsAndProofs gates the UI only: the CLI never reads handouts.json, so the full PDF is still built, uploaded and fetchable on R2, which matters when the flag is meant to keep solutions from a class. The rest is an audit pass. The \EnvId naming rule was stated whole in both skills, so handout-editor now owns it (plus the two halves only the finalizer carried: names are unique within a file, and the competition source spoils a problem the same way the technique does) and the finalizer points at it. The dash rule no longer says "his prose" and "he asked for it here", which a fresh reader has to decode, and the editor drops an opening persona line its own description already carries. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
1 participant
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.
Two rules the handouts already follow but neither skill stated:
\eqnoearns its number only when the text points back at it from further than the paragraph directly below. A display cited once in the next sentence stays untagged, and a block's surviving numbers run from(1).hideSolutionsAndProofsgates the UI only. The CLI never readshandouts.json, so the full-solutions PDF is still built, uploaded and fetchable at its R2 URL — worth saying out loud when the flag is being used to keep solutions from a class.The rest is an audit pass against the current skill-authoring docs:
\EnvIdnaming rule was stated whole in both skills.handout-editornow owns it, including the two halves only the finalizer carried (a name is unique within its file, and the competition source spoils a problem the same way the technique does), andhandout-finalizepoints at it instead of restating it.handout-editordrops an opening persona line that its own description already carries into the system prompt.🤖 Generated with Claude Code