feat(cli): skip missing sync files (#30)

Change-Id: Iaebd03ec16a7ef559025240041ec68cd751582c5

Author: heimanba

docs/reference/cli.md

@@ -73,6 +73,7 @@ Export a provider's remote configuration into a local `agents.yaml`.
 | `--provider <name>` | Source provider to sync from. |
 | `-o, --out <path>` | Output file (default `agents.synced.yaml`). |
 | `--force` | Overwrite the output file if it exists. |
+| `--skip-missing-files` | Do not prompt for remote files that cannot be downloaded; omit them from the synced output. |
 
 ## `agents migrate`
 

packages/cli/src/commands/sync.ts

@@ -27,7 +27,13 @@ export function ensureSyncOutputWritable(outPath: string, force?: boolean): void
 	}
 }
 
-export async function syncCommand(options: { file: string; provider?: string; out?: string; force?: boolean }) {
+export async function syncCommand(options: {
+	file: string;
+	provider?: string;
+	out?: string;
+	force?: boolean;
+	skipMissingFiles?: boolean;
+}) {
 	const outPath = options.out ?? DEFAULT_SYNC_OUTPUT;
 	ensureSyncOutputWritable(outPath, options.force);
 
@@ -38,8 +44,10 @@ export async function syncCommand(options: { file: string; provider?: string; ou
 
 	const baseDir = dirname(outPath);
 
-	// Interactive file path association (before writing yaml — skipped files are removed)
-	const removedFiles = await promptFileAssociation(result.config, baseDir);
+	// File path association (before writing yaml — skipped files are removed)
+	const removedFiles = options.skipMissingFiles
+		? removeMissingFileAssociations(result.config, baseDir)
+		: await promptFileAssociation(result.config, baseDir);
 	if (removedFiles.length > 0) {
 		const files = (result.config.files ?? {}) as Record<string, unknown>;
 		for (const key of removedFiles) {
@@ -84,6 +92,19 @@ export async function syncCommand(options: { file: string; provider?: string; ou
 	}
 }
 
+export function removeMissingFileAssociations(config: Record<string, unknown>, baseDir: string): string[] {
+	const files = (config.files ?? {}) as Record<string, Record<string, unknown>>;
+	const removed = Object.entries(files)
+		.filter(([, decl]) => !fileExistsSync(join(baseDir, decl.source as string)))
+		.map(([key]) => key);
+
+	if (removed.length > 0) {
+		log.info(`${removed.length} file(s) removed (skipped).`);
+	}
+
+	return removed;
+}
+
 async function syncFromConfig(
 	configPath: string,
 	explicitProvider?: string,

packages/cli/src/program.ts

@@ -145,6 +145,7 @@ program
 	.addOption(providerOption("Source provider to sync from (defaults from config when -f is set)"))
 	.option("-o, --out <path>", "Output file path", "agents.synced.yaml")
 	.option("--force", "Overwrite the output file if it already exists")
+	.option("--skip-missing-files", "Do not prompt for remote files that cannot be downloaded; omit them from output")
 	.action(withResolvedConfigFile(syncCommand));
 
 program

packages/cli/tests/unit/sync-file.test.ts

@@ -3,7 +3,7 @@ import { mkdtemp, rm } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { UserError } from "@openagentpack/sdk";
-import { syncCommand } from "../../src/commands/sync.ts";
+import { removeMissingFileAssociations, syncCommand } from "../../src/commands/sync.ts";
 
 const tempDirs: string[] = [];
 
@@ -30,3 +30,17 @@ test("syncCommand requires --provider when config file is missing", async () =>
 		"--provider when no config file exists",
 	);
 });
+
+test("removeMissingFileAssociations keeps existing sources and skips missing files", async () => {
+	const dir = await makeTempDir();
+	await Bun.write(join(dir, "present.txt"), "present\n");
+
+	const config = {
+		files: {
+			present: { source: "present.txt" },
+			missing: { source: "missing.txt" },
+		},
+	};
+
+	expect(removeMissingFileAssociations(config, dir)).toEqual(["missing"]);
+});