Skip to content

fix(dir): store the project index in .codegraph/ instead of .codegraph-vba/ - #325

Merged
ardelperal merged 1 commit into
mainfrom
fix/issue-324-codegraph-dir
Sep 19, 2026
Merged

ardelperal merged 1 commit into
mainfrom
fix/issue-324-codegraph-dir

Conversation

@ardelperal

Copy link
Copy Markdown
Owner

Closes #324

What changes

  • The default per-project data directory is now .codegraph/ (src/directory.ts), so init, status, uninit and the MCP projectPath walk-up all use the path upstream CodeGraph and the agent harness expect.
  • isCodeGraphDataDir goes back to the upstream shape. Its .codegraph-* prefix match still covers a legacy .codegraph-vba/, so a stale one in a consumer repo is never indexed or watched.
  • Tests, README, CLAUDE.md, docs and scripts now use .codegraph/ instead of .codegraph-vba/.
  • New tests: the default init writes .codegraph/codegraph.db and never creates .codegraph-vba/, and .codegraph-vba is still recognized as a data dir.
  • CHANGELOG [Unreleased] has a Breaking Changes entry: existing projects must run init again.

Deliberately untouched

  • Binary and package names (codegraph-vba, @aroman22/codegraph-vba): out of scope per the issue.
  • The npm staging-dir cleanup (.codegraph-vba-<hash> in upgrade/ and postinstall-cleanup.js): those are npm install dirs, not the project index.
  • Archived openspec/ notes, historical CHANGELOG entries and the benchmark record in docs/vba-extraction-perf.md, which record paths as they were at the time.
  • No automatic migration of an existing .codegraph-vba/ (out of scope per the issue).

Verification

  • npm run build passes.
  • Passed locally: foundation, daemon-, exclude-config, vba-targets-config, dump-index-schema, external-integration-doc, extraction-vba-, resolution, upgrade.
  • 2 codegraph_init tests in mcp-tools-lifecycle fail on my machine because a dependency is missing (fast-wrap-ansi for @clack/core), and more of them fail against main. CI is the source of truth here.

🤖 Generated with Claude Code

…h-vba/

Align the per-project data directory with upstream CodeGraph and the
agent harness, which look for .codegraph/. A legacy .codegraph-vba/
left in a consumer repo is still skipped by indexing and watching.

Closes #324
@ardelperal
ardelperal merged commit 77262c5 into main Sep 19, 2026
5 checks passed
@ardelperal
ardelperal deleted the fix/issue-324-codegraph-dir branch September 19, 2026 09:41
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.

fork: alineá el path del artefacto con la convención canónica (.codegraph/ en vez de .codegraph-vba/)

1 participant