Repository navigation
feat: improve relationship recall and agent query contracts - #325
Merged
Merged
Conversation
GQAdonis
added a commit
to GQAdonis/compass
that referenced
this pull request
Sep 22, 2026
Absorb ten upstream commits (PR crabbuild#325): source-backed alias/import/export usage evidence in callers/impact/affected, live CompassQL node degree, historical reads that neutralize checkout filters, the bounded `compass architecture` view, and the shared agent output formats. Conflict resolution kept both sides wherever they were disjoint: - compass-model/src/search.rs: our `searchable_node_terms`, which the Surreal projection depends on, alongside upstream's new `RELATIONSHIP_SEARCH_EDGE_KINDS`. Neither side had the other's symbol. - compass-cli/src/install_commands.rs: our focused-skill collection preflight and consumer updates beside upstream's managed-skill probe and state-event journaling. All eight functions have live call sites. - compass-cli/src/help.rs: upstream's newer option text in all seven hunks, with our `--engine ...|surreal` option re-applied to each. One resolution takes upstream over this fork's version. Our `RELATIONSHIP_TERM_INDEX_CAPABILITY_V2` guarded postings that are byte-identical to the ones `_V1` guards: `af0bb1d0` added `_V2` without removing `_V1`, `direct_call_source_identifier_postings` is unchanged from the merge base, and the base already stored the `(source, term, target)` evidence that commit claimed as new. Keeping `_V2` would make this fork report incomplete relationship coverage for every upstream-written snapshot — a false negative on complete, immutable data — so the marker returns to `_V1` and the three remaining references follow it. No migration is required: snapshots from any 0.3.x-lineage builder stay mutually readable. The now-false v2 claims in CHANGELOG.md and COMPATIBILITY.md are corrected. Re-pin the canonical skill digest; upstream documented its new `architecture` command in SKILL.md, which the build-time guard rejects until the digest is refreshed. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
6 tasks
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Verification
cargo fmt --all -- --checkcargo test -p compass-model --lockedcargo test -p compass-query --locked --lib --test code_traversalcargo test -p compass-output -p compass-files -p compass-history --locked --libcargo test -p compass-graph --locked --lib --test store_snapshotcargo test -p compass-cli --locked --lib --test code_query_cli --test coverage_paths --test help_cli --test history_cli --test install_clicargo clippy -p compass-model -p compass-query -p compass-output -p compass-files -p compass-cli --lib --locked -- -D warningssh scripts/check_product_boundary.shcargo test -p compass-cypher --locked --test tckcargo test -p compass-query --locked --test opencypher_tckpython3 scripts/check_compassql_support.py