@aspruyt/xfg 3.7.7 → 3.8.0

package/dist/output/settings-report.js

@@ -0,0 +1,300 @@
+import { appendFileSync } from "node:fs";
+import chalk from "chalk";
+import { formatPropertyTree, } from "../settings/rulesets/formatter.js";
+// =============================================================================
+// Helpers
+// =============================================================================
+function formatValue(val) {
+    if (val === null)
+        return "null";
+    if (val === undefined)
+        return "undefined";
+    if (typeof val === "string")
+        return `"${val}"`;
+    if (typeof val === "boolean")
+        return val ? "true" : "false";
+    return String(val);
+}
+function formatRulesetConfig(config, indent) {
+    const lines = [];
+    function renderObject(obj, currentIndent) {
+        for (const [k, v] of Object.entries(obj)) {
+            renderValue(k, v, currentIndent);
+        }
+    }
+    function renderValue(key, value, currentIndent) {
+        const pad = "    ".repeat(currentIndent);
+        if (value === null || value === undefined)
+            return;
+        if (Array.isArray(value)) {
+            if (value.length === 0) {
+                lines.push(chalk.green(`${pad}+ ${key}: []`));
+            }
+            else if (value.every((v) => typeof v !== "object")) {
+                lines.push(chalk.green(`${pad}+ ${key}: [${value.map((v) => (typeof v === "string" ? `"${v}"` : String(v))).join(", ")}]`));
+            }
+            else {
+                lines.push(chalk.green(`${pad}+ ${key}:`));
+                for (let i = 0; i < value.length; i++) {
+                    const item = value[i];
+                    if (typeof item === "object" && item !== null) {
+                        const obj = item;
+                        const typeLabel = "type" in obj ? ` (${obj.type})` : "";
+                        lines.push(chalk.green(`${pad}    + [${i}]${typeLabel}:`));
+                        renderObject(obj, currentIndent + 2);
+                    }
+                    else {
+                        lines.push(chalk.green(`${pad}    + ${formatValue(item)}`));
+                    }
+                }
+            }
+        }
+        else if (typeof value === "object") {
+            lines.push(chalk.green(`${pad}+ ${key}:`));
+            renderObject(value, currentIndent + 1);
+        }
+        else {
+            lines.push(chalk.green(`${pad}+ ${key}: ${formatValue(value)}`));
+        }
+    }
+    for (const [key, value] of Object.entries(config)) {
+        if (key === "name")
+            continue; // Name is in the header
+        renderValue(key, value, indent);
+    }
+    return lines;
+}
+function formatSummary(totals) {
+    const parts = [];
+    const settingsTotal = totals.settings.add + totals.settings.change;
+    const rulesetsTotal = totals.rulesets.create + totals.rulesets.update + totals.rulesets.delete;
+    if (settingsTotal > 0) {
+        const settingWord = settingsTotal === 1 ? "setting" : "settings";
+        const actions = [];
+        if (totals.settings.add > 0)
+            actions.push(`${totals.settings.add} to add`);
+        if (totals.settings.change > 0)
+            actions.push(`${totals.settings.change} to change`);
+        parts.push(`${settingsTotal} ${settingWord} (${actions.join(", ")})`);
+    }
+    if (rulesetsTotal > 0) {
+        const rulesetWord = rulesetsTotal === 1 ? "ruleset" : "rulesets";
+        const actions = [];
+        if (totals.rulesets.create > 0)
+            actions.push(`${totals.rulesets.create} to create`);
+        if (totals.rulesets.update > 0)
+            actions.push(`${totals.rulesets.update} to update`);
+        if (totals.rulesets.delete > 0)
+            actions.push(`${totals.rulesets.delete} to delete`);
+        parts.push(`${rulesetsTotal} ${rulesetWord} (${actions.join(", ")})`);
+    }
+    if (parts.length === 0) {
+        return "No changes";
+    }
+    return `Plan: ${parts.join(", ")}`;
+}
+// =============================================================================
+// CLI Formatter
+// =============================================================================
+export function formatSettingsReportCLI(report) {
+    const lines = [];
+    for (const repo of report.repos) {
+        if (repo.settings.length === 0 &&
+            repo.rulesets.length === 0 &&
+            !repo.error) {
+            continue;
+        }
+        // Repo header
+        lines.push(chalk.yellow(`~ ${repo.repoName}`));
+        // Settings
+        for (const setting of repo.settings) {
+            // Skip settings where both values are undefined
+            if (setting.oldValue === undefined && setting.newValue === undefined) {
+                continue;
+            }
+            if (setting.action === "add") {
+                lines.push(chalk.green(`    + ${setting.name}: ${formatValue(setting.newValue)}`));
+            }
+            else {
+                lines.push(chalk.yellow(`    ~ ${setting.name}: ${formatValue(setting.oldValue)} → ${formatValue(setting.newValue)}`));
+            }
+        }
+        // Rulesets
+        for (const ruleset of repo.rulesets) {
+            if (ruleset.action === "create") {
+                lines.push(chalk.green(`    + ruleset "${ruleset.name}"`));
+                if (ruleset.config) {
+                    lines.push(...formatRulesetConfig(ruleset.config, 2));
+                }
+            }
+            else if (ruleset.action === "update") {
+                lines.push(chalk.yellow(`    ~ ruleset "${ruleset.name}"`));
+                if (ruleset.propertyDiffs && ruleset.propertyDiffs.length > 0) {
+                    const treeLines = formatPropertyTree(ruleset.propertyDiffs);
+                    for (const line of treeLines) {
+                        lines.push(`        ${line}`);
+                    }
+                }
+            }
+            else if (ruleset.action === "delete") {
+                lines.push(chalk.red(`    - ruleset "${ruleset.name}"`));
+            }
+        }
+        // Error
+        if (repo.error) {
+            lines.push(chalk.red(`    Error: ${repo.error}`));
+        }
+        lines.push(""); // Blank line between repos
+    }
+    // Summary
+    lines.push(formatSummary(report.totals));
+    return lines;
+}
+// =============================================================================
+// Markdown Formatter
+// =============================================================================
+function formatValuePlain(val) {
+    if (val === null)
+        return "null";
+    if (val === undefined)
+        return "undefined";
+    if (typeof val === "string")
+        return `"${val}"`;
+    if (typeof val === "boolean")
+        return val ? "true" : "false";
+    return String(val);
+}
+function formatRulesetConfigPlain(config, indent) {
+    const lines = [];
+    function renderObject(obj, currentIndent) {
+        for (const [k, v] of Object.entries(obj)) {
+            renderValue(k, v, currentIndent);
+        }
+    }
+    function renderValue(key, value, currentIndent) {
+        const pad = "    ".repeat(currentIndent);
+        if (value === null || value === undefined)
+            return;
+        if (Array.isArray(value)) {
+            if (value.length === 0) {
+                lines.push(`${pad}+ ${key}: []`);
+            }
+            else if (value.every((v) => typeof v !== "object")) {
+                lines.push(`${pad}+ ${key}: [${value.map((v) => (typeof v === "string" ? `"${v}"` : String(v))).join(", ")}]`);
+            }
+            else {
+                lines.push(`${pad}+ ${key}:`);
+                for (let i = 0; i < value.length; i++) {
+                    const item = value[i];
+                    if (typeof item === "object" && item !== null) {
+                        const obj = item;
+                        const typeLabel = "type" in obj ? ` (${obj.type})` : "";
+                        lines.push(`${pad}    + [${i}]${typeLabel}:`);
+                        renderObject(obj, currentIndent + 2);
+                    }
+                    else {
+                        lines.push(`${pad}    + ${formatValuePlain(item)}`);
+                    }
+                }
+            }
+        }
+        else if (typeof value === "object") {
+            lines.push(`${pad}+ ${key}:`);
+            renderObject(value, currentIndent + 1);
+        }
+        else {
+            lines.push(`${pad}+ ${key}: ${formatValuePlain(value)}`);
+        }
+    }
+    for (const [key, value] of Object.entries(config)) {
+        if (key === "name")
+            continue;
+        renderValue(key, value, indent);
+    }
+    return lines;
+}
+export function formatSettingsReportMarkdown(report, dryRun) {
+    const lines = [];
+    // Title
+    const titleSuffix = dryRun ? " (Dry Run)" : "";
+    lines.push(`## Repository Settings Summary${titleSuffix}`);
+    lines.push("");
+    // Dry-run warning
+    if (dryRun) {
+        lines.push("> [!WARNING]");
+        lines.push("> This was a dry run — no changes were applied");
+        lines.push("");
+    }
+    // Diff block
+    const diffLines = [];
+    for (const repo of report.repos) {
+        if (repo.settings.length === 0 &&
+            repo.rulesets.length === 0 &&
+            !repo.error) {
+            continue;
+        }
+        diffLines.push(`~ ${repo.repoName}`);
+        for (const setting of repo.settings) {
+            // Skip settings where both values are undefined
+            if (setting.oldValue === undefined && setting.newValue === undefined) {
+                continue;
+            }
+            if (setting.action === "add") {
+                diffLines.push(`    + ${setting.name}: ${formatValuePlain(setting.newValue)}`);
+            }
+            else {
+                diffLines.push(`    ~ ${setting.name}: ${formatValuePlain(setting.oldValue)} → ${formatValuePlain(setting.newValue)}`);
+            }
+        }
+        for (const ruleset of repo.rulesets) {
+            if (ruleset.action === "create") {
+                diffLines.push(`    + ruleset "${ruleset.name}"`);
+                if (ruleset.config) {
+                    diffLines.push(...formatRulesetConfigPlain(ruleset.config, 2));
+                }
+            }
+            else if (ruleset.action === "update") {
+                diffLines.push(`    ~ ruleset "${ruleset.name}"`);
+                if (ruleset.propertyDiffs && ruleset.propertyDiffs.length > 0) {
+                    for (const diff of ruleset.propertyDiffs) {
+                        const path = diff.path.join(".");
+                        if (diff.action === "add") {
+                            diffLines.push(`        + ${path}: ${formatValuePlain(diff.newValue)}`);
+                        }
+                        else if (diff.action === "change") {
+                            diffLines.push(`        ~ ${path}: ${formatValuePlain(diff.oldValue)} → ${formatValuePlain(diff.newValue)}`);
+                        }
+                        else if (diff.action === "remove") {
+                            diffLines.push(`        - ${path}`);
+                        }
+                    }
+                }
+            }
+            else if (ruleset.action === "delete") {
+                diffLines.push(`    - ruleset "${ruleset.name}"`);
+            }
+        }
+        if (repo.error) {
+            diffLines.push(`    ! Error: ${repo.error}`);
+        }
+    }
+    if (diffLines.length > 0) {
+        lines.push("```diff");
+        lines.push(...diffLines);
+        lines.push("```");
+        lines.push("");
+    }
+    // Summary
+    lines.push(`**${formatSummary(report.totals)}**`);
+    return lines.join("\n");
+}
+// =============================================================================
+// File Writer
+// =============================================================================
+export function writeSettingsReportSummary(report, dryRun) {
+    const summaryPath = process.env.GITHUB_STEP_SUMMARY;
+    if (!summaryPath)
+        return;
+    const markdown = formatSettingsReportMarkdown(report, dryRun);
+    appendFileSync(summaryPath, "\n" + markdown + "\n");
+}

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

@@ -0,0 +1,24 @@
+export interface SyncReport {
+    repos: RepoFileChanges[];
+    totals: {
+        files: {
+            create: number;
+            update: number;
+            delete: number;
+        };
+    };
+}
+export interface RepoFileChanges {
+    repoName: string;
+    files: FileChange[];
+    prUrl?: string;
+    mergeOutcome?: "manual" | "auto" | "force" | "direct";
+    error?: string;
+}
+export interface FileChange {
+    path: string;
+    action: "create" | "update" | "delete";
+}
+export declare function formatSyncReportCLI(report: SyncReport): string[];
+export declare function formatSyncReportMarkdown(report: SyncReport, dryRun: boolean): string;
+export declare function writeSyncReportSummary(report: SyncReport, dryRun: boolean): void;

package/dist/output/sync-report.js

@@ -0,0 +1,99 @@
+// src/output/sync-report.ts
+import { appendFileSync } from "node:fs";
+import chalk from "chalk";
+function formatSummary(totals) {
+    const total = totals.files.create + totals.files.update + totals.files.delete;
+    if (total === 0) {
+        return "No changes";
+    }
+    const parts = [];
+    if (totals.files.create > 0)
+        parts.push(`${totals.files.create} to create`);
+    if (totals.files.update > 0)
+        parts.push(`${totals.files.update} to update`);
+    if (totals.files.delete > 0)
+        parts.push(`${totals.files.delete} to delete`);
+    const fileWord = total === 1 ? "file" : "files";
+    return `Plan: ${total} ${fileWord} (${parts.join(", ")})`;
+}
+export function formatSyncReportCLI(report) {
+    const lines = [];
+    for (const repo of report.repos) {
+        if (repo.files.length === 0 && !repo.error) {
+            continue;
+        }
+        // Repo header
+        lines.push(chalk.yellow(`~ ${repo.repoName}`));
+        // Files
+        for (const file of repo.files) {
+            if (file.action === "create") {
+                lines.push(chalk.green(`    + ${file.path}`));
+            }
+            else if (file.action === "update") {
+                lines.push(chalk.yellow(`    ~ ${file.path}`));
+            }
+            else if (file.action === "delete") {
+                lines.push(chalk.red(`    - ${file.path}`));
+            }
+        }
+        // Error
+        if (repo.error) {
+            lines.push(chalk.red(`    Error: ${repo.error}`));
+        }
+        lines.push(""); // Blank line between repos
+    }
+    // Summary
+    lines.push(formatSummary(report.totals));
+    return lines;
+}
+export function formatSyncReportMarkdown(report, dryRun) {
+    const lines = [];
+    // Title
+    const titleSuffix = dryRun ? " (Dry Run)" : "";
+    lines.push(`## Config Sync Summary${titleSuffix}`);
+    lines.push("");
+    // Dry-run warning
+    if (dryRun) {
+        lines.push("> [!WARNING]");
+        lines.push("> This was a dry run — no changes were applied");
+        lines.push("");
+    }
+    // Diff block
+    const diffLines = [];
+    for (const repo of report.repos) {
+        if (repo.files.length === 0 && !repo.error) {
+            continue;
+        }
+        diffLines.push(`~ ${repo.repoName}`);
+        for (const file of repo.files) {
+            if (file.action === "create") {
+                diffLines.push(`    + ${file.path}`);
+            }
+            else if (file.action === "update") {
+                diffLines.push(`    ~ ${file.path}`);
+            }
+            else if (file.action === "delete") {
+                diffLines.push(`    - ${file.path}`);
+            }
+        }
+        if (repo.error) {
+            diffLines.push(`    ! Error: ${repo.error}`);
+        }
+    }
+    if (diffLines.length > 0) {
+        lines.push("```diff");
+        lines.push(...diffLines);
+        lines.push("```");
+        lines.push("");
+    }
+    // Summary
+    lines.push(`**${formatSummary(report.totals)}**`);
+    return lines.join("\n");
+}
+export function writeSyncReportSummary(report, dryRun) {
+    const summaryPath = process.env.GITHUB_STEP_SUMMARY;
+    if (!summaryPath)
+        return;
+    const markdown = formatSyncReportMarkdown(report, dryRun);
+    appendFileSync(summaryPath, "\n" + markdown + "\n");
+}

package/dist/settings/index.d.ts

@@ -1,3 +1,2 @@
 export * from "./rulesets/index.js";
 export * from "./repo-settings/index.js";
-export { rulesetResultToResources, syncResultToResources, repoSettingsResultToResources, } from "./resource-converters.js";

package/dist/settings/index.js

@@ -2,5 +2,3 @@
 export * from "./rulesets/index.js";
 // Repo settings
 export * from "./repo-settings/index.js";
-// Resource converters
-export { rulesetResultToResources, syncResultToResources, repoSettingsResultToResources, } from "./resource-converters.js";

package/dist/settings/repo-settings/formatter.d.ts

@@ -2,6 +2,8 @@ import type { RepoSettingsChange } from "./diff.js";
 export interface RepoSettingsPlanEntry {
     property: string;
     action: "add" | "change";
+    oldValue?: unknown;
+    newValue?: unknown;
 }
 export interface RepoSettingsPlanResult {
     lines: string[];

package/dist/settings/repo-settings/formatter.js

@@ -54,12 +54,21 @@ export function formatRepoSettingsPlan(changes) {
         if (change.action === "add") {
             lines.push(chalk.green(`    + ${change.property}: ${formatValue(change.newValue)}`));
             adds++;
-            entries.push({ property: change.property, action: "add" });
+            entries.push({
+                property: change.property,
+                action: "add",
+                newValue: change.newValue,
+            });
         }
         else if (change.action === "change") {
             lines.push(chalk.yellow(`    ~ ${change.property}: ${formatValue(change.oldValue)} → ${formatValue(change.newValue)}`));
             changesCount++;
-            entries.push({ property: change.property, action: "change" });
+            entries.push({
+                property: change.property,
+                action: "change",
+                oldValue: change.oldValue,
+                newValue: change.newValue,
+            });
         }
     }
     return { lines, adds, changes: changesCount, warnings, entries };

package/dist/settings/repo-settings/processor.js

@@ -69,8 +69,14 @@ export class RepoSettingsProcessor {
                     planOutput,
                 };
             }
-            // Apply changes
-            await this.applyChanges(githubRepo, desiredSettings, strategyOptions);
+            // Apply changes - only send settings that actually changed
+            const changedSettings = changes.reduce((acc, change) => {
+                if (change.action !== "unchanged") {
+                    acc[change.property] = change.newValue;
+                }
+                return acc;
+            }, {});
+            await this.applyChanges(githubRepo, changedSettings, strategyOptions);
             return {
                 success: true,
                 repoName,

package/dist/settings/rulesets/formatter.d.ts

@@ -1,4 +1,5 @@
 import { type RulesetChange, type RulesetAction } from "./diff.js";
+import type { Ruleset } from "../../config/index.js";
 import { type PropertyDiff } from "./index.js";
 export type { DiffAction, PropertyDiff } from "./index.js";
 export { computePropertyDiffs } from "./index.js";
@@ -11,6 +12,8 @@ export interface RulesetPlanEntry {
         changed: number;
         removed: number;
     };
+    propertyDiffs?: PropertyDiff[];
+    config?: Ruleset;
 }
 export interface RulesetPlanResult {
     lines: string[];

package/dist/settings/rulesets/formatter.js

@@ -264,7 +264,12 @@ export function formatRulesetPlan(changes) {
             const propertyCount = change.desired
                 ? Object.keys(change.desired).length
                 : 0;
-            entries.push({ name: change.name, action: "create", propertyCount });
+            entries.push({
+                name: change.name,
+                action: "create",
+                propertyCount,
+                config: change.desired,
+            });
             lines.push(""); // Blank line between rulesets
         }
     }
@@ -289,6 +294,7 @@ export function formatRulesetPlan(changes) {
                     name: change.name,
                     action: "update",
                     propertyChanges: { added, changed, removed },
+                    propertyDiffs: diffs,
                 });
             }
             else {

package/dist/sync/index.d.ts

@@ -7,7 +7,7 @@ export { CommitPushManager } from "./commit-push-manager.js";
 export { formatCommitMessage } from "./commit-message.js";
 export { FileSyncOrchestrator } from "./file-sync-orchestrator.js";
 export { PRMergeHandler } from "./pr-merge-handler.js";
-export type { IFileWriter, FileWriteContext, FileWriterDeps, FileWriteAllResult, FileWriteResult, IManifestManager, OrphanProcessResult, OrphanDeleteOptions, OrphanDeleteDeps, IBranchManager, BranchSetupOptions, IAuthOptionsBuilder, AuthResult, IRepositorySession, SessionOptions, SessionContext, ICommitPushManager, CommitPushOptions, CommitPushResult, GitOpsFactory, IRepositoryProcessor, ProcessorOptions, ProcessorResult, IFileSyncOrchestrator, FileSyncResult, IPRMergeHandler, PRHandlerOptions, } from "./types.js";
+export type { IFileWriter, FileWriteContext, FileWriterDeps, FileWriteAllResult, FileWriteResult, IManifestManager, OrphanProcessResult, OrphanDeleteOptions, OrphanDeleteDeps, IBranchManager, BranchSetupOptions, IAuthOptionsBuilder, AuthResult, IRepositorySession, SessionOptions, SessionContext, ICommitPushManager, CommitPushOptions, CommitPushResult, GitOpsFactory, IRepositoryProcessor, ProcessorOptions, ProcessorResult, FileChangeDetail, IFileSyncOrchestrator, FileSyncResult, IPRMergeHandler, PRHandlerOptions, } from "./types.js";
 export { RepositoryProcessor } from "./repository-processor.js";
 export { createEmptyManifest, loadManifest, saveManifest, getManagedFiles, getManagedRulesets, updateManifest, updateManifestRulesets, MANIFEST_FILENAME, type XfgManifest, type XfgManifestConfigEntry, } from "./manifest.js";
 export { getFileStatus, formatStatusBadge, formatDiffLine, generateDiff, createDiffStats, incrementDiffStats, type FileStatus, type DiffStats, } from "./diff-utils.js";

package/dist/sync/pr-merge-handler.d.ts

@@ -3,9 +3,9 @@ import type { RepoInfo } from "../shared/repo-detector.js";
 import type { ILogger } from "../shared/logger.js";
 import { type FileAction } from "../vcs/pr-creator.js";
 import type { DiffStats } from "./diff-utils.js";
-import type { ProcessorResult, PRHandlerOptions, IPRMergeHandler } from "./types.js";
+import type { ProcessorResult, PRHandlerOptions, IPRMergeHandler, FileChangeDetail } from "./types.js";
 export declare class PRMergeHandler implements IPRMergeHandler {
     private readonly log;
     constructor(log: ILogger);
-    createAndMerge(repoInfo: RepoInfo, repoConfig: RepoConfig, options: PRHandlerOptions, changedFiles: FileAction[], repoName: string, diffStats?: DiffStats): Promise<ProcessorResult>;
+    createAndMerge(repoInfo: RepoInfo, repoConfig: RepoConfig, options: PRHandlerOptions, changedFiles: FileAction[], repoName: string, diffStats?: DiffStats, fileChanges?: FileChangeDetail[]): Promise<ProcessorResult>;
 }

package/dist/sync/pr-merge-handler.js

@@ -4,7 +4,7 @@ export class PRMergeHandler {
     constructor(log) {
         this.log = log;
     }
-    async createAndMerge(repoInfo, repoConfig, options, changedFiles, repoName, diffStats) {
+    async createAndMerge(repoInfo, repoConfig, options, changedFiles, repoName, diffStats, fileChanges) {
         this.log.info("Creating pull request...");
         const prResult = await createPR({
             repoInfo,
@@ -57,6 +57,7 @@ export class PRMergeHandler {
             prUrl: prResult.url,
             mergeResult,
             diffStats,
+            fileChanges,
         };
     }
 }

package/dist/sync/repository-processor.js

@@ -8,6 +8,14 @@ import { loadManifest, saveManifest, updateManifestRulesets, MANIFEST_FILENAME,
 import { GitHubAppTokenManager } from "../vcs/github-app-token-manager.js";
 import { formatCommitMessage } from "./commit-message.js";
 import { FileWriter, ManifestManager, BranchManager, AuthOptionsBuilder, RepositorySession, CommitPushManager, FileSyncOrchestrator, PRMergeHandler, } from "./index.js";
+function mapToFileChangeDetails(changedFiles) {
+    return changedFiles
+        .filter((f) => f.action !== "skip")
+        .map((f) => ({
+        path: f.fileName,
+        action: f.action,
+    }));
+}
 export class RepositoryProcessor {
     gitOpsFactory;
     log;
@@ -87,6 +95,8 @@ export class RepositoryProcessor {
             });
             // Process files and manifest
             const { fileChanges, diffStats, changedFiles, hasChanges } = await this.fileSyncOrchestrator.sync(repoConfig, repoInfo, session, options);
+            // Map file changes for reporting
+            const fileChangeDetails = mapToFileChangeDetails(changedFiles);
             if (!hasChanges) {
                 return {
                     success: true,
@@ -94,6 +104,7 @@ export class RepositoryProcessor {
                     message: "No changes detected",
                     skipped: true,
                     diffStats,
+                    fileChanges: fileChangeDetails,
                 };
             }
             // Commit and push
@@ -122,6 +133,7 @@ export class RepositoryProcessor {
                     message: "No changes detected after staging",
                     skipped: true,
                     diffStats,
+                    fileChanges: fileChangeDetails,
                 };
             }
             // Direct mode: no PR
@@ -132,6 +144,7 @@ export class RepositoryProcessor {
                     repoName,
                     message: `Pushed directly to ${session.baseBranch}`,
                     diffStats,
+                    fileChanges: fileChangeDetails,
                 };
             }
             // Create and merge PR
@@ -144,7 +157,7 @@ export class RepositoryProcessor {
                 prTemplate,
                 token: authResult.token,
                 executor,
-            }, changedFiles, repoName, diffStats);
+            }, changedFiles, repoName, diffStats, fileChangeDetails);
         }
         finally {
             try {
@@ -190,12 +203,17 @@ export class RepositoryProcessor {
                     skipped: true,
                 };
             }
+            // Prepare manifest file change details for reporting
+            const manifestFileChange = [
+                { path: MANIFEST_FILENAME, action: "update" },
+            ];
             if (dryRun) {
                 this.log.info(`Would update ${MANIFEST_FILENAME} with rulesets`);
                 return {
                     success: true,
                     repoName,
                     message: "Would update manifest (dry-run)",
+                    fileChanges: manifestFileChange,
                 };
             }
             // Setup branch and commit
@@ -246,6 +264,7 @@ export class RepositoryProcessor {
                     repoName,
                     message: "No changes detected after staging",
                     skipped: true,
+                    fileChanges: manifestFileChange,
                 };
             }
             if (isDirectMode) {
@@ -253,6 +272,7 @@ export class RepositoryProcessor {
                     success: true,
                     repoName,
                     message: `Manifest updated directly on ${session.baseBranch}`,
+                    fileChanges: manifestFileChange,
                 };
             }
             // Create and merge PR
@@ -264,7 +284,7 @@ export class RepositoryProcessor {
                 retries,
                 token: authResult.token,
                 executor,
-            }, [{ fileName: MANIFEST_FILENAME, action: "update" }], repoName);
+            }, [{ fileName: MANIFEST_FILENAME, action: "update" }], repoName, undefined, manifestFileChange);
         }
         finally {
             try {