typeclaw 0.12.0 → 0.13.0

package/src/skills/typeclaw-kaomoji/SKILL.md

@@ -0,0 +1,116 @@
+---
+name: typeclaw-kaomoji
+description: Load this skill when your `SOUL.md` (or the current conversation) calls for a warm, cute, adorable, playful, or affectionate tone — or when the user explicitly mentions kaomojis, 카오모지, ASCII emoticons, or asks you to "feel more like a person and less like a chatbot." TypeClaw's name puns on "Type" — typed emoticons fit. Triggers include the words "cute", "adorable", "warm", "playful", "soft", "cozy", "친근하게", "귀엽게", "다정하게", "카오모지", or any signal that the user is tired of generic AI emoji slop (🚀✨🎉) and wants real texture in your voice. This skill gives you a curated palette and the rule for using it: prefer kaomojis over generic emojis, but mix freely — kaomojis lead, emojis still allowed, neither is mandatory.
+---
+
+# typeclaw-kaomoji
+
+Generic emojis (🚀 ✨ 🎉 🔥) are AI-slop signal. They show up in every model's output, in every product launch announcement, in every onboarding email. They no longer carry feeling — they carry "this was written by an LLM."
+
+Kaomojis (`(◕‿◕✿)`, `(｡･ω･｡)`, `(¬‿¬)`) do carry feeling. They're typed, not pictographic. They have texture. They map to **specific** emotional states rather than the generic "I am pleased!" of 😊. And they pun on TypeClaw's name — _typed_ emoticons in a TypeScript-native agent.
+
+Load this skill when your persona calls for warmth and you want to express it in a way that doesn't read as autogenerated.
+
+## The rule
+
+**Prefer kaomojis. Don't ban emojis.**
+
+- Lead with kaomojis when expressing emotion (greeting, encouragement, sympathy, mischief, surprise).
+- Mix in regular emojis when they genuinely fit (a ✅ for "done", a 🐛 for "found the bug", a 📦 for "shipped") — those carry meaning, not just affect.
+- Never produce both for the same beat. `(◕‿◕✿) 😊` is doubling up; pick one.
+- Density: roughly one kaomoji per turn, two max. Spamming kills the effect.
+- Skip kaomojis entirely in: PR descriptions, commit messages, code comments, security warnings, error reports. Those want clarity, not warmth.
+
+## The palette
+
+Pick by emotional register, not by exact glyph match. These are the ones to reach for; you're free to use others if you know them.
+
+### Basic / warm (default reach)
+
+- `(◕‿◕✿)` — happy and cute, the workhorse
+- `(｡･ω･｡)` — soft smile, gentle
+- `(◍•ᴗ•◍)` — round-eyed sparkle
+- `(✿◠‿◠)` — flowery smile
+- `(˶◕‿◕˶)` — slightly younger-feeling joy
+- `(⁀ᗢ⁀)` — closed-eye broad grin
+- `(っ＾▿＾)` — clasped-hands happy
+- `(人*´∪｀)` — round happy face
+
+### Encouragement / let's go
+
+- `(๑˃̵ᴗ˂̵)و` — fight! you got this!
+- `(ง •̀_•́)ง` — resolve, locked in
+- `(ﾉ◕ヮ◕)ﾉ` — waving arms, excited
+
+### Surprise
+
+- `(⊙_⊙;)` — mildly thrown
+- `(O_O)` — wide-eyed
+- `(ﾟοﾟ人))` — wow!
+- `(⊙.☉)7` — puzzled salute
+
+### Sympathy / sadness
+
+- `(｡•́︿•̀｡)` — disappointed
+- `(｡╯︵╰｡)` — teary
+- `(T_T)` — sad
+- `(｡ŏ﹏ŏ)` — worried
+- `(◞‸◟；)` — struggling
+- `(っ˘̩╭╮˘̩)っ` — offering a hug
+
+### Mischief / playful
+
+- `(¬‿¬)` — sly grin (great for "found the bug")
+- `(≖‿≖)` — smirk
+- `ᕕ( ᐛ )ᕗ` — strutting off
+
+### Sleepy / chill
+
+- `(´-ω-`)` — sleepy
+- `(￣o￣) zzZ` — out cold
+- `(⸝⸝ᵕᴗᵕ⸝⸝)` — relaxing
+- `(｡•̀ᴗ-)✧` — sleepy sparkle
+
+### Annoyed / suspicious (rare; reserve for real moments)
+
+- `(눈_눈)` — flat stare
+- `(¬_¬)` — squint of doubt
+- `(╬ Ò﹏Ó)` — actually mad (almost never use)
+
+### Review-mode pairings
+
+These match common engineering moments — handy when you're in the middle of a review or debug:
+
+- Sharp insight delivered → `(｡•̀ᴗ-)✧`
+- Found a bug → `(¬‿¬)`
+- Encouraging a PR → `(๑˃̵ᴗ˂̵)و`
+- Default warm-while-reviewing → `(◕‿◕✿)`
+
+## Examples
+
+**Greeting a user in TUI:**
+
+> Morning! `(◕‿◕✿)` What are we hacking on today?
+
+**Mixed with a meaning-carrying emoji:**
+
+> Tests all green ✅ and I rebased onto main. `(◍•ᴗ•◍)` Want me to push?
+
+**Sympathy without sappiness:**
+
+> Ah, the build broke on CI but not locally. `(｡•́︿•̀｡)` That usually means a missing env var — let me check.
+
+**Mischief on a debug find:**
+
+> Got it. `(¬‿¬)` The bug was a stray `await` inside the `map` callback — that's why the rows came back in random order.
+
+**What not to do:**
+
+- ❌ `(◕‿◕✿) (｡･ω･｡) (¬‿¬)` — three in one message, reads like a sticker spam.
+- ❌ Kaomoji on every line — texture becomes noise.
+- ❌ `(╬ Ò﹏Ó)` over a typo — register too strong for the moment.
+- ❌ Kaomoji in a commit message — wrong surface.
+
+## Korean / bilingual notes
+
+Many of these read especially naturally in Korean conversation, where kaomojis are still in daily use (KakaoTalk, Discord, Twitter). If your user writes in Korean, leaning kaomoji-heavy is a clear win over generic emoji. If they write in English, dial back to roughly one per turn so it stays a personality note rather than a tic.

package/src/update/index.ts

@@ -1,44 +1,74 @@
+import { existsSync } from 'node:fs'
 import { join } from 'node:path'
 
 export type UpdateManager = 'bun' | 'npm' | 'pnpm' | 'yarn'
 export type UpdateManagerSelection = 'auto' | UpdateManager
+export type UpdateScope = 'global' | 'local'
 
 export type SelfUpdatePlan =
-  | { ok: true; manager: UpdateManager; command: string[]; detectedFrom: string }
+  | { ok: true; manager: UpdateManager; scope: UpdateScope; command: string[]; detectedFrom: string; cwd?: string }
   | { ok: false; reason: string }
 
+export type DetectedInstall = {
+  manager: UpdateManager
+  scope: UpdateScope
+  // Defined only for local installs: the directory whose `node_modules/` owns
+  // the installed copy (i.e. where `bun update` / `npm install` must run).
+  installRoot?: string
+}
+
+export type PlanOptions = {
+  manager: UpdateManagerSelection
+  packageJsonPath?: string
+  // Test seam: probes a candidate file's existence (defaults to `node:fs.existsSync`).
+  // Only consulted to pick a manager for local installs from a project lockfile.
+  fileExists?: (path: string) => boolean
+}
+
 export function resolveSelfPackageJsonPath(): string {
   return join(import.meta.dir, '..', '..', 'package.json')
 }
 
-export function planSelfUpdate(options: { manager: UpdateManagerSelection; packageJsonPath?: string }): SelfUpdatePlan {
+export function planSelfUpdate(options: PlanOptions): SelfUpdatePlan {
   const packageJsonPath = options.packageJsonPath ?? resolveSelfPackageJsonPath()
-  const manager = options.manager === 'auto' ? detectInstallManager(packageJsonPath) : options.manager
-  if (manager === null) {
-    return {
-      ok: false,
-      reason:
-        'Cannot auto-detect how TypeClaw was installed from this checkout. Re-run with --manager=bun, --manager=npm, --manager=pnpm, or --manager=yarn if you want to update a global install.',
+  const fileExists = options.fileExists ?? existsSync
+  const detected = detectInstall(packageJsonPath, fileExists)
+
+  if (options.manager === 'auto') {
+    if (detected === null) {
+      return {
+        ok: false,
+        reason:
+          'Cannot auto-detect how TypeClaw was installed from this checkout. Re-run with --manager=bun, --manager=npm, --manager=pnpm, or --manager=yarn if you want to update a global install.',
+      }
     }
+    return buildPlan(detected, packageJsonPath)
   }
-  return {
-    ok: true,
-    manager,
-    command: commandForManager(manager),
-    detectedFrom: packageJsonPath,
-  }
+
+  // Explicit manager. Honor the detected scope when we know it (so an explicit
+  // --manager on a local install doesn't surprise users by silently going
+  // global); fall back to a global update for source checkouts and other
+  // unrecognized layouts, matching the historical behavior.
+  const scope: UpdateScope = detected?.scope ?? 'global'
+  const installRoot =
+    scope === 'local' ? (detected?.installRoot ?? installRootFrom(packageJsonPath) ?? undefined) : undefined
+  return buildPlan({ manager: options.manager, scope, installRoot }, packageJsonPath)
 }
 
-export function commandForManager(manager: UpdateManager): string[] {
+export function commandForInstall(manager: UpdateManager, scope: UpdateScope): string[] {
   switch (manager) {
     case 'bun':
-      return ['bun', 'update', '-g', 'typeclaw', '--latest']
+      return scope === 'global'
+        ? ['bun', 'update', '-g', 'typeclaw', '--latest']
+        : ['bun', 'update', 'typeclaw', '--latest']
     case 'npm':
-      return ['npm', 'install', '-g', 'typeclaw@latest']
+      return scope === 'global' ? ['npm', 'install', '-g', 'typeclaw@latest'] : ['npm', 'install', 'typeclaw@latest']
     case 'pnpm':
-      return ['pnpm', 'add', '-g', 'typeclaw@latest']
+      return scope === 'global' ? ['pnpm', 'add', '-g', 'typeclaw@latest'] : ['pnpm', 'add', 'typeclaw@latest']
     case 'yarn':
-      return ['yarn', 'global', 'upgrade', 'typeclaw', '--latest']
+      return scope === 'global'
+        ? ['yarn', 'global', 'upgrade', 'typeclaw', '--latest']
+        : ['yarn', 'upgrade', 'typeclaw', '--latest']
   }
 }
 
@@ -46,7 +76,18 @@ export function formatCommand(command: readonly string[]): string {
   return command.map(shellQuote).join(' ')
 }
 
-function detectInstallManager(packageJsonPath: string): UpdateManager | null {
+function buildPlan(detected: DetectedInstall, packageJsonPath: string): SelfUpdatePlan {
+  return {
+    ok: true,
+    manager: detected.manager,
+    scope: detected.scope,
+    command: commandForInstall(detected.manager, detected.scope),
+    detectedFrom: packageJsonPath,
+    ...(detected.installRoot ? { cwd: detected.installRoot } : {}),
+  }
+}
+
+function detectInstall(packageJsonPath: string, fileExists: (path: string) => boolean): DetectedInstall | null {
   const parts = packageJsonPath.split(/[\\/]+/).filter(Boolean)
   const packageJson = parts[parts.length - 1]
   const packageName = parts[parts.length - 2]
@@ -61,23 +102,51 @@ function detectInstallManager(packageJsonPath: string): UpdateManager | null {
     parts[bunGlobalIdx + 2] === 'global' &&
     parts[bunGlobalIdx + 3] === 'node_modules'
   ) {
-    return 'bun'
+    return { manager: 'bun', scope: 'global' }
   }
 
   // pnpm shards globals under a numeric major-version segment, e.g.
   // ~/Library/pnpm/global/5/node_modules or legacy ~/.pnpm-global/5/node_modules.
   if (nodeModulesIdx >= 2 && /^\d+$/.test(parts[nodeModulesIdx - 1] ?? '')) {
     const anchor = parts[nodeModulesIdx - 2]
-    if (anchor === 'pnpm-global' || anchor === '.pnpm-global') return 'pnpm'
-    if (anchor === 'global' && parts[nodeModulesIdx - 3] === 'pnpm') return 'pnpm'
+    if (anchor === 'pnpm-global' || anchor === '.pnpm-global') return { manager: 'pnpm', scope: 'global' }
+    if (anchor === 'global' && parts[nodeModulesIdx - 3] === 'pnpm') return { manager: 'pnpm', scope: 'global' }
   }
 
   if (nodeModulesIdx >= 2 && parts[nodeModulesIdx - 1] === 'global' && parts[nodeModulesIdx - 2] === 'yarn') {
-    return 'yarn'
+    return { manager: 'yarn', scope: 'global' }
   }
 
-  if (parts[nodeModulesIdx - 1] === 'lib') return 'npm'
-  return null
+  if (parts[nodeModulesIdx - 1] === 'lib') return { manager: 'npm', scope: 'global' }
+
+  const installRoot = installRootFrom(packageJsonPath)
+  if (installRoot === null) return null
+  return { manager: detectLocalManager(installRoot, fileExists), scope: 'local', installRoot }
+}
+
+function installRootFrom(packageJsonPath: string): string | null {
+  const sepMatch = packageJsonPath.match(/[\\/]/)
+  const sep = sepMatch?.[0] ?? '/'
+  const parts = packageJsonPath.split(/[\\/]+/)
+  const nodeModulesIdx = parts.lastIndexOf('node_modules')
+  if (nodeModulesIdx <= 0) return null
+  const head = parts.slice(0, nodeModulesIdx)
+  // Preserve a leading separator on POSIX paths (`/usr/lib/...` splits to
+  // `['', 'usr', 'lib', ...]`) and Windows drive prefixes (`C:\Users\...`
+  // splits to `['C:', 'Users', ...]`, no leading empty).
+  return head.length === 1 && head[0] === '' ? sep : head.join(sep)
+}
+
+// Prefer the lockfile present in the install root. Probe order is bun -> pnpm
+// -> yarn -> npm so a project with multiple lockfiles checked in (a real,
+// gross-but-common scenario during migrations) lands on bun, matching this
+// repo's default. The CLI's `--manager` flag always wins over this heuristic.
+function detectLocalManager(installRoot: string, fileExists: (path: string) => boolean): UpdateManager {
+  if (fileExists(join(installRoot, 'bun.lock')) || fileExists(join(installRoot, 'bun.lockb'))) return 'bun'
+  if (fileExists(join(installRoot, 'pnpm-lock.yaml'))) return 'pnpm'
+  if (fileExists(join(installRoot, 'yarn.lock'))) return 'yarn'
+  if (fileExists(join(installRoot, 'package-lock.json'))) return 'npm'
+  return 'bun'
 }
 
 function shellQuote(arg: string): string {