copilot-tap-extension 0.2.5 → 1.0.1

package/README.md

@@ -204,7 +204,7 @@ PLAN.md                         # ubiquitous language and design decisions
 | [Reference](./docs/reference.md) | Look up tool parameters, config fields, or the event pipeline |
 | [Use cases and patterns](./docs/use-cases.md) | Recipes for deploy watchers, PR monitors, log tailers, and more |
 | [Evals](./docs/evals.md) | Run or extend the automated test suite |
-| [Copilot instructions](./.github/copilot-instructions.md) | Understand or customize how the agent uses this extension |
+| [Copilot instructions](./src/copilot-instructions.md) | Understand or customize how the agent uses this extension |
 | [Implementation plan](./PLAN.md) | Ubiquitous language and naming conventions for contributors |
 | [Evolution of the ※ icon](./docs/evolution-of-tap-icon.html) | 20 metaphors, 10 variants, one mark — the design story behind ※ tap |
 

package/bin/install.mjs

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
-import { existsSync, mkdirSync, copyFileSync } from "node:fs";
+import { existsSync, mkdirSync, copyFileSync, readFileSync } from "node:fs";
+import { execFileSync } from "node:child_process";
 import { fileURLToPath } from "node:url";
 import path from "node:path";
 import os from "node:os";
@@ -11,6 +12,14 @@ const distDir = path.join(pkgRoot, "dist");
 const BRAND = "※ tap";
 const EXT_DIR_NAME = "tap";
 
+function getPackageVersion() {
+  try {
+    return JSON.parse(readFileSync(path.join(distDir, "version.json"), "utf8")).version;
+  } catch {
+    return JSON.parse(readFileSync(path.join(pkgRoot, "package.json"), "utf8")).version;
+  }
+}
+
 function usage() {
   console.log(`
 ${BRAND} — Copilot CLI extension installer
@@ -18,14 +27,18 @@ ${BRAND} — Copilot CLI extension installer
 Usage:
   npx copilot-tap-extension [options]
 
+If ※ tap is already installed, updates core files (extension + version)
+and preserves customizable artifacts. If fresh, does a full install.
+
 Options:
   --global, -g     Install to ~/.copilot/  (default)
   --local,  -l     Install to .github/  (project-scoped)
-  --force,  -f     Overwrite existing files without prompting
-  --help,   -h     Show this help message
+  --full           Force a full install even if already installed
+  --help,  -h      Show this help message
 
 Installs:
   extensions/tap/extension.mjs    The bundled ※ tap extension
+  extensions/tap/version.json     Installed version metadata
   skills/loop/SKILL.md            The /loop skill for prompt-based loops
   copilot-instructions.md         Agent instructions for using ※ tap
 `);
@@ -33,7 +46,7 @@ Installs:
 
 function parseArgs(argv) {
   const args = argv.slice(2);
-  const flags = { scope: "global", force: false, help: false };
+  const flags = { scope: "global", full: false, help: false };
   for (const arg of args) {
     switch (arg) {
       case "--global":
@@ -44,9 +57,14 @@ function parseArgs(argv) {
       case "-l":
         flags.scope = "local";
         break;
+      case "--full":
+        flags.full = true;
+        break;
+      // Keep legacy flags working
       case "--force":
       case "-f":
-        flags.force = true;
+      case "--update":
+      case "-u":
         break;
       case "--help":
       case "-h":
@@ -61,40 +79,94 @@ function parseArgs(argv) {
   return flags;
 }
 
+function getCopilotHome() {
+  return process.env.COPILOT_HOME || path.join(os.homedir(), ".copilot");
+}
+
 function getTargetRoot(scope) {
   if (scope === "global") {
-    return path.join(os.homedir(), ".copilot");
+    return getCopilotHome();
   }
   return path.join(process.cwd(), ".github");
 }
 
-function copyArtifact(src, dest, label, flags) {
+function copyArtifact(src, dest, label) {
   if (!existsSync(src)) {
     console.error(`  ✗ ${label}: source not found (${src})`);
     return false;
   }
-  if (existsSync(dest) && !flags.force) {
-    console.log(`  ⊘ ${label}: already exists, skipping (use --force to overwrite)`);
-    return true;
-  }
   mkdirSync(path.dirname(dest), { recursive: true });
   copyFileSync(src, dest);
   console.log(`  ✓ ${label}`);
   return true;
 }
 
+function getInstalledVersion(targetRoot) {
+  try {
+    const versionFile = path.join(targetRoot, "extensions", EXT_DIR_NAME, "version.json");
+    return JSON.parse(readFileSync(versionFile, "utf8")).version;
+  } catch {
+    return null;
+  }
+}
+
+function isAlreadyInstalled(targetRoot) {
+  return existsSync(path.join(targetRoot, "extensions", EXT_DIR_NAME, "extension.mjs"));
+}
+
+function isCopilotCliInstalled() {
+  if (existsSync(getCopilotHome())) {
+    return true;
+  }
+  try {
+    execFileSync("copilot", ["--version"], { stdio: "ignore", timeout: 5000 });
+    return true;
+  } catch {
+    return false;
+  }
+}
+
 function install(flags) {
   const targetRoot = getTargetRoot(flags.scope);
   const scopeLabel = flags.scope === "global" ? "global (~/.copilot)" : "local (.github)";
+  const packageVersion = getPackageVersion();
+
+  if (flags.scope === "global" && !isCopilotCliInstalled()) {
+    console.log(`\n⚠  Copilot CLI does not appear to be installed.`);
+    console.log(`   Install it first: https://docs.github.com/en/copilot/github-copilot-in-the-cli`);
+    console.log(`   Then re-run: npx copilot-tap-extension\n`);
+    process.exit(1);
+  }
+
+  const installed = isAlreadyInstalled(targetRoot);
+  const isUpdate = installed && !flags.full;
 
-  console.log(`\n${BRAND} — installing (${scopeLabel})\n`);
+  if (isUpdate) {
+    const installedVersion = getInstalledVersion(targetRoot);
+    if (installedVersion && installedVersion === packageVersion) {
+      console.log(`\n${BRAND} — already up to date (v${installedVersion})\n`);
+      process.exit(0);
+    }
+    const fromLabel = installedVersion ? `v${installedVersion}` : "unknown";
+    console.log(`\n${BRAND} — updating ${fromLabel} → v${packageVersion} (${scopeLabel})\n`);
+  } else {
+    console.log(`\n${BRAND} — installing v${packageVersion} (${scopeLabel})\n`);
+  }
 
-  const artifacts = [
+  const coreArtifacts = [
     {
       src: path.join(distDir, "extension.mjs"),
       dest: path.join(targetRoot, "extensions", EXT_DIR_NAME, "extension.mjs"),
       label: "extensions/tap/extension.mjs"
     },
+    {
+      src: path.join(distDir, "version.json"),
+      dest: path.join(targetRoot, "extensions", EXT_DIR_NAME, "version.json"),
+      label: "extensions/tap/version.json"
+    }
+  ];
+
+  const ancillaryArtifacts = [
     {
       src: path.join(distDir, "skills", "loop", "SKILL.md"),
       dest: path.join(targetRoot, "skills", "loop", "SKILL.md"),
@@ -107,18 +179,21 @@ function install(flags) {
     }
   ];
 
+  const artifacts = isUpdate ? coreArtifacts : [...coreArtifacts, ...ancillaryArtifacts];
+
   let allOk = true;
   for (const { src, dest, label } of artifacts) {
-    if (!copyArtifact(src, dest, label, flags)) {
+    if (!copyArtifact(src, dest, label)) {
       allOk = false;
     }
   }
 
   console.log();
   if (allOk) {
-    console.log(`✓ ${BRAND} installed to ${targetRoot}`);
+    const verb = isUpdate ? "updated" : "installed";
+    console.log(`✓ ${BRAND} ${verb} to ${targetRoot}`);
   } else {
-    console.error(`⚠  Some artifacts could not be installed.`);
+    console.error(`⚠  Some artifacts could not be ${isUpdate ? "updated" : "installed"}.`);
     process.exit(1);
   }
 }

package/dist/extension.mjs

@@ -2,7 +2,7 @@
 // https://github.com/amitse/copilot-tap-extension
 
 
-// .github/extensions/tap/extension.mjs
+// src/extension.mjs
 import { joinSession } from "@github/copilot-sdk/extension";
 
 // src/consts.mjs
@@ -13,7 +13,7 @@ var CONFIG_LOCATIONS = [
   CONFIG_FILENAME,
   `${GITHUB_DIR}${path.sep}${CONFIG_FILENAME}`
 ];
-var COPILOT_INSTRUCTIONS_PATH = `${GITHUB_DIR}/copilot-instructions.md`;
+var COPILOT_INSTRUCTIONS_PATH = "src/copilot-instructions.md";
 var MAX_STREAM_ENTRIES = 200;
 var DEFAULT_STREAM = "main";
 var DEFAULT_STREAM_DESCRIPTION = "Extension events";
@@ -766,12 +766,6 @@ function createLineRouter({ streams, notifications, sessionPort }) {
         monitorName: emitter.name,
         stream: STREAM.PROMPT
       });
-      notifications.enqueue({
-        channel: emitter.stream,
-        monitorName: emitter.name,
-        stream: STREAM.PROMPT,
-        text
-      });
     }
   }
   return { handleLine, handleTextBlock, handlePromptResult, appendSystemMessage };
@@ -951,19 +945,11 @@ function createLifecycle({ lineRouter, sessionPort }) {
   }
   async function runPromptIteration(emitter) {
     try {
-      const response = await sessionPort.sendAndWait(emitter.prompt);
+      await sessionPort.send(emitter.prompt);
       if (emitter.stopRequested) {
         return { ok: true };
       }
-      const responseText = toText(response?.data?.content ?? response?.data ?? response);
-      if (responseText.trim()) {
-        lineRouter.handlePromptResult(emitter, responseText);
-      } else {
-        lineRouter.appendSystemMessage(
-          emitter,
-          `Emitter '${emitter.name}' received an empty response from prompt work.`
-        );
-      }
+      emitter.lineCount += 1;
       return { ok: true };
     } catch (error) {
       return {
@@ -1537,6 +1523,116 @@ function createTools(deps) {
   ];
 }
 
+// src/update/checker.mjs
+import { existsSync as existsSync2, readFileSync as readFileSync2, writeFileSync as writeFileSync2, mkdirSync } from "node:fs";
+import { fileURLToPath } from "node:url";
+import path4 from "node:path";
+import os from "node:os";
+var PKG_NAME = "copilot-tap-extension";
+var REGISTRY_URL = `https://registry.npmjs.org/${PKG_NAME}/latest`;
+function getCopilotHome() {
+  return process.env.COPILOT_HOME || path4.join(os.homedir(), ".copilot");
+}
+var CHECK_INTERVAL_MS = 24 * 60 * 60 * 1e3;
+function getInstalledVersion() {
+  try {
+    const extensionDir = path4.dirname(fileURLToPath(import.meta.url));
+    const candidates = [
+      path4.join(extensionDir, "version.json"),
+      path4.join(extensionDir, "..", "version.json"),
+      path4.join(extensionDir, "..", "..", "dist", "version.json")
+    ];
+    for (const candidate of candidates) {
+      if (existsSync2(candidate)) {
+        return JSON.parse(readFileSync2(candidate, "utf8")).version;
+      }
+    }
+    return null;
+  } catch {
+    return null;
+  }
+}
+function getUpdateStateFile() {
+  return path4.join(getCopilotHome(), ".tap-update-state.json");
+}
+function readUpdateState() {
+  try {
+    return JSON.parse(readFileSync2(getUpdateStateFile(), "utf8"));
+  } catch {
+    return {};
+  }
+}
+function writeUpdateState(state) {
+  try {
+    const stateDir = getCopilotHome();
+    mkdirSync(stateDir, { recursive: true });
+    writeFileSync2(getUpdateStateFile(), JSON.stringify(state, null, 2) + "\n");
+  } catch {
+  }
+}
+function shouldCheck() {
+  const state = readUpdateState();
+  if (!state.lastCheckAt) {
+    return true;
+  }
+  return Date.now() - state.lastCheckAt > CHECK_INTERVAL_MS;
+}
+function recordCheck(latest) {
+  const state = readUpdateState();
+  state.lastCheckAt = Date.now();
+  if (latest) {
+    state.latestVersion = latest;
+  }
+  writeUpdateState(state);
+}
+async function fetchLatestVersion() {
+  const res = await fetch(REGISTRY_URL);
+  if (!res.ok) {
+    return null;
+  }
+  const data = await res.json();
+  return data.version ?? null;
+}
+function isNewer(installed, latest) {
+  if (!installed || !latest) {
+    return !!latest;
+  }
+  const pa = installed.split(".").map(Number);
+  const pb = latest.split(".").map(Number);
+  for (let i = 0; i < 3; i++) {
+    if ((pb[i] || 0) > (pa[i] || 0)) {
+      return true;
+    }
+    if ((pb[i] || 0) < (pa[i] || 0)) {
+      return false;
+    }
+  }
+  return false;
+}
+async function checkForUpdate(sessionPort) {
+  try {
+    if (!shouldCheck()) {
+      return;
+    }
+    const installed = getInstalledVersion();
+    if (!installed) {
+      return;
+    }
+    const latest = await fetchLatestVersion();
+    if (!latest) {
+      return;
+    }
+    recordCheck(latest);
+    if (!isNewer(installed, latest)) {
+      return;
+    }
+    await sessionPort.log(
+      `Update available: v${installed} \u2192 v${latest}. Run \`npx ${PKG_NAME}\` to update.`
+    );
+  } catch {
+  }
+}
+
 // src/hooks.mjs
 function sessionInjectorSummary(streams) {
   const subscribed = streams.list().filter((stream) => stream.sessionInjector.enabled);
@@ -1589,6 +1685,8 @@ function createHooks({ streams, configStore, supervisor, sessionPort, setBaseCwd
   return {
     onSessionStart: async (input) => {
       streams.ensure(DEFAULT_STREAM, DEFAULT_STREAM_DESCRIPTION);
+      checkForUpdate(sessionPort).catch(() => {
+      });
       let configSummary = "No config loaded.";
       try {
         configSummary = await applyPersistentConfig({
@@ -1664,7 +1762,7 @@ function createCopilotChannelsRuntime(options = {}) {
   };
 }
 
-// .github/extensions/tap/extension.mjs
+// src/extension.mjs
 var runtime = createCopilotChannelsRuntime({
   cwd: process.cwd()
 });

package/dist/skills/loop/SKILL.md

@@ -67,12 +67,11 @@ When this skill is invoked:
 
 ## Why subscribe defaults to false
 
-PromptEmitter output is already delivered through two paths:
+PromptEmitter output is delivered through a single path:
 
-1. **`session.sendAndWait()`** -- the prompt runs inside the session, so Copilot processes and responds to it directly.
-2. **Notification dispatcher** -- each result line is also enqueued via `handlePromptResult` and injected as a background event stream update via `session.send()`.
+1. **`session.send()`** -- the prompt is dispatched fire-and-forget; Copilot processes and responds to it directly inside the session.
 
-The `subscribe` flag controls a third layer: the **SessionInjector**. When enabled, it additionally pushes system-level messages (emitter started, stopped, errored) into the session.
+The `subscribe` flag controls the **SessionInjector**. When enabled, it additionally pushes system-level messages (emitter started, stopped, errored) into the session.
 
 For PromptEmitters, the main results already reach the session without the SessionInjector. Setting `subscribe = true` adds system noise on top of content that is already being delivered. Default to `false` to keep things clean.
 

package/dist/version.json

@@ -0,0 +1,3 @@
+{
+  "version": "1.0.1"
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "copilot-tap-extension",
-  "version": "0.2.5",
+  "version": "1.0.1",
   "description": "Copilot CLI extension for background event emitters, event streams, and session injection.",
   "type": "module",
   "license": "MIT",
@@ -20,7 +20,7 @@
   "scripts": {
     "build": "node scripts/build.mjs",
     "prepublishOnly": "npm run build",
-    "check": "node --check ./src/tap-runtime.mjs && node --check ./.github/extensions/tap/extension.mjs && node --check ./evals/run.mjs && node --check ./examples/heartbeat.mjs",
+    "check": "node --check ./src/tap-runtime.mjs && node --check ./src/extension.mjs && node --check ./evals/run.mjs && node --check ./examples/heartbeat.mjs",
     "demo:heartbeat": "node ./examples/heartbeat.mjs",
     "evals:list": "node ./evals/run.mjs list",
     "evals:smoke": "node ./evals/run.mjs smoke",