Skip to content

Normalize count-mode output format validation - #4984

Merged
Widthdom merged 7 commits into
mainfrom
fix-issue4878
Jul 29, 2026
Merged

Normalize count-mode output format validation#4984
Widthdom merged 7 commits into
mainfrom
fix-issue4878

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Normalize --count and --format count before output compatibility validation.
  • Preserve the rightmost explicit format while rejecting incompatible row formats, JSON stream/projection modes, and compact bounded responses instead of silently overriding count output.
  • Keep count output script-compatible across numeric, JSON object, array, and envelope shapes, including audit's structured JSON usage errors.

Validation

  • Full Release net8.0: 10,841 passed, 7 skipped, 0 failed.
  • Full Release net9.0: 10,367 passed, 420 skipped, 0 failed.
  • After the final origin/main sync:
  • Changelog fragment validation: 55 fragments validated.
  • Repository index: final HEAD is fresh, matches the workspace, and has no failed readiness checks.
  • git diff --check origin/main...HEAD.

Documentation and changelog

  • Updated the English and Japanese count-mode contracts in USER_GUIDE.md and TESTING_GUIDE.md.
  • Added changelog.d/unreleased/4878.fixed.md.

Adversarial review

  • Round 1 identified the --results-only count-schema conflict; the conflict is now rejected with regression coverage.
  • Round 2 identified explicit count format loss through later modifiers and audit's structured JSON error bypass; both findings are fixed with regression coverage.
  • All actionable findings from the two workflow review rounds were addressed and revalidated.

Follow-up

Fixes #4878

@Widthdom
Widthdom merged commit bfe8771 into main Jul 29, 2026
12 checks passed
@Widthdom
Widthdom deleted the fix-issue4878 branch July 29, 2026 13:59
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.

Normalize count-mode output format validation

1 participant