Skip to content

Conversation

@pyramation
Copy link
Contributor

@pyramation pyramation commented Jan 14, 2026

Summary

Adds table_template_module to the metadata export flow and removes the old module configs it replaces. This is a companion change to constructive-db PR #340 which introduced the table_template_module table.

Changes:

  • Added config entry for table_template_module with field definitions (id, database_id, schema_id, private_schema_id, table_id, owner_table_id, table_name, node_type, data)
  • Added queryAndParse call to export data from metaschema_modules_public.table_template_module
  • Removed config entries for user_profiles_module, user_settings_module, and organization_settings_module (these tables no longer exist - replaced by table_template_module)

Review & Testing Checklist for Human

  • Verify field names and types match the actual table_template_module table schema in constructive-db
  • Confirm that the old modules (user_profiles_module, user_settings_module, organization_settings_module) have been removed from constructive-db before merging this PR
  • Test export flow against a database provisioned with table_template_module to verify it exports correctly

Recommended test plan:

  1. Ensure PR Graphile simple inflector #340 is merged in constructive-db first
  2. Run the export against a constructive database that has been provisioned with table templates
  3. Verify the exported SQL includes the table_template_module INSERT statements

Notes

This follows the existing pattern for all other module exports in export-meta.ts. The export will gracefully skip if the table doesn't exist (handled by the try/catch in queryAndParse).

Link to Devin run: https://app.devin.ai/sessions/f4b2066a3e3e434da15de84fba001ac3
Requested by: Dan Lynch (@pyramation)

@devin-ai-integration
Copy link
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

…, organization_settings_module) from export flow
@devin-ai-integration devin-ai-integration bot changed the title Add table_template_module to export flow Add table_template_module to export flow and remove old modules Jan 14, 2026
@pyramation pyramation merged commit 38a6b79 into main Jan 14, 2026
36 checks passed
@pyramation pyramation deleted the devin/1768372802-table-template-module branch January 14, 2026 07:25
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.

2 participants