Skip to content

feat: add DataRealtime node type definition#1096

Merged
pyramation merged 1 commit intomainfrom
feat/add-data-realtime-definition
May 10, 2026
Merged

feat: add DataRealtime node type definition#1096
pyramation merged 1 commit intomainfrom
feat/add-data-realtime-definition

Conversation

@pyramation
Copy link
Copy Markdown
Contributor

Summary

Adds the DataRealtime node type definition to the node-type-registry, corresponding to the data_realtime.sql generator that was added in constructive-db (PRs #1082/#1085/#1086). This registers the type so it appears in metaschema_public.node_type_registry at deploy time.

What it defines:

  • DataRealtime — category data, slug data_realtime
  • parameter_schema with two properties:
    • operations: which DML operations to track (default: all three)
    • subscriber_table_name: optional override for the subscriber table name (no static default — the SQL generator computes {source_table}_subscriber at runtime)

Notable design choice: Unlike most other Data* types, DataRealtime has no column-ref properties. This is intentional — the SQL generator dynamically introspects source table SELECT policies from the registry to discover which columns to create on the subscriber table, rather than declaring them statically here.

Review & Testing Checklist for Human

  • Verify parameter_schema matches what data_realtime.sql actually reads from the data jsonb — specifically that operations and subscriber_table_name are the correct keys and types
  • After merging, regenerate the SQL seed in constructive-db (pnpm generate in packages/node-type-registry, then pgpm package in pgpm-modules/metaschema-schema) to populate the registry row

Notes

Related PRs across other repos in this batch:

  • constructive-db: skills updates (node-type-registry + data-modules installed copies)
  • constructive-private-skills: DataRealtime section added to data-modules skill
  • constructive-skills: Realtime Subscriptions section added to features catalog

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

@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 and CI monitoring

@pyramation pyramation merged commit 364ab3b into main May 10, 2026
54 checks passed
@pyramation pyramation deleted the feat/add-data-realtime-definition branch May 10, 2026 07:03
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