Skip to content

Add missing template answers (repoName, username) to exportMigrations #534

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pyramation merged 2 commits into from
Dec 31, 2025
Merged

Add missing template answers (repoName, username) to exportMigrations #534

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
67f708a
Add missing template answers (repoName, username) to exportMigrations
Dec 31, 2025
694df01
Make repoName and username optional parameters instead of hardcoded
Dec 31, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
41 changes: 34 additions & 7 deletions pgpm/core/src/export/export-migrations.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,10 @@ interface ExportMigrationsToDiskOptions {
metaExtensionName: string;
metaExtensionDesc?: string;
prompter?: Prompter;
/** Repository name for module scaffolding. Defaults to module name if not provided. */
repoName?: string;
/** GitHub username/org for module scaffolding. Required for non-interactive use. */
username?: string;
}

interface ExportOptions {
Expand All @@ -162,6 +166,10 @@ interface ExportOptions {
metaExtensionName: string;
metaExtensionDesc?: string;
prompter?: Prompter;
/** Repository name for module scaffolding. Defaults to module name if not provided. */
repoName?: string;
/** GitHub username/org for module scaffolding. Required for non-interactive use. */
username?: string;
}

const exportMigrationsToDisk = async ({
Expand All @@ -177,7 +185,9 @@ const exportMigrationsToDisk = async ({
extensionDesc,
metaExtensionName,
metaExtensionDesc,
prompter
prompter,
repoName,
username
}: ExportMigrationsToDiskOptions): Promise<void> => {
outdir = outdir + '/';

Expand Down Expand Up @@ -241,7 +251,9 @@ const exportMigrationsToDisk = async ({
name,
description: dbExtensionDesc,
extensions: [...DB_REQUIRED_EXTENSIONS],
prompter
prompter,
repoName,
username
});

// Install missing modules if user confirmed (now that module exists)
Expand Down Expand Up @@ -274,7 +286,9 @@ const exportMigrationsToDisk = async ({
name: metaExtensionName,
description: metaDesc,
extensions: [...SERVICE_REQUIRED_EXTENSIONS],
prompter
prompter,
repoName,
username
});

// Install missing modules if user confirmed (now that module exists)
Expand Down Expand Up @@ -347,7 +361,9 @@ export const exportMigrations = async ({
extensionDesc,
metaExtensionName,
metaExtensionDesc,
prompter
prompter,
repoName,
username
}: ExportOptions): Promise<void> => {
for (let v = 0; v < dbInfo.database_ids.length; v++) {
const databaseId = dbInfo.database_ids[v];
Expand All @@ -364,7 +380,9 @@ export const exportMigrations = async ({
schema_names,
author,
outdir,
prompter
prompter,
repoName,
username
});
}
};
Expand All @@ -378,6 +396,10 @@ interface PreparePackageOptions {
description: string;
extensions: string[];
prompter?: Prompter;
/** Repository name for module scaffolding. Defaults to module name if not provided. */
repoName?: string;
/** GitHub username/org for module scaffolding. Required for non-interactive use. */
username?: string;
}

interface Schema {
Expand Down Expand Up @@ -408,7 +430,9 @@ const preparePackage = async ({
name,
description,
extensions,
prompter
prompter,
repoName,
username
}: PreparePackageOptions): Promise<string> => {
const curDir = process.cwd();
const pgpmDir = path.resolve(path.join(outdir, name));
Expand All @@ -430,7 +454,10 @@ const preparePackage = async ({
access: 'restricted',
license: 'CLOSED',
fullName,
...(email && { email })
...(email && { email }),
// Use provided values or sensible defaults
repoName: repoName || name,
...(username && { username })
}
});
} else {
Expand Down