Skip to content

Adopts ex_quality 0.14.0 format check-mode - #16

Merged
johnnyt merged 1 commit into
mainfrom
sp-987-format-check-mode
Aug 22, 2026
Merged

Adopts ex_quality 0.14.0 format check-mode#16
johnnyt merged 1 commit into
mainfrom
sp-987-format-check-mode

Conversation

@johnnyt

@johnnyt johnnyt commented Aug 22, 2026

Copy link
Copy Markdown
Member

Why

The Format stage's write-mode default means formatting can never fail the
gate: unformatted code passes while the gate silently rewrites the working
tree mid-run. The fleet decision on sui-b5y (operator-confirmed 2026-08-22)
moves every repo's gate to check-mode instead.

What

  • Bumps ex_quality to ~> 0.14 (0.14.0 in the lock).
  • Sets format: [check: true] in .quality.exs: drift fails the stage
    with the file list and nothing is rewritten.
  • Updates the CLAUDE.md gate guidance that claimed the gate formats code
    for you; agents now run mix format themselves before committing.

Notes

  • No pre-flip format commit was needed: the tree was already fully
    formatted, so the flip commit is pure config.
  • Red path verified locally: a deliberately unformatted scratch file
    turned the Format stage red with the file listed and its content byte
    identical afterward, then was removed.
  • Full mix quality green on this tree (scope all, no profile; Format
    reports files_needing_format: 0); mix gate.verify attests it.
    Doctor/Gettext/Sobelow are not installed in this project and are
    declared not-applicable.

Closes: sp-987

Bumps ex_quality to ~> 0.14 and sets `format: [check: true]` in
.quality.exs (sui-b5y, fleet-wide decision 2026-08-22): the gate now
fails on formatting drift instead of rewriting files, so unformatted
code can no longer pass and agents run `mix format` themselves before
committing.

No file needed a pre-flip format pass - the tree was already clean.
Red path verified locally: a deliberately unformatted scratch file
turned the Format stage red with the file listed and its content
untouched, then was removed.

Updates the CLAUDE.md gate guidance that claimed the gate formats
code for you.

Refs: sp-987
@johnnyt
johnnyt merged commit cf3b09b into main Aug 22, 2026
1 check passed
@johnnyt
johnnyt deleted the sp-987-format-check-mode branch August 22, 2026 19:18
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.

1 participant