claude-agent-skills 1.3.1 → 1.3.3

package/lib/prompts.js

@@ -1,5 +1,6 @@
 import { select, multiselect, confirm, isCancel } from '@clack/prompts';
 import ansis from 'ansis';
+import { skillColor, white, muted, divider } from './theme.js';
 
 export class CliCancel extends Error {}
 
@@ -26,11 +27,23 @@ export async function pickScope(flags = {}) {
 export async function pickSkills(names, flags = {}, descriptions = {}) {
   if (flags.all) return names;
   if (flags.skill?.length) return flags.skill;
+
+  // Calculate description width from actual terminal columns.
+  // prefix=4 (clack's "□ "), nameCol=32, divider=3 (" │ ")
+  const termWidth = process.stdout.columns || 100;
+  const descWidth = Math.max(20, termWidth - 4 - 32 - 3);
+
+  function fitDesc(s) {
+    if (!s) return '';
+    return s.length <= descWidth ? s : s.slice(0, descWidth - 1) + '…';
+  }
+
   return guard(await multiselect({
     message: 'Select skills  (space to toggle, a for all, enter to confirm)',
-    options: names.map(n => {
-      const desc = descriptions[n] ? ansis.dim('  ' + descriptions[n].slice(0, 50)) : '';
-      return { value: n, label: n.padEnd(34) + desc };
+    options: names.map((n, i) => {
+      const name = skillColor(i)(ansis.bold(n.padEnd(32)));
+      const desc = descriptions[n] ? white(fitDesc(descriptions[n])) : '';
+      return { value: n, label: name + divider + desc };
     }),
     required: true,
   }));

package/lib/summary.js

@@ -1,20 +1,25 @@
-import { brand, muted, success, warn } from './theme.js';
+import { brand, muted, success, warn, white, skillColor, divider } from './theme.js';
 import { VERSION } from './constants.js';
 
-const PACK = `claude-skills v${VERSION}`;
+const PACK = `claude-agent-skills v${VERSION}`;
 
 function trunc(s, n) { return s.length <= n ? s : `${s.slice(0, n - 1)}…`; }
 
-// Pad before colouring so escape codes don't break column widths
-function colorStatus(label, healthy) {
+function colorStatus(label) {
   const padded = label.padEnd(10);
-  if (label === 'ok' || label === 'synced')        return success(padded);
-  if (label === 'installed' || label === 'updated') return brand(padded);
-  if (label === 'update')                           return brand(padded);
-  if (label === 'skipped')                          return muted(padded);
+  if (label === 'ok' || label === 'synced')         return success(padded);
+  if (label === 'installed' || label === 'updated')  return brand(padded);
+  if (label === 'update')                            return brand(padded);
+  if (label === 'skipped')                           return muted(padded);
   return warn(padded); // broken / missing / modified / removed / failed
 }
 
+// Pride-colored name with divider + white detail text
+function skillRow(index, name, detail = '') {
+  const colored = skillColor(index)(trunc(name, 30).padEnd(30));
+  return detail ? `  ${colored}${divider}${white(detail)}` : `  ${colored}`;
+}
+
 function header({ scope, destination, lockPath }) {
   const lines = [
     `${brand('Pack')}:        ${PACK}`,
@@ -26,11 +31,11 @@ function header({ scope, destination, lockPath }) {
 }
 
 export function renderInstallSummary({ scope, skillsDir, lockPath, installed, updated, skipped }) {
-  // Each entry is { name, dependencyOf? } or a plain string
+  let idx = 0;
   const row = (entry, label) => {
     const name = typeof entry === 'string' ? entry : entry.name;
-    const dep = typeof entry === 'object' && entry.dependencyOf ? muted(` (dependency of ${entry.dependencyOf})`) : '';
-    return `  ${colorStatus(label, label !== 'skipped')} ${name}${dep}`;
+    const dep = typeof entry === 'object' && entry.dependencyOf ? `dep of ${entry.dependencyOf}` : '';
+    return `  ${colorStatus(label)} ${skillColor(idx++)(trunc(name, 28).padEnd(28))}${dep ? divider + muted(dep) : ''}`;
   };
   const rows = [
     ...installed.map(e => row(e, 'installed')),
@@ -48,38 +53,41 @@ export function renderInstallSummary({ scope, skillsDir, lockPath, installed, up
 }
 
 export function renderSyncSummary({ scope, skillsDir, lockPath, synced, ok: upToDate }) {
+  let idx = 0;
   const rows = [
-    ...synced.map(n =>    `  ${colorStatus('synced', true)} ${trunc(n, 30).padEnd(30)} ${muted('—')}`),
-    ...upToDate.map(n =>  `  ${colorStatus('ok', true)}     ${trunc(n, 30).padEnd(30)} ${muted('—')}`),
+    ...synced.map(n =>   `  ${colorStatus('synced')} ${skillRow(idx++, n)}`),
+    ...upToDate.map(n => `  ${colorStatus('ok')}     ${skillRow(idx++, n)}`),
   ];
   return [
     ...header({ scope, destination: skillsDir, lockPath }),
     '',
     `Synced: ${synced.length}  Up to date: ${upToDate.length}`,
     '',
-    `${'STATUS'.padEnd(12)} ${'NAME'.padEnd(30)} DEPS`,
+    `${'STATUS'.padEnd(12)} NAME`,
     ...rows,
   ].join('\n');
 }
 
 export function renderListSummary({ scope, skillsDir, lockPath, rows }) {
-  const tableRows = rows.map(r => {
+  const tableRows = rows.map((r, i) => {
     const label = r.healthy ? 'ok' : r.status;
-    return `  ${colorStatus(label, r.healthy)} ${trunc(r.name, 30).padEnd(30)} ${r.linkType.padEnd(8)} ${r.hash}`;
+    const name = skillColor(i)(trunc(r.name, 28).padEnd(28));
+    return `  ${colorStatus(label)} ${name}${divider}${white(r.linkType.padEnd(8))}${muted(r.hash)}`;
   });
   return [
     ...header({ scope, destination: skillsDir, lockPath }),
     '',
-    `${'STATUS'.padEnd(12)} ${'NAME'.padEnd(30)} ${'LINK'.padEnd(8)} HASH`,
+    `${'STATUS'.padEnd(12)} ${'NAME'.padEnd(30)} ${'LINK'.padEnd(11)} HASH`,
     ...tableRows,
   ].join('\n');
 }
 
 export function renderCheckSummary({ scope, skillsDir, lockPath, rows, counts }) {
   const legend = { ok: 0, update: 0, modified: 0, missing: 0, broken: 0, ...counts };
-  const tableRows = rows.map(r =>
-    `  ${colorStatus(r.status, r.status === 'ok')} ${trunc(r.name, 30).padEnd(30)} ${r.linkType}`
-  );
+  const tableRows = rows.map((r, i) => {
+    const name = skillColor(i)(trunc(r.name, 28).padEnd(28));
+    return `  ${colorStatus(r.status)} ${name}${divider}${white(r.linkType)}`;
+  });
   return [
     ...header({ scope, destination: skillsDir, lockPath }),
     '',
@@ -94,7 +102,7 @@ export function renderUpdateSummary({ scope, skillsDir, lockPath, updated }) {
   return [
     ...header({ scope, destination: skillsDir, lockPath }),
     '',
-    ...updated.map(n => `  ${colorStatus('updated', true)} ${n}`),
+    ...updated.map((n, i) => `  ${colorStatus('updated')} ${skillColor(i)(n)}`),
   ].join('\n');
 }
 
@@ -102,7 +110,7 @@ export function renderRemoveSummary({ scope, skillsDir, lockPath, removed, faile
   return [
     ...header({ scope, destination: skillsDir, lockPath }),
     '',
-    ...removed.map(n => `  ${colorStatus('removed', false)} ${n}`),
-    ...failed.map(f => `  ${colorStatus('failed', false)} ${f.name} — ${f.error}`),
+    ...removed.map((n, i) => `  ${colorStatus('removed')} ${skillColor(i)(n)}`),
+    ...failed.map((f, i) => `  ${colorStatus('failed')}  ${skillColor(i)(f.name)}${divider}${warn(f.error)}`),
   ].join('\n');
 }

package/lib/theme.js

@@ -8,4 +8,29 @@ export const brand   = t => ok() ? ansis.hex(BRAND_HEX)(t) : t;
 export const success = t => ok() ? ansis.green(t) : t;
 export const warn    = t => ok() ? ansis.yellow(t) : t;
 export const muted   = t => ok() ? ansis.dim(t) : t;
+export const white   = t => ok() ? ansis.white(t) : t;
 export const strip   = t => ansis.strip(t);
+
+// ── Pride mode ─────────────────────────────────────────────────────────────
+// Set to false after June to revert skill names to brand pink.
+export const PRIDE_MODE = true;
+
+const PRIDE_PALETTE = [
+  [220,  30,  30],   // red
+  [255, 140,   0],   // orange
+  [220, 200,   0],   // yellow
+  [ 30, 185,  30],   // green
+  [ 30, 100, 255],   // blue
+  [150,  30, 230],   // purple
+];
+
+// Returns a color function for the nth skill. Cycles through pride palette in
+// PRIDE_MODE, falls back to brand pink otherwise.
+export function skillColor(index) {
+  if (!ok()) return t => t;
+  if (!PRIDE_MODE) return brand;
+  const [r, g, b] = PRIDE_PALETTE[index % PRIDE_PALETTE.length];
+  return t => ansis.rgb(r, g, b)(t);
+}
+
+export const divider = ok() ? ansis.dim(' │ ') : ' | ';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-agent-skills",
-  "version": "1.3.1",
+  "version": "1.3.3",
   "description": "Install and manage Pavi's Claude Code agent skills",
   "license": "MIT",
   "type": "module",

package/skills.json

@@ -1,7 +1,7 @@
 {
   "schema_version": 1,
   "name": "claude-agent-skills",
-  "version": "1.3.1",
+  "version": "1.3.3",
   "skills": [
     "ask-matt",
     "brainstorming",