How to run prettier against changed files using jj fix

[joefiorini]

I'm working in a TypeScript/React codebase that consists of json, html, TypeScript and CSS files. Prettier is a great formatter that can handle all of these filetypes. However, when using stdin/stdout it needs to be explicitly told what parser to use for formatting (ie. html, typescript, etc). I tried setting up a fix.tool-command that passes --parser typescript but obviously that breaks as soon as it tries to run against package.json. Has anyone found a way to use prettier with jj fix like this? Does jj pass the filename in any way to the tool command?

[bnjmnt4n]

Here's my Prettier setup:

fix.tools.prettier.command = ["npx", "prettier", "--stdin-filepath=$path"]
fix.tools.prettier.patterns = ["glob:'**/*.tsx'", "glob:'**/*.ts'", "glob:'**/*.jsx'", "glob:'**/*.js'", "glob:'**/*.css'", "glob:'**/*.html'"]

You can use $path, and Prettier should be able to infer the filetype from the pathname.

[joefiorini]

@bnjmnt4n Tried that but running jj fix with that config gives me the error:

configuration property "fix.tool-command" not found

Do you have anything set for that?

[bnjmnt4n]

Looks like you're on a slightly older version of jj. v0.20.0 changed the configuration syntax to allow multiple tools to be used.

[joefiorini]

Yes, I'm actually running on your openssh branch. Thanks for making it available and keeping it up-to-date. Just updated to the latest and it works great!