style: run the formatter on three files that drifted - #18
Merged
Conversation
`vp fmt --check` fails on `apps/server/src/serverSettings.ts`, `docs/internals/glossary.md` and `packages/effect-codex-app-server/src/schema.test.ts`. All three are correctly formatted upstream, so the drift is the fork's own and predates the current sync. Nothing caught it because the upstream `CI` workflow targets Blacksmith runners this fork cannot reach, so it has never run here. Formatter output only. `serverSettings.ts` is byte-identical once whitespace is stripped. `schema.test.ts` also loses a trailing comma from an argument list, which JavaScript ignores. `glossary.md` repads a table separator row, which renders identically. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
github-actions
Bot
added
vouch:trusted
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
1 participant
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.
vp fmt --checkfails on three files:apps/server/src/serverSettings.tsdocs/internals/glossary.mdpackages/effect-codex-app-server/src/schema.test.tsAll three are correctly formatted in
pingdotgg/t3code, verified by checking out the upstream versions and running the formatter against them. The drift is this fork's own and predates the current sync. Nothing caught it because the upstreamCIworkflow targets Blacksmith runners this fork cannot reach, so it has never completed here. See #17.This is formatter output, nothing else.
serverSettings.tsis byte-identical to its previous content once all whitespace is stripped, so it is purely line joining.schema.test.tsalso drops a trailing comma from an argument list, which JavaScript ignores.glossary.mdrepads a table separator row, which renders identically.vp fmt --checkis clean across all 3599 files afterwards, and the two files that carry tests pass 52 tests between them.Model and harness: Claude Opus 5 via Claude Code, in T3 Code.