@sentry/junior 0.203.0 → 0.205.0

package/dist/cli/check.js

@@ -1,886 +1,593 @@
-import {
-  parseSkillFile
-} from "../chunk-A5L27LE4.js";
-import "../chunk-WCNQJKFT.js";
-import {
-  parseInlinePluginManifest,
-  parsePluginManifest
-} from "../chunk-7GWQ5S5R.js";
-import "../chunk-CEMMLV5L.js";
-import "../chunk-N7TMSOZH.js";
-import "../chunk-DWPN6SCH.js";
-import "../chunk-TXKUKMVO.js";
-import {
-  JUNIOR_CONVERSATION_WORK_CALLBACK_ROUTE,
-  JUNIOR_HEARTBEAT_ROUTE,
-  LEGACY_JUNIOR_CONVERSATION_WORK_FUNCTION
-} from "../chunk-WS7NAFTM.js";
-import "../chunk-PEQXYSSE.js";
-import "../chunk-WDZR3QQM.js";
-
-// src/cli/check.ts
-import fs from "fs/promises";
-import path from "path";
-import { pathToFileURL } from "url";
-var DEFAULT_IO = {
-  info: console.log,
-  warn: console.warn,
-  error: console.error
+import { E as JUNIOR_HEARTBEAT_ROUTE, j as LEGACY_JUNIOR_CONVERSATION_WORK_FUNCTION, w as JUNIOR_CONVERSATION_WORK_CALLBACK_ROUTE } from "../logging-BMJzdDlP.js";
+import { d as parseInlinePluginManifest, f as parsePluginManifest } from "../registry-BA0__8ai.js";
+import { i as parseSkillFile } from "../skills-DJOlYdBs.js";
+import { pathToFileURL } from "node:url";
+import path from "node:path";
+import fs from "node:fs/promises";
+//#region src/cli/check.ts
+const DEFAULT_IO = {
+	info: console.log,
+	warn: console.warn,
+	error: console.error
 };
-var ANSI = {
-  reset: "\x1B[0m",
-  bold: "\x1B[1m",
-  dim: "\x1B[2m",
-  green: "\x1B[32m",
-  yellow: "\x1B[33m",
-  red: "\x1B[31m",
-  cyan: "\x1B[36m"
+const ANSI = {
+	reset: "\x1B[0m",
+	bold: "\x1B[1m",
+	dim: "\x1B[2m",
+	green: "\x1B[32m",
+	yellow: "\x1B[33m",
+	red: "\x1B[31m",
+	cyan: "\x1B[36m"
 };
 function supportsColor(stream) {
-  if (process.env.NO_COLOR !== void 0) {
-    return false;
-  }
-  if (process.env.FORCE_COLOR && process.env.FORCE_COLOR !== "0") {
-    return true;
-  }
-  return Boolean(stream?.isTTY);
-}
-var COLOR_ENABLED = supportsColor(process.stdout) || supportsColor(process.stderr);
+	if (process.env.NO_COLOR !== void 0) return false;
+	if (process.env.FORCE_COLOR && process.env.FORCE_COLOR !== "0") return true;
+	return Boolean(stream?.isTTY);
+}
+const COLOR_ENABLED = supportsColor(process.stdout) || supportsColor(process.stderr);
 function color(text, ...codes) {
-  if (!COLOR_ENABLED || codes.length === 0) {
-    return text;
-  }
-  return `${codes.join("")}${text}${ANSI.reset}`;
+	if (!COLOR_ENABLED || codes.length === 0) return text;
+	return `${codes.join("")}${text}${ANSI.reset}`;
 }
 function contentRoot(rootDir, subdir) {
-  return path.resolve(rootDir, "app", subdir);
+	return path.resolve(rootDir, "app", subdir);
 }
 async function pathIsDirectory(targetPath) {
-  try {
-    return (await fs.stat(targetPath)).isDirectory();
-  } catch {
-    return false;
-  }
+	try {
+		return (await fs.stat(targetPath)).isDirectory();
+	} catch {
+		return false;
+	}
 }
 async function pathIsFile(targetPath) {
-  try {
-    return (await fs.stat(targetPath)).isFile();
-  } catch {
-    return false;
-  }
+	try {
+		return (await fs.stat(targetPath)).isFile();
+	} catch {
+		return false;
+	}
 }
 async function readJsonFile(targetPath) {
-  try {
-    return JSON.parse(await fs.readFile(targetPath, "utf8"));
-  } catch {
-    return void 0;
-  }
+	try {
+		return JSON.parse(await fs.readFile(targetPath, "utf8"));
+	} catch {
+		return;
+	}
 }
 async function readTextFile(targetPath) {
-  try {
-    return await fs.readFile(targetPath, "utf8");
-  } catch {
-    return void 0;
-  }
+	try {
+		return await fs.readFile(targetPath, "utf8");
+	} catch {
+		return;
+	}
 }
 async function readRootPackageJson(rootDir) {
-  return await readJsonFile(path.join(rootDir, "package.json"));
+	return await readJsonFile(path.join(rootDir, "package.json"));
 }
 function packageInstallDir(rootDir, packageName) {
-  return path.join(rootDir, "node_modules", ...packageName.split("/"));
+	return path.join(rootDir, "node_modules", ...packageName.split("/"));
 }
 function declaredPackages(pkg) {
-  if (!pkg) {
-    return [];
-  }
-  const packages = /* @__PURE__ */ new Map();
-  for (const deps of [
-    pkg.dependencies,
-    pkg.optionalDependencies,
-    pkg.devDependencies
-  ]) {
-    for (const [name, spec] of Object.entries(deps ?? {})) {
-      if (!packages.has(name)) {
-        packages.set(name, spec);
-      }
-    }
-  }
-  return [...packages.entries()].map(([name, spec]) => ({ name, spec })).sort((left, right) => left.name.localeCompare(right.name));
+	if (!pkg) return [];
+	const packages = /* @__PURE__ */ new Map();
+	for (const deps of [
+		pkg.dependencies,
+		pkg.optionalDependencies,
+		pkg.devDependencies
+	]) for (const [name, spec] of Object.entries(deps ?? {})) if (!packages.has(name)) packages.set(name, spec);
+	return [...packages.entries()].map(([name, spec]) => ({
+		name,
+		spec
+	})).sort((left, right) => left.name.localeCompare(right.name));
 }
 async function readInstalledPackageVersion(rootDir, packageName) {
-  return (await readJsonFile(
-    path.join(packageInstallDir(rootDir, packageName), "package.json")
-  ))?.version;
+	return (await readJsonFile(path.join(packageInstallDir(rootDir, packageName), "package.json")))?.version;
 }
 async function packageHasPluginContent(rootDir, packageName) {
-  const packageDir = packageInstallDir(rootDir, packageName);
-  return await pathIsFile(path.join(packageDir, "plugin.yaml")) || await pathIsDirectory(path.join(packageDir, "plugins")) || await pathIsDirectory(path.join(packageDir, "skills"));
+	const packageDir = packageInstallDir(rootDir, packageName);
+	return await pathIsFile(path.join(packageDir, "plugin.yaml")) || await pathIsDirectory(path.join(packageDir, "plugins")) || await pathIsDirectory(path.join(packageDir, "skills"));
 }
 function comparableVersion(version) {
-  return version?.match(/\d+\.\d+\.\d+(?:[-+][0-9A-Za-z.-]+)?/)?.[0];
+	return version?.match(/\d+\.\d+\.\d+(?:[-+][0-9A-Za-z.-]+)?/)?.[0];
 }
 async function collectPackageWarnings(rootDir, packages) {
-  const warnings = [];
-  const packageSpecByName = new Map(
-    packages.map((declaredPackage) => [
-      declaredPackage.name,
-      declaredPackage.spec
-    ])
-  );
-  const coreVersion = comparableVersion(
-    await readInstalledPackageVersion(rootDir, "@sentry/junior") ?? packageSpecByName.get("@sentry/junior")
-  );
-  if (!coreVersion) {
-    return warnings;
-  }
-  for (const declaredPackage of packages) {
-    if (!declaredPackage.name.startsWith("@sentry/junior-")) {
-      continue;
-    }
-    if (!await packageHasPluginContent(rootDir, declaredPackage.name)) {
-      continue;
-    }
-    const pluginVersion = comparableVersion(
-      await readInstalledPackageVersion(rootDir, declaredPackage.name) ?? declaredPackage.spec
-    );
-    if (pluginVersion && pluginVersion !== coreVersion) {
-      warnings.push(
-        `${path.join(rootDir, "package.json")}: ${declaredPackage.name} version ${pluginVersion} does not match @sentry/junior version ${coreVersion}`
-      );
-    }
-  }
-  return warnings;
+	const warnings = [];
+	const packageSpecByName = new Map(packages.map((declaredPackage) => [declaredPackage.name, declaredPackage.spec]));
+	const coreVersion = comparableVersion(await readInstalledPackageVersion(rootDir, "@sentry/junior") ?? packageSpecByName.get("@sentry/junior"));
+	if (!coreVersion) return warnings;
+	for (const declaredPackage of packages) {
+		if (!declaredPackage.name.startsWith("@sentry/junior-")) continue;
+		if (!await packageHasPluginContent(rootDir, declaredPackage.name)) continue;
+		const pluginVersion = comparableVersion(await readInstalledPackageVersion(rootDir, declaredPackage.name) ?? declaredPackage.spec);
+		if (pluginVersion && pluginVersion !== coreVersion) warnings.push(`${path.join(rootDir, "package.json")}: ${declaredPackage.name} version ${pluginVersion} does not match @sentry/junior version ${coreVersion}`);
+	}
+	return warnings;
 }
 async function validateSkillDirectory(skillDir, duplicateNames) {
-  const skillFile = path.join(skillDir, "SKILL.md");
-  const errors = [];
-  const warnings = [];
-  let raw;
-  try {
-    raw = await fs.readFile(skillFile, "utf8");
-  } catch {
-    errors.push(`${skillFile}: missing SKILL.md`);
-    return { skillFile, errors, warnings };
-  }
-  const parsed = parseSkillFile(raw, path.basename(skillDir));
-  if (!parsed.ok) {
-    errors.push(`${skillFile}: ${parsed.error}`);
-    return { skillFile, errors, warnings };
-  }
-  const name = parsed.skill.name;
-  const firstSeen = duplicateNames.get(name);
-  if (firstSeen) {
-    errors.push(
-      `${skillFile}: duplicate skill name "${name}" (already defined in ${firstSeen})`
-    );
-  } else {
-    duplicateNames.set(name, skillFile);
-  }
-  if (!parsed.skill.body) {
-    warnings.push(`${skillFile}: no skill instructions after frontmatter`);
-  }
-  if (/\b(?:searchTools|searchMcpTools|useTool|callMcpTool|available_tools)\b|<active-mcp-(?:tools|catalogs)>/.test(
-    parsed.skill.body
-  )) {
-    errors.push(
-      `${skillFile}: skill instructions must not hardcode harness tool-discovery or MCP dispatcher mechanics`
-    );
-  }
-  return { skillFile, skill: parsed.skill, errors, warnings };
+	const skillFile = path.join(skillDir, "SKILL.md");
+	const errors = [];
+	const warnings = [];
+	let raw;
+	try {
+		raw = await fs.readFile(skillFile, "utf8");
+	} catch {
+		errors.push(`${skillFile}: missing SKILL.md`);
+		return {
+			skillFile,
+			errors,
+			warnings
+		};
+	}
+	const parsed = parseSkillFile(raw, path.basename(skillDir));
+	if (!parsed.ok) {
+		errors.push(`${skillFile}: ${parsed.error}`);
+		return {
+			skillFile,
+			errors,
+			warnings
+		};
+	}
+	const name = parsed.skill.name;
+	const firstSeen = duplicateNames.get(name);
+	if (firstSeen) errors.push(`${skillFile}: duplicate skill name "${name}" (already defined in ${firstSeen})`);
+	else duplicateNames.set(name, skillFile);
+	if (!parsed.skill.body) warnings.push(`${skillFile}: no skill instructions after frontmatter`);
+	if (/\b(?:searchTools|searchMcpTools|useTool|callMcpTool|available_tools)\b|<active-mcp-(?:tools|catalogs)>/.test(parsed.skill.body)) errors.push(`${skillFile}: skill instructions must not hardcode harness tool-discovery or MCP dispatcher mechanics`);
+	return {
+		skillFile,
+		skill: parsed.skill,
+		errors,
+		warnings
+	};
 }
 async function validatePluginDirectory(pluginDir, duplicatePluginNames, duplicateProviderDomains) {
-  const manifestPath = path.join(pluginDir, "plugin.yaml");
-  try {
-    const raw = await fs.readFile(manifestPath, "utf8");
-    const manifest = parsePluginManifest(raw, pluginDir);
-    return validatePluginManifest(
-      manifestPath,
-      manifest,
-      duplicatePluginNames,
-      duplicateProviderDomains
-    );
-  } catch (error) {
-    return {
-      manifestPath,
-      errors: [
-        `${manifestPath}: ${error instanceof Error ? error.message : String(error)}`
-      ]
-    };
-  }
+	const manifestPath = path.join(pluginDir, "plugin.yaml");
+	try {
+		const raw = await fs.readFile(manifestPath, "utf8");
+		return validatePluginManifest(manifestPath, parsePluginManifest(raw, pluginDir), duplicatePluginNames, duplicateProviderDomains);
+	} catch (error) {
+		return {
+			manifestPath,
+			errors: [`${manifestPath}: ${error instanceof Error ? error.message : String(error)}`]
+		};
+	}
 }
 function validatePluginManifest(manifestPath, manifest, duplicatePluginNames, duplicateProviderDomains) {
-  const errors = [];
-  const firstSeen = duplicatePluginNames.get(manifest.name);
-  if (firstSeen) {
-    errors.push(
-      `${manifestPath}: duplicate plugin name "${manifest.name}" (already defined in ${firstSeen})`
-    );
-  }
-  const domains = [
-    .../* @__PURE__ */ new Set([
-      ...manifest.credentials?.domains ?? [],
-      ...manifest.domains ?? []
-    ])
-  ];
-  for (const domain of domains) {
-    const firstDomainSeen = duplicateProviderDomains.get(domain);
-    if (firstDomainSeen) {
-      errors.push(
-        `${manifestPath}: duplicate provider domain "${domain}" (already defined in ${firstDomainSeen})`
-      );
-    }
-  }
-  if (errors.length > 0) {
-    return { manifestPath, manifest, errors };
-  }
-  duplicatePluginNames.set(manifest.name, manifestPath);
-  for (const domain of domains) {
-    duplicateProviderDomains.set(domain, manifestPath);
-  }
-  return { manifestPath, manifest, errors: [] };
+	const errors = [];
+	const firstSeen = duplicatePluginNames.get(manifest.name);
+	if (firstSeen) errors.push(`${manifestPath}: duplicate plugin name "${manifest.name}" (already defined in ${firstSeen})`);
+	const domains = [.../* @__PURE__ */ new Set([...manifest.credentials?.domains ?? [], ...manifest.domains ?? []])];
+	for (const domain of domains) {
+		const firstDomainSeen = duplicateProviderDomains.get(domain);
+		if (firstDomainSeen) errors.push(`${manifestPath}: duplicate provider domain "${domain}" (already defined in ${firstDomainSeen})`);
+	}
+	if (errors.length > 0) return {
+		manifestPath,
+		manifest,
+		errors
+	};
+	duplicatePluginNames.set(manifest.name, manifestPath);
+	for (const domain of domains) duplicateProviderDomains.set(domain, manifestPath);
+	return {
+		manifestPath,
+		manifest,
+		errors: []
+	};
 }
 function packageExportsEntryPath(exportsValue) {
-  if (typeof exportsValue === "string") {
-    return exportsValue;
-  }
-  if (!isRecord(exportsValue)) {
-    return void 0;
-  }
-  const rootExport = exportsValue["."] ?? exportsValue;
-  if (typeof rootExport === "string") {
-    return rootExport;
-  }
-  if (!isRecord(rootExport)) {
-    return void 0;
-  }
-  for (const condition of ["default", "import", "node"]) {
-    const target = rootExport[condition];
-    if (typeof target === "string") {
-      return target;
-    }
-  }
-  return void 0;
+	if (typeof exportsValue === "string") return exportsValue;
+	if (!isRecord(exportsValue)) return;
+	const rootExport = exportsValue["."] ?? exportsValue;
+	if (typeof rootExport === "string") return rootExport;
+	if (!isRecord(rootExport)) return;
+	for (const condition of [
+		"default",
+		"import",
+		"node"
+	]) {
+		const target = rootExport[condition];
+		if (typeof target === "string") return target;
+	}
 }
 async function resolvePackageEntryPath(packageDir) {
-  const packageJson = await readJsonFile(
-    path.join(packageDir, "package.json")
-  );
-  const entry = packageExportsEntryPath(packageJson?.exports) ?? packageJson?.main;
-  const candidates = [
-    ...entry ? [entry] : [],
-    "./index.js",
-    "./dist/index.js"
-  ];
-  for (const candidate of candidates) {
-    if (path.isAbsolute(candidate) || candidate.startsWith("../")) {
-      continue;
-    }
-    const normalizedCandidate = candidate.startsWith("./") ? candidate : `./${candidate}`;
-    const entryPath = path.resolve(packageDir, normalizedCandidate);
-    if (await pathIsFile(entryPath)) {
-      return entryPath;
-    }
-  }
-  return void 0;
+	const packageJson = await readJsonFile(path.join(packageDir, "package.json"));
+	const entry = packageExportsEntryPath(packageJson?.exports) ?? packageJson?.main;
+	const candidates = [
+		...entry ? [entry] : [],
+		"./index.js",
+		"./dist/index.js"
+	];
+	for (const candidate of candidates) {
+		if (path.isAbsolute(candidate) || candidate.startsWith("../")) continue;
+		const normalizedCandidate = candidate.startsWith("./") ? candidate : `./${candidate}`;
+		const entryPath = path.resolve(packageDir, normalizedCandidate);
+		if (await pathIsFile(entryPath)) return entryPath;
+	}
 }
 function isPluginFactoryExport(name, value) {
-  return typeof value === "function" && (name === "default" || name.endsWith("Plugin"));
+	return typeof value === "function" && (name === "default" || name.endsWith("Plugin"));
 }
 function isPluginRegistrationLike(value) {
-  return isRecord(value) && isRecord(value.manifest);
+	return isRecord(value) && isRecord(value.manifest);
 }
 async function collectPackageJsPluginSources(packageDir, packageName) {
-  const entryPath = await resolvePackageEntryPath(packageDir);
-  if (!entryPath) {
-    return [];
-  }
-  let moduleExports;
-  try {
-    moduleExports = await import(pathToFileURL(entryPath).href);
-  } catch {
-    return [];
-  }
-  const pluginSources = [];
-  for (const [exportName, exportedValue] of Object.entries(moduleExports).sort(
-    ([left], [right]) => left.localeCompare(right)
-  )) {
-    if (!isPluginFactoryExport(exportName, exportedValue)) {
-      continue;
-    }
-    let registration;
-    try {
-      registration = await exportedValue();
-    } catch {
-      continue;
-    }
-    if (!isPluginRegistrationLike(registration)) {
-      continue;
-    }
-    const manifestPath = `${entryPath}#${exportName}`;
-    try {
-      pluginSources.push({
-        pluginDir: packageDir,
-        packageName,
-        manifestPath,
-        manifest: parseInlinePluginManifest(registration.manifest, packageDir)
-      });
-    } catch (error) {
-      pluginSources.push({
-        pluginDir: packageDir,
-        packageName,
-        manifestPath,
-        errors: [
-          `${manifestPath}: ${error instanceof Error ? error.message : String(error)}`
-        ]
-      });
-    }
-  }
-  return pluginSources;
+	const entryPath = await resolvePackageEntryPath(packageDir);
+	if (!entryPath) return [];
+	let moduleExports;
+	try {
+		moduleExports = await import(pathToFileURL(entryPath).href);
+	} catch {
+		return [];
+	}
+	const pluginSources = [];
+	for (const [exportName, exportedValue] of Object.entries(moduleExports).sort(([left], [right]) => left.localeCompare(right))) {
+		if (!isPluginFactoryExport(exportName, exportedValue)) continue;
+		let registration;
+		try {
+			registration = await exportedValue();
+		} catch {
+			continue;
+		}
+		if (!isPluginRegistrationLike(registration)) continue;
+		const manifestPath = `${entryPath}#${exportName}`;
+		try {
+			pluginSources.push({
+				pluginDir: packageDir,
+				packageName,
+				manifestPath,
+				manifest: parseInlinePluginManifest(registration.manifest, packageDir)
+			});
+		} catch (error) {
+			pluginSources.push({
+				pluginDir: packageDir,
+				packageName,
+				manifestPath,
+				errors: [`${manifestPath}: ${error instanceof Error ? error.message : String(error)}`]
+			});
+		}
+	}
+	return pluginSources;
 }
 async function collectPluginDirectories(rootDir) {
-  const pluginDirs = [];
-  let entries;
-  try {
-    entries = await fs.readdir(contentRoot(rootDir, "plugins"), {
-      withFileTypes: true
-    });
-  } catch {
-    return pluginDirs;
-  }
-  for (const entry of entries) {
-    if (!entry.isDirectory()) {
-      continue;
-    }
-    const pluginDir = path.join(contentRoot(rootDir, "plugins"), entry.name);
-    if (await pathIsFile(path.join(pluginDir, "plugin.yaml"))) {
-      pluginDirs.push(pluginDir);
-    }
-  }
-  return pluginDirs.sort((left, right) => left.localeCompare(right));
+	const pluginDirs = [];
+	let entries;
+	try {
+		entries = await fs.readdir(contentRoot(rootDir, "plugins"), { withFileTypes: true });
+	} catch {
+		return pluginDirs;
+	}
+	for (const entry of entries) {
+		if (!entry.isDirectory()) continue;
+		const pluginDir = path.join(contentRoot(rootDir, "plugins"), entry.name);
+		if (await pathIsFile(path.join(pluginDir, "plugin.yaml"))) pluginDirs.push(pluginDir);
+	}
+	return pluginDirs.sort((left, right) => left.localeCompare(right));
 }
 async function collectPackagedContent(rootDir, packages) {
-  const pluginSources = [];
-  const skillRoots = [];
-  for (const declaredPackage of packages) {
-    const packageDir = packageInstallDir(rootDir, declaredPackage.name);
-    if (!await pathIsDirectory(packageDir)) {
-      continue;
-    }
-    const rootManifestPath = path.join(packageDir, "plugin.yaml");
-    const hasRootManifest = await pathIsFile(rootManifestPath);
-    const packageSkillsRoot = path.join(packageDir, "skills");
-    const hasPackageSkills = await pathIsDirectory(packageSkillsRoot);
-    if (hasRootManifest) {
-      pluginSources.push({
-        pluginDir: packageDir,
-        manifestPath: rootManifestPath,
-        packageName: declaredPackage.name
-      });
-    } else if (hasPackageSkills) {
-      const jsPluginSources = await collectPackageJsPluginSources(
-        packageDir,
-        declaredPackage.name
-      );
-      if (jsPluginSources.length > 0) {
-        pluginSources.push(...jsPluginSources);
-      } else {
-        skillRoots.push({
-          root: packageSkillsRoot,
-          packageName: declaredPackage.name
-        });
-      }
-    }
-    const nestedPluginsRoot = path.join(packageDir, "plugins");
-    let nestedEntries;
-    try {
-      nestedEntries = await fs.readdir(nestedPluginsRoot, {
-        withFileTypes: true
-      });
-    } catch {
-      continue;
-    }
-    for (const entry of nestedEntries) {
-      if (!entry.isDirectory()) {
-        continue;
-      }
-      const pluginDir = path.join(nestedPluginsRoot, entry.name);
-      if (await pathIsFile(path.join(pluginDir, "plugin.yaml"))) {
-        pluginSources.push({
-          pluginDir,
-          manifestPath: path.join(pluginDir, "plugin.yaml"),
-          packageName: declaredPackage.name
-        });
-      }
-    }
-  }
-  return {
-    pluginSources: pluginSources.sort(
-      (left, right) => `${left.packageName}:${left.pluginDir}`.localeCompare(
-        `${right.packageName}:${right.pluginDir}`
-      )
-    ),
-    skillRoots: skillRoots.sort(
-      (left, right) => `${left.packageName}:${left.root}`.localeCompare(
-        `${right.packageName}:${right.root}`
-      )
-    )
-  };
+	const pluginSources = [];
+	const skillRoots = [];
+	for (const declaredPackage of packages) {
+		const packageDir = packageInstallDir(rootDir, declaredPackage.name);
+		if (!await pathIsDirectory(packageDir)) continue;
+		const rootManifestPath = path.join(packageDir, "plugin.yaml");
+		const hasRootManifest = await pathIsFile(rootManifestPath);
+		const packageSkillsRoot = path.join(packageDir, "skills");
+		const hasPackageSkills = await pathIsDirectory(packageSkillsRoot);
+		if (hasRootManifest) pluginSources.push({
+			pluginDir: packageDir,
+			manifestPath: rootManifestPath,
+			packageName: declaredPackage.name
+		});
+		else if (hasPackageSkills) {
+			const jsPluginSources = await collectPackageJsPluginSources(packageDir, declaredPackage.name);
+			if (jsPluginSources.length > 0) pluginSources.push(...jsPluginSources);
+			else skillRoots.push({
+				root: packageSkillsRoot,
+				packageName: declaredPackage.name
+			});
+		}
+		const nestedPluginsRoot = path.join(packageDir, "plugins");
+		let nestedEntries;
+		try {
+			nestedEntries = await fs.readdir(nestedPluginsRoot, { withFileTypes: true });
+		} catch {
+			continue;
+		}
+		for (const entry of nestedEntries) {
+			if (!entry.isDirectory()) continue;
+			const pluginDir = path.join(nestedPluginsRoot, entry.name);
+			if (await pathIsFile(path.join(pluginDir, "plugin.yaml"))) pluginSources.push({
+				pluginDir,
+				manifestPath: path.join(pluginDir, "plugin.yaml"),
+				packageName: declaredPackage.name
+			});
+		}
+	}
+	return {
+		pluginSources: pluginSources.sort((left, right) => `${left.packageName}:${left.pluginDir}`.localeCompare(`${right.packageName}:${right.pluginDir}`)),
+		skillRoots: skillRoots.sort((left, right) => `${left.packageName}:${left.root}`.localeCompare(`${right.packageName}:${right.root}`))
+	};
 }
 async function collectSkillDirectories(root) {
-  const skillDirs = [];
-  let entries;
-  try {
-    entries = await fs.readdir(root, { withFileTypes: true });
-  } catch {
-    return skillDirs;
-  }
-  for (const entry of entries) {
-    if (entry.isDirectory()) {
-      skillDirs.push(path.join(root, entry.name));
-    }
-  }
-  return skillDirs.sort((left, right) => left.localeCompare(right));
+	const skillDirs = [];
+	let entries;
+	try {
+		entries = await fs.readdir(root, { withFileTypes: true });
+	} catch {
+		return skillDirs;
+	}
+	for (const entry of entries) if (entry.isDirectory()) skillDirs.push(path.join(root, entry.name));
+	return skillDirs.sort((left, right) => left.localeCompare(right));
 }
 function formatDisplayPath(rootDir, targetPath) {
-  const relativePath = path.relative(rootDir, targetPath);
-  const displayPath = relativePath.length > 0 && !relativePath.startsWith("..") && !path.isAbsolute(relativePath) ? relativePath : targetPath;
-  return displayPath.split(path.sep).join("/");
+	const relativePath = path.relative(rootDir, targetPath);
+	return (relativePath.length > 0 && !relativePath.startsWith("..") && !path.isAbsolute(relativePath) ? relativePath : targetPath).split(path.sep).join("/");
 }
 function formatStatus(errorCount, warningCount) {
-  if (errorCount > 0) {
-    return "error";
-  }
-  if (warningCount > 0) {
-    return "warn";
-  }
-  return "ok";
+	if (errorCount > 0) return "error";
+	if (warningCount > 0) return "warn";
+	return "ok";
 }
 function statusIcon(status) {
-  switch (status) {
-    case "ok":
-      return color("\u2713", ANSI.green, ANSI.bold);
-    case "warn":
-      return color("\u26A0", ANSI.yellow, ANSI.bold);
-    case "error":
-      return color("\u2716", ANSI.red, ANSI.bold);
-  }
+	switch (status) {
+		case "ok": return color("✓", ANSI.green, ANSI.bold);
+		case "warn": return color("⚠", ANSI.yellow, ANSI.bold);
+		case "error": return color("✖", ANSI.red, ANSI.bold);
+	}
 }
 function formatHeading(status, label) {
-  const styledLabel = status === "ok" ? color(label, ANSI.bold) : status === "warn" ? color(label, ANSI.bold, ANSI.yellow) : color(label, ANSI.bold, ANSI.red);
-  return `${statusIcon(status)} ${styledLabel}`;
+	const styledLabel = status === "ok" ? color(label, ANSI.bold) : status === "warn" ? color(label, ANSI.bold, ANSI.yellow) : color(label, ANSI.bold, ANSI.red);
+	return `${statusIcon(status)} ${styledLabel}`;
 }
 function reportSkillResult(result, io, indent, isLast) {
-  const status = formatStatus(result.errors.length, result.warnings.length);
-  const skillName = result.skill?.name ?? path.basename(path.dirname(result.skillFile));
-  const branch = isLast ? "\u2514\u2500" : "\u251C\u2500";
-  io.info(`${indent}${branch} ${formatHeading(status, `skill ${skillName}`)}`);
+	const status = formatStatus(result.errors.length, result.warnings.length);
+	const skillName = result.skill?.name ?? path.basename(path.dirname(result.skillFile));
+	const branch = isLast ? "└─" : "├─";
+	io.info(`${indent}${branch} ${formatHeading(status, `skill ${skillName}`)}`);
 }
 function reportPluginResult(result, io) {
-  const skillErrorCount = result.skillResults.reduce(
-    (count, skillResult) => count + skillResult.errors.length,
-    0
-  );
-  const skillWarningCount = result.skillResults.reduce(
-    (count, skillResult) => count + skillResult.warnings.length,
-    0
-  );
-  const status = formatStatus(
-    result.errors.length + skillErrorCount,
-    skillWarningCount
-  );
-  const pluginName = result.manifest?.name ?? path.basename(result.pluginDir);
-  const label = result.packageName ? `packaged plugin ${pluginName} (${result.packageName})` : `plugin ${pluginName}`;
-  io.info(formatHeading(status, label));
-  for (const [index, skillResult] of result.skillResults.entries()) {
-    reportSkillResult(
-      skillResult,
-      io,
-      "  ",
-      index === result.skillResults.length - 1
-    );
-  }
+	const skillErrorCount = result.skillResults.reduce((count, skillResult) => count + skillResult.errors.length, 0);
+	const skillWarningCount = result.skillResults.reduce((count, skillResult) => count + skillResult.warnings.length, 0);
+	const status = formatStatus(result.errors.length + skillErrorCount, skillWarningCount);
+	const pluginName = result.manifest?.name ?? path.basename(result.pluginDir);
+	const label = result.packageName ? `packaged plugin ${pluginName} (${result.packageName})` : `plugin ${pluginName}`;
+	io.info(formatHeading(status, label));
+	for (const [index, skillResult] of result.skillResults.entries()) reportSkillResult(skillResult, io, "  ", index === result.skillResults.length - 1);
 }
 function reportSkillGroup(label, skillResults, io) {
-  const errorCount = skillResults.reduce(
-    (count, skillResult) => count + skillResult.errors.length,
-    0
-  );
-  const warningCount = skillResults.reduce(
-    (count, skillResult) => count + skillResult.warnings.length,
-    0
-  );
-  const status = formatStatus(errorCount, warningCount);
-  io.info(formatHeading(status, label));
-  for (const [index, skillResult] of skillResults.entries()) {
-    reportSkillResult(skillResult, io, "  ", index === skillResults.length - 1);
-  }
+	const status = formatStatus(skillResults.reduce((count, skillResult) => count + skillResult.errors.length, 0), skillResults.reduce((count, skillResult) => count + skillResult.warnings.length, 0));
+	io.info(formatHeading(status, label));
+	for (const [index, skillResult] of skillResults.entries()) reportSkillResult(skillResult, io, "  ", index === skillResults.length - 1);
 }
 function reportAppSkills(skillResults, io) {
-  reportSkillGroup("app skills", skillResults, io);
+	reportSkillGroup("app skills", skillResults, io);
 }
 function isRecord(value) {
-  return Boolean(value && typeof value === "object" && !Array.isArray(value));
+	return Boolean(value && typeof value === "object" && !Array.isArray(value));
 }
 async function hasJuniorDeploymentMarkers(rootDir, packages, nitroConfig) {
-  if (packages.some(
-    (declaredPackage) => declaredPackage.name === "@sentry/junior"
-  )) {
-    return true;
-  }
-  for (const source of [
-    nitroConfig,
-    await readTextFile(path.join(rootDir, "server.ts")),
-    await readTextFile(path.join(rootDir, "server.js"))
-  ]) {
-    if (source?.includes("@sentry/junior") || /\bjuniorNitro\s*\(/.test(source ?? "")) {
-      return true;
-    }
-  }
-  const appDir = path.resolve(rootDir, "app");
-  return await pathIsDirectory(appDir) && await hasJuniorAppMarkers(appDir);
+	if (packages.some((declaredPackage) => declaredPackage.name === "@sentry/junior")) return true;
+	for (const source of [
+		nitroConfig,
+		await readTextFile(path.join(rootDir, "server.ts")),
+		await readTextFile(path.join(rootDir, "server.js"))
+	]) if (source?.includes("@sentry/junior") || /\bjuniorNitro\s*\(/.test(source ?? "")) return true;
+	const appDir = path.resolve(rootDir, "app");
+	return await pathIsDirectory(appDir) && await hasJuniorAppMarkers(appDir);
 }
 async function validateDeploymentConfig(rootDir, packages) {
-  const errors = [];
-  const warnings = [];
-  let checked = false;
-  const nitroConfigPath = path.join(rootDir, "nitro.config.ts");
-  const nitroConfig = await readTextFile(nitroConfigPath);
-  const hasJuniorMarkers = await hasJuniorDeploymentMarkers(
-    rootDir,
-    packages,
-    nitroConfig
-  );
-  if (nitroConfig !== void 0 && hasJuniorMarkers) {
-    checked = true;
-    if (!/\bjuniorNitro\s*\(/.test(nitroConfig)) {
-      errors.push(
-        `${nitroConfigPath}: missing juniorNitro(). The Nitro module emits Junior's Vercel queue trigger and heartbeat cron into the deployed build output.`
-      );
-    }
-  }
-  const vercelConfigPath = path.join(rootDir, "vercel.json");
-  const vercelConfigText = await readTextFile(vercelConfigPath);
-  if (vercelConfigText === void 0) {
-    return { checked, errors, warnings };
-  }
-  const containsLegacyJuniorQueuePath = vercelConfigText.includes(
-    LEGACY_JUNIOR_CONVERSATION_WORK_FUNCTION
-  );
-  if (!hasJuniorMarkers && !containsLegacyJuniorQueuePath) {
-    return { checked, errors, warnings };
-  }
-  let vercelConfig;
-  try {
-    vercelConfig = JSON.parse(vercelConfigText);
-  } catch (error) {
-    errors.push(
-      `${vercelConfigPath}: invalid JSON (${error instanceof Error ? error.message : String(error)})`
-    );
-    return { checked, errors, warnings };
-  }
-  if (hasJuniorMarkers && vercelConfig.framework !== void 0 && vercelConfig.framework !== "nitro") {
-    checked = true;
-    errors.push(
-      `${vercelConfigPath}: framework must be "nitro" for Junior's Vercel deployment wiring.`
-    );
-  }
-  if (isRecord(vercelConfig.functions)) {
-    const legacyFunctionConfig = vercelConfig.functions[LEGACY_JUNIOR_CONVERSATION_WORK_FUNCTION];
-    if (legacyFunctionConfig !== void 0) {
-      checked = true;
-      errors.push(
-        `${vercelConfigPath}: functions.${LEGACY_JUNIOR_CONVERSATION_WORK_FUNCTION} targets a source file that Nitro does not deploy. Remove it; juniorNitro() emits the ${JUNIOR_CONVERSATION_WORK_CALLBACK_ROUTE} queue trigger through Nitro functionRules.`
-      );
-    }
-  }
-  if (hasJuniorMarkers && Array.isArray(vercelConfig.crons)) {
-    const hasHeartbeatCron = vercelConfig.crons.some(
-      (cron) => isRecord(cron) && cron.path === JUNIOR_HEARTBEAT_ROUTE
-    );
-    if (hasHeartbeatCron) {
-      checked = true;
-      warnings.push(
-        `${vercelConfigPath}: ${JUNIOR_HEARTBEAT_ROUTE} cron is now emitted by juniorNitro() into Nitro's Vercel Build Output config. Remove the root cron entry to avoid deployment drift.`
-      );
-    }
-  }
-  return { checked, errors, warnings };
+	const errors = [];
+	const warnings = [];
+	let checked = false;
+	const nitroConfigPath = path.join(rootDir, "nitro.config.ts");
+	const nitroConfig = await readTextFile(nitroConfigPath);
+	const hasJuniorMarkers = await hasJuniorDeploymentMarkers(rootDir, packages, nitroConfig);
+	if (nitroConfig !== void 0 && hasJuniorMarkers) {
+		checked = true;
+		if (!/\bjuniorNitro\s*\(/.test(nitroConfig)) errors.push(`${nitroConfigPath}: missing juniorNitro(). The Nitro module emits Junior's Vercel queue trigger and heartbeat cron into the deployed build output.`);
+	}
+	const vercelConfigPath = path.join(rootDir, "vercel.json");
+	const vercelConfigText = await readTextFile(vercelConfigPath);
+	if (vercelConfigText === void 0) return {
+		checked,
+		errors,
+		warnings
+	};
+	const containsLegacyJuniorQueuePath = vercelConfigText.includes(LEGACY_JUNIOR_CONVERSATION_WORK_FUNCTION);
+	if (!hasJuniorMarkers && !containsLegacyJuniorQueuePath) return {
+		checked,
+		errors,
+		warnings
+	};
+	let vercelConfig;
+	try {
+		vercelConfig = JSON.parse(vercelConfigText);
+	} catch (error) {
+		errors.push(`${vercelConfigPath}: invalid JSON (${error instanceof Error ? error.message : String(error)})`);
+		return {
+			checked,
+			errors,
+			warnings
+		};
+	}
+	if (hasJuniorMarkers && vercelConfig.framework !== void 0 && vercelConfig.framework !== "nitro") {
+		checked = true;
+		errors.push(`${vercelConfigPath}: framework must be "nitro" for Junior's Vercel deployment wiring.`);
+	}
+	if (isRecord(vercelConfig.functions)) {
+		if (vercelConfig.functions["api/internal/agent/continue.ts"] !== void 0) {
+			checked = true;
+			errors.push(`${vercelConfigPath}: functions.${LEGACY_JUNIOR_CONVERSATION_WORK_FUNCTION} targets a source file that Nitro does not deploy. Remove it; juniorNitro() emits the ${JUNIOR_CONVERSATION_WORK_CALLBACK_ROUTE} queue trigger through Nitro functionRules.`);
+		}
+	}
+	if (hasJuniorMarkers && Array.isArray(vercelConfig.crons)) {
+		if (vercelConfig.crons.some((cron) => isRecord(cron) && cron.path === "/api/internal/heartbeat")) {
+			checked = true;
+			warnings.push(`${vercelConfigPath}: ${JUNIOR_HEARTBEAT_ROUTE} cron is now emitted by juniorNitro() into Nitro's Vercel Build Output config. Remove the root cron entry to avoid deployment drift.`);
+		}
+	}
+	return {
+		checked,
+		errors,
+		warnings
+	};
 }
 async function validateAppSourceFiles(rootDir, registeredConfigKeys) {
-  const errors = [];
-  const warnings = [];
-  for (const fileName of ["server.ts", "server.js", "nitro.config.ts"]) {
-    const sourcePath = path.join(rootDir, fileName);
-    let source;
-    try {
-      source = await fs.readFile(sourcePath, "utf8");
-    } catch {
-      continue;
-    }
-    if (/\bpluginPackages\s*:/.test(source)) {
-      errors.push(
-        `${sourcePath}: pluginPackages is no longer supported. Export a defineJuniorPlugins(...) set and point juniorNitro({ plugins: "./plugins" }) at it.`
-      );
-    }
-    if (/\bplugins\s*:\s*\{\s*packages\s*:/.test(source)) {
-      errors.push(
-        `${sourcePath}: plugins.packages is no longer supported. Export a defineJuniorPlugins(...) set and point juniorNitro({ plugins: "./plugins" }) at it.`
-      );
-    }
-    for (const defaultsBlock of source.matchAll(
-      /\bconfigDefaults\s*:\s*\{([\s\S]*?)\}/g
-    )) {
-      const block = defaultsBlock[1] ?? "";
-      for (const keyMatch of block.matchAll(/["']([^"']+)["']\s*:/g)) {
-        const key = keyMatch[1];
-        if (key && !registeredConfigKeys.has(key)) {
-          warnings.push(
-            `${sourcePath}: configDefaults key "${key}" is not a registered plugin config key`
-          );
-        }
-      }
-    }
-  }
-  return { errors, warnings };
+	const errors = [];
+	const warnings = [];
+	for (const fileName of [
+		"server.ts",
+		"server.js",
+		"nitro.config.ts"
+	]) {
+		const sourcePath = path.join(rootDir, fileName);
+		let source;
+		try {
+			source = await fs.readFile(sourcePath, "utf8");
+		} catch {
+			continue;
+		}
+		if (/\bpluginPackages\s*:/.test(source)) errors.push(`${sourcePath}: pluginPackages is no longer supported. Export a defineJuniorPlugins(...) set and point juniorNitro({ plugins: "./plugins" }) at it.`);
+		if (/\bplugins\s*:\s*\{\s*packages\s*:/.test(source)) errors.push(`${sourcePath}: plugins.packages is no longer supported. Export a defineJuniorPlugins(...) set and point juniorNitro({ plugins: "./plugins" }) at it.`);
+		for (const defaultsBlock of source.matchAll(/\bconfigDefaults\s*:\s*\{([\s\S]*?)\}/g)) {
+			const block = defaultsBlock[1] ?? "";
+			for (const keyMatch of block.matchAll(/["']([^"']+)["']\s*:/g)) {
+				const key = keyMatch[1];
+				if (key && !registeredConfigKeys.has(key)) warnings.push(`${sourcePath}: configDefaults key "${key}" is not a registered plugin config key`);
+			}
+		}
+	}
+	return {
+		errors,
+		warnings
+	};
 }
 async function validateAppFiles(appDir) {
-  const errors = [];
-  const warnings = [];
-  if (await pathIsFile(path.join(appDir, "ABOUT.md"))) {
-    errors.push(
-      `${path.join(appDir, "ABOUT.md")}: ABOUT.md is no longer supported. Rename to WORLD.md (operational context) and DESCRIPTION.md (user-facing description).`
-    );
-  }
-  if (!await pathIsFile(path.join(appDir, "SOUL.md"))) {
-    warnings.push(`${path.join(appDir, "SOUL.md")}: missing SOUL.md`);
-  }
-  if (!await pathIsFile(path.join(appDir, "WORLD.md"))) {
-    warnings.push(`${path.join(appDir, "WORLD.md")}: missing WORLD.md`);
-  }
-  if (!await pathIsFile(path.join(appDir, "DESCRIPTION.md"))) {
-    warnings.push(
-      `${path.join(appDir, "DESCRIPTION.md")}: missing DESCRIPTION.md`
-    );
-  }
-  return { errors, warnings };
+	const errors = [];
+	const warnings = [];
+	if (await pathIsFile(path.join(appDir, "ABOUT.md"))) errors.push(`${path.join(appDir, "ABOUT.md")}: ABOUT.md is no longer supported. Rename to WORLD.md (operational context) and DESCRIPTION.md (user-facing description).`);
+	if (!await pathIsFile(path.join(appDir, "SOUL.md"))) warnings.push(`${path.join(appDir, "SOUL.md")}: missing SOUL.md`);
+	if (!await pathIsFile(path.join(appDir, "WORLD.md"))) warnings.push(`${path.join(appDir, "WORLD.md")}: missing WORLD.md`);
+	if (!await pathIsFile(path.join(appDir, "DESCRIPTION.md"))) warnings.push(`${path.join(appDir, "DESCRIPTION.md")}: missing DESCRIPTION.md`);
+	return {
+		errors,
+		warnings
+	};
 }
 async function hasJuniorAppMarkers(appDir) {
-  for (const fileName of [
-    "SOUL.md",
-    "WORLD.md",
-    "DESCRIPTION.md",
-    "ABOUT.md"
-  ]) {
-    if (await pathIsFile(path.join(appDir, fileName))) {
-      return true;
-    }
-  }
-  for (const dirName of ["skills", "plugins"]) {
-    if (await pathIsDirectory(path.join(appDir, dirName))) {
-      return true;
-    }
-  }
-  return false;
+	for (const fileName of [
+		"SOUL.md",
+		"WORLD.md",
+		"DESCRIPTION.md",
+		"ABOUT.md"
+	]) if (await pathIsFile(path.join(appDir, fileName))) return true;
+	for (const dirName of ["skills", "plugins"]) if (await pathIsDirectory(path.join(appDir, dirName))) return true;
+	return false;
 }
 async function runCheck(rootDir = process.cwd(), io = DEFAULT_IO) {
-  const resolvedRoot = path.resolve(rootDir);
-  if (!await pathIsDirectory(resolvedRoot)) {
-    throw new Error(
-      `validation root does not exist or is not a directory: ${resolvedRoot}`
-    );
-  }
-  const rootPackageJson = await readRootPackageJson(resolvedRoot);
-  const packages = declaredPackages(rootPackageJson);
-  const packageWarnings = await collectPackageWarnings(resolvedRoot, packages);
-  const packagedContent = await collectPackagedContent(resolvedRoot, packages);
-  const appPluginDirs = await collectPluginDirectories(resolvedRoot);
-  const packagedPluginSources = packagedContent.pluginSources;
-  const pluginSources = [
-    ...appPluginDirs.map((pluginDir) => ({
-      pluginDir,
-      manifestPath: path.join(pluginDir, "plugin.yaml")
-    })),
-    ...packagedPluginSources
-  ];
-  const appSkillsRoot = contentRoot(resolvedRoot, "skills");
-  const appSkillDirs = await collectSkillDirectories(appSkillsRoot);
-  const pluginSkillDirs = /* @__PURE__ */ new Map();
-  for (const { pluginDir } of pluginSources) {
-    pluginSkillDirs.set(
-      pluginDir,
-      await collectSkillDirectories(path.join(pluginDir, "skills"))
-    );
-  }
-  const packagedSkillDirsByPackage = /* @__PURE__ */ new Map();
-  for (const skillRoot of packagedContent.skillRoots) {
-    packagedSkillDirsByPackage.set(
-      skillRoot.packageName,
-      await collectSkillDirectories(skillRoot.root)
-    );
-  }
-  const packagedStandaloneSkillDirs = [
-    ...packagedSkillDirsByPackage.values()
-  ].flat();
-  const appAndLocalPluginSkillDirs = [
-    ...appSkillDirs,
-    ...appPluginDirs.flatMap(
-      (pluginDir) => pluginSkillDirs.get(pluginDir) ?? []
-    )
-  ].sort((left, right) => left.localeCompare(right));
-  const packagedSkillDirs = [
-    ...packagedPluginSources.flatMap(
-      ({ pluginDir }) => pluginSkillDirs.get(pluginDir) ?? []
-    ),
-    ...packagedStandaloneSkillDirs
-  ].sort((left, right) => left.localeCompare(right));
-  const skillDirs = [...appAndLocalPluginSkillDirs, ...packagedSkillDirs].sort(
-    (left, right) => left.localeCompare(right)
-  );
-  const duplicateSkillNames = /* @__PURE__ */ new Map();
-  const duplicatePluginNames = /* @__PURE__ */ new Map();
-  const duplicateProviderDomains = /* @__PURE__ */ new Map();
-  const duplicatePackagedSkillNames = /* @__PURE__ */ new Map();
-  const duplicatePackagedPluginNames = /* @__PURE__ */ new Map();
-  const duplicatePackagedProviderDomains = /* @__PURE__ */ new Map();
-  const warnings = [];
-  const errors = [];
-  const pluginResults = [];
-  const skillResultsByDir = /* @__PURE__ */ new Map();
-  warnings.push(...packageWarnings);
-  const deploymentConfigResult = await validateDeploymentConfig(
-    resolvedRoot,
-    packages
-  );
-  warnings.push(...deploymentConfigResult.warnings);
-  errors.push(...deploymentConfigResult.errors);
-  for (const source of pluginSources) {
-    const { pluginDir, packageName } = source;
-    const pluginNameMap = packageName ? duplicatePackagedPluginNames : duplicatePluginNames;
-    const providerDomainMap = packageName ? duplicatePackagedProviderDomains : duplicateProviderDomains;
-    const result = source.errors ? {
-      manifestPath: source.manifestPath,
-      ...source.manifest ? { manifest: source.manifest } : void 0,
-      errors: source.errors
-    } : source.manifest ? validatePluginManifest(
-      source.manifestPath,
-      source.manifest,
-      pluginNameMap,
-      providerDomainMap
-    ) : await validatePluginDirectory(
-      pluginDir,
-      pluginNameMap,
-      providerDomainMap
-    );
-    pluginResults.push({
-      pluginDir,
-      manifestPath: result.manifestPath,
-      ...packageName ? { packageName } : void 0,
-      ...result.manifest ? { manifest: result.manifest } : void 0,
-      errors: result.errors,
-      skillResults: []
-    });
-    errors.push(...result.errors);
-  }
-  const registeredConfigKeys = new Set(
-    pluginResults.flatMap((result) => result.manifest?.configKeys ?? [])
-  );
-  const appSourceResult = await validateAppSourceFiles(
-    resolvedRoot,
-    registeredConfigKeys
-  );
-  warnings.push(...appSourceResult.warnings);
-  errors.push(...appSourceResult.errors);
-  for (const skillDir of appAndLocalPluginSkillDirs) {
-    const result = await validateSkillDirectory(skillDir, duplicateSkillNames);
-    skillResultsByDir.set(skillDir, result);
-    warnings.push(...result.warnings);
-    errors.push(...result.errors);
-  }
-  for (const skillDir of packagedSkillDirs) {
-    const result = await validateSkillDirectory(
-      skillDir,
-      duplicatePackagedSkillNames
-    );
-    skillResultsByDir.set(skillDir, result);
-    warnings.push(...result.warnings);
-    errors.push(...result.errors);
-  }
-  for (const pluginResult of pluginResults) {
-    pluginResult.skillResults = (pluginSkillDirs.get(pluginResult.pluginDir) ?? []).map((skillDir) => skillResultsByDir.get(skillDir)).filter((result) => Boolean(result));
-  }
-  const appSkillResults = appSkillDirs.map((skillDir) => skillResultsByDir.get(skillDir)).filter((result) => Boolean(result));
-  const packagedStandaloneSkillResultsByPackage = /* @__PURE__ */ new Map();
-  for (const [packageName, packageSkillDirs] of packagedSkillDirsByPackage) {
-    const packageSkillResults = packageSkillDirs.map((skillDir) => skillResultsByDir.get(skillDir)).filter((result) => Boolean(result));
-    if (packageSkillResults.length > 0) {
-      packagedStandaloneSkillResultsByPackage.set(
-        packageName,
-        packageSkillResults
-      );
-    }
-  }
-  const appDir = path.resolve(resolvedRoot, "app");
-  let appFileResult = {
-    errors: [],
-    warnings: []
-  };
-  const shouldValidateAppFiles = await pathIsDirectory(appDir) && await hasJuniorAppMarkers(appDir);
-  if (shouldValidateAppFiles) {
-    appFileResult = await validateAppFiles(appDir);
-    warnings.push(...appFileResult.warnings);
-    errors.push(...appFileResult.errors);
-  }
-  io.info(
-    `${color("Checking", ANSI.bold, ANSI.cyan)} ${color(
-      formatDisplayPath(resolvedRoot, resolvedRoot),
-      ANSI.dim
-    )}`
-  );
-  if (shouldValidateAppFiles) {
-    const appFileStatus = formatStatus(
-      appFileResult.errors.length,
-      appFileResult.warnings.length
-    );
-    io.info(formatHeading(appFileStatus, "app files"));
-  }
-  if (deploymentConfigResult.checked) {
-    const deploymentConfigStatus = formatStatus(
-      deploymentConfigResult.errors.length,
-      deploymentConfigResult.warnings.length
-    );
-    io.info(formatHeading(deploymentConfigStatus, "deployment config"));
-  }
-  for (const pluginResult of pluginResults) {
-    reportPluginResult(pluginResult, io);
-  }
-  for (const [
-    packageName,
-    packageSkillResults
-  ] of packagedStandaloneSkillResultsByPackage) {
-    reportSkillGroup(
-      `packaged skills (${packageName})`,
-      packageSkillResults,
-      io
-    );
-  }
-  if (appSkillResults.length > 0) {
-    reportAppSkills(appSkillResults, io);
-  }
-  for (const warning of warnings) {
-    io.warn(`${statusIcon("warn")} warning: ${warning}`);
-  }
-  if (errors.length > 0) {
-    for (const error of errors) {
-      io.error(`${statusIcon("error")} error: ${error}`);
-    }
-    throw new Error(
-      `Validation failed (${errors.length} error${errors.length === 1 ? "" : "s"}, ${pluginSources.length} plugin manifest${pluginSources.length === 1 ? "" : "s"}, ${skillDirs.length} skill director${skillDirs.length === 1 ? "y" : "ies"} checked).`
-    );
-  }
-  io.info(
-    `${formatHeading("ok", `Validation passed (${pluginSources.length} plugin manifest${pluginSources.length === 1 ? "" : "s"}, ${skillDirs.length} skill director${skillDirs.length === 1 ? "y" : "ies"} checked).`)}`
-  );
-}
-export {
-  runCheck
-};
+	const resolvedRoot = path.resolve(rootDir);
+	if (!await pathIsDirectory(resolvedRoot)) throw new Error(`validation root does not exist or is not a directory: ${resolvedRoot}`);
+	const packages = declaredPackages(await readRootPackageJson(resolvedRoot));
+	const packageWarnings = await collectPackageWarnings(resolvedRoot, packages);
+	const packagedContent = await collectPackagedContent(resolvedRoot, packages);
+	const appPluginDirs = await collectPluginDirectories(resolvedRoot);
+	const packagedPluginSources = packagedContent.pluginSources;
+	const pluginSources = [...appPluginDirs.map((pluginDir) => ({
+		pluginDir,
+		manifestPath: path.join(pluginDir, "plugin.yaml")
+	})), ...packagedPluginSources];
+	const appSkillDirs = await collectSkillDirectories(contentRoot(resolvedRoot, "skills"));
+	const pluginSkillDirs = /* @__PURE__ */ new Map();
+	for (const { pluginDir } of pluginSources) pluginSkillDirs.set(pluginDir, await collectSkillDirectories(path.join(pluginDir, "skills")));
+	const packagedSkillDirsByPackage = /* @__PURE__ */ new Map();
+	for (const skillRoot of packagedContent.skillRoots) packagedSkillDirsByPackage.set(skillRoot.packageName, await collectSkillDirectories(skillRoot.root));
+	const packagedStandaloneSkillDirs = [...packagedSkillDirsByPackage.values()].flat();
+	const appAndLocalPluginSkillDirs = [...appSkillDirs, ...appPluginDirs.flatMap((pluginDir) => pluginSkillDirs.get(pluginDir) ?? [])].sort((left, right) => left.localeCompare(right));
+	const packagedSkillDirs = [...packagedPluginSources.flatMap(({ pluginDir }) => pluginSkillDirs.get(pluginDir) ?? []), ...packagedStandaloneSkillDirs].sort((left, right) => left.localeCompare(right));
+	const skillDirs = [...appAndLocalPluginSkillDirs, ...packagedSkillDirs].sort((left, right) => left.localeCompare(right));
+	const duplicateSkillNames = /* @__PURE__ */ new Map();
+	const duplicatePluginNames = /* @__PURE__ */ new Map();
+	const duplicateProviderDomains = /* @__PURE__ */ new Map();
+	const duplicatePackagedSkillNames = /* @__PURE__ */ new Map();
+	const duplicatePackagedPluginNames = /* @__PURE__ */ new Map();
+	const duplicatePackagedProviderDomains = /* @__PURE__ */ new Map();
+	const warnings = [];
+	const errors = [];
+	const pluginResults = [];
+	const skillResultsByDir = /* @__PURE__ */ new Map();
+	warnings.push(...packageWarnings);
+	const deploymentConfigResult = await validateDeploymentConfig(resolvedRoot, packages);
+	warnings.push(...deploymentConfigResult.warnings);
+	errors.push(...deploymentConfigResult.errors);
+	for (const source of pluginSources) {
+		const { pluginDir, packageName } = source;
+		const pluginNameMap = packageName ? duplicatePackagedPluginNames : duplicatePluginNames;
+		const providerDomainMap = packageName ? duplicatePackagedProviderDomains : duplicateProviderDomains;
+		const result = source.errors ? {
+			manifestPath: source.manifestPath,
+			...source.manifest ? { manifest: source.manifest } : void 0,
+			errors: source.errors
+		} : source.manifest ? validatePluginManifest(source.manifestPath, source.manifest, pluginNameMap, providerDomainMap) : await validatePluginDirectory(pluginDir, pluginNameMap, providerDomainMap);
+		pluginResults.push({
+			pluginDir,
+			manifestPath: result.manifestPath,
+			...packageName ? { packageName } : void 0,
+			...result.manifest ? { manifest: result.manifest } : void 0,
+			errors: result.errors,
+			skillResults: []
+		});
+		errors.push(...result.errors);
+	}
+	const appSourceResult = await validateAppSourceFiles(resolvedRoot, new Set(pluginResults.flatMap((result) => result.manifest?.configKeys ?? [])));
+	warnings.push(...appSourceResult.warnings);
+	errors.push(...appSourceResult.errors);
+	for (const skillDir of appAndLocalPluginSkillDirs) {
+		const result = await validateSkillDirectory(skillDir, duplicateSkillNames);
+		skillResultsByDir.set(skillDir, result);
+		warnings.push(...result.warnings);
+		errors.push(...result.errors);
+	}
+	for (const skillDir of packagedSkillDirs) {
+		const result = await validateSkillDirectory(skillDir, duplicatePackagedSkillNames);
+		skillResultsByDir.set(skillDir, result);
+		warnings.push(...result.warnings);
+		errors.push(...result.errors);
+	}
+	for (const pluginResult of pluginResults) pluginResult.skillResults = (pluginSkillDirs.get(pluginResult.pluginDir) ?? []).map((skillDir) => skillResultsByDir.get(skillDir)).filter((result) => Boolean(result));
+	const appSkillResults = appSkillDirs.map((skillDir) => skillResultsByDir.get(skillDir)).filter((result) => Boolean(result));
+	const packagedStandaloneSkillResultsByPackage = /* @__PURE__ */ new Map();
+	for (const [packageName, packageSkillDirs] of packagedSkillDirsByPackage) {
+		const packageSkillResults = packageSkillDirs.map((skillDir) => skillResultsByDir.get(skillDir)).filter((result) => Boolean(result));
+		if (packageSkillResults.length > 0) packagedStandaloneSkillResultsByPackage.set(packageName, packageSkillResults);
+	}
+	const appDir = path.resolve(resolvedRoot, "app");
+	let appFileResult = {
+		errors: [],
+		warnings: []
+	};
+	const shouldValidateAppFiles = await pathIsDirectory(appDir) && await hasJuniorAppMarkers(appDir);
+	if (shouldValidateAppFiles) {
+		appFileResult = await validateAppFiles(appDir);
+		warnings.push(...appFileResult.warnings);
+		errors.push(...appFileResult.errors);
+	}
+	io.info(`${color("Checking", ANSI.bold, ANSI.cyan)} ${color(formatDisplayPath(resolvedRoot, resolvedRoot), ANSI.dim)}`);
+	if (shouldValidateAppFiles) {
+		const appFileStatus = formatStatus(appFileResult.errors.length, appFileResult.warnings.length);
+		io.info(formatHeading(appFileStatus, "app files"));
+	}
+	if (deploymentConfigResult.checked) {
+		const deploymentConfigStatus = formatStatus(deploymentConfigResult.errors.length, deploymentConfigResult.warnings.length);
+		io.info(formatHeading(deploymentConfigStatus, "deployment config"));
+	}
+	for (const pluginResult of pluginResults) reportPluginResult(pluginResult, io);
+	for (const [packageName, packageSkillResults] of packagedStandaloneSkillResultsByPackage) reportSkillGroup(`packaged skills (${packageName})`, packageSkillResults, io);
+	if (appSkillResults.length > 0) reportAppSkills(appSkillResults, io);
+	for (const warning of warnings) io.warn(`${statusIcon("warn")} warning: ${warning}`);
+	if (errors.length > 0) {
+		for (const error of errors) io.error(`${statusIcon("error")} error: ${error}`);
+		throw new Error(`Validation failed (${errors.length} error${errors.length === 1 ? "" : "s"}, ${pluginSources.length} plugin manifest${pluginSources.length === 1 ? "" : "s"}, ${skillDirs.length} skill director${skillDirs.length === 1 ? "y" : "ies"} checked).`);
+	}
+	io.info(`${formatHeading("ok", `Validation passed (${pluginSources.length} plugin manifest${pluginSources.length === 1 ? "" : "s"}, ${skillDirs.length} skill director${skillDirs.length === 1 ? "y" : "ies"} checked).`)}`);
+}
+//#endregion
+export { runCheck };