Skip to content

feat(enterprise): persist report artifacts in db#76

Closed
Gracker wants to merge 1 commit into
feature/enterprise-multi-tenant-trace-db-metadatafrom
feature/enterprise-multi-tenant-report-db-metadata
Closed

feat(enterprise): persist report artifacts in db#76
Gracker wants to merge 1 commit into
feature/enterprise-multi-tenant-trace-db-metadatafrom
feature/enterprise-multi-tenant-report-db-metadata

Conversation

@Gracker
Copy link
Copy Markdown
Owner

@Gracker Gracker commented May 8, 2026

Summary

  • close §0.3.4 by making enterprise mode persist report metadata into report_artifacts
  • move enterprise report bodies into data/{tenantId}/{workspaceId}/reports/{reportId}/report.html with sidecar report.json
  • keep legacy logs/reports persistence when enterprise mode is off
  • load/delete enterprise reports through DB metadata plus scoped files, preserving owner guard behavior
  • pass trace/run context from agent report generation into persistReport
  • add enterprise report route coverage for DB reload, scoped files, cross-workspace 404, and delete cleanup
  • mark README §0.3.4 complete

Validation

  • PATH="$HOME/.nvm/versions/node/v24.15.0/bin:$PATH" npx jest --runInBand --forceExit src/routes/__tests__/enterpriseReportRoutes.test.ts in backend
  • PATH="$HOME/.nvm/versions/node/v24.15.0/bin:$PATH" npx jest --runInBand --forceExit src/routes/__tests__/ownerGuardRoutes.test.ts src/routes/__tests__/workspaceResourceRoutes.test.ts src/routes/__tests__/enterpriseReportRoutes.test.ts in backend
  • PATH="$HOME/.nvm/versions/node/v24.15.0/bin:$PATH" npm run typecheck in backend
  • PATH="$HOME/.nvm/versions/node/v24.15.0/bin:$PATH" npm run test:core in backend
  • git diff --check
  • PATH="$HOME/.nvm/versions/node/v24.15.0/bin:$PATH" npm run verify:pr

mr-torto pushed a commit to mr-torto/SmartPerfetto that referenced this pull request May 8, 2026
…racker#63, Gracker#67, Gracker#76, Gracker#87, Gracker#99)

DomainSkillEvalContract is the gate for every domain Skill change.
Five facets bound together:
- SkillEvalCase[] mapping caseId -> trace path + skill under test
- assertions: Record<caseId, SkillEvalAssertion[]> using JSONPath-style
  paths into the Skill output (Spark Gracker#99)
- SubAgentSpec[] for domain expert sub-agent expansion (Spark Gracker#87)
- SkillEvalRunResult[] capturing pass/fail/flaky/skipped status with
  per-assertion failure traces
- importers[] with required flag for atrace, simpleperf, bpftrace,
  macrobenchmark hooks (Spark Gracker#61, Gracker#63, Gracker#67, Gracker#76)

The harness runner (npm run test:scene-trace-regression) will become
contract-driven in a follow-up so the canonical six traces and their
assertions live in a single source of truth.

Plan: docs/superpowers/spark/plans/18-domain-skill-eval-harness.md

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Gracker
Copy link
Copy Markdown
Owner Author

Gracker commented May 9, 2026

已通过 commit 67856ec (merge) + 023a84a (frontend rebuild) 合入 main,关闭此 PR。

@Gracker Gracker closed this May 9, 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.

1 participant