feat: add jsonc-parser for surgical package.json edits

.claude/skills/sindresorhus-log-update/SKILL.md

@@ -1,76 +1,62 @@
 ---
 name: sindresorhus-log-update
-description: "ALWAYS use when writing code importing \"log-update\". Consult for debugging, best practices, or modifying log-update, log update."
+description: "Log by overwriting the previous output in the terminal. Useful for rendering progress bars, animations, etc. ALWAYS use when writing code importing \"log-update\". Consult for debugging, best practices, or modifying log-update, log update."
 metadata:
-  version: 7.1.0
+  version: 7.2.0
+  generated_by: cached
+  generated_at: 2026-03-24
 ---
 
 # sindresorhus/log-update `log-update`
 
-**Version:** 7.1.0 (1 week ago)
-**Deps:** ansi-escapes@^7.1.0, cli-cursor@^5.0.0, slice-ansi@^7.1.2, strip-ansi@^7.1.2, wrap-ansi@^9.0.2
-**Tags:** latest: 7.1.0 (1 week ago)
+> Log by overwriting the previous output in the terminal. Useful for rendering progress bars, animations, etc.
 
-**References:** [package.json](./.skilld/pkg/package.json) • [GitHub Issues](./.skilld/issues/_INDEX.md) • [Releases](./.skilld/releases/_INDEX.md)
+**Version:** 7.2.0
+**Deps:** ansi-escapes@^7.3.0, cli-cursor@^5.0.0, slice-ansi@^8.0.0, strip-ansi@^7.2.0, wrap-ansi@^10.0.0
+**Tags:** latest: 7.2.0
 
-## Search
-
-Use `npx -y skilld search` instead of grepping `.skilld/` directories — hybrid semantic + keyword search across all indexed docs, issues, and releases.
+**References:** [package.json](./.skilld/pkg/package.json) — exports, entry points • [GitHub Issues](./.skilld/issues/_INDEX.md) — bugs, workarounds, edge cases • [Releases](./.skilld/releases/_INDEX.md) — changelog, breaking changes, new APIs
 
-```bash
-npx -y skilld search "query" -p log-update
-npx -y skilld search "issues:error handling" -p log-update
-npx -y skilld search "releases:deprecated" -p log-update
-```
+## Search
 
-Filters: `docs:`, `issues:`, `releases:` prefix narrows by source type.
+Use `skilld search "query" -p log-update` instead of grepping `.skilld/` directories. Run `skilld search --guide -p log-update` for full syntax, filters, and operators.
 
+<!-- skilld:api-changes -->
 ## API Changes
 
-✨ `.persist(...text)` — new in v7.0, writes text that stays in scrollback (like `console.log`) without clearing the update area [source](./.skilld/releases/v7.0.0.md)
-
-✨ `defaultWidth` / `defaultHeight` options — new in v7.0 for `createLogUpdate()`, controls fallback dimensions when stream lacks `columns`/`rows` (default: 80×24) [source](./.skilld/releases/v7.0.0.md)
+This section documents version-specific API changes — prioritize recent major/minor releases.
 
-✨ Partial diff rendering — v7.0 only redraws changed lines instead of erasing all, reduces flicker [source](./.skilld/releases/v7.0.0.md)
+- BREAKING: Node.js requirement updated from 18 to 20 in v7.0.0 — code targeting earlier Node.js versions will fail at runtime [source](./.skilld/releases/v7.0.0.md#breaking)
 
-✨ Synchronized output (`?2026h`/`?2026l`) — v7.1 wraps writes in DEC synchronized output sequences on TTYs, eliminates tearing [source](./.skilld/releases/v7.1.0.md)
+- NEW: `persist(...text)` method added in v7.0.0 — writes output that persists in terminal scrollback, unlike the main `logUpdate()` which updates in place [source](./.skilld/releases/v7.0.0.md#improvements)
 
-⚠️ Node.js 20+ required — v7.0 dropped Node 18 support [source](./.skilld/releases/v7.0.0.md)
+- NEW: `defaultWidth` option added in v7.0.0 — allows specifying terminal width (default: 80) when the stream doesn't provide `columns` property, useful for piped or redirected output [source](./.skilld/releases/v7.0.0.md#improvements)
 
-⚠️ `logUpdate.create()` removed in v5 — use named export `createLogUpdate` instead [source](./.skilld/releases/v5.0.0.md)
+- NEW: `defaultHeight` option added in v7.0.0 — allows specifying terminal height (default: 24) when the stream doesn't provide `rows` property, useful for piped or redirected output [source](./.skilld/releases/v7.0.0.md#improvements)
 
-⚠️ `logUpdate.stderr` removed in v5 — use named export `logUpdateStderr` instead [source](./.skilld/releases/v5.0.0.md)
-
-⚠️ Pure ESM since v5 — no `require()`, use `import logUpdate from 'log-update'` [source](./.skilld/releases/v5.0.0.md)
+**Also changed:** Partial update rendering optimization v7.0.0 · Trailing newline handling fixes v7.0.1–v7.0.2
+<!-- /skilld:api-changes -->
 
+<!-- skilld:best-practices -->
 ## Best Practices
 
-✅ Use `.persist()` for permanent output between updating sections — it writes to scrollback history then resets the update region, unlike `.done()` which just freezes the current frame [source](./.skilld/pkg/readme.md)
-
-```ts
-logUpdate('Downloading...')
-logUpdate.persist('✓ Download complete')  // stays in scrollback
-logUpdate('Installing...')                 // new update region starts
-
-```
-✅ Call `.done()` when finished to restore the cursor — the default export hides the cursor on first call and only restores it on `.done()` [source](./.skilld/pkg/index.js)
+- Use `.persist()` for permanent terminal output that should remain in scrollback history — it writes like `console.log()` instead of updating in-place, ideal for results or status messages that users need to scroll back to review [source](./.skilld/pkg/index.d.ts:L45:66)
 
-✅ Set `showCursor: true` via `createLogUpdate` when your CLI also accepts user input — the default singleton hides the cursor which breaks interactive prompts [source](./.skilld/pkg/readme.md)
+- Call `.done()` before starting a new log session to persist the current output and prepare for fresh updates below — prevents scrollback contamination [source](./.skilld/pkg/readme.md#logUpdateDone)
 
-```ts
-const log = createLogUpdate(process.stdout, { showCursor: true })
-```
+- Enable `showCursor: true` when your CLI accepts user input or displays a prompt — otherwise the hidden cursor creates a poor UX for interactive interfaces [source](./.skilld/pkg/readme.md#showCursor)
 
-✅ Set `defaultWidth`/`defaultHeight` when output may be piped or redirected — `stream.columns`/`stream.rows` are undefined in non-TTY contexts, defaults are 80×24 [source](./.skilld/pkg/readme.md)
+- Set `defaultWidth` and `defaultHeight` options when output is piped, redirected, or running in non-TTY environments — without this, the library assumes 80x24 which causes incorrect line wrapping and erasure [source](./.skilld/releases/v7.0.0.md#improvements)
 
-✅ Output is automatically clipped to terminal height (bottom lines kept, top removed) — cannot be disabled, design your output with the most important info at the bottom [source](./.skilld/issues/issue-51.md)
+- Create separate `logUpdate` instances via `createLogUpdate(stream)` for each independent update context instead of reusing the default export — prevents one instance's updates from clobbering another's [source](./.skilld/pkg/index.d.ts:L74:87)
 
-✅ Content exceeding terminal width is hard-wrapped per-character (not word-wrapped) — ANSI-colored strings are handled correctly but long unbroken lines will split mid-word [source](./.skilld/pkg/index.js)
+- Understand that log-update can only erase and rewrite lines currently visible in the terminal viewport — content already in scrollback cannot be manipulated, so extremely long outputs will overflow and "leak" [source](./.skilld/issues/issue-10.md)
 
-✅ Multiple string arguments are joined with spaces, not newlines — `logUpdate('a', 'b')` produces `"a b"`, use template literals or `\n` for multiline [source](./.skilld/pkg/index.js)
+- Account for ANSI escape sequences in output width calculations — the library properly handles colorized text with tools like chalk/yoctocolors and accounts for their escape codes when calculating line wrapping [source](./.skilld/issues/issue-12.md)
 
-✅ Use `createLogUpdate` for multiple independent update regions — the default export is a singleton; two modules sharing it will clobber each other's output. Each `createLogUpdate` call tracks its own state [source](./.skilld/issues/issue-48.md)
+- Use `logUpdateStderr` for error messages and warnings instead of `logUpdate` — maintains separation between stdout and stderr streams [source](./.skilld/pkg/index.d.ts:L69:81)
 
-✅ v7.1.0 uses synchronized output (`\x1b[?2026h/l`) on TTYs to eliminate flicker — no action needed, but be aware this wraps every write in DEC private mode sequences that some non-standard terminals may not support [source](./.skilld/releases/v7.1.0.md)
+- Leverage partial/synchronized rendering in v7.1.0+ for reduced flicker when updating large outputs — the library now intelligently redraws only changed lines instead of erasing everything [source](./.skilld/releases/v7.1.0.md)
 
-✅ Identical consecutive frames are skipped (no-op) — safe to call `logUpdate()` at high frequency without performance concern, the library diffs and only writes changed lines [source](./.skilld/pkg/index.js)
+- Be aware that multiple consecutive blank lines at the end of output may collapse to a single blank line during rendering optimization — design output that doesn't rely on exact blank line preservation [source](./.skilld/issues/issue-62.md)
+<!-- /skilld:best-practices -->

.claude/skills/skilld-lock.yaml

@@ -1,10 +1,11 @@
 skills:
   sindresorhus-log-update:
     packageName: log-update
-    version: 7.1.0
+    version: 7.2.0
+    packages: "log-update@7.2.0"
     repo: sindresorhus/log-update
-    source: "https://raw.githubusercontent.com/sindresorhus/log-update/main/readme.md"
-    syncedAt: 2026-02-13
+    source: "ungh://sindresorhus/log-update"
+    syncedAt: 2026-03-24
     generator: skilld
   microsoft-typescript:
     packageName: typescript

package.json

@@ -63,6 +63,7 @@
     "citty": "catalog:deps",
     "consola": "catalog:deps",
     "giget": "catalog:",
+    "jsonc-parser": "catalog:",
     "log-update": "catalog:deps",
     "mdast-util-from-markdown": "catalog:deps",
     "mdast-util-frontmatter": "catalog:deps",

pnpm-workspace.yaml

@@ -1,5 +1,4 @@
 catalogMode: prefer
-
 shellEmulator: true
 
 trustPolicy: no-downgrade
@@ -13,6 +12,7 @@ catalog:
   '@types/semver': ^7.7.1
   bumpp: ^11.0.1
   giget: ^3.1.2
+  jsonc-parser: ^3.3.1
   mdream: ^1.0.3
   ofetch: ^1.5.1
   pathe: ^2.0.3

src/cli-helpers.ts

@@ -7,10 +7,12 @@ import type { AgentType, OptimizeModel } from './agent/index.ts'
 import type { ProjectState } from './core/skills.ts'
 import { existsSync, readFileSync } from 'node:fs'
 import * as p from '@clack/prompts'
+import { parseTree } from 'jsonc-parser'
 import { join } from 'pathe'
 import { detectCurrentAgent } from 'unagent/env'
 import { agents, detectInstalledAgents, detectProjectAgents, detectTargetAgent, getAgentVersion, getModelName } from './agent/index.ts'
 import { readConfig, updateConfig } from './core/config.ts'
+import { editJsonProperty, patchPackageJson } from './core/package-json.ts'
 import { version } from './version.ts'
 
 export type { AgentType, OptimizeModel }
@@ -440,31 +442,74 @@ export async function pickModel<T extends { provider: string, providerName: stri
 }
 
 /**
- * Suggest adding `skilld prepare` to package.json "prepare" script.
- * Returns true if the suggestion was shown (i.e. hook is missing).
+ * Prompt to add `skilld prepare` to package.json "prepare" script.
+ * In non-interactive environments, falls back to an info log.
+ * Returns true if the hook was added or already present.
  */
-export function suggestPrepareHook(cwd: string = process.cwd()): boolean {
+export async function suggestPrepareHook(cwd: string = process.cwd()): Promise<boolean> {
   const pkgJsonPath = join(cwd, 'package.json')
   if (!existsSync(pkgJsonPath))
     return false
 
-  const pkgJson = JSON.parse(readFileSync(pkgJsonPath, 'utf-8'))
-  if (pkgJson.scripts?.prepare?.includes('skilld'))
+  const raw = readFileSync(pkgJsonPath, 'utf-8')
+  const pkgJson = JSON.parse(raw)
+  const existing: string | undefined = pkgJson.scripts?.prepare
+
+  if (existing?.includes('skilld'))
+    return true
+
+  const prepareCmd = buildPrepareScript(existing)
+
+  if (!isInteractive()) {
+    p.log.info(
+      `\x1B[90mAdd to package.json scripts:\n`
+      + `  \x1B[36m"prepare": "${prepareCmd}"\x1B[0m\n`
+      + `  \x1B[90mRestores references and shipped skills on install.\x1B[0m`,
+    )
     return false
+  }
 
-  const prepareCmd = pkgJson.scripts?.prepare
-    ? `${pkgJson.scripts.prepare} && skilld prepare`
-    : 'skilld prepare'
+  const confirmed = await p.confirm({
+    message: `Add \x1B[36m"prepare": "${prepareCmd}"\x1B[0m to package.json?`,
+    initialValue: true,
+  })
+  if (p.isCancel(confirmed) || !confirmed)
+    return false
 
-  p.log.info(
-    `\x1B[90mAdd to package.json scripts:\n`
-    + `  \x1B[36m"prepare": "${prepareCmd}"\x1B[0m\n`
-    + `  \x1B[90mRestores references and shipped skills on install. Notifies when updates are available.\n`
-    + `  Commit skilld-lock.yaml so teammates can run \`skilld install\`.\x1B[0m`,
+  // Ensure "scripts" object exists in the JSON
+  const tree = parseTree(raw)
+  const hasScripts = tree?.children?.some(c =>
+    c.type === 'property' && c.children?.[0]?.value === 'scripts',
   )
+
+  let patched = raw
+  if (!hasScripts)
+    patched = editJsonProperty(patched, ['scripts'], {})
+
+  patched = editJsonProperty(patched, ['scripts', 'prepare'], prepareCmd)
+
+  patchPackageJson(pkgJsonPath, () => patched)
+  p.log.success('Added \x1B[36mskilld prepare\x1B[0m to package.json')
   return true
 }
 
+/**
+ * Build the full prepare script value, safely appending to any existing command.
+ */
+function buildPrepareScript(existing: string | undefined): string {
+  if (!existing || !existing.trim())
+    return 'skilld prepare'
+
+  const trimmed = existing.trim()
+
+  // Strip trailing && or ; that would leave a dangling operator
+  const cleaned = trimmed.replace(/[&|;]+\s*$/, '').trim()
+  if (!cleaned)
+    return 'skilld prepare'
+
+  return `${cleaned} && skilld prepare`
+}
+
 export function getRepoHint(name: string, cwd: string): string | undefined {
   const pkgJsonPath = join(cwd, 'node_modules', name, 'package.json')
   if (!existsSync(pkgJsonPath))

src/cli.ts

@@ -514,7 +514,7 @@ const main = defineCommand({
       )
 
       // Team advice: suggest prepare hook + lockfile
-      suggestPrepareHook(cwd)
+      await suggestPrepareHook(cwd)
       return
     }
 

src/commands/author.ts

@@ -18,6 +18,7 @@ import {
 import { guard } from '../cli-helpers.ts'
 import { defaultFeatures, readConfig } from '../core/config.ts'
 import { timedSpinner } from '../core/formatting.ts'
+import { appendToJsonArray, patchPackageJson } from '../core/package-json.ts'
 import { sanitizeMarkdown } from '../core/sanitize.ts'
 import {
   fetchGitHubDiscussions,
@@ -314,38 +315,20 @@ export function patchPackageJsonFiles(packageDir: string): void {
   if (!existsSync(pkgPath))
     return
 
-  const raw = readFileSync(pkgPath, 'utf-8')
-  const pkg = JSON.parse(raw)
+  const wrote = patchPackageJson(pkgPath, (raw, pkg) => {
+    if (!Array.isArray(pkg.files)) {
+      p.log.warn('No `files` array in package.json. Add `"skills"` to your files array manually.')
+      return null
+    }
 
-  if (!Array.isArray(pkg.files)) {
-    p.log.warn('No `files` array in package.json. Add `"skills"` to your files array manually.')
-    return
-  }
+    if ((pkg.files as string[]).some((f: string) => f === 'skills' || f === 'skills/' || f === 'skills/**'))
+      return null
 
-  if (pkg.files.some((f: string) => f === 'skills' || f === 'skills/' || f === 'skills/**'))
-    return
+    return appendToJsonArray(raw, ['files'], 'skills')
+  })
 
-  // Targeted insertion: find the closing bracket of the files array and insert before it
-  // This preserves the original formatting (indentation, trailing newlines, etc.)
-  const filesMatch = raw.match(/"files"\s*:\s*\[([^\]]*)\]/)
-  if (!filesMatch) {
-    // Fallback: full rewrite
-    pkg.files.push('skills')
-    writeFileSync(pkgPath, `${JSON.stringify(pkg, null, 2)}\n`)
+  if (wrote)
     p.log.success('Added `"skills"` to package.json files array')
-    return
-  }
-
-  const inner = filesMatch[1]
-  const trimmed = inner.trimEnd()
-  // Detect indentation from existing entries
-  const entryMatch = inner.match(/\n(\s+)"/)
-  const indent = entryMatch ? entryMatch[1] : '    '
-  const needsComma = trimmed.length > 0 && !trimmed.endsWith(',')
-  const insertion = `${needsComma ? ',' : ''}\n${indent}"skills"`
-  const patched = raw.replace(filesMatch[0], `"files": [${trimmed}${insertion}\n${indent.slice(2) || '  '}]`)
-  writeFileSync(pkgPath, patched)
-  p.log.success('Added `"skills"` to package.json files array')
 }
 
 // ── Core author flow for a single package ──

src/commands/sync-parallel.ts

@@ -386,7 +386,7 @@ export async function syncPackagesParallel(config: ParallelSyncConfig): Promise<
   p.outro(`${pastVerb} ${successfulPkgs.length}/${packages.length} packages`)
 
   const { suggestPrepareHook } = await import('../cli-helpers.ts')
-  suggestPrepareHook(cwd)
+  await suggestPrepareHook(cwd)
 }
 
 type UpdateFn = (pkg: string, status: PackageStatus, message: string, version?: string) => void

src/commands/sync.ts

@@ -715,7 +715,7 @@ async function syncSinglePackage(packageSpec: string, config: SyncConfig): Promi
   const relDir = relative(cwd, skillDir)
   p.outro(config.mode === 'update' ? `Updated ${packageName}${ejectMsg}` : `Synced ${packageName} → ${relDir}${ejectMsg}`)
 
-  suggestPrepareHook(cwd)
+  await suggestPrepareHook(cwd)
 }
 
 // ── Citty command definitions (lazy-loaded by cli.ts) ──

src/core/index.ts

@@ -2,6 +2,7 @@ export * from './config.ts'
 export * from './formatting.ts'
 export * from './lockfile.ts'
 export * from './markdown.ts'
+export * from './package-json.ts'
 export * from './sanitize.ts'
 export * from './shared.ts'
 export * from './skills.ts'