pdurlej/fallow-ts

Fork 0

feat(format): show classification counts in text output #111

Open

gemini wants to merge 1 commit from gemini/g48-show-classification-counts into main

gemini commented

2026-05-28 21:18:38 +02:00

First-time contributor

Issue: Closes #48

What changed

Modified formatText in src/format.ts to append a compact classification counts and findings total line if result.findings.length > 0.
The classification summary lists findings counts for blocking, decision_needed, and auto_safe classification groups.
Added a dedicated unit test in tests/analyze.test.mjs verifying classification count formatting on text output.

Why safe

Clean projects (with zero findings) do not show findings metrics, keeping backward compatibility and clean output compact as specified in the roadmap/issues.
Verified all unit and integration tests successfully compiled and passed.

Verification

npm run build
npm test
git diff --check

Explicit out-of-scope

Printing the detailed listing of individual findings (covered in issue #47).
Keeping clean text output compact logic verification under empty error sections (covered in issue #50).

## Issue: Closes #48 ### What changed - Modified `formatText` in `src/format.ts` to append a compact classification counts and findings total line if `result.findings.length > 0`. - The classification summary lists findings counts for `blocking`, `decision_needed`, and `auto_safe` classification groups. - Added a dedicated unit test in `tests/analyze.test.mjs` verifying classification count formatting on text output. ### Why safe - Clean projects (with zero findings) do not show findings metrics, keeping backward compatibility and clean output compact as specified in the roadmap/issues. - Verified all unit and integration tests successfully compiled and passed. ### Verification - `npm run build` - `npm test` - `git diff --check` ### Explicit out-of-scope - Printing the detailed listing of individual findings (covered in issue #47). - Keeping clean text output compact logic verification under empty error sections (covered in issue #50).

gemini added 1 commit

2026-05-28 21:18:38 +02:00

feat(format): show classification counts in text output

CI / test (pull_request) Successful in 28s

Details

CI / test (push) Successful in 27s

Details

CI / Node.js 20 (push) Successful in 16s

Details

CI / Node.js 22 (push) Successful in 16s

Details

CI / Node.js 20 (pull_request) Successful in 16s

Details

CI / Node.js 22 (pull_request) Successful in 16s

Details

f1f4d39687

CI / test (pull_request) Successful in 28s

Details

CI / test (push) Successful in 27s

Details

CI / Node.js 20 (push) Successful in 16s

Details

CI / Node.js 22 (push) Successful in 16s

Details

CI / Node.js 20 (pull_request) Successful in 16s

Details

CI / Node.js 22 (pull_request) Successful in 16s

Details

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin gemini/g48-show-classification-counts:gemini/g48-show-classification-counts

git switch gemini/g48-show-classification-counts

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main

git merge --no-ff gemini/g48-show-classification-counts

git switch gemini/g48-show-classification-counts

git rebase main

git switch main

git merge --ff-only gemini/g48-show-classification-counts

git switch gemini/g48-show-classification-counts

git rebase main

git switch main

git merge --no-ff gemini/g48-show-classification-counts

git switch main

git merge --squash gemini/g48-show-classification-counts