Skip to content

#191: Print each ask hit's location first and its neighbours under it - #192

Merged
devmaxxx merged 4 commits into
mainfrom
fix/ask-text-output
Oct 17, 2026
Merged

devmaxxx merged 4 commits into
mainfrom
fix/ask-text-output

Conversation

@devmaxxx

@devmaxxx devmaxxx commented Oct 16, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • zerograph ask text output now leads each line with path:line, so terminals make it clickable.
  • sym:, file: and entity: ids are dropped when they only repeat the location or the name. Document ids (BE-M10/T05) stay.
  • Neighbours print under their seed with └ instead of after every seed. ← via stays only for a neighbour whose seed is not shown. --json is unchanged.

Before:

sym:packages/db/src/schema/salon/scheduling.ts::CANCELLATION_CONSEQUENCES  packages/db/src/schema/salon/scheduling.ts:110  CANCELLATION_CONSEQUENCES

After:

packages/db/src/schema/salon/scheduling.ts:110  CANCELLATION_CONSEQUENCES
packages/db/migrations/0008_first_slice.sql:808  salon_ctx/cancellation_policy  cancellation_policy
  └ packages/db/sql/0002_rls.sql:135  salon_ctx/cancellation_policy  cancellation_policy

Bench impact: answers cost fewer tokens. rank_of in bench/compare counts ids in reading order, so a seed printed after a neighbour now ranks one position lower.

The README example and docs/commands.md show the new shape.

Closes #191

Test plan

  • cargo test: full suite green, three new render tests
  • cargo clippy --all-targets clean
  • bench/compare test_run passes
  • cargo clippy --all-targets -- -D warnings clean after the review fix
  • Real ask cancellation on the beauty-crm fixture prints the new shape

A code hit printed its path and name twice, once inside the sym: id and
once beside it. Code, file and entity ids are now dropped where they only
repeat the location or the name; document ids stay. Neighbours print
under the seed they came from instead of after every seed.

Closes #191
@devmaxxx devmaxxx added this to the 0.6.1 milestone Oct 16, 2026
@devmaxxx
devmaxxx merged commit feb4f2c into main Oct 17, 2026
4 checks passed
@devmaxxx
devmaxxx deleted the fix/ask-text-output branch October 17, 2026 13:35
devmaxxx added a commit that referenced this pull request Oct 17, 2026
The same-size rebuild test rewrote the binary within one Linux ctime
tick and read the old install's key, which turned main's CI red; it now
waits past the tick.

`serve` checks for `stop` before the poll, so a poll that embeds a pull
can no longer outlast `stop`'s wait and make it report failure while the
server is leaving.

graph-model.md and how-it-works.md describe the `ask` line as it is
printed since #192: location first, the id only when it adds something,
neighbours under their seed.
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.

ask: text output repeats code ids and leaves neighbours far from their seed

1 participant