You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
#3774 (PR #3775) overhauled the lint/format toolchain and added a format:check script:
npm run format:check # prettier --check '**/*.{js,jsx,ts,tsx}'
Running it against develop reports 626 tracked files that do not match the repository's Prettier config.
Why this is a separate issue
The drift is entirely pre-existing — it is not caused by #3774. Reformatting 626 files inside that PR would bury a tooling change in enormous unrelated churn, so it was deliberately scoped out. For the same reason, PR #3775 does not wire format:check into required CI: a gate that can never pass is worse than no gate.
Work required
Run npm run format:js (or prettier --write) across the tracked JS/TS files, ideally in reviewed batches by directory rather than one unreviewable commit.
Confirm the repository test suite still passes after each batch.
Consider extending the same treatment to .cjs files, which lint-staged's *.{js,jsx,ts,tsx} pattern does not cover — several config files (for example eslint.config.cjs) are non-conformant for that reason.
Notes
DEVELOPMENT.md currently documents format:check as informational with this backlog; update that note once the gate is enabled.
Context
#3774 (PR #3775) overhauled the lint/format toolchain and added a
format:checkscript:npm run format:check # prettier --check '**/*.{js,jsx,ts,tsx}'Running it against
developreports 626 tracked files that do not match the repository's Prettier config.Why this is a separate issue
The drift is entirely pre-existing — it is not caused by #3774. Reformatting 626 files inside that PR would bury a tooling change in enormous unrelated churn, so it was deliberately scoped out. For the same reason, PR #3775 does not wire
format:checkinto required CI: a gate that can never pass is worse than no gate.Work required
npm run format:js(orprettier --write) across the tracked JS/TS files, ideally in reviewed batches by directory rather than one unreviewable commit.format:checkis clean, wire it back as a CI job (it was removed from.github/workflows/lint.ymlin chore: improve code-quality tooling, OpenCode diagnostics, and lint performance (#3774) #3775) so the standard is enforced going forward..cjsfiles, which lint-staged's*.{js,jsx,ts,tsx}pattern does not cover — several config files (for exampleeslint.config.cjs) are non-conformant for that reason.Notes
DEVELOPMENT.mdcurrently documentsformat:checkas informational with this backlog; update that note once the gate is enabled.docs/mustache-repo-templatesis now correctly ignored via the.prettierignorefix in chore: improve code-quality tooling, OpenCode diagnostics, and lint performance (#3774) #3775.