Addresses the automated review findings on PR #1175:
- prettier --write the 3 files flagged by `quality / format` CI check
(query.ts, typescript-hof-callbacks.test.ts, typescript-jsx-as-call.test.ts).
- [medium] typescript-jsx-as-call.test.ts: tighten the combined HOF+JSX
assertion from `toBeGreaterThan(0)` to `toHaveLength(1)`. A single
`<Foo />` is one logical invocation; the bounds-only assertion would
have masked a duplicate-CALLS-edge regression (e.g. if both
`jsx_self_closing_element` and a generic call pattern matched the
same site).
- [medium] typescript-hof-callbacks.test.ts: replace the vacuously-true
`for (c of calls) expect(...)` Zustand assertion with a structural
one. Old form passed unconditionally when `calls` was empty (any
change that silenced ALL CALLS edges from store.ts would have
slipped through). New form asserts both: (a) at least one File-rooted
edge exists (proving the `isCallerAnchorLabel` fallback fires), and
(b) no edge sources from anything else (proving the fallback fires
exclusively).
- [low] finalize-algorithm.ts (`findExportByName`): rephrase the
comment to make the language-agnostic nature of the tie-break rule
explicit. The implementation was already correct for all migrated
languages; only the comment overplayed the TypeScript specificity.
- [low] captures.ts (arity synthesis): add a comment explaining why
JSX call anchors (`jsx_self_closing_element` / `jsx_opening_element`)
intentionally don't synthesize `@reference.arity`. Name-only
resolution is correct for React (components aren't overloaded in the
current graph model); a JSX-aware synthesizer counting jsx_attribute
children would be needed if that ever changes.
No production behavior change. All 8/8 HOF + 7/7 JSX + 236/236
typescript + 11/11 api-deep-flow integration tests still pass.
gitnexus and gitnexus-shared typechecks clean.
Made-with: Cursor
ReidenXerx
851d2ab749