Skip to content

docs(decisions): withdraw the wrong sc-1465 reference from PR #377's note - #381

Open
norvalbv wants to merge 1 commit into
mainfrom
fix/withdraw-wrong-sc-1465-reference
Open

docs(decisions): withdraw the wrong sc-1465 reference from PR #377's note#381
norvalbv wants to merge 1 commit into
mainfrom
fix/withdraw-wrong-sc-1465-reference

Conversation

@norvalbv

Copy link
Copy Markdown
Owner

PR #377 shipped its decision note under the header sc-1465 follow-up. That reference is wrong and is withdrawn here.

sc-1465 is an unrelated bug: core.hooksPath is shared across linked worktrees and resolves to an absolute path under the main checkout, so a worktree's commits run the main checkout's pre-commit hook instead of its own committed copy. Its acceptance criterion is a per-worktree override (git config --worktree core.hooksPath with extensions.worktreeConfig enabled). That work is in flight on fix/sc-1465-hookspath-owner and is not done.

Nothing in #377 touches it — the self-terminating run log, the phase stamping, and the setup/repository manifest fail-closed assertions are all unrelated to worktree hook resolution.

How it happened: the ticket number was inferred from a local worktree directory name (.claude/worktrees/sc-1465-hookspath-owner) without reading the story. The real origin of that work is an autonomous P1 report claiming devkit review exits 0 on a drifted core.hooksPath — whose exit-code half did not reproduce, and whose log-opacity half did.

Recorded as an append-only correction rather than an edit, per the store's own rule. Docs-only; no behaviour change.

🤖 Generated with Claude Code

…note

PR #377 shipped its decision note under the header `sc-1465 follow-up`. That reference is wrong and is withdrawn here.

**sc-1465** is an unrelated bug: `core.hooksPath` is shared across linked worktrees and resolves to an absolute path under the *main* checkout, so a worktree's commits run the main checkout's `pre-commit` hook instead of its own committed copy. Its acceptance criterion is a per-worktree override (`git config --worktree core.hooksPath` with `extensions.worktreeConfig` enabled). That work is in flight on `fix/sc-1465-hookspath-owner` and is **not** done.

Nothing in #377 touches it — the self-terminating run log, the phase stamping, and the setup/repository manifest fail-closed assertions are all unrelated to worktree hook resolution.

**How it happened:** the ticket number was inferred from a local worktree directory name (`.claude/worktrees/sc-1465-hookspath-owner`) without reading the story. The real origin of that work is an autonomous P1 report claiming `devkit review` exits 0 on a drifted `core.hooksPath` — whose exit-code half did not reproduce, and whose log-opacity half did.

Recorded as an append-only correction rather than an edit, per the store's own rule. Docs-only; no behaviour change.

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

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@norvalbv, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 22 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 20a8f837-8332-431e-bb86-67bafe2d18aa

📥 Commits

Reviewing files that changed from the base of the PR and between 7ed8ed7 and f1f0d8b.

📒 Files selected for processing (1)
  • docs/decisions/review-gate-in-chain.md

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.

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