yaaw-se 0.3.11 → 0.3.13

package/dist/installer/plan.js

@@ -17,85 +17,43 @@ export class ManagedConflictError extends Error {
         this.name = "ManagedConflictError";
     }
 }
-async function exists(path) {
-    try {
-        await access(path);
-        return true;
-    }
-    catch {
-        return false;
-    }
+async function exists(path) { try {
+    await access(path);
+    return true;
 }
-async function walk(dir) {
-    const entries = await readdir(dir, { withFileTypes: true });
-    const out = [];
-    for (const entry of entries) {
-        const full = join(dir, entry.name);
-        if (entry.isDirectory())
-            out.push(...await walk(full));
-        else
-            out.push(full);
-    }
-    return out;
-}
-async function canonicalSkills(payloadRoot) {
-    const registry = JSON.parse(await readFile(join(payloadRoot, "yaaw-core", "system", "registries", "skills.json"), "utf8"));
-    return Object.keys(registry).sort().map(id => ({ id, source: join(payloadRoot, "skills", id, "SKILL.md") }));
-}
-export async function resolveSkillSelection(payloadRoot, request) {
-    const all = (await canonicalSkills(payloadRoot)).map(x => x.id);
-    const core = ["yaaw-orchestrator", "yaaw-prd", "yaaw-planner", "yaaw-implement", "yaaw-review"];
-    if (Array.isArray(request)) {
-        const unknown = request.filter(x => !all.includes(x));
-        if (unknown.length)
-            throw new Error(`Unknown skills: ${unknown.join(", ")}`);
-        return [...new Set(request)];
-    }
-    if (request === "standard")
-        return all;
-    if (request === "core")
-        return core;
-    const requested = request.split(",").map(x => x.trim()).filter(Boolean);
-    const unknown = requested.filter(x => !all.includes(x));
+catch {
+    return false;
+} }
+async function walk(dir) { const out = []; for (const entry of await readdir(dir, { withFileTypes: true })) {
+    const full = join(dir, entry.name);
+    if (entry.isDirectory())
+        out.push(...await walk(full));
+    else
+        out.push(full);
+} return out; }
+async function canonicalSkills(payloadRoot) { const kernel = JSON.parse(await readFile(join(payloadRoot, "yaaw-core", "system", "kernel.yaml"), "utf8")); return Object.keys(kernel.skills ?? {}).sort().map(id => ({ id, source: join(payloadRoot, "skills", id, "SKILL.md") })); }
+export async function resolveSkillSelection(payloadRoot, request) { const all = (await canonicalSkills(payloadRoot)).map(x => x.id), core = ["yaaw-orchestrator", "yaaw-prd", "yaaw-planner", "yaaw-implement", "yaaw-review"]; if (Array.isArray(request)) {
+    const unknown = request.filter(x => !all.includes(x));
     if (unknown.length)
         throw new Error(`Unknown skills: ${unknown.join(", ")}`);
-    return [...new Set(requested)];
-}
-function rel(projectRoot, path) {
-    return relative(projectRoot, path).replaceAll("\\", "/");
-}
-function queueEmptyParentCleanup(operations, projectRoot, path, owner) {
-    const protectedDirs = new Set([
-        projectRoot,
-        join(projectRoot, ".yaaw-core"),
-        join(projectRoot, ".yaaw-core", "project"),
-        join(projectRoot, ".yaaw-core", "install")
-    ]);
-    let current = dirname(path);
-    while (current !== projectRoot && !protectedDirs.has(current)) {
-        // Cleanup is intentionally repeatable. Earlier attempts may see sibling
-        // managed files that are removed later in the same transaction; the final
-        // attempt then removes the now-empty adapter/framework directory.
-        operations.push({ type: "remove-empty-dir", path: current, owner });
-        current = dirname(current);
-    }
-}
-async function currentFileHash(path) {
-    try {
-        return sha256Bytes(await readFile(path));
-    }
-    catch {
-        return null;
-    }
+    return [...new Set(request)];
+} if (request === "standard")
+    return all; if (request === "core")
+    return core; const requested = request.split(",").map(x => x.trim()).filter(Boolean), unknown = requested.filter(x => !all.includes(x)); if (unknown.length)
+    throw new Error(`Unknown skills: ${unknown.join(", ")}`); return [...new Set(requested)]; }
+function rel(projectRoot, path) { return relative(projectRoot, path).replaceAll("\\", "/"); }
+function queueEmptyParentCleanup(operations, projectRoot, path, owner) { const protectedDirs = new Set([projectRoot, join(projectRoot, ".yaaw-core"), join(projectRoot, ".yaaw-core", "project"), join(projectRoot, ".yaaw-core", "install")]); let current = dirname(path); while (current !== projectRoot && !protectedDirs.has(current)) {
+    operations.push({ type: "remove-empty-dir", path: current, owner });
+    current = dirname(current);
+} }
+async function currentFileHash(path) { try {
+    return sha256Bytes(await readFile(path));
 }
+catch {
+    return null;
+} }
 export async function chooseManagedFile(params) {
-    const { ctx, previous, path, owner, operations, records, conflicts, backupStamp } = params;
-    const pathRel = rel(ctx.projectRoot, path);
-    const expected = params.source ? await readFile(params.source) : Buffer.from(params.content ?? "");
-    const expectedHash = sha256Bytes(expected);
-    const diskHash = await currentFileHash(path);
-    const prior = previous?.managedFiles?.[pathRel];
-    const hasConflict = diskHash !== null && (prior ? (prior.localOverride === true || diskHash !== prior.sha256) : diskHash !== expectedHash);
+    const { ctx, previous, path, owner, operations, records, conflicts, backupStamp } = params, pathRel = rel(ctx.projectRoot, path), expected = params.source ? await readFile(params.source) : Buffer.from(params.content ?? ""), expectedHash = sha256Bytes(expected), diskHash = await currentFileHash(path), prior = previous?.managedFiles?.[pathRel], hasConflict = diskHash !== null && (prior ? (prior.localOverride === true || diskHash !== prior.sha256) : diskHash !== expectedHash);
     if (hasConflict && ctx.conflictPolicy === "fail") {
         conflicts.push(pathRel);
         return;
@@ -116,18 +74,13 @@ export async function chooseManagedFile(params) {
     records[pathRel] = { owner, sha256: expectedHash };
 }
 export async function chooseManagedSection(params) {
-    const { ctx, previous, op, sectionRecords, operations, conflicts, backupStamp } = params;
-    const pathRel = rel(ctx.projectRoot, op.path);
+    const { ctx, previous, op, sectionRecords, operations, conflicts, backupStamp } = params, pathRel = rel(ctx.projectRoot, op.path);
     let original = "";
     try {
         original = await readFile(op.path, "utf8");
     }
     catch { }
-    const existing = extractManagedSection(original, op.sectionId);
-    const expectedHash = managedSectionHash(op.content);
-    const prior = previous?.managedSections?.[pathRel]?.[op.sectionId];
-    const existingHash = existing === null ? null : managedSectionHash(existing);
-    const conflict = existing !== null && (prior ? (prior.localOverride === true || existingHash !== prior.sha256) : existingHash !== expectedHash);
+    const existing = extractManagedSection(original, op.sectionId), expectedHash = managedSectionHash(op.content), prior = previous?.managedSections?.[pathRel]?.[op.sectionId], existingHash = existing === null ? null : managedSectionHash(existing), conflict = existing !== null && (prior ? (prior.localOverride === true || existingHash !== prior.sha256) : existingHash !== expectedHash);
     if (conflict && ctx.conflictPolicy === "fail") {
         conflicts.push(`${pathRel}#${op.sectionId}`);
         return;
@@ -138,16 +91,14 @@ export async function chooseManagedSection(params) {
         return;
     }
     if (conflict && ctx.conflictPolicy === "backup-replace") {
-        const backupPath = join(ctx.projectRoot, ".yaaw-core", "install", "backups", backupStamp, pathRel);
-        operations.push({ type: "write-managed-file", path: backupPath, content: original, owner: "installer:backup" });
+        operations.push({ type: "write-managed-file", path: join(ctx.projectRoot, ".yaaw-core", "install", "backups", backupStamp, pathRel), content: original, owner: "installer:backup" });
     }
     operations.push(op);
     sectionRecords[pathRel] ??= {};
     sectionRecords[pathRel][op.sectionId] = { owner: op.owner, sha256: expectedHash };
 }
 export async function chooseManagedConfigKeys(params) {
-    const { ctx, previous, op, configRecords, operations, conflicts, backupStamp } = params;
-    const pathRel = rel(ctx.projectRoot, op.path);
+    const { ctx, previous, op, configRecords, operations, conflicts, backupStamp } = params, pathRel = rel(ctx.projectRoot, op.path);
     let original = "";
     try {
         original = await readFile(op.path, "utf8");
@@ -158,11 +109,7 @@ export async function chooseManagedConfigKeys(params) {
     const ownedEntries = [];
     let backupQueued = false;
     for (const entry of op.entries) {
-        const prior = previous?.managedConfigKeys?.[pathRel]?.[entry.key];
-        const current = readTomlManagedValue(original, entry.key);
-        const expectedHash = semanticConfigHash(entry.value);
-        const currentHash = current === undefined ? null : semanticConfigHash(current);
-        const namespacedRole = entry.key.startsWith("agents.yaaw_");
+        const prior = previous?.managedConfigKeys?.[pathRel]?.[entry.key], current = readTomlManagedValue(original, entry.key), expectedHash = semanticConfigHash(entry.value), currentHash = current === undefined ? null : semanticConfigHash(current), namespacedRole = entry.key.startsWith("agents.yaaw_");
         if (!prior) {
             if (current === undefined) {
                 ownedEntries.push(entry);
@@ -170,17 +117,14 @@ export async function chooseManagedConfigKeys(params) {
                 configRecords[pathRel][entry.key] = { owner: op.owner, sha256: expectedHash, value: entry.value };
                 continue;
             }
-            if (!namespacedRole && currentHash === expectedHash) {
-                // Compatible user-owned value satisfies the request. Do not claim ownership.
+            if (!namespacedRole && currentHash === expectedHash)
                 continue;
-            }
             if (ctx.conflictPolicy === "fail" || ctx.conflictPolicy === "keep") {
                 conflicts.push(`${pathRel}#${entry.key}`);
                 continue;
             }
             if (ctx.conflictPolicy === "backup-replace" && !backupQueued) {
-                const backupPath = join(ctx.projectRoot, ".yaaw-core", "install", "backups", backupStamp, pathRel);
-                operations.push({ type: "write-managed-file", path: backupPath, content: original, owner: "installer:backup" });
+                operations.push({ type: "write-managed-file", path: join(ctx.projectRoot, ".yaaw-core", "install", "backups", backupStamp, pathRel), content: original, owner: "installer:backup" });
                 backupQueued = true;
             }
             ownedEntries.push(entry);
@@ -196,19 +140,12 @@ export async function chooseManagedConfigKeys(params) {
         if (modified && ctx.conflictPolicy === "keep") {
             if (current !== undefined) {
                 configRecords[pathRel] ??= {};
-                configRecords[pathRel][entry.key] = {
-                    owner: op.owner,
-                    sha256: currentHash,
-                    value: current,
-                    packageSha256: expectedHash,
-                    localOverride: true
-                };
+                configRecords[pathRel][entry.key] = { owner: op.owner, sha256: currentHash, value: current, packageSha256: expectedHash, localOverride: true };
             }
             continue;
         }
         if (modified && ctx.conflictPolicy === "backup-replace" && !backupQueued) {
-            const backupPath = join(ctx.projectRoot, ".yaaw-core", "install", "backups", backupStamp, pathRel);
-            operations.push({ type: "write-managed-file", path: backupPath, content: original, owner: "installer:backup" });
+            operations.push({ type: "write-managed-file", path: join(ctx.projectRoot, ".yaaw-core", "install", "backups", backupStamp, pathRel), content: original, owner: "installer:backup" });
             backupQueued = true;
         }
         ownedEntries.push(entry);
@@ -219,8 +156,7 @@ export async function chooseManagedConfigKeys(params) {
         operations.push({ ...op, entries: ownedEntries });
 }
 export async function protectConfigRemoval(params) {
-    const { ctx, pathRel, records, operations, conflicts, backupStamp } = params;
-    const path = join(ctx.projectRoot, pathRel);
+    const { ctx, pathRel, records, operations, conflicts, backupStamp } = params, path = join(ctx.projectRoot, pathRel);
     let original = "";
     try {
         original = await readFile(path, "utf8");
@@ -235,8 +171,7 @@ export async function protectConfigRemoval(params) {
         const current = readTomlManagedValue(original, key);
         if (current === undefined)
             continue;
-        const currentHash = semanticConfigHash(current);
-        const modified = record.localOverride === true || currentHash !== record.sha256;
+        const currentHash = semanticConfigHash(current), modified = record.localOverride === true || currentHash !== record.sha256;
         if (modified && ctx.conflictPolicy === "fail") {
             conflicts.push(`${pathRel}#${key}`);
             continue;
@@ -244,8 +179,7 @@ export async function protectConfigRemoval(params) {
         if (modified && ctx.conflictPolicy === "keep")
             continue;
         if (modified && ctx.conflictPolicy === "backup-replace" && !backupQueued) {
-            const backupPath = join(ctx.projectRoot, ".yaaw-core", "install", "backups", backupStamp, pathRel);
-            operations.push({ type: "write-managed-file", path: backupPath, content: original, owner: "installer:backup" });
+            operations.push({ type: "write-managed-file", path: join(ctx.projectRoot, ".yaaw-core", "install", "backups", backupStamp, pathRel), content: original, owner: "installer:backup" });
             backupQueued = true;
         }
         removable.push(key);
@@ -255,62 +189,32 @@ export async function protectConfigRemoval(params) {
         queueEmptyParentCleanup(operations, ctx.projectRoot, path, "integration:codex");
     }
 }
-async function protectRemoval(params) {
-    const { ctx, previous, pathRel, record, operations, conflicts, backupStamp } = params;
-    const path = join(ctx.projectRoot, pathRel);
-    const diskHash = await currentFileHash(path);
-    if (diskHash === null)
-        return;
-    const modified = record.localOverride === true || diskHash !== record.sha256;
-    if (modified && ctx.conflictPolicy === "fail") {
-        conflicts.push(pathRel);
-        return;
-    }
-    if (modified && ctx.conflictPolicy === "keep") {
-        operations.push({ type: "preserve", path, reason: "local managed-file override retained during removal" });
-        return;
-    }
-    if (modified && ctx.conflictPolicy === "backup-replace") {
-        const backupPath = join(ctx.projectRoot, ".yaaw-core", "install", "backups", backupStamp, pathRel);
-        operations.push({ type: "write-managed-file", path: backupPath, content: await readFile(path), owner: "installer:backup" });
-    }
-    operations.push({ type: "remove-managed-file", path, owner: record.owner });
-    queueEmptyParentCleanup(operations, ctx.projectRoot, path, record.owner);
-}
-async function protectSectionRemoval(params) {
-    const { ctx, pathRel, sectionId, record, operations, conflicts, backupStamp } = params;
-    const path = join(ctx.projectRoot, pathRel);
-    let original = "";
-    try {
-        original = await readFile(path, "utf8");
-    }
-    catch {
-        return;
-    }
-    const existing = extractManagedSection(original, sectionId);
-    if (existing === null)
-        return;
-    const modified = record.localOverride === true || managedSectionHash(existing) !== record.sha256;
-    if (modified && ctx.conflictPolicy === "fail") {
-        conflicts.push(`${pathRel}#${sectionId}`);
-        return;
-    }
-    if (modified && ctx.conflictPolicy === "keep") {
-        operations.push({ type: "preserve", path, reason: "local managed-section override retained during removal" });
-        return;
-    }
-    if (modified && ctx.conflictPolicy === "backup-replace") {
-        const backupPath = join(ctx.projectRoot, ".yaaw-core", "install", "backups", backupStamp, pathRel);
-        operations.push({ type: "write-managed-file", path: backupPath, content: original, owner: "installer:backup" });
-    }
-    operations.push({ type: "remove-managed-section", path, sectionId, owner: record.owner });
+async function protectRemoval(params) { const { ctx, pathRel, record, operations, conflicts, backupStamp } = params, path = join(ctx.projectRoot, pathRel), diskHash = await currentFileHash(path); if (diskHash === null)
+    return; const modified = record.localOverride === true || diskHash !== record.sha256; if (modified && ctx.conflictPolicy === "fail") {
+    conflicts.push(pathRel);
+    return;
+} if (modified && ctx.conflictPolicy === "keep") {
+    operations.push({ type: "preserve", path, reason: "local managed-file override retained during removal" });
+    return;
+} if (modified && ctx.conflictPolicy === "backup-replace")
+    operations.push({ type: "write-managed-file", path: join(ctx.projectRoot, ".yaaw-core", "install", "backups", backupStamp, pathRel), content: await readFile(path), owner: "installer:backup" }); operations.push({ type: "remove-managed-file", path, owner: record.owner }); queueEmptyParentCleanup(operations, ctx.projectRoot, path, record.owner); }
+async function protectSectionRemoval(params) { const { ctx, pathRel, sectionId, record, operations, conflicts, backupStamp } = params, path = join(ctx.projectRoot, pathRel); let original = ""; try {
+    original = await readFile(path, "utf8");
 }
+catch {
+    return;
+} const existing = extractManagedSection(original, sectionId); if (existing === null)
+    return; const modified = record.localOverride === true || managedSectionHash(existing) !== record.sha256; if (modified && ctx.conflictPolicy === "fail") {
+    conflicts.push(`${pathRel}#${sectionId}`);
+    return;
+} if (modified && ctx.conflictPolicy === "keep") {
+    operations.push({ type: "preserve", path, reason: "local managed-section override retained during removal" });
+    return;
+} if (modified && ctx.conflictPolicy === "backup-replace")
+    operations.push({ type: "write-managed-file", path: join(ctx.projectRoot, ".yaaw-core", "install", "backups", backupStamp, pathRel), content: original, owner: "installer:backup" }); operations.push({ type: "remove-managed-section", path, sectionId, owner: record.owner }); }
 export async function buildInstallPlan(ctx, previous) {
-    const warnings = [];
-    const operations = [{ type: "mkdir", path: ctx.projectRoot, owner: "layout" }];
-    const conflicts = [];
-    const now = new Date().toISOString();
-    const backupStamp = now.replace(/[:.]/g, "-");
+    const warnings = [], operations = [{ type: "mkdir", path: ctx.projectRoot, owner: "layout" }], conflicts = [];
+    const now = new Date().toISOString(), backupStamp = now.replace(/[:.]/g, "-");
     previous = previous ? migrateInstallationManifest(previous) : null;
     if (ctx.action === "uninstall") {
         if (!previous)
@@ -320,27 +224,15 @@ export async function buildInstallPlan(ctx, previous) {
                 continue;
             await protectRemoval({ ctx, previous, pathRel, record, operations, conflicts, backupStamp });
         }
-        for (const [pathRel, sections] of Object.entries(previous.managedSections)) {
-            for (const [sectionId, record] of Object.entries(sections)) {
+        for (const [pathRel, sections] of Object.entries(previous.managedSections))
+            for (const [sectionId, record] of Object.entries(sections))
                 await protectSectionRemoval({ ctx, previous, pathRel, sectionId, record, operations, conflicts, backupStamp });
-            }
-        }
-        for (const [pathRel, records] of Object.entries(previous.managedConfigKeys ?? {})) {
+        for (const [pathRel, records] of Object.entries(previous.managedConfigKeys ?? {}))
             await protectConfigRemoval({ ctx, pathRel, records, operations, conflicts, backupStamp });
-        }
         if (conflicts.length)
             throw new ManagedConflictError(conflicts);
-        operations.push({ type: "remove-managed-file", path: join(ctx.projectRoot, MANIFEST_RELATIVE_PATH), owner: "installer" });
-        operations.push({
-            type: "write-managed-file",
-            path: join(ctx.projectRoot, ".yaaw-core", "install", "uninstalled.json"),
-            content: JSON.stringify({ schema: "yaaw.uninstalled/v1", version: previous.yaawVersion, uninstalledAt: now, projectMemory: ".yaaw-core/project" }, null, 2) + "\n",
-            owner: "installer"
-        });
-        return {
-            plan: { action: ctx.action, projectRoot: ctx.projectRoot, operations, selectedIntegrations: [], selectedSkills: [], warnings: ["Durable .yaaw-core/project data will be preserved."] },
-            manifest: null
-        };
+        operations.push({ type: "remove-managed-file", path: join(ctx.projectRoot, MANIFEST_RELATIVE_PATH), owner: "installer" }, { type: "write-managed-file", path: join(ctx.projectRoot, ".yaaw-core", "install", "uninstalled.json"), content: JSON.stringify({ schema: "yaaw.uninstalled/v1", version: previous.yaawVersion, uninstalledAt: now, projectMemory: ".yaaw-core/project" }, null, 2) + "\n", owner: "installer" });
+        return { plan: { action: ctx.action, projectRoot: ctx.projectRoot, operations, selectedIntegrations: [], selectedSkills: [], warnings: ["Durable .yaaw-core/project data will be preserved."] }, manifest: null };
     }
     if (previous) {
         if (previous.installationSchema > CURRENT_INSTALLATION_SCHEMA)
@@ -351,89 +243,46 @@ export async function buildInstallPlan(ctx, previous) {
             throw new Error(`Installed project schema ${previous.projectSchema} is newer than this package supports (${CURRENT_PROJECT_SCHEMA}); downgrade is blocked.`);
         for (const integrationId of ctx.selectedIntegrations) {
             const installed = previous.integrations[integrationId];
-            if (installed && installed.adapterVersion > getIntegration(integrationId).adapterVersion) {
+            if (installed && installed.adapterVersion > getIntegration(integrationId).adapterVersion)
                 throw new Error(`Installed ${integrationId} adapter v${installed.adapterVersion} is newer than this package supports; downgrade is blocked.`);
-            }
         }
     }
     const manifest = emptyManifest(ctx.packageVersion, previous?.installedAt ?? now);
     manifest.updatedAt = now;
     manifest.skills = [...ctx.selectedSkills];
-    const coreRoot = join(ctx.payloadRoot, "yaaw-core", "system");
-    const coreFiles = await walk(coreRoot);
-    const desiredFilePaths = new Set();
+    const coreRoot = join(ctx.payloadRoot, "yaaw-core", "system"), coreFiles = await walk(coreRoot), desiredFilePaths = new Set();
     for (const source of coreFiles) {
-        const coreRel = relative(coreRoot, source);
-        const target = join(ctx.projectRoot, ".yaaw-core", "system", coreRel);
+        const coreRel = relative(coreRoot, source), target = join(ctx.projectRoot, ".yaaw-core", "system", coreRel);
         desiredFilePaths.add(rel(ctx.projectRoot, target));
         await chooseManagedFile({ ctx, previous, path: target, owner: "package:system", source, operations, records: manifest.managedFiles, conflicts, backupStamp });
     }
     operations.push(...await planProjectInitialization(ctx.payloadRoot, ctx.projectRoot));
     if (["quick-update", "modify", "repair"].includes(ctx.action)) {
-        for (const name of ["observed-state.json", "handoff.json", "intent.json", "dispatch-failures.json"]) {
-            operations.push({
-                type: "remove-runtime-file",
-                path: join(ctx.projectRoot, ".yaaw-core", "runtime", name),
-                owner: "runtime:invalidate"
-            });
-        }
+        for (const name of ["observed-state.json", "handoff.json", "intent.json", "dispatch-failures.json"])
+            operations.push({ type: "remove-runtime-file", path: join(ctx.projectRoot, ".yaaw-core", "runtime", name), owner: "runtime:invalidate" });
         warnings.push("Replaceable YAAW runtime coordination caches will be invalidated after package changes.");
     }
-    if (previous) {
-        for (const migration of migrationPath(projectMigrations, previous.projectSchema, CURRENT_PROJECT_SCHEMA)) {
-            operations.push(...await migration.plan({
-                projectRoot: ctx.projectRoot,
-                payloadRoot: ctx.payloadRoot,
-                fromVersion: migration.from,
-                toVersion: migration.to
-            }));
-        }
-    }
-    const configurationSummaries = [];
-    const skillMap = new Map((await canonicalSkills(ctx.payloadRoot)).map(x => [x.id, x]));
-    const selectedCanonical = ctx.selectedSkills.map(id => {
-        const skill = skillMap.get(id);
-        if (!skill)
-            throw new Error(`Unknown canonical skill: ${id}`);
-        return skill;
-    });
+    if (previous)
+        for (const migration of migrationPath(projectMigrations, previous.projectSchema, CURRENT_PROJECT_SCHEMA))
+            operations.push(...await migration.plan({ projectRoot: ctx.projectRoot, payloadRoot: ctx.payloadRoot, fromVersion: migration.from, toVersion: migration.to }));
+    const configurationSummaries = [], skillMap = new Map((await canonicalSkills(ctx.payloadRoot)).map(x => [x.id, x]));
+    const selectedCanonical = ctx.selectedSkills.map(id => { const skill = skillMap.get(id); if (!skill)
+        throw new Error(`Unknown canonical skill: ${id}`); return skill; });
     for (const integrationId of ctx.selectedIntegrations) {
-        const adapter = getIntegration(integrationId);
-        const previousRecord = previous?.integrations?.[integrationId];
-        const previousConfiguration = previousRecord?.configuration;
+        const adapter = getIntegration(integrationId), previousRecord = previous?.integrations?.[integrationId], previousConfiguration = previousRecord?.configuration;
         let settings = ctx.integrationSettings?.[integrationId] ?? previousConfiguration?.settings ?? previousRecord?.runtime;
         if (adapter.configuration && settings === undefined)
             settings = adapter.configuration.defaultSettings();
-        const ictx = { projectRoot: ctx.projectRoot, payloadRoot: ctx.payloadRoot, settings };
-        const runtimeOps = adapter.configuration ? await adapter.configuration.plan(ictx) : adapter.planRuntime ? await adapter.planRuntime(ictx) : [];
-        const adapterOps = [...await adapter.planSkills(ictx, selectedCanonical), ...await adapter.planBootstrap(ictx), ...runtimeOps];
-        const installationRecord = {
-            adapterVersion: adapter.adapterVersion,
-            skillsRoot: rel(ctx.projectRoot, adapter.skillsRoot(ctx.projectRoot)),
-            bootstrap: adapter.bootstrapRelativePath
-        };
+        const ictx = { projectRoot: ctx.projectRoot, payloadRoot: ctx.payloadRoot, settings }, runtimeOps = adapter.configuration ? await adapter.configuration.plan(ictx) : adapter.planRuntime ? await adapter.planRuntime(ictx) : [], adapterOps = [...await adapter.planSkills(ictx, selectedCanonical), ...await adapter.planBootstrap(ictx), ...runtimeOps];
+        const installationRecord = { adapterVersion: adapter.adapterVersion, skillsRoot: rel(ctx.projectRoot, adapter.skillsRoot(ctx.projectRoot)), bootstrap: adapter.bootstrapRelativePath };
         if (adapter.configuration) {
-            const normalizedSettings = adapter.configuration.normalize(settings);
-            const preserveReviewedState = Boolean(previousConfiguration && ["quick-update", "repair"].includes(ctx.action));
-            const configuration = preserveReviewedState
-                ? structuredClone(previousConfiguration)
-                : {
-                    schema: "yaaw.integration-config/v1",
-                    appliedRevision: adapter.configuration.revision,
-                    notifiedRevision: adapter.configuration.revision,
-                    profile: ctx.integrationProfiles?.[integrationId] ?? previousConfiguration?.profile ?? { id: "custom", revision: adapter.configuration.revision },
-                    settings: normalizedSettings
-                };
+            const normalizedSettings = adapter.configuration.normalize(settings), preserveReviewedState = Boolean(previousConfiguration && ["quick-update", "repair"].includes(ctx.action)), configuration = preserveReviewedState ? structuredClone(previousConfiguration) : { schema: "yaaw.integration-config/v1", appliedRevision: adapter.configuration.revision, notifiedRevision: adapter.configuration.revision, profile: ctx.integrationProfiles?.[integrationId] ?? previousConfiguration?.profile ?? { id: "custom", revision: adapter.configuration.revision }, settings: normalizedSettings };
             if (ctx.action === "quick-update" && ctx.acknowledgeConfigurationUpdates?.includes(integrationId))
                 configuration.notifiedRevision = adapter.configuration.revision;
             configuration.settings = preserveReviewedState ? previousConfiguration.settings : normalizedSettings;
             installationRecord.configuration = configuration;
             installationRecord.runtime = configuration.settings;
-            configurationSummaries.push({
-                integrationId,
-                profile: configuration.profile ? `${configuration.profile.id} r${configuration.profile.revision}` : "custom",
-                description: adapter.configuration.describe(configuration.settings)
-            });
+            configurationSummaries.push({ integrationId, profile: configuration.profile ? `${configuration.profile.id} r${configuration.profile.revision}` : "custom", description: adapter.configuration.describe(configuration.settings) });
         }
         else if (adapter.planRuntime)
             installationRecord.runtime = settings ?? null;
@@ -447,47 +296,28 @@ export async function buildInstallPlan(ctx, previous) {
                 desiredFilePaths.add(rel(ctx.projectRoot, op.path));
                 await chooseManagedFile({ ctx, previous, path: op.path, owner: op.owner, content: op.content, operations, records: manifest.managedFiles, conflicts, backupStamp });
             }
-            else if (op.type === "update-managed-section") {
+            else if (op.type === "update-managed-section")
                 await chooseManagedSection({ ctx, previous, op, sectionRecords: manifest.managedSections, operations, conflicts, backupStamp });
-            }
-            else if (op.type === "update-managed-config-keys") {
+            else if (op.type === "update-managed-config-keys")
                 await chooseManagedConfigKeys({ ctx, previous, op, configRecords: manifest.managedConfigKeys, operations, conflicts, backupStamp });
-            }
         }
     }
     if (previous) {
-        for (const [pathRel, record] of Object.entries(previous.managedFiles)) {
-            if (!desiredFilePaths.has(pathRel) && record.owner !== "installer:backup") {
+        for (const [pathRel, record] of Object.entries(previous.managedFiles))
+            if (!desiredFilePaths.has(pathRel) && record.owner !== "installer:backup")
                 await protectRemoval({ ctx, previous, pathRel, record, operations, conflicts, backupStamp });
-            }
-        }
-        for (const [pathRel, sections] of Object.entries(previous.managedSections)) {
-            for (const [sectionId, record] of Object.entries(sections)) {
-                if (!manifest.managedSections[pathRel]?.[sectionId]) {
+        for (const [pathRel, sections] of Object.entries(previous.managedSections))
+            for (const [sectionId, record] of Object.entries(sections))
+                if (!manifest.managedSections[pathRel]?.[sectionId])
                     await protectSectionRemoval({ ctx, previous, pathRel, sectionId, record, operations, conflicts, backupStamp });
-                }
-            }
-        }
         for (const [pathRel, records] of Object.entries(previous.managedConfigKeys ?? {})) {
             const obsolete = Object.fromEntries(Object.entries(records).filter(([key]) => !manifest.managedConfigKeys[pathRel]?.[key]));
-            if (Object.keys(obsolete).length) {
+            if (Object.keys(obsolete).length)
                 await protectConfigRemoval({ ctx, pathRel, records: obsolete, operations, conflicts, backupStamp });
-            }
         }
     }
     if (conflicts.length)
         throw new ManagedConflictError(conflicts);
-    return {
-        plan: {
-            action: ctx.action,
-            projectRoot: ctx.projectRoot,
-            operations,
-            selectedIntegrations: [...ctx.selectedIntegrations],
-            selectedSkills: [...ctx.selectedSkills],
-            warnings,
-            configurationSummaries
-        },
-        manifest
-    };
+    return { plan: { action: ctx.action, projectRoot: ctx.projectRoot, operations, selectedIntegrations: [...ctx.selectedIntegrations], selectedSkills: [...ctx.selectedSkills], warnings, configurationSummaries }, manifest };
 }
 //# sourceMappingURL=plan.js.map