osborn 0.9.178 → 0.9.180

package/dist/claude-llm.js

@@ -167,7 +167,7 @@ ensureCompactionSettings();
 // (1M) window; without it opus runs at its 200K default. NOTE: 1M activation
 // also depends on account entitlement (auto on Team seats; else usage credits).
 process.env.ENABLE_1M_CONTEXT = '1';
-process.env.CLAUDE_AUTOCOMPACT_PCT_OVERRIDE = '60';
+process.env.CLAUDE_AUTOCOMPACT_PCT_OVERRIDE = '75'; // auto-compact at 75% of 1M context (750k tokens) — prevents compaction at extreme tail
 // Research mode tools — full research capabilities
 // Named sub-agents — the orchestrator delegates to these specialists. Each has
 // a specific role, model, and tool set. Module-level + exported so the HTTP
@@ -276,13 +276,13 @@ export const NAMED_AGENTS = {
     },
     writer: {
         description: [
-            'Execution agent with file write/edit permissions (Sonnet).',
+            'Execution agent with file write/edit permissions (Opus).',
             'Handles ALL file operations: code, config, docs, scripts, data files.',
             'VERIFY-FIRST workflow: checks assumptions before making changes, runs tests after.',
             'If anything is unclear, asks the main agent for clarification before touching files.',
         ].join(' '),
         tools: ['Read', 'Write', 'Edit', 'MultiEdit', 'Bash', 'Glob', 'Grep', 'NotebookRead', 'NotebookEdit'],
-        model: 'sonnet',
+        model: 'opus',
         prompt: [
             'You are Osborn\'s writer agent. You execute file changes with a verify-first approach.',
             '',
@@ -421,13 +421,13 @@ export const NAMED_AGENTS = {
     },
     planner: {
         description: [
-            'Planning agent (Opus). Use for: decomposing a large or ambiguous request into a concrete,',
+            'Planning agent (Sonnet). Use for: decomposing a large or ambiguous request into a concrete,',
             'ordered sequence of atomic writer-safe steps. Returns a self-contained brief the writer',
             'can execute without further clarification. Slow but thorough — only use for genuinely',
             'complex multi-file changes or when the approach is uncertain.',
         ].join(' '),
         tools: ['Read', 'Glob', 'Grep', 'WebSearch'],
-        model: 'opus',
+        model: 'sonnet',
         prompt: [
             'You are Osborn\'s planning agent. Your job is to decompose complex tasks into clear, atomic steps.',
             '',
@@ -474,13 +474,13 @@ export const NAMED_AGENTS = {
     },
     reviewer: {
         description: [
-            'Code-review agent (Opus). Use for: the VERIFY step in a generator-verifier loop — after the',
+            'Code-review agent (Sonnet). Use for: the VERIFY step in a generator-verifier loop — after the',
             'writer completes a change, the reviewer reads the diff, checks correctness, spec/requirement',
             'adherence, obvious bugs, and security issues, then tags each finding BLOCKER/MAJOR/MINOR/NIT',
             'and returns an ACCEPT or REJECT verdict with specific, actionable feedback. May write documentation files (.md etc.) only.',
         ].join(' '),
         tools: ['Read', 'Glob', 'Grep', 'Bash', 'Write', 'Edit'],
-        model: 'opus',
+        model: 'sonnet',
         prompt: [
             'You are Osborn\'s reviewer agent. You are the VERIFY step in a generator-verifier loop.',
             '',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "osborn",
-  "version": "0.9.178",
+  "version": "0.9.180",
   "description": "Voice AI coding assistant - local agent that connects to Osborn frontend",
   "type": "module",
   "bin": {

package/tests/autocompact-pct.test.ts

@@ -0,0 +1,77 @@
+/**
+ * Regression tests for CLAUDE_AUTOCOMPACT_PCT_OVERRIDE in claude-llm.ts
+ *
+ * Scope: verifies that the compaction threshold is set to a value that
+ * leaves meaningful context headroom before compaction fires.
+ *
+ * Derived from: requirements documented in CLAUDE.md (ENABLE_1M_CONTEXT,
+ * compaction rationale comments in claude-llm.ts) and the 0.9.177 → 0.9.178
+ * change that lowered the threshold from 92 → 60.
+ * NOT derived from the implementation diff.
+ */
+
+import { strict as assert } from 'node:assert'
+import { readFileSync } from 'node:fs'
+import { dirname, join } from 'node:path'
+import { fileURLToPath } from 'node:url'
+
+const here = dirname(fileURLToPath(import.meta.url))
+const src = readFileSync(join(here, '../src/claude-llm.ts'), 'utf8')
+
+let passed = 0
+let failed = 0
+
+function test(name: string, fn: () => void) {
+  try {
+    fn()
+    console.log(`  ✅ ${name}`)
+    passed++
+  } catch (err: any) {
+    console.error(`  ❌ ${name}`)
+    console.error(`     ${err.message}`)
+    failed++
+  }
+}
+
+// Extract the numeric value of CLAUDE_AUTOCOMPACT_PCT_OVERRIDE from source
+const match = src.match(/process\.env\.CLAUDE_AUTOCOMPACT_PCT_OVERRIDE\s*=\s*'(\d+)'/)
+const pctValue = match ? parseInt(match[1], 10) : NaN
+
+console.log('\n=== autocompact PCT override regression tests ===\n')
+console.log(`  (detected value: CLAUDE_AUTOCOMPACT_PCT_OVERRIDE = '${pctValue}')`)
+console.log()
+
+test('CLAUDE_AUTOCOMPACT_PCT_OVERRIDE is set in claude-llm.ts', () => {
+  assert.ok(
+    !isNaN(pctValue),
+    'CLAUDE_AUTOCOMPACT_PCT_OVERRIDE must be explicitly set in claude-llm.ts'
+  )
+})
+
+test('CLAUDE_AUTOCOMPACT_PCT_OVERRIDE is a valid percentage (1–99)', () => {
+  assert.ok(
+    pctValue >= 1 && pctValue <= 99,
+    `CLAUDE_AUTOCOMPACT_PCT_OVERRIDE must be between 1 and 99, got ${pctValue}`
+  )
+})
+
+test('CLAUDE_AUTOCOMPACT_PCT_OVERRIDE is ≤ 92 (prevents compaction at extreme tail)', () => {
+  // Values above 92 leave almost no headroom before hitting the token limit.
+  // The long-standing default was 92; anything higher than that risks the
+  // "compacting every message" bug described in memory/osborn-1m-context-and-worker-restart.md
+  assert.ok(
+    pctValue <= 92,
+    `CLAUDE_AUTOCOMPACT_PCT_OVERRIDE is ${pctValue} which is above 92 — ` +
+    'values this high risk compaction firing on every message at the context ceiling'
+  )
+})
+
+test('ENABLE_1M_CONTEXT is also set in claude-llm.ts', () => {
+  assert.ok(
+    src.includes("process.env.ENABLE_1M_CONTEXT = '1'"),
+    "ENABLE_1M_CONTEXT = '1' must be set alongside CLAUDE_AUTOCOMPACT_PCT_OVERRIDE"
+  )
+})
+
+console.log(`\n--- ${passed + failed} tests: ${passed} passed, ${failed} failed ---\n`)
+if (failed > 0) process.exit(1)