Clarify Applies to: placement to follow the title heading#8
Merged
Conversation
- Replace "first line of every memory" with explicit placement under the `# Title` heading in the synced applies-to block (SYNC-BLOCKS.md + both SKILL.md copies). - Align the standalone instruction in continuous-learning to match.
- Promote the forcing-function check in continuous-learning Stage A from a soft prompt to a hard gate, anchoring the "Do Not Save" table's `[Forcing-function]` label. - Correct the Bash row in the Tool Reference to describe its actual purpose (resolving the git repo name for `Applies to:`) instead of memory deletion. - Sync README's required mcs version with techpack.yaml and fix a heading-depth typo in a self-link.
- Refer to the subsection by name instead of leaking the markdown nesting depth.
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.
Why
The skill prose said
Applies to:belongs on the "first line of every memory," which conflicted with the templates (title heading first, then**Applies to:**). Models took the literal phrasing and started writing**Applies to:**above the# Learning:/# Decision:heading.Changes
Reword the synced applies-to block and the standalone instruction so both pin placement to the line immediately under the title heading. No template, hook, or settings changes.
Test plan
SYNC-BLOCKS.md→ expect "OK: identical across all three files" for all three tags.# Learning: …(or# Decision: …) and place**Applies to:**directly under it.