name: issue-triage-quickwins-2026-06-02 date: 2026-06-02 17:55 project: codegraph branch: main

summary: Triaged all 72 open issues against the changelog, then shipped the already-fixed closes + 10 quick-win fixes (PR #654) + status --json CI fields (PR #655); both merged.

Handoff: open-issue triage → quick-win fixes + status --json (#329)

Resume here — read this first

Current state: All of this session's work is merged to main (tip 7b62356), working tree clean, nothing in progress. 18 issues closed, 2 PRs merged, 2 superseded PRs closed. The triage surfaced a backlog of real bugs/features that are documented below but not started. Immediate next step: Pick the next work item — either start a Tier-1 correctness bug or review the 4 in-flight contributor PRs. Highest-leverage correctness bug with the clearest entry point is #629 (Svelte re-export barrels → false "0 callers").

Suggested next message: "Let's fix #629 — start by making the default-export branch of findExportedSymbol (src/resolution/import-resolver.ts:~1250) also match node kind component so export { default as X } from './X.svelte' resolves, then check the bare ./-index and package-subpath barrel forms. Reproduce on a SvelteKit repo first."

Goal

Work the open-issue backlog: close what's already fixed, ship the cheap wins, and tee up the real bugs. This session's slice is done and merged; the goal now is to keep going on the remaining triaged items (Tier-1 correctness bugs, in-flight PR reviews, or the multi-root feature cluster).

Key findings

Full triage of 72 open issues was done via 6 parallel general-purpose agents, each anchoring issues to the reporter's version vs CHANGELOG.md (per CLAUDE.md's version-anchoring rule). Result buckets are in the conversation; the still-open real work is under "Open threads" below.
#329 design decision: shipped version, indexPath, lastIndexed (ISO-8601 string, or null); dropped agentCount — it had no clear consumer, two conflicting meanings (configured integrations vs. live daemon sessions), and didn't fit the issue's own CI use case. Field names match the issue (not eddieran's codegraphVersion/ms+ISO scheme).
#329 impl: QueryBuilder.getLastIndexedAt() = SELECT MAX(indexed_at) FROM files (epoch-ms; indexed_at is Date.now() at index time, schema.sql:64). Exposed as CodeGraph.getLastIndexedAt(): number|null (src/index.ts) and surfaced in both JSON branches of the status action (src/bin/codegraph.ts).
Quick-win fix locations (all on main now): .codegraph/.gitignore → *+!.gitignore (src/directory.ts, BOTH write sites ~86 and ~248); MCP resources/list/prompts/list empty replies (src/mcp/session.ts + src/mcp/proxy.ts); extension map adds (src/extraction/grammars.ts); getUnresolvedReferencesByFiles chunking (src/db/queries.ts:~1588); git ls-files -z (src/extraction/index.ts, BOTH calls in collectGitFiles); Go generic-receiver regex (src/extraction/languages/go.ts:~60); anonymous-body visit (src/extraction/tree-sitter.ts:~633); impact contains-edge exclusion (src/graph/traversal.ts:~525).
Competing-PR handling (confirmed pattern): for #329's two PRs (#333, #480) we shipped a fresh combined impl, credited BOTH via Co-Authored-By: trailers, and closed each with a specific thanks. Saved to memory feedback_pr_improve_on_contributor_branch.

Gotchas

#583 is only HALF done. Shipped RC1 (generic-receiver regex). RC2 — receiver methods declared in a different file from their struct lose the struct→method contains edge (src/extraction/tree-sitter.ts:~788 restricts owner lookup to same file) — is still open, needs a resolution-phase package-wide owner join. Don't tell anyone #583 is closed.
gh issue close --comment is unreliable — comment first (gh issue comment --body-file), then close, then verify the comment is the last activity. Used this throughout.
main is REVIEW_REQUIRED → merge with gh pr merge <N> --squash --admin --delete-branch. Co-authored-by needs numeric IDs: gh api users/<login> --jq .id → <id>+<login>@users.noreply.github.com (12122J=199902626, eddieran=8403607).
No git add -p in this env. To split mixed working-tree changes into 2 branches: commit all to tmp/snapshot, branch each off main, git checkout tmp/snapshot -- <owned files>, and for shared files start from main + re-apply hunks via Edit. CHANGELOG [Unreleased] auto-merges when one PR adds ### New Features and the other adds to ### Fixes.
Spawn-based tests (mcp-initialize, status-json) exec dist/bin/codegraph.js — rebuild dist before running them.
codegraph status takes a positional path arg, not --path (that flag silently no-ops for status).

How to test & validate

npm run build && npm test → 59 files, ~1122 passed / 2 skipped on main. (Spawn tests need the build first.)
npx vitest run __tests__/<file>.test.ts for a single file.
Smoke test the real binary: CODEGRAPH_NO_DAEMON=1 node dist/bin/codegraph.js status --json (initialized) / ... status /tmp/empty --json (uninitialized).
For new language/framework coverage bugs (#629, #645, #608, #578…): follow the CLAUDE.md validation methodology — deterministic probe on the built dist/ + agent A/B; verify "the flow EXISTS end-to-end in the graph" and node count is stable (no explosion).

Repo state

branch main, last commit 7b62356 feat(cli): add version, indexPath, lastIndexed to status --json (#329)
previous: ddb1a8f fix: issue-triage quick wins (#654)
uncommitted: clean

Open threads / TODO

Tier-1 correctness bugs (silently-wrong / zero-recall — highest value):

#645 C++ method calls via singletons/factories/chained getters → wrong class (needs C++ return-type extraction; L)
#629 Svelte default re-export barrels → false 0 callers (start: match kind component in findExportedSymbol, import-resolver.ts:~1250; M)
#608 PHP Cls::for($x)->method() static-factory chains drop the edge (M)
#578 Python module.func() after from pkg import module → zero recall (import-resolver.ts resolveViaImport; M)
#583-RC2 Go cross-file receiver methods lose contains edge (tree-sitter.ts:~788; M) → unblocks #584 (Go structural implements edges)
#527 symlink read-escape (validatePathWithinRoot has no realpath check; isPathWithinRootReal exists but unused in read paths; security; S/M)

Platform: [ ] #237 (C# primary ctors — stale tree-sitter-wasms C# grammar, current is 0.23.5) · [ ] #448 (WAL on SMB/network drives) · [ ] #208 (Windows NTFS scan hang, mis-titled) · [ ] #576 (Windows daemon lingers after Ctrl+C)

In-flight contributor PRs to review/merge: [ ] #597 (#515 C macros) · [ ] #301 (#300 Drupal patterns) · [ ] #306 (#305 docs Contributing) · [ ] #57 (#55 MQL5; cookbook PR #97 was closed)

Framework coverage: [ ] #307 (Hono sub-routers) · [ ] #491/#490/#489 (Spring @ConditionalOnProperty / Feign→controller / MyBatis SqlSessionTemplate) · [ ] #634 (TS string-literal type args)

Feature cluster (one design pass, not piecemeal): [ ] multi-root / scan-boundary — #542, #518, #511, #499, #452, #304, #281, #141, #514, #636

Needs-info / verify-version-then-maybe-close: [ ] #641 (Codex MCP — PATH/timeout) · [ ] #501 (Antigravity 2.0 — UI enable?) · [ ] #493 (sync delete — confirm reporter <0.9.5) · [ ] #535 (opencode Windows %APPDATA% — needs real Windows VM) · [ ] #613 (10GB-DB MCP timeout — large-repo query profiling)

Recent transcript (last 7 turns, oldest → newest)

Turn 1 — "Scan all open issues; which are already fixed / quick wins / real?"

Claude: gh issue list (72 open) + read CHANGELOG; built a version→fix map; fanned out 6 Agent(general-purpose) batches to triage 12 issues each, each verifying against src/.
Outcome: Bucketed triage report: 7 already-fixed, ~11 quick wins, ~20 real bugs, ~20 features, tracking/dups.

Turn 2 — "take care of the quick wins" (+ idea: append `&& codegraph install` to README curl/irm)

Claude: Verified install.sh/install.ps1 do NOT refresh current-shell PATH → chaining would fail "command not found". Implemented 10 quick wins + tests + CHANGELOG; npm test green (1122).
Outcome: All quick wins done locally; README reworked to an explicit codegraph install step "in a new terminal".

Turn 3 — "handle these [#86 #316 #339 #342 #507 #551 #565], then look at 329 and the PR"

Claude: Pinned fix-versions via git tag --contains; gh issue comment --body-file then gh issue close then verified, for all 7. Found two competing PRs for #329 (#333 by 12122J, #480 by eddieran).
Outcome: 7 already-fixed issues closed. Surfaced the #329 PR conflict + the agentCount ambiguity.

Turn 4 — "what is this feature?" → "why would anyone need agentCount" → "yes"

Claude: Explained #329 (machine-readable status for CI). Argued agentCount has no real consumer → recommended dropping it. Implemented version/indexPath/lastIndexed + getLastIndexedAt() + __tests__/status-json.test.ts; green.
Outcome: #329 implemented, agentCount dropped, validated end-to-end.

Turn 5 — "Option A" (two separate PRs)

Claude: Split the mixed working tree via tmp/snapshot + selective git checkout + Edit; built two clean branches (fix/issue-triage-quick-wins, feat/status-json-ci-fields), each green independently.
Outcome: Two committed branches; main untouched.

Turn 6 — "Yes" (push + open PRs)

Claude: git push both; gh pr create → #654 (closes 10 issues, refs #583) and #655 (closes #329). Both MERGEABLE.
Outcome: Two PRs open and verified.

Turn 7 — "do it" (merge + close superseded)

Claude: gh pr merge 654 --squash --admin; merged main into #655 branch (CHANGELOG auto-merged), pushed, gh pr merge 655 --squash --admin with co-author trailers. Closed #333/#480 with specific thanks; left a #583 RC2 note. Verified all auto-closes.
Outcome: Everything merged; 18 issues closed; #583 open with note. Updated memory feedback_pr_improve_on_contributor_branch.

issue-triage-quickwins-2026-06-02.md 10 KB История Исходник