[BUG] optimize command: missing test coverage for --output=overwrite

[SanidhyaMadheshia]

Describe the bug.

Ran npm run test --if-present and noticed low test coverage in
src/apps/cli/commands/optimize.ts.

Specifically, there is no test covering the --output=overwrite path:

asyncapi optimize file.yml --output=overwrite --no-tty

This code path overwrites the original AsyncAPI file, so missing coverage could allow regressions to go unnoticed.

Expected behavior

The optimize command should overwrite the original AsyncAPI file when --output=overwrite --no-tty is used, and this behavior should be covered by an automated test to ensure regressions are detected.

Screenshots

How to Reproduce

1. run npm run test --if-present
2. see the coverage

🖥️ Device Information [optional]

• Operating System (OS):
• Browser:
• Browser Version:

👀 Have you checked for similar open issues?

• I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

• I have read the Contributing Guidelines

Are you willing to work on this issue ?

Yes I am willing to submit a PR!

[github-actions]

Welcome to AsyncAPI. Thanks a lot for reporting your first issue. Please check out our contributors guide and the instructions about a basic recommended setup useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

[SanidhyaMadheshia]

hey @AayushSaini101
Please have a view on this issue and let me know if I should raise a PR

[timoh99]

Hey @SanidhyaMadheshia I’m interested in working on this issue to add a test for the --output=overwrite --no-tty path in the asyncapi optimize command.
I’ve read the contributing guidelines and would appreciate any guidance on how to best structure the test to ensure it safely overwrites a temporary file without affecting other tests.

[SanidhyaMadheshia]

hey @princerajpoot20
Please have a view on this issue and let me know if I should raise a PR

[mishraa-G]

Hey @Shurtu-gal , @AayushSaini101 , @princerajpoot20

Quick check before I start, does it make sense for me to add an integration
test covering the asyncapi optimize --output=overwrite --no-tty path?
The plan would be to run it on a temporary copy of a fixture and assert the
file is safely overwritten and still valid.

wanted to check if I can open a PR for this?