feat: upstream pgpm-modules from constructive-db#70
Merged
Conversation
Major changes: - partman: add create_parent_with_retention procedure, update create_parent_by_id to support retention params - metaschema-modules: rename encrypted_secrets_module → config_secrets_user_module, secrets_module → user_state_module, levels_module → events_module; add config_secrets_org_module, inference_log_module, rate_limit_meters_module; remove table_template_module; update billing_module (meter_credits), entity_type_provision (invite achievements, multi-module storage), limits_module, realtime_module, storage_module - metaschema-schema: add partition columns to table, add embedding_chunks columns (search_indexes, embedding_model, embedding_provider), update node_type_registry seed (73 node types) - services: add enable_bulk column to api_settings and database_settings - object-tree: fix set_and_commit revert - object-store/object-tree: update READMEs - database-jobs: update tests for JWT claims-based db_id - all packages: add forceExit to jest configs, bump pgpm to ^4.24.4
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
The deploy creates both set_and_commit and set_props_and_commit, so the revert must drop both to allow the schema to be dropped cleanly.
…apshots - Revert verify files for blueprint, relation_provision, secure_table_provision (constructive-db verifies referenced columns that don't exist in deploy) - Revert database-jobs test changes (function signature mismatch) - Update metaschema-modules test snapshots for new module table names
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Syncs all diverged
pgpm-modules/*packages fromconstructive-dbinto this repo. Key changes by package:partman (user-requested focus):
create_parent_with_retentionprocedure — wrapspartman.create_parent+ sets retention/retention_keep_table onpart_configcreate_parent_by_idto delegate tocreate_parent_with_retentionmetaschema-modules:
encrypted_secrets_module→config_secrets_user_module,secrets_module→user_state_module,levels_module→events_moduleconfig_secrets_org_module,inference_log_module,rate_limit_meters_moduletable_template_modulebilling_module(meter_credits columns),entity_type_provision(invite achievements, array-only multi-module storage config),limits_module,realtime_module,storage_modulemetaschema-schema:
table: added partition columns (partitioned,partition_strategy,partition_key_names,partition_key_types)embedding_chunks: addedsearch_indexes,embedding_model,embedding_providercolumnsnode_type_registryseed: updated from 61 → 73 node typesservices: Added
enable_bulkcolumn toapi_settingsanddatabase_settingsobject-store / object-tree: Updated READMEs
Cross-cutting: Added
forceExit: trueto all jest configsItems NOT upstreamed (constructive-db has stale/incompatible versions):
blueprint,relation_provision,secure_table_provision— constructive-db verifies referencenode_type/node_datacolumns that don't exist in the deploy (deploy usesnodes). Kept pgpm-modules originals.add_scheduled_jobsignature (nodb_idparam) that doesn't match the deployed function.DROP FUNCTION set_props_and_commitfrom the revert, but the deploy still creates it. Kept both DROPs so revert works cleanly.^4.24.3→^4.24.4) — deferred to avoid lockfile churn; caret range already covers it.Review & Testing Checklist for Human
partman.create_parent_with_retentionprocedure logic — especially theUPDATE partman.part_config SET retention = ...pathconfig_secrets_org_modulerevert/verify — these were created since constructive-db was missing them (modeled after config_secrets_user_module pattern)pgpm deploy --createdblocally to verify full deploy/verify cycleNotes
Link to Devin session: https://app.devin.ai/sessions/2cd4b3b9432a45acab518af28204ced2
Requested by: @pyramation