Hide bundled skills from human skills list

The default skills list is meant for skills users can inspect and manage in the current environment. Bundled skills remain available internally and in JSON metadata, but the human table now omits bundled rows and removes the Source column.

Constraint: --json remains machine-readable with full source metadata.

Confidence: high

Scope-risk: narrow

Tested: bun test src/cli/handlers/skills.test.ts src/skills/loadSkillsDir.test.ts src/commands.test.ts

Tested: bun run build

Tested: node dist/cli.mjs skills list

Tested: node dist/cli.mjs skills list --json

Tested: git diff --check

Tested: bun run smoke

Author: anandh8x

src/cli/handlers/skills.test.ts

@@ -102,29 +102,73 @@ test('formats skills list as an aligned human table', () => {
       skill(
         'batch',
         'Research and plan a large-scale change, then execute it in parallel across 5–30 isolated worktree agents that each open a PR.',
+        'projectSettings',
+      ),
+      skill(
+        'debug',
+        'Enable debug logging for this session and help diagnose issues.',
+        'userSettings',
       ),
-      skill('debug', 'Enable debug logging for this session and help diagnose issues.'),
       skill(
         'loop',
         'Run a prompt on a fixed interval or dynamically reschedule it, including bare maintenance-mode loops.',
+        'projectSettings',
       ),
       skill(
         'simplify',
         'Review changed code for reuse, quality, and efficiency, then fix any issues found.',
+        'projectSettings',
       ),
       skill(
         'update-config',
         'Use this skill to configure the Claude Code harness via settings.json. Automated behaviors require hooks.',
+        'projectSettings',
       ),
     ],
     80,
   )
 
   assert.match(output, /^Skills: 5 enabled/)
-  assert.match(output, /Name\s+Status\s+Source\s+Description/)
+  assert.match(output, /Name\s+Status\s+Description/)
+  assert.doesNotMatch(output, /\bSource\b/)
   assert.doesNotMatch(output, /source: bundled \| trust:/)
-  assert.match(output, /batch\s+enabled\s+bundled\s+Research and plan/)
-  assert.match(output, /update-config\s+enabled\s+bundled\s+Configure the Claude Code harness via/)
+  assert.doesNotMatch(output, /\bbundled\b/)
+  assert.match(output, /batch\s+enabled\s+Research and plan/)
+  assert.match(output, /update-config\s+enabled\s+Configure the Claude Code harness via/)
+})
+
+test('omits source column while preserving installed rows', () => {
+  const output = formatSkillsListForDisplay(
+    [
+      skill('docs-writer', 'Writes project documentation.', 'projectSettings'),
+      skill('pr-review', 'Reviews pull requests.', 'userSettings'),
+      skill('debug', 'Enable debug logging.', 'bundled'),
+    ],
+    100,
+  )
+
+  assert.doesNotMatch(output, /\bSource\b/)
+  assert.doesNotMatch(output, /docs-writer\s+enabled\s+project\s+/)
+  assert.doesNotMatch(output, /pr-review\s+enabled\s+user\s+/)
+  assert.match(output, /docs-writer\s+enabled\s+Writes project documentation\./)
+  assert.match(output, /pr-review\s+enabled\s+Reviews pull requests\./)
+  assert.doesNotMatch(output, /\bdebug\b/)
+  assert.doesNotMatch(output, /Enable debug logging/)
+})
+
+test('omits bundled skills from the human table', () => {
+  const output = formatSkillsListForDisplay(
+    [
+      skill('debug', 'Enable debug logging.', 'bundled'),
+      skill('docs-writer', 'Writes project documentation.', 'projectSettings'),
+    ],
+    100,
+  )
+
+  assert.match(output, /^Skills: 1 enabled/)
+  assert.doesNotMatch(output, /\bdebug\b/)
+  assert.doesNotMatch(output, /Enable debug logging/)
+  assert.match(output, /docs-writer\s+enabled\s+Writes project documentation\./)
 })
 
 test('wraps description continuations under the Description column', () => {
@@ -133,16 +177,17 @@ test('wraps description continuations under the Description column', () => {
       skill(
         'batch',
         'Research and plan a large-scale change, then execute it in parallel across 5–30 isolated worktree agents that each open a PR.',
+        'projectSettings',
       ),
     ],
-    70,
+    45,
   )
   const lines = output.split('\n')
   const header = lines.find(line => line.includes('Description'))
   assert.ok(header)
   const descriptionColumn = header.indexOf('Description')
   const continuation = lines.find(line =>
-    line.trim().startsWith('then execute'),
+    line.trim().startsWith('large-scale change'),
   )
   assert.ok(continuation)
   assert.equal(continuation.search(/\S/), descriptionColumn)
@@ -155,6 +200,16 @@ test('formats empty skills list cleanly', () => {
   )
 })
 
+test('formats all-bundled skills as empty in the human table', () => {
+  assert.equal(
+    formatSkillsListForDisplay(
+      [skill('debug', 'Enable debug logging.', 'bundled')],
+      100,
+    ),
+    'Skills: 0 enabled\n\nNo skills found.',
+  )
+})
+
 test('formats skills list json as machine-readable metadata', () => {
   const description = 'Full description should remain in JSON. Extra sentence stays.'
   const parsed = JSON.parse(

src/cli/handlers/skillsListFormat.ts

@@ -100,21 +100,18 @@ function formatSkillListRow({
   state,
   nameWidth,
   statusWidth,
-  sourceWidth,
   descriptionWidth,
   descriptionIndent,
 }: {
   skill: SkillListCommand
   state: string | undefined
   nameWidth: number
   statusWidth: number
-  sourceWidth: number
   descriptionWidth: number
   descriptionIndent: string
 }): string {
   const status = state ?? 'enabled'
-  const source = sourceLabel(skill)
-  const prefix = `${getCommandName(skill).padEnd(nameWidth)}  ${status.padEnd(statusWidth)}   ${source.padEnd(sourceWidth)}   `
+  const prefix = `${getCommandName(skill).padEnd(nameWidth)}  ${status.padEnd(statusWidth)}   `
   const descriptionLines = wrapSkillDescription(
     descriptionSummary(skill.description),
     descriptionWidth,
@@ -166,11 +163,14 @@ export function formatSkillsListForDisplay(
 ): string {
   const sortedSkills = skills
     .slice()
+    .filter(skill => sourceLabel(skill) !== 'bundled')
     .sort((a, b) => getCommandName(a).localeCompare(getCommandName(b)))
   const states = getResolutionState(skills)
-  const enabledCount = [...states.values()].filter(s => s === 'enabled').length
+  const enabledCount = sortedSkills.filter(
+    skill => states.get(skill) === 'enabled',
+  ).length
 
-  if (skills.length === 0) {
+  if (sortedSkills.length === 0) {
     return ['Skills: 0 enabled', '', 'No skills found.'].join('\n')
   }
 
@@ -182,23 +182,17 @@ export function formatSkillsListForDisplay(
     'Status'.length,
     ...sortedSkills.map(skill => (states.get(skill) ?? 'enabled').length),
   )
-  const sourceWidth = Math.max(
-    'Source'.length,
-    ...sortedSkills.map(skill => sourceLabel(skill).length),
-  )
-  const descriptionStart =
-    nameWidth + 2 + statusWidth + 3 + sourceWidth + 3
+  const descriptionStart = nameWidth + 2 + statusWidth + 3
   const descriptionWidth = Math.max(20, columns - descriptionStart)
   const descriptionIndent = ' '.repeat(descriptionStart)
-  const header = `${'Name'.padEnd(nameWidth)}  ${'Status'.padEnd(statusWidth)}   ${'Source'.padEnd(sourceWidth)}   Description`
-  const rule = `${separator(nameWidth)}  ${separator(statusWidth)}   ${separator(sourceWidth)}   ${separator(descriptionWidth)}`
+  const header = `${'Name'.padEnd(nameWidth)}  ${'Status'.padEnd(statusWidth)}   Description`
+  const rule = `${separator(nameWidth)}  ${separator(statusWidth)}   ${separator(descriptionWidth)}`
   const rows = sortedSkills.map(skill =>
     formatSkillListRow({
       skill,
       state: states.get(skill),
       nameWidth,
       statusWidth,
-      sourceWidth,
       descriptionWidth,
       descriptionIndent,
     }),