@pencil-agent/nano-pencil 2.0.1 → 2.0.3

package/dist/extensions/builtin/browser/src/browser_harness/run.py

@@ -1,103 +1,103 @@
-"""Browser Harness CLI-compatible runner used by NanoPencil subprocess tools.
-
-[WHO]: Provides main() command dispatch, daemon auto-start, setup/doctor/update/reload flags, and snippet execution
-[FROM]: Depends on browser_harness.admin, browser_harness.helpers, urllib, os, and sys
-[TO]: Consumed by NanoPencil browser/browser_admin tools via python -m browser_harness.run
-[HERE]: extensions/builtin/browser/src/browser_harness/run.py within vendored Browser Harness package
-"""
-
-import os, sys, urllib.request
-
-# Windows default stdout encoding is cp1252, which can't encode the 🟢 marker
-# helpers prepend to tab titles (or anything else outside Latin-1). Force UTF-8
-# so `print(page_info())` doesn't UnicodeEncodeError on Windows. Issue #124(4).
-if hasattr(sys.stdout, "reconfigure"):
-    try: sys.stdout.reconfigure(encoding="utf-8", errors="replace")
-    except Exception: pass
-
-from .admin import (
-    _version,
-    NAME,
-    daemon_alive,
-    ensure_daemon,
-    list_cloud_profiles,
-    list_local_profiles,
-    print_update_banner,
-    restart_daemon,
-    run_doctor,
-    run_setup,
-    run_update,
-    start_remote_daemon,
-    stop_remote_daemon,
-    sync_local_profile,
-)
-from .helpers import *
-
-HELP = """Browser Harness
-
-Read SKILL.md for the default workflow and examples.
-
-Typical usage:
-  browser-harness -c '
-  ensure_real_tab()
-  print(page_info())
-  '
-
-Helpers are pre-imported. The daemon auto-starts and connects to the running browser.
-
-Commands:
-  browser-harness --version        print the installed version
-  browser-harness --doctor         diagnose install, daemon, and browser state
-  browser-harness --setup          interactively attach to your running browser
-  browser-harness --update [-y]    pull the latest version (agents: pass -y)
-  browser-harness --reload         stop the daemon so next call picks up code changes
-"""
-
-
-# Probe /json/version (not a bare TCP connect) so a non-Chrome process bound to
-# 9222/9223 doesn't masquerade as Chrome and skip the cloud bootstrap. Mirrors
-# daemon.py's fallback probe.
-def _local_chrome_listening():
-    for port in (9222, 9223):
-        try:
-            urllib.request.urlopen(f"http://127.0.0.1:{port}/json/version", timeout=0.3).close()
-            return True
-        except OSError: pass
-    return False
-
-
-def main():
-    args = sys.argv[1:]
-    if args and args[0] in {"-h", "--help"}:
-        print(HELP)
-        return
-    if args and args[0] == "--version":
-        print(_version() or "unknown")
-        return
-    if args and args[0] == "--doctor":
-        sys.exit(run_doctor())
-    if args and args[0] == "--setup":
-        sys.exit(run_setup())
-    if args and args[0] == "--update":
-        yes = any(a in {"-y", "--yes"} for a in args[1:])
-        sys.exit(run_update(yes=yes))
-    if args and args[0] == "--reload":
-        restart_daemon()
-        print("daemon stopped — will restart fresh on next call")
-        return
-    if args and args[0] == "--debug-clicks":
-        os.environ["BH_DEBUG_CLICKS"] = "1"
-        args = args[1:]
-    if not args or args[0] != "-c":
-        sys.exit("Usage: browser-harness -c \"print(page_info())\"")
-    if len(args) < 2:
-        sys.exit("Usage: browser-harness -c \"print(page_info())\"")
-    print_update_banner()
-    if not daemon_alive() and not _local_chrome_listening() and os.environ.get("BROWSER_USE_API_KEY"):
-        start_remote_daemon(NAME)
-    ensure_daemon()
-    exec(args[1], globals())
-
-
-if __name__ == "__main__":
-    main()
+"""Browser Harness CLI-compatible runner used by NanoPencil subprocess tools.
+
+[WHO]: Provides main() command dispatch, daemon auto-start, setup/doctor/update/reload flags, and snippet execution
+[FROM]: Depends on browser_harness.admin, browser_harness.helpers, urllib, os, and sys
+[TO]: Consumed by NanoPencil browser/browser_admin tools via python -m browser_harness.run
+[HERE]: extensions/builtin/browser/src/browser_harness/run.py within vendored Browser Harness package
+"""
+
+import os, sys, urllib.request
+
+# Windows default stdout encoding is cp1252, which can't encode the 🟢 marker
+# helpers prepend to tab titles (or anything else outside Latin-1). Force UTF-8
+# so `print(page_info())` doesn't UnicodeEncodeError on Windows. Issue #124(4).
+if hasattr(sys.stdout, "reconfigure"):
+    try: sys.stdout.reconfigure(encoding="utf-8", errors="replace")
+    except Exception: pass
+
+from .admin import (
+    _version,
+    NAME,
+    daemon_alive,
+    ensure_daemon,
+    list_cloud_profiles,
+    list_local_profiles,
+    print_update_banner,
+    restart_daemon,
+    run_doctor,
+    run_setup,
+    run_update,
+    start_remote_daemon,
+    stop_remote_daemon,
+    sync_local_profile,
+)
+from .helpers import *
+
+HELP = """Browser Harness
+
+Read SKILL.md for the default workflow and examples.
+
+Typical usage:
+  browser-harness -c '
+  ensure_real_tab()
+  print(page_info())
+  '
+
+Helpers are pre-imported. The daemon auto-starts and connects to the running browser.
+
+Commands:
+  browser-harness --version        print the installed version
+  browser-harness --doctor         diagnose install, daemon, and browser state
+  browser-harness --setup          interactively attach to your running browser
+  browser-harness --update [-y]    pull the latest version (agents: pass -y)
+  browser-harness --reload         stop the daemon so next call picks up code changes
+"""
+
+
+# Probe /json/version (not a bare TCP connect) so a non-Chrome process bound to
+# 9222/9223 doesn't masquerade as Chrome and skip the cloud bootstrap. Mirrors
+# daemon.py's fallback probe.
+def _local_chrome_listening():
+    for port in (9222, 9223):
+        try:
+            urllib.request.urlopen(f"http://127.0.0.1:{port}/json/version", timeout=0.3).close()
+            return True
+        except OSError: pass
+    return False
+
+
+def main():
+    args = sys.argv[1:]
+    if args and args[0] in {"-h", "--help"}:
+        print(HELP)
+        return
+    if args and args[0] == "--version":
+        print(_version() or "unknown")
+        return
+    if args and args[0] == "--doctor":
+        sys.exit(run_doctor())
+    if args and args[0] == "--setup":
+        sys.exit(run_setup())
+    if args and args[0] == "--update":
+        yes = any(a in {"-y", "--yes"} for a in args[1:])
+        sys.exit(run_update(yes=yes))
+    if args and args[0] == "--reload":
+        restart_daemon()
+        print("daemon stopped — will restart fresh on next call")
+        return
+    if args and args[0] == "--debug-clicks":
+        os.environ["BH_DEBUG_CLICKS"] = "1"
+        args = args[1:]
+    if not args or args[0] != "-c":
+        sys.exit("Usage: browser-harness -c \"print(page_info())\"")
+    if len(args) < 2:
+        sys.exit("Usage: browser-harness -c \"print(page_info())\"")
+    print_update_banner()
+    if not daemon_alive() and not _local_chrome_listening() and os.environ.get("BROWSER_USE_API_KEY"):
+        start_remote_daemon(NAME)
+    ensure_daemon()
+    exec(args[1], globals())
+
+
+if __name__ == "__main__":
+    main()

package/dist/extensions/builtin/debug/index.js

@@ -1,4 +1,4 @@
-var D=Object.defineProperty;var c=(a,e)=>D(a,"name",{value:e,configurable:!0});import{Box as C,Container as E,Spacer as $,Text as P}from"@pencil-agent/tui";import{collectSystemInfo as h,collectModelInfo as y,collectSessionInfo as b,collectConfigInfo as I,collectGitInfo as O,collectAgentState as M,collectPreferencesInfo as U,sanitizeForLLM as _,formatDiagnosticData as L}from"./collectors.js";const S="debug",v="[DEBUG:",A="[DEBUG]",j=`You are a diagnostic analyst for nanoPencil (a terminal-native AI coding agent).
+var D=Object.defineProperty;var c=(r,e)=>D(r,"name",{value:e,configurable:!0});import{Box as C,Container as E,Spacer as $,Text as P}from"@pencil-agent/tui";import{collectSystemInfo as h,collectModelInfo as y,collectSessionInfo as b,collectConfigInfo as I,collectGitInfo as O,collectAgentState as M,collectPreferencesInfo as U,sanitizeForLLM as _,formatDiagnosticData as L}from"./collectors.js";const S="debug",v="[DEBUG:",A="[DEBUG]",j=`You are a diagnostic analyst for nanoPencil (a terminal-native AI coding agent).
 Analyze the provided system state and produce a structured three-layer diagnostic report.
 
 Output format (use these exact section headers):
@@ -28,22 +28,22 @@ Rules:
 - If no specific issue, perform a general health assessment
 - Use concise language; prefer tables and bullet lists over prose
 - If a diagnostic collection failed, treat that failure itself as a diagnostic signal
-- Do NOT use any tools \u2014 this is a pure analysis task`;let p;function z(a){return a.startsWith(v)}c(z,"isDebugPrompt");const k=[{value:"env",label:"env",description:"Show terminal and system details"},{value:"session",label:"session",description:"Show current conversation state"},{value:"model",label:"model",description:"Show active model and provider details"},{value:"preferences",label:"preferences",description:"Show saved preferences"}],N=[{value:"zh",label:"zh",description:"Use Chinese for future replies"},{value:"en",label:"en",description:"Use English for future replies"}];function w(a){const e=a.trim().toLowerCase();return e==="env"?{subcommand:"env"}:e==="session"?{subcommand:"session"}:e==="model"?{subcommand:"model"}:e==="preferences"?{subcommand:"preferences"}:{subcommand:"full",issueDescription:a.trim()||void 0}}c(w,"parseDebugArgs");async function R(a,e,r){const t=w(a);e.ui.setStatus("debug","Collecting diagnostics...");try{const[n,s,o,i,u,l]=await Promise.allSettled([h(),y(e),b(e),I(e),O(e.cwd),M(e)]),m={system:n.status==="fulfilled"?n.value:{data:null,error:String(n.reason)},model:s.status==="fulfilled"?s.value:{data:null,error:String(s.reason)},session:o.status==="fulfilled"?o.value:{data:null,error:String(o.reason)},config:i.status==="fulfilled"?i.value:{data:null,error:String(i.reason)},git:u.status==="fulfilled"?u.value:{data:null,error:String(u.reason)},agent:l.status==="fulfilled"?l.value:{data:null,error:String(l.reason)}},g=_(m);e.ui.setStatus("debug",void 0);const d=[];d.push(`${A} Perform a three-layer diagnostic analysis.`),t.issueDescription&&d.push(`
+- Do NOT use any tools \u2014 this is a pure analysis task`;let p;function k(r){return r.startsWith(v)}c(k,"isDebugPrompt");const z=[{value:"env",label:"env",description:"Show terminal and system details"},{value:"session",label:"session",description:"Show current conversation state"},{value:"model",label:"model",description:"Show active model and provider details"},{value:"preferences",label:"preferences",description:"Show saved preferences"}],N=[{value:"zh",label:"zh",description:"Use Chinese for future replies"},{value:"en",label:"en",description:"Use English for future replies"}];function w(r){const e=r.trim().toLowerCase();return e==="env"?{subcommand:"env"}:e==="session"?{subcommand:"session"}:e==="model"?{subcommand:"model"}:e==="preferences"?{subcommand:"preferences"}:{subcommand:"full",issueDescription:r.trim()||void 0}}c(w,"parseDebugArgs");async function R(r,e,o){const t=w(r);e.ui.setStatus("debug","Collecting diagnostics...");try{const[n,s,a,i,u,l]=await Promise.allSettled([h(),y(e),b(e),I(e),O(e.cwd),M(e)]),m={system:n.status==="fulfilled"?n.value:{data:null,error:String(n.reason)},model:s.status==="fulfilled"?s.value:{data:null,error:String(s.reason)},session:a.status==="fulfilled"?a.value:{data:null,error:String(a.reason)},config:i.status==="fulfilled"?i.value:{data:null,error:String(i.reason)},git:u.status==="fulfilled"?u.value:{data:null,error:String(u.reason)},agent:l.status==="fulfilled"?l.value:{data:null,error:String(l.reason)}},g=_(m);e.ui.setStatus("debug",void 0);const d=[];d.push(`${A} Perform a three-layer diagnostic analysis.`),t.issueDescription&&d.push(`
 User-Reported Issue: ${t.issueDescription}`),d.push(`
 Collected Diagnostics:
 `),d.push(L(g));const f=`${v}${Date.now()}]
 ${d.join(`
-`)}`;p=f,r.sendUserMessage(f,{deliverAs:"followUp"})}catch(n){e.ui.setStatus("debug",void 0);const s=n instanceof Error?n.message:String(n);e.ui.notify(`Debug error: ${s}`,"error")}}c(R,"handleFullDiagnostic");async function T(a,e,r){let t;switch(a){case"env":{const n=await h();t=n.data?`| System | |
+`)}`;p=f,o.sendUserMessage(f,{deliverAs:"followUp"})}catch(n){e.ui.setStatus("debug",void 0);const s=n instanceof Error?n.message:String(n);e.ui.notify(`Debug error: ${s}`,"error")}}c(R,"handleFullDiagnostic");async function T(r,e,o){let t;switch(r){case"env":{const n=await h();t=n.data?`| System | |
 |---|---|
-${Object.entries(n.data).map(([s,o])=>`| ${s} | ${o} |`).join(`
+${Object.entries(n.data).map(([s,a])=>`| ${s} | ${a} |`).join(`
 `)}`:`> Collection failed: ${n.error}`;break}case"session":{const n=await b(e);t=n.data?`| Session | |
 |---|---|
-${Object.entries(n.data).map(([s,o])=>`| ${s} | ${o} |`).join(`
+${Object.entries(n.data).map(([s,a])=>`| ${s} | ${a} |`).join(`
 `)}`:`> Collection failed: ${n.error}`;break}case"model":{const n=await y(e);t=n.data?`| Model | |
 |---|---|
-${Object.entries(n.data).map(([s,o])=>`| ${s} | ${o} |`).join(`
+${Object.entries(n.data).map(([s,a])=>`| ${s} | ${a} |`).join(`
 `)}`:`> Collection failed: ${n.error}`;break}case"preferences":{const n=await U(e);t=n.data?`| Preferences | |
 |---|---|
-${Object.entries(n.data).map(([s,o])=>`| ${s} | ${typeof o=="string"?o:JSON.stringify(o)} |`).join(`
-`)}`:`> Collection failed: ${n.error}`;break}}r.sendMessage({customType:S,content:t,display:!0})}c(T,"handleQuickSub");async function B(a,e,r){const t=w(a);t.subcommand!=="full"?await T(t.subcommand,e,r):await R(a,e,r)}c(B,"handleDebugCommand");async function G(a){a.registerMessageRenderer(S,(e,r,t)=>{const n=typeof e.content=="string"?e.content:e.content.filter(i=>i.type==="text").map(i=>i.text).join(`
-`),s=new C(1,1,i=>t.bg("customMessageBg",i));s.addChild(new P(t.fg("dim",n),0,0));const o=new E;return o.addChild(new $(1)),o.addChild(s),o}),a.on("before_agent_start",e=>{if(!(!z(e.prompt)||e.prompt!==p))return{appendSystemPrompt:j}}),a.on("agent_end",()=>{p&&(p=void 0)}),a.registerCommand("debug",{description:"Check NanoPencil health or investigate an issue. Usage: /debug [env|session|model|preferences|<issue>]",getArgumentCompletions:c((e,r)=>{if(r&&r.tokenIndex>0)return null;const t=e.trim().toLowerCase(),n=k.filter(s=>s.value.startsWith(t));return n.length>0?n:null},"getArgumentCompletions"),handler:c((e,r)=>B(e,r,a),"handler")}),a.registerCommand("set-locale",{description:"Set language preference (/set-locale zh|en)",getArgumentCompletions:c((e,r)=>{if(r&&r.tokenIndex>0)return null;const t=e.trim().toLowerCase(),n=N.filter(s=>s.value.startsWith(t));return n.length>0?n:null},"getArgumentCompletions"),handler:c(async(e,r)=>{const t=e.trim().toLowerCase();if(t!=="zh"&&t!=="en"){r.ui.notify("Usage: /set-locale zh or /set-locale en","info");return}const n=await import("node:os"),s=await import("node:fs"),o=await import("node:path"),i=process.env.NANOMEM_MEMORY_DIR||o.join(n.homedir(),".nanopencil","agent","memory"),u=o.join(i,"preferences.json");try{let l=[];s.existsSync(u)&&(l=JSON.parse(s.readFileSync(u,"utf-8")));const m=l.findIndex(d=>{const f=d.name||"";return f.includes("\u7528\u6237\u504F\u597D")||f.includes("language preference")||f.includes("locale")}),g={id:`set-locale-${Date.now()}`,type:"preference",name:t==="zh"?"\u7528\u6237\u504F\u597D\u4E2D\u6587":"Language Preference (English)",summary:t==="zh"?"\u7528\u6237\u5E0C\u671B\u6211\u7528\u4E2D\u6587\u56DE\u590D":"User prefers English",detail:t==="zh"?"\u7528\u6237\u901A\u8FC7 /set-locale \u547D\u4EE4\u660E\u786E\u8BBE\u7F6E\u8BED\u8A00\u4E3A\u4E2D\u6587":"User explicitly set language to English via /set-locale command",content:t==="zh"?"\u7528\u6237\u5E0C\u671B\u7528\u4E2D\u6587\u56DE\u590D":"User prefers English responses",tags:["locale","language",t==="zh"?"\u4E2D\u6587":"english"],importance:10,strength:1e3,created:new Date().toISOString(),eventTime:new Date().toISOString(),accessCount:0,retention:"core",salience:10,stability:"stable",relations:[]};m>=0?l[m]=g:l.push(g),s.writeFileSync(u,JSON.stringify(l,null,2)),r.ui.notify(`Locale set to ${t==="zh"?"\u4E2D\u6587":"English"}. Restart or run /debug preferences to verify.`,"info")}catch(l){r.ui.notify(`Failed to set locale: ${l}`,"error")}},"handler")})}c(G,"debugExtension");export{G as default};
+${Object.entries(n.data).map(([s,a])=>`| ${s} | ${typeof a=="string"?a:JSON.stringify(a)} |`).join(`
+`)}`:`> Collection failed: ${n.error}`;break}}o.sendMessage({customType:S,content:t,display:!0})}c(T,"handleQuickSub");async function B(r,e,o){const t=w(r);t.subcommand!=="full"?await T(t.subcommand,e,o):await R(r,e,o)}c(B,"handleDebugCommand");async function G(r){r.registerMessageRenderer(S,(e,o,t)=>{const n=typeof e.content=="string"?e.content:e.content.filter(i=>i.type==="text").map(i=>i.text).join(`
+`),s=new C(1,1,i=>t.bg("customMessageBg",i));s.addChild(new P(t.fg("dim",n),0,0));const a=new E;return a.addChild(new $(1)),a.addChild(s),a}),r.on("before_agent_start",e=>{if(!(!k(e.prompt)||e.prompt!==p))return{appendSystemPrompt:j}}),r.on("agent_end",()=>{p&&(p=void 0)}),r.registerCommand("debug",{description:"Check NanoPencil health or investigate an issue. Usage: /debug [env|session|model|preferences|<issue>]",getArgumentCompletions:c((e,o)=>{if(o&&o.tokenIndex>0)return null;const t=e.trim().toLowerCase(),n=z.filter(s=>s.value.startsWith(t));return n.length>0?n:null},"getArgumentCompletions"),handler:c((e,o)=>B(e,o,r),"handler")}),r.registerCommand("set-locale",{description:"Set language preference (/set-locale zh|en)",getArgumentCompletions:c((e,o)=>{if(o&&o.tokenIndex>0)return null;const t=e.trim().toLowerCase(),n=N.filter(s=>s.value.startsWith(t));return n.length>0?n:null},"getArgumentCompletions"),handler:c(async(e,o)=>{const t=e.trim().toLowerCase();if(t!=="zh"&&t!=="en"){o.ui.notify("Usage: /set-locale zh or /set-locale en","info");return}const n=await import("node:os"),s=await import("node:fs"),a=await import("node:path"),i=process.env.NANOMEM_MEMORY_DIR||a.join(n.homedir(),".nanopencil","agent","memory"),u=a.join(i,"preferences.json");try{let l=[];s.existsSync(u)&&(l=JSON.parse(s.readFileSync(u,"utf-8")));const m=l.findIndex(d=>{const f=d.name||"";return f.includes("\u7528\u6237\u504F\u597D")||f.includes("language preference")||f.includes("locale")}),g={id:`set-locale-${Date.now()}`,type:"preference",name:t==="zh"?"\u7528\u6237\u504F\u597D\u4E2D\u6587":"Language Preference (English)",summary:t==="zh"?"\u7528\u6237\u5E0C\u671B\u6211\u7528\u4E2D\u6587\u56DE\u590D":"User prefers English",detail:t==="zh"?"\u7528\u6237\u901A\u8FC7 /set-locale \u547D\u4EE4\u660E\u786E\u8BBE\u7F6E\u8BED\u8A00\u4E3A\u4E2D\u6587":"User explicitly set language to English via /set-locale command",content:t==="zh"?"\u7528\u6237\u5E0C\u671B\u7528\u4E2D\u6587\u56DE\u590D":"User prefers English responses",tags:["locale","language",t==="zh"?"\u4E2D\u6587":"english"],importance:10,strength:1e3,created:new Date().toISOString(),eventTime:new Date().toISOString(),accessCount:0,retention:"core",salience:10,stability:"stable",relations:[]};m>=0?l[m]=g:l.push(g),s.mkdirSync(i,{recursive:!0}),s.writeFileSync(u,JSON.stringify(l,null,2)),o.ui.notify(`Locale set to ${t==="zh"?"\u4E2D\u6587":"English"}. Restart or run /debug preferences to verify.`,"info")}catch(l){o.ui.notify(`Failed to set locale: ${l}`,"error")}},"handler")})}c(G,"debugExtension");export{G as default};

package/dist/extensions/builtin/discipline/skills/brainstorming/SKILL.md

@@ -1,33 +1,33 @@
----
-name: brainstorming
-description: Use before creative or behavioral work when intent, scope, design choices, acceptance criteria, or trade-offs are not already explicit.
----
-
-# Brainstorming
-
-Turn an idea into an implementable design before writing code.
-
-## Gate
-
-Do not edit implementation files until the design is clear enough to state:
-
-- The user-visible goal
-- The constrained scope
-- The recommended approach and at least one rejected alternative
-- The main files or modules likely affected
-- Acceptance criteria and verification commands
-
-For small changes, the design can be short. The gate is clarity, not ceremony.
-
-## Process
-
-1. Explore current project context with read-only tools.
-2. Ask one high-value clarification question at a time only when the answer cannot be inferred safely.
-3. Present 2-3 viable approaches with trade-offs.
-4. Recommend one approach and explain why it fits the existing codebase.
-5. Get user approval before implementation when the change is broad, risky, or changes behavior.
-6. If the design will take multiple implementation steps, transition to `writing-plans`.
-
-## Output Standard
-
-Use evidence from the codebase. State assumptions explicitly. Keep the design actionable enough that another agent could implement it without rediscovering the architecture.
+---
+name: brainstorming
+description: Use before creative or behavioral work when intent, scope, design choices, acceptance criteria, or trade-offs are not already explicit.
+---
+
+# Brainstorming
+
+Turn an idea into an implementable design before writing code.
+
+## Gate
+
+Do not edit implementation files until the design is clear enough to state:
+
+- The user-visible goal
+- The constrained scope
+- The recommended approach and at least one rejected alternative
+- The main files or modules likely affected
+- Acceptance criteria and verification commands
+
+For small changes, the design can be short. The gate is clarity, not ceremony.
+
+## Process
+
+1. Explore current project context with read-only tools.
+2. Ask one high-value clarification question at a time only when the answer cannot be inferred safely.
+3. Present 2-3 viable approaches with trade-offs.
+4. Recommend one approach and explain why it fits the existing codebase.
+5. Get user approval before implementation when the change is broad, risky, or changes behavior.
+6. If the design will take multiple implementation steps, transition to `writing-plans`.
+
+## Output Standard
+
+Use evidence from the codebase. State assumptions explicitly. Keep the design actionable enough that another agent could implement it without rediscovering the architecture.

package/dist/extensions/builtin/discipline/skills/executing-plans/SKILL.md

@@ -1,25 +1,25 @@
----
-name: executing-plans
-description: Use when implementing an approved written plan in the current session.
----
-
-# Executing Plans
-
-Execute the approved plan task by task with verification checkpoints.
-
-## Process
-
-1. Read the plan completely.
-2. Check for contradictions, missing files, missing tests, or unsafe assumptions.
-3. Raise blockers before editing.
-4. For each task:
-   - Mark the task in progress.
-   - Follow the stated steps.
-   - Use `test-driven-development` for behavior changes.
-   - Run the specified verification.
-   - Mark the task complete only after evidence supports it.
-5. After all tasks, use `verification-before-completion`.
-
-## Stop Conditions
-
-Stop when a plan instruction is ambiguous enough to change behavior, a required dependency is missing, verification fails repeatedly, or the plan conflicts with current codebase evidence.
+---
+name: executing-plans
+description: Use when implementing an approved written plan in the current session.
+---
+
+# Executing Plans
+
+Execute the approved plan task by task with verification checkpoints.
+
+## Process
+
+1. Read the plan completely.
+2. Check for contradictions, missing files, missing tests, or unsafe assumptions.
+3. Raise blockers before editing.
+4. For each task:
+   - Mark the task in progress.
+   - Follow the stated steps.
+   - Use `test-driven-development` for behavior changes.
+   - Run the specified verification.
+   - Mark the task complete only after evidence supports it.
+5. After all tasks, use `verification-before-completion`.
+
+## Stop Conditions
+
+Stop when a plan instruction is ambiguous enough to change behavior, a required dependency is missing, verification fails repeatedly, or the plan conflicts with current codebase evidence.

package/dist/extensions/builtin/discipline/skills/finishing-development-branch/SKILL.md

@@ -1,25 +1,25 @@
----
-name: finishing-development-branch
-description: Use after implementation is complete and verified, when deciding how to merge, push, preserve, or discard branch work.
----
-
-# Finishing a Development Branch
-
-Complete branch work with evidence and a clear integration choice.
-
-## Process
-
-1. Run the verification that proves the completed work.
-2. Inspect git status and current branch.
-3. Detect whether the workspace is a normal checkout, linked worktree, or detached/headless workspace.
-4. Present concrete options:
-   - Merge locally
-   - Push and create a PR
-   - Keep branch/worktree as-is
-   - Discard work, with explicit confirmation
-5. Execute the chosen path.
-6. Re-run verification after merge when merging locally.
-
-## Safety
-
-Never discard work without typed confirmation. Never remove a worktree unless you can prove it was created for this workflow and is not owned by the harness.
+---
+name: finishing-development-branch
+description: Use after implementation is complete and verified, when deciding how to merge, push, preserve, or discard branch work.
+---
+
+# Finishing a Development Branch
+
+Complete branch work with evidence and a clear integration choice.
+
+## Process
+
+1. Run the verification that proves the completed work.
+2. Inspect git status and current branch.
+3. Detect whether the workspace is a normal checkout, linked worktree, or detached/headless workspace.
+4. Present concrete options:
+   - Merge locally
+   - Push and create a PR
+   - Keep branch/worktree as-is
+   - Discard work, with explicit confirmation
+5. Execute the chosen path.
+6. Re-run verification after merge when merging locally.
+
+## Safety
+
+Never discard work without typed confirmation. Never remove a worktree unless you can prove it was created for this workflow and is not owned by the harness.

package/dist/extensions/builtin/discipline/skills/receiving-code-review/SKILL.md

@@ -1,22 +1,22 @@
----
-name: receiving-code-review
-description: Use when applying human, agent, or external review feedback.
----
-
-# Receiving Code Review
-
-Treat review as technical input to verify, not a script to follow blindly.
-
-## Process
-
-1. Read all feedback before editing.
-2. Restate unclear items or ask for clarification.
-3. Verify each item against current codebase behavior.
-4. Accept technically valid feedback and implement one item at a time.
-5. Push back with evidence when feedback is incorrect, breaks existing behavior, or violates scope.
-6. Run focused verification after each substantive fix.
-7. Run broader verification before completion.
-
-## YAGNI Check
-
-If feedback asks for "proper" infrastructure, search for actual usage. If nothing uses the path, consider deleting or deferring rather than expanding unused surface area.
+---
+name: receiving-code-review
+description: Use when applying human, agent, or external review feedback.
+---
+
+# Receiving Code Review
+
+Treat review as technical input to verify, not a script to follow blindly.
+
+## Process
+
+1. Read all feedback before editing.
+2. Restate unclear items or ask for clarification.
+3. Verify each item against current codebase behavior.
+4. Accept technically valid feedback and implement one item at a time.
+5. Push back with evidence when feedback is incorrect, breaks existing behavior, or violates scope.
+6. Run focused verification after each substantive fix.
+7. Run broader verification before completion.
+
+## YAGNI Check
+
+If feedback asks for "proper" infrastructure, search for actual usage. If nothing uses the path, consider deleting or deferring rather than expanding unused surface area.

package/dist/extensions/builtin/discipline/skills/requesting-code-review/SKILL.md

@@ -1,31 +1,31 @@
----
-name: requesting-code-review
-description: Use after meaningful implementation tasks, before merging, or when a fresh review would reduce risk.
----
-
-# Requesting Code Review
-
-Ask for review with precise context and explicit requirements.
-
-## Review Packet
-
-Provide the reviewer:
-
-- Summary of the intended change
-- Requirements or plan being checked
-- Files changed
-- Base and head SHAs when available
-- Verification already run
-- Known risks or areas needing attention
-
-## Review Standard
-
-Ask the reviewer to lead with findings ordered by severity. Findings need file and line references, a concrete failure mode, and a suggested correction. Require distinction between:
-
-- Critical: must fix before proceeding
-- Important: should fix before merge
-- Minor: optional improvement
-
-## After Review
-
-Use `receiving-code-review` before applying feedback.
+---
+name: requesting-code-review
+description: Use after meaningful implementation tasks, before merging, or when a fresh review would reduce risk.
+---
+
+# Requesting Code Review
+
+Ask for review with precise context and explicit requirements.
+
+## Review Packet
+
+Provide the reviewer:
+
+- Summary of the intended change
+- Requirements or plan being checked
+- Files changed
+- Base and head SHAs when available
+- Verification already run
+- Known risks or areas needing attention
+
+## Review Standard
+
+Ask the reviewer to lead with findings ordered by severity. Findings need file and line references, a concrete failure mode, and a suggested correction. Require distinction between:
+
+- Critical: must fix before proceeding
+- Important: should fix before merge
+- Minor: optional improvement
+
+## After Review
+
+Use `receiving-code-review` before applying feedback.

package/dist/extensions/builtin/discipline/skills/systematic-debugging/SKILL.md

@@ -1,28 +1,28 @@
----
-name: systematic-debugging
-description: Use for any bug, test failure, build failure, performance issue, flaky behavior, or unexpected runtime result before proposing fixes.
----
-
-# Systematic Debugging
-
-Find the root cause before changing code.
-
-## Gate
-
-No fixes before root-cause investigation. A patch that only explains the symptom is not ready.
-
-## Process
-
-1. Read the exact error, stack trace, failing assertion, or observed behavior.
-2. Reproduce the issue or document why reproduction is not currently possible.
-3. Check recent changes and nearby working examples.
-4. Trace the bad value, state, or control flow backward to its source.
-5. Form one explicit hypothesis: "X causes Y because Z."
-6. Test the hypothesis with the smallest evidence-gathering action.
-7. Add a failing regression test or minimal reproduction before the fix when the codebase can support it.
-8. Implement one root-cause fix.
-9. Run focused verification, then broader regression checks as risk requires.
-
-## Stop Conditions
-
-Stop and re-analyze when a hypothesis fails. If three fix attempts fail or each fix reveals a new unrelated problem, question the architecture instead of stacking patches.
+---
+name: systematic-debugging
+description: Use for any bug, test failure, build failure, performance issue, flaky behavior, or unexpected runtime result before proposing fixes.
+---
+
+# Systematic Debugging
+
+Find the root cause before changing code.
+
+## Gate
+
+No fixes before root-cause investigation. A patch that only explains the symptom is not ready.
+
+## Process
+
+1. Read the exact error, stack trace, failing assertion, or observed behavior.
+2. Reproduce the issue or document why reproduction is not currently possible.
+3. Check recent changes and nearby working examples.
+4. Trace the bad value, state, or control flow backward to its source.
+5. Form one explicit hypothesis: "X causes Y because Z."
+6. Test the hypothesis with the smallest evidence-gathering action.
+7. Add a failing regression test or minimal reproduction before the fix when the codebase can support it.
+8. Implement one root-cause fix.
+9. Run focused verification, then broader regression checks as risk requires.
+
+## Stop Conditions
+
+Stop and re-analyze when a hypothesis fails. If three fix attempts fail or each fix reveals a new unrelated problem, question the architecture instead of stacking patches.