@senpi/runtime 1.1.0-dev.runtime-phase-2.20260513122539 → 1.1.0-dev.runtime-phase-2.20260514134437

package/dist/skills-manager/version-manager.js

@@ -40,29 +40,165 @@ export function parseSkillMdVersion(content) {
     return versionMatch ? versionMatch[1].trim() : null;
 }
 /**
- * Read the installed version for a skill from its local SKILL.md.
- * Returns null when the file is absent or the version field is missing.
- * Logs a warning for unexpected errors (anything other than ENOENT).
+ * Extract the `description` value from a SKILL.md YAML frontmatter block.
+ *
+ * Skill descriptions can be authored a few different ways. This helper
+ * handles the real-world shapes that appear in the senpi-skills repo today:
+ *
+ *   1. Single-line, double-quoted:
+ *        description: "Configure, deploy, ..."
+ *   2. Single-line, single-quoted or unquoted:
+ *        description: Manage runtimes ...
+ *   3. YAML block scalar (folded or literal, optional chomping indicator):
+ *        description: >
+ *          A short summary
+ *          continues on this line.
+ *        description: >-       # strip trailing newlines (this PR's senpi-skills repo)
+ *          A short summary
+ *        description: |        # literal — preserves newlines (collapsed to spaces here)
+ *          A short summary
+ *        description: |+       # keep trailing newlines (same handling)
+ *
+ *      The block scalar header is `[>|][-+]?` per the YAML 1.2 spec:
+ *      `>` = folded, `|` = literal; `-` strips trailing newlines,
+ *      `+` keeps them, no indicator clips. For notification rendering we
+ *      always collapse to a single line joined by spaces, so the chomping
+ *      indicator only affects whether we match the block-scalar branch.
+ *
+ * Returns null when the field is absent or the frontmatter block is missing.
+ *
+ * Note: kept as a regex extractor rather than introducing a full YAML
+ * parse here — `parseSkillMdVersion` uses the same approach, and skill
+ * frontmatters are constrained enough that the regex covers the shapes we
+ * actually encounter. If we ever adopt a full YAML parser for this file,
+ * both helpers should switch together.
  */
-export async function readLocalVersion(skillsDir, skillName, logger) {
-    const filePath = join(skillsDir, skillName, "SKILL.md");
-    try {
-        const content = await readFile(filePath, "utf8");
-        return parseSkillMdVersion(content);
+export function parseSkillMdDescription(content) {
+    const match = /^---\r?\n([\s\S]*?)\r?\n---/.exec(content.trim());
+    if (!match)
+        return null;
+    const frontmatter = match[1];
+    // Shape 3: block scalar — `description:` followed by a `>` or `|` header
+    // (optionally with a `-` / `+` chomping indicator) on its own line, then
+    // indented continuation lines. Stop at the first un-indented line or the
+    // end of frontmatter. We collapse the indent and join lines into a single
+    // string so the notification renderer doesn't emit raw YAML continuations.
+    //
+    // Pre-fix this regex was `[>|]` (no indicator). A SKILL.md authored with
+    // `description: >-` (the YAML strip-trailing-newlines variant — what the
+    // senpi-skills repo actually uses) didn't match, fell through to the
+    // inline branch, and ended up capturing the literal "`>-`" as the value.
+    // That's the bug report this fixes: the Telegram notification showed
+    // `>-` instead of the real description.
+    const foldedMatch = /(?:^|\n)description:[ \t]*[>|][-+]?[ \t]*\r?\n([\s\S]*?)(?=\n[^\s]|\n?$)/i.exec(frontmatter);
+    if (foldedMatch) {
+        const lines = foldedMatch[1]
+            .split(/\r?\n/)
+            .map((l) => l.replace(/^[ \t]+/, "").trim())
+            .filter((l) => l.length > 0);
+        if (lines.length > 0)
+            return lines.join(" ");
     }
-    catch (err) {
-        if (err.code !== "ENOENT") {
+    // Shapes 1 + 2: single-line value. Allow optional surrounding quotes.
+    // The value runs to the end of the line; trailing whitespace + matching
+    // closing quote stripped.
+    const inlineMatch = /(?:^|\n)description:[ \t]*(?:"((?:[^"\\]|\\.)*)"|'((?:[^'\\]|\\.)*)'|([^\r\n]+?))[ \t]*(?:\r?\n|$)/i
+        .exec(frontmatter);
+    if (inlineMatch) {
+        // Unescape `\"` and `\\` for double-quoted form; single-quoted is taken
+        // literally per YAML (no escapes). Unquoted form passes through.
+        const raw = inlineMatch[1] ?? inlineMatch[2] ?? inlineMatch[3] ?? "";
+        const value = inlineMatch[1]
+            ? raw.replace(/\\(["\\])/g, "$1")
+            : raw;
+        const trimmed = value.trim();
+        return trimmed.length > 0 ? trimmed : null;
+    }
+    return null;
+}
+// `summarizeDescription` (sentence-cut + char-cap truncator) was removed
+// alongside the notification reformat in PR #78. The new nested-bullet
+// notification format renders the full description verbatim so users get
+// the complete context for the skill they're being notified about.
+// Telegram's own 4096-char message limit is the only remaining ceiling,
+// and skill descriptions in the senpi-skills repo are nowhere near that.
+/**
+ * Read both the installed version and description for a skill from its local
+ * SKILL.md. Shared core for `readLocalVersion` and `readLocalDescription` so a
+ * tick that needs both pays a single file read per dir rather than two.
+ *
+ * Multi-dir search path: tries each entry in order and returns the first
+ * readable SKILL.md's parsed fields. Either field may be null if absent in
+ * the frontmatter — that's distinct from "no SKILL.md found at all" (which
+ * returns nulls only after every dir has been tried).
+ */
+export async function readLocalSkillMetadata(skillsDir, skillName, logger) {
+    const dirs = typeof skillsDir === "string" ? [skillsDir] : skillsDir;
+    for (const dir of dirs) {
+        const filePath = join(dir, skillName, "SKILL.md");
+        try {
+            const content = await readFile(filePath, "utf8");
+            // First readable SKILL.md wins, even if its fields are missing —
+            // a present-but-versionless file is meaningful state (skill predates
+            // versioning), not a "not found" signal that should let the next dir
+            // override it.
+            return {
+                version: parseSkillMdVersion(content),
+                description: parseSkillMdDescription(content),
+            };
+        }
+        catch (err) {
+            if (err.code === "ENOENT") {
+                continue; // try next dir
+            }
             logger?.warn(`[senpi-skills-manager] failed to read SKILL.md for "${skillName}" | path=${filePath} err=${formatErr(err)}`);
+            // Non-ENOENT (permissions, I/O error) — log and keep trying the
+            // remaining dirs rather than short-circuit; if one mirror is broken,
+            // a healthy mirror further down the path should still serve metadata.
         }
-        return null;
     }
+    return { version: null, description: null };
 }
 /**
- * Fetch the remote SKILL.md for a skill and return its version.
+ * Read the installed version for a skill from its local SKILL.md.
+ *
+ * Accepts either a single dir (legacy callers) or an array of dirs (new
+ * multi-dir search path). When given an array, tries each entry in order and
+ * returns the first non-null match — this mirrors the documented behaviour
+ * that `skillsDir[0]` is the *primary* (writes go here) and the remaining
+ * entries are read-only fallbacks scanned during version checks.
+ *
+ * Returns null when no readable SKILL.md is found across any of the
+ * candidate dirs, or when the version field is absent. Logs a warning per
+ * dir for unexpected errors (anything other than ENOENT).
+ */
+export async function readLocalVersion(skillsDir, skillName, logger) {
+    return (await readLocalSkillMetadata(skillsDir, skillName, logger)).version;
+}
+/**
+ * Read the installed description for a skill from its local SKILL.md.
+ * Multi-dir search path semantics identical to `readLocalVersion`.
+ *
+ * Returns null when no readable SKILL.md is found across any of the
+ * candidate dirs, or when the description field is absent.
+ */
+export async function readLocalDescription(skillsDir, skillName, logger) {
+    return (await readLocalSkillMetadata(skillsDir, skillName, logger)).description;
+}
+/**
+ * Fetch the remote SKILL.md for a skill and return both the version and the
+ * description in one network round-trip.
+ *
+ * Notification text uses the description from the **remote** SKILL.md (not
+ * the local one) so the "About:" line reflects the version the user is being
+ * asked about — a description on disk for v1.x may be stale relative to the
+ * v2.0.0 the major-update notification is announcing.
  *
- * Returns null when the skill does not exist in the remote repo (HTTP 404) —
- * that's a legitimate state ("skill removed or not yet published"), not an
- * error, so it should not pollute failure counters.
+ * Returns null fields when the skill does not exist in the remote repo
+ * (HTTP 404) — that's a legitimate state ("skill removed or not yet
+ * published"), not an error, so it should not pollute failure counters.
+ * Both fields may also be null when the remote SKILL.md is present but
+ * its frontmatter omits the corresponding field.
  *
  * Throws on transient/operational failures (network errors, timeouts, 5xx
  * responses, body-parse failures) so the caller's failure-recording path can
@@ -70,7 +206,7 @@ export async function readLocalVersion(skillsDir, skillName, logger) {
  *
  * @param fetchImpl Injectable for unit testing — defaults to global `fetch`.
  */
-export async function fetchRemoteVersion(skillName, rawBaseUrl = SKILLS_RAW_BASE_URL, fetchImpl = fetch) {
+export async function fetchRemoteSkillMetadata(skillName, rawBaseUrl = SKILLS_RAW_BASE_URL, fetchImpl = fetch) {
     const url = `${rawBaseUrl}/${encodeURIComponent(skillName)}/SKILL.md`;
     let res;
     try {
@@ -79,22 +215,36 @@ export async function fetchRemoteVersion(skillName, rawBaseUrl = SKILLS_RAW_BASE
     catch (err) {
         throw new Error(`Network error fetching remote SKILL.md for "${skillName}": ${formatErr(err)}`);
     }
-    // 404 = skill is not in the remote repo. Treated as "no remote version"
+    // 404 = skill is not in the remote repo. Treated as "no remote metadata"
     // rather than a failure so the caller can skip cleanly without escalating.
     if (res.status === 404) {
-        return null;
+        return { version: null, description: null };
     }
     if (!res.ok) {
         throw new Error(`Failed to fetch remote SKILL.md for "${skillName}": HTTP ${res.status} ${res.statusText}`);
     }
     try {
         const text = await res.text();
-        return parseSkillMdVersion(text);
+        return {
+            version: parseSkillMdVersion(text),
+            description: parseSkillMdDescription(text),
+        };
     }
     catch (err) {
         throw new Error(`Failed to read response body for "${skillName}": ${formatErr(err)}`);
     }
 }
+/**
+ * Fetch the remote SKILL.md for a skill and return its version.
+ *
+ * Thin wrapper over {@link fetchRemoteSkillMetadata} kept for backward
+ * compatibility with callers (and tests) that only need the version field.
+ * New code that wants both version + description in a single round-trip
+ * should call `fetchRemoteSkillMetadata` directly.
+ */
+export async function fetchRemoteVersion(skillName, rawBaseUrl = SKILLS_RAW_BASE_URL, fetchImpl = fetch) {
+    return (await fetchRemoteSkillMetadata(skillName, rawBaseUrl, fetchImpl)).version;
+}
 /**
  * Compare the local and remote versions for a skill and return a classified diff.
  *
@@ -107,36 +257,43 @@ export async function fetchRemoteVersion(skillName, rawBaseUrl = SKILLS_RAW_BASE
  *   skip.
  */
 export async function checkSkillVersion(skillsDir, skillName, rawBaseUrl = SKILLS_RAW_BASE_URL, fetchImpl = fetch, logger) {
+    // Fetch the remote SKILL.md once and extract both version + description.
+    // Description is reported back to the coordinator so notifications can
+    // surface the *remote* (i.e. upcoming-version) description rather than the
+    // local one, which may be stale relative to the version the notification
+    // is announcing.
+    //
     // Log the specific fetch failure for traceability, then re-throw so the
     // coordinator records it as a check failure (consecutiveFailures,
     // lastFailureReason). 404 is intentionally returned as null inside
-    // fetchRemoteVersion and never reaches this catch.
-    const remoteVersionPromise = fetchRemoteVersion(skillName, rawBaseUrl, fetchImpl).catch((err) => {
-        logger?.warn(`[senpi-skills-manager] failed to fetch remote version for "${skillName}" | err=${formatErr(err)}`);
+    // fetchRemoteSkillMetadata and never reaches this catch.
+    const remoteMetadataPromise = fetchRemoteSkillMetadata(skillName, rawBaseUrl, fetchImpl).catch((err) => {
+        logger?.warn(`[senpi-skills-manager] failed to fetch remote SKILL.md for "${skillName}" | err=${formatErr(err)}`);
         throw err;
     });
-    const [localVersion, remoteVersion] = await Promise.all([
+    const [localVersion, remoteMetadata] = await Promise.all([
         readLocalVersion(skillsDir, skillName, logger),
-        remoteVersionPromise,
+        remoteMetadataPromise,
     ]);
+    const { version: remoteVersion, description: remoteDescription } = remoteMetadata;
     const normalizedRemote = normalizeVersion(remoteVersion ?? "");
     if (!remoteVersion || !parseSemver(normalizedRemote)) {
-        return { localVersion, remoteVersion, updateKind: "unknown" };
+        return { localVersion, remoteVersion, updateKind: "unknown", remoteDescription };
     }
     // Skill has never had a version in SKILL.md (null) — predates the versioning
     // system or was installed before the field was added. Treat as a minor update
     // so it is auto-applied without requiring a manual major-version approval.
     if (localVersion === null) {
-        return { localVersion, remoteVersion, updateKind: "minor" };
+        return { localVersion, remoteVersion, updateKind: "minor", remoteDescription };
     }
     const normalizedLocal = normalizeVersion(localVersion);
     // Local version string exists but is not valid semver even after normalisation
     // (e.g. "latest", "nightly"). We cannot safely compare, so skip.
     if (!parseSemver(normalizedLocal)) {
-        return { localVersion, remoteVersion, updateKind: "unknown" };
+        return { localVersion, remoteVersion, updateKind: "unknown", remoteDescription };
     }
     // classifyUpdate returns AutoUpdateKind which is a strict subset of SkillUpdateKind
     const updateKind = classifyUpdate(normalizedLocal, normalizedRemote);
-    return { localVersion, remoteVersion, updateKind };
+    return { localVersion, remoteVersion, updateKind, remoteDescription };
 }
 //# sourceMappingURL=version-manager.js.map

package/dist/skills-manager/version-manager.js.map

@@ -1 +1 @@
-{"version":3,"file":"version-manager.js","sourceRoot":"","sources":["../../src/skills-manager/version-manager.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAC5C,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,kCAAkC,CAAC;AAC/E,OAAO,EAAE,mBAAmB,EAAE,MAAM,gBAAgB,CAAC;AAErD,OAAO,EAAE,SAAS,EAAE,MAAM,YAAY,CAAC;AAEvC;;;;;;GAMG;AACH,MAAM,UAAU,gBAAgB,CAAC,CAAS;IACxC,OAAO,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC;AAC7D,CAAC;AAED;;;;;;;;;;GAUG;AACH,MAAM,UAAU,mBAAmB,CAAC,OAAe;IACjD,MAAM,KAAK,GAAG,6BAA6B,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC;IACjE,IAAI,CAAC,KAAK;QAAE,OAAO,IAAI,CAAC;IACxB,MAAM,WAAW,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;IAC7B,MAAM,YAAY,GAAG,uDAAuD,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;IAC/F,OAAO,YAAY,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC;AACtD,CAAC;AAED;;;;GAIG;AACH,MAAM,CAAC,KAAK,UAAU,gBAAgB,CACpC,SAAiB,EACjB,SAAiB,EACjB,MAAmB;IAEnB,MAAM,QAAQ,GAAG,IAAI,CAAC,SAAS,EAAE,SAAS,EAAE,UAAU,CAAC,CAAC;IACxD,IAAI,CAAC;QACH,MAAM,OAAO,GAAG,MAAM,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;QACjD,OAAO,mBAAmB,CAAC,OAAO,CAAC,CAAC;IACtC,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,IAAK,GAA6B,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;YACrD,MAAM,EAAE,IAAI,CACV,uDAAuD,SAAS,YAAY,QAAQ,QAAQ,SAAS,CAAC,GAAG,CAAC,EAAE,CAC7G,CAAC;QACJ,CAAC;QACD,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;GAYG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,SAAiB,EACjB,aAAqB,mBAAmB,EACxC,YAA0B,KAAK;IAE/B,MAAM,GAAG,GAAG,GAAG,UAAU,IAAI,kBAAkB,CAAC,SAAS,CAAC,WAAW,CAAC;IAEtE,IAAI,GAAa,CAAC;IAClB,IAAI,CAAC;QACH,GAAG,GAAG,MAAM,SAAS,CAAC,GAAG,EAAE,EAAE,MAAM,EAAE,WAAW,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;IACtE,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,MAAM,IAAI,KAAK,CACb,+CAA+C,SAAS,MAAM,SAAS,CAAC,GAAG,CAAC,EAAE,CAC/E,CAAC;IACJ,CAAC;IAED,wEAAwE;IACxE,2EAA2E;IAC3E,IAAI,GAAG,CAAC,MAAM,KAAK,GAAG,EAAE,CAAC;QACvB,OAAO,IAAI,CAAC;IACd,CAAC;IACD,IAAI,CAAC,GAAG,CAAC,EAAE,EAAE,CAAC;QACZ,MAAM,IAAI,KAAK,CACb,wCAAwC,SAAS,WAAW,GAAG,CAAC,MAAM,IAAI,GAAG,CAAC,UAAU,EAAE,CAC3F,CAAC;IACJ,CAAC;IAED,IAAI,CAAC;QACH,MAAM,IAAI,GAAG,MAAM,GAAG,CAAC,IAAI,EAAE,CAAC;QAC9B,OAAO,mBAAmB,CAAC,IAAI,CAAC,CAAC;IACnC,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,MAAM,IAAI,KAAK,CACb,qCAAqC,SAAS,MAAM,SAAS,CAAC,GAAG,CAAC,EAAE,CACrE,CAAC;IACJ,CAAC;AACH,CAAC;AAED;;;;;;;;;;GAUG;AACH,MAAM,CAAC,KAAK,UAAU,iBAAiB,CACrC,SAAiB,EACjB,SAAiB,EACjB,aAAqB,mBAAmB,EACxC,YAA0B,KAAK,EAC/B,MAAmB;IAEnB,wEAAwE;IACxE,kEAAkE;IAClE,mEAAmE;IACnE,mDAAmD;IACnD,MAAM,oBAAoB,GAAG,kBAAkB,CAAC,SAAS,EAAE,UAAU,EAAE,SAAS,CAAC,CAAC,KAAK,CACrF,CAAC,GAAG,EAAE,EAAE;QACN,MAAM,EAAE,IAAI,CACV,8DAA8D,SAAS,WAAW,SAAS,CAAC,GAAG,CAAC,EAAE,CACnG,CAAC;QACF,MAAM,GAAG,CAAC;IACZ,CAAC,CACF,CAAC;IACF,MAAM,CAAC,YAAY,EAAE,aAAa,CAAC,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC;QACtD,gBAAgB,CAAC,SAAS,EAAE,SAAS,EAAE,MAAM,CAAC;QAC9C,oBAAoB;KACrB,CAAC,CAAC;IAEH,MAAM,gBAAgB,GAAG,gBAAgB,CAAC,aAAa,IAAI,EAAE,CAAC,CAAC;IAC/D,IAAI,CAAC,aAAa,IAAI,CAAC,WAAW,CAAC,gBAAgB,CAAC,EAAE,CAAC;QACrD,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,UAAU,EAAE,SAAS,EAAE,CAAC;IAChE,CAAC;IAED,6EAA6E;IAC7E,8EAA8E;IAC9E,2EAA2E;IAC3E,IAAI,YAAY,KAAK,IAAI,EAAE,CAAC;QAC1B,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,UAAU,EAAE,OAAO,EAAE,CAAC;IAC9D,CAAC;IAED,MAAM,eAAe,GAAG,gBAAgB,CAAC,YAAY,CAAC,CAAC;IAEvD,+EAA+E;IAC/E,iEAAiE;IACjE,IAAI,CAAC,WAAW,CAAC,eAAe,CAAC,EAAE,CAAC;QAClC,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,UAAU,EAAE,SAAS,EAAE,CAAC;IAChE,CAAC;IAED,oFAAoF;IACpF,MAAM,UAAU,GAAG,cAAc,CAAC,eAAe,EAAE,gBAAgB,CAAoB,CAAC;IACxF,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,UAAU,EAAE,CAAC;AACrD,CAAC"}
+{"version":3,"file":"version-manager.js","sourceRoot":"","sources":["../../src/skills-manager/version-manager.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAC5C,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,kCAAkC,CAAC;AAC/E,OAAO,EAAE,mBAAmB,EAAE,MAAM,gBAAgB,CAAC;AAErD,OAAO,EAAE,SAAS,EAAE,MAAM,YAAY,CAAC;AAEvC;;;;;;GAMG;AACH,MAAM,UAAU,gBAAgB,CAAC,CAAS;IACxC,OAAO,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC;AAC7D,CAAC;AAED;;;;;;;;;;GAUG;AACH,MAAM,UAAU,mBAAmB,CAAC,OAAe;IACjD,MAAM,KAAK,GAAG,6BAA6B,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC;IACjE,IAAI,CAAC,KAAK;QAAE,OAAO,IAAI,CAAC;IACxB,MAAM,WAAW,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;IAC7B,MAAM,YAAY,GAAG,uDAAuD,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;IAC/F,OAAO,YAAY,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC;AACtD,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAiCG;AACH,MAAM,UAAU,uBAAuB,CAAC,OAAe;IACrD,MAAM,KAAK,GAAG,6BAA6B,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC;IACjE,IAAI,CAAC,KAAK;QAAE,OAAO,IAAI,CAAC;IACxB,MAAM,WAAW,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;IAE7B,yEAAyE;IACzE,yEAAyE;IACzE,yEAAyE;IACzE,0EAA0E;IAC1E,2EAA2E;IAC3E,EAAE;IACF,yEAAyE;IACzE,yEAAyE;IACzE,qEAAqE;IACrE,yEAAyE;IACzE,qEAAqE;IACrE,wCAAwC;IACxC,MAAM,WAAW,GAAG,2EAA2E,CAAC,IAAI,CAClG,WAAW,CACZ,CAAC;IACF,IAAI,WAAW,EAAE,CAAC;QAChB,MAAM,KAAK,GAAG,WAAW,CAAC,CAAC,CAAC;aACzB,KAAK,CAAC,OAAO,CAAC;aACd,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,SAAS,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;aAC3C,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;QAC/B,IAAI,KAAK,CAAC,MAAM,GAAG,CAAC;YAAE,OAAO,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IAC/C,CAAC;IAED,sEAAsE;IACtE,wEAAwE;IACxE,0BAA0B;IAC1B,MAAM,WAAW,GAAG,qGAAqG;SACtH,IAAI,CAAC,WAAW,CAAC,CAAC;IACrB,IAAI,WAAW,EAAE,CAAC;QAChB,wEAAwE;QACxE,iEAAiE;QACjE,MAAM,GAAG,GAAG,WAAW,CAAC,CAAC,CAAC,IAAI,WAAW,CAAC,CAAC,CAAC,IAAI,WAAW,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACrE,MAAM,KAAK,GAAG,WAAW,CAAC,CAAC,CAAC;YAC1B,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,YAAY,EAAE,IAAI,CAAC;YACjC,CAAC,CAAC,GAAG,CAAC;QACR,MAAM,OAAO,GAAG,KAAK,CAAC,IAAI,EAAE,CAAC;QAC7B,OAAO,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC;IAC7C,CAAC;IAED,OAAO,IAAI,CAAC;AACd,CAAC;AAED,yEAAyE;AACzE,uEAAuE;AACvE,yEAAyE;AACzE,mEAAmE;AACnE,wEAAwE;AACxE,yEAAyE;AAEzE;;;;;;;;;GASG;AACH,MAAM,CAAC,KAAK,UAAU,sBAAsB,CAC1C,SAAqC,EACrC,SAAiB,EACjB,MAAmB;IAEnB,MAAM,IAAI,GAAG,OAAO,SAAS,KAAK,QAAQ,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;IACrE,KAAK,MAAM,GAAG,IAAI,IAAI,EAAE,CAAC;QACvB,MAAM,QAAQ,GAAG,IAAI,CAAC,GAAG,EAAE,SAAS,EAAE,UAAU,CAAC,CAAC;QAClD,IAAI,CAAC;YACH,MAAM,OAAO,GAAG,MAAM,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;YACjD,iEAAiE;YACjE,qEAAqE;YACrE,qEAAqE;YACrE,eAAe;YACf,OAAO;gBACL,OAAO,EAAE,mBAAmB,CAAC,OAAO,CAAC;gBACrC,WAAW,EAAE,uBAAuB,CAAC,OAAO,CAAC;aAC9C,CAAC;QACJ,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,IAAK,GAA6B,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;gBACrD,SAAS,CAAC,eAAe;YAC3B,CAAC;YACD,MAAM,EAAE,IAAI,CACV,uDAAuD,SAAS,YAAY,QAAQ,QAAQ,SAAS,CAAC,GAAG,CAAC,EAAE,CAC7G,CAAC;YACF,gEAAgE;YAChE,qEAAqE;YACrE,sEAAsE;QACxE,CAAC;IACH,CAAC;IACD,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,CAAC;AAC9C,CAAC;AAED;;;;;;;;;;;;GAYG;AACH,MAAM,CAAC,KAAK,UAAU,gBAAgB,CACpC,SAAqC,EACrC,SAAiB,EACjB,MAAmB;IAEnB,OAAO,CAAC,MAAM,sBAAsB,CAAC,SAAS,EAAE,SAAS,EAAE,MAAM,CAAC,CAAC,CAAC,OAAO,CAAC;AAC9E,CAAC;AAED;;;;;;GAMG;AACH,MAAM,CAAC,KAAK,UAAU,oBAAoB,CACxC,SAAqC,EACrC,SAAiB,EACjB,MAAmB;IAEnB,OAAO,CAAC,MAAM,sBAAsB,CAAC,SAAS,EAAE,SAAS,EAAE,MAAM,CAAC,CAAC,CAAC,WAAW,CAAC;AAClF,CAAC;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,MAAM,CAAC,KAAK,UAAU,wBAAwB,CAC5C,SAAiB,EACjB,aAAqB,mBAAmB,EACxC,YAA0B,KAAK;IAE/B,MAAM,GAAG,GAAG,GAAG,UAAU,IAAI,kBAAkB,CAAC,SAAS,CAAC,WAAW,CAAC;IAEtE,IAAI,GAAa,CAAC;IAClB,IAAI,CAAC;QACH,GAAG,GAAG,MAAM,SAAS,CAAC,GAAG,EAAE,EAAE,MAAM,EAAE,WAAW,CAAC,OAAO,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;IACtE,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,MAAM,IAAI,KAAK,CACb,+CAA+C,SAAS,MAAM,SAAS,CAAC,GAAG,CAAC,EAAE,CAC/E,CAAC;IACJ,CAAC;IAED,yEAAyE;IACzE,2EAA2E;IAC3E,IAAI,GAAG,CAAC,MAAM,KAAK,GAAG,EAAE,CAAC;QACvB,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,EAAE,CAAC;IAC9C,CAAC;IACD,IAAI,CAAC,GAAG,CAAC,EAAE,EAAE,CAAC;QACZ,MAAM,IAAI,KAAK,CACb,wCAAwC,SAAS,WAAW,GAAG,CAAC,MAAM,IAAI,GAAG,CAAC,UAAU,EAAE,CAC3F,CAAC;IACJ,CAAC;IAED,IAAI,CAAC;QACH,MAAM,IAAI,GAAG,MAAM,GAAG,CAAC,IAAI,EAAE,CAAC;QAC9B,OAAO;YACL,OAAO,EAAE,mBAAmB,CAAC,IAAI,CAAC;YAClC,WAAW,EAAE,uBAAuB,CAAC,IAAI,CAAC;SAC3C,CAAC;IACJ,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,MAAM,IAAI,KAAK,CACb,qCAAqC,SAAS,MAAM,SAAS,CAAC,GAAG,CAAC,EAAE,CACrE,CAAC;IACJ,CAAC;AACH,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,SAAiB,EACjB,aAAqB,mBAAmB,EACxC,YAA0B,KAAK;IAE/B,OAAO,CAAC,MAAM,wBAAwB,CAAC,SAAS,EAAE,UAAU,EAAE,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC;AACpF,CAAC;AAED;;;;;;;;;;GAUG;AACH,MAAM,CAAC,KAAK,UAAU,iBAAiB,CACrC,SAAqC,EACrC,SAAiB,EACjB,aAAqB,mBAAmB,EACxC,YAA0B,KAAK,EAC/B,MAAmB;IAEnB,yEAAyE;IACzE,uEAAuE;IACvE,2EAA2E;IAC3E,yEAAyE;IACzE,iBAAiB;IACjB,EAAE;IACF,wEAAwE;IACxE,kEAAkE;IAClE,mEAAmE;IACnE,yDAAyD;IACzD,MAAM,qBAAqB,GAAG,wBAAwB,CAAC,SAAS,EAAE,UAAU,EAAE,SAAS,CAAC,CAAC,KAAK,CAC5F,CAAC,GAAG,EAAE,EAAE;QACN,MAAM,EAAE,IAAI,CACV,+DAA+D,SAAS,WAAW,SAAS,CAAC,GAAG,CAAC,EAAE,CACpG,CAAC;QACF,MAAM,GAAG,CAAC;IACZ,CAAC,CACF,CAAC;IACF,MAAM,CAAC,YAAY,EAAE,cAAc,CAAC,GAAG,MAAM,OAAO,CAAC,GAAG,CAAC;QACvD,gBAAgB,CAAC,SAAS,EAAE,SAAS,EAAE,MAAM,CAAC;QAC9C,qBAAqB;KACtB,CAAC,CAAC;IACH,MAAM,EAAE,OAAO,EAAE,aAAa,EAAE,WAAW,EAAE,iBAAiB,EAAE,GAAG,cAAc,CAAC;IAElF,MAAM,gBAAgB,GAAG,gBAAgB,CAAC,aAAa,IAAI,EAAE,CAAC,CAAC;IAC/D,IAAI,CAAC,aAAa,IAAI,CAAC,WAAW,CAAC,gBAAgB,CAAC,EAAE,CAAC;QACrD,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,UAAU,EAAE,SAAS,EAAE,iBAAiB,EAAE,CAAC;IACnF,CAAC;IAED,6EAA6E;IAC7E,8EAA8E;IAC9E,2EAA2E;IAC3E,IAAI,YAAY,KAAK,IAAI,EAAE,CAAC;QAC1B,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,UAAU,EAAE,OAAO,EAAE,iBAAiB,EAAE,CAAC;IACjF,CAAC;IAED,MAAM,eAAe,GAAG,gBAAgB,CAAC,YAAY,CAAC,CAAC;IAEvD,+EAA+E;IAC/E,iEAAiE;IACjE,IAAI,CAAC,WAAW,CAAC,eAAe,CAAC,EAAE,CAAC;QAClC,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,UAAU,EAAE,SAAS,EAAE,iBAAiB,EAAE,CAAC;IACnF,CAAC;IAED,oFAAoF;IACpF,MAAM,UAAU,GAAG,cAAc,CAAC,eAAe,EAAE,gBAAgB,CAAoB,CAAC;IACxF,OAAO,EAAE,YAAY,EAAE,aAAa,EAAE,UAAU,EAAE,iBAAiB,EAAE,CAAC;AACxE,CAAC"}

package/openclaw.plugin.json

@@ -74,11 +74,10 @@
       "skillsManager": {
         "type": "object",
         "additionalProperties": false,
-        "description": "Skills manager configuration. Polls the senpi-skills repo for updates to declared skills and applies patch/minor updates automatically (major updates are notify-only). Omit this block to use defaults.",
+        "description": "Skills manager configuration. Checks the senpi-skills repo for updates to declared skills after each plugin auto-update tick and applies patch/minor updates automatically (major updates are notify-only). Cadence is inherited from `autoUpdate.pollIntervalMinutes`; this block has no independent poll interval. Omit this block to use defaults.",
         "default": {
           "enabled": true,
-          "skills": ["senpi-trading-runtime"],
-          "pollIntervalMinutes": 60
+          "skills": ["senpi-trading-runtime"]
         },
         "properties": {
           "enabled": {
@@ -86,12 +85,11 @@
             "description": "Enable or disable the skills manager (default: true)"
           },
           "skillsDir": {
-            "type": "string",
-            "description": "Directory where managed skills are installed (default: /data/.openclaw/skills, override via SENPI_SKILLS_DIR env)"
-          },
-          "pollIntervalMinutes": {
-            "type": "number",
-            "description": "Minutes between version checks (default: 60, range: 1-1440)"
+            "oneOf": [
+              { "type": "string" },
+              { "type": "array", "items": { "type": "string", "minLength": 1 }, "minItems": 1 }
+            ],
+            "description": "Directory (or directories) where managed skills are installed. Accepts a single string for one dir, or an array for a search path. With multiple, the first is the *primary* (writes go here on bootstrap/install); the rest are read-only mirrors scanned during version checks and discovery. Default: /data/.openclaw/skills. Env override: SENPI_SKILLS_DIR (comma-separated for multi-dir; use the JSON array form for paths containing commas)."
           },
           "skills": {
             "type": "array",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@senpi/runtime",
-  "version": "1.1.0-dev.runtime-phase-2.20260513122539",
+  "version": "1.1.0-dev.runtime-phase-2.20260514134437",
   "description": "Senpi runtime plugin for OpenClaw",
   "type": "module",
   "main": "dist/index.js",