Add lint:fix commands and update agent guidance#5303
Add lint:fix commands and update agent guidance#5303jamesricky wants to merge 2 commits intomainfrom
lint:fix commands and update agent guidance#5303Conversation
jamesricky
force-pushed
the
add-lint-fix-commands
branch
from
78ab8b9 to
1c66801
Compare
Add `lint:fix` scripts to all packages, demo services, docs, and storybook to provide a single command for auto-fixing ESLint, Prettier, and Stylelint issues. A root-level `lint:fix` script orchestrates all workspaces. `AGENTS.md` is updated to document these commands and instruct agents to run `lint:fix` after making code changes.
jamesricky
force-pushed
the
add-lint-fix-commands
branch
from
1c66801 to
ba58c2f
Compare
demo/site/package.json
Outdated
| "lint:fix:eslint": "eslint --fix --max-warnings 0 src/ '**/*.json' --no-warn-ignored", | ||
| "lint:fix:prettier": "pnpm exec prettier --write '*.{ts,js,json,md,yml,yaml,css,scss}'", | ||
| "lint:fix:style": "pnpm exec stylelint --fix '**/*.{css,scss}'", |
There was a problem hiding this comment.
What do you think about reusing the existing scripts?
"lint:fix:eslint": "pnpm run lint:eslint --fix",
"lint:fix:prettier": "pnpm run lint:prettier --write",
"lint:fix:style": "pnpm run lint:style --fix",
Then we (or the agent) won't forget to update one of the two scripts when making some changes.
There was a problem hiding this comment.
Good point, I did this for eslint and stylelint.
It's not possible for prettier unfortunately, as this needs either the --check or the --write argument, so appending one to the other doesn't work.
We could have a base-prettier script, when separate :check and :fix scripts, but I think that's overkill.
There was a problem hiding this comment.
It's possible to use both check and write, see
johnnyomair
left a comment
johnnyomair
left a comment
There was a problem hiding this comment.
IMO we could reuse lint:prettier for lint:fix:prettier, but I'm fine with the duplication as well.
3 participants
Add
lint:fixscripts to all packages, demo services, docs, and storybook to provide a single command for auto-fixing ESLint, Prettier, and Stylelint issues. A root-levellint:fixscript orchestrates all workspaces.AGENTS.mdis updated to document these commands and instruct agents to runlint:fixafter making code changes.