@av-pi-studio/client 0.0.110 → 0.0.112

package/dist/pistudio-client.d.ts

@@ -1,5 +1,6 @@
-import type { AgentCloneResponse, AgentCompactResponse, AgentCycleModelResponse, AgentExportHtmlResponse, AgentForkMessagesResponse, AgentForkResponse, AgentLastAssistantTextResponse, AgentListCommandsResponse, AgentNewSessionResponse, AgentSessionStatsResponse, AgentSetModelResponse, AgentSetSessionNameResponse, AgentSetThinkingResponse, AgentStreamEvent, AgentSwitchSessionResponse, AgentThinkingLevelsResponse, AgentUiPendingRequest, AgentUiRequest, AgentUiResolved, AgentUiResponse, AgentUiSurface, CreateAgentRequest, ExtensionPacksListResponse, ExtensionPacksSetResponse, FetchAgentTimelineResponse, ProviderAuthInfo, ProviderAuthType, SessionMessage, TimelineDirection, ViewerSettings, ViewerSettingsPatchEntry } from "@av-pi-studio/protocol";
+import type { AgentCloneResponse, AgentCompactResponse, AgentCycleModelResponse, AgentExportHtmlResponse, AgentForkMessagesResponse, AgentForkResponse, AgentLastAssistantTextResponse, AgentListCommandsResponse, AgentNewSessionResponse, AgentSessionStatsResponse, AgentSetModelResponse, AgentSetSessionNameResponse, AgentSetThinkingResponse, AgentStreamEvent, AgentSwitchSessionResponse, AgentThinkingLevelsResponse, AgentUiPendingRequest, AgentUiRequest, AgentUiResolved, AgentUiResponse, AgentUiSurface, CreateAgentRequest, ExtensionPacksListResponse, ExtensionPacksSetResponse, FetchAgentTimelineResponse, ProviderAuthInfo, ProviderAuthType, SessionMessage, SkillsInstallResponse, SkillsListResponse, SkillsPreviewResponse, SkillsRemoveResponse, SkillsSetEnabledResponse, SkillsUpdateAllResponse, TimelineDirection, ViewerSettings, ViewerSettingsPatchEntry } from "@av-pi-studio/protocol";
 import type { DaemonClient } from "./daemon-client.js";
+export { detectSkillSource } from "@av-pi-studio/protocol";
 /**
  * High-level `PiStudioClient` SDK facade over the low-level `DaemonClient` driver.
  *
@@ -22,6 +23,28 @@ import type { DaemonClient } from "./daemon-client.js";
  * this call (`rpcTimeoutMs ≠ socket death`); compaction progress keeps following the stream.
  */
 export declare const COMPACT_TIMEOUT_MS: number;
+/**
+ * RPC timeout for `skills_remove_request`. The daemon bounds its own spawn at 600 s
+ * (`swe/architecture/pi-resource-management.md` § Timeouts); the SDK timeout is set above that so
+ * a slow `skills remove`/`pi remove` process reports its own `timeout` domain error before the
+ * client gives up and treats the socket as dead.
+ */
+export declare const REMOVE_SKILL_TIMEOUT_MS: number;
+/**
+ * RPC timeout for `skills_update_all_request`. Updates both engines (`agent-skills`, `pi-packages`)
+ * sequentially, each bounded at 600 s daemon-side; 1 300 s covers both spawns plus margin.
+ */
+export declare const UPDATE_ALL_SKILLS_TIMEOUT_MS: number;
+/**
+ * RPC timeout for `skills_preview_request`. The daemon bounds its own clone/fetch spawn at 120 s
+ * (`swe/architecture/pi-resource-management.md` § Timeouts); the SDK timeout is set above that.
+ */
+export declare const PREVIEW_SKILL_SOURCE_TIMEOUT_MS: number;
+/**
+ * RPC timeout for `skills_install_request`. The daemon bounds its own `pi install`/`skills add`
+ * spawn at 600 s; 660 s covers one spawn plus margin (same idiom as `REMOVE_SKILL_TIMEOUT_MS`).
+ */
+export declare const INSTALL_SKILLS_TIMEOUT_MS: number;
 export type PiStudioAgentUpdateHandler = (update: AgentUpdateMessage) => void;
 export type PiStudioWorkspaceUpdateHandler = (update: WorkspaceUpdateMessage) => void;
 /** Metadata carried alongside a live `agent_stream` event (daemon-owned, not provider-replayed). */
@@ -484,6 +507,54 @@ export declare class PiStudioClient {
      * from "does this daemon speak the `agent_ui_*` RPC family at all".
      */
     extensionUiAvailable(): boolean;
+    /** True iff the daemon advertised the `skills` capability in `server_info.features`. */
+    hasSkillsCapability(): boolean;
+    /** Every resolved Agent Skill plus every resolved `settings.json` package — `skills_list_request`.
+     *  A domain failure (e.g. unreadable `settings.json`) resolves `{ok:false, error}`, never throws. */
+    listSkills(): Promise<SkillsListResponse>;
+    /** Enables/disables one skill by path — `skills_set_enabled_request`. Resolves to the full
+     *  post-mutation `{skills, packages}` document (domain `{ok:false}` on failure, never thrown). */
+    setSkillEnabled(path: string, enabled: boolean): Promise<SkillsSetEnabledResponse>;
+    /** Removes one top-level skill by `path`, or one `settings.json` package by `packageSource` —
+     *  exactly one must be set — `skills_remove_request`. Resolves to the full post-mutation
+     *  `{skills, packages}` document (domain `{ok:false, error}` on failure, never thrown); the
+     *  daemon's own spawn bound is 600 s, so the SDK timeout (`REMOVE_SKILL_TIMEOUT_MS`) is set
+     *  above that rather than the client's default. */
+    removeSkill(target: {
+        path: string;
+        packageSource?: undefined;
+    } | {
+        path?: undefined;
+        packageSource: string;
+    }): Promise<SkillsRemoveResponse>;
+    /** Updates every removable skill engine (`agent-skills` via `skills update -g -y`, `pi-packages`
+     *  via `pi update --extensions`) — `skills_update_all_request`. Resolves to a per-engine
+     *  `results: ItemResult[]` (a mixed ok/failed array is expected, not an overall failure) plus the
+     *  full post-mutation `{skills, packages}` document. Both spawns are bounded at 600 s
+     *  daemon-side; `UPDATE_ALL_SKILLS_TIMEOUT_MS` covers both plus margin. */
+    updateAllSkills(): Promise<SkillsUpdateAllResponse>;
+    /** Previews one install source's picklist before installing — `skills_preview_request`.
+     *  `pi_npm`/`pi_git` always resolve `{ok:true, kind, source, skills: []}` (package kinds install
+     *  as a unit, nothing to pick); `skills_repo`/`local_path` resolve the candidate `skills` array.
+     *  Domain failure (e.g. `invalid_source`) resolves `{ok:false, error}`, never thrown; the daemon
+     *  bounds its own clone/fetch spawn at 120 s, so `PREVIEW_SKILL_SOURCE_TIMEOUT_MS` (150 s) is set
+     *  above that rather than the client's default. */
+    previewSkillSource(source: string): Promise<SkillsPreviewResponse>;
+    /** Installs one source — `skills_install_request`. `skills` is the picked names from a prior
+     *  `previewSkillSource()` call (meaningful for `skills_repo` only; `pi_npm`/`pi_git` install
+     *  every skill the package ships, `local_path` registers the whole folder); `replace` resends an
+     *  already-installed picked name instead of excluding it as `name_exists`. Resolves one
+     *  `ItemResult` per engine/picked-name outcome plus the full post-mutation `{skills, packages}`
+     *  document (domain `{ok:false, error}` on failure, never thrown); the daemon's own spawn bound
+     *  is 600 s, so `INSTALL_SKILLS_TIMEOUT_MS` (660 s) is set above that. */
+    installSkills(opts: {
+        source: string;
+        skills?: string[];
+        replace?: boolean;
+    }): Promise<SkillsInstallResponse>;
+    /** Subscribe to the daemon's `skills_changed` broadcast — fired after any mutation (this
+     *  client's own or another's) or an external filesystem change the daemon detected. */
+    onSkillsChanged(handler: (reason: string) => void): () => void;
 }
 /** Convenience re-export of the import operation (not agent-scoped). */
 export declare function importAgentSession(daemon: DaemonClient, args: {

package/dist/pistudio-client.js

@@ -1,3 +1,6 @@
+// Pure skill-install-source classifier (sprint-077/task-002, `@av-pi-studio/protocol`) —
+// re-exported so the web client can classify a pasted source string without a round trip.
+export { detectSkillSource } from "@av-pi-studio/protocol";
 /**
  * High-level `PiStudioClient` SDK facade over the low-level `DaemonClient` driver.
  *
@@ -20,6 +23,28 @@
  * this call (`rpcTimeoutMs ≠ socket death`); compaction progress keeps following the stream.
  */
 export const COMPACT_TIMEOUT_MS = 10 * 60_000;
+/**
+ * RPC timeout for `skills_remove_request`. The daemon bounds its own spawn at 600 s
+ * (`swe/architecture/pi-resource-management.md` § Timeouts); the SDK timeout is set above that so
+ * a slow `skills remove`/`pi remove` process reports its own `timeout` domain error before the
+ * client gives up and treats the socket as dead.
+ */
+export const REMOVE_SKILL_TIMEOUT_MS = 660 * 1000;
+/**
+ * RPC timeout for `skills_update_all_request`. Updates both engines (`agent-skills`, `pi-packages`)
+ * sequentially, each bounded at 600 s daemon-side; 1 300 s covers both spawns plus margin.
+ */
+export const UPDATE_ALL_SKILLS_TIMEOUT_MS = 1300 * 1000;
+/**
+ * RPC timeout for `skills_preview_request`. The daemon bounds its own clone/fetch spawn at 120 s
+ * (`swe/architecture/pi-resource-management.md` § Timeouts); the SDK timeout is set above that.
+ */
+export const PREVIEW_SKILL_SOURCE_TIMEOUT_MS = 150 * 1000;
+/**
+ * RPC timeout for `skills_install_request`. The daemon bounds its own `pi install`/`skills add`
+ * spawn at 600 s; 660 s covers one spawn plus margin (same idiom as `REMOVE_SKILL_TIMEOUT_MS`).
+ */
+export const INSTALL_SKILLS_TIMEOUT_MS = 660 * 1000;
 /** Narrows an inbound session message to the `provider_auth_flow_event` push. */
 export function isProviderAuthFlowEvent(message) {
     const m = message;
@@ -469,6 +494,67 @@ export class PiStudioClient {
     extensionUiAvailable() {
         return this.daemon.hasFeature("extensionUi");
     }
+    // ─── Skills (skills_*, sprint-075) ─────────────────────────────────────────
+    /** True iff the daemon advertised the `skills` capability in `server_info.features`. */
+    hasSkillsCapability() {
+        return this.daemon.hasFeature("skills");
+    }
+    /** Every resolved Agent Skill plus every resolved `settings.json` package — `skills_list_request`.
+     *  A domain failure (e.g. unreadable `settings.json`) resolves `{ok:false, error}`, never throws. */
+    async listSkills() {
+        return this.daemon.request("skills_list_request", {});
+    }
+    /** Enables/disables one skill by path — `skills_set_enabled_request`. Resolves to the full
+     *  post-mutation `{skills, packages}` document (domain `{ok:false}` on failure, never thrown). */
+    async setSkillEnabled(path, enabled) {
+        return this.daemon.request("skills_set_enabled_request", {
+            path,
+            enabled,
+        });
+    }
+    /** Removes one top-level skill by `path`, or one `settings.json` package by `packageSource` —
+     *  exactly one must be set — `skills_remove_request`. Resolves to the full post-mutation
+     *  `{skills, packages}` document (domain `{ok:false, error}` on failure, never thrown); the
+     *  daemon's own spawn bound is 600 s, so the SDK timeout (`REMOVE_SKILL_TIMEOUT_MS`) is set
+     *  above that rather than the client's default. */
+    async removeSkill(target) {
+        return this.daemon.request("skills_remove_request", target, REMOVE_SKILL_TIMEOUT_MS);
+    }
+    /** Updates every removable skill engine (`agent-skills` via `skills update -g -y`, `pi-packages`
+     *  via `pi update --extensions`) — `skills_update_all_request`. Resolves to a per-engine
+     *  `results: ItemResult[]` (a mixed ok/failed array is expected, not an overall failure) plus the
+     *  full post-mutation `{skills, packages}` document. Both spawns are bounded at 600 s
+     *  daemon-side; `UPDATE_ALL_SKILLS_TIMEOUT_MS` covers both plus margin. */
+    async updateAllSkills() {
+        return this.daemon.request("skills_update_all_request", {}, UPDATE_ALL_SKILLS_TIMEOUT_MS);
+    }
+    /** Previews one install source's picklist before installing — `skills_preview_request`.
+     *  `pi_npm`/`pi_git` always resolve `{ok:true, kind, source, skills: []}` (package kinds install
+     *  as a unit, nothing to pick); `skills_repo`/`local_path` resolve the candidate `skills` array.
+     *  Domain failure (e.g. `invalid_source`) resolves `{ok:false, error}`, never thrown; the daemon
+     *  bounds its own clone/fetch spawn at 120 s, so `PREVIEW_SKILL_SOURCE_TIMEOUT_MS` (150 s) is set
+     *  above that rather than the client's default. */
+    async previewSkillSource(source) {
+        return this.daemon.request("skills_preview_request", { source }, PREVIEW_SKILL_SOURCE_TIMEOUT_MS);
+    }
+    /** Installs one source — `skills_install_request`. `skills` is the picked names from a prior
+     *  `previewSkillSource()` call (meaningful for `skills_repo` only; `pi_npm`/`pi_git` install
+     *  every skill the package ships, `local_path` registers the whole folder); `replace` resends an
+     *  already-installed picked name instead of excluding it as `name_exists`. Resolves one
+     *  `ItemResult` per engine/picked-name outcome plus the full post-mutation `{skills, packages}`
+     *  document (domain `{ok:false, error}` on failure, never thrown); the daemon's own spawn bound
+     *  is 600 s, so `INSTALL_SKILLS_TIMEOUT_MS` (660 s) is set above that. */
+    async installSkills(opts) {
+        return this.daemon.request("skills_install_request", opts, INSTALL_SKILLS_TIMEOUT_MS);
+    }
+    /** Subscribe to the daemon's `skills_changed` broadcast — fired after any mutation (this
+     *  client's own or another's) or an external filesystem change the daemon detected. */
+    onSkillsChanged(handler) {
+        return this.daemon.onSessionMessage((msg) => {
+            if (msg.type === "skills_changed")
+                handler(msg.reason);
+        });
+    }
 }
 // ─── Handle classes ──────────────────────────────────────────────────────────────
 class TimelineHandle {

package/dist/test-support/scripted-daemon.js

@@ -245,6 +245,95 @@ export function makeScriptedDaemon(opts) {
                 });
                 return;
             }
+            case "skills_list_request": {
+                reply({
+                    type: "skills_list_response",
+                    requestId,
+                    ok: true,
+                    skills: [
+                        {
+                            name: "brave-search",
+                            path: "/home/u/.pi/agent/skills/brave-search",
+                            enabled: true,
+                            origin: "agent_dir",
+                            originLabel: "User",
+                            removable: true,
+                        },
+                    ],
+                    packages: [],
+                });
+                return;
+            }
+            case "skills_set_enabled_request": {
+                reply({
+                    type: "skills_set_enabled_response",
+                    requestId,
+                    ok: true,
+                    skills: [
+                        {
+                            name: "brave-search",
+                            path: msg.path,
+                            enabled: msg.enabled,
+                            origin: "agent_dir",
+                            originLabel: "User",
+                            removable: true,
+                        },
+                    ],
+                    packages: [],
+                });
+                return;
+            }
+            case "skills_remove_request": {
+                reply({
+                    type: "skills_remove_response",
+                    requestId,
+                    ok: true,
+                    skills: [],
+                    packages: [],
+                });
+                return;
+            }
+            case "skills_update_all_request": {
+                reply({
+                    type: "skills_update_all_response",
+                    requestId,
+                    ok: true,
+                    results: [{ target: "agent-skills", ok: true }],
+                    skills: [],
+                    packages: [],
+                });
+                return;
+            }
+            case "skills_preview_request": {
+                reply({
+                    type: "skills_preview_response",
+                    requestId,
+                    ok: true,
+                    kind: "skills_repo",
+                    source: msg.source,
+                    skills: [
+                        {
+                            name: "code-review",
+                            description: "Review a diff",
+                            relPath: "skills/code-review",
+                            valid: true,
+                            exists: false,
+                        },
+                    ],
+                });
+                return;
+            }
+            case "skills_install_request": {
+                reply({
+                    type: "skills_install_response",
+                    requestId,
+                    ok: true,
+                    results: [{ target: String(msg.source), ok: true }],
+                    skills: [],
+                    packages: [],
+                });
+                return;
+            }
             default:
                 reply({ type: `${String(msg.type)}_response`, requestId, payload: { ok: true } });
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@av-pi-studio/client",
-  "version": "0.0.110",
+  "version": "0.0.112",
   "files": [
     "dist",
     "!dist/**/*.map",
@@ -20,7 +20,7 @@
     "clean": "rm -rf dist *.tsbuildinfo"
   },
   "dependencies": {
-    "@av-pi-studio/protocol": "^0.0.110",
-    "@av-pi-studio/relay": "^0.0.110"
+    "@av-pi-studio/protocol": "^0.0.112",
+    "@av-pi-studio/relay": "^0.0.112"
   }
 }