Repository navigation
plugin: the places view opens with what the question resolved to (#1939) - #1940
Merged
swapnilpaliwal-sd merged 1 commit intoOct 12, 2026
Merged
Conversation
At the front door (MCP, the installed command) impact, path and tests answer as ten numbered places built from the verb's --json document, and the verb's summary was prose only, so it never reached that view. `path main '*'` on a tree with 22 functions named `main` opened at a test script's row 1 and read as one start, with no total and no count per hop; `impact getCsvHeader` never said it was the union of 27 declarations. The verbs now write it as data: path a `closure` (label, declarations, methods, files, tests, by_hop) and, for a chain, `ends`; impact a `declarations` count on each target, read from the label every resolver already writes it into. ax_grep.head turns those into the opening lines of both the places view and --grep, with the graph named when several languages answer. A single declaration, a chain between single declarations and a document without the fields render exactly as before. Co-authored-by: axiomcode-bot[bot] <334110751+axiomcode-bot[bot]@users.noreply.github.com>
swapnilpaliwal-sd
requested review from
JaredHLZhang,
Whua689 and
suyashpaliwal26
as code owners
October 12, 2026 23:16
Contributor
Author
|
csharp: 319/319 check(s) passed in 60 case(s). |
This was referenced Oct 13, 2026
Closed
Merged
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
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.
Fixes #1939.
At the front door (MCP and the installed
axiomcode),impact,pathandtestsanswer as ten numbered places built from the verb's--jsondocument. The verb's summary existed only as prose, so the places view never showed it: a closure had no total and no count per hop, and a name that matched N declarations read as one.Change
axiomcode-pathwritesclosure(label, declarations, methods, files, tests,by_hop) for'*'queries andendsfor chains.axiomcode-impactaddsdeclarationsto each target, taken from the label every resolver already writes the count into.ax_grep.head()turns those fields into the opening lines of both the places view (ax_blocks) and--grep. Each closure line names its graph when more than one language answers.Before → after (MCP surface, a 3,250-method TypeScript tree with 22 functions named
main)path main *1. src/test/…/sweep.ts:66 [hop 1][typescript] everything reached from main (22 declarations): 4465 method(s) in 263 file(s), 1211 of them test · by hop: 1:216, 2:211, 3:946, …, then a line saying the name matches 22 declarationspath src/index.ts:19 *1. src/extract.ts:90 [hop 1]everything reached from main (at src/index.ts:19): 3250 method(s) … · by hop: 1:1, 2:30, 3:92, …path main extractProject1. src/index.ts:38 …`main` matches 22 declarations …, then the same rowsimpact getCsvHeader1. …python-project-analyzer.ts:543`getCsvHeader` matches 27 declarations …, then the same rowsControls (unchanged)
path src/index.ts:19 extractProjectandimpact EnumConstantExtractor.extractFromEnum: identical output.Tests
tests/answer_head.py: 13 checks with no engine, including the controls, all passing.tests/run.py: typescript 352/352, python 436/436, javascript 354/354, csharp (see comment).library-auto-discovers-imported-dependencies, which needs a decompiler that isn't installed here, and it fails the same way onapps/integration-0.1.10without this change.tests/surfaces.pyhas one failure (README names--fresh), which is also there on the base branch.🤖 Generated with Claude Code