@skaile/workspaces 1.3.0 → 1.5.0

package/dist/{chunk-BZMTR5WW.js → chunk-MZV6WCF3.js}

@@ -1,8 +1,8 @@
 import { AssetKindRegistryFrozenError, AssetKindConflictError, AssetKindPatternConflictError, AssetKindPathAmbiguityError } from './chunk-JXT46QFC.js';
-import { decodeSkaileManifest, SKAILE_MANIFEST_FILENAME } from './chunk-DZCK7GRW.js';
+import { decodeSkaileManifest, SKAILE_MANIFEST_FILENAME } from './chunk-P4VHQA4N.js';
 import { parseFrontmatter } from './chunk-BQCGFPHW.js';
 import { isValidAssetName } from './chunk-GEEW6SY2.js';
-import { validatePersona, validateRuleset, validateSkill, validateAgent, validateConnector, validateMcpServer, validateContract, validatePrompt, validatePreset } from './chunk-GB4SEEK4.js';
+import { validatePersona, validateRuleset, validateSkill, validateAgent, validateConnector, validateMcpServer, validateContract, validatePrompt, validatePreset } from './chunk-IRORTHGA.js';
 import * as fs from 'fs';
 import * as path from 'path';
 import * as yaml from 'js-yaml';
@@ -597,7 +597,7 @@ var SourceConfigSchema = z.object({
    * project itself (e.g. `ai-assets-dev/`). Excluded from a normal discovery
    * run; re-included only when discovery is invoked with `includeDev` (the
    * `--dev` flag on `skaile source sync` / `source add`). Conventionally a
-   * single top-level folder. See `docs/discovery/dev-assets.md`.
+   * single top-level folder. See `docs/sections/discovery/dev-assets.md`.
    */
   dev_paths: z.array(z.string().min(1)).default([]),
   /** Optional sync configuration for the catalog backend. */
@@ -656,6 +656,9 @@ function collectWarnings(config) {
   }
   return warnings;
 }
+function barePublisher(publisher) {
+  return publisher.startsWith("@") ? publisher.slice(1) : publisher;
+}
 function atPublisher(publisher) {
   return publisher.startsWith("@") ? publisher : `@${publisher}`;
 }
@@ -768,13 +771,19 @@ function mergeSourceConfigs(base, overlay) {
   return merged;
 }
 function loadMergedSourceConfig(rootPath, sidecarPath) {
+  return loadMergedSourceConfigFromFile(
+    rootPath,
+    sidecarPath ? path.join(sidecarPath, SKAILE_MANIFEST_FILENAME) : void 0
+  );
+}
+function loadMergedSourceConfigFromFile(rootPath, overlayManifestFile) {
   const baseResult = loadSourceConfigFromDir(rootPath);
   if (baseResult && !baseResult.ok) return baseResult;
   const base = baseResult?.ok ? baseResult.config : void 0;
   let overlay;
   const overlayWarnings = [];
-  if (sidecarPath) {
-    const overlayResult = loadSourceConfigFromDir(sidecarPath);
+  if (overlayManifestFile) {
+    const overlayResult = loadSourceConfigFromFile(overlayManifestFile);
     if (overlayResult && !overlayResult.ok) return overlayResult;
     if (overlayResult?.ok) {
       overlay = overlayResult.config;
@@ -799,7 +808,9 @@ function loadMergedSourceConfig(rootPath, sidecarPath) {
   return warnings.length > 0 ? { ok: true, config: merged, warnings } : { ok: true, config: merged };
 }
 function loadSourceConfigFromDir(dir) {
-  const filePath = path.join(dir, SKAILE_MANIFEST_FILENAME);
+  return loadSourceConfigFromFile(path.join(dir, SKAILE_MANIFEST_FILENAME));
+}
+function loadSourceConfigFromFile(filePath) {
   if (!fs.existsSync(filePath)) return void 0;
   try {
     const text = fs.readFileSync(filePath, "utf-8");
@@ -821,12 +832,14 @@ function loadSourceConfigFromDir(dir) {
 function discoverFromManifest(rootPath, sourceConfig, registry, includeDev = false) {
   const assets = [];
   const errors = [];
+  const warnings = [];
   const allEdges = [];
   const devPaths = sourceConfig.dev_paths ?? [];
   const sortedEntries = [...sourceConfig.assets].sort((a, b) => a.path < b.path ? -1 : 1);
   for (const entry of sortedEntries) {
     if (!includeDev && isExcludedDevPath(entry.path, devPaths)) continue;
     const result = processEntry(rootPath, entry, registry);
+    warnings.push(...result.warnings);
     if (result.errors.length > 0) {
       errors.push(...result.errors);
       if (!result.asset) continue;
@@ -837,10 +850,11 @@ function discoverFromManifest(rootPath, sourceConfig, registry, includeDev = fal
     }
   }
   const graph = buildRequiresGraph(allEdges);
-  return { assets, graph, errors, warnings: [] };
+  return { assets, graph, errors, warnings };
 }
 function processEntry(rootPath, entry, registry) {
   const errors = [];
+  const warnings = [];
   const assetRoot = path.resolve(rootPath, entry.path);
   const assetRootRelative = entry.path;
   const files = entry.files ?? expandRootFiles(assetRoot);
@@ -849,7 +863,7 @@ function processEntry(rootPath, entry, registry) {
       path: assetRootRelative,
       error: `ManifestFileMissing: root "${entry.root ?? entry.path}" contains no files`
     });
-    return { asset: null, errors };
+    return { asset: null, errors, warnings };
   }
   const manifestFileRel = pickManifestFile(entry.path, files, entry.kind, registry);
   const manifestAbsPath = path.join(assetRoot, manifestFileRel);
@@ -864,7 +878,7 @@ function processEntry(rootPath, entry, registry) {
       path: assetRootRelative,
       error: `ManifestFileMissing: ${missing.join(", ")}`
     });
-    return { asset: null, errors };
+    return { asset: null, errors, warnings };
   }
   const manifest = parseManifest(manifestAbsPath);
   if (!manifest) {
@@ -872,12 +886,12 @@ function processEntry(rootPath, entry, registry) {
       path: assetRootRelative,
       error: `ManifestSchemaError: failed to parse manifest file ${manifestFileRel}`
     });
-    return { asset: null, errors };
+    return { asset: null, errors, warnings };
   }
   if (!registry.getProvider(entry.kind)) {
-    errors.push({
+    warnings.push({
       path: assetRootRelative,
-      error: `Unknown asset kind "${entry.kind}" \u2014 ghost-installing without provider validation`
+      warning: `Unknown asset kind "${entry.kind}" \u2014 ghost-installing without provider validation`
     });
   }
   const requiresErrors = validateRequiresSyntax(entry.requires);
@@ -886,7 +900,7 @@ function processEntry(rootPath, entry, registry) {
       path: assetRootRelative,
       error: `ManifestRequiresError: ${requiresErrors.join("; ")}`
     });
-    return { asset: null, errors };
+    return { asset: null, errors, warnings };
   }
   const { sha256: computedHash, missing: hashMissing } = computeDeterministicHashDetailed(
     assetRoot,
@@ -897,22 +911,24 @@ function processEntry(rootPath, entry, registry) {
       path: assetRootRelative,
       error: `ManifestFileMissing: ${hashMissing.join(", ")} (became unreadable during hashing)`
     });
-    return { asset: null, errors };
+    return { asset: null, errors, warnings };
   }
   if (entry.sha256 && entry.sha256 !== computedHash) {
     errors.push({
       path: assetRootRelative,
       error: `ManifestHashMismatch: declared ${entry.sha256}, computed ${computedHash}`
     });
-    return { asset: null, errors };
+    return { asset: null, errors, warnings };
   }
   if (!entry.sha256) {
-    errors.push({
+    warnings.push({
       path: assetRootRelative,
-      error: `Missing sha256 \u2014 computed ${computedHash}. Re-run \`skaile source build-manifest --write\` to pin it.`
+      warning: "Missing sha256 \u2014 auto-computed locally (only needed for store publish).",
+      code: "missing_sha256"
     });
   }
-  const ref = `${entry.publisher}/${entry.name}@${entry.version}`;
+  const publisher = barePublisher(entry.publisher);
+  const ref = `@${publisher}/${entry.name}@${entry.version}`;
   const requires = entry.requires.map((to) => ({
     from: ref,
     to,
@@ -925,7 +941,7 @@ function processEntry(rootPath, entry, registry) {
   const asset = {
     ref,
     kind: entry.kind,
-    publisher: entry.publisher,
+    publisher,
     name: entry.name,
     version: entry.version,
     manifest: mergedManifest,
@@ -938,7 +954,7 @@ function processEntry(rootPath, entry, registry) {
     sha256: computedHash,
     requires
   };
-  return { asset, errors };
+  return { asset, errors, warnings };
 }
 var MANIFEST_FILENAMES = /* @__PURE__ */ new Set([
   "SKILL.md",
@@ -1461,13 +1477,15 @@ function normalizeDiscoveryArgs(rootPath, optionsOrSourceConfig) {
   let sourceConfig;
   let registry;
   let sidecarPath;
+  let sidecarManifestFile;
   let includeDev = false;
   let domainFiles;
   let atomization = DEFAULT_ATOMIZATION_POLICY;
-  if (optionsOrSourceConfig && ("registry" in optionsOrSourceConfig || "sourceConfig" in optionsOrSourceConfig || "sidecarPath" in optionsOrSourceConfig || "domainFiles" in optionsOrSourceConfig || "atomization" in optionsOrSourceConfig || "includeDev" in optionsOrSourceConfig)) {
+  if (optionsOrSourceConfig && ("registry" in optionsOrSourceConfig || "sourceConfig" in optionsOrSourceConfig || "sidecarPath" in optionsOrSourceConfig || "sidecarManifestFile" in optionsOrSourceConfig || "domainFiles" in optionsOrSourceConfig || "atomization" in optionsOrSourceConfig || "includeDev" in optionsOrSourceConfig)) {
     sourceConfig = optionsOrSourceConfig.sourceConfig;
     registry = optionsOrSourceConfig.registry;
     sidecarPath = optionsOrSourceConfig.sidecarPath;
+    sidecarManifestFile = optionsOrSourceConfig.sidecarManifestFile;
     includeDev = optionsOrSourceConfig.includeDev ?? false;
     domainFiles = optionsOrSourceConfig.domainFiles;
     atomization = optionsOrSourceConfig.atomization ?? DEFAULT_ATOMIZATION_POLICY;
@@ -1477,13 +1495,21 @@ function normalizeDiscoveryArgs(rootPath, optionsOrSourceConfig) {
   if (!registry) {
     registry = createDefaultRegistry();
   }
-  if (!sourceConfig && sidecarPath) {
-    const merged = loadMergedSourceConfig(rootPath, sidecarPath);
+  if (!sourceConfig && (sidecarManifestFile || sidecarPath)) {
+    const merged = sidecarManifestFile ? loadMergedSourceConfigFromFile(rootPath, sidecarManifestFile) : loadMergedSourceConfig(rootPath, sidecarPath);
     if (merged.ok) {
       sourceConfig = merged.config;
     }
   }
-  return { sourceConfig, registry, includeDev, sidecarPath, domainFiles, atomization };
+  return {
+    sourceConfig,
+    registry,
+    includeDev,
+    sidecarPath,
+    sidecarManifestFile,
+    domainFiles,
+    atomization
+  };
 }
 function processDiscoveredEntry(entry, ctx, acc) {
   const { absPath, relativePath, kind } = entry;
@@ -1497,7 +1523,7 @@ function processDiscoveredEntry(entry, ctx, acc) {
     const publisher = resolvePublisher(relativePath, sourceConfig);
     const name = deriveName(relativePath, kind, manifest);
     const version = coerceVersion(manifest);
-    const ref = `${publisher}/${name}@${version}`;
+    const ref = `@${publisher}/${name}@${version}`;
     const fileFilter = registry.defaultFileFilter(kind, absPath) ?? {
       assetRoot: path.dirname(absPath),
       files: [path.basename(absPath)]
@@ -1630,19 +1656,19 @@ function parseManifest2(absPath) {
 function resolvePublisher(relativePath, sourceConfig) {
   if (!sourceConfig) {
     const domain = relativePath.split("/")[0] ?? "unknown";
-    return `@${domain}`;
+    return domain;
   }
   for (const override of sourceConfig.publisher_overrides) {
     if (relativePath.startsWith(override.path)) {
-      return override.publisher;
+      return barePublisher(override.publisher);
     }
   }
   const template = sourceConfig.publisher_default;
   if (template.includes("<domain>")) {
     const domain = relativePath.split("/")[0] ?? "unknown";
-    return template.replace("<domain>", domain);
+    return barePublisher(template.replace("<domain>", domain));
   }
-  return template;
+  return barePublisher(template);
 }
 function coerceVersion(manifest) {
   const scalar = (v) => typeof v === "string" ? v : typeof v === "number" ? String(v) : null;
@@ -1813,7 +1839,7 @@ async function buildAssetFromBlob(relativePath, kind, blobByPath, fetchBlob, res
   const publisher = resolvePublisher2(relativePath, resolvedSourceConfig);
   const name = deriveName2(relativePath, kind, manifest);
   const version = coerceVersion2(manifest);
-  const ref = `${publisher}/${name}@${version}`;
+  const ref = `@${publisher}/${name}@${version}`;
   const fileFilter = computeFileFilter(kind, relativePath, blobByPath);
   warnings.push(
     ...await applyAtomizationPolicyTree(
@@ -1878,19 +1904,19 @@ function parseManifestText(relativePath, text) {
 function resolvePublisher2(relativePath, sourceConfig) {
   if (!sourceConfig) {
     const domain = relativePath.split("/")[0] ?? "unknown";
-    return `@${domain}`;
+    return domain;
   }
   for (const override of sourceConfig.publisher_overrides) {
     if (relativePath.startsWith(override.path)) {
-      return override.publisher;
+      return barePublisher(override.publisher);
     }
   }
   const template = sourceConfig.publisher_default;
   if (template.includes("<domain>")) {
     const domain = relativePath.split("/")[0] ?? "unknown";
-    return template.replace("<domain>", domain);
+    return barePublisher(template.replace("<domain>", domain));
   }
-  return template;
+  return barePublisher(template);
 }
 function coerceVersion2(manifest) {
   const scalar = (v) => typeof v === "string" ? v : typeof v === "number" ? String(v) : null;
@@ -2156,12 +2182,14 @@ async function resolveSourceConfigFromTree(explicit, blobByPath, fetchBlob) {
 async function discoverFromManifestTree(sourceConfig, blobByPath, fetchBlob, registry, includeDev = false) {
   const assets = [];
   const errors = [];
+  const warnings = [];
   const allEdges = [];
   const devPaths = sourceConfig.dev_paths ?? [];
   const sortedEntries = [...sourceConfig.assets].sort((a, b) => a.path < b.path ? -1 : 1);
   for (const entry of sortedEntries) {
     if (!includeDev && isExcludedDevPath(entry.path, devPaths)) continue;
     const result = await processManifestEntryTree(entry, blobByPath, fetchBlob, registry);
+    warnings.push(...result.warnings);
     if (result.errors.length > 0) {
       errors.push(...result.errors);
       if (!result.asset) continue;
@@ -2172,13 +2200,14 @@ async function discoverFromManifestTree(sourceConfig, blobByPath, fetchBlob, reg
     }
   }
   const graph = buildRequiresGraph(allEdges);
-  return { assets, graph, errors, warnings: [] };
+  return { assets, graph, errors, warnings };
 }
 async function processManifestEntryTree(entry, blobByPath, fetchBlob, registry) {
   const errors = [];
+  const warnings = [];
   const assetRootRelative = entry.path;
   const files = resolveEntryFilesTree(entry, blobByPath, errors);
-  if (!files) return { asset: null, errors };
+  if (!files) return { asset: null, errors, warnings };
   const manifestFileRel = pickManifestFileTree(entry.path, files, entry.kind, registry);
   const manifestFull = joinPosix(entry.path, manifestFileRel);
   const manifest = await fetchAndParseManifestTree(
@@ -2189,11 +2218,11 @@ async function processManifestEntryTree(entry, blobByPath, fetchBlob, registry)
     fetchBlob,
     errors
   );
-  if (!manifest) return { asset: null, errors };
+  if (!manifest) return { asset: null, errors, warnings };
   if (!registry.getProvider(entry.kind)) {
-    errors.push({
+    warnings.push({
       path: assetRootRelative,
-      error: `Unknown asset kind "${entry.kind}" \u2014 ghost-installing without provider validation`
+      warning: `Unknown asset kind "${entry.kind}" \u2014 ghost-installing without provider validation`
     });
   }
   const requiresErrors = validateRequiresSyntax2(entry.requires);
@@ -2202,7 +2231,7 @@ async function processManifestEntryTree(entry, blobByPath, fetchBlob, registry)
       path: assetRootRelative,
       error: `ManifestRequiresError: ${requiresErrors.join("; ")}`
     });
-    return { asset: null, errors };
+    return { asset: null, errors, warnings };
   }
   const computedHash = await computeAndVerifyHashTree(
     entry,
@@ -2210,10 +2239,12 @@ async function processManifestEntryTree(entry, blobByPath, fetchBlob, registry)
     assetRootRelative,
     blobByPath,
     fetchBlob,
-    errors
+    errors,
+    warnings
   );
-  if (computedHash === null) return { asset: null, errors };
-  const ref = `${entry.publisher}/${entry.name}@${entry.version}`;
+  if (computedHash === null) return { asset: null, errors, warnings };
+  const publisher = barePublisher(entry.publisher);
+  const ref = `@${publisher}/${entry.name}@${entry.version}`;
   const requires = entry.requires.map((to) => ({
     from: ref,
     to,
@@ -2226,7 +2257,7 @@ async function processManifestEntryTree(entry, blobByPath, fetchBlob, registry)
   const asset = {
     ref,
     kind: entry.kind,
-    publisher: entry.publisher,
+    publisher,
     name: entry.name,
     version: entry.version,
     manifest: mergedManifest,
@@ -2239,7 +2270,7 @@ async function processManifestEntryTree(entry, blobByPath, fetchBlob, registry)
     sha256: computedHash,
     requires
   };
-  return { asset, errors };
+  return { asset, errors, warnings };
 }
 function resolveEntryFilesTree(entry, blobByPath, errors) {
   const assetRootRelative = entry.path;
@@ -2294,7 +2325,7 @@ async function fetchAndParseManifestTree(manifestFull, manifestFileRel, assetRoo
   }
   return manifest;
 }
-async function computeAndVerifyHashTree(entry, files, assetRootRelative, blobByPath, fetchBlob, errors) {
+async function computeAndVerifyHashTree(entry, files, assetRootRelative, blobByPath, fetchBlob, errors, warnings) {
   let computedHash;
   try {
     computedHash = await computeManifestHashTree(entry.path, files, blobByPath, fetchBlob);
@@ -2313,9 +2344,10 @@ async function computeAndVerifyHashTree(entry, files, assetRootRelative, blobByP
     return null;
   }
   if (!entry.sha256) {
-    errors.push({
+    warnings.push({
       path: assetRootRelative,
-      error: `Missing sha256 \u2014 computed ${computedHash}. Re-run \`skaile source build-manifest --write\` to pin it.`
+      warning: "Missing sha256 \u2014 auto-computed locally (only needed for store publish).",
+      code: "missing_sha256"
     });
   }
   return computedHash;
@@ -2385,6 +2417,6 @@ function validateRequiresSyntax2(requires) {
 // discovery/src/index.ts
 var VERSION = "0.0.0";
 
-export { AGENT_COMPANION_DIRS, AssetKindRegistry, BUILTIN_PROVIDERS, DEFAULT_ATOMIZATION_POLICY, DOMAIN_MD_FILENAME, SEMVER_RE, SKILL_COMPANION_DIRS, SourceConfigSchema, VERSION, agentProvider, assignDomain, buildDomainIndex, buildRequiresGraph, closeOverReferences, computeDeterministicHash, computeDeterministicHashDetailed, connectorProvider, contractProvider, createDefaultRegistry, createNodeAssetFs, decodeManifestSourceConfig, detectCycles, discoverAssetsInTree, discoverAssetsInTreeEntries, discoverFromManifest, extractPathCandidates, extractRequires, extractSlashCommands, isDomainMdPath, isExcludedDevPath, isManifestMode, lintUncoveredReferences, loadMergedSourceConfig, loadSourceConfig, manifestToSourceConfig, mcpServerProvider, mergeSourceConfigs, parseDomainMd, parseSourceConfig, personaProvider, presetProvider, promptProvider, readManifestSourceConfig, rulesetProvider, scanAssetReferences, scanSlashCommandNames, skillProvider, validateDomains, validateSourceConfig };
-//# sourceMappingURL=chunk-BZMTR5WW.js.map
-//# sourceMappingURL=chunk-BZMTR5WW.js.map
+export { AGENT_COMPANION_DIRS, AssetKindRegistry, BUILTIN_PROVIDERS, DEFAULT_ATOMIZATION_POLICY, DOMAIN_MD_FILENAME, SEMVER_RE, SKILL_COMPANION_DIRS, SourceConfigSchema, VERSION, agentProvider, assignDomain, barePublisher, buildDomainIndex, buildRequiresGraph, closeOverReferences, computeDeterministicHash, computeDeterministicHashDetailed, connectorProvider, contractProvider, createDefaultRegistry, createNodeAssetFs, decodeManifestSourceConfig, detectCycles, discoverAssetsInTree, discoverAssetsInTreeEntries, discoverFromManifest, extractPathCandidates, extractRequires, extractSlashCommands, isDomainMdPath, isExcludedDevPath, isManifestMode, lintUncoveredReferences, loadMergedSourceConfig, loadMergedSourceConfigFromFile, loadSourceConfig, manifestToSourceConfig, mcpServerProvider, mergeSourceConfigs, parseDomainMd, parseSourceConfig, personaProvider, presetProvider, promptProvider, readManifestSourceConfig, rulesetProvider, scanAssetReferences, scanSlashCommandNames, skillProvider, validateDomains, validateSourceConfig };
+//# sourceMappingURL=chunk-MZV6WCF3.js.map
+//# sourceMappingURL=chunk-MZV6WCF3.js.map