Skip to content

Fix #83: Add extension handling to TerminalReport.generate_to_file#154

Open
fazalpsinfo-cmyk wants to merge 1 commit into
OpenAgentHQ:mainfrom
fazalpsinfo-cmyk:fix-83-terminal-extension
Open

Fix #83: Add extension handling to TerminalReport.generate_to_file#154
fazalpsinfo-cmyk wants to merge 1 commit into
OpenAgentHQ:mainfrom
fazalpsinfo-cmyk:fix-83-terminal-extension

Conversation

@fazalpsinfo-cmyk

Copy link
Copy Markdown

Description

Fixes #83 - TerminalReport.generate_to_file() did no extension handling or default filename appending, unlike every other report generator (HTMLReport, MarkdownReport, JSONReport, ComparisonReport) which all check the output path and either append a default filename or replace the extension.

Changes

  • Added the same extension-handling pattern used by HTMLReport:
    • No extension → treat as directory, append report.txt
    • Wrong extension (not .txt) → replace with .txt
    • Already .txt → use as-is

Related

Fixes #83


Support open-source contributions: https://buymeacoffee.com/muhamedfazalps

@himanshu231204

Copy link
Copy Markdown
Contributor

This branch has conflicts that must be resolved

@himanshu231204

Copy link
Copy Markdown
Contributor

Description

Fixes #83 - TerminalReport.generate_to_file() did no extension handling or default filename appending, unlike every other report generator (HTMLReport, MarkdownReport, JSONReport, ComparisonReport) which all check the output path and either append a default filename or replace the extension.

Changes

  • Added the same extension-handling pattern used by HTMLReport:

    • No extension → treat as directory, append report.txt
    • Wrong extension (not .txt) → replace with .txt
    • Already .txt → use as-is

Related

Fixes #83

Support open-source contributions: https://buymeacoffee.com/muhamedfazalps

This branch has conflicts that must be resolved

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.

reports(terminal): generate_to_file missing extension handling (inconsistent with all other generators)

2 participants