Skip to content

sync: port the infrastructure-surface module tables from constructive-db - #122

Merged
pyramation merged 2 commits into
mainfrom
feat/sync-infrastructure-surface
Aug 17, 2026
Merged

sync: port the infrastructure-surface module tables from constructive-db#122
pyramation merged 2 commits into
mainfrom
feat/sync-infrastructure-surface

Conversation

@pyramation

Copy link
Copy Markdown
Contributor

Summary

Ports the module-registry side of https://github.com/constructive-io/constructive-db/pull/3170 (all 59 checks green) — the source of truth for these tables — so the two trees stay identical. Bundles regenerated with pgpm package; pgpm test-packages --full-cycle passes (deploy → verify → revert, all 29 modules).

  • http_route_module is deleted (table + plan entry + revert/verify). Generated HTTP routes are gone; route resolution is routing_public.resolve_route over the catalog, so nothing configures a route plane any more.
  • catalog_module gains images_table_id/images_table_name — the catalog grows an images kind, so a scope's published images are readable down-frame the way functions already is.
  • repository_module gains search jsonb — per-install search/embedding config (dimensions, has_chunks, embedding model/provider) the generator turns into SearchFullText/SearchVector/ProcessChunks nodes on proposals, proposal_comments, proposal_reviews, repositories.
  • The four usage-log modules lose entity_fk_table_id/actor_fk_table_id. Those knobs existed to point the entity/actor columns at an arbitrary table; the columns now carry AuthzEntityMembership SELECT policies keyed on entity_id instead, so nothing needs a configurable FK target.
  • function_resolution.image_catalog_row(name) — new reader, the images twin of bucket_catalog_row: nearest reachable frame wins.

Test fixtures that INSERT INTO metaschema_modules_public.catalog_module needed images_table_id added: it is NOT NULL DEFAULT uuid_nil() with a FK to metaschema_public.table (matching its sibling columns), so an omitted column fails the FK on the nil default rather than being ignored.

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

@pyramation pyramation self-assigned this Aug 16, 2026
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 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, CI, and merge conflict monitoring

@pyramation
pyramation merged commit 963ddcc into main Aug 17, 2026
29 checks passed
@pyramation
pyramation deleted the feat/sync-infrastructure-surface branch August 17, 2026 00:36
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