Skip to content

chore: streamline formatting and pre-commit checks - #40

Merged
66HEX merged 1 commit into
masterfrom
chore/oxfmt-precommit
Aug 13, 2026
Merged

chore: streamline formatting and pre-commit checks#40
66HEX merged 1 commit into
masterfrom
chore/oxfmt-precommit

Conversation

@66HEX

@66HEX 66HEX commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores

    • Replaced Prettier-based formatting with Oxfmt across the repository.
    • Added consistent formatting support for Svelte and Tailwind CSS.
    • Updated formatting and validation scripts, including pre-commit checks.
  • Refactor

    • Standardized formatting across source code, tests, and configuration files.
    • No functional or observable behavior changes.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: fc0bfbf1-2813-442f-9290-ed06c9a7c708

📥 Commits

Reviewing files that changed from the base of the PR and between b4f66de and 13d5ea0.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (20)
  • .githooks/pre-commit
  • .oxfmtrc.json
  • .prettierignore
  • .prettierrc
  • apps/web/package.json
  • apps/web/src/routes/+layout.svelte
  • apps/web/src/routes/[section]/+layout.svelte
  • package.json
  • packages/motion-gpu/.prettierignore
  • packages/motion-gpu/package.json
  • packages/motion-gpu/src/lib/core/frame-registry.ts
  • packages/motion-gpu/src/lib/core/types.ts
  • packages/motion-gpu/src/tests/FragCanvas.runtime.test.ts
  • packages/motion-gpu/src/tests/core/compute-comprehensive.test.ts
  • packages/motion-gpu/src/tests/core/passes.test.ts
  • packages/motion-gpu/src/tests/core/renderer.test.ts
  • packages/motion-gpu/src/tests/fixtures/MotionGPUUserTypedNamespaceProbe.svelte
  • packages/motion-gpu/src/tests/react-frag-canvas.runtime.test.tsx
  • packages/motion-gpu/src/tests/vue-frag-canvas.runtime.test.ts
  • pnpm-workspace.yaml
💤 Files with no reviewable changes (3)
  • packages/motion-gpu/.prettierignore
  • .prettierignore
  • .prettierrc

📝 Walkthrough

Walkthrough

The repository replaces Prettier with Oxfmt, updates formatting scripts and hooks, adds Oxfmt configuration, and reformats affected TypeScript and Svelte declarations without changing runtime behavior.

Changes

Formatting migration

Layer / File(s) Summary
Oxfmt tooling and scripts
.oxfmtrc.json, package.json, pnpm-workspace.yaml, .githooks/pre-commit, apps/web/package.json, packages/motion-gpu/package.json, .prettierignore, packages/motion-gpu/.prettierignore
Oxfmt replaces Prettier in repository, web, and package scripts. The workspace catalog and pre-commit hook use the new formatting workflow.
Web template formatting
apps/web/src/routes/+layout.svelte, apps/web/src/routes/[section]/+layout.svelte
JSON-LD script element expressions use double-quoted string syntax without changing structured-data rendering.
Source and test declaration formatting
packages/motion-gpu/src/lib/core/*, packages/motion-gpu/src/tests/*
Union type declarations, type assertions, and a generic parameter list use Oxfmt-compatible formatting without changing types or test behavior.

Estimated code review effort: 2 (Simple) | ~10 minutes

Mergeability Score: ⚪ Minimal · up to 13d5e

This change is merge-ready after normal checks and review; no actionable merge-blocking risk remains.

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the formatting migration and pre-commit check updates in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/oxfmt-precommit

Comment @coderabbitai help to get the list of available commands.

@66HEX
66HEX merged commit af79f95 into master Aug 13, 2026
5 checks passed
@66HEX
66HEX deleted the chore/oxfmt-precommit branch August 13, 2026 21:05
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