Skip to content

docs(skill): combine CCG search with source lookup - #115

Merged
tae2089 merged 2 commits into
mainfrom
codex/ccg-hybrid-search
Aug 11, 2026
Merged

docs(skill): combine CCG search with source lookup#115
tae2089 merged 2 commits into
mainfrom
codex/ccg-hybrid-search

Conversation

@tae2089

@tae2089 tae2089 commented Aug 11, 2026

Copy link
Copy Markdown
Owner

What changed

  • add a hybrid search workflow that pairs direct source lookup with CCG intent and relationship search
  • prefer rg for exact repository text, then verify CCG candidates against current source
  • require both search modes plus graph-freshness checks before claiming code or behavior is absent
  • deduplicate corroborating results while preserving whether they came from exact text, identifiers, recorded reasons, or graph relationships
  • narrow the annotation guidance so it triggers only when coverage or next actually says reasons are missing

Why

The skill previously treated grep/read and CCG as separate routing choices. That left agents without an operational rule for cases where indexed intent finds a candidate that direct text misses, direct text finds a candidate ranked off a CCG page, or either source alone appears empty.

Impact

Agents can combine high-recall direct lookup with annotation-aware search without treating either result source as proof it cannot provide. The completion contract now requires source corroboration for absence and completeness claims.

Verification

  • CGO_ENABLED=1 go test -tags fts5 ./internal/adapters/inbound/cli ./internal/app/search/wire -count=1
  • git diff --check
  • isolated SQLite: ccg migrate, ccg build ., ccg docs --out docs, and ccg lint
  • generated docs produced no tracked diff; lint reported the existing quality counts only (0 orphan, missing, stale, contradiction, dead-ref, or drifted)

@tae2089 tae2089 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-review completed against the published patch; no actionable P0–P3 findings.

Reviewed contracts:

  • the simple exact-file/string lookup exception remains intact
  • hybrid lookup is required only when intent, relationships, or completeness can change the answer
  • absence claims now require graph freshness plus both indexed and direct-source evidence
  • provenance is preserved instead of treating a grep hit as relationship proof or a CCG hit as exact-current-source proof
  • the formerly absolute empty-result guidance now delegates to annotation only when coverage or next provides that evidence
  • the procedure remains host-neutral and checkable; no production code, test expectation, or generated artifact changed

Local validation passed: skill contract tests, git diff --check, and isolated SQLite migrate/build/docs/lint. CI was still pending at the time of this review.

@tae2089

tae2089 commented Aug 11, 2026

Copy link
Copy Markdown
Owner Author

CI follow-up: both verify and verify-postgres completed successfully after the review was submitted.

@tae2089
tae2089 marked this pull request as ready for review August 11, 2026 10:56
@tae2089
tae2089 merged commit 9d183e1 into main Aug 11, 2026
2 checks passed
@tae2089
tae2089 deleted the codex/ccg-hybrid-search branch August 11, 2026 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant