@oas-framework/pi 0.4.0 → 0.5.0

package/extension/index.ts

@@ -17,7 +17,7 @@ import { StringEnum } from "@earendil-works/pi-ai";
 import {
   appendLogEntry, createAgent, defaultRepo, ensureRoot, findAgent, findRoot, listAgentDefs, listAgents,
   listInstances, retireInstance, spawnInstance, upsertTmpAgent, workspaceOf, writeSoul, PACKAGED_SKILLS_DIR,
-  workspaceConfigSkillDirs, isOasWorkspace, resolveOasConfig, runLifecycleHooks,
+  workspaceConfigSkillDirs, isOasWorkspace,
 } from "./core-loader.mjs";
 import { existsSync, readFileSync } from "node:fs";
 import { dirname, join, resolve } from "node:path";
@@ -94,11 +94,15 @@ export default function (pi: ExtensionAPI) {
   const agentHome = process.env.PI_AGENT_HOME;
   const isInstance = !!agentHome && existsSync(join(agentHome ?? "", "instance.json"));
 
-  // ---------- kernel + workspace skills: only inside OAS workspaces ----------
-  // (a workspace/repo with an oas-config, an agents/ root, or an instance home —
-  //  agents elsewhere on the machine do not get OAS skills)
+  // ---------- OAS skills ----------
+  // getting-started is GLOBAL (it bootstraps — must be discoverable before any OAS
+  // workspace exists). The rest of the bundle is gated to OAS contexts (instance
+  // home, or a workspace/repo with an oas-config / agents root). Workspace and
+  // config skills stay trust-gated.
   pi.on("resources_discover", async (event, ctx) => {
     const paths: string[] = [];
+    const gettingStarted = join(PACKAGED_SKILLS_DIR, "oas-getting-started");
+    if (existsSync(gettingStarted)) paths.push(gettingStarted);
     const inOas = isInstance || isOasWorkspace(event.cwd);
     if (inOas && existsSync(PACKAGED_SKILLS_DIR)) paths.push(PACKAGED_SKILLS_DIR);
     if (ctx.isProjectTrusted()) {
@@ -135,25 +139,9 @@ export default function (pi: ExtensionAPI) {
       }, { deliverAs: "steer" });
     });
 
-    // Post-commit lifecycle: after a successful `git commit` by this instance, fire
-    // the integrations' post-commit hooks (e.g. oas-okf spawns a knowledge-harvest
-    // agent attached to this instance's work tree). Kernel stays layer-agnostic.
-    pi.on("tool_result", async (event) => {
-      if (event.toolName !== "bash" || event.isError) return;
-      const cmd = String((event.input as { command?: string })?.command ?? "");
-      if (!/\bgit\b[^\n;&|]*\bcommit\b/.test(cmd)) return;
-      try {
-        const meta = JSON.parse(readFileSync(join(agentHome!, "instance.json"), "utf8"));
-        const root = findRoot(agentHome!);
-        if (!root || !meta.repo) return;
-        runLifecycleHooks("post-commit", {
-          home: agentHome!, instance: meta.instance, agentName: meta.agent,
-          soulDir: join(agentHome!, "soul"), contextDir: meta.repo,
-          workspaceDir: workspaceOf(root), rootDir: root,
-          resolved: resolveOasConfig(meta.repo), priorMeta: meta.providerMeta || {},
-        });
-      } catch { /* hooks never break the session */ }
-    });
+    // (post-commit harvest is agent-initiated now — the okf injection/skill tell the
+    // instance to run the integration's `harvest` entry after committing with notes;
+    // the kernel no longer watches git commits.)
 
     // Resume nudge: on any later session in this home (restart, model switch, /new),
     // point the model at its own memory before it does anything else.
@@ -202,9 +190,10 @@ export default function (pi: ExtensionAPI) {
       purpose: Type.Optional(Type.String({ description: "Short kebab slug describing what this instance is for; names the instance <agent>-<purpose>" })),
       model: Type.Optional(Type.String({ description: "Model as provider/id (must be enabled in pi); defaults to the soul's model or pi's default" })),
       repo: Type.Optional(Type.String({ description: "Target repo (absolute, or relative to the agents root's parent). Defaults to the soul's repo, else the current git repo" })),
-      work: Type.Optional(StringEnum(["worktree", "checkout"] as const, {
-        description: "worktree = own git worktree + branch; checkout = work directly on the repo's checked-out branch. Defaults to the soul's setting",
+      work: Type.Optional(StringEnum(["worktree", "checkout", "attached"] as const, {
+        description: "worktree = own git worktree + branch; checkout = work directly on the repo's checked-out branch; attached = share ANOTHER instance's work tree (requires workDir). Defaults to the soul's setting",
       })),
+      workDir: Type.Optional(Type.String({ description: "attached mode: the owning instance's work tree (its <home>/work) to attach to" })),
       runtime: Type.Optional(StringEnum(["pi", "claude"] as const, { description: "Session runtime (default: soul's setting, else pi)" })),
       branch: Type.Optional(Type.String({ description: "Branch name for worktree mode (default: agents/<instance>)" })),
       launch: Type.Optional(Type.Boolean({ description: "Launch in tmux (default true). false = scaffold only and return the launch command" })),
@@ -235,7 +224,7 @@ export default function (pi: ExtensionAPI) {
 
       const result = spawnInstance(root, agent!, {
         instance: undefined, purpose: p.purpose, repo: p.repo || agent!.repo || defaultRepo(ctx.cwd),
-        work: p.work, runtime: p.runtime, model, task: p.task, branch: p.branch, launch: p.launch,
+        work: p.work, workDir: p.workDir, runtime: p.runtime, model, task: p.task, branch: p.branch, launch: p.launch,
       });
       return { ...text(JSON.stringify(result, null, 2)), details: result };
     },
@@ -418,7 +407,7 @@ export default function (pi: ExtensionAPI) {
     parameters: Type.Object({
       name: Type.String(),
       repo: Type.Optional(Type.String({ description: "Target repo (absolute, or relative to the agents root's parent)" })),
-      work: Type.Optional(StringEnum(["worktree", "checkout"] as const)),
+      work: Type.Optional(StringEnum(["worktree", "checkout", "attached"] as const)),
       runtime: Type.Optional(StringEnum(["pi", "claude"] as const)),
       model: Type.Optional(Type.String({ description: "Default model as provider/id (must be enabled in pi)" })),
       description: Type.Optional(Type.String()),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oas-framework/pi",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "description": "OAS pi adapter — agent tools (spawn_agent, agents_status, retire_agent) and skill discovery for pi, over the globally installed @oas-framework/oas kernel",
   "keywords": [
     "pi-package",