copilot-tap-extension 1.1.4 → 2.0.0

package/README.md

@@ -62,7 +62,7 @@ npx copilot-tap-extension
 npx copilot-tap-extension --local
 ```
 
-This installs the bundled extension, the `/loop` skill, the `/monitor` skill, and the agent instructions to the appropriate Copilot directory. Run `npx copilot-tap-extension --help` for all options.
+This installs the bundled extension, the `/tap-loop` skill, the `/tap-monitor` skill, and the agent instructions to the appropriate Copilot directory. Run `npx copilot-tap-extension --help` for all options.
 
 To update to the latest version, re-run the same command with `--force`:
 
@@ -103,13 +103,13 @@ The config file tells the extension which emitters to auto-start. The example de
 
 This runs the heartbeat script on session start, drops boot messages, injects warnings and errors, and keeps everything else in the stream.
 
-Once inside the session, describe what you want in natural language. You can also use `/loop` to set up scheduled prompts directly:
+Once inside the session, describe what you want in natural language. You can also use `/tap-loop` to set up scheduled prompts directly:
 
 > _"Watch my build logs and tell me if anything fails"_
 
-> _"/loop 5m check for new PR review comments"_
+> _"/tap-loop 5m check for new PR review comments"_
 
-> _"/monitor tail -f /var/log/app.log"_
+> _"/tap-monitor tail -f /var/log/app.log"_
 
 > _"Tail the API logs, inject errors, drop health checks"_
 
@@ -175,11 +175,11 @@ You keep coding. Twenty minutes later, Copilot interrupts: "Run 48291: deploymen
 
 **Monitor a command with self-tuning filters**
 
-Use `/monitor` to run a shell command continuously while a companion agent periodically reads the output and updates the filter expressions to separate noise from signal automatically.
+Use `/tap-monitor` to run a shell command continuously while a companion agent periodically reads the output and updates the filter expressions to separate noise from signal automatically.
 
 ```
-/monitor tail -f /var/log/app.log
-/monitor 10m docker logs -f mycontainer
+/tap-monitor tail -f /var/log/app.log
+/tap-monitor 10m docker logs -f mycontainer
 ```
 
 The command stream starts with a sensible initial `notifyPattern`. Every few minutes (configurable) the companion reviews recent log lines and calls `tap_set_event_filter` if the patterns need adjustment. The filter tightens itself based on real output — no manual tuning required.
@@ -189,7 +189,7 @@ The command stream starts with a sensible initial `notifyPattern`. Every few min
 A PromptEmitter re-runs an agent prompt at a fixed interval. Useful for PR comments, CI status, or ticket queues.
 
 ```
-/loop 15m Check for new failing CI runs or PR review comments.
+/tap-loop 15m Check for new failing CI runs or PR review comments.
          Summarize only actionable items.
 ```
 
@@ -197,10 +197,10 @@ Every 15 minutes the agent scans and reports back. No news means no interruption
 
 **Run a prompt when idle**
 
-Use `/loop idle` to re-run a prompt whenever the session has nothing else to do. Set `maxRuns` to cap iterations.
+Use `/tap-loop idle` to re-run a prompt whenever the session has nothing else to do. Set `maxRuns` to cap iterations.
 
 ```
-/loop idle Scan for new issues labeled urgent. Summarize what changed.
+/tap-loop idle Scan for new issues labeled urgent. Summarize what changed.
 ```
 
 The prompt fires immediately, then re-fires after each idle period. It stops after reaching the iteration limit.
@@ -222,8 +222,9 @@ Rules can be added or changed while the emitter is running. You never need to re
 ```text
 .github/
   extensions/tap/extension.mjs  # extension entry point (loads the runtime)
-  skills/loop/                  # /loop skill for scheduled and idle prompts
-  skills/monitor/               # /monitor skill for self-tuning command monitors
+  skills/tap-loop/                  # /tap-loop skill for scheduled and idle prompts
+  skills/tap-monitor/               # /tap-monitor skill for self-tuning command monitors
+  skills/tap-create-provider/       # /tap-create-provider skill for scaffolding external tool providers
   copilot-instructions.md       # agent guidance for using this extension
 src/
   emitter/                      # supervisor, lifecycle, spawn, line router

package/bin/install.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { existsSync, mkdirSync, copyFileSync, readFileSync } from "node:fs";
+import { existsSync, mkdirSync, copyFileSync, readFileSync, unlinkSync } from "node:fs";
 import { execFileSync } from "node:child_process";
 import { fileURLToPath } from "node:url";
 import path from "node:path";
@@ -33,22 +33,22 @@ and preserves customizable artifacts. If fresh, does a full install.
 Options:
   --global, -g     Install to ~/.copilot/  (default)
   --local,  -l     Install to .github/  (project-scoped)
-  --full           Force a full install even if already installed
+  --force, -f      Force a full reinstall 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
-  skills/create-provider/SKILL.md The /create-provider skill for scaffolding providers
-  skills/monitor/SKILL.md         The /monitor skill for self-tuning command monitors
+  skills/tap-loop/SKILL.md            The /tap-loop skill for prompt-based loops
+  skills/tap-create-provider/SKILL.md The /tap-create-provider skill for scaffolding providers
+  skills/tap-monitor/SKILL.md         The /tap-monitor skill for self-tuning command monitors
   copilot-instructions.md         Agent instructions for using ※ tap
 `);
 }
 
 function parseArgs(argv) {
   const args = argv.slice(2);
-  const flags = { scope: "global", full: false, help: false };
+  const flags = { scope: "global", force: false, help: false };
   for (const arg of args) {
     switch (arg) {
       case "--global":
@@ -59,12 +59,12 @@ function parseArgs(argv) {
       case "-l":
         flags.scope = "local";
         break;
+      case "--force":
+      case "-f":
       case "--full":
-        flags.full = true;
+        flags.force = true;
         break;
       // Keep legacy flags working
-      case "--force":
-      case "-f":
       case "--update":
       case "-u":
         break;
@@ -128,6 +128,36 @@ function isCopilotCliInstalled() {
   }
 }
 
+function removeDeprecatedSkills(targetRoot) {
+  const deprecated = ["loop", "monitor", "create-provider"];
+  let allOk = true;
+  let removedAny = false;
+
+  for (const name of deprecated) {
+    const oldPath = path.join(targetRoot, "skills", name, "SKILL.md");
+    if (!existsSync(oldPath)) {
+      continue;
+    }
+    try {
+      unlinkSync(oldPath);
+      if (!removedAny) {
+        console.log();
+        removedAny = true;
+      }
+      console.log(`  ✓ Removed deprecated skill: skills/${name}/SKILL.md`);
+    } catch {
+      allOk = false;
+      console.warn(`  ⚠  Could not remove deprecated skill at ${oldPath} — remove it manually`);
+    }
+  }
+
+  if (removedAny) {
+    console.log(`\n  Use the new namespaced commands: /tap-loop  /tap-monitor  /tap-create-provider`);
+  }
+
+  return allOk;
+}
+
 function install(flags) {
   const targetRoot = getTargetRoot(flags.scope);
   const scopeLabel = flags.scope === "global" ? "global (~/.copilot)" : "local (.github)";
@@ -141,16 +171,20 @@ function install(flags) {
   }
 
   const installed = isAlreadyInstalled(targetRoot);
-  const isUpdate = installed && !flags.full;
+  const isUpdate = installed && !flags.force;
+  const isReinstall = installed && flags.force;
+  const installedVersion = installed ? getInstalledVersion(targetRoot) : null;
 
   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 if (isReinstall) {
+    const fromLabel = installedVersion ? `v${installedVersion}` : "unknown";
+    console.log(`\n${BRAND} — reinstalling ${fromLabel} → v${packageVersion} (${scopeLabel})\n`);
   } else {
     console.log(`\n${BRAND} — installing v${packageVersion} (${scopeLabel})\n`);
   }
@@ -170,19 +204,19 @@ function install(flags) {
 
   const ancillaryArtifacts = [
     {
-      src: path.join(distDir, "skills", "loop", "SKILL.md"),
-      dest: path.join(targetRoot, "skills", "loop", "SKILL.md"),
-      label: "skills/loop/SKILL.md"
+      src: path.join(distDir, "skills", "tap-loop", "SKILL.md"),
+      dest: path.join(targetRoot, "skills", "tap-loop", "SKILL.md"),
+      label: "skills/tap-loop/SKILL.md"
     },
     {
-      src: path.join(distDir, "skills", "create-provider", "SKILL.md"),
-      dest: path.join(targetRoot, "skills", "create-provider", "SKILL.md"),
-      label: "skills/create-provider/SKILL.md"
+      src: path.join(distDir, "skills", "tap-create-provider", "SKILL.md"),
+      dest: path.join(targetRoot, "skills", "tap-create-provider", "SKILL.md"),
+      label: "skills/tap-create-provider/SKILL.md"
     },
     {
-      src: path.join(distDir, "skills", "monitor", "SKILL.md"),
-      dest: path.join(targetRoot, "skills", "monitor", "SKILL.md"),
-      label: "skills/monitor/SKILL.md"
+      src: path.join(distDir, "skills", "tap-monitor", "SKILL.md"),
+      dest: path.join(targetRoot, "skills", "tap-monitor", "SKILL.md"),
+      label: "skills/tap-monitor/SKILL.md"
     },
     {
       src: path.join(distDir, "copilot-instructions.md"),
@@ -191,7 +225,12 @@ function install(flags) {
     }
   ];
 
-  const artifacts = isUpdate ? coreArtifacts : [...coreArtifacts, ...ancillaryArtifacts];
+  // During updates, also install ancillary artifacts that don't yet exist at the destination
+  // (e.g. new skills added in a newer version). Existing ones are preserved to keep user customizations.
+  const newAncillaryArtifacts = isUpdate
+    ? ancillaryArtifacts.filter(({ dest }) => !existsSync(dest))
+    : ancillaryArtifacts;
+  const artifacts = [...coreArtifacts, ...newAncillaryArtifacts];
 
   let allOk = true;
   for (const { src, dest, label } of artifacts) {
@@ -200,14 +239,20 @@ function install(flags) {
     }
   }
 
+  if (installed && !removeDeprecatedSkills(targetRoot)) {
+    allOk = false;
+  }
+
   console.log();
   if (allOk) {
-    const verb = isUpdate ? "updated" : "installed";
+    const verb = isUpdate ? "updated" : isReinstall ? "reinstalled" : "installed";
     console.log(`✓ ${BRAND} ${verb} to ${targetRoot}`);
-  } else {
-    console.error(`⚠  Some artifacts could not be ${isUpdate ? "updated" : "installed"}.`);
-    process.exit(1);
+    return;
   }
+
+  const verb = isUpdate ? "updated" : isReinstall ? "reinstalled" : "installed";
+  console.error(`⚠  Some artifacts could not be ${verb}.`);
+  process.exit(1);
 }
 
 const flags = parseArgs(process.argv);

package/dist/copilot-instructions.md

@@ -146,7 +146,7 @@ If the work is mostly reasoning rather than data collection, prefer a PromptEmit
 - prompt once for a background check (oneTime)
 - prompt + `runInterval` for a fixed maintenance loop (timed)
 
-This is the closest analogue to Claude's session-scoped `/loop` behavior in this extension.
+This is the closest analogue to Claude's session-scoped `/tap-loop` behavior in this extension.
 
 ## Borrow from the official SDK examples
 

package/dist/skills/{create-provider → tap-create-provider}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: create-provider
+name: tap-create-provider
 description: "Create an external tool provider that connects to ※ tap. Use when the user says 'create a provider', 'build a provider', 'scaffold a provider', 'add an external tool', 'connect a service to tap', or wants to extend Copilot with tools written in any language."
 argument-hint: "<what the provider should do>"
 user-invocable: true

package/dist/skills/{loop → tap-loop}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: loop
+name: tap-loop
 description: "Run a prompt on a recurring schedule or when idle. Use when the user says 'loop', 'every 5 minutes', 'check periodically', 'keep watching', 'repeat this', 'run when idle', or wants any prompt to re-run automatically."
 argument-hint: "<interval|idle> <prompt>"
 user-invocable: true
@@ -17,7 +17,7 @@ Interpret the invocation as:
 Example (timed):
 
 ```text
-/loop 5m check the deploy
+/tap-loop 5m check the deploy
 ```
 
 means:
@@ -28,7 +28,7 @@ means:
 Example (idle):
 
 ```text
-/loop idle check the deploy
+/tap-loop idle check the deploy
 ```
 
 means:

package/dist/skills/{monitor → tap-monitor}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: monitor
+name: tap-monitor
 description: "Start a self-tuning command monitor. Use when the user says 'monitor', 'watch', 'tail', 'track', 'keep an eye on', or wants a shell command to run continuously while Copilot automatically reviews and tunes the output filters over time."
 argument-hint: "<shell-command>"
 user-invocable: true
@@ -14,7 +14,7 @@ The entire invocation is the shell command to run continuously.
 Example:
 
 ```text
-/monitor tail -f /var/log/app.log
+/tap-monitor tail -f /var/log/app.log
 ```
 
 means:

package/dist/version.json

@@ -1,3 +1,3 @@
 {
-  "version": "1.1.4"
+  "version": "2.0.0"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "copilot-tap-extension",
-  "version": "1.1.4",
+  "version": "2.0.0",
   "description": "Copilot CLI extension for background event emitters, event streams, and session injection.",
   "type": "module",
   "license": "MIT",