@skaile/workspaces 6.14.0 → 6.16.0

package/dist/{chunk-OC3AIMZH.js → chunk-KDRA6FYE.js}

@@ -1,17 +1,17 @@
-import { openLibrary } from './chunk-VKGNMXGZ.js';
-import { getKindRegistry } from './chunk-HPQ2WDYC.js';
-import { S, logErr, logWarn, logOk, logInfo } from './chunk-5FO4U3OL.js';
-import { resolveBinding, UnknownCompositionKindError } from './chunk-LZCX2TCZ.js';
-import { filesEntryPaths, resolveIndexPath, LocalIndex, assetRootProblem, stampManifestPublisher, upsertManifestAsset } from './chunk-7FXOOUOC.js';
-import { SEMVER_RE, invalidSemverMessage, flowFileNameError, parseManifestFile, requiresSyntaxErrors } from './chunk-DRUAQ7IY.js';
+import { openLibrary } from './chunk-Z7HLX4V2.js';
+import { getKindRegistry } from './chunk-U2WPJHRA.js';
+import { S, logErr, logWarn, logOk, logInfo } from './chunk-WW22FS74.js';
 import { parseFlowDefinition } from './chunk-JRN5QF4C.js';
-import { SKILL_DESCRIPTION_PLACEHOLDER_PREFIX, FLOW_DESCRIPTION_PLACEHOLDER, FLOW_NODE_INSTRUCTION_PLACEHOLDER, PRESET_DESCRIPTION_PLACEHOLDER, PRESET_ITEM_PLACEHOLDER_REF, isScaffoldBody, composeSlotDiagnostics, agentDeprecationDiagnostics, composeRefTarget } from './chunk-LCXQ4LVP.js';
-import { checkAgentSkillConformance } from './chunk-MEU2HHGV.js';
-import { matchPin } from './chunk-BNLXZXZN.js';
+import { resolveBinding, UnknownCompositionKindError } from './chunk-4ZNKEDYH.js';
+import { filesEntryPaths, resolveIndexPath, LocalIndex, assetRootProblem, stampManifestPublisher, upsertManifestAsset } from './chunk-RPRMD7XV.js';
+import { SEMVER_RE, invalidSemverMessage, parseManifestFile, requiresSyntaxErrors, flowFileNameError } from './chunk-QUPZKTF3.js';
+import { checkAsset, assetPathProblem } from './chunk-U7P22GEB.js';
+import { matchPin } from './chunk-BRAOB37W.js';
 import { globalSkaileDir } from './chunk-L5VSIPQD.js';
-import { SKAILE_MANIFEST_FILENAME, loadSkPublishManifest, listAssetFiles, expandManifestFiles, listLibrarySymlinks, librarySymlinkMessage, isNonAssetPath, isLibraryManifestAtAssetRoot } from './chunk-MS2IXLTC.js';
-import { readAssetBody, parseFrontmatter } from './chunk-VS6XC5JE.js';
-import { parseDep, PUBLISHER_HINT, isValidPublisher, isValidAssetName, slugifyAssetName, parseAssetRef, ASSET_NAME_HINT, assetRefProblem } from './chunk-7HRZACI4.js';
+import { SKAILE_MANIFEST_FILENAME, manifestTextPublisherProblems, loadSkPublishManifest, listAssetFiles, expandManifestFiles, listLibrarySymlinks, librarySymlinkMessage, isNonAssetPath, isLibraryManifestAtAssetRoot } from './chunk-PMXTBWFD.js';
+import { readAssetBody, parseFrontmatter } from './chunk-2MTRRK37.js';
+import { parseDep, parsePublisher, isValidAssetName, slugifyAssetName, assetRefProblem } from './chunk-HW4BDJPC.js';
+import { composeSlotDiagnostics, agentDeprecationDiagnostics, composeRefTarget } from './chunk-LCXQ4LVP.js';
 import * as fs3 from 'fs';
 import { rmSync, readFileSync, mkdirSync, writeFileSync, chmodSync } from 'fs';
 import * as path3 from 'path';
@@ -199,34 +199,6 @@ function manifestKindOfFile(file) {
 function manifestParseErrorText(reason) {
   return `parse error: ${reason}`;
 }
-var UNUSED_KEYS = ["reads_from", "writes_to"];
-function checkAssetManifest(kind, parsed, body, dirName) {
-  const out = [];
-  if (kind === "skill") {
-    for (const f of checkAgentSkillConformance(parsed, { dirName })) {
-      out.push({ tag: "skill conformance", text: `${f.field}: ${f.message}`, conformance: true });
-    }
-  }
-  for (const key of UNUSED_KEYS) {
-    if (key in parsed) {
-      out.push({
-        tag: kind,
-        text: `${key}: unknown key; nothing in skaile reads it, so it has no effect. Remove it`,
-        conformance: false
-      });
-    }
-  }
-  if (isScaffoldBody(kind, body)) {
-    const noun = kind === "skill" ? "skill prompt" : kind;
-    out.push({
-      tag: kind,
-      text: `body is still the scaffold placeholder; write the ${noun}`,
-      conformance: false,
-      scaffoldBody: true
-    });
-  }
-  return out;
-}
 function declaredVersion(parsed) {
   const scalar = (v) => typeof v === "string" ? v : typeof v === "number" ? String(v) : void 0;
   const meta = parsed.metadata;
@@ -275,66 +247,6 @@ function withManifestDefaults(kind, parsed) {
   if (out.version == null && nested == null) out.version = DEFAULT_VERSION;
   return out;
 }
-function manifestIdentityError(kind, parsed) {
-  const identity = kind === "flow" ? parsed.id : parsed.name;
-  if (typeof identity !== "string" || identity.length === 0) return void 0;
-  const publisher = typeof parsed.publisher === "string" ? parsed.publisher : void 0;
-  if (publisher !== void 0) {
-    try {
-      parseAssetRef(`${kind}:@${publisher}/${identity}`);
-      return void 0;
-    } catch (e) {
-      return e instanceof Error ? e.message : String(e);
-    }
-  }
-  if (!isValidAssetName(identity)) {
-    return `invalid asset ${kind === "flow" ? "id" : "name"} "${identity}" \u2014 ${ASSET_NAME_HINT}`;
-  }
-  return void 0;
-}
-function scaffoldDefinitionErrors(kind, parsed) {
-  const out = [];
-  if (kind === "flow") {
-    if (parsed.description === FLOW_DESCRIPTION_PLACEHOLDER) {
-      out.push("description is still the scaffold placeholder; say what the flow does");
-    }
-    if (parsed.nodes === void 0) {
-      out.push("nodes is missing; a flow needs at least one node");
-    } else if (Array.isArray(parsed.nodes) && parsed.nodes.length === 0) {
-      out.push("nodes is still the scaffold placeholder (nodes: []); add the flow's nodes");
-    } else if (Array.isArray(parsed.nodes)) {
-      for (const node of parsed.nodes) {
-        if (typeof node !== "object" || node === null || !("run" in node)) continue;
-        const run = node.run;
-        if (typeof run !== "object" || run === null || !("instruction" in run)) continue;
-        if (run.instruction === FLOW_NODE_INSTRUCTION_PLACEHOLDER) {
-          const id = "id" in node ? String(node.id) : "?";
-          out.push(
-            `node "${id}" instruction is still the scaffold placeholder; say what the step does`
-          );
-        }
-      }
-    }
-  }
-  if (kind === "preset") {
-    if (parsed.description === PRESET_DESCRIPTION_PLACEHOLDER) {
-      out.push(
-        `description is still the scaffold placeholder ("${PRESET_DESCRIPTION_PLACEHOLDER}"); say what the preset does`
-      );
-    }
-    const items = Array.isArray(parsed.items) ? parsed.items : [];
-    const placeholderItem = items.some((item) => {
-      const ref = typeof item === "string" ? item : item?.ref;
-      return typeof ref === "string" && ref.trim() === PRESET_ITEM_PLACEHOLDER_REF;
-    });
-    if (placeholderItem) {
-      out.push(
-        `item ${PRESET_ITEM_PLACEHOLDER_REF} is still the scaffold placeholder; replace it with the preset's real items`
-      );
-    }
-  }
-  return out;
-}
 function contentRequiresEntries(parsed, fileName) {
   if (fileName === "agent.yaml") return [];
   const metadata = parsed.metadata;
@@ -403,86 +315,92 @@ function nameDescriptionProblem(kind, parsed, dirName, e) {
   }
   return void 0;
 }
-function checkAsset(kind, parsed, body, dirName, registry, rawText, fileName, agentFiles) {
-  const errors = [];
-  const error2 = (text) => {
-    errors.push({ tag: kind, text, conformance: false });
-  };
-  const identity = manifestIdentityError(kind, parsed);
-  if (identity) error2(identity);
-  const flowFileError = kind === "flow" ? flowFileNameError(parsed, fileName) : void 0;
-  if (flowFileError) error2(flowFileError);
-  const semver = versionProblemText(parsed, rawText);
-  if (semver) error2(semver);
+function runAssetChecks(command, subject, opts = {}) {
+  const { registry, agentFiles, ...input } = subject;
+  const { kind, parsed, dirName, rawText, fileName } = input;
+  const error2 = (text) => ({ tag: kind, text, conformance: false });
   const effective = withManifestDefaults(kind, parsed);
-  const versionKey = ["skill", "prompt", "contract"].includes(kind) ? "metadata.version" : "version";
-  const provider = registry.getProvider(kind);
   const reported = /* @__PURE__ */ new Set();
-  if (!provider) {
-    error2(`unknown_asset_kind: kind "${kind}" is not registered`);
-  } else {
-    const result = provider.validateManifest(effective);
-    if (!result.ok) {
+  let semver;
+  const deps = {
+    file: () => {
+      const text = kind === "flow" ? flowFileNameError(parsed, fileName) : void 0;
+      return text ? [error2(text)] : [];
+    },
+    version: () => {
+      semver = versionProblemText(parsed, rawText);
+      return semver ? [error2(semver)] : [];
+    },
+    schema: () => {
+      const errors = [];
+      const versionKey = ["skill", "prompt", "contract"].includes(kind) ? "metadata.version" : "version";
+      const provider = registry.getProvider(kind);
+      if (!provider) {
+        errors.push(error2(`unknown_asset_kind: kind "${kind}" is not registered`));
+        return errors;
+      }
+      const result = provider.validateManifest(effective);
+      if (result.ok) return errors;
       for (const e of result.errors) {
         if (e.path === "version" || e.path === "metadata.version") {
           if (!semver) {
-            error2(`${e.path}: must be a semver string \u2014 set ${versionKey}: ${DEFAULT_VERSION}`);
+            errors.push(
+              error2(`${e.path}: must be a semver string \u2014 set ${versionKey}: ${DEFAULT_VERSION}`)
+            );
           }
           continue;
         }
         reported.add(e.path);
         const plain = nameDescriptionProblem(kind, parsed, dirName, e);
-        if (plain) {
-          error2(plain);
-          continue;
-        }
-        error2(`${e.path ? `${e.path}: ` : ""}${e.message}`);
+        errors.push(error2(plain ?? `${e.path ? `${e.path}: ` : ""}${e.message}`));
       }
-    }
-  }
-  const description = parsed.description;
-  if (kind !== "skill" && typeof description === "string" && description.trim().startsWith(SKILL_DESCRIPTION_PLACEHOLDER_PREFIX)) {
-    error2(
-      `description: description is still the scaffold placeholder (${SKILL_DESCRIPTION_PLACEHOLDER_PREFIX} \u2026) \u2014 say what the ${kind} does and when to use it`
-    );
-  }
-  for (const text of scaffoldDefinitionErrors(kind, parsed)) error2(text);
-  for (const text of referenceErrors(kind, parsed, registry, fileName)) error2(text);
-  if (kind === "flow") {
-    if (!reported.has("description")) {
-      const plain = nameDescriptionProblem(kind, parsed, dirName, {
-        path: "description",
-        message: ""
-      });
-      if (plain) {
-        error2(plain);
-        reported.add("description");
+      return errors;
+    },
+    references: () => ({
+      errors: referenceErrors(kind, parsed, registry, fileName).map(error2),
+      warnings: unheldRequiresFindings(kind, parsed, fileName, agentFiles?.composeRefs)
+    }),
+    definition: () => {
+      const errors = [];
+      const warnings = [];
+      if (kind === "flow") {
+        if (!reported.has("description")) {
+          const plain = nameDescriptionProblem(kind, parsed, dirName, {
+            path: "description",
+            message: ""
+          });
+          if (plain) {
+            errors.push(error2(plain));
+            reported.add("description");
+          }
+        }
+        const v2WithoutVersion = flowMissingSchemaVersionError(parsed);
+        if (v2WithoutVersion) errors.push(error2(v2WithoutVersion));
+        const flow = parseFlowDefinition(effective);
+        if (!flow.ok) {
+          for (const e of flow.errors) {
+            if (reported.has(e.path)) continue;
+            errors.push(error2(`invalid flow definition: ${e.path || "<root>"}: ${e.message}`));
+          }
+        }
       }
-    }
-    const v2WithoutVersion = flowMissingSchemaVersionError(parsed);
-    if (v2WithoutVersion) error2(v2WithoutVersion);
-    const flow = parseFlowDefinition(effective);
-    if (!flow.ok) {
-      for (const e of flow.errors) {
-        if (reported.has(e.path)) continue;
-        error2(`invalid flow definition: ${e.path || "<root>"}: ${e.message}`);
+      if (kind === "agent") {
+        if (fileName === "agent.yaml") {
+          const definition = checkAgentDefinition(parsed, registry);
+          errors.push(...definition.errors);
+          warnings.push(...definition.advisories);
+        }
+        if (agentFiles) {
+          errors.push(...checkAgentFiles(parsed, agentFiles.dir, agentFiles.libraryRoot));
+        }
+        if (agentFiles?.composeRefs) {
+          errors.push(...agentComposeRefProblems(parsed, agentFiles.composeRefs));
+        }
       }
+      return { errors, warnings };
     }
-  }
-  const advisories = checkAssetManifest(kind, parsed, body, dirName);
-  advisories.push(...unheldRequiresFindings(kind, parsed, fileName, agentFiles?.composeRefs));
-  if (kind === "agent") {
-    if (fileName === "agent.yaml") {
-      const definition = checkAgentDefinition(parsed, registry);
-      errors.push(...definition.errors);
-      advisories.push(...definition.advisories);
-    }
-    if (agentFiles) errors.push(...checkAgentFiles(parsed, agentFiles.dir, agentFiles.libraryRoot));
-    if (agentFiles?.composeRefs) {
-      errors.push(...agentComposeRefProblems(parsed, agentFiles.composeRefs));
-    }
-  }
-  return { errors, advisories };
+  };
+  return checkAsset(command, input, deps, opts);
 }
 function flowMissingSchemaVersionError(parsed) {
   if ("schemaVersion" in parsed || !Array.isArray(parsed.nodes)) return void 0;
@@ -529,26 +447,34 @@ function findEntryManifestFile(libraryPath, entry) {
   }
   return null;
 }
-function checkAssetFile(file, kind, registry, libraryRoot, composeRefs) {
+function checkAssetFileFull(command, file, kind, registry, libraryRoot, composeRefs) {
   let text;
   try {
     text = fs3.readFileSync(file, "utf8");
   } catch {
-    return [];
+    return { errors: [], warnings: [] };
   }
   const res = parseManifestFile(file, text);
-  if (!res.ok) return [];
-  const checked = checkAsset(
+  if (!res.ok) return { errors: [], warnings: [] };
+  return runAssetChecks(command, {
     kind,
-    res.data,
-    readAssetBody(file, text),
-    path3.basename(path3.dirname(file)),
+    parsed: res.data,
+    body: readAssetBody(file, text),
+    dirName: path3.basename(path3.dirname(file)),
     registry,
-    text,
-    path3.basename(file),
-    libraryRoot === void 0 ? void 0 : { dir: path3.dirname(file), libraryRoot, ...composeRefs ? { composeRefs } : {} }
-  );
-  return [...checked.errors, ...checked.advisories.filter((f) => f.scaffoldBody)];
+    rawText: text,
+    fileName: path3.basename(file),
+    ...libraryRoot === void 0 ? {} : {
+      agentFiles: {
+        dir: path3.dirname(file),
+        libraryRoot,
+        ...composeRefs ? { composeRefs } : {}
+      }
+    }
+  });
+}
+function checkAssetFile(command, file, kind, registry, libraryRoot, composeRefs) {
+  return checkAssetFileFull(command, file, kind, registry, libraryRoot, composeRefs).errors;
 }
 function libraryAssetRefusals(asset, libraryPath, registry) {
   let file = asset.source;
@@ -565,7 +491,7 @@ function libraryAssetRefusals(asset, libraryPath, registry) {
   } catch {
     return [];
   }
-  return checkAssetFile(file, asset.kind, registry, libraryPath).map(
+  return checkAssetFile("index", file, asset.kind, registry, libraryPath).map(
     (f) => `${rel} [${f.tag}] \u2014 ${f.text}`
   );
 }
@@ -1044,42 +970,24 @@ async function manifestDependencyProblems(dir) {
 async function unresolvedManifestDependencies(dir) {
   return (await manifestDependencyProblems(dir)).map((p) => p.message);
 }
-function invalidPublisherMessage(raw) {
-  return `Invalid publisher "${raw}": ${PUBLISHER_HINT}.`;
-}
 function requireValidPublisherOrExit(raw) {
-  const bare = raw.trim().replace(/^@/, "");
-  if (!isValidPublisher(bare)) {
-    logErr(invalidPublisherMessage(raw));
+  const parsed = parsePublisher(raw);
+  if (!parsed.ok) {
+    logErr(parsed.message);
     process.exit(1);
   }
-  return bare;
+  return parsed.publisher;
 }
 function manifestPublisherProblems(dir) {
-  let doc;
+  let text;
   try {
-    doc = parse(fs3.readFileSync(path3.join(dir, SKAILE_MANIFEST_FILENAME), "utf-8"));
+    text = fs3.readFileSync(path3.join(dir, SKAILE_MANIFEST_FILENAME), "utf-8");
   } catch {
     return [];
   }
-  if (!doc || typeof doc !== "object") return [];
-  const { publisher, assets } = doc;
-  const problems = [];
-  const check = (value, where) => {
-    if (typeof value !== "string") return;
-    if (!isValidPublisher(value.trim().replace(/^@/, ""))) {
-      problems.push(`${SKAILE_MANIFEST_FILENAME} ${where} \u2014 ${invalidPublisherMessage(value)}`);
-    }
-  };
-  check(publisher, "[publisher]");
-  if (Array.isArray(assets)) {
-    for (const a of assets) {
-      if (!a || typeof a !== "object") continue;
-      const { kind, name, publisher: assetPublisher } = a;
-      check(assetPublisher, `[assets] ${String(kind)}:${String(name)}`);
-    }
-  }
-  return problems;
+  return manifestTextPublisherProblems(text).map(
+    (p) => `${SKAILE_MANIFEST_FILENAME} ${p.where} \u2014 ${p.message}`
+  );
 }
 function exitOnManifestPublisherProblems(dir, libName) {
   const problems = manifestPublisherProblems(dir);
@@ -1148,7 +1056,7 @@ function manifestRootProblems(dir) {
     if (!a || typeof a !== "object") continue;
     const { kind, name, root, source, files } = a;
     if (typeof root !== "string" || source !== void 0) continue;
-    const problem = assetRootProblem(dir, root) ?? (files === void 0 ? emptyRootProblem(dir, root) : void 0);
+    const problem = assetRootProblem(dir, root) ?? assetPathProblem(root) ?? (files === void 0 ? emptyRootProblem(dir, root) : void 0);
     if (problem) {
       problems.push(
         `${SKAILE_MANIFEST_FILENAME} [assets] ${String(kind)}:${String(name)} \u2014 ${problem}`
@@ -1317,7 +1225,8 @@ function resolveLibraryPublisherDetailed(lib, opts, storeIdentity) {
   const declared = loadSkPublishManifest(lib.path)?.publisher?.trim();
   if (declared) return { publisher: bare(declared), source: "manifest" };
   const identity = (storeIdentity?.githubLogin ?? storeIdentity?.username)?.trim().toLowerCase();
-  if (identity && isValidPublisher(identity)) return { publisher: identity, source: "identity" };
+  const parsedIdentity = identity === void 0 ? void 0 : parsePublisher(identity);
+  if (parsedIdentity?.ok) return { publisher: parsedIdentity.publisher, source: "identity" };
   try {
     const slug = slugifyAssetName(lib.name).slice(0, 39).replace(/-+$/, "");
     return { publisher: slug, source: "library-name" };
@@ -1459,6 +1368,6 @@ function warnUntrackedAssetFiles(libPath) {
   return count;
 }
 
-export { DEFAULT_VERSION, DOMAIN_REMOVED_HINT, MANIFEST_GLOBS, assetEntryRef, assetNamespace, bareName, checkAsset, checkAssetFile, checkComposeKinds, composeRefContext, declaredVersion, deviceLogin, duplicateAssetMessage, exitOnEmptyAsset, exitOnLibrarySymlinks, exitOnManifestPathProblems, exitOnManifestPublisherProblems, exitOnRootSkillExtraAssets, exitOnRootSkillLibrary, findAssetManifestFile, findEntryManifestFile, formatNamespaceAvailability, getAccessToken, getStoreIdentity, hasStoredAuth, indexLibrary, invalidPublisherMessage, invalidSemverText, libraryAssetRefusals, loadStoreAuthConfig, logout, manifestDependencyProblems, manifestEntryFilesRefusals, manifestEntryNameProblems, manifestEntryRenames, manifestEntryRootRefusals, manifestFilesProblems, manifestNamespaces, manifestParseErrorText, manifestParseFailureLine, manifestPublisherProblems, manifestRequiresProblems, manifestRootProblems, manifestVersionMismatchMessage, plannedRefs, preflightNamespace, publishToStore, readOnlyLibraryAt, refFor, registerLibraryAsset, registeredAssetKeys, rejectRemovedDomainOption, removedDomainOption, renameFileFix, reportLibrarySymlinks, requireValidPublisherOrExit, resolveLibraryPublisherDetailed, rootLevelAssetFileProblem, rootSkillExtraAssetProblems, unheldRequiresOfFile, unresolvedManifestDependencies, warnIgnoredDomainFiles, warnLegacyBundleFiles, warnLibraryNameFallback, warnUntrackedAssetFiles };
-//# sourceMappingURL=chunk-OC3AIMZH.js.map
-//# sourceMappingURL=chunk-OC3AIMZH.js.map
+export { DEFAULT_VERSION, DOMAIN_REMOVED_HINT, MANIFEST_GLOBS, assetEntryRef, assetNamespace, bareName, checkAssetFile, checkAssetFileFull, checkComposeKinds, composeRefContext, declaredVersion, deviceLogin, duplicateAssetMessage, exitOnEmptyAsset, exitOnLibrarySymlinks, exitOnManifestPathProblems, exitOnManifestPublisherProblems, exitOnRootSkillExtraAssets, exitOnRootSkillLibrary, findAssetManifestFile, findEntryManifestFile, formatNamespaceAvailability, getAccessToken, getStoreIdentity, hasStoredAuth, indexLibrary, invalidSemverText, libraryAssetRefusals, loadStoreAuthConfig, logout, manifestDependencyProblems, manifestEntryFilesRefusals, manifestEntryNameProblems, manifestEntryRenames, manifestEntryRootRefusals, manifestFilesProblems, manifestKindOfFile, manifestNamespaces, manifestParseErrorText, manifestParseFailureLine, manifestPublisherProblems, manifestRequiresProblems, manifestRootProblems, manifestVersionMismatchMessage, plannedRefs, preflightNamespace, publishToStore, readOnlyLibraryAt, refFor, registerLibraryAsset, registeredAssetKeys, rejectRemovedDomainOption, removedDomainOption, renameFileFix, reportLibrarySymlinks, requireValidPublisherOrExit, resolveLibraryPublisherDetailed, rootLevelAssetFileProblem, rootSkillExtraAssetProblems, runAssetChecks, unheldRequiresOfFile, unresolvedManifestDependencies, warnIgnoredDomainFiles, warnLegacyBundleFiles, warnLibraryNameFallback, warnUntrackedAssetFiles };
+//# sourceMappingURL=chunk-KDRA6FYE.js.map
+//# sourceMappingURL=chunk-KDRA6FYE.js.map