Skip to content

Show known commit dates in CLI and MCP search snapshot provenance - #454

Draft
jlitola wants to merge 9 commits into
mainfrom
jlitola/search-commit-dates
Draft

jlitola wants to merge 9 commits into
mainfrom
jlitola/search-commit-dates

Conversation

@jlitola

@jlitola jlitola commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Repository search provenance now shows independently known absolute UTC commit dates for the served snapshot and a different resolved requested commit. Dates stay attached to their own SHA; usable hits still lead to the exact served read, and existing conditional HEAD/exact-ref wait advice is unchanged.

The shared GraphQL selection, nullable parser, normalizer and lean projection now retain backend committedAt values. Search and retained status share the same copy; JSON retains full known timestamps and omits unknown values. Existing shared read and legacy CLI navigation provenance also gain the metadata; public list/grep use separate queries. Permanent contract, examples and evidence are in docs/implementation/search-snapshot-presentation.md.

Validation:

  • 668 focused tests pass across 14 files, with 3,258 expectations, covering wire fields, independent date cases, date-free actions and CLI/MCP initial/status text and JSON parity.
  • Typecheck, root/MCP builds, scoped Biome, diff checks and public-package validation pass.
  • Required dev CLI/MCP smokes pass auth handling; authenticated cohorts skip AUTH_REQUIRED.
  • Live CLI business verification was unavailable: file auth required login and the normal auth path blocked on keychain access. Supplied upstream backend dev search/status/read evidence was inspected; no successful new client live query is claimed.
  • Targeted Claude agent eval failed before tool use (Not logged in); no qualitative quality or performance claim.

Rollout: confirm production backend schema support before client release or hosted MCP adoption. An early rollout makes all read requests fail validation; search/status reach the existing fallback that drops target provenance. Historical date enrichment stays backend scope. No runtime fallback/backfill, metadata-only requests, version bumps, release or deployment are included. Both public artifacts carry pending minor impact.

Review: Internal preflight and fresh Claude Opus 5.5 loop clean after documentation/test cleanup. The same fresh-context final-check subagent covered the full delta; its last minor doc wording fix is applied. The completed plan was removed after review, with durable knowledge retained in implementation docs.

CI is green on ad5491a: Pull Request build/test/compatibility run and MCP package validation. Linux/Windows tests, root/MCP builds, package artifact checks, built CLI/MCP smokes and Bun/Node 20/22/24/26 compatibility all pass. MCP publication and unlabeled agent-eval jobs are skipped as expected on this PR.

Commits: 4b52bc6 implementation; 5b2ba65 review closure; ad5491a permanent verification and completed-plan cleanup.

@jlitola jlitola added the enhancement New feature or request label Oct 5, 2026
@jlitola
jlitola force-pushed the jlitola/search-commit-dates branch from ad5491a to d29541f Compare October 6, 2026 06:08
Carry backend commit timestamps through shared transport and lean metadata, and attach independent UTC dates to served and requested snapshot copy. Preserve read-now guidance and exact served locators; cover initial/status parity and navigation JSON, and document the backend rollout prerequisite.
Document that unified reads have no schema fallback and fail before backend support, while shared legacy navigation differs from public list and grep. Close review wording findings and remove a duplicate date assignment in snapshot tests.
Record the clean Claude full-delta review and precise whole-document read validation requirement. Preserve contract, evidence and rollout limits in permanent implementation docs, then remove the completed working plan.
Record the approved compact CLI/MCP row format, verified provenance contracts, minimal list selection, and focused acceptance checks. Include the clean internal and Claude plan review closure before implementation.
Render proved served snapshots and actual preparation work with shared compact wording. Preserve per-tool actions, raw content and JSON while selecting minimal list provenance and keeping browser tools free of service imports.
Verify pending n8n source calls on CLI and local MCP with minimal waits. Keep the package alias under actual work without repeating a tag that has no independent SHA, and update parallel search documentation to match the shared sections.
Select original repository and package identity fields for text provenance so unavailable targets retain their full Requested label. Keep detailed recovery fields and silent selection unchanged, and cover the compact wire projection and repository/package output.
Update the working checkpoint to reflect the implemented and independently verified identity regressions before external review.
Enable partial search by default while preserving explicit atomic requests. Keep indexed alternatives with preparation targets, retain healthy provenance for text, and close reviewed indexing explanation, empty-list guidance and wrapping defects.
@jlitola
jlitola force-pushed the jlitola/search-commit-dates branch from a8bde7e to d7d09ce Compare October 6, 2026 09:58

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant