Skip to content

Update report-template.html#91

Open
Taya1800 wants to merge 1 commit intobrowserbase:mainfrom
Taya1800:patch-1
Open

Update report-template.html#91
Taya1800 wants to merge 1 commit intobrowserbase:mainfrom
Taya1800:patch-1

Conversation

@Taya1800
Copy link
Copy Markdown

@Taya1800 Taya1800 commented May 1, 2026

Note

High Risk
The PR deletes the entire report-template.html body, leaving an empty template that will cause compile_report.mjs to generate blank/invalid index.html unless an alternate template is provided.

Overview
Removes the full HTML/CSS report template from skills/company-research/references/report-template.html, leaving the file empty.

This effectively disables the default report rendering used by compile_report.mjs (auto-detected template), so generated reports will be blank/invalid unless callers pass a different --template.

Reviewed by Cursor Bugbot for commit c66f6f9. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Fix All in Cursor

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

Reviewed by Cursor Bugbot for commit c66f6f9. Configure here.

@@ -1,139 +1 @@
<!DOCTYPE html>
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HTML template content entirely deleted, breaking report generation

High Severity

The entire 139-line HTML template has been replaced with a single blank line. compile_report.mjs reads this file at runtime (line 56) and performs placeholder replacements ({{TITLE}}, {{TABLE_ROWS}}, etc.) to generate index.html. With an empty template, the script will produce a blank/broken report file, completely breaking the company research report generation feature.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit c66f6f9. Configure here.

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.

1 participant