feat(code-habits): forbid referencing gitignored paths from shared surfaces#21
Open
amitkot wants to merge 2 commits into
Open
feat(code-habits): forbid referencing gitignored paths from shared surfaces#21amitkot wants to merge 2 commits into
amitkot wants to merge 2 commits into
Conversation
…rfaces `plans-and-investigations-location` already routed personal-process markdown to `.local/<kind>/`; this adds the missing companion rule that once a path is gitignored you must not cite it from issues, PRs, commit messages, committed code/docs, skills, or any other artefact other readers will see. Two remediation paths are spelled out: promote-to- shared, or summarise-inline. Motivating case: a session referenced a `.local/plans/<slug>.md` path in a GitHub issue body, producing a dangling pointer for everyone but the author. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Replace the bulleted list, surface-by-surface enumeration, and separate Why with a two-paragraph rule. The substance is the same — gitignored paths exist on one machine, so don't cite them from anywhere others will read; promote-to-shared or summarise-inline are the two fixes; TodoWrite/live transcript are exempt — but the bloat was hiding it. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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
plans-and-investigations-locationskill with a new "CRITICAL: never reference gitignored paths from shared surfaces" section. The skill already routed personal markdown to.local/<kind>/; this adds the companion rule that gitignored paths must not be cited from issues, PRs, commit messages, committed code/docs, skills, or any other artefact other readers will see.Why
The original skill covered where to put personal-process markdown but not what happens once you cite that gitignored path from a public surface. Recent session: a
.local/plans/<slug>.mdreference landed in a GitHub issue body, producing a dangling pointer for everyone but the author. The fix had to be inline summarisation after the fact; encoding the rule at the skill level prevents the same shape at reference-creation time. Applies generally to anythinggit check-ignoreflags, not just.local/.Test plan
SKILL.mdand confirm the new section reads cleanly.jqthe manifests to confirm versions (marketplace 1.19.0, code-habits plugin 0.5.0, SKILL.md 0.5.0)..local/foo.mdfrom a shared surface.🤖 Generated with Claude Code