Skip to content

docs(claude): sync the org-standards block - #143

Merged
LukasWodka merged 1 commit into
developfrom
docs/1602-org-standards-sync
Aug 26, 2026
Merged

docs(claude): sync the org-standards block#143
LukasWodka merged 1 commit into
developfrom
docs/1602-org-standards-sync

Conversation

@tracebloc-release-train

Copy link
Copy Markdown
Contributor

Managed sync of the org-standards block into this repo's CLAUDE.md — canonical
source: tracebloc/.github/org-standards.md. Do not hand-edit the block; to change
a rule, open a PR against tracebloc/.github and the sync propagates it.

Part of tracebloc/backend#1602 (org-wide engineering standards).

🤖 Generated with Claude Code

Managed sync from tracebloc/.github/org-standards.md.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@LukasWodkaLukasWodka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Content is fine — byte-identical managed sync of the org-standards block.

Red on the same required check as the whole batch: set-status / closing-ref, step "The title names a ticket, so the PR must link it". The gate reads backend#1602 from the title's trailing parens and wants it in the PR's link graph; Part of tracebloc/backend#1602 is a mention, not a link.

Don't follow the gate's printed advice here. Adding Closes tracebloc/backend#1602 would close the epic when the first of these twenty PRs merges, with nineteen still open — cross-repo closing keywords do fire in the owner/repo#n form.

The fix is in the sync generator: drop (backend#1602) from the title so the gate returns NOTHING_NAMED, and keep Part of … in the body for the cross-reference. Full write-up on tracebloc/.github#344.

Holding my review until the titles are regenerated — the content is right, the blocker isn't in this diff.

@LukasWodkaLukasWodka changed the title docs(claude): sync org-standards block (backend#1602)docs(claude): sync the org-standards blockAug 26, 2026

@LukasWodkaLukasWodka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Title fix worked — set-status / closing-ref is green here and across the batch. I approved the identical content on rfcs#54.

Holding my approval on the remaining red, which is structural rather than a finding:

Bugbot review gate: FAIL
No Bugbot check run on head …
Waited 900s over 46 attempt(s) and Bugbot never reported a terminal verdict.
Measured Bugbot latency is p50 164s / max 635s over 40 runs, so this is not slowness.

The author is app/tracebloc-release-train and Bugbot doesn't review bot PRs, so the verdict the gate waits for cannot arrive. Every managed sync into a Bugbot-gated repo burns 900s and reports red, permanently. rfcs has no Bugbot caller, which is the only reason it went green.

bugbot / review isn't a required context, so it doesn't block the merge — but I don't approve over a red check. Full write-up and the two options on tracebloc/.github#344; the better one is exempting bot-authored PRs in the caller rather than timing out on a verdict that structurally cannot come.

@LukasWodka
LukasWodka requested review from LukasWodka and saadqbal and removed request for saadqbalAugust 26, 2026 11:46
@LukasWodka
LukasWodka merged commit 224f67a into developAug 26, 2026
12 of 15 checks passed
@LukasWodka
LukasWodka deleted the docs/1602-org-standards-sync branch August 26, 2026 12:36
@LukasWodka
LukasWodka requested review from saqlainsyed007 and removed request for LukasWodkaAugust 26, 2026 13:16
Sign up for freeto 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

@LukasWodka@saqlainsyed007