feat: sync pgpm modules from constructive-db#100
Merged
Conversation
Sync SQL sources, plans, and generated bundles from constructive-db/pgpm-modules: - add new uuid module - metaschema-modules: new module tables (infra_config/infra_secrets/internal_secrets/ webhook/integration_providers), platform-scope constraints, updated test+snapshot - metaschema-schema: is_valid_step_up validator + step_up field wiring - database-jobs: schedule_min_interval_seconds + add_scheduled_job execute grant - services/utils: sync source trees
🤖 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 sync's directory replacement overwrote the standalone repo's verify/revert trees with constructive-db's (which is less complete), removing revert scripts required by the integration-test full-cycle. Restore them from main and add verify/revert for the two genuinely-new tables (integration_providers_module, db_preset_module).
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 the standalone workspace from
constructive-db/pgpm-modules(source of truth), porting SQL sources (deploy/verify/revert),pgpm.planentries, and the affected generatedsql/*.sqlbundles. Standalone adaptations are preserved:workspace:*deps,pgsql-testtest imports (exceptmetaschema-modules, which already usesconstructive-testin both repos), and each module's existing frozen extension version (bundles regenerated in place — no version churn).New module
uuid— copied verbatim from db; adaptedpackage.json(version→0.32.1,@pgpm/verify→workspace:*, droppedgitHead). Added a singlepackages/uuidimporter topnpm-lock.yaml(mirrorsbase32) sopnpm install --frozen-lockfilepasses; no other lockfile churn. Not added to the CI test matrix (it has no tests, matching db).Module content synced
metaschema-modules— new module tables:infra_config_module,infra_secrets_module,internal_secrets_module,webhook_module,integration_providers_module,db_preset_module;one_platform_scopeconstraints on the*_log_module/graph/events/db_usagetables; droppedconfig_secrets_module(superseded). Test + snapshot updated from db;sessions_modulecolumns renamedsessions_table→sessions_table_name,session_credentials_table→session_credentials_table_name.metaschema-schema—metaschema_private.is_valid_step_upvalidator wired into thetablechange'srequires.database-jobs—schedule_min_interval_secondsestimator;authenticatedEXECUTE grant onadd_scheduled_jobfor INVOKER trigger support.services,utils— source-tree sync (services revert tree;enforce_identity_providers_quota).Notable non-obvious change
The larger module set makes the
information_schema.constraint_column_usageFK-introspection test straddle the old 30s limit (≈4s warm, ≈32s cold), so the timeout is bumped. Pure config; no assertion changes.Testing
pnpm install --frozen-lockfile(pnpm 9, as CI) +pnpm testfor every changed module — all green:utils,database-jobs,metaschema-schema,services,metaschema-modules(8/8, snapshots regenerated to includewebhook_module, which db's own committed snapshot still lacks).Link to Devin session: https://app.devin.ai/sessions/a118097bf58b4160938c3c8eadfaba62
Requested by: @pyramation