kennel 0.2.23 → 0.2.24

package/dist/cli/dispatch.js

@@ -1,6 +1,4 @@
-import { cpSync, existsSync, mkdirSync, readdirSync, readFileSync, statSync, writeFileSync } from 'node:fs';
-import { dirname, resolve } from 'node:path';
-import { fileURLToPath } from 'node:url';
+import { existsSync } from 'node:fs';
 import { usage } from './usage.js';
 export async function dispatch(argv) {
     const [, , command, ...args] = argv;
@@ -13,55 +11,9 @@ export async function dispatch(argv) {
         await viewer([]);
         return;
     }
-    if (command === 'plugin' && args[0] === 'add') {
-        const url = args[1];
-        let directory;
-        for (let index = 2; index < args.length; index += 1) {
-            const arg = args[index];
-            if (arg === '--path' || arg === '--directory') {
-                directory = args[index + 1];
-                if (!directory)
-                    throw new Error(`${arg} requires a plugin subdirectory`);
-                index += 1;
-            }
-            else {
-                throw new Error('Usage: kennel plugin add <git-repository-url-or-local-path> [--path <subdirectory>]');
-            }
-        }
-        if (!url)
-            throw new Error('Usage: kennel plugin add <git-repository-url-or-local-path> [--path <subdirectory>]');
-        const { installExternalPlugin } = await import('../plugins/external.js');
-        const plugin = await installExternalPlugin(url, { directory });
-        const source = plugin.path ?? plugin.url;
-        console.log(`Installed plugin ${plugin.id} from ${source} at ${plugin.commit}`);
-        return;
-    }
-    if (command === 'plugin' && args[0] === 'new') {
-        const id = args[1];
-        if (!id || args.length > 2)
-            throw new Error('Usage: kennel plugin new <id>');
-        if (!/^[a-z][a-z0-9-]*$/u.test(id))
-            throw new Error(`Invalid plugin id: ${id}`);
-        const target = resolve(id);
-        if (existsSync(target))
-            throw new Error(`Refusing to overwrite existing path: ${target}`);
-        const root = resolve(dirname(fileURLToPath(import.meta.url)), '../..');
-        const template = resolve(root, 'templates/plugin');
-        mkdirSync(target, { recursive: true });
-        cpSync(template, target, { recursive: true });
-        const replacePlaceholders = (folder) => {
-            for (const name of readdirSync(folder)) {
-                const path = resolve(folder, name);
-                if (statSync(path).isDirectory()) {
-                    replacePlaceholders(path);
-                    continue;
-                }
-                const content = readFileSync(path, 'utf8').replaceAll('__PLUGIN_ID__', id);
-                writeFileSync(path, content);
-            }
-        };
-        replacePlaceholders(target);
-        console.log(`Created Kennel plugin ${id} in ${target}`);
+    if (command === 'plugin') {
+        const { plugin } = await import('./plugin.js');
+        await plugin(args);
         return;
     }
     if (command === 'core') {

package/dist/cli/plugin.d.ts

@@ -0,0 +1 @@
+export declare function plugin(args: string[]): Promise<void>;

package/dist/cli/plugin.js

@@ -0,0 +1,103 @@
+import { cpSync, existsSync, mkdirSync, readdirSync, readFileSync, statSync, writeFileSync } from 'node:fs';
+import { dirname, resolve } from 'node:path';
+import { fileURLToPath } from 'node:url';
+const USAGE = `Usage:
+  kennel plugin add <git-repository-url-or-local-path> [--path <subdirectory>] [--install-skill]
+  kennel plugin skill install <plugin-id> [--force]
+  kennel plugin list
+  kennel plugin new <id>`;
+function usageError() {
+    throw new Error(USAGE);
+}
+async function installSkill(id, force) {
+    const { installPluginSkill } = await import('../plugins/external.js');
+    const skill = installPluginSkill(id, { force });
+    console.log(`Installed skill ${skill.name} at ${skill.path} (commit ${skill.commit})`);
+    for (const duplicate of skill.duplicates) {
+        console.warn(`Warning: another skill named ${skill.name} exists at ${duplicate}; Kennel did not change it.`);
+    }
+}
+export async function plugin(args) {
+    const [subcommand, ...rest] = args;
+    if (subcommand === 'new') {
+        const id = rest[0];
+        if (!id || rest.length > 1)
+            throw new Error('Usage: kennel plugin new <id>');
+        if (!/^[a-z][a-z0-9-]*$/u.test(id))
+            throw new Error(`Invalid plugin id: ${id}`);
+        const target = resolve(id);
+        if (existsSync(target))
+            throw new Error(`Refusing to overwrite existing path: ${target}`);
+        const root = resolve(dirname(fileURLToPath(import.meta.url)), '../..');
+        const template = resolve(root, 'templates/plugin');
+        mkdirSync(target, { recursive: true });
+        cpSync(template, target, { recursive: true });
+        const replacePlaceholders = (folder) => {
+            for (const name of readdirSync(folder)) {
+                const path = resolve(folder, name);
+                if (statSync(path).isDirectory()) {
+                    replacePlaceholders(path);
+                    continue;
+                }
+                const content = readFileSync(path, 'utf8').replaceAll('__PLUGIN_ID__', id);
+                writeFileSync(path, content);
+            }
+        };
+        replacePlaceholders(target);
+        console.log(`Created Kennel plugin ${id} in ${target}`);
+        return;
+    }
+    if (subcommand === 'add') {
+        const [url, ...flags] = rest;
+        let directory;
+        let withSkill = false;
+        for (let index = 0; index < flags.length; index += 1) {
+            const arg = flags[index];
+            if (arg === '--path' || arg === '--directory') {
+                directory = flags[index + 1];
+                if (!directory)
+                    throw new Error(`${arg} requires a plugin subdirectory`);
+                index += 1;
+            }
+            else if (arg === '--install-skill') {
+                withSkill = true;
+            }
+            else {
+                usageError();
+            }
+        }
+        if (!url || url.startsWith('-'))
+            usageError();
+        const { installExternalPlugin } = await import('../plugins/external.js');
+        const entry = await installExternalPlugin(url, { directory });
+        console.log(`Installed plugin ${entry.id} from ${entry.path ?? entry.url} at ${entry.commit}`);
+        if (withSkill)
+            await installSkill(entry.id, false);
+        else if (entry.skill)
+            console.log(`Plugin declares skill ${entry.skill.name}; install it with \`kennel plugin skill install ${entry.id}\``);
+        return;
+    }
+    if (subcommand === 'skill' && rest[0] === 'install') {
+        const [, id, ...flags] = rest;
+        if (!id || id.startsWith('-') || flags.some((flag) => flag !== '--force'))
+            usageError();
+        await installSkill(id, flags.includes('--force'));
+        return;
+    }
+    if (subcommand === 'list' && rest.length === 0) {
+        const { listPluginStatus } = await import('../plugins/external.js');
+        const plugins = listPluginStatus();
+        if (plugins.length === 0) {
+            console.log('No external plugins installed.');
+            return;
+        }
+        for (const item of plugins) {
+            const skill = item.skill.name
+                ? `skill ${item.skill.name}: ${item.skill.state} (${item.skill.path})`
+                : 'skill: missing (none declared)';
+            console.log(`${item.id}\t${item.packageName}@${item.commit.slice(0, 12)}\t${skill}`);
+        }
+        return;
+    }
+    usageError();
+}

package/dist/cli/usage.js

@@ -2,7 +2,9 @@ export function usage() {
     console.log(`Usage:
   knl core [folder] [--host <host>] [--port <port>] [--api-only] [--web-root <dir>]
   kennel init [path] [--plugin <id>] [--force] [--list]
-  kennel plugin add <git-repository-url-or-local-path> [--path <subdirectory>]
+  kennel plugin add <git-repository-url-or-local-path> [--path <subdirectory>] [--install-skill]
+  kennel plugin skill install <plugin-id> [--force]
+  kennel plugin list
   kennel plugin new <id>
   kennel core-folders [path] [--json]
   kennel [folder] [--host <host>] [--port <port>] [--lan|--tailscale]

package/dist/plugins/external.d.ts

@@ -1,3 +1,4 @@
+import { type SkillState } from "./skills.js";
 export type ExternalPluginEntry = {
     url: string;
     requestedRef?: string;
@@ -15,6 +16,15 @@ export type ExternalPluginEntry = {
         api?: string;
         css?: string;
     };
+    skill?: {
+        entry: string;
+        name: string;
+        installed?: {
+            path: string;
+            commit: string;
+            digest: string;
+        };
+    };
 };
 export declare function kennelConfigPath(): string;
 export declare const PLUGIN_SOURCE_IGNORED: Set<string>;
@@ -25,6 +35,28 @@ export declare function readExternalPlugins(): ExternalPluginEntry[];
 export declare function installExternalPlugin(url: string, options?: {
     directory?: string;
 }): Promise<ExternalPluginEntry>;
+/** Installs a declared skill from local source or the pinned cached checkout. */
+export declare function installPluginSkill(id: string, options?: {
+    force?: boolean;
+}): {
+    duplicates: string[];
+    path: string;
+    commit: string;
+    digest: string;
+    name: string;
+};
+export type PluginStatus = {
+    id: string;
+    packageName: string;
+    commit: string;
+    skill: {
+        name?: string;
+        state: SkillState;
+        path?: string;
+    };
+};
+/** Reports installed plugins and the real state of each declared skill. */
+export declare function listPluginStatus(): PluginStatus[];
 /** The compiled host web app this CLI serves. */
 export declare function webRoot(): string;
 /**

package/dist/plugins/external.js

@@ -14,6 +14,7 @@ import { basename, dirname, isAbsolute, join, relative, resolve, } from "node:pa
 import { pathToFileURL, fileURLToPath } from "node:url";
 import { spawnSync } from "node:child_process";
 import { z } from "zod";
+import { duplicateSkills, readPluginSkill, skillDestination, skillState, writeSkill } from "./skills.js";
 const CONFIG_FILE = join("kennel", "config.json");
 const VALID_ID = /^[a-z][a-z0-9-]*$/u;
 const CORE_IDS = new Set(["agents", "notes"]);
@@ -142,9 +143,13 @@ function resolveCommit(url, requestedRef) {
         throw new Error(`Could not resolve ref ${requestedRef} for ${url}`);
     return commit;
 }
-function sourceCheckout(url, commit) {
+function sourceDir(url) {
     const dir = join(cacheRoot(), "plugins", "sources", hash(url), "repo");
     assertUnderCache(dir);
+    return dir;
+}
+function sourceCheckout(url, commit) {
+    const dir = sourceDir(url);
     mkdirSync(dirname(dir), { recursive: true });
     if (!existsSync(join(dir, ".git")))
         run("git", ["clone", "--no-checkout", url, dir]);
@@ -400,6 +405,7 @@ export async function validateExternalPlugin(inputUrl, options = {}) {
     const api = entryByConvention(root, "api.ts", false);
     const cssValue = metadata?.exports?.css;
     const css = cssValue === undefined ? undefined : safeRelativeFile(root, cssValue, "css export path");
+    const skill = metadata?.skill === undefined ? undefined : readPluginSkill(root, metadata.skill);
     const entry = {
         url: source.url,
         ...(source.requestedRef ? { requestedRef: source.requestedRef } : {}),
@@ -417,6 +423,7 @@ export async function validateExternalPlugin(inputUrl, options = {}) {
             ...(api ? { api } : {}),
             ...(css ? { css } : {}),
         },
+        ...(skill ? { skill: { entry: skill.entry, name: skill.name } } : {}),
     };
     installPluginDependencies(root, pkg.dependencies);
     await validateImports(entry);
@@ -449,6 +456,58 @@ export async function installExternalPlugin(url, options = {}) {
     }
     return entry;
 }
+/** Uses local source or checks out the pinned cached commit without fetching. */
+function pinnedPackageRoot(plugin) {
+    if (plugin.path)
+        return pluginRoot(realpathSync(plugin.path), plugin.directory);
+    const dir = sourceDir(plugin.url);
+    if (!existsSync(join(dir, ".git")))
+        throw new Error(`Cached source for ${plugin.id} is missing; run \`kennel plugin add\` again`);
+    run("git", ["checkout", "--force", "--quiet", plugin.commit], { cwd: dir });
+    if (run("git", ["rev-parse", "HEAD"], { cwd: dir }) !== plugin.commit)
+        throw new Error(`Cached source for ${plugin.id} is not at ${plugin.commit}`);
+    return pluginRoot(realpathSync(dir), plugin.directory);
+}
+/** Installs a declared skill from local source or the pinned cached checkout. */
+export function installPluginSkill(id, options = {}) {
+    const path = kennelConfigPath();
+    const config = readJson(path);
+    const plugins = config.externalPlugins ?? [];
+    const plugin = plugins.find((item) => item.id === id);
+    if (!plugin)
+        throw new Error(`No installed plugin with id ${id}`);
+    if (!plugin.skill)
+        throw new Error(`Plugin ${id} declares no skill (kennel.plugin.skill or kennel.plugin.json > skill)`);
+    const skill = readPluginSkill(pinnedPackageRoot(plugin), plugin.skill.entry);
+    if (skill.name !== plugin.skill.name)
+        throw new Error(`Pinned skill name ${skill.name} does not match recorded name ${plugin.skill.name}`);
+    const duplicates = duplicateSkills(skill.name);
+    const commit = plugin.path ? contentHash(pluginRoot(plugin.path, plugin.directory)) : plugin.commit;
+    const written = writeSkill(skill, options);
+    const installed = { path: written.path, commit, digest: written.digest };
+    const nextConfig = {
+        ...config,
+        externalPlugins: plugins.map((item) => item.id === id ? { ...item, skill: { entry: skill.entry, name: skill.name, installed } } : item),
+    };
+    assertUserConfig(nextConfig, path);
+    writeJsonAtomic(path, nextConfig);
+    return { name: skill.name, ...installed, duplicates };
+}
+/** Reports installed plugins and the real state of each declared skill. */
+export function listPluginStatus() {
+    return readExternalPlugins().map((plugin) => ({
+        id: plugin.id,
+        packageName: plugin.packageName,
+        commit: plugin.commit,
+        skill: plugin.skill
+            ? {
+                name: plugin.skill.name,
+                state: skillState(plugin.skill.name, plugin.skill.installed?.digest),
+                path: skillDestination(plugin.skill.name),
+            }
+            : { state: "missing" },
+    }));
+}
 function cliRoot() {
     return resolve(dirname(fileURLToPath(import.meta.url)), "../..");
 }

package/dist/plugins/skills.d.ts

@@ -0,0 +1,30 @@
+export type PluginSkill = {
+    entry: string;
+    name: string;
+    directory: string;
+};
+export type SkillState = "missing" | "installed" | "conflict";
+/** Canonical global agent store: `${AGENTS_HOME:-$HOME/.agents}/skills`. */
+export declare function agentSkillsRoot(): string;
+export declare function skillDestination(name: string): string;
+/**
+ * Resolves `package.json > kennel.plugin.skill` inside a plugin package root.
+ * The entry must be a contained, non-symlinked `SKILL.md` with a valid name and description.
+ */
+export declare function readPluginSkill(packageRoot: string, entry: unknown): PluginSkill;
+/** Content digest of a skill directory: relative paths plus file bytes. */
+export declare function skillDigest(directory: string): string;
+/** Reports the installed skill state at the canonical destination. */
+export declare function skillState(name: string, digest?: string): SkillState;
+/** Same-name skills in known agent stores other than the canonical one. Read only. */
+export declare function duplicateSkills(name: string): string[];
+/**
+ * Copies a resolved skill directory into the canonical agent store.
+ * An existing destination is a conflict unless `force` is set.
+ */
+export declare function writeSkill(skill: PluginSkill, options?: {
+    force?: boolean;
+}): {
+    path: string;
+    digest: string;
+};

package/dist/plugins/skills.js

@@ -0,0 +1,175 @@
+import { createHash } from "node:crypto";
+import { cpSync, existsSync, lstatSync, mkdirSync, readdirSync, readFileSync, realpathSync, renameSync, rmSync, } from "node:fs";
+import { homedir } from "node:os";
+import { basename, dirname, isAbsolute, join, relative, resolve, sep } from "node:path";
+import { parse as parseYaml } from "yaml";
+/** Agent skill folder names follow the Agent Skills naming rule. */
+const VALID_SKILL_NAME = /^[a-z0-9][a-z0-9-]{0,63}$/u;
+/** Never copy repository metadata when a skill lives at a repository root. */
+const SKIPPED_ENTRIES = new Set([".git"]);
+function isInside(root, target) {
+    const rel = relative(root, target);
+    return rel === "" || (!rel.startsWith("..") && !isAbsolute(rel));
+}
+/** Canonical global agent store: `${AGENTS_HOME:-$HOME/.agents}/skills`. */
+export function agentSkillsRoot() {
+    const home = process.env.AGENTS_HOME?.trim() || join(homedir(), ".agents");
+    if (!isAbsolute(home))
+        throw new Error(`AGENTS_HOME must be an absolute path: ${home}`);
+    return resolve(home, "skills");
+}
+export function skillDestination(name) {
+    if (!VALID_SKILL_NAME.test(name))
+        throw new Error(`Invalid skill name: ${name}`);
+    const root = agentSkillsRoot();
+    const target = resolve(root, name);
+    if (dirname(target) !== root)
+        throw new Error(`Refusing skill path outside ${root}: ${name}`);
+    return target;
+}
+function frontmatter(path) {
+    const text = readFileSync(path, "utf8");
+    const match = /^---\r?\n([\s\S]*?)\r?\n---(?:\r?\n|$)/u.exec(text);
+    if (!match)
+        throw new Error(`Skill ${path} has no YAML frontmatter`);
+    const data = parseYaml(match[1] ?? "");
+    if (!data || typeof data !== "object" || Array.isArray(data))
+        throw new Error(`Skill ${path} frontmatter is not a mapping`);
+    return data;
+}
+/**
+ * Resolves `package.json > kennel.plugin.skill` inside a plugin package root.
+ * The entry must be a contained, non-symlinked `SKILL.md` with a valid name and description.
+ */
+export function readPluginSkill(packageRoot, entry) {
+    if (typeof entry !== "string" || !entry.trim())
+        throw new Error("kennel.plugin.skill must be a relative path to SKILL.md");
+    if (isAbsolute(entry) || entry.includes("\0") || entry.includes("\\"))
+        throw new Error(`Unsafe skill path: ${entry}`);
+    if (basename(entry) !== "SKILL.md")
+        throw new Error(`Skill path must name a SKILL.md file: ${entry}`);
+    const root = realpathSync(packageRoot);
+    const file = resolve(root, entry);
+    if (!isInside(root, file) || file === root)
+        throw new Error(`Unsafe skill path: ${entry}`);
+    if (!existsSync(file))
+        throw new Error(`Missing skill file: ${entry}`);
+    const directory = realpathSync(dirname(file));
+    if (!isInside(root, directory))
+        throw new Error(`Skill directory escapes the plugin package: ${entry}`);
+    const stat = lstatSync(join(directory, "SKILL.md"));
+    if (!stat.isFile())
+        throw new Error(`Skill path is not a regular file: ${entry}`);
+    const data = frontmatter(join(directory, "SKILL.md"));
+    const name = typeof data.name === "string" ? data.name.trim() : "";
+    const description = typeof data.description === "string" ? data.description.trim() : "";
+    if (!name)
+        throw new Error(`Skill ${entry} frontmatter needs a nonempty name`);
+    if (!description)
+        throw new Error(`Skill ${entry} frontmatter needs a nonempty description`);
+    if (!VALID_SKILL_NAME.test(name))
+        throw new Error(`Skill ${entry} frontmatter name must match ${VALID_SKILL_NAME}: ${name}`);
+    return { entry: relative(root, join(directory, "SKILL.md")).split(sep).join("/"), name, directory };
+}
+/** Lists regular files under a skill directory. Symlinks and special files are refused. */
+function skillFiles(directory, prefix = "") {
+    return readdirSync(join(directory, prefix), { withFileTypes: true })
+        .filter((item) => !SKIPPED_ENTRIES.has(item.name))
+        .sort((a, b) => a.name.localeCompare(b.name))
+        .flatMap((item) => {
+        const rel = prefix ? join(prefix, item.name) : item.name;
+        if (item.isDirectory())
+            return skillFiles(directory, rel);
+        if (item.isFile())
+            return [rel];
+        throw new Error(`Skill contains an unsupported entry (symlink or special file): ${join(directory, rel)}`);
+    });
+}
+/** Content digest of a skill directory: relative paths plus file bytes. */
+export function skillDigest(directory) {
+    const hash = createHash("sha256");
+    for (const file of skillFiles(directory)) {
+        hash.update(file.split(sep).join("/")).update("\0").update(readFileSync(join(directory, file))).update("\0");
+    }
+    return hash.digest("hex");
+}
+/** Reports the installed skill state at the canonical destination. */
+export function skillState(name, digest) {
+    const target = skillDestination(name);
+    if (!existsSync(target) && !isSymlink(target))
+        return "missing";
+    try {
+        if (!lstatSync(target).isDirectory())
+            return "conflict";
+        return digest && skillDigest(target) === digest ? "installed" : "conflict";
+    }
+    catch {
+        return "conflict";
+    }
+}
+function isSymlink(path) {
+    try {
+        return lstatSync(path).isSymbolicLink();
+    }
+    catch {
+        return false;
+    }
+}
+/** Same-name skills in known agent stores other than the canonical one. Read only. */
+export function duplicateSkills(name) {
+    const canonical = agentSkillsRoot();
+    const roots = [
+        process.env.CODEX_HOME?.trim() ? join(process.env.CODEX_HOME.trim(), "skills") : undefined,
+        join(homedir(), ".codex", "skills"),
+        join(homedir(), ".claude", "skills"),
+    ].filter((root) => Boolean(root));
+    const canonicalReal = existsSync(canonical) ? realpathSync(canonical) : canonical;
+    const seen = new Set();
+    return roots.flatMap((root) => {
+        const real = existsSync(root) ? realpathSync(root) : resolve(root);
+        if (real === canonicalReal || seen.has(real))
+            return [];
+        seen.add(real);
+        const path = join(root, name);
+        return existsSync(path) || isSymlink(path) ? [path] : [];
+    });
+}
+/**
+ * Copies a resolved skill directory into the canonical agent store.
+ * An existing destination is a conflict unless `force` is set.
+ */
+export function writeSkill(skill, options = {}) {
+    const root = agentSkillsRoot();
+    const target = skillDestination(skill.name);
+    const exists = existsSync(target) || isSymlink(target);
+    if (exists && !options.force) {
+        throw new Error(`Skill conflict: ${target} already exists. Re-run with --force to replace it.`);
+    }
+    const files = skillFiles(skill.directory);
+    mkdirSync(root, { recursive: true });
+    const temp = join(root, `.${skill.name}.kennel-${process.pid}-${Date.now()}`);
+    try {
+        for (const file of files) {
+            const from = join(skill.directory, file);
+            const to = join(temp, file);
+            if (!isInside(skill.directory, from) || !isInside(temp, to))
+                throw new Error(`Unsafe skill file: ${file}`);
+            mkdirSync(dirname(to), { recursive: true });
+            cpSync(from, to, { errorOnExist: true, force: false, preserveTimestamps: true });
+        }
+        const digest = skillDigest(temp);
+        if (exists) {
+            const previous = `${temp}.previous`;
+            renameSync(target, previous);
+            renameSync(temp, target);
+            rmSync(previous, { force: true, recursive: true });
+        }
+        else {
+            renameSync(temp, target);
+        }
+        return { path: target, digest };
+    }
+    finally {
+        rmSync(temp, { force: true, recursive: true });
+    }
+}

package/dist/site/folder-icons.d.ts

@@ -20,13 +20,6 @@ export declare function assetKey(value: Buffer): string;
 export declare function encodeBase64Url(value: string): string;
 export declare function decodeBase64Url(value: string): string;
 export declare function iconUrl(folder: string, hash: string, mediaType: IconMediaType): string;
-/**
- * Nearest ancestor icon for a folder, walking up parent directories so
- * nested folders inherit the closest icon above them. The bundled
- * `@raggle-ai/local` does not export the ancestor-aware discovery helper
- * yet, so the walk is implemented here over `discoverProjectIcon`.
- */
-export declare function discoverAncestorIcon(folder: string): string | undefined;
 export declare function folderIcon(folder: string, assets: Record<string, IconAsset>, assetFiles: Map<string, {
     mediaType: IconMediaType;
     path: string;

package/dist/site/folder-icons.js

@@ -1,7 +1,7 @@
 import { createHash } from 'node:crypto';
 import { readFile, stat } from 'node:fs/promises';
-import { dirname, extname } from 'node:path';
-import { discoverProjectIcon } from '@raggle-ai/local';
+import { extname } from 'node:path';
+import { discoverFolderIcon } from '@raggle-ai/local';
 /**
  * Folder icons are served as immutable, content-addressed assets, so the
  * cap is generous: it only guards against pathological files, not normal
@@ -34,26 +34,8 @@ export function decodeBase64Url(value) {
 export function iconUrl(folder, hash, mediaType) {
     return `/api/folders/${encodeBase64Url(folder)}/assets/${hash}.${iconExtensions.get(mediaType)}`;
 }
-/**
- * Nearest ancestor icon for a folder, walking up parent directories so
- * nested folders inherit the closest icon above them. The bundled
- * `@raggle-ai/local` does not export the ancestor-aware discovery helper
- * yet, so the walk is implemented here over `discoverProjectIcon`.
- */
-export function discoverAncestorIcon(folder) {
-    let current = folder;
-    while (true) {
-        const icon = discoverProjectIcon(current);
-        if (icon)
-            return icon;
-        const parent = dirname(current);
-        if (parent === current)
-            return undefined;
-        current = parent;
-    }
-}
 export async function folderIcon(folder, assets, assetFiles, fallback) {
-    const icon = discoverAncestorIcon(folder);
+    const icon = discoverFolderIcon(folder);
     const mediaType = icon ? iconTypes.get(extname(icon).toLowerCase()) : null;
     const details = icon && mediaType ? await stat(icon).catch(() => undefined) : undefined;
     if (!icon || !mediaType || !details?.size || details.size > maxIconBytes) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kennel",
-  "version": "0.2.23",
+  "version": "0.2.24",
   "description": "Open-source pop-up previews for AI output: markdown, HTML artifacts, email, images, code, tables, notes, and calendars, from the terminal or your agent session.",
   "type": "module",
   "license": "MIT",

package/schemas/config.schema.json

@@ -79,6 +79,7 @@
           "type": "string",
           "pattern": "^(?!\\.{1,2}$)[^/\\\\]+$"
         },
+        "skill": { "$ref": "#/$defs/pluginSkill" },
         "entries": {
           "description": "Resolved absolute convention entry files. api is optional.",
           "oneOf": [
@@ -88,6 +89,35 @@
         }
       }
     },
+    "pluginSkill": {
+      "description": "Agent skill declared by package.json > kennel.plugin.skill or kennel.plugin.json > skill. Managed by `kennel plugin add` and `kennel plugin skill install`.",
+      "type": "object",
+      "additionalProperties": false,
+      "required": ["entry", "name"],
+      "properties": {
+        "entry": {
+          "description": "SKILL.md path relative to the plugin package root.",
+          "type": "string",
+          "pattern": "^(?!/)(?:[^\\\\]*/)?SKILL\\.md$"
+        },
+        "name": {
+          "description": "Skill frontmatter name; the agent-store folder name.",
+          "type": "string",
+          "pattern": "^[a-z0-9][a-z0-9-]{0,63}$"
+        },
+        "installed": {
+          "description": "Last install into ${AGENTS_HOME:-$HOME/.agents}/skills.",
+          "type": "object",
+          "additionalProperties": false,
+          "required": ["path", "commit", "digest"],
+          "properties": {
+            "path": { "$ref": "#/$defs/absolutePath" },
+            "commit": { "description": "Pinned Git commit or local source SHA-256.", "type": "string", "pattern": "^(?:[0-9a-f]{40}|[0-9a-f]{64})$" },
+            "digest": { "description": "SHA-256 of the installed skill files.", "type": "string", "pattern": "^[0-9a-f]{64}$" }
+          }
+        }
+      }
+    },
     "entriesWithoutApi": {
       "type": "object",
       "additionalProperties": false,