@rfxlamia/skillkit 1.0.0 → 1.2.0

package/src/scope.js

@@ -1,57 +1,26 @@
+// installer/src/scope.js
 import { select, log } from '@clack/prompts'
-import { homedir } from 'os'
-import { join } from 'path'
-import { existsSync } from 'fs'
-
-export async function selectScope() {
-  const userSkillsPath = join(homedir(), '.claude', 'skills')
-  const projectSkillsPath = join(process.cwd(), '.claude', 'skills')
-  const projectExists = existsSync(join(process.cwd(), '.claude', 'skills'))
 
+export async function selectScope(selectedTools = []) {
   const scope = await select({
     message: 'Install to:',
     options: [
       {
         value: 'user',
-        label: `User scope  ~/.claude/skills/`,
+        label: 'User scope',
         hint: 'available in all projects'
       },
       {
         value: 'project',
-        label: `Project scope  ./.claude/skills/`,
-        hint: projectExists ? 'detected .claude/skills/ here' : `will create ${projectSkillsPath}`
+        label: 'Project scope',
+        hint: 'this project only'
       }
     ]
   })
 
-  if (scope === 'user') {
-    return {
-      scope: 'user',
-      skillsDir: userSkillsPath,
-      agentsDir: join(homedir(), '.claude', 'agents')
-    }
-  }
-
-  return {
-    scope: 'project',
-    skillsDir: projectSkillsPath,
-    agentsDir: join(process.cwd(), '.claude', 'agents')
-  }
-}
-
-// Export for testing
-export function getUserScope() {
-  return {
-    scope: 'user',
-    skillsDir: join(homedir(), '.claude', 'skills'),
-    agentsDir: join(homedir(), '.claude', 'agents')
+  if (selectedTools.includes('codex') && scope === 'project') {
+    log.warn('Codex does not support project scope — skills will be installed to ~/.agents/skills/skillkit/')
   }
-}
 
-export function getProjectScope() {
-  return {
-    scope: 'project',
-    skillsDir: join(process.cwd(), '.claude', 'skills'),
-    agentsDir: join(process.cwd(), '.claude', 'agents')
-  }
+  return scope
 }

package/src/scope.test.js

@@ -1,19 +1,15 @@
+// installer/src/scope.test.js
 import { test } from 'node:test'
 import assert from 'node:assert'
-import { homedir } from 'os'
-import { join } from 'path'
-import { getUserScope, getProjectScope } from './scope.js'
+import { selectScope } from './scope.js'
 
-test('getUserScope returns correct paths', () => {
-  const scope = getUserScope()
-  assert.strictEqual(scope.scope, 'user')
-  assert.strictEqual(scope.skillsDir, join(homedir(), '.claude', 'skills'))
-  assert.strictEqual(scope.agentsDir, join(homedir(), '.claude', 'agents'))
+test('selectScope is an async function', () => {
+  const AsyncFunction = (async () => {}).constructor
+  assert.ok(selectScope instanceof AsyncFunction)
 })
 
-test('getProjectScope returns correct paths', () => {
-  const scope = getProjectScope()
-  assert.strictEqual(scope.scope, 'project')
-  assert.strictEqual(scope.skillsDir, join(process.cwd(), '.claude', 'skills'))
-  assert.strictEqual(scope.agentsDir, join(process.cwd(), '.claude', 'agents'))
+test('getUserScope and getProjectScope are no longer exported', async () => {
+  const mod = await import('./scope.js')
+  assert.strictEqual(mod.getUserScope, undefined)
+  assert.strictEqual(mod.getProjectScope, undefined)
 })

package/src/tools.js

@@ -0,0 +1,76 @@
+// installer/src/tools.js
+import { multiselect } from '@clack/prompts'
+import { homedir } from 'os'
+import { join } from 'path'
+
+export async function selectTools() {
+  return multiselect({
+    message: 'Install to which tools?',
+    options: [
+      { value: 'claude-code', label: 'Claude Code', hint: '~/.claude/skills/' },
+      { value: 'opencode', label: 'OpenCode', hint: '~/.config/opencode/skills/' },
+      { value: 'codex', label: 'Codex', hint: '~/.agents/skills/' },
+      { value: 'copilot', label: 'GitHub Copilot', hint: '~/.copilot/skills/' }
+    ],
+    required: true
+  })
+}
+
+export function getToolTargets(selectedTools, scope) {
+  if (!selectedTools.length) return []
+
+  const home = homedir()
+  const cwd = process.cwd()
+  const isUser = scope === 'user'
+
+  const resolve = {
+    'claude-code': {
+      name: 'Claude Code',
+      scope,
+      skillsDir: isUser ? join(home, '.claude', 'skills') : join(cwd, '.claude', 'skills'),
+      agentsDir: isUser ? join(home, '.claude', 'agents') : join(cwd, '.claude', 'agents')
+    },
+    'opencode': {
+      name: 'OpenCode',
+      scope,
+      skillsDir: isUser
+        ? join(home, '.config', 'opencode', 'skills', 'skillkit')
+        : join(cwd, '.opencode', 'skills'),
+      agentsDir: isUser
+        ? join(home, '.config', 'opencode', 'agents')
+        : join(cwd, '.opencode', 'agents')
+    },
+    'codex': {
+      name: 'Codex',
+      scope: 'user',
+      skillsDir: join(home, '.agents', 'skills', 'skillkit'),
+      agentsDir: null
+    },
+    'copilot': {
+      name: 'GitHub Copilot',
+      scope: isUser ? 'user' : 'project',
+      skillsDir: join(home, '.copilot', 'skills'),
+      agentsDir: isUser
+        ? join(home, '.claude', 'agents')
+        : join(cwd, '.github', 'agents')
+    }
+  }
+
+  const targets = selectedTools
+    .filter(id => resolve[id])
+    .map(id => ({ ...resolve[id] }))
+
+  // Dedup: if skillsDir or agentsDir appears more than once, null it in later targets
+  const seenSkills = new Set()
+  const seenAgents = new Set()
+
+  for (const t of targets) {
+    if (seenSkills.has(t.skillsDir)) t.skillsDir = null
+    else seenSkills.add(t.skillsDir)
+
+    if (t.agentsDir && seenAgents.has(t.agentsDir)) t.agentsDir = null
+    else if (t.agentsDir) seenAgents.add(t.agentsDir)
+  }
+
+  return targets.filter(t => t.skillsDir || t.agentsDir)
+}

package/src/tools.test.js

@@ -0,0 +1,80 @@
+// installer/src/tools.test.js
+import { test } from 'node:test'
+import assert from 'node:assert'
+import { homedir } from 'os'
+import { join } from 'path'
+import { getToolTargets } from './tools.js'
+
+const home = homedir()
+const cwd = process.cwd()
+
+test('getToolTargets returns empty array for empty selection', () => {
+  const result = getToolTargets([], 'user')
+  assert.deepStrictEqual(result, [])
+})
+
+test('getToolTargets claude-code user scope', () => {
+  const [t] = getToolTargets(['claude-code'], 'user')
+  assert.strictEqual(t.name, 'Claude Code')
+  assert.strictEqual(t.scope, 'user')
+  assert.strictEqual(t.skillsDir, join(home, '.claude', 'skills'))
+  assert.strictEqual(t.agentsDir, join(home, '.claude', 'agents'))
+})
+
+test('getToolTargets claude-code project scope', () => {
+  const [t] = getToolTargets(['claude-code'], 'project')
+  assert.strictEqual(t.scope, 'project')
+  assert.strictEqual(t.skillsDir, join(cwd, '.claude', 'skills'))
+  assert.strictEqual(t.agentsDir, join(cwd, '.claude', 'agents'))
+})
+
+test('getToolTargets opencode user scope', () => {
+  const [t] = getToolTargets(['opencode'], 'user')
+  assert.strictEqual(t.name, 'OpenCode')
+  assert.strictEqual(t.skillsDir, join(home, '.config', 'opencode', 'skills', 'skillkit'))
+  assert.strictEqual(t.agentsDir, join(home, '.config', 'opencode', 'agents'))
+})
+
+test('getToolTargets opencode project scope', () => {
+  const [t] = getToolTargets(['opencode'], 'project')
+  assert.strictEqual(t.skillsDir, join(cwd, '.opencode', 'skills'))
+  assert.strictEqual(t.agentsDir, join(cwd, '.opencode', 'agents'))
+})
+
+test('getToolTargets codex user scope — agentsDir is null', () => {
+  const [t] = getToolTargets(['codex'], 'user')
+  assert.strictEqual(t.name, 'Codex')
+  assert.strictEqual(t.skillsDir, join(home, '.agents', 'skills', 'skillkit'))
+  assert.strictEqual(t.agentsDir, null)
+})
+
+test('getToolTargets codex project scope — still resolves to user path', () => {
+  const [t] = getToolTargets(['codex'], 'project')
+  assert.strictEqual(t.scope, 'user')
+  assert.strictEqual(t.skillsDir, join(home, '.agents', 'skills', 'skillkit'))
+  assert.strictEqual(t.agentsDir, null)
+})
+
+test('getToolTargets copilot user scope', () => {
+  const [t] = getToolTargets(['copilot'], 'user')
+  assert.strictEqual(t.name, 'GitHub Copilot')
+  assert.strictEqual(t.skillsDir, join(home, '.copilot', 'skills'))
+  assert.strictEqual(t.agentsDir, join(home, '.claude', 'agents'))
+})
+
+test('getToolTargets copilot project scope — skills still user path, agents project path', () => {
+  const [t] = getToolTargets(['copilot'], 'project')
+  assert.strictEqual(t.skillsDir, join(home, '.copilot', 'skills'))
+  assert.strictEqual(t.agentsDir, join(cwd, '.github', 'agents'))
+})
+
+test('getToolTargets deduplicates agentsDir for claude-code + copilot (user)', () => {
+  const targets = getToolTargets(['claude-code', 'copilot'], 'user')
+  assert.strictEqual(targets.length, 2)
+  const agentsDirs = targets.map(t => t.agentsDir).filter(Boolean)
+  const uniqueAgentsDirs = [...new Set(agentsDirs)]
+  assert.strictEqual(agentsDirs.length, uniqueAgentsDirs.length, 'no duplicate agentsDirs')
+  // Claude Code gets agentsDir, Copilot gets null (deduped)
+  assert.strictEqual(targets[0].agentsDir, join(home, '.claude', 'agents'))
+  assert.strictEqual(targets[1].agentsDir, null)
+})