Skip to content

fix: remove node_type_registry from metaschema-schema (belongs in constructive-db only)#73

Merged
pyramation merged 1 commit into
mainfrom
fix/remove-node-type-registry
May 25, 2026
Merged

fix: remove node_type_registry from metaschema-schema (belongs in constructive-db only)#73
pyramation merged 1 commit into
mainfrom
fix/remove-node-type-registry

Conversation

@pyramation
Copy link
Copy Markdown
Contributor

Summary

Removes the node_type_registry table and seed fixture from packages/metaschema-schema. This table is not referenced by any code in pgpm-modules — it's only consumed at runtime in constructive-db (by rls_parser and embedding_chunks_trigger).

Changes:

  • Removed deploy/revert/verify SQL files for node_type_registry/table and node_type_registry/fixtures/node_type_registry_seed
  • Removed corresponding entries from pgpm.plan
  • Repackaged compiled SQL (version aligned to 0.26.0 matching package.json)

Review & Testing Checklist for Human

  • Verify no other package in pgpm-modules references node_type_registry at runtime
  • Verify constructive-db still has its own node_type_registry table definition (it does — in packages/metaschema)

Notes

  • Both metaschema-schema and metaschema-modules tests pass locally after this removal
  • The compiled SQL version was bumped from 0.15.5 to 0.26.0 by pgpm package to match the current package.json version

Link to Devin session: https://app.devin.ai/sessions/8820b639fbd94ac8bfae86cfb01cf827
Requested by: @pyramation

…structive-db only)

The node_type_registry table and seed fixture are not referenced by any
code in pgpm-modules. They are only consumed at runtime in constructive-db
(rls_parser, embedding_chunks_trigger). Removing to keep pgpm-modules lean.
@devin-ai-integration
Copy link
Copy Markdown

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@pyramation pyramation merged commit c53545d into main May 25, 2026
24 checks passed
@pyramation pyramation deleted the fix/remove-node-type-registry branch May 25, 2026 23:43
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