fix(cli): lint with multiple api files results in invalid output

[harshit078]

What/Why/How?

• Added logic to gather all lint results into allResults during per-file loop, then call formatProblems() once after with the combined totals.

Reference

#2692

Testing

Screenshots (optional)

Check yourself

• This PR follows the contributing guide
• All new/updated code is covered by tests
• Core code changed? - Tested with other Redocly products (internal contributions only)
• New package installed? - Tested in different environments (browser/node)
• Documentation update has been considered

Security

• The security impact of the change has been considered
• Code follows company security practices and guidelines

---

Note

Low Risk
Scoped to checkstyle output aggregation in the lint command; other formats and ignore-file generation are unchanged.

Overview
lint --format=checkstyle now emits one valid Checkstyle XML document when you pass multiple API paths in a single run. Previously each API triggered its own formatProblems call, which concatenated separate XML roots and broke downstream parsers.

The handler buffers problems per file during the loop and calls formatProblems once after all APIs finish, using combined totals. Other formats still format per API as before. Docs drop the old “lint one API at a time” warning and describe the combined <file> behavior. A unit test asserts a single aggregated formatProblems call for two APIs.

^{Reviewed by Cursor Bugbot for commit fc23c4e. Bugbot is set up for automated code reviews on this repo. Configure here.}

[changeset-bot]

🦋 Changeset detected

Latest commit: fc23c4e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages

Name	Type
@redocly/cli	Patch
@redocly/openapi-core	Patch
@redocly/respect-core	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[Daryna-del]

Thanks for the contribution! The overall approach looks good, but there are
a couple of things worth addressing before merging. Could you also take a
look at why the build and test job is failing? Left a few comments.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 0c7cf32. Configure here.}

[Daryna-del]

Hey @harshit078
Left some comments, could you, please, take a look?

[harshit078]

Hey @Daryna-del , I have addressed the comments you left and updated the docs along with it. Can you please review it again ? Thanks !

[Daryna-del]

Hi @harshit078 !
Thanks for the contribution! We’re almost there 🙂
Could you please fix formatting issue by running npm run format?

[harshit078]

Hey @Daryna-del , pushed a fix for the linting issue. Thanks !

[harshit078]

Hey @Daryna-del , can you review the PR again post all test fixes ? Thanks !

[Daryna-del]

Hey @harshit078!
Thanks again for the contribution! Everything looks good to me, let's wait for a review from technical writers 🙂
Regarding the changes I made to your PR, I was trying to adjust the release snapshot version job for testing purposes, but it didn’t work as expected, so I reverted those changes. Sorry for the confusion.

[Daryna-del]

Hey @harshit078!
Fix should be available in 2.31.6 version.
Thank you.