Skip to content

Set up scoped Oxfmt formatting - #371

Draft
donatj with Copilot wants to merge 2 commits into
masterfrom
copilot/set-up-oxfmt
Draft

Set up scoped Oxfmt formatting#371
donatj with Copilot wants to merge 2 commits into
masterfrom
copilot/set-up-oxfmt

Conversation

Copilot AI commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Adds Oxfmt alongside Oxlint and limits formatting to frontend TypeScript and Sass sources.

  • Formatter tooling

    • Adds oxfmt as a development dependency.
    • Adds .oxfmtrc.json to retain tab indentation.
  • Project command

    • Adds make fmt, scoped to ts/ and scss/.
    ./node_modules/.bin/oxfmt -c .oxfmtrc.json ts/ scss/
  • Formatting baseline

    • Applies Oxfmt to the existing TypeScript and Sass sources.
    • Regenerates affected browser assets.
    • Documents the formatter command in the README.

Copilot AI linked an issue Sep 6, 2026 that may be closed by this pull request
Co-authored-by: donatj <133747+donatj@users.noreply.github.com>
Copilot AI changed the title [WIP] Set up Oxfmt configuration for /ts and /scss Set up scoped Oxfmt formatting Sep 6, 2026
Copilot AI requested a review from donatj September 6, 2026 11:50
@donatj
donatj requested a balanced review from Copilot September 6, 2026 16:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

All reviewed changes are scoped to formatter setup, formatting, and generated assets, with no unresolved issues.

Pull request overview

Adds scoped Oxfmt tooling for frontend TypeScript and Sass sources, establishing a formatting baseline and regenerating browser assets.

Changes:

  • Adds Oxfmt configuration, dependency, and make fmt.
  • Formats ts/ and scss/ sources.
  • Regenerates assets and documents the formatter command.
File summaries
File Description
ts/main.ts Applies formatting.
ts/Home.ts Applies formatting.
ts/Dashboard.tsx Reformats dashboard JSX and expressions.
ts/Controllers/ApiExampleController.ts Reformats controller code and examples.
ts/api/tokens.ts Applies formatting.
ts/api/shields.ts Applies formatting.
ts/api/request.ts Applies formatting.
ts/api/env.ts Applies formatting.
ts/api/authed.ts Applies formatting.
ts/AbstractController.ts Reformats class declarations.
static/style/style.css Regenerates compiled CSS.
static/main.js Regenerates the browser bundle.
scss/style.scss Applies Sass formatting.
scss/_variables.scss Normalizes whitespace.
scss/_shared.scss Reformats shared styles.
scss/_reset.scss Reformats selector lists.
scss/_mixins.scss Applies formatting.
scss/_home.scss Normalizes spacing and decimals.
scss/_dashboard.scss Normalizes dashboard styles.
scss/_code.scss Reformats font lists and whitespace.
README.md Documents make fmt.
package.json Adds the Oxfmt dependency.
package-lock.json Locks Oxfmt and platform packages.
Makefile Adds the scoped formatter target.
.oxfmtrc.json Configures tab indentation.
Review details

Files not reviewed (1)

  • static/style/style.css: Generated file
  • Files reviewed: 21/25 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

Set up Oxfmt

3 participants