Skip to content

Replace the per-repo reference files with a template - #44

Merged
jasperboerhof merged 1 commit into
mainfrom
lift/generic-repo-references
Sep 7, 2026
Merged

jasperboerhof merged 1 commit into
mainfrom
lift/generic-repo-references

Conversation

@jasperboerhof

Copy link
Copy Markdown
Contributor

Why

The catalog shipped three real repos' house rules as reference files. A public catalog carrying a file named crit.md reads as a dependency, not an example. Readers concluded they need access to the crit repo.

This is also what the repo's own CLAUDE.md already asks for: skills are generalised on lift, "abstracting away project-specific config". These three files were the one place that never happened.

What

Deleted:

  • skills/shepard/references/crit.md
  • skills/worktree/references/crit.md
  • skills/worktree/references/kendo.md

Added: references/_template.md in each skill. It carries the section structure both SKILL.md files already document, says what each section is for, and states in its first line that it is a template, not a dependency. Running on the defaults is the normal path, not a gap.

Pointers updated. shepard/SKILL.md § Adding a repo reference file said "references/crit.md is the model." worktree/SKILL.md said the same of kendo.md, and its frontmatter said "(kendo has one)". All three now point at the template. Both skills' load steps now say the no-file path is the normal one.

Two more crit mentions in shepard/SKILL.md, same defect class:

  • The reviewer-resolves row named crit as its example. The row already describes the camp; the parenthetical is gone.
  • The town-crier paragraph now leads with the bus surface being optional, and says its env-file default points at one personal checkout as a convenience.

Not changed

pr-watch.sh:84 still defaults TOWN_CRIER_ENV_FILE to $HOME/Code/crit/.env. Changing it would silently break the bus surface for whoever relies on that default, so it is a behaviour decision rather than a docs fix. The paragraph above it now says what it is. Raised separately.

Records under docs/plans/ still name crit.md. They are dated accounts of what was done at the time, not standing guidance, so they are left as history.

Test plan

  • pr-watch.test.sh 16/16, ci-failures.test.sh 33/33, verify-citations.test.sh 32/32.
  • grep -rniE '\bcrit\b|Code/crit' skills/ --include=*.md returns nothing.
  • Both ](references/_template.md) links resolve.
  • No pointer to a deleted file remains in skills/ or README.md.
  • CI green — this repo has no CI. The three suites above are run by hand.

🤖 Generated with Claude Code

https://claude.ai/code/session_01TfPDz4Gk4CcneEykXSKUoa

The catalog shipped three real repos' house rules: shepard/references/crit.md,
worktree/references/crit.md and worktree/references/kendo.md. A public catalog
carrying a file named crit.md reads as a dependency. Readers concluded they need
access to the crit repo.

Each skill now ships references/_template.md instead. It carries the section
structure both SKILL.md files already document, says what each section is for,
and states plainly that no file here is required. Running on the defaults is the
normal path, not a gap.

This is what the repo's own CLAUDE.md asks for. Skills are generalised on lift,
with project-specific config abstracted away. These three files were the one
place that never happened.

Nothing operational is lost. The live copies under ~/.claude/skills/ are what
actually run, they already hold more repos than the catalog did, and all three
catalog files had drifted from them.

Also generalised two crit mentions in shepard/SKILL.md: the reviewer-resolves row
no longer names a repo, and the town-crier paragraph now says the bus surface is
optional and its env-file default is one person's convenience.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TfPDz4Gk4CcneEykXSKUoa
@jasperboerhof jasperboerhof added the Agent Review Requested Ask an agent to review this PR label Sep 7, 2026
@jasperboerhof
jasperboerhof merged commit ac6cff4 into main Sep 7, 2026
@jasperboerhof
jasperboerhof deleted the lift/generic-repo-references branch September 7, 2026 11:36
bjvle added a commit that referenced this pull request Oct 2, 2026
Finding #43 gains the retest after d508795: a manual /compact answered
end to end under the resolved Git bash. Finding #44 records the first
trigger:'auto' compaction answered by the module, with core never run,
and the fork's cost at ~158K cache-read tokens. design.md narrows the
"Still unmeasured" list accordingly.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Agent Review Requested Ask an agent to review this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant