Repository navigation
feat(trace): parse Pi coding sessions - #3547
waveywaves wants to merge 1 commit into
Conversation
PR validation —
|
| Status | Policy | Material | Messages |
|---|---|---|---|
pr-min-approvals |
pr-info |
PR/MR #3547 has 0 approving reviews, 1 required. | |
| ✅ Passed | pr-description-required |
pr-info |
- |
| ✅ Passed | pr-user-story-linked |
pr-info |
- |
AI Session Checks — ⚠️ no AI session found
Missing AI Coding Sessions
This organization requires every PR to be backed by a Chainloop Trace AI coding session, and none was found for this one.
Please make sure the AI coding session evidence has been sent by the Chainloop CLI, or add the skip-ai-session label to this PR to bypass this check.
Learn more about Chainloop Trace.
Security Checks — ✅ 6 passing
✅ secret-scan
| Status | Policy | Messages |
|---|---|---|
| ✅ Passed | secrets-detection |
- |
✅ sast-scan
| Status | Policy | Messages |
|---|---|---|
| ✅ Passed | owasp-top10-2025 |
- |
| ✅ Passed | sast |
- |
| ✅ Passed | cwe-top25 |
- |
| ✅ Passed | cwe-top26-40-cusp |
- |
✅ iac-scan
| Status | Policy | Messages |
|---|---|---|
| ✅ Passed | iac-misconfiguration |
- |
Scans not applied (2)
| Scan | Reason |
|---|---|
vulnerability-scan |
no manifest/lockfile changed |
github-actions-scan |
no workflow files changed |
Security context
This change touches code with 2 recorded security-fix advisories. These are pointers to what past fixes established, not findings in this diff, and they never fail the check.
View in Chainloop ↗ · How this works ↗
Powered by Chainloop and Chainloop Trace
There was a problem hiding this comment.
All reported issues were addressed across 11 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
7662966 to
c389246
Compare
Copy persisted Pi JSONL safely, follow the selected v3 session branch, and map deterministic usage, tools, conversation, and raw evidence. Implements R-006 and R-007, plus the provider-copy portion of R-005 in Spec 005. Refs: chainloop-dev#3520 Assisted-by: pi Signed-off-by: Vibhav Bobade <vibhav.bobde@gmail.com>
c389246 to
0c53304
Compare
|
Thanks for your contribution, we're looking into expanding testing and evaluations for this feature so we'll come back to your PR soon! |
Summary
Implements the provider/parser slice of spec issue-3520:
This PR implements R-006, R-007, and the provider-copy portion of R-005. It intentionally does not register Pi or add extension/hooks/CLI flags; those are the next implementation slice.
Compatibility and safety
trace.ErrSessionDataNotFreshis opt-in. Existing providers keep their current behavior of attempting their previous copy after an ordinary refresh error.Verification
env -u CHAINLOOP_TOKEN -u CHAINLOOP_ORGANIZATION go test ./app/cli/...go test -race ./app/cli/internal/trace/pi ./app/cli/pkg/actiongit diff --checkRefs #3520
AI assistance
pi assisted with implementation and review. The commit carries an
Assisted-by: pitrailer.