Skip to content

chore(sync): record CLAUDE.md and the bench scaling as skipped - #439

Merged
IgorShevchik merged 1 commit into
mainfrom
sync/nuxt-6f82268
Aug 19, 2026
Merged

chore(sync): record CLAUDE.md and the bench scaling as skipped#439
IgorShevchik merged 1 commit into
mainfrom
sync/nuxt-6f82268

Conversation

@IgorShevchik

Copy link
Copy Markdown
Collaborator

First two of six in the current queue. Both deliberately not taken; adjacent, so batched per PORTING.md §6 step 4b. No code changes.

1. nuxt/ui@6f82268badd CLAUDE.md importing AGENTS.mdskip

Three lines adding a tracked CLAUDE.md that imports AGENTS.md. We have AGENTS.md (156 lines) and no CLAUDE.md, so it would apply cleanly — and would still be wrong:

# AI
CLAUDE.md
.claude/settings.local.json
.claude/channels/
.claude/debug/

CLAUDE.md is in our .gitignore at line 51. The company it keeps settles it: this fork classifies the file as per-developer Claude Code state, not as tracked instructions. Porting the commit would mean deleting that line and reversing a standing decision — not something a sync port should do on its own authority.

The trade-off is named rather than hidden: upstream's comment says Claude Code auto-loads only CLAUDE.md, so our tracked AGENTS.md is not loaded automatically, and each developer supplies their own ignored copy. If that is judged a mistake, it is a two-line change — .gitignore line 51 plus this commit — and it should be made on its own merits.

2. nuxt/ui@3d2129d2scale tv benchmarks past codspeed's noise floorskip

Wraps every bench body in a 100-iteration loop, because CodSpeed runs them under an instruction-counting simulator and GitHub's runners alternate between Intel and AMD, so sub-millisecond bodies collect phantom regressions.

We have test/bench/tv.bench.ts and the change would apply cleanly. But this fork does not run CodSpeed: bench is a plain vitest bench --project vue, no workflow invokes it, and nothing compares runs across machines.

So it would buy nothing, cost a hundredfold longer local run, and add a comment describing CI we do not have. Worth taking the day benchmarks are wired into CI — and taking the comment with it.

A larger finding, surfaced by the first one

Checking why CLAUDE.md was ignored led to git log -S 'CLAUDE.md' -- .gitignoreb55bd3e7, a deps port. Its body never mentions CLAUDE.md, but it does say:

Skipped: ai + @ai-sdk/* (b24ui stays on the deferred v6 line)

Five other ledger entries say the same. That is what prompted #438: the ai v6 line was a deliberate, repeatedly reaffirmed deferral rather than drift, deferred for exactly the DeepSeek provider coupling that #425 then broke. Recorded in both logs here, and as a §2 invariant in #438.

Ledger

cursor3d2129d2. Both entries added with decision: skip and the reasoning in .sync/log/. .sync/dep-parity.json refreshed to the new cursor per the §6 step from #429 — neither commit touches a manifest, so only the cursor line moves and all 148 versions are identical.

Verify (CI=true)

lint and the test/utils suite (495 tests, 25 files). Documentation and ledger only; nothing executable changes.

Remaining in the queue: ae2bd5eb (Splitter, adopting), 2beb2345 + a630c943 (showcase, no-ops), 14ac2438 (ProgressGroup plus the --percent fix for the existing Progress).


Generated by Claude Code

Two contiguous upstream commits, both deliberately not taken, batched per
PORTING.md §6 step 4b. No code changes.

nuxt/ui@6f82268b `chore: add CLAUDE.md importing AGENTS.md` — skip. We have
`AGENTS.md` and no `CLAUDE.md`, so it would apply cleanly, and would still
be wrong: `CLAUDE.md` is in our `.gitignore` at line 51, under a `# AI`
heading, alongside `.claude/settings.local.json`, `.claude/channels/` and
`.claude/debug/`. The company it keeps settles it — this fork classifies
the file as per-developer Claude Code state, not tracked instructions.
Porting it would mean deleting that line and reversing a standing decision,
which a sync port is not the place to do.

The trade-off is named rather than hidden. Upstream's comment says Claude
Code auto-loads only `CLAUDE.md`, so our tracked `AGENTS.md` is not loaded
automatically and each developer supplies their own ignored copy. If that
is judged a mistake it should be changed on its own merits — `.gitignore`
line 51 and this commit — not carried in as a port.

nuxt/ui@3d2129d2 `test(bench): scale tv benchmarks past codspeed's noise
floor` — skip. It wraps every bench body in a 100-iteration loop because
CodSpeed runs them under an instruction-counting simulator on runners that
alternate between Intel and AMD. We have the file, but this fork does not
run CodSpeed: `bench` is a plain `vitest bench --project vue`, no workflow
invokes it, and nothing compares runs across machines. The change would buy
nothing, cost a hundredfold longer local run, and add a comment describing
CI we do not have. Worth taking the day benchmarks are wired into CI.

Also refreshes `.sync/dep-parity.json` to the new cursor, per §6. Neither
commit touches a manifest, so only the cursor line moves and all 148
versions are identical.

Verify (CI=true): lint and the `test/utils` suite (495 tests, 25 files).
Documentation and ledger only; nothing executable changes.
@IgorShevchik
IgorShevchik merged commit 167cd6f into main Aug 19, 2026
1 check passed
@IgorShevchik
IgorShevchik deleted the sync/nuxt-6f82268 branch August 19, 2026 13:54
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.

2 participants