Skip to content

[codex] Default symbol node types to hidden#212

Merged
joesobo merged 2 commits into
mainfrom
codex/symbol-defaults-false
May 22, 2026
Merged

[codex] Default symbol node types to hidden#212
joesobo merged 2 commits into
mainfrom
codex/symbol-defaults-false

Conversation

@joesobo
Copy link
Copy Markdown
Owner

@joesobo joesobo commented May 22, 2026

Summary

  • Default core symbol, variable, symbol-kind, and Godot class_name symbol node types to hidden.
  • Keep the extension shared graph-control defaults in sync with the core defaults.
  • Add core regression coverage and update extension default visibility snapshots.
  • Add a patch changeset for @codegraphy-dev/core and @codegraphy-dev/extension.

Validation

  • pnpm --filter @codegraphy-dev/extension exec vitest run --config vitest.config.ts tests/shared/graphControls/defaults/maps.test.ts tests/shared/graphControls/defaults/nodeTypes.test.ts tests/extension/graphView/controls/definitions/snapshot.test.ts
  • pnpm --filter @codegraphy-dev/core exec vitest run --config vitest.config.ts tests/graphControls/defaults/definitions.test.ts
  • pnpm --filter @codegraphy-dev/extension exec vitest run --config vitest.config.ts tests/extension/graphView/provider/runtime.test.ts tests/webview/app/shell/view.test.tsx
  • pnpm run lint
  • pnpm run typecheck
  • pnpm run build

Notes

  • pnpm run test reached the full unit gate but failed in two existing timing-heavy extension files under parallel load; rerunning those files directly passed.
  • pnpm run test:playwright currently fails because the Playwright harness stays on Loading graph...; the current main harness code sends graph data but does not send APP_BOOTSTRAP_COMPLETE, so I left that separate from this small defaults PR.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 22, 2026

🦋 Changeset detected

Latest commit: d8dabf4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@codegraphy-dev/core Patch
@codegraphy-dev/extension Patch
@codegraphy-dev/mcp Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@joesobo joesobo marked this pull request as ready for review May 22, 2026 19:28
@joesobo joesobo merged commit 4948d74 into main May 22, 2026
9 checks passed
@joesobo joesobo deleted the codex/symbol-defaults-false branch May 22, 2026 19:35
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.

1 participant