clikit-plugin 0.2.9 → 0.2.11

package/dist/cli.js

@@ -6,7 +6,7 @@ import * as fs from "fs";
 import * as path from "path";
 import * as os from "os";
 var PLUGIN_NAME = "clikit-plugin";
-var VERSION = "0.2.0";
+var VERSION = "0.2.11";
 function getRealHome() {
   if (process.env.SNAP_REAL_HOME) {
     return process.env.SNAP_REAL_HOME;

package/dist/index.js

@@ -3495,8 +3495,7 @@ var require_gray_matter = __commonJS((exports, module) => {
 var import_gray_matter = __toESM(require_gray_matter(), 1);
 import * as fs from "fs";
 import * as path from "path";
-var __dirname = "/mnt/c/Users/Kira/Desktop/clikit/.opencode/src/agents";
-var AGENTS_DIR = __dirname;
+var AGENTS_DIR = path.join(import.meta.dir, "../src/agents");
 function parseAgentMarkdown(filePath) {
   try {
     const content = fs.readFileSync(filePath, "utf-8");
@@ -3549,8 +3548,7 @@ function getBuiltinAgents() {
 var import_gray_matter2 = __toESM(require_gray_matter(), 1);
 import * as fs2 from "fs";
 import * as path2 from "path";
-var __dirname = "/mnt/c/Users/Kira/Desktop/clikit/.opencode/src/commands";
-var COMMANDS_DIR = path2.join(__dirname, "../../command");
+var COMMANDS_DIR = path2.join(import.meta.dir, "../command");
 function parseCommandMarkdown(filePath) {
   try {
     const content = fs2.readFileSync(filePath, "utf-8");
@@ -4791,8 +4789,7 @@ var RITUAL_FILE = path8.join(process.cwd(), ".opencode", "memory", "ritual-state
 var import_gray_matter3 = __toESM(require_gray_matter(), 1);
 import * as fs8 from "fs";
 import * as path9 from "path";
-var __dirname = "/mnt/c/Users/Kira/Desktop/clikit/.opencode/src/skills";
-var SKILLS_DIR = path9.join(__dirname, "..", "..", "skill");
+var SKILLS_DIR = path9.join(import.meta.dir, "../skill");
 function getBuiltinSkills() {
   const skills = {};
   if (!fs8.existsSync(SKILLS_DIR)) {

package/memory/handoffs/2026-02-15-complete-fix.md

@@ -0,0 +1,140 @@
+---
+date: 2026-02-15
+phase: implementing
+branch: main
+---
+
+# Handoff: CliKit Complete Type Guard & SQLite Fix
+
+---
+
+## Status Summary
+
+CliKit plugin v0.2.1 → v0.2.9 published with comprehensive type guards across all hooks and tools. Fixed SQLite schema migration issues, removed backup logic, and added defensive programming patterns throughout the codebase. Plugin should now handle all edge cases where OpenCode passes unexpected data types.
+
+---
+
+## Version History
+
+| Version | Fix |
+|---------|-----|
+| 0.2.1 | Type guards in hooks (truncator, security-check, compaction) |
+| 0.2.2 | Abort on JSON parse error |
+| 0.2.3 | Fix JSONC comment stripping breaking URLs |
+| 0.2.4 | Remove backup logic, fix `todos.filter` error |
+| 0.2.5 | Fix `path.join` error - guards for projectDir/cwd params |
+| 0.2.6 | Fix `text.toLowerCase` error in subagent-question-blocker |
+| 0.2.7 | Fix remaining type guards (git-guard, swarm-enforcer) |
+| 0.2.8 | SQLite migration - add missing columns |
+| 0.2.9 | Type guards for ALL tool functions |
+
+---
+
+## Task Status
+
+### ✅ Completed
+- [x] Fix installer to MERGE with existing opencode.json
+- [x] Remove backup creation from cli.ts
+- [x] Add type guards to all hooks (14 hooks)
+- [x] Add type guards to all tools (6 tool files)
+- [x] Fix SQLite schema migration with ALTER TABLE
+- [x] Publish v0.2.1 → v0.2.9 to npm
+
+### 📋 Not Started
+- [ ] Test: `opencode` starts without errors
+- [ ] Test: Plugin loads correctly after fresh install
+
+---
+
+## Files Modified (22 files)
+
+| File | Changes |
+|------|---------|
+| `package.json` | v0.2.9 |
+| `cli.ts` | Removed backup, fixed JSONC parsing |
+| `config.ts` | Type guard for loadCliKitConfig |
+| `index.ts` | Array.isArray for todos |
+| `hooks/auto-format.ts` | Type guards for filePath, projectDir |
+| `hooks/comment-checker.ts` | Type guard for content |
+| `hooks/compaction.ts` | Type guards for projectDir, raw content |
+| `hooks/env-context.ts` | Type guard for cwd |
+| `hooks/git-guard.ts` | Type guard for command |
+| `hooks/security-check.ts` | Type guard for content |
+| `hooks/session-notification.ts` | Type guards for sessionId, error |
+| `hooks/subagent-question-blocker.ts` | Type guards for text, toolName |
+| `hooks/swarm-enforcer.ts` | Type guards for filePath, toolName |
+| `hooks/todo-enforcer.ts` | Type guard for todos array |
+| `hooks/truncator.ts` | Type guard for content |
+| `hooks/typecheck-gate.ts` | Type guards for filePath, projectDir, output |
+| `tools/observation.ts` | Type guard + SQLite migration |
+| `tools/memory.ts` | Type guards + SQLite migration |
+| `tools/beads-memory-sync.ts` | Type guard + SQLite migration |
+| `tools/context-summary.ts` | Type guard + SQLite migration |
+| `tools/quick-research.ts` | Type guard for params |
+| `tools/swarm.ts` | Type guard for params |
+
+---
+
+## Git State
+
+- **Branch:** `main`
+- **Last commit:** `31b200c` - Update README with simplified installation
+- **Uncommitted changes:** 22 modified files, 5 handoff files
+
+---
+
+## Known Issues
+
+None - all reported runtime errors have been addressed with type guards.
+
+---
+
+## Next Steps
+
+1. [ ] User test: `rm -rf ~/.cache/opencode/node_modules/clikit-plugin`
+2. [ ] User test: `bun x clikit-plugin@latest install`
+3. [ ] User test: `opencode` - should start without errors
+4. [ ] If errors persist, check error message and add appropriate type guard
+
+---
+
+## Context for Resumption
+
+### Type Guard Pattern Used Throughout
+
+```typescript
+export function myFunc(params: unknown): Result | null {
+  if (!params || typeof params !== "object") {
+    return defaultResult;
+  }
+  const p = params as Partial<Params>;
+  if (!p.requiredField) {
+    return defaultResult;
+  }
+  // ... rest of function
+}
+```
+
+### SQLite Migration Pattern
+
+```typescript
+// Create table with all columns
+db.exec(`CREATE TABLE IF NOT EXISTS observations (...)`);
+
+// Migration: add missing columns if they don't exist
+try { db.exec(`ALTER TABLE observations ADD COLUMN new_col TEXT`); } catch {}
+```
+
+### Key Files to Review
+- `.opencode/src/hooks/*.ts` — All hooks have type guards
+- `.opencode/src/tools/*.ts` — All tools have type guards + SQLite migration
+- `.opencode/src/cli.ts` — Installer logic, JSONC parsing
+
+---
+
+## Resume Command
+
+To resume, use `/resume` and it will:
+1. Load this handoff
+2. Check for new runtime errors
+3. Propose adding appropriate type guards if needed

package/memory/handoffs/2026-02-15-importmeta-fix.md

@@ -0,0 +1,131 @@
+---
+date: 2026-02-15
+phase: verifying
+branch: main
+---
+
+# Handoff: CliKit `import.meta.dir` Path Fix (v0.2.10)
+
+---
+
+## Status Summary
+
+Fixed critical bug where bun build inlined `__dirname` to absolute build-time paths, causing plugin to fail on user machines. Replaced with `import.meta.dir` which is preserved as runtime expression.
+
+---
+
+## Bug Description
+
+**Problem:** After `bun build`, the bundled code had hardcoded paths:
+
+```javascript
+// Before (broken):
+var __dirname = "/mnt/c/Users/Kira/Desktop/clikit/.opencode/src/agents";
+```
+
+When users installed via npm, the plugin looked for files at the BUILD machine's path, not the installed location.
+
+**Root Cause:** Bun bundler inlines `__dirname` to the build-time absolute path. `import.meta.dir` is preserved as a runtime expression.
+
+---
+
+## Fix Applied
+
+| File | Before | After |
+|------|--------|-------|
+| `src/agents/index.ts:6` | `const AGENTS_DIR = __dirname;` | `const AGENTS_DIR = import.meta.dir;` |
+| `src/commands/index.ts:6` | `path.join(__dirname, "../../command")` | `path.join(import.meta.dir, "../../command")` |
+| `src/skills/index.ts:5` | `path.join(__dirname, "..", "..", "skill")` | `path.join(import.meta.dir, "../../skill")` |
+
+---
+
+## Verification
+
+```bash
+# Published package shows runtime expressions (not inlined):
+npm pack clikit-plugin@latest && tar -xzf clikit-plugin-0.2.10.tgz
+grep import.meta.dir package/dist/index.js
+
+# Output:
+# var AGENTS_DIR = import.meta.dir;
+# var COMMANDS_DIR = path2.join(import.meta.dir, "../../command");
+# var SKILLS_DIR = path9.join(import.meta.dir, "../../skill");
+```
+
+---
+
+## Version History
+
+| Version | Fix |
+|---------|-----|
+| 0.2.1-0.2.9 | Type guards for hooks/tools |
+| 0.2.10 | `__dirname` → `import.meta.dir` path fix |
+
+---
+
+## Next Steps
+
+1. [ ] User test: `rm -rf ~/.cache/opencode/node_modules/clikit-plugin`
+2. [ ] User test: `bun x clikit-plugin@latest install`
+3. [ ] User test: `opencode` - should start without errors
+4. [ ] Verify agents/commands/skills load correctly
+
+---
+
+## Files Modified (3 files)
+
+| File | Changes |
+|------|---------|
+| `src/agents/index.ts` | `__dirname` → `import.meta.dir` |
+| `src/commands/index.ts` | `__dirname` → `import.meta.dir` |
+| `src/skills/index.ts` | `__dirname` → `import.meta.dir` |
+| `package.json` | v0.2.10 |
+| `src/cli.ts` | VERSION = "0.2.10" |
+
+---
+
+## Git State
+
+- **Branch:** `main`
+- **Uncommitted changes:** All previous fixes + this fix
+
+---
+
+## Context for Resumption
+
+### Why `import.meta.dir` Works
+
+Bun's bundler treats `import.meta.dir` specially - it's preserved as a runtime expression that evaluates to the actual file's directory at runtime, not build time. This is standard ESM behavior.
+
+### Path Resolution After Fix
+
+```
+Installed location: ~/.cache/opencode/node_modules/clikit-plugin/
+
+dist/index.js
+├── import.meta.dir → ~/.cache/opencode/node_modules/clikit-plugin/dist/
+├── AGENTS_DIR = import.meta.dir → ~/.cache/.../dist/ (contains compiled index.js)
+│   └── Resolved from src/agents/*.md via package.json "files": ["src/agents"]
+├── COMMANDS_DIR = import.meta.dir + "../../command" → ~/.cache/.../command/
+└── SKILLS_DIR = import.meta.dir + "../../skill" → ~/.cache/.../skill/
+```
+
+Note: `AGENTS_DIR` points to `dist/` which doesn't contain `.md` files directly. The `src/agents` directory IS included in package.json `files` array, but the path resolution may need adjustment.
+
+### Potential Issue
+
+Looking at the package structure:
+- `dist/index.js` is the entry point
+- `import.meta.dir` in `dist/index.js` evaluates to `.../clikit-plugin/dist/`
+- But `src/agents/*.md` files are at `.../clikit-plugin/src/agents/`
+
+The agents directory resolution may need adjustment. Check if agents load correctly after install.
+
+---
+
+## Resume Command
+
+To resume, use `/resume` and verify:
+1. Plugin starts without errors
+2. Agents load (check `[CliKit] Loaded X agents` log)
+3. Commands load (check `[CliKit] Loaded X commands` log)

package/package.json

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

package/src/agents/index.ts

@@ -3,7 +3,7 @@ import * as fs from "fs";
 import * as path from "path";
 import matter from "gray-matter";
 
-const AGENTS_DIR = __dirname;
+const AGENTS_DIR = path.join(import.meta.dir, "../src/agents");
 
 function parseAgentMarkdown(filePath: string): AgentConfig | null {
   try {