@inetafrica/open-claudia 3.4.19 → 3.4.20

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 3.4.20
+
+- Make `claude-opus-5-5` the default Claude model. 3.4.19 only added it to the picker; the default stayed on `claude-opus-4-8`, so a room that had never run `/model` kept the old model.
+- The default governs main conversation turns only. Utility work remains tier-pinned — `low` to `claude-haiku-4-5-20251001` and `medium` to `claude-sonnet-4-6` — so recall, pack review, the dedup adjudicator and the enforcer do not reprice with it, and a room's own `/model` choice still wins over the default.
+- Note for operators: `CLAUDE_MODEL` in `~/.open-claudia/.env` (or the process env) overrides this. An install that pinned the old default there keeps it until that line is changed. Pod images carry no `CLAUDE_MODEL` and the entrypoint never writes one, so pods resolve the shipped constant.
+- `claude-opus-5-5` requires Claude Code >= 2.1.280; an older CLI refuses it with an API 400 rather than degrading. The pod image installs the CLI unpinned, so an image built before that release must not run this default.
+
 ## 3.4.19
 
 - Offer `claude-opus-5-5` in the Claude model picker. Verified against the live API rather than assumed: on Claude Code 2.1.280 the API bills the request to `claude-opus-5-5` (1M context, 128k max output), while 2.1.258 refuses it with "does not support this model; version 2.1.280 or newer is required". An older CLI returns that API 400 instead of degrading, so a pod on a stale image errors on the model rather than falling back — the same constraint already recorded for `claude-fable-5-1` (>= 2.1.251).

package/core/providers/claude.js

@@ -24,7 +24,10 @@ const CLAUDE_MODELS = Object.freeze([
   "claude-haiku-4-5-20251001",
 ]);
 const CLAUDE_EFFORT_VALUES = Object.freeze(["low", "medium", "high", "xhigh", "max"]);
-const CLAUDE_DEFAULT_MODEL = config.config.CLAUDE_MODEL || process.env.CLAUDE_MODEL || "claude-opus-4-8";
+// The default governs MAIN turns only: utility work is pinned by tier below
+// (low -> haiku, medium -> sonnet), so recall, pack review and the enforcer do
+// not move with it. A room's own /model pin always wins over this.
+const CLAUDE_DEFAULT_MODEL = config.config.CLAUDE_MODEL || process.env.CLAUDE_MODEL || "claude-opus-5-5";
 const REQUIRED_HELP_FLAGS = Object.freeze([
   "--append-system-prompt",
   "--output-format",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inetafrica/open-claudia",
-  "version": "3.4.19",
+  "version": "3.4.20",
   "description": "An always-on, provider-agnostic coding-agent harness for Claude Code and OpenAI Codex via chat",
   "main": "bot.js",
   "bin": {

package/test-provider-claude.js

@@ -6,7 +6,7 @@ const assert = require("assert");
 const fs = require("fs");
 const path = require("path");
 const { validateInvocation, validateProvider } = require("./core/providers/contract");
-const { createClaudeProvider } = require("./core/providers/claude");
+const { createClaudeProvider, claudeProvider, CLAUDE_MODELS } = require("./core/providers/claude");
 const { getProvider } = require("./core/providers");
 
 function deepFreeze(value) {
@@ -49,6 +49,25 @@ assert.ok(provider.modelChoices().includes("claude-fable-5-1"));
 assert.ok(provider.modelChoices().includes("claude-opus-5-5"), "opus 5.5 is offered in the model picker");
 assert.ok(provider.modelChoices().includes("claude-fable-5"));
 assert.ok(provider.modelChoices().includes("claude-sonnet-4-6"));
+
+// The SHIPPED default, asserted on the real module rather than the fixture above,
+// so changing the constant cannot pass unnoticed. Read from source because the
+// resolved value is deliberately overridable: loadEnv merges CLAUDE_MODEL from
+// ~/.open-claudia/.env over process.env, and a machine that carries that pin
+// (Sumeet's laptop does, set 2026-07-21) would otherwise mask a regression here.
+const claudeSource = fs.readFileSync(path.join(__dirname, "core", "providers", "claude.js"), "utf8");
+assert.match(
+  claudeSource,
+  /CLAUDE_DEFAULT_MODEL = config\.config\.CLAUDE_MODEL \|\| process\.env\.CLAUDE_MODEL \|\| "claude-opus-5-5";/,
+  "shipped Claude default is claude-opus-5-5",
+);
+// The default governs main turns only: utility tiers stay cheap. These are what
+// stop a default bump from silently repricing recall, pack review and the enforcer.
+assert.strictEqual(claudeProvider.tierModel("low"), "claude-haiku-4-5-20251001", "low-tier utility work stays on haiku");
+assert.strictEqual(claudeProvider.tierModel("medium"), "claude-sonnet-4-6", "medium-tier utility work stays on sonnet");
+if (!process.env.CLAUDE_MODEL && !claudeSource.includes("__fixture__")) {
+  assert.ok(CLAUDE_MODELS.includes(claudeProvider.defaultModel()), "the resolved default is a model the picker offers");
+}
 assert.deepStrictEqual(provider.compatibilityStatus(), {
   state: "compatible",
   version: "2.1.150",

package/test-provider-language.js

@@ -477,7 +477,17 @@ assert.match(pkg.description, /provider-agnostic coding-agent harness/i);
 // unreleased Kazee fail-closed room gate, so a runtime behaviour change and an
 // additive picker entry can be rolled back independently. Local Claude Code went
 // 2.1.258 -> 2.1.280 first; the model is refused outright below that.
-assert.strictEqual(pkg.version, "3.4.19", "release version must remain unchanged without explicit approval");
+// 3.4.20 approved by Sumeet 2026-09-23 ("If not please make it the default and
+// push them roll out to the whole fleet", answering whether 3.4.19 had already
+// made opus 5.5 the default — it had not, it only added the picker entry):
+// claude-opus-5-5 becomes CLAUDE_DEFAULT_MODEL. It governs MAIN turns only —
+// utility work is tier-pinned (low -> haiku, medium -> sonnet) and a room's own
+// /model pin still wins — so the cost moves with conversation turns, not recall.
+// Safe on the fleet because the pod env carries no CLAUDE_MODEL and the
+// entrypoint never writes one, so the constant is what pods resolve; the pod
+// image installs Claude Code unpinned, which must be >= 2.1.280 or the model is
+// refused with an API 400 rather than degrading.
+assert.strictEqual(pkg.version, "3.4.20", "release version must remain unchanged without explicit approval");
 for (const keyword of ["claude", "codex", "coding-agent", "provider-agnostic"]) {
   assert.ok(pkg.keywords.includes(keyword), `package keyword missing: ${keyword}`);
 }