Skip to content

Format the repository with Prettier - #42

Open
matthewnanas wants to merge 2 commits into
matt/abstractionsfrom
matt/format-repo
Open

matthewnanas wants to merge 2 commits into
matt/abstractionsfrom
matt/format-repo

Conversation

@matthewnanas

@matthewnanas matthewnanas commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Stack: 2 of 2, on top of #41. Review only after #41; this PR's diff is just formatting.

Summary

  • Adds Prettier 3.9.10 as a pinned dev dependency, with .prettierrc.json setting printWidth: 100. Everything else uses Prettier defaults, which already match the repo's style: double quotes, semicolons.
  • Adds pnpm format and pnpm format:check, and runs format:check in pnpm check, so CI now enforces formatting.
  • .prettierignore skips Markdown, so agent instructions, skills, and docs aren't rewritten, along with the lockfile and build output.
  • Commit 2 is only the output of pnpm format (128 files), with no hand edits.

Review notes

  • Review commit 1 (Add Prettier…). Skim commit 2.
  • The lockfile diff only adds prettier. The undici/zod security overrides are unchanged.
  • Open PRs that touch the same files will need a rebase and a pnpm format after this merges.
  • If this is merged without squashing, add commit 2's SHA to .git-blame-ignore-revs so git blame skips it.

Testing

  • pnpm lint, pnpm typecheck, and prettier --check . pass.
  • pnpm test: 421 passed, 101 skipped. That's identical to matt/abstractions with the same node_modules.

🤖 Generated with Claude Code

matthewnanas and others added 2 commits November 3, 2026 12:12
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
No behavior changes: output of `pnpm format` only.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@matthewnanas
matthewnanas added this pull request to stack #43 November 3, 2026 20:12
@matthewnanas matthewnanas changed the title matt/format repo Format the repository with Prettier Nov 3, 2026
@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedprettier@​3.9.10981009797100

View full report

@matthewnanas
matthewnanas marked this pull request as ready for review November 3, 2026 20:14
@matthewnanas
matthewnanas requested a review from a team as a code owner November 3, 2026 20:14
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