fix: repair oc-switch agent model selection

Author: juancarlosrodicio

README.md

@@ -327,7 +327,7 @@ After a tagged release is published, you can install the same CLI directly
 from GitHub without keeping a checkout:
 
 ```bash
-npm install --global "git+https://github.com/jcarlosrodicio/opencode-agent-orchestration-kit.git#v1.0.40"
+npm install --global "git+https://github.com/jcarlosrodicio/opencode-agent-orchestration-kit.git#v1.0.41"
 oc-switch
 ```
 
@@ -741,7 +741,7 @@ Validate only the canonical identity and current release note with:
 
 ```bash
 npm run check:version
-node scripts/version.mjs --check-tag v1.0.40
+node scripts/version.mjs --check-tag v1.0.41
 ```
 
 Tag validation compares an explicitly supplied tag with the package identity.

docs/models.md

@@ -26,7 +26,7 @@ oc-switch
 Once a release tag exists, a checkout is optional:
 
 ```bash
-npm install --global "git+https://github.com/jcarlosrodicio/opencode-agent-orchestration-kit.git#v1.0.40"
+npm install --global "git+https://github.com/jcarlosrodicio/opencode-agent-orchestration-kit.git#v1.0.41"
 oc-switch
 ```
 

docs/releases/v1.0.41.md

@@ -0,0 +1,42 @@
+# v1.0.41 - Safe lifecycle and canonical release identity
+
+This patch release fixes agent-row selection in the interactive OpenCode model
+switcher. Selecting a model for an agent now updates the agent assignment instead
+of rejecting the selection as an invalid target.
+
+## Highlights
+
+- Fix `oc-switch` agent-row selections by mapping focused agent rows to the
+  internal `agents` assignment target.
+- Add a regression test covering a focused agent selecting and saving a model.
+
+## Installation or upgrade
+
+Install or upgrade the public kit using the normal manifest-owned flow:
+
+```bash
+./install.sh --dry-run
+./install.sh
+./upgrade.sh --dry-run
+./upgrade.sh
+```
+
+After installation, run `oak check` and review the installed target before
+using the harness.
+
+## Migration
+
+No migration is required. Existing model-switcher state and shell routing
+remain compatible.
+
+## Validation performed
+
+The focused selector tests, full 1,068-test unit and script suite, contract
+check, package smoke, syntax checks, and dry-run package inspection passed.
+
+## Public safety boundary
+
+The public release excludes private OpenCode and Pi configuration, local
+providers and MCP wiring, credentials, authentication files, raw transcripts,
+private evidence, absolute machine paths, and runtime state. Pi uses its own
+native model catalog and is not included in this OpenCode package.

package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-agent-orchestration-kit",
-  "version": "1.0.40",
+  "version": "1.0.41",
   "private": false,
   "license": "Apache-2.0",
   "type": "module",

scripts/benchmark-tool-paths.mjs

@@ -143,7 +143,7 @@ async function benchmarkFixture(fixture, iterations, temporaryRoot) {
         const result = await createInstallationManager({
           sourceRoot: fixture.sourceRoot,
           repositoryRoot: fixture.root,
-          versionProvider: () => "1.0.40",
+          versionProvider: () => "1.0.41",
         }).run("install", { targetRoot });
         if (result.exitCode !== 0) throw invalid(`installation benchmark failed for ${fixture.id}`);
       } finally {

scripts/oc-switch.mjs

@@ -411,7 +411,8 @@ export async function runTerminalOcSwitcher({ input = process.stdin, output = pr
         const targetAgents = pickerTarget.kind === "agent"
           ? [...(selectedAgents.size ? selectedAgents : new Set([pickerTarget.key]))]
           : [];
-        state = applyModelSelection(state, { target: pickerTarget.key, agents: targetAgents, modelRef: available[pickerFocusedIndex] });
+        const target = pickerTarget.kind === "agent" ? "agents" : pickerTarget.key;
+        state = applyModelSelection(state, { target, agents: targetAgents, modelRef: available[pickerFocusedIndex] });
         dirty = true; selectedAgents.clear(); screen = "main"; status = "Change pending save.";
       } else if (name === "escape") { screen = "main"; pickerQuery = ""; }
       else if (name === "backspace") { pickerQuery = pickerQuery.slice(0, -1); pickerFocusedIndex = 0; }

scripts/oc-switch.test.mjs

@@ -139,6 +139,44 @@ test("OpenCode TUI changes the default model and persists on q", async () => {
   assert.match(output.writes.join(""), /Loading models/);
 });
 
+test("OpenCode TUI changes the focused agent model", async () => {
+  const input = createTuiInput();
+  const output = { isTTY: true, rows: 30, writes: [], write(chunk) { this.writes.push(String(chunk)); } };
+  const saved = [];
+  const state = {
+    schemaVersion: 1,
+    defaultModel: "nan/model-a",
+    smallModel: "nan/model-a",
+    agents: Object.fromEntries(AGENT_IDS.map((agent) => [agent, "nan/model-a"])),
+  };
+  const running = runTerminalOcSwitcher({
+    input,
+    output,
+    initialState: state,
+    initialModels: ["nan/model-a", "openai/model-b"],
+    refreshModels: async () => ["nan/model-a", "openai/model-b"],
+    saveState: async (nextState) => {
+      saved.push(nextState);
+      return nextState;
+    },
+  });
+
+  await nextTick();
+  emitKey(input, "down");
+  emitKey(input, "down");
+  emitKey(input, "return", "\r");
+  await nextTick();
+  emitKey(input, "down");
+  emitKey(input, "return", "\r");
+  await nextTick();
+  emitKey(input, "q");
+  const result = await running;
+
+  assert.equal(result.status, "saved");
+  assert.equal(saved.length, 1);
+  assert.equal(saved[0].agents.lead, "openai/model-b");
+});
+
 test("OpenCode search treats shortcut letters as query text", async () => {
   const input = createTuiInput();
   const output = { isTTY: true, rows: 30, writes: [], write(chunk) { this.writes.push(String(chunk)); } };

scripts/package-smoke.test.mjs

@@ -17,7 +17,7 @@ import {
 
 const ROOT = path.dirname(path.dirname(fileURLToPath(import.meta.url)));
 const HASH = "a".repeat(64);
-const BASENAME = "opencode-agent-orchestration-kit-1.0.40.tgz";
+const BASENAME = "opencode-agent-orchestration-kit-1.0.41.tgz";
 const REQUIRED = [
   "package/package.json",
   "package/supply-chain.json",
@@ -210,7 +210,7 @@ test("smokeTarball validates both archive listings before extraction", async ()
   await assert.rejects(
     smokeTarball({
       repositoryRoot: ROOT,
-      tarball: "/public/opencode-agent-orchestration-kit-1.0.40.tgz",
+      tarball: "/public/opencode-agent-orchestration-kit-1.0.41.tgz",
       captureTarball(_source, destination) {
         fs.writeFileSync(destination, "fixture");
         return { size: 1 };

scripts/version.test.mjs

@@ -109,7 +109,7 @@ test("[V004] direct version entrypoints are exact and mutation-free", (t) => {
   for (const args of [["scripts/version.mjs"], ["scripts/manage-installation.mjs", "--version"]]) {
     const result = run(process.execPath, args, { HOME: home });
     assert.equal(result.status, 0, result.stderr);
-    assert.equal(result.stdout, "opencode-agent-orchestration-kit 1.0.40\n");
+    assert.equal(result.stdout, "opencode-agent-orchestration-kit 1.0.41\n");
     assert.equal(fs.existsSync(target), false);
   }
 });
@@ -118,7 +118,7 @@ test("[V005] all lifecycle wrappers expose the canonical version", () => {
   for (const wrapper of ["install.sh", "upgrade.sh", "doctor.sh", "uninstall.sh", "rollback.sh"]) {
     const result = run(path.join(REPOSITORY_ROOT, wrapper), ["--version"]);
     assert.equal(result.status, 0, `${wrapper}: ${result.stderr}`);
-    assert.equal(result.stdout, "opencode-agent-orchestration-kit 1.0.40\n", wrapper);
+    assert.equal(result.stdout, "opencode-agent-orchestration-kit 1.0.41\n", wrapper);
   }
 });