gsd-opencode 1.20.2 → 1.20.4

package/get-shit-done/bin/test/set-profile.test.cjs

@@ -0,0 +1,301 @@
+/**
+ * Unit tests for set-profile.cjs
+ *
+ * Tests for profile switching, validation, and the three operation modes:
+ * 1. Mode 1 (no profile name): Validate and apply current profile
+ * 2. Mode 2 (profile name): Switch to specified profile
+ * 3. Mode 3 (inline JSON): Create new profile from definition
+ *
+ * Includes validation checks, dry-run functionality, and rollback mechanisms.
+ */
+
+import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
+import fs from 'fs';
+import path from 'path';
+import os from 'os';
+
+// Mock console.log and console.error to capture output
+const originalLog = console.log;
+const originalError = console.error;
+const originalExit = process.exit;
+
+// Test fixtures
+const VALID_CONFIG = {
+  current_oc_profile: 'smart',
+  profiles: {
+    presets: {
+      simple: {
+        planning: 'bailian-coding-plan/qwen3.5-plus',
+        execution: 'bailian-coding-plan/qwen3.5-plus',
+        verification: 'bailian-coding-plan/qwen3.5-plus'
+      },
+      smart: {
+        planning: 'bailian-coding-plan/qwen3.5-plus',
+        execution: 'bailian-coding-plan/qwen3.5-plus',
+        verification: 'bailian-coding-plan/qwen3.5-plus'
+      },
+      genius: {
+        planning: 'bailian-coding-plan/qwen3.5-plus',
+        execution: 'bailian-coding-plan/qwen3.5-plus',
+        verification: 'bailian-coding-plan/qwen3.5-plus'
+      }
+    }
+  }
+};
+
+describe('set-profile.cjs', () => {
+  let testDir;
+  let planningDir;
+  let configPath;
+  let opencodePath;
+  let capturedLog;
+  let capturedError;
+  let exitCode;
+  let allLogs;
+  let allErrors;
+
+  beforeEach(() => {
+    // Create isolated test directory
+    testDir = fs.mkdtempSync(path.join(os.tmpdir(), 'set-profile-test-'));
+    planningDir = path.join(testDir, '.planning');
+    configPath = path.join(planningDir, 'oc_config.json');
+    opencodePath = path.join(testDir, 'opencode.json');
+
+    fs.mkdirSync(planningDir, { recursive: true });
+
+    // Reset captured output
+    capturedLog = null;
+    capturedError = null;
+    exitCode = null;
+    allLogs = [];
+    allErrors = [];
+
+    // Mock console.log to capture all output
+    console.log = (msg) => {
+      allLogs.push(msg);
+      capturedLog = msg;
+    };
+    console.error = (msg) => {
+      allErrors.push(msg);
+      capturedError = msg;
+    };
+    process.exit = (code) => {
+      exitCode = code;
+      throw new Error(`process.exit(${code})`);
+    };
+  });
+
+  afterEach(() => {
+    // Restore original functions
+    console.log = originalLog;
+    console.error = originalError;
+    process.exit = originalExit;
+
+    // Cleanup test directory
+    try {
+      fs.rmSync(testDir, { recursive: true, force: true });
+    } catch (err) {
+      // Ignore cleanup errors
+    }
+  });
+
+  // Import setProfile inside tests to use mocked functions
+  const importSetProfile = () => {
+    const modulePath = '../gsd-oc-commands/set-profile.cjs';
+    delete require.cache[require.resolve(modulePath)];
+    return require(modulePath);
+  };
+
+  describe('Export verification', () => {
+    it('exports setProfile function', () => {
+      const setProfile = importSetProfile();
+      expect(typeof setProfile).toBe('function');
+    });
+
+    it('function name is setProfile', () => {
+      const setProfile = importSetProfile();
+      expect(setProfile.name).toBe('setProfilePhase16'); // Function was renamed from phase16
+    });
+  });
+
+  describe('Basic functionality', () => {
+    function writeOpencodeJson() {
+      const opencode = {
+        $schema: 'https://opencode.ai/schema.json',
+        agent: {
+          'gsd-planner': {
+            model: 'bailian-coding-plan/qwen3.5-plus',
+            tools: ['*']
+          },
+          'gsd-executor': {
+            model: 'bailian-coding-plan/qwen3.5-plus',
+            tools: ['*']
+          }
+        }
+      };
+      fs.writeFileSync(opencodePath, JSON.stringify(opencode, null, 2) + '\n', 'utf8');
+    }
+
+    beforeEach(() => {
+      fs.writeFileSync(configPath, JSON.stringify(VALID_CONFIG, null, 2) + '\n', 'utf8');
+      writeOpencodeJson();
+    });
+
+    it('setProfile updates profile when profile name provided', () => {
+      const setProfile = importSetProfile();
+      
+      try {
+        setProfile(testDir, ['genius']);
+      } catch (err) {
+        // Expected to throw due to process.exit mock
+      }
+
+      expect(exitCode).toBe(0);
+      const output = JSON.parse(capturedLog);
+      expect(output.success).toBe(true);
+      expect(output.data.profile).toBe('genius');
+    });
+
+    it('setProfile processes dry-run flag', () => {
+      const setProfile = importSetProfile();
+      
+      try {
+        setProfile(testDir, ['smart', '--dry-run']);
+      } catch (err) {
+        // Expected
+      }
+
+      expect(exitCode).toBe(0);
+      const output = JSON.parse(capturedLog);
+      expect(output.success).toBe(true);
+      expect(output.data.dryRun).toBe(true);
+      expect(output.data.action).toBe('switch_profile');
+    });
+
+    it('setProfile validates required keys for inline profiles', () => {
+      const setProfile = importSetProfile();
+      const inlineProfile = 'test_profile:{"planning":"bailian-coding-plan/qwen3.5-plus","execution":"bailian-coding-plan/qwen3.5-plus","verification":"bailian-coding-plan/qwen3.5-plus"}';
+      
+      try {
+        setProfile(testDir, [inlineProfile]);
+      } catch (err) {
+        // Expected
+      }
+
+      const output = JSON.parse(capturedLog);
+      expect(output.success).toBe(true);
+      expect(output.data.profile).toBe('test_profile');
+    });
+
+    it('setProfile handles Mode 1 (no profile name) scenario', () => {
+      const setProfile = importSetProfile();
+      
+      try {
+        setProfile(testDir, []);
+      } catch (err) {
+        // Expected
+      }
+
+      expect(exitCode).toBe(0);
+      const output = JSON.parse(capturedLog);
+      expect(output.success).toBe(true);
+      expect(output.data.profile).toBe('smart'); // From initial current_oc_profile
+    });
+
+    it('setProfile validates invalid models before modification', () => {
+      const setProfile = importSetProfile();
+      const inlineProfile = 'bad_profile:{"planning":"bad_model","execution":"bad_model","verification":"bad_model"}';
+      
+      try {
+        setProfile(testDir, [inlineProfile]);
+      } catch (err) {
+        // Expected - should error
+      }
+
+      expect(exitCode).toBe(1);
+    });
+
+    it('setProfile rejects invalid inline profile definitions', () => {
+      const setProfile = importSetProfile();
+      // Invalid JSON
+      const badDef = 'bad_profile:{"planning:"model","execution":"model","verification":"model"}';
+      
+      try {
+        setProfile(testDir, [badDef]);
+      } catch (err) {
+        // Expected - should error
+      }
+
+      expect(exitCode).toBe(1);
+      const error = JSON.parse(capturedError);
+      expect(error.error.code).toBe('INVALID_SYNTAX');
+    });
+
+    it('setProfile rejects incomplete profile definitions', () => {
+      const setProfile = importSetProfile();
+      // Missing verification property
+      const badDef = 'bad_profile:{"planning":"bailian-coding-plan/qwen3.5-plus","execution":"bailian-coding-plan/qwen3.5-plus"}';
+      
+      try {
+        setProfile(testDir, [badDef]);
+      } catch (err) {
+        // Expected - should error
+      }
+
+      expect(exitCode).toBe(1);
+      const error = JSON.parse(capturedError);
+      expect(error.error.code).toBe('INCOMPLETE_PROFILE');
+    });
+  });
+
+  describe('Error handling', () => {
+    it('handles missing config.json gracefully', () => {
+      const setProfile = importSetProfile();
+      
+      try {
+        setProfile(testDir, ['test']);
+      } catch (err) {
+        // Expected to throw
+      }
+
+      expect(exitCode).toBe(1);
+      const error = JSON.parse(capturedError);
+      expect(error.error.code).toBe('CONFIG_NOT_FOUND');
+    });
+
+    it('sets exit code 1 for invalid profile', () => {
+      const setProfile = importSetProfile();
+      
+      // Set up a valid config with presets
+      const configData = {...VALID_CONFIG};
+      fs.writeFileSync(configPath, JSON.stringify(configData, null, 2) + '\n', 'utf8');
+      const opencodeData = {
+        $schema: 'https://opencode.ai/schema.json',
+        agent: {}
+      };
+      fs.writeFileSync(opencodePath, JSON.stringify(opencodeData, null, 2) + '\n', 'utf8');
+      
+      try {
+        setProfile(testDir, ['non-existent-profile']);
+      } catch (err) {
+        // Expected
+      }
+
+      expect(exitCode).toBe(1);
+    });
+
+    it('rejects too many arguments', () => {
+      const setProfile = importSetProfile();
+
+      try {
+        setProfile(testDir, ['profile1', 'profile2']);
+      } catch (err) {
+        // Expected
+      }
+
+      expect(exitCode).toBe(1);
+      const error = JSON.parse(capturedError);
+      expect(error.error.code).toBe('INVALID_ARGS');
+    });
+  });
+});

package/get-shit-done/workflows/oc-check-profile.md

@@ -0,0 +1,181 @@
+<role>
+You are executing the `/gsd-check-profile` command. Validate gsd-opencode profile configuration across both `opencode.json` and `.planning/oc_config.json`, then report results.
+
+This is a **read-only diagnostic**. Do NOT modify any files or attempt to fix issues. When problems are found, recommend `/gsd-set-profile` and stop.
+</role>
+
+<required_reading>
+read all files referenced by the invoking prompt's execution_context before starting.
+</required_reading>
+
+<context>
+## What Gets Validated
+
+| Check | File | Validates |
+|-------|------|-----------|
+| `check-opencode-json` | `opencode.json` | All agent model IDs exist in the opencode models catalog |
+| `check-config-json` | `.planning/oc_config.json` | gsd-opencode profile structure is valid, current profile exists in presets, all stage model IDs exist in catalog |
+
+## CLI Tool
+
+All validation runs through `gsd-oc-tools.cjs`. Both commands output a JSON envelope with `success`, `data`, and optional `error` fields. Exit code 0 = valid, exit code 1 = issues found.
+
+## JSON Response Shapes
+
+**check-opencode-json** (exit 0 or 1):
+```json
+{
+  "success": true,
+  "data": { "valid": true|false, "total": N, "validCount": N, "invalidCount": N, "issues": [{ "agent": "...", "model": "...", "reason": "..." }] },
+  "error": { "code": "INVALID_MODEL_ID", "message": "..." }
+}
+```
+
+Note: When `opencode.json` does not exist, the tool returns exit 1 with `error.code = "CONFIG_NOT_FOUND"`. This is **not** an error for gsd-opencode profile validation — see Step 2 for handling.
+
+**check-config-json** (exit 0 or 1):
+```json
+{
+  "success": true|false,
+  "data": { "passed": true|false, "current_oc_profile": "...", "profile_data": {...}, "issues": [{ "field": "...", "value": "...", "reason": "..." }] },
+  "error": { "code": "INVALID_PROFILE|CONFIG_NOT_FOUND|INVALID_JSON", "message": "..." }
+}
+```
+</context>
+
+<behavior>
+
+## Step 1: Run both validations
+
+Execute both checks and capture their output and exit codes:
+
+```bash
+node ~/.config/opencode/get-shit-done/bin/gsd-oc-tools.cjs check-opencode-json
+```
+
+```bash
+node ~/.config/opencode/get-shit-done/bin/gsd-oc-tools.cjs check-config-json
+```
+
+Parse both JSON responses.
+
+## Step 2: Classify results by severity
+
+### opencode.json classification
+
+| Tool result | Severity | Meaning |
+|-------------|----------|---------|
+| exit 0, `data.valid = true` | OK | All model IDs valid |
+| exit 1, `error.code = "CONFIG_NOT_FOUND"` | WARNING | No `opencode.json` — agents will use the default/current model. This is acceptable. |
+| exit 1, `error.code = "INVALID_MODEL_ID"` | ERROR | One or more model IDs are invalid. Must be fixed. |
+| exit 1, `error.code = "INVALID_JSON"` | ERROR | File is malformed JSON. Must be fixed. |
+
+### .planning/oc_config.json classification
+
+| Tool result | Severity | Meaning |
+|-------------|----------|---------|
+| exit 0, `data.passed = true` | OK | gsd-opencode profile configuration valid |
+| exit 1, `error.code = "CONFIG_NOT_FOUND"` | ERROR | No gsd-opencode profile configured yet |
+| exit 1, `error.code = "INVALID_PROFILE"` | ERROR | gsd-opencode profile structure is invalid |
+| exit 1, `error.code = "INVALID_JSON"` | ERROR | File is malformed JSON |
+
+## Step 3: Report results
+
+Determine the overall status:
+- **All OK (no ERRORs, no WARNINGs)**: report success
+- **WARNINGs only (no ERRORs)**: report success with warnings
+- **Any ERRORs**: report errors with fix instructions
+
+---
+
+### All OK — no errors, no warnings
+
+```
+gsd-opencode profile: OK
+
+  opencode.json              All model IDs valid
+  .planning/oc_config.json   gsd-opencode profile valid
+```
+
+**Stop here.**
+
+---
+
+### OK with warnings (opencode.json missing, but oc_config.json is valid)
+
+```
+gsd-opencode profile: OK
+
+  opencode.json              Not found (agents will use the default/current model)
+  .planning/oc_config.json   gsd-opencode profile valid
+```
+
+**Stop here.**
+
+---
+
+### Errors found
+
+Display a structured diagnostic. Use the severity labels (WARNING / ERROR) to make the impact clear.
+
+```
+gsd-opencode profile: ERRORS FOUND
+
+--- opencode.json ---
+
+[If OK]
+  All model IDs valid
+
+[If WARNING — CONFIG_NOT_FOUND]
+  WARNING: opencode.json not found. Agents will use the default/current model.
+
+[If ERROR — INVALID_MODEL_ID — iterate over data.issues]
+  ERROR: {N} invalid model ID(s):
+
+    Agent:   {issue.agent}
+    Model:   {issue.model}
+    Reason:  {issue.reason}
+
+    (repeat for each issue)
+
+[If ERROR — INVALID_JSON]
+  ERROR: opencode.json is not valid JSON.
+
+--- .planning/oc_config.json ---
+
+[If OK]
+  gsd-opencode profile valid
+
+[If ERROR — CONFIG_NOT_FOUND]
+  ERROR: .planning/oc_config.json not found — no gsd-opencode profile configured.
+
+[If ERROR — INVALID_PROFILE — iterate over data.issues]
+  ERROR: {N} gsd-opencode profile issue(s):
+
+    Field:   {issue.field}
+    Value:   {issue.value}
+    Reason:  {issue.reason}
+
+    (repeat for each issue)
+
+[If ERROR — INVALID_JSON]
+  ERROR: .planning/oc_config.json is not valid JSON.
+
+--- Fix ---
+
+Run /gsd-set-profile or /gsd-set-profile <simple|smart|genius> to fix gsd-opencode profile configuration.
+```
+
+**Stop here.** Do not offer to fix anything. Do not edit files.
+
+</behavior>
+
+<notes>
+- This workflow is strictly diagnostic — never modify `opencode.json`, `.planning/oc_config.json`, or any other file.
+- When errors are found, always recommend `/gsd-set-profile` or `/gsd-set-profile <simple|smart|genius>` as the resolution path. Do not suggest manual editing.
+- Always display full model IDs (e.g., `bailian-coding-plan/qwen3-coder-plus`), never abbreviate.
+- Missing `opencode.json` is a WARNING, not an error. The user simply hasn't customized agent models — agents fall back to the default/current model. Do not include it in the "Fix" section.
+- Missing `.planning/oc_config.json` IS an error — it means no gsd-opencode profile has been set up.
+- Always use the term "gsd-opencode profile" (not just "profile") when referring to the profile system.
+- Both `check-config-json` and `check-oc-config-json` route to the same validator. Use `check-config-json` (shorter).
+</notes>