Make etincel lint print the full audit, not three bare terms - #27
Merged
Merged
Conversation
The CLI discarded almost everything auditText() computes: whole-piece
rhythm findings (the tool's core differentiator vs. word-list linters),
the summary, replacement hints, and strengths all got dropped in favor
of a `topFindings` slice of three terms. That was the first thing a new
user saw and undercut the "we measure prose shape" pitch.
- FileLintResult now carries the full findings list (not a 3-item
slice), categoryBreakdown, summary, strengths, and the register that
was actually applied.
- formatText groups findings by category (structural first, vocabulary
bucketed together), sorts by severity within a group, prints notes
and replacement hints, and shows line:col positions on lexical
findings via a new engine/offsets.ts helper. Structural findings are
never truncated; vocabulary findings cap at 20 with a count of what
was omitted.
- formatJson is a strict superset of formatText's data now that
FileLintResult carries everything.
No engine changes. action.yml/orb.yml only care about the exit code,
so neither needed updates. README's sample output block still needs
regenerating from a real run (follow-up, not done here).
Bumps version 0.9.5 -> 0.10.0 (new exported surface: LintFinding,
offsetToLineCol/buildLineStarts) across package.json, package-lock.json,
server.json, and .claude-plugin/{plugin,marketplace}.json.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01U24Csv17sJtoAqZtS1eNLU
1 task
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The CLI discarded almost everything auditText() computes: whole-piece rhythm findings (the tool's core differentiator vs. word-list linters), the summary, replacement hints, and strengths all got dropped in favor of a
topFindingsslice of three terms. That was the first thing a new user saw and undercut the "we measure prose shape" pitch.No engine changes. action.yml/orb.yml only care about the exit code, so neither needed updates. README's sample output block still needs regenerating from a real run (follow-up, not done here).
Bumps version 0.9.5 -> 0.10.0 (new exported surface: LintFinding, offsetToLineCol/buildLineStarts) across package.json, package-lock.json, server.json, and .claude-plugin/{plugin,marketplace}.json.
Claude-Session: https://claude.ai/code/session_01U24Csv17sJtoAqZtS1eNLU