Add evidence spine phase one#31
Conversation
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
There was a problem hiding this comment.
Code Review
This pull request implements Phase 1 of the Tree Ring Evidence Spine and establishes a shared action foundation to unify CLI and TUI durable operations. It introduces a file-backed evidence reader and index (evidence-index.json), integrates an /evidence browser mode into the TUI, and refactors the SQLite storage layer to split private internals into dedicated modules (schema, write, search, lifecycle) behind the public SQLiteMemoryStore facade. Additionally, the CLI commands have been refactored to delegate execution to these shared actions, ensuring behavior-stable execution across interfaces. As there are no review comments provided, we have no feedback on the reviewer's comments.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
Caution Review failedAn error occurred during the review process. Please try again later. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (28)
📝 WalkthroughWalkthroughIntroduces a shared ChangesShared action layer, SQLite internal split, and CLI/TUI wiring
Estimated code review effort: 4 (Complex) | ~75 minutes Evidence spine and TUI /evidence mode
Estimated code review effort: 3 (Moderate) | ~30 minutes Sequence Diagram(s)sequenceDiagram
participant User
participant TUI as tui::app
participant Render as tui::render
participant Evidence as evidence.rs
participant Cert as certify-tree-ring.sh
User->>TUI: "/evidence refresh"
TUI->>TUI: schedule PendingAction::refresh_certification(command)
User->>TUI: confirm pending action
TUI->>TUI: set status "run externally: sh scripts/certify-tree-ring.sh"
Note over Cert: user runs script externally
Cert->>Cert: write evidence-index.json
User->>TUI: "/evidence"
TUI->>Evidence: load_snapshot(certification_dir)
Evidence-->>TUI: EvidenceSnapshot
TUI->>Render: render_evidence_list/detail(evidence_snapshot)
Render-->>User: certification status and metrics
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
target/tree-ring-certification/evidence-index.jsonbeside existingmetrics.jsonandsummary.md/evidenceand/proofbrowsing plus confirmation-gated/evidence refreshStack note
This PR targets
mainas requested and is currently stacked on top of #30 (codex/shared-action-foundation). Once #30 merges, this PR should show only the evidence-spine commits.Verification
cargo fmt --checkcargo test --lockedcargo clippy --locked --all-targetsgit diff --checksh scripts/certify-tree-ring.shtarget/tree-ring-certification/evidence-index.jsonCertification snapshot
overall_status: pass,path: metrics.json,summary_path: summary.md,missing: [harness, recall_quality]Summary by CodeRabbit
New Features
/evidenceview in the TUI for browsing certification status and details.Bug Fixes
Chores