@skaile/workspaces 6.0.0 → 6.2.0

package/dist/{chunk-TKHJXPSB.js → chunk-2XM3ZH36.js}

@@ -1,17 +1,17 @@
-import { openLibrary } from './chunk-DHYSUJ3H.js';
-import { S, logErr, logWarn, logOk, logInfo } from './chunk-KX2PT5Q4.js';
-import { getKindRegistry } from './chunk-ARWKK325.js';
-import { resolveBinding, UnknownCompositionKindError } from './chunk-VJRTDKXS.js';
-import { filesEntryPaths, assetRootProblem, stampManifestPublisher, upsertManifestAsset, resolveIndexPath, LocalIndex } from './chunk-UL22SDGR.js';
-import { SEMVER_RE, invalidSemverMessage, flowFileNameError, parseManifestFile, requiresSyntaxErrors } from './chunk-QOM5Z62G.js';
-import { parseFlowDefinition } from './chunk-X3RMRJNR.js';
-import { SKILL_DESCRIPTION_PLACEHOLDER_PREFIX, FLOW_DESCRIPTION_PLACEHOLDER, PRESET_DESCRIPTION_PLACEHOLDER, PRESET_ITEM_PLACEHOLDER_REF, isScaffoldBody, composeSlotDiagnostics, agentDeprecationDiagnostics, composeRefTarget } from './chunk-E572OHGY.js';
-import { checkAgentSkillConformance } from './chunk-L4YKW5AY.js';
-import { matchPin } from './chunk-OHNKYHRR.js';
+import { openLibrary } from './chunk-BEYNH2LS.js';
+import { S, logErr, logWarn, logOk, logInfo } from './chunk-T2VBPVWZ.js';
+import { getKindRegistry } from './chunk-JJJ2G4K2.js';
+import { resolveBinding, UnknownCompositionKindError } from './chunk-SJNZ4WPA.js';
+import { filesEntryPaths, assetRootProblem, stampManifestPublisher, upsertManifestAsset, resolveIndexPath, LocalIndex } from './chunk-6CYKTHNT.js';
+import { SEMVER_RE, invalidSemverMessage, flowFileNameError, parseManifestFile, requiresSyntaxErrors } from './chunk-MAZ3UXJK.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-76FLBODK.js';
+import { checkAgentSkillConformance } from './chunk-BRK5QZKI.js';
+import { matchPin } from './chunk-KJSC2ZD6.js';
 import { globalSkaileDir } from './chunk-L5VSIPQD.js';
-import { SKAILE_MANIFEST_FILENAME, loadSkPublishManifest, listLibrarySymlinks, librarySymlinkMessage } from './chunk-UQ33Y4DM.js';
-import { readAssetBody, parseFrontmatter } from './chunk-GOCWM7FN.js';
-import { parseDep, PUBLISHER_HINT, isValidPublisher, slugifyAssetName, parseAssetRef, isValidAssetName, ASSET_NAME_HINT, assetRefProblem } from './chunk-WDXZA7QW.js';
+import { SKAILE_MANIFEST_FILENAME, loadSkPublishManifest, expandManifestFiles, listLibrarySymlinks, librarySymlinkMessage } from './chunk-KPKGWXUJ.js';
+import { readAssetBody, parseFrontmatter } from './chunk-JMIKGAUA.js';
+import { parseDep, PUBLISHER_HINT, isValidPublisher, isValidAssetName, slugifyAssetName, parseAssetRef, ASSET_NAME_HINT, assetRefProblem } from './chunk-IZQOO6MN.js';
 import * as fs3 from 'fs';
 import { rmSync, readFileSync, mkdirSync, writeFileSync, chmodSync } from 'fs';
 import * as path3 from 'path';
@@ -174,6 +174,30 @@ function agentComposeRefProblems(parsed, context) {
   });
   return out;
 }
+var MANIFEST_GLOBS = [
+  { pattern: /\/SKILL\.md$/, kind: "skill" },
+  { pattern: /\/agent\.yaml$/, kind: "agent" },
+  { pattern: /\/AGENT\.md$/, kind: "agent" },
+  { pattern: /\/CONTRACT\.md$/, kind: "contract" },
+  { pattern: /\/PERSONA\.md$/, kind: "persona" },
+  { pattern: /\/RULESET\.md$/, kind: "ruleset" },
+  { pattern: /\.flow\.yaml$/, kind: "flow" },
+  { pattern: /\.flow\.json$/, kind: "flow" },
+  { pattern: /\/CONNECTOR\.md$/, kind: "connector" },
+  { pattern: /\.prompt\.md$/, kind: "prompt" },
+  { pattern: /\/MCP\.md$/, kind: "mcp-server" },
+  { pattern: /\/knowledge\.yaml$/, kind: "knowledge" },
+  { pattern: /\/knowledge\/index\.yaml$/, kind: "knowledge" },
+  { pattern: /\.preset\.yaml$/, kind: "preset" }
+];
+function manifestKindOfFile(file) {
+  const posix = file.split(path3.sep).join("/");
+  const hay = posix.startsWith("/") ? posix : `/${posix}`;
+  return MANIFEST_GLOBS.find((g) => g.pattern.test(hay))?.kind;
+}
+function manifestParseErrorText(reason) {
+  return `parse error: ${reason}`;
+}
 var UNUSED_KEYS = ["reads_from", "writes_to"];
 function checkAssetManifest(kind, parsed, body, dirName) {
   const out = [];
@@ -277,6 +301,18 @@ function scaffoldDefinitionErrors(kind, parsed) {
       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") {
@@ -341,6 +377,31 @@ function referenceErrors(kind, parsed, registry, fileName) {
   }
   return out;
 }
+function nameDescriptionProblem(kind, parsed, dirName, e) {
+  if (e.path === "name") {
+    const name = parsed.name;
+    if ((name ?? null) === null) {
+      return `name is required: add \`name: ${dirName}\` (lowercase kebab-case, the name of the ${kind})`;
+    }
+    if (typeof name !== "string") {
+      return `name must be text: write \`name: ${dirName}\` (lowercase kebab-case, the name of the ${kind})`;
+    }
+    return void 0;
+  }
+  if (e.path === "description") {
+    const description = parsed.description;
+    if ((description ?? null) === null) {
+      return `description is required: say what the ${kind} does and when to use it`;
+    }
+    if (typeof description !== "string") {
+      return `description must be text: say what the ${kind} does and when to use it`;
+    }
+    if (description.trim() === "" && !e.message.startsWith("description is empty")) {
+      return `description: description is empty \u2014 say what the ${kind} does and when to use it`;
+    }
+  }
+  return void 0;
+}
 function checkAsset(kind, parsed, body, dirName, registry, rawText, fileName, agentFiles) {
   const errors = [];
   const error2 = (text) => {
@@ -355,6 +416,7 @@ function checkAsset(kind, parsed, body, dirName, registry, rawText, fileName, ag
   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 {
@@ -367,8 +429,10 @@ function checkAsset(kind, parsed, body, dirName, registry, rawText, fileName, ag
           }
           continue;
         }
-        if (e.path === "description" && (parsed.description ?? null) === null) {
-          error2(`description is required: say what the ${kind} does and when to use it`);
+        reported.add(e.path);
+        const plain = nameDescriptionProblem(kind, parsed, dirName, e);
+        if (plain) {
+          error2(plain);
           continue;
         }
         error2(`${e.path ? `${e.path}: ` : ""}${e.message}`);
@@ -384,9 +448,22 @@ function checkAsset(kind, parsed, body, dirName, registry, rawText, fileName, ag
   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");
+      }
+    }
+    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}`);
       }
     }
@@ -406,6 +483,16 @@ function checkAsset(kind, parsed, body, dirName, registry, rawText, fileName, ag
   }
   return { errors, advisories };
 }
+function flowMissingSchemaVersionError(parsed) {
+  if ("schemaVersion" in parsed || !Array.isArray(parsed.nodes)) return void 0;
+  for (const node of parsed.nodes) {
+    if (typeof node !== "object" || node === null || !("run" in node)) continue;
+    if (typeof node.run !== "object" || node.run === null) continue;
+    const id = "id" in node ? String(node.id) : "?";
+    return `schemaVersion is missing: node "${id}" has a run: block, the v2 shape; add schemaVersion: 2 as the first line (a v2 flow takes defaults:, not globals:)`;
+  }
+  return void 0;
+}
 function findAssetManifestFile(dir, kind, name) {
   const candidates = [];
   if (kind === "skill") candidates.push("SKILL.md");
@@ -462,6 +549,25 @@ function checkAssetFile(file, kind, registry, libraryRoot, composeRefs) {
   );
   return [...checked.errors, ...checked.advisories.filter((f) => f.scaffoldBody)];
 }
+function libraryAssetRefusals(asset, libraryPath, registry) {
+  let file = asset.source;
+  try {
+    if (fs3.statSync(file).isDirectory()) file = findAssetManifestFile(file, asset.kind, asset.name);
+  } catch {
+    return [];
+  }
+  if (!file || manifestKindOfFile(file) !== asset.kind) return [];
+  const rel = path3.relative(libraryPath, file).split(path3.sep).join("/");
+  try {
+    const parsed = parseManifestFile(file, fs3.readFileSync(file, "utf8"));
+    if (!parsed.ok) return [`${rel} [${asset.kind}] \u2014 ${manifestParseErrorText(parsed.reason)}`];
+  } catch {
+    return [];
+  }
+  return checkAssetFile(file, asset.kind, registry, libraryPath).map(
+    (f) => `${rel} [${f.tag}] \u2014 ${f.text}`
+  );
+}
 function unheldRequiresOfFile(file, kind, context) {
   try {
     const res = parseManifestFile(file, fs3.readFileSync(file, "utf8"));
@@ -821,7 +927,7 @@ function manifestRequiresProblems(dir) {
     for (const message of requiresSyntaxErrors(entries)) {
       problems.push({
         asset: `${String(kind)}:${String(name)}`,
-        message: `ManifestRequiresError: ${message}`
+        message
       });
     }
   }
@@ -957,9 +1063,14 @@ function exitOnManifestPublisherProblems(dir, libName) {
   process.exit(1);
 }
 function manifestEntryNameProblems(dir) {
+  return manifestEntryRenames(dir).map(
+    (r) => `${SKAILE_MANIFEST_FILENAME} [assets] ${r.kind}:${r.name} \u2014 ${r.file} now names "${r.now}": the entry is a ref \`add\` cannot resolve. Run \`skaile library index <library>\` to rename it, or restore the name.`
+  );
+}
+function manifestEntryRenames(dir) {
   const manifest = loadSkPublishManifest(dir);
   if (!manifest || manifest.source) return [];
-  const problems = [];
+  const renames = [];
   for (const entry of manifest.assets ?? []) {
     if (!entry.root || entry.source) continue;
     if (assetRootProblem(dir, entry.root)) continue;
@@ -974,11 +1085,17 @@ function manifestEntryNameProblems(dir) {
       continue;
     }
     if (typeof parsedName !== "string" || parsedName === "" || parsedName === entry.name) continue;
-    problems.push(
-      `${SKAILE_MANIFEST_FILENAME} [assets] ${entry.kind}:${entry.name} \u2014 ${path3.relative(dir, file)} now names "${parsedName}": the entry is a ref \`add\` cannot resolve. Run \`skaile library index <library>\` to rename it, or restore the name.`
-    );
+    if (!isValidAssetName(parsedName)) continue;
+    const publisher = entry.publisher ?? manifest.publisher;
+    renames.push({
+      kind: entry.kind,
+      name: entry.name,
+      ...publisher ? { publisher } : {},
+      file: path3.relative(dir, file),
+      now: parsedName
+    });
   }
-  return problems;
+  return renames;
 }
 function manifestRootProblems(dir) {
   let doc;
@@ -1004,8 +1121,34 @@ function manifestRootProblems(dir) {
   }
   return problems;
 }
-function exitOnManifestRootProblems(dir, verb) {
-  const problems = manifestRootProblems(dir);
+function manifestFilesProblems(dir) {
+  const manifest = loadSkPublishManifest(dir);
+  if (!manifest || manifest.source) return [];
+  const problems = [];
+  for (const a of manifest.assets ?? []) {
+    if (a.source || !Array.isArray(a.files) || a.files.length === 0) continue;
+    const ref = assetEntryRef(a, manifest);
+    const patterns = a.files.map((f) => typeof f === "string" ? f : f.path).filter(Boolean);
+    const outside = patterns.filter(
+      (p) => path3.isAbsolute(p) || p.split(/[\\/]+/).includes("..") || /^[a-zA-Z]:/.test(p)
+    );
+    if (outside.length > 0) {
+      problems.push(
+        `${SKAILE_MANIFEST_FILENAME} [assets] ${ref} \u2014 files: ${outside.join(", ")} leaves the library: \`add\` ignores it and a published repo has no file for it. Remove the entry from the \`files:\` list or move the file into the library.`
+      );
+      continue;
+    }
+    const { unmatched } = expandManifestFiles(dir, patterns);
+    if (unmatched.length > 0) {
+      problems.push(
+        `${SKAILE_MANIFEST_FILENAME} [assets] ${ref} \u2014 files: ${unmatched.join(", ")} not found. Fix the \`files:\` list or drop the entry.`
+      );
+    }
+  }
+  return problems;
+}
+function exitOnManifestPathProblems(dir, verb) {
+  const problems = [...manifestRootProblems(dir), ...manifestFilesProblems(dir)];
   if (problems.length === 0) return;
   for (const message of problems) logErr(message);
   logErr(`Fix or remove the entry in ${SKAILE_MANIFEST_FILENAME}, then re-run \`${verb}\`.`);
@@ -1022,17 +1165,24 @@ function exitOnLibrarySymlinks(dir, verb) {
   logErr(`${count} symlink(s) in ${dir}: remove them, then re-run \`${verb}\`.`);
   process.exit(1);
 }
+function manifestParseFailureLine(libPath, rel) {
+  const file = path3.join(libPath, rel);
+  try {
+    if (!fs3.statSync(file).isFile()) return void 0;
+    const res = parseManifestFile(file, fs3.readFileSync(file, "utf-8"));
+    if (res.ok) return void 0;
+    const kind = manifestKindOfFile(file) ?? "asset";
+    return `${rel.split(path3.sep).join("/")} [${kind}] \u2014 ${manifestParseErrorText(res.reason)}`;
+  } catch {
+    return void 0;
+  }
+}
 function parseFailureText(libPath, error2) {
   const m = /^(.*): ManifestSchemaError: failed to parse manifest file (.+)$/.exec(error2);
   if (!m) return error2;
   for (const rel of [m[1], path3.join(m[1], m[2]), m[2]]) {
-    const file = path3.join(libPath, rel);
-    try {
-      if (!fs3.statSync(file).isFile()) continue;
-      const res = parseManifestFile(file, fs3.readFileSync(file, "utf-8"));
-      if (!res.ok) return `${rel.split(path3.sep).join("/")}: ${res.reason}`;
-    } catch {
-    }
+    const line = manifestParseFailureLine(libPath, rel);
+    if (line) return line;
   }
   return error2;
 }
@@ -1184,6 +1334,6 @@ function warnLegacyBundleFiles(root) {
   return files.length;
 }
 
-export { DOMAIN_REMOVED_HINT, assetEntryRef, assetNamespace, bareName, checkAsset, checkAssetFile, checkComposeKinds, composeRefContext, declaredVersion, deviceLogin, duplicateAssetMessage, exitOnEmptyAsset, exitOnLibrarySymlinks, exitOnManifestPublisherProblems, exitOnManifestRootProblems, findAssetManifestFile, findEntryManifestFile, formatNamespaceAvailability, getAccessToken, getStoreIdentity, hasStoredAuth, indexLibrary, invalidPublisherMessage, invalidSemverText, loadStoreAuthConfig, logout, manifestDependencyProblems, manifestEntryNameProblems, manifestNamespaces, manifestPublisherProblems, manifestRequiresProblems, manifestRootProblems, manifestVersionMismatchMessage, plannedRefs, preflightNamespace, publishToStore, refFor, registerLibraryAsset, registeredAssetKeys, rejectRemovedDomainOption, removedDomainOption, reportLibrarySymlinks, requireValidPublisherOrExit, resolveLibraryPublisherDetailed, unheldRequiresOfFile, unresolvedManifestDependencies, warnIgnoredDomainFiles, warnLegacyBundleFiles, warnLibraryNameFallback };
-//# sourceMappingURL=chunk-TKHJXPSB.js.map
-//# sourceMappingURL=chunk-TKHJXPSB.js.map
+export { DOMAIN_REMOVED_HINT, MANIFEST_GLOBS, assetEntryRef, assetNamespace, bareName, checkAsset, checkAssetFile, checkComposeKinds, composeRefContext, declaredVersion, deviceLogin, duplicateAssetMessage, exitOnEmptyAsset, exitOnLibrarySymlinks, exitOnManifestPathProblems, exitOnManifestPublisherProblems, findAssetManifestFile, findEntryManifestFile, formatNamespaceAvailability, getAccessToken, getStoreIdentity, hasStoredAuth, indexLibrary, invalidPublisherMessage, invalidSemverText, libraryAssetRefusals, loadStoreAuthConfig, logout, manifestDependencyProblems, manifestEntryNameProblems, manifestEntryRenames, manifestFilesProblems, manifestNamespaces, manifestParseErrorText, manifestParseFailureLine, manifestPublisherProblems, manifestRequiresProblems, manifestRootProblems, manifestVersionMismatchMessage, plannedRefs, preflightNamespace, publishToStore, refFor, registerLibraryAsset, registeredAssetKeys, rejectRemovedDomainOption, removedDomainOption, reportLibrarySymlinks, requireValidPublisherOrExit, resolveLibraryPublisherDetailed, unheldRequiresOfFile, unresolvedManifestDependencies, warnIgnoredDomainFiles, warnLegacyBundleFiles, warnLibraryNameFallback };
+//# sourceMappingURL=chunk-2XM3ZH36.js.map
+//# sourceMappingURL=chunk-2XM3ZH36.js.map