Skip to content

Oxfmt #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Smef merged 2 commits into from
May 21, 2026
Merged

Oxfmt #9

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
60f46a2
add oxfmt
Smef May 21, 2026
e3a0608
sort tailwind
Smef May 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/workflows/format-check.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ on:

jobs:
format-check:
name: ESLint & Prettier
name: Format Checks
runs-on: ubuntu-latest
steps:
- name: Checkout
Expand All @@ -29,6 +29,6 @@ jobs:
- name: Run ESLint
run: pnpm lint

- name: Run Prettier
- name: Run Oxfmt
if: always()
run: pnpm prettier:check
run: pnpm fmt:check
3 changes: 2 additions & 1 deletion .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@ node_modules
.DS_Store
.fleet
.idea
.vscode
.vscode/*
!.vscode/extensions.json

# Cache files
*.tsbuildinfo
Expand Down
11 changes: 11 additions & 0 deletions .oxfmtrc.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
{
"$schema": "./node_modules/oxfmt/configuration_schema.json",
"printWidth": 120,
"sortPackageJson": false,
"ignorePatterns": ["docs/content/index.md", "docs/.nuxt", "docs/.data", "docs/.output"],
"sortTailwindcss": {
"stylesheet": "docs/assets/css/main.css",
"functions": ["clsx", "cn"],
"preserveWhitespace": true
}
}
5 changes: 0 additions & 5 deletions .prettierignore
View file
Open in desktop

This file was deleted.

3 changes: 0 additions & 3 deletions .prettierrc
View file
Open in desktop

This file was deleted.

3 changes: 3 additions & 0 deletions .vscode/extensions.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
{
"recommendations": ["oxc.oxc-vscode"]
}
1 change: 0 additions & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ Vasta is a type-safe Object Relational Mapper (ORM) layer for [Kysely](https://k

Vasta provides an [active record pattern](https://en.wikipedia.org/wiki/Active_record_pattern) interface for querying and manipulating your data, making it easy to work with your database in a model-instance oriented way. You can define your models with attributes, relationships, methods, and more. You can use these model properties to perform actions directly on your models before easily saving those changes to the database.


```js
const pet = await Pet.findOrFail(1);
pet.name = "Fluffy";
Expand Down
4 changes: 2 additions & 2 deletions docs/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This is the default Docus starter template that provides everything you need to
## ✨ Features

- 🎨 **Beautiful Design** - Clean, modern documentation theme
- 📱 **Responsive** - Mobile-first responsive design
- 📱 **Responsive** - Mobile-first responsive design
- 🌙 **Dark Mode** - Built-in dark/light mode support
- 🔍 **Search** - Full-text search functionality
- 📝 **Markdown Enhanced** - Extended markdown with custom components
Expand Down Expand Up @@ -79,4 +79,4 @@ The built files will be in the `.output` directory, ready for deployment to any

## 📄 License

[MIT License](https://opensource.org/licenses/MIT)
[MIT License](https://opensource.org/licenses/MIT)
2 changes: 1 addition & 1 deletion docs/content/1.getting-started/1.introduction.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,4 +33,4 @@ In this example, the `where` method is type-safe and will only allow valid colum

![Example type safety not allowing invalid columns, and a type error when an invalid column is entered](/img/content/1.getting-started/invalid-column.png)

![Example type safety not allowing values, and a type error when an invalid value is entered](/img/content/1.getting-started/invalid-value.png)
![Example type safety not allowing values, and a type error when an invalid value is entered](/img/content/1.getting-started/invalid-value.png)
4 changes: 2 additions & 2 deletions docs/content/3.contributing/1.local-development.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,15 +30,15 @@ pnpm test:reset
```

## Testing the package locally

You can bundle this package for local testing as an independent package without publishing to npm. This is useful for making sure that all peer dependencies are correct and that the package works as expected when installed as a dependency in another project.

1. Build the package with `pnpm build`
1. Bundle the package with `pnpm pack`
2. Install the tarball in your project with `pnpm install /path/to/tarball.tgz`
1. Install the tarball in your project with `pnpm install /path/to/tarball.tgz`

When you want to update the package in your project, you'll need to repeat these steps, which should also refresh any dependencies that have changed.


## Docs

Run the docs server locally with hot-reloading for development:
Expand Down
10 changes: 5 additions & 5 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,9 +46,9 @@
"db:seed": "kysely seed:run",
"lint": "eslint",
"lint:fix": "eslint --fix",
"prettier:check": "prettier \"**/*.{js,ts,mjs,vue}\" --check",
"prettier:write": "prettier \"**/*.{js,ts,mjs,vue}\" --write",
"format": "pnpm run prettier:write && pnpm run lint:fix",
"fmt": "oxfmt",
"fmt:check": "oxfmt --check",
"format": "pnpm run fmt && pnpm run lint:fix",
"publish": "pnpm run build && npm publish --access public"
},
"main": "dist/cjs/index.js",
Expand All @@ -62,8 +62,8 @@
"eslint-config-prettier": "^9.1.2",
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot May 21, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
rg -nP -C2 'eslint-config-prettier|"prettier"' --hidden -g '!pnpm-lock.yaml' -g '!node_modules'
fd -H -t f 'eslint.config' --exec cat {}

Repository: Smef/vasta

Length of output: 1026

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== package.json: prettier/oxfmt/devDeps/scripts =="
rg -n '"(prettier|oxfmt|eslint-config-prettier)"' package.json || true
rg -n '"scripts"\s*:' -n package.json || true
cat -n package.json | sed -n '1,140p' | sed -n '1,140p'

Repository: Smef/vasta

Length of output: 3107

🏁 Script executed:

#!/bin/bash
set -euo pipefail
echo "== eslint config: formatting-related references =="
ls -1 .
fd -H -t f 'eslint.config.*' -a . --exec sed -n '1,200p' {} || true

Repository: Smef/vasta

Length of output: 910

Don’t treat eslint-config-prettier as leftover—ESLint still extends it
eslint-config-prettier@^9.1.2 is still imported and included in extends in eslint.config.mjs, so it’s actively referenced (not dead weight). Remove it only if you also drop eslintConfigPrettier from the ESLint config; otherwise keep the devDependency.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@package.json` at line 62, The package.json currently lists
"eslint-config-prettier" but your ESLint setup (eslint.config.mjs) still extends
it—either keep the devDependency or remove the extend entry; to fix, decide
which approach you want and make both places consistent: if you want to keep
using Prettier rules, leave "eslint-config-prettier" in package.json and do
nothing else, otherwise remove the "eslint-config-prettier" entry from the
"extends" array in eslint.config.mjs (or remove the package.json devDependency
if you remove the extends); reference the package name "eslint-config-prettier"
and the ESLint config file "eslint.config.mjs" when making the matching change.

"kysely": "^0.29.0",
"kysely-ctl": "^0.21.0",
"oxfmt": "^0.51.0",
"pg": "^8.20.0",
"prettier": "^3.8.3",
"tsc-alias": "^1.8.17",
"typescript": "^5.9.3",
"typescript-eslint": "^8.59.3",
Expand All @@ -76,5 +76,5 @@
"unimport": "5.6.0",
"citty": "0.1.6"
},
"packageManager": "pnpm@11.1.2"
"packageManager": "pnpm@11.1.3"
}
Loading
Loading