fix(install): don't duplicate guidelines into a hand-merged CLAUDE.md#20
Merged
Merged
Conversation
… current task A passing thought dropped mid-task reads as a new request, so the agent rushes the big task to get to it. /sidenote reframes it as a parked later-task: log it verbatim to .claude/sidenotes.md, one-line ack, resume with scope and pace unchanged. Bare /sidenote flushes the list. Wired into the installer (copy on install, remove on uninstall) and documented in the README alongside /scalable, including the plain-text SIDENOTE: prefix for agents without slash commands.
…times-i-think-about
fenceInto only stripped its own <!-- karpathy-skills --> block before re-appending, so running the installer against a file where the guidelines were hand-merged (verbatim '## 1. Think Before Coding' heading, no markers) appended a second full copy. Detect that heading outside our block and skip with a warning instead; wire up the previously-dead --force flag as the override. Mirrors the activate hook's dedup so both paths behave the same.
…times-i-think-about
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.
Problem
An agent ran
npx @clawnify/greybeardagainst a~/.claude/CLAUDE.mdthat already contained the guidelines hand-merged (verbatim## 1. Think Before Codingheading, no<!-- karpathy-skills -->markers).fenceIntoonly strips its own fenced block before re-appending, so it didn't recognize the hand-merged copy and appended a second full copy — silent duplication in the user's config. Confirmed happening in the wild.Fix
hooks/greybeard-activate.jskeys on, so the installer and the activate hook now dedupe identically (comment cross-references to keep them in sync).--forceflag as the override (append anyway), documented in--help.Kept it as skip + warn, not auto-convert: wrapping an arbitrarily-interleaved hand-merge in markers means guessing its boundaries in someone's config file — too risky.
--forceis the escape hatch.Verification
Ran the real installer against throwaway HOMEs (not just typecheck):
--force→ appends the fenced block.npm run check-syncgreen; guidelines source untouched.