Repository navigation
feat: add trustworthy change evidence across hosts - #477
Conversation
|
Validation complete for 3d74979. All 15 PR checks are SUCCESS, including Node 22/24 CI, all six platform/path jobs, CodeQL, and release labels. Fresh CI: https://github.com/Helweg/open-codebase-index/actions/runs/40674544143 . The initial red run was repaired in code, not retried unchanged. Local final acceptance: 3,082 tests passed with coverage and short temporary roots; both packed package identities passed; actual built stdio MCP verified closed whole-line source and correct credit at 128 tokens with long identities and real graph neighbors. The unchanged pre-edit gate passed all three queries with Hit@5/MRR/graph-neighbor recall 1.0 and p95 response size 707.7 tokens; all per-query deltas were inspected. Mock evaluation is routing/harness evidence, not semantic embedding-quality evidence. Review workers are settled/released. PR remains OPEN; no merge, tag, publication, or installed omp-extension change. |
Summary
Deliver the approved agent-reliability work across the shared operations, human CLI, MCP, OpenCode, Pi, and omp:
index_status({ checkFreshness: true })/ocbi status --check-freshness --json, separate from catalog readiness and coordinator activity. Default status remains cheap; inspection does not refresh indexes.codebase_edit_context({ includeTests: true }), reusing AST-backed JS/TS direct-target and one-hop caller discovery. Complete budgeted citations and selected-only structured metadata; static links are not runtime coverage.pr_impact({ workingTree: true })/ocbi impact --working-tree --jsonfor staged, unstaged, and untracked files, with explicit refresh requirements, bounded local Git status, provenance, and UNKNOWN for incomplete graph evidence.Correctness and review
Verification
npm run build, thennpm run build:tsfor final TypeScript-only repairs; Rust source unchanged.npm run typechecknpm run lintTMPDIR=/tmp npx vitest run --coverage— 3,082 passed, 7 skipped; 154 files passed / 1 skipped on macOS ARM64, Node 26.12.1. The initial full gate also rannpm run test:run.npm run smoke:package— both package identities; strict standalone and SDK-interoperability declarations, ESM/CJS loading, CLI aliases and MCP initialize/tools-list.Linux-only invalid-byte filename coverage cannot run on macOS/APFS and is left to Linux CI. The pre-edit gate caught and drove repair of declaration-source loss and excessive source-block size; failed reports were preserved, and thresholds/baselines were not relaxed. These deterministic-mock evaluations validate harness/routing behavior, not semantic embedding quality. CI must verify the final PR revision on supported Node/platform jobs before merge.
The initial CI matrix exposed source-fence clipping during final response fitting and a path-length-dependent graph-omission fixture. Both were repaired with failing-before/passing-after source regressions and a caller identity that explicitly exceeds the small response budget. The corrected revision must complete fresh CI; this is not an unchanged rerun.
Release Notes
Category: feature. Semver: minor. This PR is for review and CI only; do not merge or publish as part of this task.