dsh-continual-evolve 0.10.6 → 0.12.0

package/lib/skill-render.js

@@ -1,69 +0,0 @@
-/** Convert a harness entry id (underscore slug) to a kebab-case skill name. */
-export function skillNameOf(id) {
-    return id.toLowerCase().replace(/_/g, "-");
-}
-/**
- * First content line usable as a routing hint: non-empty, not a Markdown
- * heading, not a list marker, not frontmatter. Undefined when the body is
- * effectively empty.
- */
-function routingHint(content) {
-    for (const rawLine of content.split("\n")) {
-        const line = rawLine.trim();
-        if (line.length === 0)
-            continue;
-        if (line.startsWith("#") || line.startsWith("---") || line.startsWith("-") || line.startsWith("*"))
-            continue;
-        return oneLine(line);
-    }
-    return undefined;
-}
-/** Max rendered frontmatter description length (loaders truncate anyway). */
-const MAX_DESCRIPTION_LENGTH = 240;
-/**
- * Render a harness skill entry as a discoverable SKILL.md document.
- *
- * 2026-08-22: the frontmatter description now carries a ROUTING HINT —
- * title plus the first meaningful content line — instead of the bare title.
- * The skill catalog matches on description; a title-only description gave
- * loaders nothing to route on (observed: materialized skills were 7-line
- * stubs with a one-line description and no use-when signal).
- */
-export function renderSkillMarkdown(entry) {
-    const hint = routingHint(entry.content);
-    const base = oneLine(entry.title);
-    let description;
-    if (base.length === 0) {
-        description = (hint ?? "").slice(0, MAX_DESCRIPTION_LENGTH);
-    }
-    else if (hint !== undefined && !base.toLowerCase().includes(hint.toLowerCase())) {
-        description = `${base} — use when: ${hint}`.slice(0, MAX_DESCRIPTION_LENGTH);
-    }
-    else {
-        description = base.slice(0, MAX_DESCRIPTION_LENGTH);
-    }
-    const lines = [
-        "---",
-        `name: ${skillNameOf(entry.id)}`,
-        `description: ${description}`,
-        "---",
-        "",
-        entry.content.trim(),
-    ];
-    const reference = entry.reference;
-    if (reference && typeof reference === "object" && Object.keys(reference).length > 0) {
-        lines.push("", "## Invocation");
-        for (const [key, value] of Object.entries(reference)) {
-            lines.push(`- ${key}: ${JSON.stringify(value)}`);
-        }
-    }
-    const args = entry.arguments;
-    if (args && typeof args === "object" && Object.keys(args).length > 0) {
-        lines.push("", "## Arguments", "```json", JSON.stringify(entry.arguments, null, 2), "```");
-    }
-    return `${lines.join("\n").trimEnd()}\n`;
-}
-function oneLine(text) {
-    return text.replace(/\s+/g, " ").trim();
-}
-//# sourceMappingURL=skill-render.js.map

package/lib/skill.d.ts

@@ -1,15 +0,0 @@
-import type { RefinementResult } from "./types.js";
-export { renderSkillMarkdown, skillNameOf } from "./skill-render.js";
-/** Resolve and defend the skill directory for an entry id. */
-export declare function skillDir(skillsRoot: string, id: string): string;
-/**
- * Apply the skill-kind edits of an applied refinement to the skills root.
- * Returns materialization warnings (rendered-SKILL.md mechanical problems
- * and dangling resource references) — the file is still written, but the
- * caller should surface them: a rendered file that fails the platform's
- * frontmatter rules would be IGNORED by the skill loader, and a body
- * referencing resources the entry does not ship would load with broken
- * links.
- */
-export declare function syncSkillsFromResult(skillsRoot: string, result: RefinementResult): string[];
-//# sourceMappingURL=skill.d.ts.map

package/lib/skill.js

@@ -1,80 +0,0 @@
-/**
- * Skill materializer: syncs skill-kind harness entries to the DSH skills
- * filesystem (`$DSH_HOME/skills/<kebab-name>/SKILL.md`) so the `skill` tool
- * and catalog can discover and load them. Writes are atomic (tmp + rename)
- * so the filesystem watcher never sees a partial file.
- *
- * Skill names must be kebab-case (the harness store ids are underscore slugs;
- * the materialized name converts `_` → `-`).
- */
-import { existsSync, mkdirSync, renameSync, rmSync, writeFileSync } from "node:fs";
-import { join, resolve, sep } from "node:path";
-import { renderSkillMarkdown, skillNameOf } from "./skill-render.js";
-import { skillResourceRefs, validateRenderedSkill } from "./skillquality.js";
-export { renderSkillMarkdown, skillNameOf } from "./skill-render.js";
-/** Resolve and defend the skill directory for an entry id. */
-export function skillDir(skillsRoot, id) {
-    const root = resolve(skillsRoot);
-    const dir = resolve(join(root, skillNameOf(id)));
-    if (dir !== root && !dir.startsWith(`${root}/`) && !dir.startsWith(`${root}${sep}`)) {
-        throw new Error(`skill path escapes skills root: ${dir}`);
-    }
-    return dir;
-}
-/**
- * Apply the skill-kind edits of an applied refinement to the skills root.
- * Returns materialization warnings (rendered-SKILL.md mechanical problems
- * and dangling resource references) — the file is still written, but the
- * caller should surface them: a rendered file that fails the platform's
- * frontmatter rules would be IGNORED by the skill loader, and a body
- * referencing resources the entry does not ship would load with broken
- * links.
- */
-export function syncSkillsFromResult(skillsRoot, result) {
-    const warnings = [];
-    for (const edit of result.appliedEdits) {
-        if (edit.kind !== "skill" || !edit.applied)
-            continue;
-        if (edit.action === "delete" || !edit.after) {
-            removeSkill(skillsRoot, edit.id);
-            continue;
-        }
-        warnings.push(...writeSkill(skillsRoot, edit.after));
-    }
-    return warnings;
-}
-/** Write one skill entry as a SKILL.md; returns materialization warnings. */
-function writeSkill(skillsRoot, entry) {
-    const dir = skillDir(skillsRoot, entry.id);
-    mkdirSync(dir, { recursive: true });
-    const temp = join(dir, `SKILL.md.${process.pid}.tmp`);
-    writeFileSync(temp, renderSkillMarkdown(entry), "utf8");
-    renameSync(temp, join(dir, "SKILL.md"));
-    return materializationWarnings(dir, entry);
-}
-/** Post-write checks on the exact file that landed on disk. */
-function materializationWarnings(dir, entry) {
-    const warnings = [];
-    for (const problem of validateRenderedSkill(entry)) {
-        warnings.push(`skill ${entry.id}: rendered SKILL.md would be ignored by the platform: ${problem}`);
-    }
-    const root = resolve(dir);
-    for (const ref of skillResourceRefs(entry.content)) {
-        const target = resolve(root, ref);
-        if (target !== root && !target.startsWith(`${root}${sep}`)) {
-            warnings.push(`skill ${entry.id}: body resource reference escapes the skill directory: ${ref}`);
-            continue;
-        }
-        if (!existsSync(target)) {
-            warnings.push(`skill ${entry.id}: body references missing resource ${ref} (expected at ${target})`);
-        }
-    }
-    return warnings;
-}
-function removeSkill(skillsRoot, id) {
-    const dir = skillDir(skillsRoot, id);
-    if (existsSync(dir)) {
-        rmSync(dir, { recursive: true, force: true });
-    }
-}
-//# sourceMappingURL=skill.js.map

package/lib/skillquality.d.ts

@@ -1,85 +0,0 @@
-import type { HarnessEntry } from "./types.js";
-/** Relative location of the skill-creator template facts. */
-export declare const SKILL_CREATOR_TEMPLATE_REL: string;
-/**
- * Read the skill-creator template facts
- * (`<skillsRoot>/skill-creator/references/template.md`; facts distilled
- * from the official deepseek-harness skills). Returns null when the skills
- * are not installed — callers fall back to the builtin distilled guide.
- * Reading is a runtime reference, never a copy: template updates in the
- * skill are picked up automatically.
- */
-export declare function readSkillCreatorTemplate(skillsRoot: string): string | null;
-/**
- * Builtin distilled skill-quality guide (fallback when the skill-creator
- * template is not installed). Condenses the template facts — frontmatter
- * schema, the 7 structural features, paragraph skeleton, and the
- * no-duplication / real-trigger rules — so a planner still authors skills
- * to the standard on installs without the skill-creator / skill-audit
- * skills.
- */
-export declare const BUILTIN_SKILL_QUALITY_GUIDE = "DSH skill quality standard (distilled by the author from the official deepseek-harness 11 skills; the full facts live in <skillsRoot>/skill-creator/references/template.md when installed):\n\nFrontmatter schema (platform-enforced; violations make the platform IGNORE the whole file):\n- name: required, kebab-case only (^[a-z0-9]+(?:-[a-z0-9]+)*$)\n- description: required, non-empty; write \"use when / do not use when\" routing so the model can select it correctly\n- invocation booleans accept true/false/yes/no/on/off/1/0; legacy camelCase keys (disableModelInvocation / modelInvocable / userInvocable) are rejected\n- whenToUse (optional): non-empty string; metadata (optional): object\n\nThe 7 structural features of the official deepseek-harness skills:\n1. Frontmatter is routing metadata, not a summary (description = when to use / when not to use)\n2. Opens with a boundary declaration (guidance, not a script; mechanical flow skills may omit the disclaimer)\n3. Prerequisites + exclusions: explicit required input, stop when missing (report the required input and stop), excluded scenarios\n4. Layered information: Sources of truth (link only, do not re-summarize) -> numbered blocking requirements -> manual checks -> verification commands -> report format; all executable, no slogans\n5. Skill interlinks: reference a single source of truth instead of duplicating it\n6. Verifiable completion criteria: explicit verification commands and report format\n7. Real use + iteration: a real trigger scenario must exist; calibration conclusions distill into references/\n\nParagraph skeleton (writing order): frontmatter -> H1 + boundary declaration -> Sources of truth -> numbered requirements / workflow (full commands) -> exclusions / stop conditions -> verification and report.\n\nCreation rules: only create a skill for a REAL trigger scenario (who, in what real task, what signal) grounded in the trajectory \u2014 never invent one to pad the store; do not duplicate the official 11 skills or existing entries; skill bodies should be a SKILL.md document (this is what materializes under <skillsRoot>/<kebab-name>/SKILL.md).";
-export interface SkillQualityGuide {
-    /** Where the guide text came from: the on-disk template or the builtin guide. */
-    source: "template" | "builtin";
-    text: string;
-}
-/**
- * The quality guide handed to the planner: the skill-creator template facts
- * when the skills are installed, otherwise the builtin distilled guide.
- * Never throws — a missing/unreadable template degrades to the builtin.
- */
-export declare function skillQualityGuide(skillsRoot: string | undefined): SkillQualityGuide;
-/** Split frontmatter out of a raw SKILL.md. Returns { yaml, body } or null when delimiters are missing. */
-export declare function splitFrontmatter(raw: string): {
-    yaml: string;
-    body: string;
-} | null;
-/**
- * Mechanical frontmatter validation of a rendered SKILL.md, mirroring
- * `skill-creator/scripts/validate-frontmatter.mjs` (and the platform's
- * skill-filesystem rules): delimiter structure, name kebab-case, non-empty
- * description, invocation-boolean spellings, legacy camelCase key rejection,
- * whenToUse/metadata types. Returns human-readable problems; an empty array
- * means the file would load.
- */
-export declare function validateRenderedSkillMarkdown(markdown: string): string[];
-/**
- * Mechanical validation of a skill entry's raw `content` (the SKILL.md body
- * that materializes under the generated frontmatter). Code-enforced at
- * apply time so a bad entry never reaches the store:
- * - empty content is rejected;
- * - content must not open with a `---` block: the materializer generates
- *   its own frontmatter, and a second frontmatter in the body would be
- *   parsed instead of the generated one (the platform reads the FIRST
- *   closing `---`), so the file could be ignored or routed wrongly;
- * - resource references (`references/…`, `scripts/…`) must be skill-local
- *   relative paths — an embedded `/../` after the category escapes the skill
- *   directory and is rejected. A leading `../` instead addresses a sibling
- *   skill (cross-skill interlink, e.g. `../skill-creator/…`) and is allowed:
- *   it is not dangling-checked since cross-skill targets live outside this
- *   entry's shipped files (see skillResourceRefs).
- * Returns human-readable problems; an empty array means the content is
- * mechanically acceptable.
- */
-export declare function validateSkillEntryContent(content: string): string[];
-/**
- * Validate the FULL rendered SKILL.md of an entry (generated frontmatter +
- * body) — the exact bytes that materialize on disk. Used as the final
- * code-enforced line after materialization; problems here mean the platform
- * would refuse to load the file.
- */
-export declare function validateRenderedSkill(entry: HarnessEntry): string[];
-/**
- * Resource references (`references/…`, `scripts/…`) found in a skill body —
- * the same scanning policy as validate-frontmatter.mjs: markdown link
- * targets starting with the category, plus backticked/prose paths carrying
- * a filename extension. Used after materialization to warn about dangling
- * references (a body referencing a resource the entry never ships).
- * Leading-`../` cross-skill interlinks are skipped, not warned: they resolve
- * against sibling skill directories, never against shipped files.
- */
-export declare function skillResourceRefs(content: string): string[];
-/** Kebab-case name under which the entry materializes (exported for diagnostics). */
-export { skillNameOf } from "./skill-render.js";
-//# sourceMappingURL=skillquality.d.ts.map

package/lib/skillquality.js

@@ -1,320 +0,0 @@
-/**
- * Skill-quality integration: makes the DSH skill quality standard — carried
- * by the author-distilled skills skill-creator / skill-audit (distilled
- * from the official deepseek-harness 11 skills, facts verified against
- * deepseek-harness 47f9438) — usable INSIDE the self-evolution loop.
- *
- * The planner and review gate are raw `ctx.llm` calls — they do not live in
- * an agent session, so they cannot load skills through the `skill` tool.
- * The skill-creator / skill-audit skills stay the single source of truth on
- * disk; this module only:
- *
- * 1. reads the template facts at runtime
- *    (`<skillsRoot>/skill-creator/references/template.md`, 85 lines) and
- *    hands them to the planner as a `<skill_quality_standard>` block —
- *    the on-disk template wins, a built-in distilled guide is the fallback
- *    for installs without these skills;
- * 2. code-enforces the mechanical frontmatter rules of
- *    `skill-creator/scripts/validate-frontmatter.mjs` (the platform would
- *    IGNORE a file that fails them), so a skill entry can never materialize
- *    a SKILL.md the platform refuses to load.
- */
-import { existsSync, readFileSync } from "node:fs";
-import { join } from "node:path";
-import { renderSkillMarkdown } from "./skill-render.js";
-/** Skill-name regex the platform enforces (skill-filesystem). */
-const NAME_RE = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
-const TRUE_WORDS = new Set(["true", "yes", "on", "1"]);
-const FALSE_WORDS = new Set(["false", "no", "off", "0"]);
-const LEGACY_KEYS = ["disableModelInvocation", "modelInvocable", "userInvocable"];
-const CANONICAL_KEYS = {
-    disableModelInvocation: "disable-model-invocation",
-    modelInvocable: "disable-model-invocation",
-    userInvocable: "user-invocable",
-};
-/** Relative location of the skill-creator template facts. */
-export const SKILL_CREATOR_TEMPLATE_REL = join("skill-creator", "references", "template.md");
-/**
- * Read the skill-creator template facts
- * (`<skillsRoot>/skill-creator/references/template.md`; facts distilled
- * from the official deepseek-harness skills). Returns null when the skills
- * are not installed — callers fall back to the builtin distilled guide.
- * Reading is a runtime reference, never a copy: template updates in the
- * skill are picked up automatically.
- */
-export function readSkillCreatorTemplate(skillsRoot) {
-    const path = join(skillsRoot, SKILL_CREATOR_TEMPLATE_REL);
-    try {
-        if (!existsSync(path))
-            return null;
-        const text = readFileSync(path, "utf8");
-        return text.trim().length > 0 ? text : null;
-    }
-    catch {
-        return null;
-    }
-}
-/**
- * Builtin distilled skill-quality guide (fallback when the skill-creator
- * template is not installed). Condenses the template facts — frontmatter
- * schema, the 7 structural features, paragraph skeleton, and the
- * no-duplication / real-trigger rules — so a planner still authors skills
- * to the standard on installs without the skill-creator / skill-audit
- * skills.
- */
-export const BUILTIN_SKILL_QUALITY_GUIDE = `DSH skill quality standard (distilled by the author from the official deepseek-harness 11 skills; the full facts live in <skillsRoot>/skill-creator/references/template.md when installed):
-
-Frontmatter schema (platform-enforced; violations make the platform IGNORE the whole file):
-- name: required, kebab-case only (^[a-z0-9]+(?:-[a-z0-9]+)*$)
-- description: required, non-empty; write "use when / do not use when" routing so the model can select it correctly
-- invocation booleans accept true/false/yes/no/on/off/1/0; legacy camelCase keys (disableModelInvocation / modelInvocable / userInvocable) are rejected
-- whenToUse (optional): non-empty string; metadata (optional): object
-
-The 7 structural features of the official deepseek-harness skills:
-1. Frontmatter is routing metadata, not a summary (description = when to use / when not to use)
-2. Opens with a boundary declaration (guidance, not a script; mechanical flow skills may omit the disclaimer)
-3. Prerequisites + exclusions: explicit required input, stop when missing (report the required input and stop), excluded scenarios
-4. Layered information: Sources of truth (link only, do not re-summarize) -> numbered blocking requirements -> manual checks -> verification commands -> report format; all executable, no slogans
-5. Skill interlinks: reference a single source of truth instead of duplicating it
-6. Verifiable completion criteria: explicit verification commands and report format
-7. Real use + iteration: a real trigger scenario must exist; calibration conclusions distill into references/
-
-Paragraph skeleton (writing order): frontmatter -> H1 + boundary declaration -> Sources of truth -> numbered requirements / workflow (full commands) -> exclusions / stop conditions -> verification and report.
-
-Creation rules: only create a skill for a REAL trigger scenario (who, in what real task, what signal) grounded in the trajectory — never invent one to pad the store; do not duplicate the official 11 skills or existing entries; skill bodies should be a SKILL.md document (this is what materializes under <skillsRoot>/<kebab-name>/SKILL.md).`;
-/**
- * The quality guide handed to the planner: the skill-creator template facts
- * when the skills are installed, otherwise the builtin distilled guide.
- * Never throws — a missing/unreadable template degrades to the builtin.
- */
-export function skillQualityGuide(skillsRoot) {
-    if (skillsRoot) {
-        const template = readSkillCreatorTemplate(skillsRoot);
-        if (template !== null) {
-            return {
-                source: "template",
-                text: `The skill-creator template facts (distilled from the official deepseek-harness 11 skills, verified against deepseek-harness 47f9438; single source of truth, read from <skillsRoot>/skill-creator/references/template.md):\n\n${template}`,
-            };
-        }
-    }
-    return { source: "builtin", text: BUILTIN_SKILL_QUALITY_GUIDE };
-}
-/** Split frontmatter out of a raw SKILL.md. Returns { yaml, body } or null when delimiters are missing. */
-export function splitFrontmatter(raw) {
-    const lines = raw.split(/\r?\n/);
-    if (lines[0] !== "---")
-        return null;
-    const close = lines.indexOf("---", 1);
-    if (close < 0)
-        return null;
-    return { yaml: lines.slice(1, close).join("\n"), body: lines.slice(close + 1).join("\n") };
-}
-/** Parse one scalar in the YAML subset the platform's schema keys use. */
-function parseScalar(raw, lineNo) {
-    const value = raw.trim();
-    if (value === "")
-        return "";
-    if (value.startsWith("'")) {
-        if (!value.endsWith("'"))
-            throw new Error(`unterminated single-quoted scalar at line ${lineNo}`);
-        return value.slice(1, -1).replace(/''/g, "'");
-    }
-    if (value.startsWith('"')) {
-        if (!value.endsWith('"'))
-            throw new Error(`unterminated double-quoted scalar at line ${lineNo}`);
-        return value
-            .slice(1, -1)
-            .replace(/\\n/g, "\n")
-            .replace(/\\t/g, "\t")
-            .replace(/\\"/g, '"')
-            .replace(/\\\\/g, "\\");
-    }
-    return value;
-}
-/** Minimal YAML-subset parser for the flat schema the platform reads (mirrors validate-frontmatter.mjs). */
-function parseMiniYaml(text) {
-    const data = {};
-    const lines = text.split("\n");
-    let i = 0;
-    while (i < lines.length) {
-        const line = lines[i] ?? "";
-        const lineNo = i + 1;
-        i += 1;
-        const trimmed = line.trim();
-        if (trimmed === "" || trimmed.startsWith("#"))
-            continue;
-        if (line.length - line.trimStart().length > 0) {
-            throw new Error(`unsupported indented construct at line ${lineNo}: ${trimmed}`);
-        }
-        const match = /^([A-Za-z0-9_-]+):(?:\s+(.*))?$/.exec(trimmed);
-        if (!match)
-            throw new Error(`unparseable line ${lineNo}: ${trimmed}`);
-        const key = match[1] ?? "";
-        let value = match[2] ?? "";
-        if (value === "|" || value === ">") {
-            const block = [];
-            while (i < lines.length && (lines[i] ?? "").trim() !== "" && (lines[i] ?? "").startsWith(" ")) {
-                block.push(lines[i] ?? "");
-                i += 1;
-            }
-            data[key] = block.join("\n");
-            continue;
-        }
-        if (value === "" && i < lines.length && (lines[i] ?? "").startsWith(" ") && (lines[i] ?? "").trim() !== "") {
-            const nested = {};
-            while (i < lines.length && (lines[i] ?? "").trim() !== "" && (lines[i] ?? "").startsWith(" ")) {
-                const nm = /^([A-Za-z0-9_-]+):(?:\s+(.*))?$/.exec((lines[i] ?? "").trim());
-                if (!nm)
-                    throw new Error(`unparseable nested line ${i + 1}: ${(lines[i] ?? "").trim()}`);
-                nested[nm[1] ?? ""] = parseScalar(nm[2] ?? "", i + 1);
-                i += 1;
-            }
-            data[key] = nested;
-            continue;
-        }
-        data[key] = parseScalar(value, lineNo);
-    }
-    return data;
-}
-/**
- * Mechanical frontmatter validation of a rendered SKILL.md, mirroring
- * `skill-creator/scripts/validate-frontmatter.mjs` (and the platform's
- * skill-filesystem rules): delimiter structure, name kebab-case, non-empty
- * description, invocation-boolean spellings, legacy camelCase key rejection,
- * whenToUse/metadata types. Returns human-readable problems; an empty array
- * means the file would load.
- */
-export function validateRenderedSkillMarkdown(markdown) {
-    const problems = [];
-    const split = splitFrontmatter(markdown);
-    if (!split) {
-        return [
-            "missing YAML frontmatter (first line `---` with a closing `---`) — platform would IGNORE this file",
-        ];
-    }
-    let data;
-    try {
-        data = parseMiniYaml(split.yaml);
-    }
-    catch (cause) {
-        return [`invalid YAML frontmatter: ${cause instanceof Error ? cause.message : String(cause)} — platform would IGNORE this file`];
-    }
-    const name = typeof data["name"] === "string" && data["name"].length > 0 ? data["name"] : undefined;
-    if (name === undefined) {
-        problems.push("frontmatter requires non-empty `name` — platform would IGNORE this file");
-    }
-    else if (!NAME_RE.test(name)) {
-        problems.push(`invalid skill name "${name}" (must match ${NAME_RE}) — platform would IGNORE this file`);
-    }
-    const description = typeof data["description"] === "string" && data["description"].length > 0 ? data["description"] : undefined;
-    if (description === undefined) {
-        problems.push("frontmatter requires non-empty `description` — platform would IGNORE this file");
-    }
-    for (const legacy of LEGACY_KEYS) {
-        if (Object.hasOwn(data, legacy)) {
-            problems.push(`legacy key "${legacy}" is unsupported; use "${CANONICAL_KEYS[legacy]}" — platform would IGNORE this file`);
-        }
-    }
-    for (const key of ["disable-model-invocation", "user-invocable"]) {
-        if (!Object.hasOwn(data, key))
-            continue;
-        const value = data[key];
-        if (typeof value === "boolean")
-            continue;
-        const word = String(value).toLowerCase();
-        if (TRUE_WORDS.has(word) || FALSE_WORDS.has(word))
-            continue;
-        problems.push(`frontmatter field "${key}" must be a boolean (accepted: true/false/yes/no/on/off/1/0), got ${JSON.stringify(value)} — platform would IGNORE this file`);
-    }
-    if (Object.hasOwn(data, "whenToUse") && !(typeof data["whenToUse"] === "string" && data["whenToUse"].length > 0)) {
-        problems.push("`whenToUse` must be a non-empty string when present");
-    }
-    if (Object.hasOwn(data, "metadata") && (typeof data["metadata"] !== "object" || data["metadata"] === null || Array.isArray(data["metadata"]))) {
-        problems.push("`metadata` must be an object when present");
-    }
-    return problems;
-}
-/**
- * Mechanical validation of a skill entry's raw `content` (the SKILL.md body
- * that materializes under the generated frontmatter). Code-enforced at
- * apply time so a bad entry never reaches the store:
- * - empty content is rejected;
- * - content must not open with a `---` block: the materializer generates
- *   its own frontmatter, and a second frontmatter in the body would be
- *   parsed instead of the generated one (the platform reads the FIRST
- *   closing `---`), so the file could be ignored or routed wrongly;
- * - resource references (`references/…`, `scripts/…`) must be skill-local
- *   relative paths — an embedded `/../` after the category escapes the skill
- *   directory and is rejected. A leading `../` instead addresses a sibling
- *   skill (cross-skill interlink, e.g. `../skill-creator/…`) and is allowed:
- *   it is not dangling-checked since cross-skill targets live outside this
- *   entry's shipped files (see skillResourceRefs).
- * Returns human-readable problems; an empty array means the content is
- * mechanically acceptable.
- */
-export function validateSkillEntryContent(content) {
-    const problems = [];
-    const trimmed = content.trim();
-    if (trimmed.length === 0) {
-        problems.push("skill content is empty");
-        return problems;
-    }
-    if (trimmed.startsWith("---")) {
-        problems.push("skill content must not start with a `---` frontmatter block (the materializer generates frontmatter from id/title; a body-level `---` would shadow it and the platform could IGNORE the file)");
-    }
-    for (const match of trimmed.matchAll(/(?<![\w])(references|scripts)\/[^\s)]+/g)) {
-        // match[0] always starts with `references/` or `scripts/` (the regex
-        // anchors it there), so only an embedded `/../` can escape the skill
-        // directory — the ../-prefix, absolute-path, and drive-letter
-        // spellings can never match and are intentionally not checked.
-        const ref = match[0] ?? "";
-        if (ref.includes("/../")) {
-            problems.push(`skill content resource reference escapes the skill directory: ${ref}`);
-        }
-    }
-    return problems;
-}
-/**
- * Validate the FULL rendered SKILL.md of an entry (generated frontmatter +
- * body) — the exact bytes that materialize on disk. Used as the final
- * code-enforced line after materialization; problems here mean the platform
- * would refuse to load the file.
- */
-export function validateRenderedSkill(entry) {
-    return validateRenderedSkillMarkdown(renderSkillMarkdown(entry));
-}
-/**
- * Resource references (`references/…`, `scripts/…`) found in a skill body —
- * the same scanning policy as validate-frontmatter.mjs: markdown link
- * targets starting with the category, plus backticked/prose paths carrying
- * a filename extension. Used after materialization to warn about dangling
- * references (a body referencing a resource the entry never ships).
- * Leading-`../` cross-skill interlinks are skipped, not warned: they resolve
- * against sibling skill directories, never against shipped files.
- */
-export function skillResourceRefs(content) {
-    const refs = new Set();
-    for (const match of content.matchAll(/\[[^\]]*\]\(([^)]+)\)/g)) {
-        const target = ((match[1] ?? "").trim().split(/\s+/)[0] ?? "").trim();
-        if (/^(references|scripts)\/[\w./-]+$/.test(target) && !target.startsWith("../")) {
-            refs.add(target);
-        }
-    }
-    const stripped = content.replace(/\[[^\]]*\]\([^)]*\)/g, "");
-    for (const match of stripped.matchAll(/(?<![\w])(references|scripts)\/[\w./-]+\.\w+/g)) {
-        const path = match[0] ?? "";
-        // Cross-skill interlinks (`../skill-creator/...`) resolve against the
-        // sibling skill's directory, not this one — skip references whose
-        // prose prefix walks up a directory (mirrors validate-frontmatter.mjs).
-        let cursor = (match.index ?? 0) - 1;
-        while (cursor >= 0 && /[\w./-]/.test(stripped[cursor] ?? ""))
-            cursor -= 1;
-        if (!stripped.slice(cursor + 1, match.index ?? 0).includes("..")) {
-            refs.add(path);
-        }
-    }
-    return [...refs];
-}
-/** Kebab-case name under which the entry materializes (exported for diagnostics). */
-export { skillNameOf } from "./skill-render.js";
-//# sourceMappingURL=skillquality.js.map

package/lib/source.d.ts

@@ -1,30 +0,0 @@
-/**
- * Trajectory citations: where distilled entries came from in the session
- * log. DSH sessions are event-sourced with contiguous seq numbers, so a
- * citation of (sessionId, seqs) expands back to the exact original
- * conversation rows (the durable log under `<dshHome>/sessions/...`).
- *
- * The extraction reads the live session log through the reader the running
- * harness generation exposes (duck-typed, no dsh-session dependency) and only
- * ever selects direct human messages — injected plugin context and tool
- * results never become citations.
- */
-import type { EntrySource } from "./types.js";
-import { type AgentLike } from "./inject.js";
-/** At most this many source user messages are cited per entry. */
-export declare const MAX_SOURCE_MESSAGES = 3;
-/**
- * The seqs of the agent's most recent direct user messages, in log order.
- * Returns [] when the agent has no readable log or no qualifying messages —
- * callers then simply omit the citation.
- */
-export declare function recentUserSeqs(agent: AgentLike | undefined, opts?: {
-    maxMessages?: number;
-}): number[];
-/**
- * Build the citation for an apply call: the session id plus the seqs of the
- * most recent direct user messages. Returns undefined when neither can be
- * determined, so callers can pass it straight through without special cases.
- */
-export declare function entrySourceOf(agent: AgentLike | undefined, sessionId: string | undefined): EntrySource | undefined;
-//# sourceMappingURL=source.d.ts.map

package/lib/source.js

@@ -1,43 +0,0 @@
-import { sessionEventsOf } from "./inject.js";
-/** At most this many source user messages are cited per entry. */
-export const MAX_SOURCE_MESSAGES = 3;
-/**
- * The seqs of the agent's most recent direct user messages, in log order.
- * Returns [] when the agent has no readable log or no qualifying messages —
- * callers then simply omit the citation.
- */
-export function recentUserSeqs(agent, opts) {
-    const events = sessionEventsOf(agent);
-    if (events.length === 0) {
-        return [];
-    }
-    const maxMessages = opts?.maxMessages ?? MAX_SOURCE_MESSAGES;
-    const seqs = [];
-    for (let i = events.length - 1; i >= 0 && seqs.length < maxMessages; i -= 1) {
-        const row = events[i];
-        if (row?.type !== "user/message") {
-            continue;
-        }
-        const source = row.data?.source;
-        if (source && source.kind !== "user") {
-            continue;
-        }
-        if (typeof row.seq === "number" && Number.isInteger(row.seq)) {
-            seqs.unshift(row.seq);
-        }
-    }
-    return seqs;
-}
-/**
- * Build the citation for an apply call: the session id plus the seqs of the
- * most recent direct user messages. Returns undefined when neither can be
- * determined, so callers can pass it straight through without special cases.
- */
-export function entrySourceOf(agent, sessionId) {
-    if (!sessionId) {
-        return undefined;
-    }
-    const seqs = recentUserSeqs(agent);
-    return seqs.length > 0 ? { sessionId, seqs } : { sessionId };
-}
-//# sourceMappingURL=source.js.map