@adityaaria/spark 6.0.10 → 6.0.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adityaaria/spark",
-  "version": "6.0.10",
+  "version": "6.0.11",
   "description": "SPARK skills and runtime bootstrap for coding agents",
   "type": "module",
   "publishConfig": {

package/src/cli/install.js

@@ -57,32 +57,6 @@ export async function runInstall(options, env) {
 
   const installResult = await adapter.install({ options, env });
 
-  // Handle global copy result
-  if (installResult?.globalCopy) {
-    printSection('Install');
-    printLine(labelValue('Harness', adapter.label));
-
-    if (installResult.cliSuccess) {
-      // Both global copy AND CLI succeeded — full install
-      printLine(statusText(`Installed SPARK for ${adapter.label}.`, 'success'));
-    } else {
-      // Global copy succeeded, CLI was skipped — still fully functional
-      printLine(statusText(`SPARK skills installed to ${installResult.globalSkillsPath}`, 'success'));
-    }
-
-    const readyLines = [
-      bullet(`SPARK skills and hooks copied to ${installResult.globalSkillsPath}.`),
-      bullet(`Open a fresh ${adapter.label} session to confirm using-spark loads before coding.`),
-    ];
-
-    if (installResult.cliSkipped) {
-      readyLines.push(bullet(`CLI integration skipped (optional — SPARK already works without it).`));
-    }
-
-    printSummary('Ready', readyLines, 'success');
-    return;
-  }
-
   if (typeof adapter.verify === 'function') {
     await adapter.verify({ options, env });
   }

package/src/installer/adapters/common.js

@@ -1,6 +1,5 @@
 import { spawnSync } from 'node:child_process';
 import { InstallerError } from '../errors.js';
-import { copyToGlobal } from './global-skills-copy.js';
 
 export function createAdapter({
   id,
@@ -18,7 +17,6 @@ export function createAdapter({
   envKeys = [],
   binaryNames = [],
   configPaths = [],
-  packageRoot = null,
 }) {
   const commandList = normalizeCommandList(commands, command);
   const automated = commandList.length > 0 || typeof customInstall === 'function';
@@ -61,17 +59,6 @@ export function createAdapter({
         return { plan: this.planInstall(), metadata };
       }
 
-      // Step 1: ALWAYS copy skills + hooks to global directory (primary mechanism)
-      let globalCopyResult = { copied: false };
-      if (packageRoot) {
-        try {
-          globalCopyResult = copyToGlobal(id, packageRoot, env);
-        } catch { /* global copy failed — will try CLI next */ }
-      }
-
-      // Step 2: Try CLI commands (optional bonus — for marketplace integration)
-      let cliSuccess = false;
-      let cliSkipped = false;
       for (const entry of commandList) {
         const result = runner(entry.file, interpolateArgs(entry.args ?? [], metadata), {
           cwd: entry.cwd ?? cwd,
@@ -80,47 +67,14 @@ export function createAdapter({
         });
 
         if (result.error) {
-          if (result.error.code === 'ENOENT') {
-            // CLI not found — not a problem if global copy succeeded
-            cliSkipped = true;
-            break;
-          }
-          // Other errors — also non-fatal if global copy succeeded
-          if (globalCopyResult.copied) {
-            cliSkipped = true;
-            break;
-          }
           throw new InstallerError(`${label} install failed: ${result.error.message}`);
         }
 
         if (result.status !== 0) {
-          if (globalCopyResult.copied) {
-            cliSkipped = true;
-            break;
-          }
           throw new InstallerError(
             `${label} install failed: ${result.stderr || result.stdout || 'unknown error'}`
           );
         }
-
-        cliSuccess = true;
-      }
-
-      // Determine result type
-      if (globalCopyResult.copied) {
-        return {
-          plan: this.planInstall(),
-          metadata,
-          globalCopy: true,
-          globalSkillsPath: globalCopyResult.globalSkillsPath,
-          cliSuccess,
-          cliSkipped,
-        };
-      }
-
-      // Neither global copy nor CLI worked — nothing we can do
-      if (!cliSuccess && commandList.length > 0) {
-        throw new InstallerError(`${label} install failed: unable to copy skills or run CLI.`);
       }
 
       return { plan: this.planInstall(), metadata };

package/src/installer/adapters/shell-hook.js

@@ -13,7 +13,6 @@ export function createClaudeCodeAdapter() {
     id: 'claude',
     label: 'Claude Code',
     kind: 'shell-hook',
-    packageRoot,
     envKeys: ['CLAUDE_PLUGIN_ROOT'],
     binaryNames: ['claude'],
     bootstrap: 'shell hook -> hooks/session-start -> using-spark',
@@ -46,7 +45,6 @@ export function createCodexAdapter() {
     id: 'codex',
     label: 'Codex CLI',
     kind: 'shell-hook',
-    packageRoot,
     envKeys: ['CLAUDE_PLUGIN_ROOT'],
     binaryNames: ['codex'],
     bootstrap: 'shell hook -> hooks/session-start-codex -> using-spark',
@@ -121,7 +119,6 @@ export function createCopilotAdapter() {
     id: 'copilot',
     label: 'Copilot CLI',
     kind: 'shell-hook',
-    packageRoot,
     envKeys: ['COPILOT_CLI'],
     binaryNames: ['copilot'],
     bootstrap: 'shell hook -> hooks/session-start -> using-spark',

package/src/installer/adapters/global-skills-copy.js

@@ -1,77 +0,0 @@
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-
-/**
- * Mapping of harness ID to its global directory structure.
- * Each entry defines where skills and hooks should be copied
- * so the harness discovers SPARK at session start.
- */
-const GLOBAL_TARGETS = {
-  codex: {
-    skillsDir: path.join('.codex', 'skills'),
-    hooksDir: path.join('.codex', 'hooks'),
-    hookFiles: [
-      { src: 'hooks/hooks-codex.json', dest: 'hooks.json' },
-      { src: 'hooks/session-start-codex', dest: 'session-start-codex' },
-      { src: 'hooks/run-hook.cmd', dest: 'run-hook.cmd' },
-    ],
-    pluginDir: path.join('.codex', '.codex-plugin'),
-    pluginSrc: '.codex-plugin',
-  },
-  claude: {
-    skillsDir: path.join('.claude', 'skills'),
-    hooksDir: path.join('.claude', 'hooks'),
-    hookFiles: [
-      { src: 'hooks/hooks.json', dest: 'hooks.json' },
-      { src: 'hooks/session-start', dest: 'session-start' },
-      { src: 'hooks/run-hook.cmd', dest: 'run-hook.cmd' },
-    ],
-    pluginDir: path.join('.claude', '.claude-plugin'),
-    pluginSrc: '.claude-plugin',
-  },
-};
-
-/**
- * Copy SPARK skills and hooks directly to a harness's global directory.
- * This is a fallback mechanism used when the harness CLI is not available.
- *
- * @param {string} id - The harness identifier (e.g. 'codex', 'claude')
- * @param {string} packageRoot - Absolute path to the SPARK package root
- * @param {object} env - Process environment (to resolve HOME)
- * @returns {{ globalSkillsPath: string, globalHooksPath: string, copied: boolean }}
- */
-export function copyToGlobal(id, packageRoot, env = process.env) {
-  const target = GLOBAL_TARGETS[id];
-  if (!target) {
-    return { globalSkillsPath: null, globalHooksPath: null, copied: false };
-  }
-
-  const homeDir = env.HOME || env.USERPROFILE || os.homedir();
-  const globalSkillsPath = path.join(homeDir, target.skillsDir);
-  const globalHooksPath = path.join(homeDir, target.hooksDir);
-
-  // 1. Copy skills
-  const sourceSkills = path.join(packageRoot, 'skills');
-  fs.cpSync(sourceSkills, globalSkillsPath, { recursive: true });
-
-  // 2. Copy hooks
-  fs.mkdirSync(globalHooksPath, { recursive: true });
-  for (const hookFile of target.hookFiles) {
-    const src = path.join(packageRoot, hookFile.src);
-    const dest = path.join(globalHooksPath, hookFile.dest);
-    fs.copyFileSync(src, dest);
-    // Preserve executable permission for shell scripts
-    const stat = fs.statSync(src);
-    fs.chmodSync(dest, stat.mode);
-  }
-
-  // 3. Copy plugin descriptor (e.g. .codex-plugin/)
-  if (target.pluginDir && target.pluginSrc) {
-    const srcPlugin = path.join(packageRoot, target.pluginSrc);
-    const destPlugin = path.join(homeDir, target.pluginDir);
-    fs.cpSync(srcPlugin, destPlugin, { recursive: true });
-  }
-
-  return { globalSkillsPath, globalHooksPath, copied: true };
-}