fix: prefer styleText for ansi

Author: harlan-zw

src/agent/clis/cli-progress.ts

@@ -1,4 +1,5 @@
 import type { StreamProgress } from './types.ts'
+import { styleText } from 'node:util'
 import { TOOL_NAMES } from './types.ts'
 
 const STATIC_REGEX_1 = /^\[(?:starting|retrying|cached)/
@@ -20,7 +21,7 @@ export function createToolProgress(log: ToolProgressLog): (progress: StreamProgr
   function emit(msg: string) {
     if (msg === lastMsg) {
       repeatCount++
-      log.message(`${msg} \x1B[90m(+${repeatCount})\x1B[0m`)
+      log.message(`${msg} ${styleText('gray', `(+${repeatCount})`)}`)
     }
     else {
       lastMsg = msg
@@ -37,18 +38,18 @@ export function createToolProgress(log: ToolProgressLog): (progress: StreamProgr
       if (now - last < TEXT_THROTTLE_MS)
         return
       lastTextEmit.set(key, now)
-      const prefix = section ? `\x1B[90m[${section}]\x1B[0m ` : ''
+      const prefix = section ? `${styleText('gray', `[${section}]`)} ` : ''
       // Count bullet items in accumulated text for meaningful progress
       const items = text ? (text.match(/^- (?:BREAKING|DEPRECATED|NEW|CHANGED|REMOVED|Use |Do |Set |Add |Avoid |Always |Never |Prefer |Check |Ensure )/gm)?.length ?? 0) : 0
-      emit(items > 0 ? `${prefix}Writing... \x1B[90m(${items} items)\x1B[0m` : `${prefix}Writing...`)
+      emit(items > 0 ? `${prefix}Writing... ${styleText('gray', `(${items} items)`)}` : `${prefix}Writing...`)
       return
     }
     if (type !== 'reasoning' || !chunk.startsWith('['))
       return
 
     // Handle status messages like [starting...], [retrying...], [cached]
     if (STATIC_REGEX_1.test(chunk)) {
-      const prefix = section ? `\x1B[90m[${section}]\x1B[0m ` : ''
+      const prefix = section ? `${styleText('gray', `[${section}]`)} ` : ''
       emit(`${prefix}${chunk.slice(1, -1)}`)
       return
     }
@@ -65,12 +66,12 @@ export function createToolProgress(log: ToolProgressLog): (progress: StreamProgr
       const rawName = names[i]!
       const hint = hints[i] ?? hints[0] ?? ''
       const verb = TOOL_NAMES[rawName]?.verb ?? rawName
-      const prefix = section ? `\x1B[90m[${section}]\x1B[0m ` : ''
+      const prefix = section ? `${styleText('gray', `[${section}]`)} ` : ''
 
       if ((rawName === 'Bash' || rawName === 'run_shell_command') && hint) {
         const searchMatch = hint.match(STATIC_REGEX_3)
         if (searchMatch) {
-          emit(`${prefix}Searching \x1B[36m"${searchMatch[1]}"\x1B[0m`)
+          emit(`${prefix}Searching ${styleText('cyan', `"${searchMatch[1]}"`)}`)
         }
         else if (hint.includes('skilld validate')) {
           emit(`${prefix}Validating...`)
@@ -82,7 +83,7 @@ export function createToolProgress(log: ToolProgressLog): (progress: StreamProgr
       }
       else {
         const path = shortenPath(hint || '...')
-        emit(`${prefix}${verb} \x1B[90m${path}\x1B[0m`)
+        emit(`${prefix}${verb} ${styleText('gray', path)}`)
       }
     }
   }

src/agent/skill-builder.ts

@@ -14,6 +14,7 @@
 import type { FeaturesConfig } from '../core/config.ts'
 import type { CustomPrompt, OptimizeModel, OptimizeResult, SkillSection, StreamProgress } from './index.ts'
 import { mkdirSync, writeFileSync } from 'node:fs'
+import { styleText } from 'node:util'
 import * as p from '@clack/prompts'
 import { join, relative } from 'pathe'
 import { createReferenceCache, listReferenceFiles } from '../cache/index.ts'
@@ -176,10 +177,10 @@ export async function enhanceSkillWithLLM(ctx: SkillContext, run: EnhanceRunOpti
     if (debugLogsDir)
       p.log.info(`Debug logs: ${relative(process.cwd(), debugLogsDir)}`)
     if (error)
-      p.log.warn(`\x1B[33mPartial failure: ${error}\x1B[0m`)
+      p.log.warn(styleText('yellow', `Partial failure: ${error}`))
     if (warnings?.length) {
       for (const w of warnings)
-        p.log.warn(`\x1B[33m${w}\x1B[0m`)
+        p.log.warn(styleText('yellow', w))
     }
   }
   else {
@@ -294,7 +295,7 @@ export function writePromptFiles(ctx: SkillContext, run: PromptRunOptions): Skil
     const relDir = relative(process.cwd(), skillDir)
     const promptFiles = written.map(s => `PROMPT_${s}.md`).join(', ')
     const outputFileList = written.map(s => SECTION_OUTPUT_FILES[s]).join(', ')
-    p.log.info(`Prompt files written to ${relDir}/.skilld/\n\x1B[2m\x1B[3m  Read each prompt file (${promptFiles}) in ${relDir}/.skilld/, read the\n  referenced files, then write your output to the matching file (${outputFileList}).\n  When done, run: skilld assemble\x1B[0m`)
+    p.log.info(`Prompt files written to ${relDir}/.skilld/\n${styleText(['dim', 'italic'], `  Read each prompt file (${promptFiles}) in ${relDir}/.skilld/, read the\n  referenced files, then write your output to the matching file (${outputFileList}).\n  When done, run: skilld assemble`)}`)
   }
 
   return written

src/agent/skill-installer.ts

@@ -15,6 +15,7 @@
 import type { SkillInfo } from '../core/lockfile.ts'
 import type { AgentType } from './index.ts'
 import { appendFileSync, existsSync, mkdirSync, readFileSync, rmSync, writeFileSync } from 'node:fs'
+import { styleText } from 'node:util'
 import * as p from '@clack/prompts'
 import { join, relative } from 'pathe'
 import { isInteractive } from '../cli/env.ts'
@@ -212,10 +213,10 @@ export async function ensureGitignore(skillsDir: string, cwd: string, isGlobal:
 
   const relSkillsDir = relative(cwd, skillsDir) || '.'
   p.log.info(
-    `\x1B[1mGit guidance:\x1B[0m\n`
-    + `  \x1B[32m✓\x1B[0m Commit: \x1B[36m${relSkillsDir}/*/SKILL.md\x1B[0m\n`
-    + `  \x1B[32m✓\x1B[0m Commit: \x1B[36m${relSkillsDir}/skilld-lock.yaml\x1B[0m\n`
-    + `  \x1B[31m✗\x1B[0m Ignore: \x1B[36m${pattern}\x1B[0m \x1B[90m(recreated by \`skilld install\`)\x1B[0m`,
+    `${styleText('bold', 'Git guidance:')}\n`
+    + `  ${styleText('green', '✓')} Commit: ${styleText('cyan', `${relSkillsDir}/*/SKILL.md`)}\n`
+    + `  ${styleText('green', '✓')} Commit: ${styleText('cyan', `${relSkillsDir}/skilld-lock.yaml`)}\n`
+    + `  ${styleText('red', '✗')} Ignore: ${styleText('cyan', pattern)} ${styleText('gray', '(recreated by `skilld install`)')}`,
   )
 
   const add = await p.confirm({
@@ -285,8 +286,8 @@ export async function ensureAgentInstructions(agent: AgentType, cwd: string, isG
       `This tells your agent to check installed skills before making\n`
       + `code changes. Without it, skills are available but may not\n`
       + `activate automatically.\n`
-      + `\n`
-      + `\x1B[90m${getMdcSkillInstructions(agent)}\x1B[0m`,
+      + `\n${
+        styleText('gray', getMdcSkillInstructions(agent))}`,
       `Create ${agentConfig.instructionFile}`,
     )
 
@@ -328,8 +329,8 @@ export async function ensureAgentInstructions(agent: AgentType, cwd: string, isG
     `This tells your agent to check installed skills before making\n`
     + `code changes. Without it, skills are available but may not\n`
     + `activate automatically.\n`
-    + `\n`
-    + `\x1B[90m${getSkillInstructions(agent).replace(/\n/g, '\n')}\x1B[0m`,
+    + `\n${
+      styleText('gray', getSkillInstructions(agent).replace(/\n/g, '\n'))}`,
     `${action} ${agentConfig.instructionFile}`,
   )
 

src/cli.ts

@@ -2,6 +2,7 @@
 import type { PackageUsage } from './agent/detect-imports.ts'
 import type { AgentType } from './agent/index.ts'
 import { existsSync, readFileSync, realpathSync } from 'node:fs'
+import { styleText } from 'node:util'
 import * as p from '@clack/prompts'
 import { defineCommand, runMain } from 'citty'
 import pLimit from 'p-limit'
@@ -49,7 +50,7 @@ function deprecatedForwarder(
       ...cmd,
       meta: { ...cmd.meta, name: oldName },
       async run(ctx: any) {
-        console.warn(`\x1B[33m⚠ \`skilld ${oldName}\` is deprecated. Use \`skilld ${newName}\` instead.\x1B[0m`)
+        console.warn(styleText('yellow', `⚠ \`skilld ${oldName}\` is deprecated. Use \`skilld ${newName}\` instead.`))
         return original(ctx)
       },
     })
@@ -128,8 +129,8 @@ const main = defineCommand({
       }
       p.log.info(
         'No agent selected - skills export as portable PROMPT_*.md files.\n'
-        + '  Run \x1B[36mskilld add <pkg>\x1B[0m to generate prompts for any package.\n'
-        + '  Run \x1B[36mskilld config\x1B[0m to set a target agent later.',
+        + `  Run ${styleText('cyan', 'skilld add <pkg>')} to generate prompts for any package.\n`
+        + `  Run ${styleText('cyan', 'skilld config')} to set a target agent later.`,
       )
       return
     }
@@ -181,14 +182,14 @@ const main = defineCommand({
 
     // Show self-update notification
     if (selfUpdate) {
-      const released = selfUpdate.releasedAt ? `\x1B[90m · ${relativeTime(new Date(selfUpdate.releasedAt))}\x1B[0m` : ''
+      const released = selfUpdate.releasedAt ? styleText('gray', ` · ${relativeTime(new Date(selfUpdate.releasedAt))}`) : ''
       const binPath = realpathSync(process.argv[1]!)
       const isLocal = binPath.startsWith(resolve(cwd, 'node_modules'))
       const flag = isLocal ? '' : ' -g'
       const cmd = `npx nypm add${flag} skilld@${selfUpdate.latest}`
       p.note(
-        `\x1B[90m${version}\x1B[0m → \x1B[1m\x1B[32m${selfUpdate.latest}\x1B[0m${released}\n\x1B[36m${cmd}\x1B[0m`,
-        '\x1B[33mUpdate available\x1B[0m',
+        `${styleText('gray', version)} → ${styleText(['bold', 'green'], selfUpdate.latest)}${released}\n${styleText('cyan', cmd)}`,
+        styleText('yellow', 'Update available'),
       )
     }
 
@@ -208,7 +209,7 @@ const main = defineCommand({
       const hasPkgJson = !!projectPkg
       const projectName = projectPkg?.parsed.name as string | undefined
       const projectLabel = projectName
-        ? `Generating skills for \x1B[36m${projectName}\x1B[0m`
+        ? `Generating skills for ${styleText('cyan', projectName)}`
         : 'Generating skills for current directory'
       p.log.step(projectLabel)
 
@@ -219,7 +220,7 @@ const main = defineCommand({
       if (state.shipped.length > 0) {
         const totalShipped = state.shipped.reduce((sum, s) => sum + s.skills.length, 0)
         const names = state.shipped.map(s => s.packageName).join(', ')
-        p.log.info(`\x1B[36m${totalShipped} ready-to-use skill${totalShipped > 1 ? 's' : ''}\x1B[0m shipped by your dependencies: ${names}`)
+        p.log.info(`${styleText('cyan', `${totalShipped} ready-to-use skill${totalShipped > 1 ? 's' : ''}`)} shipped by your dependencies: ${names}`)
       }
 
       p.log.info('Tip: Add skills for packages with complex APIs or frequent breaking changes - not every dependency needs one.')
@@ -228,7 +229,7 @@ const main = defineCommand({
       let setupComplete = false
       while (!setupComplete) {
         const shippedOption = state.shipped.length > 0
-          ? [{ label: 'Install shipped skills', value: 'shipped' as const, hint: `\x1B[36m${state.shipped.reduce((sum, s) => sum + s.skills.length, 0)} ready to use\x1B[0m` }]
+          ? [{ label: 'Install shipped skills', value: 'shipped' as const, hint: styleText('cyan', `${state.shipped.reduce((sum, s) => sum + s.skills.length, 0)} ready to use`) }]
           : []
 
         const source = hasPkgJson
@@ -250,7 +251,7 @@ const main = defineCommand({
         }
 
         if (source === 'skip') {
-          p.log.info('Run \x1B[36mskilld add <pkg>\x1B[0m or \x1B[36mskilld\x1B[0m anytime to add skills.')
+          p.log.info(`Run ${styleText('cyan', 'skilld add <pkg>')} or ${styleText('cyan', 'skilld')} anytime to add skills.`)
           return
         }
 
@@ -344,7 +345,7 @@ const main = defineCommand({
               const hint = sourceMap.get(name) === 'preset' ? 'nuxt module' : undefined
               const pad = ' '.repeat(maxLen - name.length + 2)
               const meta = [ver, hint, repo].filter(Boolean).join('  ')
-              return { label: meta ? `${name}${pad}\x1B[90m${meta}\x1B[39m` : name, value: name }
+              return { label: meta ? `${name}${pad}${styleText('gray', meta)}` : name, value: name }
             }),
             initialValues: preselect,
           })
@@ -384,7 +385,7 @@ const main = defineCommand({
           const previewLines = previewContent.split('\n').slice(0, 20).join('\n').replace(/\x1B\[[0-?]*[ -/]*[@-~]/g, '').replace(/\x1B\].*?(?:\x07|\x1B\\)/g, '')
           const fileSize = (Buffer.byteLength(previewContent) / 1024).toFixed(1)
           p.note(
-            `\x1B[90m${previewLines}\n...\x1B[0m`,
+            styleText('gray', `${previewLines}\n...`),
             `${agents[agent].skillsDir}/${previewSkill.name}/SKILL.md (${fileSize} KB)`,
           )
         }
@@ -408,18 +409,18 @@ const main = defineCommand({
       }
       const verifyLine = agentInstalled
         ? (verifyTips[agent] ?? '')
-        : `Skills are ready in ${agents[agent].skillsDir}/.\n\x1B[90m${agentName} was not detected on this machine.\nInstall it to use these skills, or run \`skilld config\` to change agents.\x1B[0m`
+        : `Skills are ready in ${agents[agent].skillsDir}/.\n${styleText('gray', `${agentName} was not detected on this machine.\nInstall it to use these skills, or run \`skilld config\` to change agents.`)}`
 
       // Build a "try it" suggestion that tests skill-specific knowledge
       const firstPkg = previewSkill?.info?.packageName || previewSkill?.name
       const trySuggestion = firstPkg
-        ? `\n\n\x1B[36mTry it:\x1B[0m ask your agent "What are the gotchas or breaking changes in ${firstPkg}?"`
+        ? `\n\n${styleText('cyan', 'Try it:')} ask your agent "What are the gotchas or breaking changes in ${firstPkg}?"`
         : ''
 
       p.note(
         `${verifyLine}${trySuggestion}\n\n`
-        + `Run \x1B[36mskilld info\x1B[0m to see installed skills.\n`
-        + `Run \x1B[36mskilld\x1B[0m again to add more, update, or search.`,
+        + `Run ${styleText('cyan', 'skilld info')} to see installed skills.\n`
+        + `Run ${styleText('cyan', 'skilld')} again to add more, update, or search.`,
         `${agentName} - next steps`,
       )
 
@@ -439,13 +440,13 @@ const main = defineCommand({
 
     let needsPrepareHook = !hasPrepareHook(cwd)
     if (needsPrepareHook) {
-      p.log.warn(`\x1B[33mNo prepare hook.\x1B[0m Skills won't auto-restore on \x1B[36mnpm install\x1B[0m.`)
+      p.log.warn(`${styleText('yellow', 'No prepare hook.')} Skills won't auto-restore on ${styleText('cyan', 'npm install')}.`)
     }
 
     if (state.shipped.length > 0) {
       const totalSkills = state.shipped.reduce((sum, s) => sum + s.skills.length, 0)
       const names = state.shipped.map(s => s.packageName).join(', ')
-      p.log.info(`\x1B[36m${totalSkills} ready-to-use skill${totalSkills > 1 ? 's' : ''}\x1B[0m shipped by your dependencies: ${names}`)
+      p.log.info(`${styleText('cyan', `${totalSkills} ready-to-use skill${totalSkills > 1 ? 's' : ''}`)} shipped by your dependencies: ${names}`)
     }
 
     const refreshState = async () => {
@@ -459,14 +460,14 @@ const main = defineCommand({
         const opts: Array<{ label: string, value: string, hint?: string }> = []
         if (state.shipped.length > 0) {
           const total = state.shipped.reduce((sum, s) => sum + s.skills.length, 0)
-          opts.push({ label: 'Install shipped skills', value: 'shipped', hint: `\x1B[36m${total} available\x1B[0m` })
+          opts.push({ label: 'Install shipped skills', value: 'shipped', hint: styleText('cyan', `${total} available`) })
         }
         opts.push({ label: 'Add new skills', value: 'install' })
         if (state.outdated.length > 0) {
-          opts.push({ label: 'Update skills', value: 'update', hint: `\x1B[33m${state.outdated.length} outdated\x1B[0m` })
+          opts.push({ label: 'Update skills', value: 'update', hint: styleText('yellow', `${state.outdated.length} outdated`) })
         }
         if (needsPrepareHook) {
-          opts.push({ label: 'Setup prepare hook', value: 'prepare-hook', hint: '\x1B[33mrecommended\x1B[0m' })
+          opts.push({ label: 'Setup prepare hook', value: 'prepare-hook', hint: styleText('yellow', 'recommended') })
         }
         opts.push(
           { label: 'Remove skills', value: 'remove' },
@@ -595,7 +596,7 @@ const main = defineCommand({
                           : undefined
                   const pad = ' '.repeat(maxLen - name.length + 2)
                   const meta = [ver, hint, repo].filter(Boolean).join('  ')
-                  return { label: meta ? `${name}${pad}\x1B[90m${meta}\x1B[39m` : name, value: name }
+                  return { label: meta ? `${name}${pad}${styleText('gray', meta)}` : name, value: name }
                 }),
                 initialValues: [],
               }))

src/cli/agent-prompt.ts

@@ -1,4 +1,5 @@
 import type { AgentType } from '../agent/index.ts'
+import { styleText } from 'node:util'
 import * as p from '@clack/prompts'
 import { agents, detectInstalledAgents, detectProjectAgents, detectTargetAgent } from '../agent/index.ts'
 import { readConfig, updateConfig } from '../core/config.ts'
@@ -95,7 +96,7 @@ export async function promptForAgent(): Promise<AgentType | 'none' | null> {
         ? `Found ${installed.map(t => agents[t].displayName).join(', ')} but couldn't determine which to use`
         : 'No agents auto-detected'
     const crossNote = sharedIds.length > 1
-      ? `\n  \x1B[90mTip: Picking Claude Code shares skills with ${sharedIds.filter(id => id !== 'claude-code').map(id => agents[id].displayName).join(', ')} automatically.\x1B[0m`
+      ? `\n  ${styleText('gray', `Tip: Picking Claude Code shares skills with ${sharedIds.filter(id => id !== 'claude-code').map(id => agents[id].displayName).join(', ')} automatically.`)}`
       : ''
     p.log.warn(`${hint}\n  Pick the agent you actively code with.${crossNote}`)
   }

src/cli/intro.ts

@@ -1,5 +1,6 @@
 import type { AgentType } from '../agent/index.ts'
 import type { ProjectState } from '../core/skills.ts'
+import { styleText } from 'node:util'
 import { join } from 'pathe'
 import { agents, detectInstalledAgents, getAgentVersion, getModelName } from '../agent/index.ts'
 import { readPackageJsonSafe } from '../core/package-json.ts'
@@ -54,10 +55,10 @@ export function getLastSynced(state: ProjectState): string | null {
 }
 
 export function introLine({ state, generators, modelId, agentId }: IntroOptions): string {
-  const name = '\x1B[1m\x1B[35mskilld\x1B[0m'
-  const ver = `\x1B[90mv${version}\x1B[0m`
+  const name = styleText(['bold', 'magenta'], 'skilld')
+  const ver = styleText('gray', `v${version}`)
   const lastSynced = getLastSynced(state)
-  const synced = lastSynced ? ` · \x1B[90msynced ${lastSynced}\x1B[0m` : ''
+  const synced = lastSynced ? ` · ${styleText('gray', `synced ${lastSynced}`)}` : ''
 
   const parts: string[] = []
   if (modelId)
@@ -67,7 +68,7 @@ export function introLine({ state, generators, modelId, agentId }: IntroOptions)
   if (agentId && agents[agentId as AgentType])
     parts.push(agents[agentId as AgentType].displayName)
   const statusLine = parts.length > 0
-    ? `\n\x1B[90m↳ ${parts.join(' → ')}\x1B[0m`
+    ? `\n${styleText('gray', `↳ ${parts.join(' → ')}`)}`
     : ''
 
   return `${name} ${ver}${synced}${statusLine}`
@@ -76,9 +77,9 @@ export function introLine({ state, generators, modelId, agentId }: IntroOptions)
 export function formatStatus(synced: number, outdated: number): string {
   const parts: string[] = []
   if (synced > 0)
-    parts.push(`\x1B[32m${synced} synced\x1B[0m`)
+    parts.push(styleText('green', `${synced} synced`))
   if (outdated > 0)
-    parts.push(`\x1B[33m${outdated} outdated\x1B[0m`)
+    parts.push(styleText('yellow', `${outdated} outdated`))
   return `Skills: ${parts.join(' · ')}`
 }