@a5c-ai/babysitter-github 5.0.1-staging.ae07dd8d → 5.0.1-staging.aedcb38c

package/bin/install-shared.js

@@ -13,7 +13,7 @@ function getUserHome() {
 }
 
 function getHarnessHome() {
-  return path.join(os.homedir(), '.copilot');
+  return path.join(os.homedir(), ".copilot");
 }
 
 function getHomePluginRoot(scope) {
@@ -454,72 +454,9 @@ function installManagedHooks(packageRoot, copilotHome) {
   mergeManagedHooksConfig(packageRoot, copilotHome);
 }
 
-function filterManagedHookEntries(eventHooks) {
-  if (!Array.isArray(eventHooks)) {
-    return [];
-  }
-  const allScriptNames = [...LEGACY_HOOK_SCRIPT_NAMES, ...HOOK_SCRIPT_NAMES];
-
-  return eventHooks
-    .map((entry) => {
-      if (!entry || typeof entry !== 'object') {
-        return null;
-      }
-
-      if (Array.isArray(entry.hooks)) {
-        const keptHooks = entry.hooks.filter((hook) => {
-          const command = String(hook && hook.command || '');
-          return !allScriptNames.some((name) => command.includes(name));
-        });
-        return keptHooks.length > 0 ? { ...entry, hooks: keptHooks } : null;
-      }
-
-      const bash = String(entry.bash || entry.command || '');
-      const ps = String(entry.powershell || '');
-      return allScriptNames.some((name) => bash.includes(name) || ps.includes(name))
-        ? null
-        : entry;
-    })
-    .filter(Boolean);
-}
-
 function removeManagedHooks(copilotHome) {
-  for (const hookName of [...LEGACY_HOOK_SCRIPT_NAMES, ...HOOK_SCRIPT_NAMES]) {
-    fs.rmSync(path.join(copilotHome, 'hooks', hookName), { force: true });
-  }
-
-  const hooksConfigPath = path.join(copilotHome, 'hooks.json');
-  if (!fs.existsSync(hooksConfigPath)) {
-    return;
-  }
-  let hooksConfig;
-  try {
-    hooksConfig = readJson(hooksConfigPath);
-  } catch {
-    return;
-  }
-  if (!hooksConfig.hooks || typeof hooksConfig.hooks !== 'object') {
-    return;
-  }
-
-  for (const eventName of ['SessionStart', 'UserPromptSubmit', 'Stop', 'sessionStart', 'sessionEnd', 'userPromptSubmitted']) {
-    const filteredEntries = filterManagedHookEntries(hooksConfig.hooks[eventName]);
-    if (filteredEntries.length > 0) {
-      hooksConfig.hooks[eventName] = filteredEntries;
-    } else {
-      delete hooksConfig.hooks[eventName];
-    }
-  }
-
-  if (Object.keys(hooksConfig.hooks).length === 0) {
-    fs.rmSync(hooksConfigPath, { force: true });
-  } else {
-    writeJson(hooksConfigPath, hooksConfig);
-  }
-}
-
-function removeLegacyHooks(copilotHome) {
-  for (const hookName of LEGACY_HOOK_SCRIPT_NAMES) {
+  const managedHookNames = [...LEGACY_HOOK_SCRIPT_NAMES, ...HOOK_SCRIPT_NAMES];
+  for (const hookName of managedHookNames) {
     fs.rmSync(path.join(copilotHome, 'hooks', hookName), { force: true });
   }
 
@@ -540,10 +477,17 @@ function removeLegacyHooks(copilotHome) {
     const eventHooks = Array.isArray(hooksConfig.hooks[eventName]) ? hooksConfig.hooks[eventName] : [];
     const filteredMatchers = eventHooks
       .map((matcher) => {
+        const directBash = String(matcher?.bash || matcher?.command || '');
+        const directPs = String(matcher?.powershell || '');
+        const hasDirectHook = directBash.length > 0 || directPs.length > 0;
+        const directIsManaged = managedHookNames.some((name) => directBash.includes(name) || directPs.includes(name));
+        if (hasDirectHook) {
+          return directIsManaged ? null : matcher;
+        }
         const hooks = Array.isArray(matcher.hooks) ? matcher.hooks : [];
         const keptHooks = hooks.filter((hook) => {
           const command = String(hook.command || '');
-          return !LEGACY_HOOK_SCRIPT_NAMES.some((name) => command.includes(name));
+          return !managedHookNames.some((name) => command.includes(name));
         });
         return keptHooks.length > 0 ? { ...matcher, hooks: keptHooks } : null;
       })
@@ -567,6 +511,8 @@ function installCopilotSurface(packageRoot, copilotHome) {
   installManagedHooks(packageRoot, copilotHome);
 }
 
+const removeLegacyHooks = removeManagedHooks;
+
 function renderCloudAgentAgentsBlock() {
   return [
     '## Babysitter Cloud Agent',
@@ -895,10 +841,19 @@ module.exports = {
   resolveCliCommand,
   runCli,
   ensureGlobalProcessLibrary,
+  PLUGIN_BUNDLE_ENTRIES,
+  copyRecursive,
+  copyPluginBundle,
+  DEFAULT_MARKETPLACE,
+  normalizeMarketplaceSourcePath,
+  ensureMarketplaceEntry,
+  removeMarketplaceEntry,
+  installManagedSkills,
+  mergeManagedHooksConfig,
+  installManagedHooks,
+  warnWindowsHooks,
   LEGACY_HOOK_SCRIPT_NAMES,
   HOOK_SCRIPT_NAMES,
-  DEFAULT_MARKETPLACE,
-  PLUGIN_BUNDLE_ENTRIES,
   CLOUD_AGENT_BUNDLE_ENTRIES,
   MANAGED_BLOCK_START,
   MANAGED_BLOCK_END,
@@ -913,12 +868,8 @@ module.exports = {
   rewriteCloudSkill,
   registerCopilotPlugin,
   deregisterCopilotPlugin,
-  installManagedSkills,
-  mergeManagedHooksConfig,
-  installManagedHooks,
-  filterManagedHookEntries,
-  removeManagedHooks,
   removeLegacyHooks,
+  removeManagedHooks,
   installCopilotSurface,
   renderCloudAgentAgentsBlock,
   renderCloudAgentCopilotInstructionsBlock,
@@ -928,12 +879,6 @@ module.exports = {
   installCloudAgentInstructions,
   installCloudAgentSetupSteps,
   installCloudAgentSurface,
-  normalizeMarketplaceSourcePath,
-  ensureMarketplaceEntry,
-  removeMarketplaceEntry,
-  warnWindowsHooks,
-  copyPluginBundle,
-  copyRecursive,
   harnessCliRoute,
   harnessInstall,
 };

package/package.json

@@ -1,28 +1,26 @@
 {
   "name": "@a5c-ai/babysitter-github",
-  "version": "5.0.1-staging.ae07dd8d",
+  "version": "5.0.1-staging.aedcb38c",
   "description": "Orchestrate complex, multi-step workflows with event-sourced state management, hook-based extensibility, and human-in-the-loop approval",
   "scripts": {
     "deploy": "npm publish --access public",
     "deploy:staging": "npm publish --access public --tag staging",
-    "postinstall": "node bin/install.js",
-    "preuninstall": "node bin/uninstall.js",
-    "team:install": "node scripts/team-install.js",
-    "test": "node scripts/sync-command-surfaces.js --check",
-    "sync:commands": "node scripts/sync-command-surfaces.js"
+    "plugin:install": "node bin/install.js --global",
+    "plugin:uninstall": "node bin/uninstall.js --global",
+    "team:install": "node scripts/team-install.js"
   },
   "bin": {
-    "babysitter-github-copilot": "bin/cli.js"
+    "babysitter-github": "bin/cli.js"
   },
   "files": [
     "bin/",
     "hooks.json",
+    "AGENTS.md",
     "hooks/",
     "skills/",
     "commands/",
     "scripts/",
     "plugin.json",
-    "AGENTS.md",
     "README.md",
     "versions.json",
     "package.json"
@@ -38,11 +36,15 @@
     "access": "public"
   },
   "dependencies": {
-    "@a5c-ai/babysitter-sdk": "5.0.1-staging.ae07dd8d"
+    "@a5c-ai/babysitter-sdk": "5.0.1-staging.aedcb38c"
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/a5c-ai/babysitter"
+    "url": "git+https://github.com/a5c-ai/babysitter.git",
+    "directory": "plugins/babysitter-github"
   },
-  "homepage": "https://github.com/a5c-ai/babysitter/tree/main/plugins/babysitter-github#readme"
+  "homepage": "https://github.com/a5c-ai/babysitter/tree/main/plugins/babysitter-github#readme",
+  "bugs": {
+    "url": "https://github.com/a5c-ai/babysitter/issues"
+  }
 }

package/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "babysitter",
-  "version": "5.0.1-staging.ae07dd8d",
+  "version": "5.0.1-staging.aedcb38c",
   "description": "Orchestrate complex, multi-step workflows with event-sourced state management, hook-based extensibility, and human-in-the-loop approval",
   "author": {
     "name": "a5c.ai"

package/scripts/sync-command-surfaces.js

@@ -1,62 +1,20 @@
+#!/usr/bin/env node
 'use strict';
 
 const path = require('path');
-const {
-  listDirectories,
-  listMarkdownBasenames,
-  reportCheckResult,
-  syncCommandMirrors,
-  syncSkillsFromCommands,
-} = require('../../../scripts/plugin-command-sync-lib.cjs');
+const { spawnSync } = require('child_process');
 
-const PACKAGE_ROOT = path.resolve(__dirname, '..');
-const REPO_ROOT = path.resolve(PACKAGE_ROOT, '..', '..');
-const ROOT_COMMANDS = path.join(REPO_ROOT, 'plugins', 'babysitter', 'commands');
-const PLUGIN_COMMANDS = path.join(PACKAGE_ROOT, 'commands');
-const PLUGIN_SKILLS = path.join(PACKAGE_ROOT, 'skills');
-const LABEL = 'babysitter-github sync';
+const REPO_ROOT = path.resolve(__dirname, '..', '..');
+const syncScript = path.join(REPO_ROOT, 'scripts', 'sync-plugin-commands.cjs');
+const args = [syncScript, '--target', 'github-copilot'];
 
-function getMirroredCommandNames() {
-  const local = new Set(listMarkdownBasenames(PLUGIN_COMMANDS));
-  return listMarkdownBasenames(ROOT_COMMANDS).filter((name) => local.has(name));
+if (process.argv.includes('--check')) {
+  args.push('--check');
 }
 
-function getDerivedSkillNames() {
-  const local = new Set(listDirectories(PLUGIN_SKILLS));
-  return listMarkdownBasenames(PLUGIN_COMMANDS).filter((name) => local.has(name));
-}
-
-function main() {
-  const check = process.argv.includes('--check');
-  const mirrorResult = syncCommandMirrors({
-    label: LABEL,
-    sourceRoot: ROOT_COMMANDS,
-    targetRoot: PLUGIN_COMMANDS,
-    names: getMirroredCommandNames(),
-    check,
-    cwd: PACKAGE_ROOT,
-  });
-  const skillsResult = syncSkillsFromCommands({
-    label: LABEL,
-    sourceRoot: PLUGIN_COMMANDS,
-    skillsRoot: PLUGIN_SKILLS,
-    names: getDerivedSkillNames(),
-    check,
-    cwd: PACKAGE_ROOT,
-  });
-
-  if (check) {
-    reportCheckResult(LABEL, [...mirrorResult.stale, ...skillsResult.stale]);
-    return;
-  }
-
-  const updated = mirrorResult.updated + skillsResult.updated;
-  if (updated === 0) {
-    console.log(`[${LABEL}] no GitHub plugin command changes were needed.`);
-    return;
-  }
-
-  console.log(`[${LABEL}] updated ${updated} GitHub plugin file(s).`);
-}
+const result = spawnSync(process.execPath, args, {
+  cwd: REPO_ROOT,
+  stdio: 'inherit',
+});
 
-main();
+process.exit(result.status ?? 1);

package/versions.json

@@ -1,3 +1,4 @@
 {
-  "sdkVersion": "5.0.1-staging.ae07dd8d"
+  "sdkVersion": "5.0.1-staging.aedcb38c",
+  "extensionVersion": "5.0.0"
 }