Skip to content

fix: reword check diagnostics in plain language - #150

Merged
PhilDL merged 1 commit into
mainfrom
fix/plain-diagnostics
Aug 23, 2026
Merged

PhilDL merged 1 commit into
mainfrom
fix/plain-diagnostics

Conversation

@PhilDL

@PhilDL PhilDL commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Diagnostics from balade check used phrasing like "claims no file this list still holds", "reaches the payload", "cannot stand on its own", "carries no expect". They now say what is wrong in ordinary words.

Before After
The filegroup X claims no file this list still holds. The filegroup X matches no file.
A fenced code block reaches the payload only at a section's top level. Fenced code blocks are only supported at the top level of a section.
The tag x cannot stand on its own. The tag x must be inside a parent tag.
matrix holds no table. matrix contains no table.
Lines 3-9 fall outside f, which holds 5 lines at abc123. Lines 3-9 are out of range: f has 5 lines at abc123.
The code reference r carries no expect. The code reference r has no expect attribute.
Prose outside a section does not reach the payload. Text outside a section is ignored.
The PR head moved 2 commits past the stamp, and it touches a.py The PR head is 2 commits ahead of the stamped commit and changes a.py

32 strings across 6 files. Diagnostic codes unchanged. One test substring updated.

https://claude.ai/code/session_01EbmbcnENzTAj8KpYDoKd7U

Messages and hints said things like "claims no file this list still holds",
"reaches the payload", "cannot stand on its own", "carries no expect".
Every diagnostic now states what is wrong and what to do in ordinary words.
Codes are unchanged; one test substring updated to the new hint.

Claude-Session: https://claude.ai/code/session_01EbmbcnENzTAj8KpYDoKd7U
@pkg-pr-new

pkg-pr-new Bot commented Aug 23, 2026

Copy link
Copy Markdown
npx https://pkg.pr.new/basaltbytes/balade@150

commit: 5968be7

@PhilDL
PhilDL merged commit f9ccb33 into main Aug 23, 2026
8 checks passed
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