tombi-format check should be quiet

[clayrosenthal]

when running the builtin linter tombi-format, it always outputs something regardless of success which is a bit ugly IMO:

ie:

$ hk run pre-commit
hk 1.53.0 by @jdx – pre-commit – fix  [=======================================] 8/8
✔ files - No unstaged changes to stash (39 files)
✔ mise
✔ tombi
✔ tombi-format
✔ tofu
✔ hclfmt
✔ whitespace
✔ newlines
✔ mixed-line-ending

tombi-format stderr:
1 file did not need formatting

I tried to change the config to pass the --quiet flag, but it doesn't seem to work for some reason:

    ["tombi-format"] = (Builtins.tombi_format) {
      depends = "tombi"
      check = "tombi format --quiet --check --diff {{ files }}"
      fix = "tombi format --quiet {{ files }}"
    }

[jdx]

Good catch — fixed in #1117, the builtin now passes --quiet to both commands.

As for why your override didn't take: Builtins.tombi_format defines check_diff, not check. hk prefers check_diff over check (both in check mode and for the "check first" pass in fix mode), so adding a check key alongside it just gets ignored — the un-quieted check_diff still ran, printed its summary line on success, and hk surfaced it as tombi-format stderr:. Your fix override was correct but never reached, since check_diff reported nothing to fix.

If you want to override it before the next release, set check_diff (an empty string disables it):

["tombi-format"] = (Builtins.tombi_format) {
  depends = "tombi"
  check_diff = "tombi format --quiet --check --diff {{ files }}"
  fix = "tombi format --quiet {{ files }}"
}

--quiet only suppresses the summary messages — the diff and the "x.toml" is not formatted error still print on failure, so nothing is lost.

This comment was generated by Claude Code.