clikit-plugin 0.2.3 → 0.2.4

package/dist/cli.js

@@ -61,18 +61,6 @@ function parseConfig(configPath) {
     return { config: {}, raw, parseError: String(err) };
   }
 }
-function backupConfig(configPath) {
-  try {
-    if (!fs.existsSync(configPath)) {
-      return null;
-    }
-    const backupPath = `${configPath}.backup-${Date.now()}`;
-    fs.copyFileSync(configPath, backupPath);
-    return backupPath;
-  } catch {
-    return null;
-  }
-}
 function writeConfig(configPath, config) {
   ensureConfigDir();
   const tmpPath = `${configPath}.tmp`;
@@ -95,16 +83,11 @@ async function install() {
   }
   const configPath = getConfigPath();
   try {
-    const backupPath = backupConfig(configPath);
-    if (backupPath) {
-      console.log(`  Backup created: ${backupPath}`);
-    }
     const result = parseConfig(configPath);
     if (result.parseError && result.raw.trim()) {
       console.error(`\u2717 Config file has syntax errors and cannot be safely modified.`);
       console.error(`  Error: ${result.parseError}`);
-      console.error(`  Please fix the config file manually or restore from backup.`);
-      console.error(`  Backup: ${backupPath}`);
+      console.error(`  Please fix the config file manually.`);
       return 1;
     }
     const config = result.config;

package/dist/hooks/todo-enforcer.d.ts

@@ -14,6 +14,6 @@ export interface TodoCheckResult {
     incomplete: TodoItem[];
     inProgress: TodoItem[];
 }
-export declare function checkTodoCompletion(todos: TodoItem[]): TodoCheckResult;
+export declare function checkTodoCompletion(todos: unknown): TodoCheckResult;
 export declare function formatIncompleteWarning(result: TodoCheckResult, sessionId?: string): string;
 //# sourceMappingURL=todo-enforcer.d.ts.map

package/dist/hooks/todo-enforcer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"todo-enforcer.d.ts","sourceRoot":"","sources":["../../src/hooks/todo-enforcer.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,MAAM,WAAW,QAAQ;IACvB,EAAE,EAAE,MAAM,CAAC;IACX,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,GAAG,aAAa,GAAG,WAAW,CAAC;CAC9C;AAED,MAAM,WAAW,eAAe;IAC9B,QAAQ,EAAE,OAAO,CAAC;IAClB,UAAU,EAAE,QAAQ,EAAE,CAAC;IACvB,UAAU,EAAE,QAAQ,EAAE,CAAC;CACxB;AAED,wBAAgB,mBAAmB,CAAC,KAAK,EAAE,QAAQ,EAAE,GAAG,eAAe,CAStE;AAED,wBAAgB,uBAAuB,CACrC,MAAM,EAAE,eAAe,EACvB,SAAS,CAAC,EAAE,MAAM,GACjB,MAAM,CAuBR"}
+{"version":3,"file":"todo-enforcer.d.ts","sourceRoot":"","sources":["../../src/hooks/todo-enforcer.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,MAAM,WAAW,QAAQ;IACvB,EAAE,EAAE,MAAM,CAAC;IACX,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,GAAG,aAAa,GAAG,WAAW,CAAC;CAC9C;AAED,MAAM,WAAW,eAAe;IAC9B,QAAQ,EAAE,OAAO,CAAC;IAClB,UAAU,EAAE,QAAQ,EAAE,CAAC;IACvB,UAAU,EAAE,QAAQ,EAAE,CAAC;CACxB;AAED,wBAAgB,mBAAmB,CAAC,KAAK,EAAE,OAAO,GAAG,eAAe,CAYnE;AAED,wBAAgB,uBAAuB,CACrC,MAAM,EAAE,eAAe,EACvB,SAAS,CAAC,EAAE,MAAM,GACjB,MAAM,CAuBR"}

package/dist/index.js

@@ -3761,6 +3761,9 @@ function filterCommands(commands, config) {
 
 // src/hooks/todo-enforcer.ts
 function checkTodoCompletion(todos) {
+  if (!Array.isArray(todos)) {
+    return { complete: true, incomplete: [], inProgress: [] };
+  }
   const incomplete = todos.filter((t) => t.status === "todo");
   const inProgress = todos.filter((t) => t.status === "in-progress");
   return {
@@ -5588,7 +5591,7 @@ var CliKitPlugin = async (ctx) => {
         const todoConfig = pluginConfig.hooks?.todo_enforcer;
         if (todoConfig?.enabled !== false) {
           const todos = props?.todos;
-          if (todos && todos.length > 0) {
+          if (Array.isArray(todos) && todos.length > 0) {
             const result = checkTodoCompletion(todos);
             if (!result.complete && todoConfig?.warn_on_incomplete !== false) {
               console.warn(formatIncompleteWarning(result, sessionID));

package/memory/handoffs/2026-02-15-runtime-fixes.md

@@ -0,0 +1,80 @@
+---
+date: 2026-02-15
+phase: implementing
+branch: main
+---
+
+# Handoff: CliKit Runtime Bug Fixes
+
+---
+
+## Status Summary
+
+CliKit plugin v0.2.3 published with installer merge fix and type guards in hooks. **NEW ISSUES IDENTIFIED**: 
+1. Backup creation is now unnecessary since merge works correctly - should be removed
+2. Runtime error `todos.filter is not a function` when OpenCode starts
+
+---
+
+## Task Status
+
+### ✅ Completed
+- [x] Fix installer to MERGE with existing opencode.json (v0.2.3)
+- [x] Add defensive type checks to hooks (truncator, security-check, compaction)
+- [x] Publish v0.2.1, v0.2.2, v0.2.3 to npm
+
+### 🔄 In Progress
+- [ ] Remove backup creation from cli.ts
+- [ ] Fix `todos.filter is not a function` runtime error
+
+### 📋 Not Started
+- [ ] Publish v0.2.4
+
+---
+
+## Files Modified
+
+| File | Status | Notes |
+|------|--------|-------|
+| `.opencode/src/cli.ts` | Needs Fix | Remove backup logic |
+| `.opencode/src/index.ts` | Needs Fix | Add Array.isArray check |
+| `.opencode/src/hooks/todo-enforcer.ts` | Needs Fix | Add array guard |
+
+---
+
+## Git State
+
+- **Branch:** `main`
+- **Last commit:** `31b200c` - Update README with simplified installation
+- **Uncommitted changes:** Yes (8 modified files)
+
+---
+
+## Next Steps
+
+1. [ ] Remove backup logic from cli.ts (delete `backupConfig`, `cleanupOldBackups` functions)
+2. [ ] Fix `todos.filter` error in `index.ts:172` and `todo-enforcer.ts:20`
+3. [ ] Run typecheck
+4. [ ] Publish v0.2.4
+
+---
+
+## Root Cause: `todos.filter` Bug
+
+```typescript
+// Current (broken) - index.ts:166-172
+const todos = props?.todos as Array<{...}> | undefined;
+if (todos && todos.length > 0) {  // Objects with .length pass this!
+  checkTodoCompletion(todos);      // But .filter fails
+}
+
+// Fix
+if (Array.isArray(todos) && todos.length > 0) {
+  checkTodoCompletion(todos);
+}
+```
+
+**Key files:**
+- `.opencode/src/cli.ts:78-107` — Backup functions to delete
+- `.opencode/src/index.ts:165-180` — Todo handling to fix
+- `.opencode/src/hooks/todo-enforcer.ts:20` — Add array guard

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "clikit-plugin",
-  "version": "0.2.3",
+  "version": "0.2.4",
   "description": "OpenCode plugin with 10 agents, 19 commands, 48 skills, 14 hooks",
   "type": "module",
   "main": "./dist/index.js",