ultra-builder-pro-cli 0.7.0 → 0.9.0

package/mcp-server/lib/state-db.test.cjs

@@ -9,11 +9,13 @@ const path = require('node:path');
 const {
   EXPECTED_VERSION,
   REQUIRED_TABLES,
+  SCHEMA_FILE,
   initStateDb,
   closeStateDb,
   openStateDb,
   tableNames,
 } = require('./state-db.cjs');
+const stateOps = require('./state-ops.cjs');
 
 function tmpDbPath(prefix = 'ubp-state') {
   const dir = fs.mkdtempSync(path.join(os.tmpdir(), `${prefix}-`));
@@ -114,3 +116,57 @@ test('Phase 8A.1 schema: tasks.parent_id column + tasks_parent partial index + s
     fs.rmSync(dir, { recursive: true, force: true });
   }
 });
+
+test('initStateDb migrates existing runtime constraints to Kimi without losing references', () => {
+  const { dir, file } = tmpDbPath('ubp-kimi-runtime-upgrade');
+  try {
+    const legacy = openStateDb(file);
+    const legacySchema = fs.readFileSync(SCHEMA_FILE, 'utf8').replaceAll(", 'kimi'", '');
+    legacy.exec(legacySchema);
+    legacy.prepare("DELETE FROM schema_version WHERE version = '9.1'").run();
+    legacy.prepare(
+      "INSERT INTO tasks (id, title, type, priority) VALUES ('task-old', 'Old', 'feature', 'P1')",
+    ).run();
+    legacy.prepare(
+      `INSERT INTO sessions
+       (sid, task_id, runtime, worktree_path, artifact_dir, lease_expires_at)
+       VALUES ('session-old', 'task-old', 'codex', '/tmp/worktree', '/tmp/artifacts', '2099-01-01T00:00:00.000Z')`,
+    ).run();
+    legacy.prepare(
+      "INSERT INTO telemetry (session_id, event_type, tool_name) VALUES ('session-old', 'tool_call', 'task.list')",
+    ).run();
+    legacy.prepare(
+      `INSERT INTO incidents
+       (id, code, severity, message, session_id)
+       VALUES ('incident-old', 'OLD', 'warning', 'preserve me', 'session-old')`,
+    ).run();
+    legacy.prepare(
+      "INSERT INTO events (type, session_id, runtime) VALUES ('session_spawned', 'session-old', 'codex')",
+    ).run();
+    closeStateDb(legacy);
+
+    const upgraded = initStateDb(file);
+    assert.equal(upgraded.schema_version, '9.1');
+    assert.equal(upgraded.db.prepare("SELECT runtime FROM sessions WHERE sid = 'session-old'").get().runtime, 'codex');
+    assert.equal(upgraded.db.prepare("SELECT COUNT(*) AS n FROM telemetry WHERE session_id = 'session-old'").get().n, 1);
+    assert.equal(upgraded.db.prepare("SELECT COUNT(*) AS n FROM incidents WHERE session_id = 'session-old'").get().n, 1);
+    assert.deepEqual(upgraded.db.pragma('foreign_key_check'), []);
+
+    upgraded.db.prepare(
+      "INSERT INTO tasks (id, title, type, priority) VALUES ('task-kimi', 'Kimi', 'feature', 'P1')",
+    ).run();
+    const session = stateOps.createSession(upgraded.db, {
+      sid: 'session-kimi',
+      task_id: 'task-kimi',
+      runtime: 'kimi',
+      worktree_path: '/tmp/kimi-worktree',
+      artifact_dir: '/tmp/kimi-artifacts',
+    });
+    assert.equal(session.runtime, 'kimi');
+    const event = stateOps.appendEvent(upgraded.db, { type: 'kimi-ready', runtime: 'kimi' });
+    assert.ok(event.event_id > 0);
+    closeStateDb(upgraded.db);
+  } finally {
+    fs.rmSync(dir, { recursive: true, force: true });
+  }
+});

package/mcp-server/tests/init-project.test.cjs

@@ -36,8 +36,8 @@ test('initProject copies bundled template into .ultra/', () => {
     assert.ok(fs.existsSync(r.state_db_path));
     const db = new Database(r.state_db_path, { readonly: true });
     try {
-      const version = db.prepare("SELECT version FROM schema_version WHERE version = '9.0'").get();
-      assert.equal(version.version, '9.0');
+      const version = db.prepare("SELECT version FROM schema_version WHERE version = '9.1'").get();
+      assert.equal(version.version, '9.1');
       assert.ok(db.prepare("SELECT name FROM sqlite_master WHERE name = 'changes'").get());
     } finally {
       db.close();

package/mcp-server/tests/state-db.test.cjs

@@ -8,7 +8,7 @@ const path = require('node:path');
 
 const stateDb = require('../lib/state-db.cjs');
 
-test('initStateDb upgrades a pre-9.0 database in place with task/event change linkage', () => {
+test('initStateDb upgrades a pre-9.1 database in place with task/event change linkage', () => {
   const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'ubp-db-upgrade-'));
   const file = path.join(dir, 'state.db');
   try {
@@ -59,8 +59,8 @@ test('initStateDb upgrades a pre-9.0 database in place with task/event change li
     assert.ok(columns.includes('change_id'));
     const eventColumns = upgraded.db.prepare('PRAGMA table_info(events)').all().map((row) => row.name);
     assert.ok(eventColumns.includes('change_id'));
-    assert.equal(upgraded.schema_version, '9.0');
-    assert.ok(upgraded.db.prepare("SELECT 1 FROM schema_version WHERE version = '9.0'").get());
+    assert.equal(upgraded.schema_version, '9.1');
+    assert.ok(upgraded.db.prepare("SELECT 1 FROM schema_version WHERE version = '9.1'").get());
     stateDb.closeStateDb(upgraded.db);
   } finally {
     fs.rmSync(dir, { recursive: true, force: true });

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "ultra-builder-pro-cli",
-  "version": "0.7.0",
-  "description": "Multi-runtime installer for the Ultra Builder Pro agent engineering system — deploys commands, agents, skills, hooks, and MCP server to Claude Code, OpenCode, and Codex CLI.",
+  "version": "0.9.0",
+  "description": "Multi-runtime installer for the Ultra Builder Pro agent engineering system — deploys commands, agents, skills, hooks, and MCP server to Claude Code, OpenCode, Codex CLI, and Kimi Code.",
   "bin": {
     "ultra-builder-pro-cli": "bin/install.js",
     "ubp": "bin/install.js",
@@ -60,6 +60,7 @@
     "claude-code",
     "opencode",
     "codex",
+    "kimi-code",
     "agent",
     "cli",
     "multi-runtime",
@@ -92,6 +93,7 @@
     "verify:release": "npm run test:all && npm run test:hooks && npm audit --omit=dev --audit-level=high",
     "prepublishOnly": "npm run verify:release",
     "install:claude": "node bin/install.js --claude --local",
+    "install:kimi": "node bin/install.js --kimi --local",
     "install:all": "node bin/install.js --all --local"
   },
   "dependencies": {

package/skills/ultra-change/SKILL.md

@@ -58,7 +58,7 @@ Choose the smallest honest kind:
 | `quick` | localized fix or tiny behavior change | diff, tests, spec |
 | `standard` | normal feature or cross-file change | diff, tests, spec, docs, review |
 | `major` | architecture, public contract, migration, or broad redesign | diff, tests, spec, docs, review |
-| `incident` | production/runtime failure requiring diagnosis | diagnosis, diff, tests |
+| `incident` | canonical debug lane for a production/runtime failure | diagnosis, diff, tests |
 
 When uncertain between two kinds, choose the higher-risk kind. Classification
 changes the proof required; it must not be used to bypass evidence.
@@ -91,14 +91,18 @@ The change directory is the bounded execution packet:
   context-manifest.json
   delta/                  # required for standard and major
   plan.md                 # required for standard and major
+  diagnosis.md            # created and structurally required for incident
   verification.md         # generated by convergence
 ```
 
 For `standard` and `major`, write delta documents under `delta/` describing
 what changes relative to the current baseline. Do not copy the whole baseline.
 Write `plan.md` with live entry points, files/contracts, tests, documentation,
-and rollback. For `quick` and `incident`, add only artifacts needed to make the
-decision and evidence inspectable.
+and rollback. For `quick`, add only artifacts needed to make the decision and
+evidence inspectable. For `incident`, complete the generated `diagnosis.md` with
+non-empty `Reproduction`, `Hypotheses`, `Root cause`, `Regression test`, and
+`Recovery` sections. Free-form convergence evidence never substitutes for this
+durable diagnosis artifact.
 
 Create at least one task through `task.create` with `change_id` set. Split only
 when ownership, dependencies, or independent verification justify it. All task
@@ -116,6 +120,11 @@ context, not permission to silently refresh or mutate the external provider.
 
 ## 6. Route execution
 
+- For `incident`, diagnosis is the first execution slice. Route it to the bundled
+  `debugger` agent when the Host exposes native subagents; otherwise the primary
+  Host follows the same four-phase method. Supply only the compiled change/task
+  context. The primary Host owns the fix, updates `diagnosis.md`, and reruns the
+  discriminating regression test.
 - Implement one linked task with `/ultra-dev <task-id>`.
 - Run `/ultra-test` for project-level evidence.
 - Run `/ultra-review` when the change kind requires review.

package/skills/ultra-deliver/SKILL.md

@@ -141,7 +141,9 @@ evidence required by kind:
 
 - `quick`: diff, tests, spec;
 - `standard` / `major`: diff, tests, spec, docs, review;
-- `incident`: diagnosis, diff, tests.
+- `incident`: diagnosis, diff, tests. The diagnosis evidence must point to the
+  completed `diagnosis.md`; convergence separately validates all five required
+  debugging sections and cannot be satisfied by the evidence string alone.
 
 Evidence must name concrete commands, artifacts, or file paths. If blockers are
 returned, stop before tag/push and route to the owning workflow. Do not weaken

package/skills/ultra-doctor/SKILL.md

@@ -66,9 +66,16 @@ remaining incident unresolved.
 For an installed hook that points to a missing cached file:
 
 1. capture the exact hook config path and missing target;
-2. verify the currently installed plugin/cache version;
-3. reinstall or regenerate the host-native plugin through the adapter;
-4. rerun read-only doctor and a host hook smoke test.
+2. run the separate read-only installer boundary check:
+   `npx ultra-builder-pro-cli --<runtime> --local --doctor --json` (or use
+   `--global` when that is the actual install scope);
+3. preserve its provenance, asset-hash, and host-contract findings;
+4. reinstall or regenerate the host-native plugin through the adapter;
+5. rerun both installer doctor and `system.doctor`, then a host hook smoke test.
+
+`system.doctor` diagnoses project `.ultra/state.db` and workflow recovery.
+Installer `ubp --doctor` diagnoses the installed plugin files and host wiring.
+Neither report substitutes for the other.
 
 Do not bypass a broken hook by changing application implementation, and do not
 treat a hook failure as project-code evidence.

package/skills/ultra-review/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: ultra-review
-description: "Parallel code review orchestration with 6 specialized agents + coordinator. Zero context pollution - all output via JSON files."
+description: "Parallel code review orchestration with 5 specialized reviewers + coordinator. Zero context pollution - all output via JSON files."
 user-invocable: true
 runtime: all
 mcp_tools_required:
@@ -32,7 +32,7 @@ Orchestrates parallel code review using specialized agents. All findings written
 
 ```
 /ultra-review              # Full review (smart skip based on diff content)
-/ultra-review all          # Force ALL 6 agents, no auto-skip (pre-merge gate)
+/ultra-review all          # Force ALL 5 reviewers, no auto-skip (pre-merge gate)
 /ultra-review quick        # Quick review (review-code only)
 /ultra-review security     # Security focus (review-code + review-errors)
 /ultra-review tests        # Test quality focus (review-tests only)
@@ -138,7 +138,7 @@ echo "<diff_files>" | grep -E "\.(test|spec)\.(ts|tsx|js|jsx)$|test_.*\.py$|.*_t
 grep -l "try\|catch\|\.catch\|throw\|Error(" <diff_files>
 ```
 
-**Mode: `all`** → Force ALL 6 agents, no auto-skip. Use for pre-merge gates (`/ultra-dev`).
+**Mode: `all`** → Force ALL 5 reviewers, no auto-skip. Use for pre-merge gates (`/ultra-dev`).
 **Mode: `quick`** → review-code only
 **Mode: `security`** → review-code + review-errors
 **Mode: `tests`** → review-tests only

package/spec/mcp-tools.yaml

@@ -367,7 +367,7 @@ tools:
         task_id:    { type: string }
         change_id:  { type: string }
         session_id: { type: string }
-        runtime:    { type: string, enum: [claude, opencode, codex] }
+        runtime:    { type: string, enum: [claude, opencode, codex, kimi] }
         type:
           type: string
           enum:
@@ -452,7 +452,7 @@ tools:
       additionalProperties: false
       properties:
         task_id:  { type: string }
-        runtime:  { type: string, enum: [claude, opencode, codex] }
+        runtime:  { type: string, enum: [claude, opencode, codex, kimi] }
         takeover: { type: boolean, default: false, description: 'override active lease (D33 takeover strategy)' }
         worktree_base: { type: string }
     output_schema:
@@ -554,7 +554,7 @@ tools:
       additionalProperties: false
       properties:
         task_id: { type: string }
-        runtime: { type: string, enum: [claude, opencode, codex] }
+        runtime: { type: string, enum: [claude, opencode, codex, kimi] }
     output_schema:
       type: object
       required: [can_spawn, recommended_action]
@@ -630,7 +630,7 @@ tools:
   # ────────────────────────── change family ──────────────────────────────
   - name: change.create
     family: change
-    description: Create a continuous feature, fix, redesign, or incident change unit and its initial context manifest.
+    description: Create a continuous feature, fix, redesign, or incident change unit and its initial context manifest; incidents also register a five-section diagnosis artifact.
     phase: '9'
     writer_role: mcp
     input_schema:
@@ -860,7 +860,7 @@ tools:
 
   - name: change.converge
     family: change
-    description: Deterministically verify task, spec, test, documentation, review, and context evidence before closure.
+    description: Deterministically verify task, spec, test, documentation, review, context, and incident diagnosis structure before closure.
     phase: '9'
     writer_role: mcp
     input_schema:

package/spec/schemas/skill-manifest.schema.json

@@ -19,10 +19,10 @@
     },
     "runtime": {
       "oneOf": [
-        { "type": "string", "enum": ["claude", "opencode", "codex", "all"] },
+        { "type": "string", "enum": ["claude", "opencode", "codex", "kimi", "all"] },
         {
           "type": "array",
-          "items": { "type": "string", "enum": ["claude", "opencode", "codex"] },
+          "items": { "type": "string", "enum": ["claude", "opencode", "codex", "kimi"] },
           "minItems": 1,
           "uniqueItems": true
         }

package/spec/schemas/state-db.sql

@@ -79,7 +79,7 @@ CREATE TABLE IF NOT EXISTS events (
   task_id       TEXT,
   change_id     TEXT,
   session_id    TEXT,
-  runtime       TEXT CHECK (runtime IS NULL OR runtime IN ('claude', 'opencode', 'codex')),
+  runtime       TEXT CHECK (runtime IS NULL OR runtime IN ('claude', 'opencode', 'codex', 'kimi')),
   payload_json  TEXT
 );
 
@@ -95,7 +95,7 @@ CREATE INDEX IF NOT EXISTS events_change  ON events(change_id, id);
 CREATE TABLE IF NOT EXISTS sessions (
   sid               TEXT PRIMARY KEY,
   task_id           TEXT NOT NULL REFERENCES tasks(id),
-  runtime           TEXT NOT NULL CHECK (runtime IN ('claude', 'opencode', 'codex')),
+  runtime           TEXT NOT NULL CHECK (runtime IN ('claude', 'opencode', 'codex', 'kimi')),
   pid               INTEGER,
   worktree_path     TEXT NOT NULL,
   artifact_dir      TEXT NOT NULL,
@@ -272,3 +272,5 @@ INSERT OR IGNORE INTO schema_version (version, description)
 VALUES ('8A.2', 'Phase 8A.2 — tasks.estimated_days preserved across MCP and v4.4 migration');
 INSERT OR IGNORE INTO schema_version (version, description)
 VALUES ('9.0', 'Continuous change units, context snapshots, trace links, incidents, projection outbox, and durable consumers');
+INSERT OR IGNORE INTO schema_version (version, description)
+VALUES ('9.1', 'Kimi runtime support in durable events and sessions');