@aspruyt/xfg 3.1.4 → 3.2.0

package/dist/authenticated-git-ops.d.ts

@@ -28,7 +28,9 @@ export interface IAuthenticatedGitOps {
         branch: string;
         method: string;
     }>;
-    lsRemote(branchName: string): Promise<string>;
+    lsRemote(branchName: string, options?: {
+        skipRetry?: boolean;
+    }): Promise<string>;
     pushRefspec(refspec: string, options?: {
         delete?: boolean;
     }): Promise<void>;
@@ -82,8 +84,13 @@ export declare class AuthenticatedGitOps implements IAuthenticatedGitOps {
     /**
      * Execute ls-remote with authentication.
      * Used by GraphQLCommitStrategy to check if branch exists on remote.
+     *
+     * @param options.skipRetry - If true, don't retry on failure. Use when checking
+     *   branch existence where failure is expected for new branches.
      */
-    lsRemote(branchName: string): Promise<string>;
+    lsRemote(branchName: string, options?: {
+        skipRetry?: boolean;
+    }): Promise<string>;
     /**
      * Execute push with custom refspec (e.g., HEAD:branchName).
      * Used by GraphQLCommitStrategy for creating/deleting remote branches.

package/dist/authenticated-git-ops.js

@@ -100,11 +100,18 @@ export class AuthenticatedGitOps {
     /**
      * Execute ls-remote with authentication.
      * Used by GraphQLCommitStrategy to check if branch exists on remote.
+     *
+     * @param options.skipRetry - If true, don't retry on failure. Use when checking
+     *   branch existence where failure is expected for new branches.
      */
-    async lsRemote(branchName) {
+    async lsRemote(branchName, options) {
         // Remote URL already has auth from clone
         const safeBranch = escapeShellArg(branchName);
-        return this.execWithRetry(`git ls-remote --exit-code --heads origin ${safeBranch}`);
+        const command = `git ls-remote --exit-code --heads origin ${safeBranch}`;
+        if (options?.skipRetry) {
+            return this.executor.exec(command, this.workDir);
+        }
+        return this.execWithRetry(command);
     }
     /**
      * Execute push with custom refspec (e.g., HEAD:branchName).

package/dist/command-executor.d.ts

@@ -2,7 +2,7 @@
  * Interface for executing shell commands.
  * Enables dependency injection for testing and alternative implementations.
  */
-export interface CommandExecutor {
+export interface ICommandExecutor {
     /**
      * Execute a shell command and return the output.
      * @param command The command to execute
@@ -16,10 +16,10 @@ export interface CommandExecutor {
  * Default implementation that uses Node.js child_process.execSync.
  * Note: Commands are escaped using escapeShellArg before being passed here.
  */
-export declare class ShellCommandExecutor implements CommandExecutor {
+export declare class ShellCommandExecutor implements ICommandExecutor {
     exec(command: string, cwd: string): Promise<string>;
 }
 /**
  * Default executor instance for production use.
  */
-export declare const defaultExecutor: CommandExecutor;
+export declare const defaultExecutor: ICommandExecutor;

package/dist/config-normalizer.js

@@ -60,14 +60,30 @@ export function mergeSettings(root, perRepo) {
     // Merge rulesets by name - each ruleset is deep merged
     const rootRulesets = root?.rulesets ?? {};
     const repoRulesets = perRepo?.rulesets ?? {};
+    // Check if repo opts out of all inherited rulesets
+    const inheritRulesets = repoRulesets?.inherit !== false;
     const allRulesetNames = new Set([
-        ...Object.keys(rootRulesets),
-        ...Object.keys(repoRulesets),
+        ...Object.keys(rootRulesets).filter((name) => name !== "inherit"),
+        ...Object.keys(repoRulesets).filter((name) => name !== "inherit"),
     ]);
     if (allRulesetNames.size > 0) {
         result.rulesets = {};
         for (const name of allRulesetNames) {
-            result.rulesets[name] = mergeRuleset(rootRulesets[name], repoRulesets[name]);
+            const rootRuleset = rootRulesets[name];
+            const repoRuleset = repoRulesets[name];
+            // Skip if repo explicitly opts out of this ruleset
+            if (repoRuleset === false) {
+                continue;
+            }
+            // Skip root rulesets if inherit: false (unless repo has override)
+            if (!inheritRulesets && !repoRuleset && rootRuleset) {
+                continue;
+            }
+            result.rulesets[name] = mergeRuleset(rootRuleset, repoRuleset);
+        }
+        // Clean up empty rulesets object
+        if (Object.keys(result.rulesets).length === 0) {
+            delete result.rulesets;
         }
     }
     // deleteOrphaned: per-repo overrides root
@@ -89,13 +105,23 @@ export function normalizeConfig(raw) {
         const gitUrls = Array.isArray(rawRepo.git) ? rawRepo.git : [rawRepo.git];
         for (const gitUrl of gitUrls) {
             const files = [];
+            // Check if repo opts out of all inherited files
+            const inheritFiles = rawRepo.files?.inherit !==
+                false;
             // Step 2: Process each file definition
             for (const fileName of fileNames) {
+                // Skip reserved key
+                if (fileName === "inherit")
+                    continue;
                 const repoOverride = rawRepo.files?.[fileName];
                 // Skip excluded files (set to false)
                 if (repoOverride === false) {
                     continue;
                 }
+                // Skip if inherit: false and no repo-specific override
+                if (!inheritFiles && !repoOverride) {
+                    continue;
+                }
                 const fileConfig = raw.files[fileName];
                 const fileStrategy = fileConfig.mergeStrategy ?? "replace";
                 // Step 3: Compute merged content for this file
@@ -190,12 +216,34 @@ export function normalizeConfig(raw) {
             });
         }
     }
+    // Normalize root settings (filter out inherit key if present)
+    let normalizedRootSettings;
+    if (raw.settings) {
+        normalizedRootSettings = {};
+        if (raw.settings.rulesets) {
+            const filteredRulesets = {};
+            for (const [name, ruleset] of Object.entries(raw.settings.rulesets)) {
+                if (name === "inherit" || ruleset === false)
+                    continue;
+                filteredRulesets[name] = ruleset;
+            }
+            if (Object.keys(filteredRulesets).length > 0) {
+                normalizedRootSettings.rulesets = filteredRulesets;
+            }
+        }
+        if (raw.settings.deleteOrphaned !== undefined) {
+            normalizedRootSettings.deleteOrphaned = raw.settings.deleteOrphaned;
+        }
+        if (Object.keys(normalizedRootSettings).length === 0) {
+            normalizedRootSettings = undefined;
+        }
+    }
     return {
         id: raw.id,
         repos: expandedRepos,
         prTemplate: raw.prTemplate,
         githubHosts: raw.githubHosts,
         deleteOrphaned: raw.deleteOrphaned,
-        settings: raw.settings,
+        settings: normalizedRootSettings,
     };
 }

package/dist/config-validator.d.ts

@@ -7,7 +7,7 @@ export declare function validateRawConfig(config: RawConfig): void;
 /**
  * Validates settings object containing rulesets.
  */
-export declare function validateSettings(settings: unknown, context: string): void;
+export declare function validateSettings(settings: unknown, context: string, rootRulesetNames?: string[]): void;
 /**
  * Validates that config is suitable for the sync command.
  * @throws Error if files section is missing or empty

package/dist/config-validator.js

@@ -49,6 +49,10 @@ export function validateRawConfig(config) {
             "Use 'files' to sync configuration files, or 'settings' to manage repository settings.");
     }
     const fileNames = hasFiles ? Object.keys(config.files) : [];
+    // Check for reserved key 'inherit' at root files level
+    if (hasFiles && "inherit" in config.files) {
+        throw new Error("'inherit' is a reserved key and cannot be used as a filename");
+    }
     // Validate each file definition
     for (const fileName of fileNames) {
         validateFileName(fileName);
@@ -127,6 +131,10 @@ export function validateRawConfig(config) {
     // Validate root settings
     if (config.settings !== undefined) {
         validateSettings(config.settings, "Root");
+        // Check for reserved key 'inherit' at root rulesets level
+        if (config.settings.rulesets && "inherit" in config.settings.rulesets) {
+            throw new Error("'inherit' is a reserved key and cannot be used as a ruleset name");
+        }
     }
     // Validate githubHosts if provided
     if (config.githubHosts !== undefined) {
@@ -161,6 +169,14 @@ export function validateRawConfig(config) {
                 throw new Error(`Repo at index ${i}: files must be an object`);
             }
             for (const fileName of Object.keys(repo.files)) {
+                // Skip reserved key 'inherit'
+                if (fileName === "inherit") {
+                    const inheritValue = repo.files.inherit;
+                    if (typeof inheritValue !== "boolean") {
+                        throw new Error(`Repo at index ${i}: files.inherit must be a boolean`);
+                    }
+                    continue;
+                }
                 // Ensure the file is defined at root level
                 if (!config.files || !config.files[fileName]) {
                     throw new Error(`Repo at index ${i} references undefined file '${fileName}'. File must be defined in root 'files' object.`);
@@ -233,7 +249,10 @@ export function validateRawConfig(config) {
         }
         // Validate per-repo settings
         if (repo.settings !== undefined) {
-            validateSettings(repo.settings, `Repo ${getGitDisplayName(repo.git)}`);
+            const rootRulesetNames = config.settings?.rulesets
+                ? Object.keys(config.settings.rulesets).filter((k) => k !== "inherit")
+                : [];
+            validateSettings(repo.settings, `Repo ${getGitDisplayName(repo.git)}`, rootRulesetNames);
         }
     }
 }
@@ -465,7 +484,7 @@ function validateRuleset(ruleset, name, context) {
 /**
  * Validates settings object containing rulesets.
  */
-export function validateSettings(settings, context) {
+export function validateSettings(settings, context, rootRulesetNames) {
     if (typeof settings !== "object" ||
         settings === null ||
         Array.isArray(settings)) {
@@ -480,6 +499,16 @@ export function validateSettings(settings, context) {
         }
         const rulesets = s.rulesets;
         for (const [name, ruleset] of Object.entries(rulesets)) {
+            // Skip reserved key
+            if (name === "inherit")
+                continue;
+            // Check for opt-out of non-existent root ruleset
+            if (ruleset === false) {
+                if (rootRulesetNames && !rootRulesetNames.includes(name)) {
+                    throw new Error(`${context}: Cannot opt out of '${name}' - not defined in root settings.rulesets`);
+                }
+                continue; // Skip further validation for false entries
+            }
             validateRuleset(ruleset, name, context);
         }
     }

package/dist/config.d.ts

@@ -239,12 +239,16 @@ export interface RawRepoFileOverride {
     deleteOrphaned?: boolean;
 }
 export interface RawRepoSettings {
-    rulesets?: Record<string, Ruleset>;
+    rulesets?: Record<string, Ruleset | false> & {
+        inherit?: boolean;
+    };
     deleteOrphaned?: boolean;
 }
 export interface RawRepoConfig {
     git: string | string[];
-    files?: Record<string, RawRepoFileOverride | false>;
+    files?: Record<string, RawRepoFileOverride | false> & {
+        inherit?: boolean;
+    };
     prOptions?: PRMergeOptions;
     settings?: RawRepoSettings;
 }

package/dist/git-ops.d.ts

@@ -1,12 +1,38 @@
-import { CommandExecutor } from "./command-executor.js";
+import { ICommandExecutor } from "./command-executor.js";
+export interface IGitOps {
+    cleanWorkspace(): void;
+    clone(gitUrl: string): Promise<void>;
+    fetch(options?: {
+        prune?: boolean;
+    }): Promise<void>;
+    createBranch(branchName: string): Promise<void>;
+    commit(message: string): Promise<boolean>;
+    push(branchName: string, options?: {
+        force?: boolean;
+    }): Promise<void>;
+    getDefaultBranch(): Promise<{
+        branch: string;
+        method: string;
+    }>;
+    writeFile(fileName: string, content: string): void;
+    setExecutable(fileName: string): Promise<void>;
+    getFileContent(fileName: string): string | null;
+    deleteFile(fileName: string): void;
+    wouldChange(fileName: string, content: string): boolean;
+    hasChanges(): Promise<boolean>;
+    getChangedFiles(): Promise<string[]>;
+    hasStagedChanges(): Promise<boolean>;
+    fileExistsOnBranch(fileName: string, branch: string): Promise<boolean>;
+    fileExists(fileName: string): boolean;
+}
 export interface GitOpsOptions {
     workDir: string;
     dryRun?: boolean;
-    executor?: CommandExecutor;
+    executor?: ICommandExecutor;
     /** Number of retries for network operations (default: 3) */
     retries?: number;
 }
-export declare class GitOps {
+export declare class GitOps implements IGitOps {
     private workDir;
     private dryRun;
     private executor;

package/dist/logger.d.ts

@@ -3,6 +3,13 @@ export interface ILogger {
     info(message: string): void;
     fileDiff(fileName: string, status: FileStatus, diffLines: string[]): void;
     diffSummary(newCount: number, modifiedCount: number, unchangedCount: number, deletedCount?: number): void;
+    setTotal(total: number): void;
+    progress(current: number, repoName: string, message: string): void;
+    success(current: number, repoName: string, message: string): void;
+    skip(current: number, repoName: string, reason: string): void;
+    error(current: number, repoName: string, error: string): void;
+    summary(): void;
+    hasFailures(): boolean;
 }
 export interface LoggerStats {
     total: number;
@@ -10,7 +17,7 @@ export interface LoggerStats {
     failed: number;
     skipped: number;
 }
-export declare class Logger {
+export declare class Logger implements ILogger {
     private stats;
     setTotal(total: number): void;
     progress(current: number, repoName: string, message: string): void;

package/dist/pr-creator.d.ts

@@ -1,6 +1,6 @@
 import { RepoInfo } from "./repo-detector.js";
 import { MergeResult, PRMergeConfig } from "./strategies/index.js";
-import { CommandExecutor } from "./command-executor.js";
+import { ICommandExecutor } from "./command-executor.js";
 export { escapeShellArg } from "./shell-utils.js";
 export interface FileAction {
     fileName: string;
@@ -18,7 +18,7 @@ export interface PROptions {
     /** Custom PR body template */
     prTemplate?: string;
     /** Optional command executor for shell commands (for testing) */
-    executor?: CommandExecutor;
+    executor?: ICommandExecutor;
     /** GitHub App installation token for authentication */
     token?: string;
 }
@@ -51,7 +51,7 @@ export interface MergePROptions {
     dryRun?: boolean;
     retries?: number;
     /** Optional command executor for shell commands (for testing) */
-    executor?: CommandExecutor;
+    executor?: ICommandExecutor;
     /** GitHub App installation token for authentication */
     token?: string;
 }

package/dist/repository-processor.d.ts

@@ -3,8 +3,14 @@ import { RepoInfo } from "./repo-detector.js";
 import { GitOpsOptions } from "./git-ops.js";
 import { IAuthenticatedGitOps, GitAuthOptions } from "./authenticated-git-ops.js";
 import { ILogger } from "./logger.js";
-import { CommandExecutor } from "./command-executor.js";
+import { ICommandExecutor } from "./command-executor.js";
 import { DiffStats } from "./diff-utils.js";
+export interface IRepositoryProcessor {
+    process(repoConfig: RepoConfig, repoInfo: RepoInfo, options: ProcessorOptions): Promise<ProcessorResult>;
+    updateManifestOnly(repoInfo: RepoInfo, repoConfig: RepoConfig, options: ProcessorOptions, manifestUpdate: {
+        rulesets: string[];
+    }): Promise<ProcessorResult>;
+}
 export interface ProcessorOptions {
     branchName: string;
     workDir: string;
@@ -14,7 +20,7 @@ export interface ProcessorOptions {
     /** Number of retries for network operations (default: 3) */
     retries?: number;
     /** Command executor for shell commands (for testing) */
-    executor?: CommandExecutor;
+    executor?: ICommandExecutor;
     /** Custom PR body template */
     prTemplate?: string;
     /** Skip deleting orphaned files even if deleteOrphaned is configured */
@@ -38,7 +44,7 @@ export interface ProcessorResult {
     };
     diffStats?: DiffStats;
 }
-export declare class RepositoryProcessor {
+export declare class RepositoryProcessor implements IRepositoryProcessor {
     private gitOps;
     private readonly gitOpsFactory;
     private readonly log;

package/dist/ruleset-processor.d.ts

@@ -1,6 +1,9 @@
 import type { RepoConfig } from "./config.js";
 import type { RepoInfo } from "./repo-detector.js";
 import { GitHubRulesetStrategy } from "./strategies/github-ruleset-strategy.js";
+export interface IRulesetProcessor {
+    process(repoConfig: RepoConfig, repoInfo: RepoInfo, options: RulesetProcessorOptions): Promise<RulesetProcessorResult>;
+}
 export interface RulesetProcessorOptions {
     configId: string;
     dryRun?: boolean;
@@ -28,7 +31,7 @@ export interface RulesetProcessorResult {
  * Processes ruleset configuration for a repository.
  * Handles create/update/delete operations via GitHub Rulesets API.
  */
-export declare class RulesetProcessor {
+export declare class RulesetProcessor implements IRulesetProcessor {
     private readonly strategy;
     constructor(strategy?: GitHubRulesetStrategy);
     /**

package/dist/strategies/azure-pr-strategy.d.ts

@@ -1,8 +1,8 @@
 import { PRResult } from "../pr-creator.js";
 import { BasePRStrategy, PRStrategyOptions, CloseExistingPROptions, MergeOptions, MergeResult } from "./pr-strategy.js";
-import { CommandExecutor } from "../command-executor.js";
+import { ICommandExecutor } from "../command-executor.js";
 export declare class AzurePRStrategy extends BasePRStrategy {
-    constructor(executor?: CommandExecutor);
+    constructor(executor?: ICommandExecutor);
     private getOrgUrl;
     private buildPRUrl;
     checkExistingPR(options: PRStrategyOptions): Promise<string | null>;

package/dist/strategies/commit-strategy-selector.d.ts

@@ -1,6 +1,6 @@
 import { RepoInfo } from "../repo-detector.js";
-import { CommitStrategy } from "./commit-strategy.js";
-import { CommandExecutor } from "../command-executor.js";
+import { ICommitStrategy } from "./commit-strategy.js";
+import { ICommandExecutor } from "../command-executor.js";
 /**
  * Checks if GitHub App credentials are configured via environment variables.
  * Both XFG_GITHUB_APP_ID and XFG_GITHUB_APP_PRIVATE_KEY must be set.
@@ -19,4 +19,4 @@ export declare function hasGitHubAppCredentials(): boolean;
  * @param repoInfo - Repository information
  * @param executor - Optional command executor for shell commands
  */
-export declare function getCommitStrategy(repoInfo: RepoInfo, executor?: CommandExecutor): CommitStrategy;
+export declare function getCommitStrategy(repoInfo: RepoInfo, executor?: ICommandExecutor): ICommitStrategy;

package/dist/strategies/commit-strategy.d.ts

@@ -27,7 +27,7 @@ export interface CommitResult {
  * Strategy interface for creating commits.
  * Implementations handle platform-specific commit mechanisms.
  */
-export interface CommitStrategy {
+export interface ICommitStrategy {
     /**
      * Create a commit with the given file changes and push to remote.
      * @returns Commit result with SHA and verification status

package/dist/strategies/git-commit-strategy.d.ts

@@ -1,13 +1,13 @@
-import { CommitStrategy, CommitOptions, CommitResult } from "./commit-strategy.js";
-import { CommandExecutor } from "../command-executor.js";
+import { ICommitStrategy, CommitOptions, CommitResult } from "./commit-strategy.js";
+import { ICommandExecutor } from "../command-executor.js";
 /**
  * Git-based commit strategy using standard git commands (add, commit, push).
  * Used with PAT authentication. Commits via this strategy are NOT verified
  * by GitHub (no signature).
  */
-export declare class GitCommitStrategy implements CommitStrategy {
+export declare class GitCommitStrategy implements ICommitStrategy {
     private executor;
-    constructor(executor?: CommandExecutor);
+    constructor(executor?: ICommandExecutor);
     /**
      * Create a commit with the given file changes and push to remote.
      * Runs: git add -A, git commit, git push (with optional --force-with-lease)

package/dist/strategies/github-ruleset-strategy.d.ts

@@ -1,6 +1,7 @@
-import { CommandExecutor } from "../command-executor.js";
+import { ICommandExecutor } from "../command-executor.js";
 import { RepoInfo } from "../repo-detector.js";
 import type { Ruleset } from "../config.js";
+import type { IRulesetStrategy } from "./ruleset-strategy.js";
 /**
  * GitHub Ruleset response from API (snake_case).
  */
@@ -50,9 +51,9 @@ export interface RulesetStrategyOptions {
  * GitHub Ruleset Strategy for managing repository rulesets via GitHub REST API.
  * Uses `gh api` CLI for authentication and API calls.
  */
-export declare class GitHubRulesetStrategy {
+export declare class GitHubRulesetStrategy implements IRulesetStrategy {
     private executor;
-    constructor(executor?: CommandExecutor);
+    constructor(executor?: ICommandExecutor);
     /**
      * Lists all rulesets for a repository.
      */

package/dist/strategies/gitlab-pr-strategy.d.ts

@@ -1,8 +1,8 @@
 import { PRResult } from "../pr-creator.js";
 import { BasePRStrategy, PRStrategyOptions, CloseExistingPROptions, MergeOptions, MergeResult } from "./pr-strategy.js";
-import { CommandExecutor } from "../command-executor.js";
+import { ICommandExecutor } from "../command-executor.js";
 export declare class GitLabPRStrategy extends BasePRStrategy {
-    constructor(executor?: CommandExecutor);
+    constructor(executor?: ICommandExecutor);
     /**
      * Build the repo flag for glab commands.
      * Format: namespace/repo (supports nested groups)

package/dist/strategies/graphql-commit-strategy.d.ts

@@ -1,5 +1,5 @@
-import { CommitStrategy, CommitOptions, CommitResult } from "./commit-strategy.js";
-import { CommandExecutor } from "../command-executor.js";
+import { ICommitStrategy, CommitOptions, CommitResult } from "./commit-strategy.js";
+import { ICommandExecutor } from "../command-executor.js";
 /**
  * Maximum payload size for GitHub GraphQL API (50MB).
  * Base64 encoding adds ~33% overhead, so raw content should be checked.
@@ -29,9 +29,9 @@ export declare function validateBranchName(branchName: string): void;
  *
  * This strategy is GitHub-only and requires the `gh` CLI to be authenticated.
  */
-export declare class GraphQLCommitStrategy implements CommitStrategy {
+export declare class GraphQLCommitStrategy implements ICommitStrategy {
     private executor;
-    constructor(executor?: CommandExecutor);
+    constructor(executor?: ICommandExecutor);
     /**
      * Create a commit with the given file changes using GitHub's GraphQL API.
      * Uses the createCommitOnBranch mutation for verified commits.

package/dist/strategies/graphql-commit-strategy.js

@@ -191,8 +191,9 @@ export class GraphQLCommitStrategy {
         // Branch name was validated in commit(), safe for shell use
         try {
             // Check if the branch exists on remote
+            // Use skipRetry because failure is expected for new branches
             if (gitOps) {
-                await gitOps.lsRemote(branchName);
+                await gitOps.lsRemote(branchName, { skipRetry: true });
             }
             else {
                 await this.executor.exec(`git ls-remote --exit-code --heads origin ${escapeShellArg(branchName)}`, workDir);

package/dist/strategies/index.d.ts

@@ -1,12 +1,12 @@
 import { RepoInfo } from "../repo-detector.js";
-import type { PRStrategy } from "./pr-strategy.js";
-import { CommandExecutor } from "../command-executor.js";
-export type { PRStrategy, PRStrategyOptions, CloseExistingPROptions, PRMergeConfig, MergeOptions, MergeResult, } from "./pr-strategy.js";
+import type { IPRStrategy } from "./pr-strategy.js";
+import { ICommandExecutor } from "../command-executor.js";
+export type { IPRStrategy, PRStrategyOptions, CloseExistingPROptions, PRMergeConfig, MergeOptions, MergeResult, } from "./pr-strategy.js";
 export { BasePRStrategy, PRWorkflowExecutor } from "./pr-strategy.js";
 export { GitHubPRStrategy } from "./github-pr-strategy.js";
 export { AzurePRStrategy } from "./azure-pr-strategy.js";
 export { GitLabPRStrategy } from "./gitlab-pr-strategy.js";
-export type { CommitStrategy, CommitOptions, CommitResult, FileChange, } from "./commit-strategy.js";
+export type { ICommitStrategy, CommitOptions, CommitResult, FileChange, } from "./commit-strategy.js";
 export { GitCommitStrategy } from "./git-commit-strategy.js";
 export { GraphQLCommitStrategy, MAX_PAYLOAD_SIZE, } from "./graphql-commit-strategy.js";
 export { getCommitStrategy, hasGitHubAppCredentials, } from "./commit-strategy-selector.js";
@@ -15,4 +15,4 @@ export { getCommitStrategy, hasGitHubAppCredentials, } from "./commit-strategy-s
  * @param repoInfo - Repository information
  * @param executor - Optional command executor for shell commands
  */
-export declare function getPRStrategy(repoInfo: RepoInfo, executor?: CommandExecutor): PRStrategy;
+export declare function getPRStrategy(repoInfo: RepoInfo, executor?: ICommandExecutor): IPRStrategy;

package/dist/strategies/pr-strategy.d.ts

@@ -1,6 +1,6 @@
 import { PRResult } from "../pr-creator.js";
 import { RepoInfo } from "../repo-detector.js";
-import { CommandExecutor } from "../command-executor.js";
+import { ICommandExecutor } from "../command-executor.js";
 import type { MergeMode, MergeStrategy } from "../config.js";
 export interface PRMergeConfig {
     mode: MergeMode;
@@ -51,7 +51,7 @@ export interface CloseExistingPROptions {
  * Strategies focus on platform-specific logic (checkExistingPR, create, merge).
  * Use PRWorkflowExecutor for full workflow orchestration with error handling.
  */
-export interface PRStrategy {
+export interface IPRStrategy {
     /**
      * Check if a PR already exists for the given branch
      * @returns PR URL if exists, null otherwise
@@ -79,10 +79,10 @@ export interface PRStrategy {
      */
     execute(options: PRStrategyOptions): Promise<PRResult>;
 }
-export declare abstract class BasePRStrategy implements PRStrategy {
+export declare abstract class BasePRStrategy implements IPRStrategy {
     protected bodyFilePath: string;
-    protected executor: CommandExecutor;
-    constructor(executor?: CommandExecutor);
+    protected executor: ICommandExecutor;
+    constructor(executor?: ICommandExecutor);
     abstract checkExistingPR(options: PRStrategyOptions): Promise<string | null>;
     abstract closeExistingPR(options: CloseExistingPROptions): Promise<boolean>;
     abstract create(options: PRStrategyOptions): Promise<PRResult>;
@@ -110,7 +110,7 @@ export declare abstract class BasePRStrategy implements PRStrategy {
  */
 export declare class PRWorkflowExecutor {
     private readonly strategy;
-    constructor(strategy: PRStrategy);
+    constructor(strategy: IPRStrategy);
     /**
      * Execute the full PR creation workflow with error handling.
      */

package/dist/strategies/ruleset-strategy.d.ts

@@ -0,0 +1,10 @@
+import type { RepoInfo } from "../repo-detector.js";
+import type { Ruleset } from "../config.js";
+import type { GitHubRuleset, RulesetStrategyOptions } from "./github-ruleset-strategy.js";
+export interface IRulesetStrategy {
+    list(repoInfo: RepoInfo, options?: RulesetStrategyOptions): Promise<GitHubRuleset[]>;
+    get(repoInfo: RepoInfo, rulesetId: number, options?: RulesetStrategyOptions): Promise<GitHubRuleset>;
+    create(repoInfo: RepoInfo, name: string, ruleset: Ruleset, options?: RulesetStrategyOptions): Promise<GitHubRuleset>;
+    update(repoInfo: RepoInfo, rulesetId: number, name: string, ruleset: Ruleset, options?: RulesetStrategyOptions): Promise<GitHubRuleset>;
+    delete(repoInfo: RepoInfo, rulesetId: number, options?: RulesetStrategyOptions): Promise<void>;
+}

package/dist/strategies/ruleset-strategy.js

@@ -0,0 +1 @@
+export {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aspruyt/xfg",
-  "version": "3.1.4",
+  "version": "3.2.0",
   "description": "CLI tool for repository-as-code",
   "type": "module",
   "main": "dist/index.js",
@@ -27,7 +27,7 @@
     "start": "node dist/cli.js",
     "dev": "ts-node src/cli.ts",
     "test": "node --import tsx scripts/run-tests.js",
-    "test:coverage": "c8 --check-coverage --lines 95 --reporter=text --reporter=lcov --all --src=src --exclude='src/**/*.test.ts' --exclude='scripts/**' npm test",
+    "test:coverage": "c8 --check-coverage --lines 95 --reporter=text --reporter=lcov --all --src=src --exclude='test/**/*.test.ts' --exclude='scripts/**' npm test",
     "test:integration:github": "npm run build && node --import tsx --test test/integration/github.test.ts",
     "test:integration:ado": "npm run build && node --import tsx --test test/integration/ado.test.ts",
     "test:integration:gitlab": "npm run build && node --import tsx --test test/integration/gitlab.test.ts",