Skip to content

feat(visualizer): absorb SimpleAudit result visualizer into Studio - #8

Merged
SushantGautam merged 1 commit into
mainfrom
agents/visualization-component-integration
Oct 2, 2026
Merged

SushantGautam merged 1 commit into
mainfrom
agents/visualization-component-integration

Conversation

@SushantGautam

Copy link
Copy Markdown
Collaborator

Ports the single-file HTML result viewer (formerly simpleaudit serve / export-html in the core) into Studio as Django views plus two self-contained static assets.

What's here

  • infra/visualizer.py — file-tree, JSON/image APIs, standalone HTML export, and the /visualizer/ + /visualizer/upload/ SPA pages
  • config/urls.py — routes for the SPA and its APIs. The SPA hardcodes /api/* fetch paths prefixed by __VISUALIZER_API_BASE, so the API routes live under /api/visualizer/api/* to match
  • simpleaudit_studio/cli.py — --results_dir + --visualize-only to browse a folder of dumped results server-side
  • templates — export-html entry points on run detail/result and compare
  • tests — shape detection, file tree, JSON/image endpoints, path traversal, login gating, standalone export

Verification

  • Full infra suite: 435 passed, 1 skipped (needs DEMO_MODE)
  • ruff clean
  • Browser end-to-end: file tree, single-file view, and experiment model picker all render with zero JS/console errors

Port the single-file HTML viewer (formerly `simpleaudit serve` /
`export-html` in the core) into Studio as Django views plus two
self-contained static assets.

- infra/visualizer.py: file-tree, JSON/image APIs, standalone HTML export,
  and the /visualizer/ + /visualizer/upload/ SPA pages
- config/urls.py: routes for the SPA and its APIs. The SPA hardcodes
  "/api/*" fetch paths prefixed by __VISUALIZER_API_BASE, so the API routes
  live under /api/visualizer/api/* to match
- simpleaudit_studio/cli.py: --results_dir + --visualize-only to browse a
  folder of dumped results server-side
- templates: export-html entry points on run detail/result and compare
- tests: cover shape detection, file tree, JSON/image endpoints, path
  traversal, login gating, and standalone export

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 2, 2026 10:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@gitguardian

gitguardian Bot commented Oct 2, 2026

Copy link
Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
37585757 Triggered Username Password 89bdb50 simpleaudit_studio/cli.py View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@SushantGautam
SushantGautam merged commit e7612e9 into main Oct 2, 2026
2 of 3 checks passed
@SushantGautam
SushantGautam deleted the agents/visualization-component-integration branch October 2, 2026 10:40
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.

2 participants