Stop formatting files this repo doesn't own - #36
Open
bdelanghe wants to merge 1 commit into
Open
Conversation
`deno task ci` fails at its first step on an untouched checkout of main, and has
since 2026-08-08 — standard.yml runs 14, 17 and 19 are all red, and every PR
opened since has inherited it:
error: Found 3 not formatted files in 111 files
CLAUDE.md · .claude/settings.json · .github/workflows/front-desk-add.yml
All three are written by the org Claude harness, not by lone, and all three
arrive with no trailing newline. `deno fmt` here takes no path arguments, so it
covers Markdown, JSON and YAML too and rejects them. baobab carries the same
files unformatted and is green only because its task is path-scoped.
NOT FIXED BY RUNNING deno fmt. Two of the three forbid it in their own text:
CLAUDE.md's diff falls inside `managed by adopt-claude-harness; do not hand-edit
this block`, and front-desk-add.yml's header names itself a rolled copy of a
canonical template that audit-front-desk-add.sh checks the deployed copies
against. A formatting commit would fail that audit, and would be undone by the
next re-roll — red again, same three files. Run 19's title is "Re-roll the org
Claude harness baseline (#32)", which is presumably how it got here.
So they are excluded from fmt instead. They are not lone's source; the gate
exists for lone's TypeScript, and lint and check are untouched. The generator
side — emit a trailing newline, and test for it — is filed at
bounded-systems/.github#247, and until that lands this is the only lever this
repo actually has.
deno task ci green: fmt, lint, check, 330 tests.
Closes #35.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011HAtCZXs1B5mUVWf6iTmdp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #35. Unblocks #34, and every other PR against this repo.
deno task cifails at its first step on an untouched checkout ofmain, and has since 2026-08-08 —standard.ymlruns 14, 17 and 19 are all red:All three are written by the org Claude harness, not by lone, and all three arrive with no trailing newline.
deno fmthere takes no path arguments, so it covers Markdown, JSON and YAML too and rejects them. baobab carries the same files unformatted and is green only because its task is path-scoped (deno fmt --check src tests cli).Why
deno fmtis not the fixTwo of the three forbid it in their own text:
CLAUDE.md— the diff falls inside<!-- bounded-systems-org:begin — managed by adopt-claude-harness; do not hand-edit this block -->..github/workflows/front-desk-add.yml— "THIS FILE IS THE CANONICAL SOURCE. reroll-front-desk-add.sh rolls it into EACH org repo … audit-front-desk-add.sh checks the deployed copies against it." Reformatting it would fail that audit.And either way a formatting commit gets undone by the next re-roll — red again, same three files. Run 19's title is "Re-roll the org Claude harness baseline (#32)", which is presumably how it got here in the first place.
So they're excluded from
fmtinstead. They aren't lone's source; the gate exists for lone's TypeScript.lintandcheckare untouched, and nothing about how lone's own code is formatted changes.The permanent fix is upstream
Filed as bounded-systems/.github#247: have the generator terminate every file it writes with
\n, and test for it. Until that lands, this exclusion is the only lever this repo actually has — and it stays correct afterwards, since these files still aren't lone's to format.Verified
deno task cigreen —fmt --check,lint,check, 330 tests. Diff is 15 lines ofdeno.jsonc, no source touched.Generated by Claude Code