opencode-swarm-plugin 0.31.7 → 0.32.0

package/CHANGELOG.md

@@ -1,5 +1,139 @@
 # opencode-swarm-plugin
 
+## 0.32.0
+
+### Minor Changes
+
+- [#54](https://github.com/joelhooks/swarm-tools/pull/54) [`358e18f`](https://github.com/joelhooks/swarm-tools/commit/358e18f0f7f18d03492ef16c2c1d3edd85c00101) Thanks [@joelhooks](https://github.com/joelhooks)! - ## 🔍 Coordinator Review Gate + UBS Removal
+
+  > _"This asynchronous back and forth between submitter and reviewer can add days to the process of getting changes made. Do Code Reviews Promptly!"_
+  > — Sam Newman, _Building Microservices_
+
+  Two changes that make swarm coordination tighter:
+
+  ### Coordinator Review Tools
+
+  New tools for coordinators to review worker output before approval:
+
+  ```
+  ┌─────────────────────────────────────────────────────┐
+  │              COORDINATOR REVIEW FLOW                │
+  ├─────────────────────────────────────────────────────┤
+  │  1. Worker completes → sends completion message     │
+  │  2. Coordinator: swarm_review(task_id, files)       │
+  │     → Gets diff + epic context + review prompt      │
+  │  3. Coordinator reviews against epic goals          │
+  │  4. swarm_review_feedback(status, issues)           │
+  │     → approved: worker can finalize                 │
+  │     → needs_changes: worker gets feedback           │
+  │  5. 3-strike rule: 3 rejections = blocked           │
+  └─────────────────────────────────────────────────────┘
+  ```
+
+  **New tools:**
+
+  - `swarm_review` - Generate review prompt with epic context + git diff
+  - `swarm_review_feedback` - Send approval/rejection with structured issues
+
+  **Updated prompts:**
+
+  - Coordinator prompt now includes review checklist
+  - Worker prompt explains the review gate
+  - Skills updated with review patterns
+
+  ### UBS Scan Removed from swarm_complete
+
+  The `skip_ubs_scan` parameter is gone. UBS was already disabled in v0.31 for performance - this cleans up the vestigial code.
+
+  **Removed:**
+
+  - `skip_ubs_scan` parameter from schema
+  - `ubs_scan` deprecation object from output
+  - All UBS-related helper functions
+  - ~100 lines of dead code
+
+  **If you need UBS scanning:** Run it manually before commit:
+
+  ```bash
+  ubs scan src/
+  ```
+
+  ### CLI Improvements
+
+  The `swarm` CLI got smarter:
+
+  - Better error messages for missing dependencies
+  - Cleaner output formatting
+  - Improved help text
+
+### Patch Changes
+
+- [#54](https://github.com/joelhooks/swarm-tools/pull/54) [`358e18f`](https://github.com/joelhooks/swarm-tools/commit/358e18f0f7f18d03492ef16c2c1d3edd85c00101) Thanks [@joelhooks](https://github.com/joelhooks)! - ## 🧪 Integration Test Coverage: 0% → 95%
+
+  > _"Many characterization tests look like 'sunny day' tests. They don't test many special conditions; they just verify that particular behaviors are present. From their presence, we can infer that refactoring hasn't broken anything."_
+  > — Michael Feathers, _Working Effectively with Legacy Code_
+
+  We had a bug that broke ALL swarm tools:
+
+  ```
+  Error: [streams/store] dbOverride parameter is required for this function.
+  PGlite getDatabase() has been removed.
+  ```
+
+  **Why didn't tests catch it?** No integration tests exercised the full tool → store → DB path.
+
+  **Now they do.**
+
+  ```
+  ┌─────────────────────────────────────────────────────────────────┐
+  │              tool-adapter.integration.test.ts                   │
+  ├─────────────────────────────────────────────────────────────────┤
+  │  20 tests | 75 assertions | 1.3s                                │
+  │                                                                 │
+  │  ✅ swarmmail_* tools (6 tests)                                 │
+  │  ✅ hive_* tools (7 tests)                                      │
+  │  ✅ swarm_progress, swarm_status (2 tests)                      │
+  │  ✅ swarm_broadcast, swarm_checkpoint (2 tests)                 │
+  │  ✅ semantic_memory_store, semantic_memory_find (2 tests)       │
+  │  ✅ Smoke test - 9 tools in sequence (1 test)                   │
+  └─────────────────────────────────────────────────────────────────┘
+  ```
+
+  ### What's Tested
+
+  Each test calls `tool.execute()` and verifies:
+
+  1. No "dbOverride required" error (the bug symptom)
+  2. Tool returns expected structure
+  3. Full path works: tool → store → DB → response
+
+  ### The Smoke Test
+
+  Runs 9 tools in sequence to catch interaction bugs:
+
+  ```
+  swarmmail_init → hive_create → swarmmail_reserve → swarm_progress
+  → semantic_memory_store → semantic_memory_find → swarmmail_send
+  → hive_close → swarmmail_release
+  ```
+
+  If ANY step throws "dbOverride required", the test fails.
+
+  ### Also Fixed
+
+  - **Auto-adapter creation** in store.ts - functions now auto-create adapters when not provided
+  - **Exported `clearAdapterCache()`** for test isolation
+  - **Migrated test files** from old `getDatabase()` to adapter pattern
+
+  ### Mandatory Coordinator Review Loop
+
+  Added `COORDINATOR_POST_WORKER_CHECKLIST` constant and `post_completion_instructions` field to `swarm_spawn_subtask`. Coordinators now get explicit instructions to review worker output before spawning the next worker.
+
+  The "dbOverride required" bug **cannot recur undetected**.
+
+- Updated dependencies [[`358e18f`](https://github.com/joelhooks/swarm-tools/commit/358e18f0f7f18d03492ef16c2c1d3edd85c00101), [`358e18f`](https://github.com/joelhooks/swarm-tools/commit/358e18f0f7f18d03492ef16c2c1d3edd85c00101), [`358e18f`](https://github.com/joelhooks/swarm-tools/commit/358e18f0f7f18d03492ef16c2c1d3edd85c00101), [`358e18f`](https://github.com/joelhooks/swarm-tools/commit/358e18f0f7f18d03492ef16c2c1d3edd85c00101)]:
+  - swarm-mail@1.3.0
+
 ## 0.31.7
 
 ### Patch Changes

package/README.md

@@ -27,6 +27,9 @@ ollama serve &
 ollama pull mxbai-embed-large
 ```
 
+> *"High-variability sequencing of whole-task problems."*  
+> — 4C/ID Instructional Design Model
+
 ## Features
 
 - **Swarm Coordination** - Break tasks into parallel subtasks, spawn worker agents
@@ -85,7 +88,7 @@ ollama pull mxbai-embed-large
 | `swarm_init`                   | Initialize swarm session                        |
 | `swarm_status`                 | Get swarm progress by epic ID                   |
 | `swarm_progress`               | Report subtask progress to coordinator          |
-| `swarm_complete`               | Complete subtask (runs UBS scan, releases)      |
+| `swarm_complete`               | Complete subtask (releases reservations)        |
 | `swarm_record_outcome`         | Record outcome for learning                     |
 | `swarm_checkpoint`             | Save progress snapshot (auto at 25/50/75%)      |
 | `swarm_recover`                | Resume from checkpoint (returns full context)   |
@@ -111,10 +114,10 @@ Ensures work survives context compaction or crashes. Proven by 9 integration tes
 
 ### Auto-Checkpoint Milestones
 
-When `swarm_progress` reports 25%, 50%, or 75% completion, a checkpoint is automatically saved to PGLite:
+When `swarm_progress` reports 25%, 50%, or 75% completion, a checkpoint is automatically saved to libSQL:
 
 ```typescript
-// Stored in .swarm-mail/ directory (no external database needed)
+// Stored in system temp directory (no external database needed)
 {
   epic_id: "hv-123",
   cell_id: "hv-123.1",
@@ -247,7 +250,7 @@ pip install -e .
 **Why install these?**
 
 - **CASS** - When you run `/swarm "Add OAuth"`, the coordinator queries CASS for similar past tasks. Without it, decomposition is based only on the current task description.
-- **UBS** - Every `swarm_complete` runs UBS to scan for bugs. Without it, you lose automatic bug detection.
+- **UBS** - Run manually on code to catch bugs before commit. Not integrated into swarm workflow but recommended for pre-commit validation.
 - **Ollama** - Enables vector similarity search for semantic memory. Without it, memory falls back to full-text search (still functional, less semantic).
 
 Run `swarm doctor` to check which dependencies are installed.