pikiloom 0.4.60 → 0.4.62

package/packages/kernel/dist/workspace/sessions.js

@@ -1,8 +1,5 @@
 import path from 'node:path';
-function splitKey(sessionKey) {
-    const i = sessionKey.indexOf(':');
-    return i < 0 ? { agent: '', sessionId: sessionKey } : { agent: sessionKey.slice(0, i), sessionId: sessionKey.slice(i + 1) };
-}
+import { makeSessionKey, splitSessionKey } from '../protocol/index.js';
 function ts(iso) {
     if (!iso)
         return 0;
@@ -14,7 +11,7 @@ function newer(a, b) {
 }
 function managedToInfo(agent, rec) {
     return {
-        sessionKey: `${agent}:${rec.sessionId}`,
+        sessionKey: makeSessionKey(agent, rec.sessionId),
         agent,
         sessionId: rec.sessionId,
         title: rec.title ?? null,
@@ -31,7 +28,7 @@ function managedToInfo(agent, rec) {
 }
 function nativeToInfo(agent, n) {
     return {
-        sessionKey: `${agent}:${n.sessionId}`,
+        sessionKey: makeSessionKey(agent, n.sessionId),
         agent,
         sessionId: n.sessionId,
         title: n.title,
@@ -72,7 +69,7 @@ export class SessionsManager {
                     if (!rec.workdir || path.resolve(rec.workdir) !== workdir)
                         continue;
                 }
-                byKey.set(`${agent}:${rec.sessionId}`, managedToInfo(agent, rec));
+                byKey.set(makeSessionKey(agent, rec.sessionId), managedToInfo(agent, rec));
             }
         }
         // Native sessions (the agents' own stores) — inherently per-workdir.
@@ -89,7 +86,7 @@ export class SessionsManager {
                     this.deps.log?.(`[sessions] ${agent}.listNativeSessions failed: ${e?.message || e}`);
                 }
                 for (const n of natives) {
-                    const key = `${agent}:${n.sessionId}`;
+                    const key = makeSessionKey(agent, n.sessionId);
                     const existing = byKey.get(key);
                     if (existing) {
                         // Same identity discovered both ways: managed record wins, but adopt the newer
@@ -118,7 +115,7 @@ export class SessionsManager {
         return typeof opts.limit === 'number' ? matched.slice(0, Math.max(0, opts.limit)) : matched;
     }
     async get(sessionKey, opts = {}) {
-        const { agent, sessionId } = splitKey(sessionKey);
+        const { agent, sessionId } = splitSessionKey(sessionKey);
         if (!agent)
             return null;
         const rec = await this.deps.store.get(agent, sessionId).catch(() => null);

package/packages/kernel/dist/workspace/skills.d.ts

@@ -21,6 +21,17 @@ export interface SkillsManagerOptions {
     agentSkillDirs?: string[];
     log?: (msg: string) => void;
 }
+export interface SkillMeta {
+    label: string | null;
+    description: string | null;
+    mcpRequires?: string[];
+}
+/**
+ * Parse a SKILL.md's frontmatter (label/name + description + mcp_requires), falling back
+ * to the first `# heading` for the label. Exported so an app scanning additional skill
+ * stores (e.g. an agent's own ~/.claude/skills) reads them with the same rules.
+ */
+export declare function parseSkillMeta(content: string): SkillMeta;
 /** Make `linkPath` a symlink to `targetDir` (idempotent; replaces a stale link/dir). */
 export declare function ensureDirSymlink(linkPath: string, targetDir: string): void;
 export declare class SkillsManager {

package/packages/kernel/dist/workspace/skills.js

@@ -1,13 +1,20 @@
 import fs from 'node:fs';
 import path from 'node:path';
+import { searchNpmPackages } from './npm-search.js';
 const DEFAULT_AGENT_SKILL_DIRS = ['.claude/skills', '.agents/skills'];
-function parseSkillMeta(content) {
+/**
+ * Parse a SKILL.md's frontmatter (label/name + description + mcp_requires), falling back
+ * to the first `# heading` for the label. Exported so an app scanning additional skill
+ * stores (e.g. an agent's own ~/.claude/skills) reads them with the same rules.
+ */
+export function parseSkillMeta(content) {
     let label = null;
     let description = null;
     let mcpRequires;
     const fm = content.match(/^---\s*\n([\s\S]*?)\n---/);
     if (fm) {
-        const lm = fm[1].match(/^label:\s*(.+)/m);
+        // Both frontmatter dialects: pikiloom's `label:` and the agent-native `name:`.
+        const lm = fm[1].match(/^label:\s*(.+)/m) || fm[1].match(/^name:\s*(.+)/m);
         if (lm)
             label = lm[1].trim();
         const dm = fm[1].match(/^description:\s*(.+)/m);
@@ -145,25 +152,12 @@ export class SkillsManager {
     /** Search installable skills on the npm registry (best-effort; [] on failure). */
     async search(query, limit = 20) {
         const q = (query || '').trim();
-        const text = encodeURIComponent(`agent skill ${q}`.trim());
-        const url = `https://registry.npmjs.org/-/v1/search?text=${text}&size=${Math.max(1, Math.min(50, limit))}`;
         try {
-            const res = await fetch(url, { headers: { accept: 'application/json' } });
-            if (!res.ok)
-                return [];
-            const data = await res.json();
-            const objects = Array.isArray(data?.objects) ? data.objects : [];
-            return objects.map((o) => {
-                const pkg = o?.package ?? {};
-                return {
-                    name: String(pkg.name ?? ''),
-                    description: pkg.description ?? null,
-                    source: 'npm',
-                    author: pkg.publisher?.username ?? pkg.author?.name ?? null,
-                    homepage: pkg.links?.homepage ?? pkg.links?.npm ?? null,
-                    version: pkg.version ?? null,
-                };
-            }).filter(s => s.name);
+            const hits = await searchNpmPackages(`agent skill ${q}`.trim(), Math.max(1, Math.min(50, limit)), fetch);
+            return hits.map(h => ({
+                name: h.name, description: h.description, source: 'npm',
+                author: h.author, homepage: h.homepage, version: h.version,
+            }));
         }
         catch (e) {
             this.log?.(`[skills] search failed: ${e?.message || e}`);