-
Notifications
You must be signed in to change notification settings - Fork 1
feat(reports): unified top-level meta (factory + central save hook), refs #917 #968
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
…with backward compatibility; add util and tests
…d/commitSha/branch/environment/createdAt) per #917; non-breaking injection in bench + req2run reports
❓ Code Generation Drift DetectionStatus: Automated by AE-Framework Codegen |
Quality Summary
|
…ide metadata (keep top-level meta only), refs #917
❓ Code Generation Drift DetectionStatus: Automated by AE-Framework Codegen |
❓ Code Generation Drift DetectionStatus: Automated by AE-Framework Codegen |
…dReportMeta; add envOverride support, refs #917
❓ Code Generation Drift DetectionStatus: Automated by AE-Framework Codegen |
Wave 2–3 plan to reach a good stopping point:\n\n- [ ] Implement src/utils/meta-factory.ts (buildReportMeta): git(commitSha/branch), env(AE_AGENT_NAME/AE_AGENT_MODEL/TRACE_ID/RUN_ID/ENVIRONMENT), createdAt(ISO), iteration; keep shape constant.\n- [ ] Apply factory at a central save hook so all reports include meta; consider artifacts/progress/summary.json if present (minimal change).\n- [ ] Tests: factory unit (env/git present/absent, CI fallback), save-path presence/shape checks.\n- [ ] Docs: docs/quality/report-meta.md (definition, precedence, CI filling, examples). Re-export factory from index as needed.\n- [ ] CI: address failures caused by this PR only; global issues go to separate PRs.\n\nUpdate PR description with before/after JSON and precedence notes when done. |
Update: Wave 2–3 meta unification & CI-aware shape
Before/After (Quality Gate report excerpt)
Before:
{
"timestamp": "...",
"environment": "testing",
"overallScore": 100,
"results": [...]
}
After:
{
"timestamp": "...",
"environment": "testing",
"overallScore": 100,
"results": [...],
"meta": {
"agent": { "name": "unknown", "version": "1.0.0" },
"model": { "provider": null, "name": null },
"traceId": null,
"iteration": 0,
"runId": null,
"commitSha": "abcdef...",
"branch": "feat/917-meta",
"environment": "local",
"createdAt": "2025-09-19T...Z"
}
}
Notes
CI Policy (internal vs external)
Precedence Summary