Skip to content

fix(meta): prettier and eslint skip other sessions' worktrees under .claude/worktrees/ - #295

Merged
PrzemekGalarowicz merged 1 commit into
mainfrom
claude/vigorous-ellis-ae8bd6
Sep 28, 2026
Merged

PrzemekGalarowicz merged 1 commit into
mainfrom
claude/vigorous-ellis-ae8bd6

Conversation

@PrzemekGalarowicz

Copy link
Copy Markdown
Contributor

Summary

A bare npm run format:check or npm run lint in a main checkout read every other Claude Code session's worktree under .claude/worktrees/<name>/. On 2026-09-28, with 9 sibling worktrees present, npm run check failed format:check with hundreds of [warn] lines, all for files inside those worktrees. They are excluded through .git/info/exclude, which prettier does not read, and ESLint 10's flat config does not skip dot-directories.

  • .prettierignore gains .claude/worktrees/.
  • eslint.config.mjs's global ignores gains .claude/worktrees/**.
  • Each entry has a comment matching the existing .markdownlint-cli2.jsonc .claude/worktrees entry: the reason, and the bound (tied to where Claude Code places worktrees today).
  • CHANGELOG.md: one dated ## [Unreleased] entry. Repo-meta only, so SKILLS_VERSION does not move.

This closes the prettier half that 6.13.1 measured and deferred.

Verification

A probe at .claude/worktrees/probe/ held a copy of HEAD plus a badly formatted .md and a .mjs with two lint errors.

format:check lint
Before exit 1, 64 files (the 2 probe files + 62 committed files the root .prettierignore excludes only at the root) exit 1, 2 errors in the probe .mjs
After exit 0 exit 0
Negative control: same files at repo root exit 1 (2 files) exit 1 (2 errors)

The probe was removed afterwards. check:changelog-entry is GREEN against origin/main.

Full npm run check did not go green locally, for two reasons outside this change:

  • In the authoring worktree it stops at check:reconcile. That worktree directory was reused, and it still held a leftover reconcile baseline from another feature (2026-09-27). All 129 of its unexplained flags were changed by commits merged since that anchor, and the other 2 are this PR's config files. CI has no baseline.
  • In a clean copy, every gate through check:reconcile passed. npm test had 1 failure out of 4256: pharn/floor/run-marker.test.mjs:564, which fails only on macOS (/var vs /private/var). It fails the same way on untouched origin/main, and it is being fixed in a separate session.

Not claimed

  • No test pins either new entry, unlike the markdownlint one in command-hygiene.test.mjs, so nothing stops one being removed.

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 25f90d78-3e30-489e-af67-cfd240b3b45c


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

…claude/worktrees/

A bare `npm run format:check` or `npm run lint` in a main checkout read every
nested Claude Code worktree under `.claude/worktrees/` (git-excluded via
`.git/info/exclude`, which prettier does not read; ESLint 10 flat config does
not skip dot-directories). `.prettierignore` gains `.claude/worktrees/` and
`eslint.config.mjs` gains `.claude/worktrees/**`, each with a comment matching
the `.markdownlint-cli2.jsonc` entry. Repo-meta only: no SKILLS_VERSION bump.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@PrzemekGalarowicz
PrzemekGalarowicz force-pushed the claude/vigorous-ellis-ae8bd6 branch from 402c454 to a1dd982 Compare September 28, 2026 11:18
@PrzemekGalarowicz
PrzemekGalarowicz merged commit a2b5f6b into main Sep 28, 2026
8 checks passed
@PrzemekGalarowicz
PrzemekGalarowicz deleted the claude/vigorous-ellis-ae8bd6 branch September 28, 2026 11:23
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