Skip to content

feat!: reset instruction surfaces to the unhobble bare baseline - #348

Merged
kyle-sexton merged 1 commit into
mainfrom
chore/unhobble-bare-baseline
Aug 11, 2026
Merged

feat!: reset instruction surfaces to the unhobble bare baseline#348
kyle-sexton merged 1 commit into
mainfrom
chore/unhobble-bare-baseline

Conversation

@kyle-sexton

Copy link
Copy Markdown
Contributor

What

Bare-baseline adoption for this repo (unhobble experiment user-global-opus-5-20260809-b70caa, fleet PR set with melodic-software/dotfiles#432):

  • Project CLAUDE.md blanked — not deleted, so it remains the canonical re-add surface with clean diff history.
  • Instruction keys (hooks, enabledPlugins) removed from .claude/settings.json where present; functional keys (env, worktree, marketplaces) kept.

Why

Two days of bare-model operation (probes + a 4h autonomous run producing a merged 78-file PR) logged zero observation-ledger rows defending any standing instruction line. Per official best-practices doctrine, lines whose removal causes no mistakes are cost. Rebuild rule: an instruction returns only with ledger evidence, each re-add citing its rows. Ledger: ~/.claude/plugins/data/claude-config-melodic-software/unhobble/user-global-opus-5-20260809-b70caa/.

Related

No linked issue — fleet-wide bare-baseline adoption executed interactively; the ledger directory above and dotfiles#432 are the tracking artifacts.

🤖 Generated with Claude Code

Bare-baseline adoption (experiment user-global-opus-5-20260809-b70caa):
project CLAUDE.md and AGENTS.md blanked -- not deleted, so they stay
the canonical re-add surfaces with clean diff history. Lines return
only with observation-ledger evidence, each re-add citing its rows.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@cursor

cursor Bot commented Aug 11, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@claude

claude Bot commented Aug 11, 2026

Copy link
Copy Markdown

Claude finished @kyle-sexton's task in 3m 29s —— View job


Review: reset instruction surfaces to the unhobble bare baseline

  • Gather context (diff, CLAUDE.md, REVIEW.md, distribution docs)
  • Understand the request (code review of this PR)
  • Review changed files for correctness/alignment
  • Post findings

Scope: AGENTS.md and CLAUDE.md both fully blanked (0 bytes; diff SHAs are the git empty-blob hash). No other files changed — despite the PR body's second bullet ("Instruction keys removed from .claude/settings.json where present"), this repo has no .claude/settings.json, so that bullet doesn't apply here; it appears to be boilerplate carried over from the fleet-wide PR template. Not a defect, just worth noting so reviewers don't go looking for a settings.json diff that doesn't exist.

Findings (posted inline)

  1. Process: no evidence of the cross-doc reconciliation self-review this repo requires for normative-doc changesAGENTS.md:29 (pre-PR content). distribution/governance-process.md defines a normative doc as any file "in conventions/ — or elsewhere in this repository — that states a rule another doc cites or assumes," and requires a cross-reference check before such a doc changes. distribution/README.md explicitly treats root AGENTS.md as exactly this kind of file (it's cited as the basis for downstream repos' periodic reconciliation obligation). The PR description points to the unhobble ledger as justification but shows no evidence the repo-internal cross-reference search was performed.

  2. Real information loss: the "Cursor Cloud specific instructions" section isn't duplicated anywhere elseAGENTS.md:75 (pre-PR content). Verified by grep that this repo's own dev-setup instructions (five npm install roots, Node version-pin gotcha, lefthook --no-verify guidance) exist nowhere else in the repo. README.md's Validation section — the doc this file pointed readers to — only lists 3 of the 5 npm roots (missing components/concurrency-policy and components/dependabot-policy, both of which have their own package.json/lockfile and back real test:*/lint:* npm scripts). This looks like genuine operational-content loss rather than "cost with no observation-ledger backing," and independently pre-existed as a gap in README.md that this deletion now makes unrecoverable from any file in the repo.

Not flagged

  • Verified via grep that no other file links to a specific anchor within AGENTS.md/CLAUDE.md (e.g. AGENTS.md#validating-a-change), so no dead-anchor links are introduced.
  • Verified the agent-orientation sync component (components/agent-orientation/orientation.md, distributed to downstream repos as their AGENTS.md) is a distinct file from the root AGENTS.md being blanked here — downstream consumers are unaffected by this change.
  • REVIEW.md is unaffected and still functions independently of CLAUDE.md.
    · Branch

Comment thread AGENTS.md
Comment thread AGENTS.md
@github-actions

Copy link
Copy Markdown

Claude has reviewed this PR 1 time. The lane skips further automatic reviews after 5; deleting this comment resets the count.

@kyle-sexton
kyle-sexton merged commit 69da22b into main Aug 11, 2026
43 checks passed
@kyle-sexton
kyle-sexton deleted the chore/unhobble-bare-baseline branch August 11, 2026 14:33
kyle-sexton added a commit to melodic-software/claude-code-plugins that referenced this pull request Aug 17, 2026
…mport (#2887)

## Summary

A one-line `CLAUDE.md` import bridges the synced `AGENTS.md` orientation
into
Claude Code sessions, per the official docs: Claude Code reads
`CLAUDE.md`, not
`AGENTS.md` (code.claude.com/docs/en/memory.md, fetched 2026-08-16), and
recommends creating a `CLAUDE.md` that imports it. Symlinks need admin
rights on
Windows, so the `@AGENTS.md` import line is the portable bridge.

## Fix

`CLAUDE.md` becomes exactly `@AGENTS.md` (single line, trailing
newline).

This is an unhobble re-add with evidence, not a restoration of prose:
the root
`CLAUDE.md` is deliberately 0 bytes from the bare-baseline resets
(melodic-software/standards#348), whose rule is that an instruction
returns only
with ledger evidence. The evidence cited here is the managed-file
silent-overwrite failure mode that `AGENTS.md` exists to prevent, plus
the
verified doc fact that `AGENTS.md` alone never loads into a Claude
session. One
import line, no restored prose.

## Verification

- `CLAUDE.md` is one line (`@AGENTS.md`) plus a trailing newline, 11
bytes.
- `AGENTS.md` confirmed non-empty at the default-branch HEAD before the
change
  (952 bytes, via `gh api` with the raw media type).
- `CLAUDE.md` is not a sync target in `melodic-software/standards`
`distribution/sync-manifest.yml` (only `AGENTS.md` is), so this edit is
  repo-owned and will not be overwritten by the next standards sync.
- No other files changed.

## Related

- melodic-software/standards#399 (orientation content)
- melodic-software/standards#397 (cloud-bootstrap component)

No linked issue

🤖 Generated with [Claude Code](https://claude.com/claude-code)

https://claude.ai/code/session_012tKxg98B3QySqEPf8UqwX3

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant