@yonpark/skillhub-cli 0.1.3 → 0.3.0

package/dist/commands/sync.js

@@ -1,31 +1,75 @@
 "use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.runSync = runSync;
-const node_child_process_1 = require("node:child_process");
-const node_util_1 = require("node:util");
-const promises_1 = __importDefault(require("node:fs/promises"));
-const node_path_1 = __importDefault(require("node:path"));
-const node_os_1 = __importDefault(require("node:os"));
+exports.runSyncMerge = runSyncMerge;
+exports.runSyncAuto = runSyncAuto;
+exports.runSyncPush = runSyncPush;
+exports.runSyncPull = runSyncPull;
+const syncCore_1 = require("../core/syncCore");
 const config_1 = require("../service/config");
 const gistService_1 = require("../service/gistService");
-const execAsync = (0, node_util_1.promisify)(node_child_process_1.exec);
-const SKILLS_LOCK_FILENAME = "skills-lock.json";
-const DEFAULT_STRATEGY = "union";
-const DEFAULT_SKILL_SOURCE_REPO = "vercel-labs/agent-skills";
-async function runSync(strategyInput) {
-    const strategy = parseStrategy(strategyInput);
+const skillsService_1 = require("../service/skillsService");
+const output_1 = require("../utils/output");
+function formatSyncSummary(summary) {
+    const prefix = summary.dryRun ? "Dry-run" : "Sync";
+    const failurePart = summary.failed.length > 0
+        ? ` (${summary.failed.length} failed - check logs or JSON output)`
+        : "";
+    const actionLine = summary.dryRun
+        ? `${prefix} ${summary.mode}: would upload ${summary.uploaded} change(s), would install ${summary.installPlanned} skill(s), would remove ${summary.removePlanned} skill(s)`
+        : `${prefix} ${summary.mode}: uploaded ${summary.uploaded} change(s), installed ${summary.installed} skill(s), removed ${summary.removed} skill(s)`;
+    const details = [
+        actionLine + failurePart,
+        `mode=${summary.mode}`,
+        `gistFound=${summary.gistFound}`,
+        `gistCreated=${summary.gistCreated}`,
+        `remoteNewer=${summary.remoteNewer === null ? "n/a" : String(summary.remoteNewer)}`,
+        `lastSyncAtUpdated=${summary.lastSyncAtUpdated}`,
+    ];
+    return details.join("\n");
+}
+function createSummary(params) {
+    return {
+        ok: params.failed.length === 0,
+        mode: params.mode,
+        dryRun: params.dryRun,
+        gistFound: params.gistFound,
+        gistCreated: params.gistCreated,
+        remoteNewer: params.remoteNewer,
+        uploaded: params.uploaded,
+        installPlanned: params.installPlanned,
+        installed: params.installed,
+        removePlanned: params.removePlanned,
+        removed: params.removed,
+        failed: params.failed,
+        lastSyncAtUpdated: params.lastSyncAtUpdated,
+    };
+}
+function finalizeWithFailures(summary, asJson) {
+    if (summary.failed.length === 0) {
+        return summary;
+    }
+    if (asJson) {
+        process.exitCode = 1;
+        return summary;
+    }
+    throw new Error(`Sync ${summary.mode} completed with ${summary.failed.length} failed operation(s). Check logs above.`);
+}
+async function ensureToken() {
     const token = await config_1.configStore.getToken();
     if (!token) {
-        throw new Error("You must login first. Run `skillhub login` and try again.");
+        throw new Error("You must login first. Run `skillhub auth login` and try again.");
     }
-    const localSkills = await getLocalSkills();
-    const localPayload = {
-        skills: uniqueSortedSkills(localSkills),
-        updatedAt: new Date().toISOString(),
-    };
+    return token;
+}
+async function safeGetPayload(octokit, gistId) {
+    try {
+        return await (0, gistService_1.getSkillhubPayload)(octokit, gistId);
+    }
+    catch {
+        return null;
+    }
+}
+async function resolveRemoteState(token) {
     const octokit = (0, gistService_1.createOctokit)(token);
     let gistId = await config_1.configStore.getGistId();
     let remotePayload = null;
@@ -38,292 +82,436 @@ async function runSync(strategyInput) {
     if (!gistId) {
         const found = await (0, gistService_1.findSkillhubGist)(octokit);
         if (found?.id) {
-            const foundId = found.id;
-            gistId = foundId;
-            await config_1.configStore.setGistId(foundId);
-            remotePayload = await safeGetPayload(octokit, foundId);
+            gistId = found.id;
+            await config_1.configStore.setGistId(found.id);
+            remotePayload = await safeGetPayload(octokit, found.id);
         }
     }
-    if (!gistId) {
-        const created = await (0, gistService_1.createSkillhubGist)(octokit, localPayload);
-        if (!created.id) {
-            throw new Error("Gist was created, but the ID could not be determined.");
-        }
-        await config_1.configStore.setGistId(created.id);
-        console.log("No existing SkillHub Gist found. A new one has been created.");
-        console.log(`Uploaded 1 change, installed 0 skills`);
-        return;
-    }
-    const resolvedRemote = remotePayload
-        ? {
-            ...remotePayload,
-            skills: normalizeSkills(remotePayload.skills ?? []),
-        }
-        : { skills: [], updatedAt: "" };
-    if (strategy === "latest") {
-        await applyLatestStrategy({
-            octokit,
-            gistId,
-            local: localPayload,
-            remote: resolvedRemote,
-        });
-        return;
-    }
-    await applyUnionStrategy({
+    return {
         octokit,
         gistId,
-        local: localPayload,
-        remote: resolvedRemote,
-    });
+        gistFound: Boolean(gistId),
+        remotePayload,
+    };
 }
-function parseStrategy(input) {
-    if (input === "latest" || input === "union") {
-        return input;
-    }
-    return DEFAULT_STRATEGY;
+function asPlanPayload(payload) {
+    return payload ?? { skills: [], updatedAt: "" };
 }
-async function getLocalSkills() {
-    // 1) Primary source: `skills list -g` output
-    // - generate-lock can be flaky (e.g. "already in lock file" without writing),
-    //   so we prefer parsing the list output when possible.
-    const listResult = await execAsync("npx skills list -g");
-    const listOutput = `${listResult.stdout ?? ""}\n${listResult.stderr ?? ""}`.trim();
-    // When there are no global skills, skills CLI prints:
-    // "No global skills found.\nTry listing project skills without -g"
-    // In that case we treat it as an empty list and avoid polluting Gist.
-    if (listOutput.includes("No global skills found") ||
-        listOutput.includes("Try listing project skills without -g")) {
-        return [];
+function splitInstallCandidates(candidates) {
+    const invalidInstallCandidates = candidates
+        .filter((skill) => !(0, skillsService_1.isValidSource)(skill.source))
+        .map((skill) => ({
+        skill,
+        reason: `Invalid source "${skill.source}". Expected owner/repo format.`,
+    }));
+    const validInstallCandidates = candidates.filter((skill) => (0, skillsService_1.isValidSource)(skill.source));
+    return {
+        invalidInstallCandidates,
+        validInstallCandidates,
+    };
+}
+async function confirmPullRemovalsIfNeeded(removeCandidates, options) {
+    if (removeCandidates.length === 0 || options.yes === true) {
+        return;
     }
-    const fromList = parseSkillsListOutput(listOutput);
-    if (fromList.length > 0) {
-        return fromList;
+    const { default: inquirer } = await import("inquirer");
+    const { confirm } = await inquirer.prompt([
+        {
+            type: "confirm",
+            name: "confirm",
+            default: false,
+            message: `Pull sync will remove ${removeCandidates.length} local skill(s). Continue?`,
+        },
+    ]);
+    if (!confirm) {
+        throw new Error("Sync pull cancelled.");
     }
-    // 2) Fallback: generate-lock + search for a skills-lock.json file
-    const { stdout, stderr } = await execAsync("npx skills generate-lock");
-    const output = `${stdout ?? ""}\n${stderr ?? ""}`.trim();
-    const candidatePaths = getCandidateSkillsLockPaths();
-    for (const lockPath of candidatePaths) {
-        const parsed = await tryReadSkillsLock(lockPath);
-        if (parsed) {
-            return parsed;
+}
+async function runSyncMerge(options = {}) {
+    const dryRun = options.dryRun === true;
+    const asJson = options.json === true;
+    const token = await ensureToken();
+    const nowIso = new Date().toISOString();
+    const localSkills = await (0, skillsService_1.getLocalSkills)();
+    const localPayload = {
+        skills: localSkills,
+        updatedAt: nowIso,
+    };
+    const { octokit, gistId, gistFound, remotePayload } = await resolveRemoteState(token);
+    if (!gistFound) {
+        if (dryRun) {
+            const summary = createSummary({
+                mode: "merge",
+                dryRun: true,
+                gistFound: false,
+                gistCreated: false,
+                remoteNewer: null,
+                uploaded: 1,
+                installPlanned: 0,
+                installed: 0,
+                removePlanned: 0,
+                removed: 0,
+                failed: [],
+                lastSyncAtUpdated: false,
+            });
+            (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+            return summary;
+        }
+        const created = await (0, gistService_1.createSkillhubGist)(octokit, localPayload);
+        if (!created.id) {
+            throw new Error("Gist was created, but the ID could not be determined.");
         }
+        await config_1.configStore.setGistId(created.id);
+        await config_1.configStore.setLastSyncAt(nowIso);
+        const summary = createSummary({
+            mode: "merge",
+            dryRun: false,
+            gistFound: false,
+            gistCreated: true,
+            remoteNewer: null,
+            uploaded: 1,
+            installPlanned: 0,
+            installed: 0,
+            removePlanned: 0,
+            removed: 0,
+            failed: [],
+            lastSyncAtUpdated: true,
+        });
+        (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+        return summary;
     }
-    if (output.includes("No installed skills found") || listOutput.includes("No project skills found")) {
-        return [];
+    const plan = (0, syncCore_1.buildMergePlan)({
+        localPayload,
+        remotePayload: asPlanPayload(remotePayload),
+        nowIso,
+    });
+    const { invalidInstallCandidates, validInstallCandidates } = splitInstallCandidates(plan.installCandidates);
+    if (dryRun) {
+        const summary = createSummary({
+            mode: "merge",
+            dryRun: true,
+            gistFound: true,
+            gistCreated: false,
+            remoteNewer: null,
+            uploaded: plan.uploadPayload ? 1 : 0,
+            installPlanned: plan.installCandidates.length,
+            installed: 0,
+            removePlanned: 0,
+            removed: 0,
+            failed: invalidInstallCandidates,
+            lastSyncAtUpdated: false,
+        });
+        (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+        return summary;
     }
-    throw new Error([
-        `Unable to construct local skills list.`,
-        `- skills list -g output:`,
-        listOutput,
-        ``,
-        `- Searched ${SKILLS_LOCK_FILENAME} paths:`,
-        ...candidatePaths.map((p) => `  - ${p}`),
-        ``,
-        `- npx skills generate-lock output:`,
-        output,
-    ].join("\n"));
-}
-function getCandidateSkillsLockPaths() {
-    const cwdPath = node_path_1.default.resolve(process.cwd(), SKILLS_LOCK_FILENAME);
-    const homePath = node_path_1.default.resolve(node_os_1.default.homedir(), SKILLS_LOCK_FILENAME);
-    // Cross-platform candidates when we don't know exactly where the tool writes
-    const homeConfigPaths = [
-        node_path_1.default.resolve(node_os_1.default.homedir(), ".config", "skills", SKILLS_LOCK_FILENAME),
-        node_path_1.default.resolve(node_os_1.default.homedir(), ".config", "skillhub", SKILLS_LOCK_FILENAME),
-        node_path_1.default.resolve(node_os_1.default.homedir(), ".skills", SKILLS_LOCK_FILENAME),
-    ];
-    // Windows-specific candidates
-    const winAppData = process.env.APPDATA;
-    const winLocalAppData = process.env.LOCALAPPDATA;
-    const windowsConfigPaths = [
-        ...(winAppData
-            ? [node_path_1.default.resolve(winAppData, "skills", SKILLS_LOCK_FILENAME)]
-            : []),
-        ...(winLocalAppData
-            ? [node_path_1.default.resolve(winLocalAppData, "skills", SKILLS_LOCK_FILENAME)]
-            : []),
+    const installResult = await (0, skillsService_1.installSkills)(validInstallCandidates, {
+        verbose: !asJson,
+    });
+    const failed = [
+        ...invalidInstallCandidates,
+        ...installResult.failed,
     ];
-    return [cwdPath, homePath, ...homeConfigPaths, ...windowsConfigPaths];
-}
-async function tryReadSkillsLock(lockPath) {
-    try {
-        const raw = await promises_1.default.readFile(lockPath, "utf-8");
-        return parseSkillsLock(raw);
+    if (plan.uploadPayload) {
+        await (0, gistService_1.updateSkillhubGist)(octokit, gistId, plan.uploadPayload);
     }
-    catch {
-        return null;
+    const summary = createSummary({
+        mode: "merge",
+        dryRun: false,
+        gistFound: true,
+        gistCreated: false,
+        remoteNewer: null,
+        uploaded: plan.uploadPayload ? 1 : 0,
+        installPlanned: plan.installCandidates.length,
+        installed: installResult.succeeded.length,
+        removePlanned: 0,
+        removed: 0,
+        failed,
+        lastSyncAtUpdated: false,
+    });
+    if (failed.length === 0) {
+        await config_1.configStore.setLastSyncAt(nowIso);
+        summary.lastSyncAtUpdated = true;
     }
+    (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+    return finalizeWithFailures(summary, asJson);
 }
-function parseSkillsLock(raw) {
-    const parsed = JSON.parse(raw);
-    // skills-lock.json에서 source 정보 추출 시도
-    const extractSkills = (items) => {
-        return items.map((item) => {
-            if (typeof item === "string") {
-                return { name: item, source: DEFAULT_SKILL_SOURCE_REPO };
-            }
-            if (typeof item === "object" && item !== null) {
-                const name = item.name || item.skill || String(item);
-                const source = item.source || item.repo || DEFAULT_SKILL_SOURCE_REPO;
-                return { name: String(name), source: String(source) };
-            }
-            return { name: String(item), source: DEFAULT_SKILL_SOURCE_REPO };
-        });
+async function runSyncAuto(options = {}) {
+    const dryRun = options.dryRun === true;
+    const asJson = options.json === true;
+    const token = await ensureToken();
+    const nowIso = new Date().toISOString();
+    const localSkills = await (0, skillsService_1.getLocalSkills)();
+    const localPayload = {
+        skills: localSkills,
+        updatedAt: nowIso,
     };
-    const fromSkills = parsed?.skills;
-    if (Array.isArray(fromSkills)) {
-        return extractSkills(fromSkills);
-    }
-    if (Array.isArray(parsed)) {
-        return extractSkills(parsed);
-    }
-    const fromInstalled = parsed?.installedSkills;
-    if (Array.isArray(fromInstalled)) {
-        return extractSkills(fromInstalled);
+    const { octokit, gistId, gistFound, remotePayload } = await resolveRemoteState(token);
+    if (!gistFound) {
+        if (dryRun) {
+            const summary = createSummary({
+                mode: "auto",
+                dryRun: true,
+                gistFound: false,
+                gistCreated: false,
+                remoteNewer: null,
+                uploaded: 1,
+                installPlanned: 0,
+                installed: 0,
+                removePlanned: 0,
+                removed: 0,
+                failed: [],
+                lastSyncAtUpdated: false,
+            });
+            (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+            return summary;
+        }
+        const created = await (0, gistService_1.createSkillhubGist)(octokit, localPayload);
+        if (!created.id) {
+            throw new Error("Gist was created, but the ID could not be determined.");
+        }
+        await config_1.configStore.setGistId(created.id);
+        await config_1.configStore.setLastSyncAt(nowIso);
+        const summary = createSummary({
+            mode: "auto",
+            dryRun: false,
+            gistFound: false,
+            gistCreated: true,
+            remoteNewer: null,
+            uploaded: 1,
+            installPlanned: 0,
+            installed: 0,
+            removePlanned: 0,
+            removed: 0,
+            failed: [],
+            lastSyncAtUpdated: true,
+        });
+        (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+        return summary;
     }
-    return [];
-}
-function parseSkillsListOutput(output) {
-    const cleaned = output.replace(/\x1b\[[0-9;]*m/g, "");
-    const lines = cleaned.split(/\r?\n/).map((line) => line.trim());
-    const skills = [];
-    for (const line of lines) {
-        if (!line)
-            continue;
-        if (line === "Global Skills")
-            continue;
-        if (line.startsWith("Agents:"))
-            continue;
-        if (line.startsWith("No project skills found"))
-            continue;
-        if (line.startsWith("Try listing global skills"))
-            continue;
-        if (line.startsWith("No global skills found"))
-            continue;
-        if (line.startsWith("Try listing project skills without -g"))
-            continue;
-        // Example: "find-skills ~\\.agents\\skills\\find-skills"
-        // We only take the first token as the skill name
-        const [name] = line.split(/\s+/);
-        if (!name)
-            continue;
-        // Skip obvious path-like tokens as a safety net
-        if (name.includes("\\") || name.includes("/") || name.includes("~"))
-            continue;
-        // skills list 출력에는 source 정보가 없으므로 기본값 사용
-        skills.push({ name, source: DEFAULT_SKILL_SOURCE_REPO });
+    const lastSyncAt = await config_1.configStore.getLastSyncAt();
+    const plan = (0, syncCore_1.buildAutoPlan)({
+        localPayload,
+        remotePayload: asPlanPayload(remotePayload),
+        lastSyncAt,
+        nowIso,
+    });
+    const { invalidInstallCandidates, validInstallCandidates } = splitInstallCandidates(plan.installCandidates);
+    if (dryRun) {
+        const summary = createSummary({
+            mode: "auto",
+            dryRun: true,
+            gistFound: true,
+            gistCreated: false,
+            remoteNewer: plan.isRemoteNewer,
+            uploaded: plan.uploadPayload ? 1 : 0,
+            installPlanned: plan.installCandidates.length,
+            installed: 0,
+            removePlanned: 0,
+            removed: 0,
+            failed: invalidInstallCandidates,
+            lastSyncAtUpdated: false,
+        });
+        (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+        return summary;
     }
-    return uniqueSortedSkills(skills);
-}
-function normalizeSkills(skills) {
-    const bannedSubstrings = [
-        "No global skills found",
-        "Try listing project skills without -g",
-        "No project skills found",
-        "Try listing global skills",
+    const installResult = await (0, skillsService_1.installSkills)(validInstallCandidates, {
+        verbose: !asJson,
+    });
+    const failed = [
+        ...invalidInstallCandidates,
+        ...installResult.failed,
     ];
-    const normalized = skills
-        .map((skill) => {
-        if (typeof skill === "string") {
-            return { name: skill, source: DEFAULT_SKILL_SOURCE_REPO };
-        }
-        return skill;
-    })
-        .filter((skill) => !!skill.name &&
-        !bannedSubstrings.some((bad) => skill.name.includes(bad)));
-    return uniqueSortedSkills(normalized);
-}
-async function safeGetPayload(octokit, gistId) {
-    try {
-        return await (0, gistService_1.getSkillhubPayload)(octokit, gistId);
+    if (plan.uploadPayload) {
+        await (0, gistService_1.updateSkillhubGist)(octokit, gistId, plan.uploadPayload);
     }
-    catch {
-        return null;
+    const summary = createSummary({
+        mode: "auto",
+        dryRun: false,
+        gistFound: true,
+        gistCreated: false,
+        remoteNewer: plan.isRemoteNewer,
+        uploaded: plan.uploadPayload ? 1 : 0,
+        installPlanned: plan.installCandidates.length,
+        installed: installResult.succeeded.length,
+        removePlanned: 0,
+        removed: 0,
+        failed,
+        lastSyncAtUpdated: false,
+    });
+    if (failed.length === 0) {
+        await config_1.configStore.setLastSyncAt(nowIso);
+        summary.lastSyncAtUpdated = true;
     }
+    (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+    return finalizeWithFailures(summary, asJson);
 }
-function uniqueSortedSkills(skills) {
-    const seen = new Set();
-    const result = [];
-    for (const skill of skills) {
-        const key = `${skill.source}:${skill.name}`;
-        if (!seen.has(key)) {
-            seen.add(key);
-            result.push(skill);
+async function runSyncPush(options = {}) {
+    const dryRun = options.dryRun === true;
+    const asJson = options.json === true;
+    const token = await ensureToken();
+    const nowIso = new Date().toISOString();
+    const localSkills = await (0, skillsService_1.getLocalSkills)();
+    const localPayload = {
+        skills: localSkills,
+        updatedAt: nowIso,
+    };
+    const { octokit, gistId, gistFound, remotePayload } = await resolveRemoteState(token);
+    if (!gistFound) {
+        if (dryRun) {
+            const summary = createSummary({
+                mode: "push",
+                dryRun: true,
+                gistFound: false,
+                gistCreated: false,
+                remoteNewer: null,
+                uploaded: 1,
+                installPlanned: 0,
+                installed: 0,
+                removePlanned: 0,
+                removed: 0,
+                failed: [],
+                lastSyncAtUpdated: false,
+            });
+            (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+            return summary;
         }
-    }
-    return result.sort((a, b) => {
-        if (a.source !== b.source) {
-            return a.source.localeCompare(b.source);
+        const created = await (0, gistService_1.createSkillhubGist)(octokit, localPayload);
+        if (!created.id) {
+            throw new Error("Gist was created, but the ID could not be determined.");
         }
-        return a.name.localeCompare(b.name);
+        await config_1.configStore.setGistId(created.id);
+        await config_1.configStore.setLastSyncAt(nowIso);
+        const summary = createSummary({
+            mode: "push",
+            dryRun: false,
+            gistFound: false,
+            gistCreated: true,
+            remoteNewer: null,
+            uploaded: 1,
+            installPlanned: 0,
+            installed: 0,
+            removePlanned: 0,
+            removed: 0,
+            failed: [],
+            lastSyncAtUpdated: true,
+        });
+        (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+        return summary;
+    }
+    const plan = (0, syncCore_1.buildPushPlan)({
+        localPayload,
+        remotePayload: asPlanPayload(remotePayload),
+        nowIso,
     });
-}
-async function installSkills(skills) {
-    const succeeded = [];
-    const failed = [];
-    for (const skill of skills) {
-        try {
-            const { stdout, stderr } = await execAsync(`npx skills add ${skill.source} --skill "${skill.name}" --global --yes`);
-            const output = `${stdout ?? ""}\n${stderr ?? ""}`.trim();
-            if (output) {
-                console.log(output);
-            }
-            succeeded.push(skill);
-        }
-        catch (error) {
-            const stdout = error?.stdout ?? "";
-            const stderr = error?.stderr ?? "";
-            const reason = `${stdout}\n${stderr}`.trim() || String(error);
-            failed.push({ skill, reason });
-            console.warn([
-                `스킬 설치 실패: ${skill.name} (from ${skill.source})`,
-                reason && `  └─ ${reason}`,
-            ]
-                .filter(Boolean)
-                .join("\n"));
-        }
+    if (dryRun) {
+        const summary = createSummary({
+            mode: "push",
+            dryRun: true,
+            gistFound: true,
+            gistCreated: false,
+            remoteNewer: null,
+            uploaded: plan.uploadPayload ? 1 : 0,
+            installPlanned: 0,
+            installed: 0,
+            removePlanned: 0,
+            removed: 0,
+            failed: [],
+            lastSyncAtUpdated: false,
+        });
+        (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+        return summary;
+    }
+    if (plan.uploadPayload) {
+        await (0, gistService_1.updateSkillhubGist)(octokit, gistId, plan.uploadPayload);
     }
-    return { succeeded, failed };
+    await config_1.configStore.setLastSyncAt(nowIso);
+    const summary = createSummary({
+        mode: "push",
+        dryRun: false,
+        gistFound: true,
+        gistCreated: false,
+        remoteNewer: null,
+        uploaded: plan.uploadPayload ? 1 : 0,
+        installPlanned: 0,
+        installed: 0,
+        removePlanned: 0,
+        removed: 0,
+        failed: [],
+        lastSyncAtUpdated: true,
+    });
+    (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+    return summary;
 }
-async function applyUnionStrategy(params) {
-    const localSkills = normalizeSkills(params.local.skills);
-    const remoteSkills = normalizeSkills(params.remote.skills);
-    const unionSkills = uniqueSortedSkills([...localSkills, ...remoteSkills]);
-    const missingLocally = unionSkills.filter((skill) => !localSkills.some((local) => local.name === skill.name && local.source === skill.source));
-    const payload = {
-        skills: unionSkills,
-        updatedAt: new Date().toISOString(),
+async function runSyncPull(options = {}) {
+    const dryRun = options.dryRun === true;
+    const asJson = options.json === true;
+    const token = await ensureToken();
+    const nowIso = new Date().toISOString();
+    const localSkills = await (0, skillsService_1.getLocalSkills)();
+    const localPayload = {
+        skills: localSkills,
+        updatedAt: nowIso,
     };
-    const { succeeded, failed } = await installSkills(missingLocally);
-    await (0, gistService_1.updateSkillhubGist)(params.octokit, params.gistId, payload);
-    const uploadCount = areSameSkills(remoteSkills, unionSkills) ? 0 : 1;
-    const installCount = succeeded.length;
-    const failedCount = failed.length;
-    console.log(`업로드 ${uploadCount}건, 설치 ${installCount}건${failedCount ? ` (실패 ${failedCount}건은 로그 참고)` : ""}`);
-}
-async function applyLatestStrategy(params) {
-    const localTime = Date.parse(params.local.updatedAt);
-    const remoteTime = Date.parse(params.remote.updatedAt);
-    const isRemoteNewer = Number.isFinite(remoteTime) && remoteTime > localTime;
-    if (isRemoteNewer) {
-        const localSkills = normalizeSkills(params.local.skills);
-        const remoteSkills = normalizeSkills(params.remote.skills);
-        const missingLocally = remoteSkills.filter((skill) => !localSkills.some((local) => local.name === skill.name && local.source === skill.source));
-        const { succeeded, failed } = await installSkills(missingLocally);
-        const failedCount = failed.length;
-        console.log(`업로드 0건, 설치 ${succeeded.length}건${failedCount ? ` (실패 ${failedCount}건은 로그 참고)` : ""}`);
-        return;
+    const { gistFound, remotePayload } = await resolveRemoteState(token);
+    if (!gistFound) {
+        throw new Error("Remote SkillHub Gist not found. Run `skillhub sync push` to create it first.");
     }
-    await (0, gistService_1.updateSkillhubGist)(params.octokit, params.gistId, params.local);
-    console.log("업로드 1건, 설치 0건");
-}
-function areSameSkills(left, right) {
-    const leftSorted = uniqueSortedSkills(left);
-    const rightSorted = uniqueSortedSkills(right);
-    if (leftSorted.length !== rightSorted.length) {
-        return false;
+    if (!remotePayload) {
+        throw new Error("Remote SkillHub payload is missing or invalid. Fix the remote `skillhub.json` and retry.");
+    }
+    const plan = (0, syncCore_1.buildPullPlan)({
+        localPayload,
+        remotePayload,
+    });
+    const { invalidInstallCandidates, validInstallCandidates } = splitInstallCandidates(plan.installCandidates);
+    if (dryRun) {
+        const summary = createSummary({
+            mode: "pull",
+            dryRun: true,
+            gistFound: true,
+            gistCreated: false,
+            remoteNewer: null,
+            uploaded: 0,
+            installPlanned: plan.installCandidates.length,
+            installed: 0,
+            removePlanned: plan.removeCandidates.length,
+            removed: 0,
+            failed: invalidInstallCandidates,
+            lastSyncAtUpdated: false,
+        });
+        (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+        return summary;
+    }
+    await confirmPullRemovalsIfNeeded(plan.removeCandidates, options);
+    const installResult = await (0, skillsService_1.installSkills)(validInstallCandidates, {
+        verbose: !asJson,
+    });
+    const removeResult = await (0, skillsService_1.removeSkills)(plan.removeCandidates, {
+        verbose: !asJson,
+    });
+    const failed = [
+        ...invalidInstallCandidates,
+        ...installResult.failed,
+        ...removeResult.failed,
+    ];
+    const summary = createSummary({
+        mode: "pull",
+        dryRun: false,
+        gistFound: true,
+        gistCreated: false,
+        remoteNewer: null,
+        uploaded: 0,
+        installPlanned: plan.installCandidates.length,
+        installed: installResult.succeeded.length,
+        removePlanned: plan.removeCandidates.length,
+        removed: removeResult.succeeded.length,
+        failed,
+        lastSyncAtUpdated: false,
+    });
+    if (failed.length === 0) {
+        await config_1.configStore.setLastSyncAt(nowIso);
+        summary.lastSyncAtUpdated = true;
     }
-    return leftSorted.every((skill, index) => skill.name === rightSorted[index].name &&
-        skill.source === rightSorted[index].source);
+    (0, output_1.emitOutput)(summary, asJson, formatSyncSummary);
+    return finalizeWithFailures(summary, asJson);
 }