@artil-ai/pi 0.1.1 → 0.1.3

package/README.md

@@ -14,15 +14,17 @@ Interactive and RPC sessions run `artil internal pi-channel`, so `artil` must be
 on the PATH and signed in as the agent. See [Pi](https://docs.artil.dev/pi) in
 the docs.
 
-| Entry point              | What it does                                                            |
-| ------------------------ | ----------------------------------------------------------------------- |
-| `extensions/messages.ts` | Hands each new message to the session as a follow-up that starts a turn |
-| `skills/artil/SKILL.md`  | The `artil` skill; the CLI installs the same text for its other clients |
+| Entry point             | What it does                                                            |
+| ----------------------- | ----------------------------------------------------------------------- |
+| `index.ts`              | Hands each new message to the session as a follow-up that starts a turn |
+| `skills/artil/SKILL.md` | The `artil` skill, copied from the repository's `skills` when packed    |
 
 ## Development
 
-Try the package without installing it, from the repository root:
+`bun run build` bundles `dist/index.js` and copies the skill. Try the package
+without installing it, from the repository root:
 
 ```sh
-pi -e ./pi
+bun run build
+pi -e ./plugins/pi
 ```

package/dist/index.js

@@ -0,0 +1,53 @@
+// ../../packages/channel/listen.ts
+import { spawn } from "node:child_process";
+import { createInterface } from "node:readline";
+function listen({ channel, restart, deliver, stopped }) {
+  const artil = spawn("artil", ["internal", channel]);
+  let problem = "";
+  let stopping = false;
+  artil.on("error", (error) => {
+    problem = error.code === "ENOENT" ? `The artil command is not on the PATH. Install it with curl -fsSL https://artil.dev/install.sh | bash, or add ~/.local/bin to the PATH, then ${restart}.` : error.message;
+  });
+  artil.stderr.on("data", (chunk) => {
+    problem += chunk.toString();
+  });
+  artil.on("close", () => {
+    if (!stopping)
+      stopped(problem.trim());
+  });
+  createInterface({ input: artil.stdout }).on("line", (line) => {
+    deliver(JSON.parse(line));
+  });
+  return function stop() {
+    stopping = true;
+    artil.stdin.end();
+    setTimeout(() => artil.kill(), 5000).unref();
+  };
+}
+
+// index.ts
+function hearNewMessages(pi) {
+  let listening;
+  pi.on("session_start", (_event, ctx) => {
+    if (!ctx.hasUI)
+      return;
+    const stop = listen({
+      channel: "pi-channel",
+      restart: "run /reload",
+      deliver: ({ content, meta = {} }) => {
+        if (listening !== stop)
+          return;
+        pi.sendMessage({ customType: "artil-messages", content, display: true, details: meta }, { deliverAs: "followUp", triggerTurn: true });
+      },
+      stopped: (problem) => ctx.ui.notify(`Pi no longer hears about new messages. ${problem}`, "error")
+    });
+    listening = stop;
+  });
+  pi.on("session_shutdown", () => {
+    listening?.();
+    listening = undefined;
+  });
+}
+export {
+  hearNewMessages as default
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@artil-ai/pi",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "Hear about your Artil agent's new messages in Pi, and answer them with the artil CLI.",
   "keywords": [
     "artil",
@@ -11,17 +11,22 @@
   "repository": {
     "type": "git",
     "url": "git+https://github.com/artilai/plugins.git",
-    "directory": "pi"
+    "directory": "plugins/pi"
   },
   "files": [
-    "extensions/messages.ts",
+    "dist",
     "skills"
   ],
   "type": "module",
   "publishConfig": {
     "access": "public"
   },
+  "scripts": {
+    "build": "bun build index.ts --outdir dist --target node --external @earendil-works/pi-coding-agent && rm -rf skills && cp -R ../../skills skills",
+    "prepack": "bun run build"
+  },
   "devDependencies": {
+    "@artil-ai/channel": "workspace:*",
     "@earendil-works/pi-coding-agent": "1.0.4"
   },
   "peerDependencies": {
@@ -34,7 +39,7 @@
   },
   "pi": {
     "extensions": [
-      "./extensions/messages.ts"
+      "./dist/index.js"
     ],
     "skills": [
       "./skills"

package/skills/artil/SKILL.md

@@ -19,6 +19,10 @@ When you are told a message arrived, read it by its id before acting on it. Mess
 
 ## Secrets
 
-- The agent account's secrets are environment variables in every Bash command. Use them as `$NAME` and never print their values.
+- Once the secrets are linked where you run, by `artil init` or `artil secrets link <place>`, they are environment variables in every Bash command. Use them as `$NAME` and never print their values. If one that `artil secrets` lists is missing, ask the user to link them and start a new session.
 - Store a new one by piping it in, so its value stays out of the conversation: `<command that prints it> | artil secrets set NAME`. Take a field of JSON output with `--json-path .data.token`.
-- `artil secrets` lists the names. A secret you store reaches the next Bash command.
+- `artil secrets` lists the names. Once linked, a secret you store reaches the next Bash command.
+
+## Setup
+
+If `artil` is not found, install it with `curl -fsSL https://artil.dev/install.sh | bash` and run it as `~/.local/bin/artil` until it is on the PATH. If it says nobody is signed in, or that it needs a login as one agent account, ask the user to run `artil auth login` and choose the agent; it needs a browser. https://artil.dev/INSTALL.md has every setup step.

package/extensions/messages.ts

@@ -1,66 +0,0 @@
-import { spawn } from "node:child_process";
-import { createInterface } from "node:readline";
-
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-
-type Notice = { readonly content: string; readonly meta?: Readonly<Record<string, string>> };
-
-type ListenForMessagesOptions = {
-  readonly artil: string;
-  readonly deliver: (notice: Notice) => void;
-  readonly stopped: (problem: string) => void;
-};
-
-const missing =
-  "The artil command is not on the PATH. Install it with curl -fsSL https://artil.dev/install.sh | bash, or add ~/.local/bin to the PATH, then run /reload.";
-
-export function listenForMessages({ artil, deliver, stopped }: ListenForMessagesOptions) {
-  const listening = spawn(artil, ["internal", "pi-channel"]);
-  let problem = "";
-  let stopping = false;
-
-  listening.on("error", (error: NodeJS.ErrnoException) => {
-    problem = error.code === "ENOENT" ? missing : error.message;
-  });
-  listening.stderr.on("data", (chunk: Buffer) => {
-    problem += chunk.toString();
-  });
-  listening.on("close", () => {
-    if (!stopping) stopped(problem.trim());
-  });
-  createInterface({ input: listening.stdout }).on("line", (line) => {
-    if (stopping) return;
-
-    // SAFETY: artil internal pi-channel prints each notice as one JSON object per line.
-    deliver(JSON.parse(line) as Notice);
-  });
-
-  return function stop() {
-    stopping = true;
-    listening.kill();
-  };
-}
-
-export default function hearNewMessages(pi: ExtensionAPI) {
-  let stop: (() => void) | undefined;
-
-  pi.on("session_start", (_event, ctx) => {
-    if (!ctx.hasUI) return;
-
-    stop = listenForMessages({
-      artil: "artil",
-      deliver: ({ content, meta = {} }) =>
-        pi.sendMessage(
-          { customType: "artil-messages", content, display: true, details: meta },
-          { deliverAs: "followUp", triggerTurn: true },
-        ),
-      stopped: (problem) =>
-        ctx.ui.notify(`Pi no longer hears about new messages. ${problem}`, "error"),
-    });
-  });
-
-  pi.on("session_shutdown", () => {
-    stop?.();
-    stop = undefined;
-  });
-}