fix: rename collections_public -> metaschema_public, meta_public -> services_public in @pgpmjs/core #550
+42
−42
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.
fix: rename schema references in @pgpmjs/core
Summary
Updates hardcoded PostgreSQL schema names in the pgpm export functionality to match the schema rename in constructive-db PR #210:
collections_public→metaschema_public(database metadata tables)meta_public→services_public(service tables like apis, sites, domains)This is a breaking change that must be coordinated with the constructive-db rename.
Files changed:
pgpm/cli/src/commands/export.ts- SQL queries for database/schema selectionpgpm/core/src/export/export-meta.ts- Table config and SQL queries for metadata exportpgpm/core/src/export/export-migrations.ts- SQL queries and comments in migration exportReview & Testing Checklist for Human
collections_publicormeta_publicreferences in the pgpm packages that may have been missedpgpm exportcommand against a database with the new schema names to verify queries workRecommended test plan: Deploy constructive-db changes first, then run
pgpm exportagainst a test database to verify the export functionality works end-to-end with the new schema names.Notes
docs/plan/NAMING-PROPOSAL.mdin constructive-dbLink to Devin run: https://app.devin.ai/sessions/6b4b0a2f64084dfaa4a82c0cef42bc43
Requested by: Dan Lynch (@pyramation)