@aspruyt/xfg 6.0.0 → 6.0.2

package/dist/config/validator.js

@@ -134,79 +134,87 @@ function buildRootSettingsContext(config) {
 /**
  * Validates settings object containing rulesets, labels, and repo settings.
  */
-function validateSettings(settings, context, rootCtx) {
-    if (!isPlainObject(settings)) {
-        throw new ValidationError(`${context}: settings must be an object`);
+function validateSettingsRulesets(settings, context, rootCtx) {
+    if (settings.rulesets === undefined)
+        return;
+    if (!isPlainObject(settings.rulesets)) {
+        throw new ValidationError(`${context}: rulesets must be an object`);
     }
-    if (settings.rulesets !== undefined) {
-        if (!isPlainObject(settings.rulesets)) {
-            throw new ValidationError(`${context}: rulesets must be an object`);
-        }
-        const rulesets = settings.rulesets;
-        for (const [name, ruleset] of Object.entries(rulesets)) {
-            // Skip reserved key
-            if (name === "inherit")
-                continue;
-            if (ruleset === false) {
-                if (rootCtx && !rootCtx.rulesetNames.includes(name)) {
-                    throw new ValidationError(`${context}: Cannot opt out of '${name}' - not defined in root settings.rulesets`);
-                }
-                continue; // Skip further validation for false entries
+    const rulesets = settings.rulesets;
+    for (const [name, ruleset] of Object.entries(rulesets)) {
+        if (name === "inherit")
+            continue;
+        if (ruleset === false) {
+            if (rootCtx && !rootCtx.rulesetNames.includes(name)) {
+                throw new ValidationError(`${context}: Cannot opt out of '${name}' - not defined in root settings.rulesets`);
             }
-            validateRuleset(ruleset, name, context);
+            continue;
         }
+        validateRuleset(ruleset, name, context);
     }
-    if (settings.labels !== undefined) {
-        if (!isPlainObject(settings.labels)) {
-            throw new ValidationError(`${context}: labels must be an object`);
-        }
-        const labels = settings.labels;
-        for (const [name, label] of Object.entries(labels)) {
-            if (name === "inherit")
-                continue;
-            if (label === false) {
-                if (rootCtx && !rootCtx.labelNames.includes(name)) {
-                    throw new ValidationError(`${context}: Cannot opt out of label '${name}' - not defined in root settings.labels`);
-                }
-                continue;
+}
+function validateSettingsLabels(settings, context, rootCtx) {
+    if (settings.labels === undefined)
+        return;
+    if (!isPlainObject(settings.labels)) {
+        throw new ValidationError(`${context}: labels must be an object`);
+    }
+    const labels = settings.labels;
+    for (const [name, label] of Object.entries(labels)) {
+        if (name === "inherit")
+            continue;
+        if (label === false) {
+            if (rootCtx && !rootCtx.labelNames.includes(name)) {
+                throw new ValidationError(`${context}: Cannot opt out of label '${name}' - not defined in root settings.labels`);
             }
-            validateLabel(label, name, context);
+            continue;
         }
+        validateLabel(label, name, context);
     }
+}
+function validateSettingsDeleteOrphaned(settings, context) {
     if (settings.deleteOrphaned !== undefined &&
         typeof settings.deleteOrphaned !== "boolean") {
         throw new ValidationError(`${context}: settings.deleteOrphaned must be a boolean`);
     }
-    if (settings.repo !== undefined) {
-        if (settings.repo === false) {
-            if (!rootCtx) {
-                // Root level — repo: false not valid here
-                throw new ValidationError(`${context}: repo: false is not valid at root level. Define repo settings or remove the field.`);
-            }
-            // Per-repo level — check root has repo settings to opt out of
-            if (!rootCtx.hasRepoSettings) {
-                throw new ValidationError(`${context}: Cannot opt out of repo settings — not defined in root settings.repo`);
-            }
-            // Valid opt-out, skip further repo validation
+}
+function validateSettingsRepo(settings, context, rootCtx) {
+    if (settings.repo === undefined)
+        return;
+    if (settings.repo === false) {
+        if (!rootCtx) {
+            throw new ValidationError(`${context}: repo: false is not valid at root level. Define repo settings or remove the field.`);
         }
-        else {
-            validateRepoSettings(settings.repo, context);
+        if (!rootCtx.hasRepoSettings) {
+            throw new ValidationError(`${context}: Cannot opt out of repo settings — not defined in root settings.repo`);
         }
+        return;
     }
-    if (settings.codeScanning !== undefined) {
-        if (settings.codeScanning === false) {
-            if (!rootCtx) {
-                throw new ValidationError(`${context}: codeScanning: false is not valid at root level. Define codeScanning settings or remove the field.`);
-            }
-            // Per-repo level — check root has codeScanning settings to opt out of
-            if (!rootCtx.hasCodeScanningSettings) {
-                throw new ValidationError(`${context}: Cannot opt out of code scanning settings — not defined in root settings.codeScanning`);
-            }
+    validateRepoSettings(settings.repo, context);
+}
+function validateSettingsCodeScanning(settings, context, rootCtx) {
+    if (settings.codeScanning === undefined)
+        return;
+    if (settings.codeScanning === false) {
+        if (!rootCtx) {
+            throw new ValidationError(`${context}: codeScanning: false is not valid at root level. Define codeScanning settings or remove the field.`);
         }
-        else {
-            validateCodeScanningSettings(settings.codeScanning, `${context} codeScanning`);
+        if (!rootCtx.hasCodeScanningSettings) {
+            throw new ValidationError(`${context}: Cannot opt out of code scanning settings — not defined in root settings.codeScanning`);
         }
+        return;
+    }
+    validateCodeScanningSettings(settings.codeScanning, `${context} codeScanning`);
+}
+function validateSettings(settings, context, rootCtx) {
+    if (!isPlainObject(settings)) {
+        throw new ValidationError(`${context}: settings must be an object`);
     }
+    validateSettingsRulesets(settings, context, rootCtx);
+    validateSettingsLabels(settings, context, rootCtx);
+    validateSettingsDeleteOrphaned(settings, context);
+    validateSettingsRepo(settings, context, rootCtx);
+    validateSettingsCodeScanning(settings, context, rootCtx);
 }
 const VALID_CODE_SCANNING_STATES = ["configured", "not-configured"];
 const VALID_CODE_SCANNING_QUERY_SUITES = ["default", "extended"];
@@ -565,9 +573,9 @@ function validateRepoFiles(config, repo, index, repoLabel) {
         for (const groupName of expandedGroups) {
             const group = config.groups[groupName];
             if (group?.files) {
-                for (const fn of Object.keys(group.files)) {
-                    if (fn !== "inherit")
-                        knownFiles.add(fn);
+                for (const fileName of Object.keys(group.files)) {
+                    if (fileName !== "inherit")
+                        knownFiles.add(fileName);
                 }
             }
         }
@@ -575,9 +583,9 @@ function validateRepoFiles(config, repo, index, repoLabel) {
     if (config.conditionalGroups) {
         for (const cg of config.conditionalGroups) {
             if (cg.files) {
-                for (const fn of Object.keys(cg.files)) {
-                    if (fn !== "inherit")
-                        knownFiles.add(fn);
+                for (const fileName of Object.keys(cg.files)) {
+                    if (fileName !== "inherit")
+                        knownFiles.add(fileName);
                 }
             }
         }
@@ -604,6 +612,28 @@ function validateRepoFiles(config, repo, index, repoLabel) {
         validateFileConfigFields(fileOverride, fileName, `Repo ${repoLabel}:`);
     }
 }
+function enrichSettingsContext(rootCtx, settings) {
+    if (!settings)
+        return;
+    if (settings.rulesets) {
+        for (const name of Object.keys(settings.rulesets)) {
+            if (name !== "inherit")
+                rootCtx.rulesetNames.push(name);
+        }
+    }
+    if (settings.labels) {
+        for (const name of Object.keys(settings.labels)) {
+            if (name !== "inherit")
+                rootCtx.labelNames.push(name);
+        }
+    }
+    if (settings.repo !== undefined && settings.repo !== false) {
+        rootCtx.hasRepoSettings = true;
+    }
+    if (settings.codeScanning !== undefined && settings.codeScanning !== false) {
+        rootCtx.hasCodeScanningSettings = true;
+    }
+}
 function validateRepoSettingsEntry(config, repo, repoLabel) {
     if (repo.settings === undefined)
         return;
@@ -611,50 +641,12 @@ function validateRepoSettingsEntry(config, repo, repoLabel) {
     if (repo.groups && config.groups) {
         const expandedGroups = expandRepoGroups(repo.groups, config.groups);
         for (const groupName of expandedGroups) {
-            const group = config.groups[groupName];
-            if (group?.settings?.rulesets) {
-                for (const name of Object.keys(group.settings.rulesets)) {
-                    if (name !== "inherit")
-                        rootCtx.rulesetNames.push(name);
-                }
-            }
-            if (group?.settings?.labels) {
-                for (const name of Object.keys(group.settings.labels)) {
-                    if (name !== "inherit")
-                        rootCtx.labelNames.push(name);
-                }
-            }
-            if (group?.settings?.repo !== undefined &&
-                group.settings.repo !== false) {
-                rootCtx.hasRepoSettings = true;
-            }
-            if (group?.settings?.codeScanning !== undefined &&
-                group.settings.codeScanning !== false) {
-                rootCtx.hasCodeScanningSettings = true;
-            }
+            enrichSettingsContext(rootCtx, config.groups[groupName]?.settings);
         }
     }
     if (config.conditionalGroups) {
         for (const cg of config.conditionalGroups) {
-            if (cg.settings?.rulesets) {
-                for (const name of Object.keys(cg.settings.rulesets)) {
-                    if (name !== "inherit")
-                        rootCtx.rulesetNames.push(name);
-                }
-            }
-            if (cg.settings?.labels) {
-                for (const name of Object.keys(cg.settings.labels)) {
-                    if (name !== "inherit")
-                        rootCtx.labelNames.push(name);
-                }
-            }
-            if (cg.settings?.repo !== undefined && cg.settings.repo !== false) {
-                rootCtx.hasRepoSettings = true;
-            }
-            if (cg.settings?.codeScanning !== undefined &&
-                cg.settings.codeScanning !== false) {
-                rootCtx.hasCodeScanningSettings = true;
-            }
+            enrichSettingsContext(rootCtx, cg.settings);
         }
     }
     validateSettings(repo.settings, `Repo ${repoLabel}`, rootCtx);

package/dist/lifecycle/ado-migration-source.d.ts

@@ -1,5 +1,5 @@
 import type { ICommandExecutor } from "../shared/command-executor.js";
-import { type RepoInfo } from "../shared/repo-detector.js";
+import { type RepoInfo } from "../repo/index.js";
 import type { IMigrationSource, LifecyclePlatform } from "./types.js";
 /**
  * Azure DevOps implementation of IMigrationSource.

package/dist/lifecycle/ado-migration-source.js

@@ -2,7 +2,7 @@ import { escapeShellArg } from "../shared/shell-utils.js";
 import { withRetry } from "../shared/retry-utils.js";
 import { toErrorMessage } from "../shared/type-guards.js";
 import { LifecycleError } from "../shared/errors.js";
-import { isAzureDevOpsRepo, } from "../shared/repo-detector.js";
+import { isAzureDevOpsRepo, } from "../repo/index.js";
 /**
  * Azure DevOps implementation of IMigrationSource.
  * Uses git clone --mirror to get all refs for migration.

package/dist/lifecycle/github-lifecycle-provider.d.ts

@@ -1,7 +1,6 @@
 import type { ICommandExecutor } from "../shared/command-executor.js";
-import { type RepoInfo } from "../shared/repo-detector.js";
 import type { DebugWarnLog } from "../shared/logger.js";
-import type { IRepoLifecycleProvider, LifecyclePlatform, CreateRepoSettings } from "./types.js";
+import type { IRepoLifecycleProvider, LifecyclePlatform, LifecycleExistsParams, LifecycleCreateParams, LifecycleForkParams, LifecycleReceiveMigrationParams } from "./types.js";
 /**
  * GitHub implementation of IRepoLifecycleProvider.
  * Uses gh CLI for all operations.
@@ -37,9 +36,9 @@ export declare class GitHubLifecycleProvider implements IRepoLifecycleProvider {
      * (e.g., "gh api --hostname host repos/owner/repo").
      */
     private buildGhApiPrefix;
-    exists(repoInfo: RepoInfo, token?: string): Promise<boolean>;
-    create(repoInfo: RepoInfo, settings?: CreateRepoSettings, token?: string): Promise<void>;
-    fork(upstream: RepoInfo, target: RepoInfo, settings?: CreateRepoSettings, token?: string): Promise<void>;
+    exists(params: LifecycleExistsParams): Promise<boolean>;
+    create(params: LifecycleCreateParams): Promise<void>;
+    fork(params: LifecycleForkParams): Promise<void>;
     /**
      * Wait for a forked repo to become available via the GitHub API.
      * GitHub forks are created asynchronously; polls exists() with a timeout.
@@ -49,7 +48,7 @@ export declare class GitHubLifecycleProvider implements IRepoLifecycleProvider {
      * Apply settings to an existing repo using gh repo edit.
      */
     private applyRepoSettings;
-    receiveMigration(repoInfo: RepoInfo, sourceDir: string, settings?: CreateRepoSettings, token?: string): Promise<void>;
+    receiveMigration(params: LifecycleReceiveMigrationParams): Promise<void>;
     /**
      * Rename a branch via the GitHub branch rename API.
      * GitHub automatically updates the default branch pointer.

package/dist/lifecycle/github-lifecycle-provider.js

@@ -1,6 +1,6 @@
 import { escapeShellArg } from "../shared/shell-utils.js";
 import { withRetry, isPermanentError, DEFAULT_PERMANENT_ERROR_PATTERNS, } from "../shared/retry-utils.js";
-import { assertGitHubRepo, } from "../shared/repo-detector.js";
+import { assertGitHubRepo, } from "../repo/index.js";
 import { toErrorMessage } from "../shared/type-guards.js";
 import { LifecycleError } from "../shared/errors.js";
 import { buildTokenEnv, getHostnameFlag } from "../shared/gh-api-utils.js";
@@ -29,8 +29,15 @@ const FORK_READY_TIMEOUT_MS = 60_000;
 /**
  * After repo creation, GitHub may return 404 due to eventual consistency.
  * Exclude 404/not-found from permanent errors so withRetry retries them.
+ * The test string must trigger all not-found-family patterns in
+ * CORE_PERMANENT_ERROR_PATTERNS: /404\b/, /not\s*found/i,
+ * /repository\s*not\s*found/i, and /does\s*not\s*exist/i.
  */
-const POST_CREATE_PERMANENT_PATTERNS = DEFAULT_PERMANENT_ERROR_PATTERNS.filter((p) => !p.test("404 Not Found"));
+const POST_CREATE_TEST_STRING = "404 Repository not found. Resource does not exist";
+const POST_CREATE_PERMANENT_PATTERNS = [
+    ...DEFAULT_PERMANENT_ERROR_PATTERNS.filter((p) => !p.test(POST_CREATE_TEST_STRING)),
+    /already\s*exists/i,
+];
 /**
  * Interval between fork readiness checks (2 seconds).
  */
@@ -119,7 +126,8 @@ export class GitHubLifecycleProvider {
         const apiPath = `repos/${escapeShellArg(repoInfo.owner)}/${escapeShellArg(repoInfo.repo)}`;
         return { tokenEnv, prefix: `gh api ${hostnamePart}`, apiPath };
     }
-    async exists(repoInfo, token) {
+    async exists(params) {
+        const { repo: repoInfo, token } = params;
         this.assertGitHub(repoInfo);
         const { tokenEnv, prefix, apiPath } = this.buildGhApiPrefix(repoInfo, token);
         const command = `${prefix}${apiPath}`;
@@ -140,7 +148,8 @@ export class GitHubLifecycleProvider {
             throw error;
         }
     }
-    async create(repoInfo, settings, token) {
+    async create(params) {
+        const { repo: repoInfo, settings, token } = params;
         this.assertGitHub(repoInfo);
         const tokenEnv = buildTokenEnv(token);
         const parts = [
@@ -175,7 +184,8 @@ export class GitHubLifecycleProvider {
         // Delete the README so xfg sync starts from a clean state.
         await this.deleteReadme(repoInfo, token);
     }
-    async fork(upstream, target, settings, token) {
+    async fork(params) {
+        const { upstream, target, settings, token } = params;
         this.assertGitHub(upstream);
         this.assertGitHub(target);
         // Guard: cannot fork a repo to the same owner
@@ -223,7 +233,7 @@ export class GitHubLifecycleProvider {
         const deadline = Date.now() + timeoutMs;
         while (Date.now() < deadline) {
             try {
-                const ready = await this.exists(repoInfo, token);
+                const ready = await this.exists({ repo: repoInfo, token });
                 if (ready) {
                     return;
                 }
@@ -259,7 +269,8 @@ export class GitHubLifecycleProvider {
             retries: this.retries,
         });
     }
-    async receiveMigration(repoInfo, sourceDir, settings, token) {
+    async receiveMigration(params) {
+        const { repo: repoInfo, sourceDir, settings, token } = params;
         this.assertGitHub(repoInfo);
         const tokenEnv = buildTokenEnv(token);
         // Remove existing "origin" remote if present (e.g., from git clone --mirror).
@@ -302,21 +313,40 @@ export class GitHubLifecycleProvider {
                 await this.executor.exec(`git -C ${escapeShellArg(sourceDir)} symbolic-ref HEAD refs/heads/${escapeShellArg(settings.defaultBranch)}`, this.cwd);
             }
         }
-        // Use gh repo create --source --push to create and mirror in one step.
-        // For bare repos (from git clone --mirror), --push mirrors all refs.
-        // This uses gh CLI authentication, avoiding raw git auth issues with GHE.
-        const parts = [
-            "gh repo create",
-            escapeShellArg(`${repoInfo.owner}/${repoInfo.repo}`),
-            "--source",
-            escapeShellArg(sourceDir),
-            "--push",
-        ];
-        buildRepoCreateFlags(parts, settings);
-        const command = parts.join(" ");
-        await withRetry(() => this.executor.exec(command, this.cwd, { env: tokenEnv }), {
+        // Split create and push into two steps. gh repo create --source --push
+        // does both atomically, but if the git backend hasn't propagated after
+        // the GraphQL create, the push fails with "Repository not found". A
+        // retry then hits "Name already exists" on the create step.
+        const repoSlug = `${repoInfo.owner}/${repoInfo.repo}`;
+        const createParts = ["gh repo create", escapeShellArg(repoSlug)];
+        buildRepoCreateFlags(createParts, settings);
+        try {
+            await withRetry(() => this.executor.exec(createParts.join(" "), this.cwd, {
+                env: tokenEnv,
+            }), {
+                retries: this.retries,
+                permanentErrorPatterns: POST_CREATE_PERMANENT_PATTERNS,
+                log: this.log
+                    ? { info: (m) => this.log.warn(m) }
+                    : undefined,
+            });
+        }
+        catch (error) {
+            if (!/already\s*exists/i.test(toErrorMessage(error))) {
+                throw error;
+            }
+        }
+        // Push mirror content via authenticated URL. Retries handle the git
+        // backend propagation delay (POST_CREATE_PERMANENT_PATTERNS allows
+        // retry on 404/not-found).
+        const remoteUrl = token
+            ? `https://x-access-token:${token}@${repoInfo.host}/${repoSlug}.git`
+            : `https://${repoInfo.host}/${repoSlug}.git`;
+        await this.executor.exec(`git -C ${escapeShellArg(sourceDir)} remote add origin ${escapeShellArg(remoteUrl)}`, this.cwd);
+        await withRetry(() => this.executor.exec(`git -C ${escapeShellArg(sourceDir)} push --mirror origin`, this.cwd, { env: tokenEnv }), {
             retries: this.retries,
             permanentErrorPatterns: POST_CREATE_PERMANENT_PATTERNS,
+            log: this.log ? { info: (m) => this.log.warn(m) } : undefined,
         });
     }
     /**

package/dist/lifecycle/lifecycle-formatter.d.ts

@@ -1,9 +1,10 @@
 import type { LifecycleResult } from "./types.js";
+import type { RepoVisibility } from "../config/index.js";
 interface FormatOptions {
     upstream?: string;
     source?: string;
     settings?: {
-        visibility?: string;
+        visibility?: RepoVisibility;
         description?: string;
     };
 }

package/dist/lifecycle/lifecycle-formatter.js

@@ -1,5 +1,5 @@
 import chalk from "chalk";
-import { getRepoDisplayName } from "../shared/repo-detector.js";
+import { getRepoDisplayName } from "../repo/index.js";
 import { SyncError } from "../shared/errors.js";
 /**
  * Format lifecycle action for output (used in both dry-run and real execution).

package/dist/lifecycle/lifecycle-helpers.d.ts

@@ -1,5 +1,5 @@
 import type { RepoConfig, GitHubRepoSettings } from "../config/index.js";
-import type { RepoInfo } from "../shared/repo-detector.js";
+import type { RepoInfo } from "../repo/index.js";
 import type { IRepoLifecycleManager, CreateRepoSettings, LifecycleResult } from "./types.js";
 interface LifecycleCheckOptions {
     dryRun: boolean;

package/dist/lifecycle/repo-lifecycle-manager.d.ts

@@ -1,4 +1,4 @@
-import { type RepoInfo } from "../shared/repo-detector.js";
+import { type RepoInfo } from "../repo/index.js";
 import { type DebugInfoWarnLog } from "../shared/logger.js";
 import type { ICommandExecutor } from "../shared/command-executor.js";
 import type { RepoConfig } from "../config/index.js";

package/dist/lifecycle/repo-lifecycle-manager.js

@@ -1,6 +1,6 @@
 import { join } from "node:path";
 import { rm } from "node:fs/promises";
-import { parseGitUrl } from "../shared/repo-detector.js";
+import { parseGitUrl } from "../repo/index.js";
 import { safeCleanup } from "../shared/cleanup-utils.js";
 import { LifecycleError } from "../shared/errors.js";
 import { NO_OP_DEBUG_LOG } from "../shared/logger.js";
@@ -31,7 +31,7 @@ export class RepoLifecycleManager {
             return { repoInfo, action: "existed" };
         }
         const { token } = options;
-        const exists = await provider.exists(repoInfo, token);
+        const exists = await provider.exists({ repo: repoInfo, token });
         if (exists) {
             // Repo exists - nothing to do (ignore upstream/source)
             return {
@@ -59,7 +59,7 @@ export class RepoLifecycleManager {
                 skipped: true,
             };
         }
-        await provider.create(repoInfo, settings, options.token);
+        await provider.create({ repo: repoInfo, settings, token: options.token });
         await this.waitForRepoReady(provider, repoInfo, options.token);
         return {
             repoInfo,
@@ -80,7 +80,12 @@ export class RepoLifecycleManager {
         const upstreamInfo = parseGitUrl(repoConfig.upstream, {
             githubHosts: options.githubHosts,
         });
-        await provider.fork(upstreamInfo, repoInfo, settings, options.token);
+        await provider.fork({
+            upstream: upstreamInfo,
+            target: repoInfo,
+            settings,
+            token: options.token,
+        });
         await this.waitForRepoReady(provider, repoInfo, options.token);
         return {
             repoInfo,
@@ -106,7 +111,12 @@ export class RepoLifecycleManager {
             await source.cloneForMigration(sourceInfo, sourceDir);
             // Create target and push content
             const provider = this.factory.getProvider(repoInfo.type);
-            await provider.receiveMigration(repoInfo, sourceDir, settings, options.token);
+            await provider.receiveMigration({
+                repo: repoInfo,
+                sourceDir,
+                settings,
+                token: options.token,
+            });
             await this.waitForRepoReady(provider, repoInfo, options.token);
             return {
                 repoInfo,
@@ -125,7 +135,7 @@ export class RepoLifecycleManager {
     async waitForRepoReady(provider, repoInfo, token, timeoutMs = 15000, pollMs = 1000) {
         const start = Date.now();
         while (Date.now() - start < timeoutMs) {
-            if (await provider.exists(repoInfo, token)) {
+            if (await provider.exists({ repo: repoInfo, token })) {
                 return;
             }
             await new Promise((resolve) => setTimeout(resolve, pollMs));

package/dist/lifecycle/types.d.ts

@@ -1,11 +1,12 @@
-import type { RepoInfo } from "../shared/repo-detector.js";
-import type { RepoConfig } from "../config/index.js";
-export type LifecyclePlatform = "github" | "azure-devops" | "gitlab";
+import type { RepoInfo, RepoPlatform } from "../repo/index.js";
+import type { RepoConfig, RepoVisibility } from "../config/index.js";
+export type LifecyclePlatform = RepoPlatform;
 export interface LifecycleResult {
     repoInfo: RepoInfo;
     action: "existed" | "created" | "forked" | "migrated";
     skipped?: boolean;
 }
+export type LifecycleActionKind = LifecycleResult["action"];
 export interface LifecycleOptions {
     dryRun: boolean;
     workDir: string;
@@ -18,12 +19,33 @@ export interface LifecycleOptions {
  * Subset of GitHubRepoSettings that makes sense for creation.
  */
 export interface CreateRepoSettings {
-    visibility?: "public" | "private" | "internal";
+    visibility?: RepoVisibility;
     description?: string;
     hasIssues?: boolean;
     hasWiki?: boolean;
     defaultBranch?: string;
 }
+export interface LifecycleExistsParams {
+    repo: RepoInfo;
+    token?: string;
+}
+export interface LifecycleCreateParams {
+    repo: RepoInfo;
+    settings?: CreateRepoSettings;
+    token?: string;
+}
+export interface LifecycleForkParams {
+    upstream: RepoInfo;
+    target: RepoInfo;
+    settings?: CreateRepoSettings;
+    token?: string;
+}
+export interface LifecycleReceiveMigrationParams {
+    repo: RepoInfo;
+    sourceDir: string;
+    settings?: CreateRepoSettings;
+    token?: string;
+}
 /**
  * Provider for platform-specific lifecycle operations.
  * Implementations handle create/fork/receive for a specific platform.
@@ -34,20 +56,20 @@ export interface IRepoLifecycleProvider {
      * Check if a repository exists on this platform.
      * @throws LifecycleError on network/auth failures (NOT for "repo not found")
      */
-    exists(repoInfo: RepoInfo, token?: string): Promise<boolean>;
+    exists(params: LifecycleExistsParams): Promise<boolean>;
     /**
      * Create an empty repository.
      */
-    create(repoInfo: RepoInfo, settings?: CreateRepoSettings, token?: string): Promise<void>;
+    create(params: LifecycleCreateParams): Promise<void>;
     /**
      * Fork from an upstream repository.
      * Optional - not all platforms support forking.
      */
-    fork?(upstream: RepoInfo, target: RepoInfo, settings?: CreateRepoSettings, token?: string): Promise<void>;
+    fork?(params: LifecycleForkParams): Promise<void>;
     /**
      * Receive migrated content (repo already created, push content).
      */
-    receiveMigration(repoInfo: RepoInfo, sourceDir: string, settings?: CreateRepoSettings, token?: string): Promise<void>;
+    receiveMigration(params: LifecycleReceiveMigrationParams): Promise<void>;
 }
 /**
  * Source for migration operations.

package/dist/output/lifecycle-report.d.ts

@@ -1,3 +1,5 @@
+import type { LifecycleActionKind } from "../lifecycle/types.js";
+import type { RepoVisibility } from "../config/index.js";
 export interface LifecycleReport {
     actions: LifecycleAction[];
     totals: {
@@ -9,11 +11,11 @@ export interface LifecycleReport {
 }
 export interface LifecycleAction {
     repoName: string;
-    action: "existed" | "created" | "forked" | "migrated";
+    action: LifecycleActionKind;
     upstream?: string;
     source?: string;
     settings?: {
-        visibility?: string;
+        visibility?: RepoVisibility;
         description?: string;
     };
 }

package/dist/output/settings-report.d.ts

@@ -1,4 +1,4 @@
-import type { PropertyDiff } from "../settings/index.js";
+import type { PropertyDiff, ActiveAction } from "../settings/index.js";
 import type { Ruleset, Label } from "../config/index.js";
 export interface SettingsReport {
     repos: RepoChanges[];
@@ -28,19 +28,19 @@ export interface RepoChanges {
 }
 export interface SettingChange {
     name: string;
-    action: "create" | "update";
+    action: Exclude<ActiveAction, "delete">;
     oldValue?: unknown;
     newValue: unknown;
 }
 export interface RulesetChange {
     name: string;
-    action: "create" | "update" | "delete";
+    action: ActiveAction;
     propertyDiffs?: PropertyDiff[];
     config?: Ruleset;
 }
 export interface LabelChange {
     name: string;
-    action: "create" | "update" | "delete";
+    action: ActiveAction;
     newName?: string;
     propertyChanges?: {
         property: string;