⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@mrtwebdesign
Copy link
Collaborator

Summary

  • Adds IDE selector UI with buttons for VS Code, Cursor, Augment, and File protocols
  • Positions selector just above Findings section for clear association with file links
  • Stores user preference in localStorage for persistence across reports
  • Clicking a file link opens directly in the selected IDE at the correct line number

Test plan

  • Generate an HTML report with json-to-html.py
  • Verify IDE selector appears above Findings section
  • Click each IDE button and verify links update (inspect href)
  • Click a file link with VS Code/Cursor selected to verify it opens in IDE
  • Refresh page and verify preference is remembered

Closes #65

🤖 Generated with Claude Code

- Add IDE selector UI with buttons for VS Code, Cursor, Augment, and File
- Position selector just above Findings section for clear context
- Store user preference in localStorage for persistence across reports
- Generate links with data-file and data-line attributes for dynamic switching
- Support IDE URL protocols: vscode://file, cursor://file, augment://file

Closes #65

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@mrtwebdesign
Copy link
Collaborator Author

I tested with VS Code and Cursor. You will have to approve the browser opening the first link, but after that it just works.
I did not test with Augment as I don't have it installed. But the link did nothing when set to Augment for me which is the intended failure mode.
Screenshot 2026-01-14 at 2 22 54 PM

@mrtwebdesign
Copy link
Collaborator Author

closes #65

@noelsaw1 noelsaw1 merged commit 0c2b070 into development Jan 15, 2026
1 check passed
@noelsaw1 noelsaw1 changed the title Add IDE selector for file links in HTML reports Add IDE selector for file links in HTML reports to Development Jan 15, 2026
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.

3 participants