@polderlabs/bizar 5.5.2 → 5.5.4

package/bizar-dash/src/server/memory-git.mjs

@@ -9,16 +9,60 @@
  * addFile, addAll.
  */
 
-import { execFileSync, execSync } from 'node:child_process';
+import { execFileSync as _execFileSync, execSync } from 'node:child_process';
 import { existsSync, readFileSync, writeFileSync, mkdirSync, unlinkSync } from 'node:fs';
 import { join } from 'node:path';
 
+/**
+ * Resolve the `git` binary path once at module load time.
+ *
+ * When the dashboard runs under systemd, PATH typically does not include
+ * /usr/bin — resulting in `spawnSync git ENOENT` even though git IS
+ * installed.  This helper locates git using `which`, then falls back to
+ * common installation paths before returning an absolute path (or 'git'
+ * as a last-resort).
+ *
+ * @returns {string} absolute path to git, or 'git' as a best-effort fallback
+ */
+export function resolveGitBinary() {
+  // Try `which git` first
+  try {
+    const out = _execFileSync('which', ['git'], {
+      encoding: 'utf8',
+      stdio: ['pipe', 'pipe', 'pipe'],
+    }).trim();
+    if (out) return out;
+  } catch { /* which not available or failed */ }
+
+  // Fallback to common absolute paths
+  const fallbacks = ['/usr/bin/git', '/usr/local/bin/git', '/opt/homebrew/bin/git', '/opt/git/bin/git'];
+  for (const f of fallbacks) {
+    try {
+      _execFileSync(f, ['--version'], { stdio: ['pipe', 'pipe', 'pipe'] });
+      return f;
+    } catch { /* keep looking */ }
+  }
+
+  return 'git'; // best-effort fallback — will only work if PATH is correct
+}
+
+/** Cached absolute path to the git binary. */
+const GIT_BIN = resolveGitBinary();
+
+/**
+ * Wrapper around execFileSync that always uses the resolved GIT_BIN path.
+ * This ensures git is found even when PATH is stripped (e.g. systemd units).
+ */
+function execFileSync(args, opts) {
+  return _execFileSync(GIT_BIN, args, opts);
+}
+
 /**
  * @returns {boolean} true if `git --version` exits 0.
  */
 export function isGitInstalled() {
   try {
-    execFileSync('git', ['--version'], { encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
+    execFileSync(GIT_BIN, ['--version'], { encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
     return true;
   } catch {
     return false;
@@ -36,7 +80,7 @@ export function isGitInstalled() {
 export function clone(remote, targetDir, { branch = 'main', depth = 1 } = {}) {
   try {
     const args = ['clone', '--branch', branch, '--depth', String(depth), remote, targetDir];
-    const output = execFileSync('git', args, { encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
+    const output = execFileSync(args, { encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
     return { ok: true, output };
   } catch (err) {
     return { ok: false, output: '', error: err.message };
@@ -53,7 +97,7 @@ export function clone(remote, targetDir, { branch = 'main', depth = 1 } = {}) {
 export function pull(dir, { rebase = true } = {}) {
   try {
     const args = rebase ? ['pull', '--rebase'] : ['pull', '--ff-only'];
-    const output = execFileSync('git', args, { cwd: dir, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
+    const output = execFileSync(args, { cwd: dir, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
     return { ok: true, output };
   } catch (err) {
     return { ok: false, output: '', error: err.message };
@@ -74,7 +118,7 @@ export function commit(dir, message, { author } = {}) {
     if (author) {
       args.push('--author', author);
     }
-    const output = execFileSync('git', args, { cwd: dir, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
+    const output = execFileSync(args, { cwd: dir, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
     return { ok: true, output };
   } catch (err) {
     return { ok: false, output: '', error: err.message };
@@ -91,7 +135,6 @@ export function commit(dir, message, { author } = {}) {
 export function push(dir, { remote = 'origin', branch = 'main' } = {}) {
   try {
     const output = execFileSync(
-      'git',
       ['push', remote, branch],
       { cwd: dir, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] },
     );
@@ -112,7 +155,7 @@ export function status(dir) {
     // Get branch name
     let branch = 'main';
     try {
-      branch = execFileSync('git', ['branch', '--show-current'], {
+      branch = execFileSync(['branch', '--show-current'], {
         cwd: dir,
         encoding: 'utf8',
         stdio: ['pipe', 'pipe', 'pipe'],
@@ -120,7 +163,7 @@ export function status(dir) {
     } catch { /* ignore */ }
 
     // Get status --porcelain
-    const raw = execFileSync('git', ['status', '--porcelain'], {
+    const raw = execFileSync(['status', '--porcelain'], {
       cwd: dir,
       encoding: 'utf8',
       stdio: ['pipe', 'pipe', 'pipe'],
@@ -143,13 +186,13 @@ export function status(dir) {
     let ahead = 0;
     let behind = 0;
     try {
-      const revparse = execFileSync('git', ['revparse', '--abbrev-ref', '@{upstream}'], {
+      const revparse = execFileSync(['rev-parse', '--abbrev-ref', '@{upstream}'], {
         cwd: dir,
         encoding: 'utf8',
         stdio: ['pipe', 'pipe', 'pipe'],
       }).trim();
       if (revparse) {
-        const aheadBehind = execFileSync('git', ['rev-list', '--left-right', '--count', `${branch}...${revparse}`], {
+        const aheadBehind = execFileSync(['rev-list', '--left-right', '--count', `${branch}...${revparse}`], {
           cwd: dir,
           encoding: 'utf8',
           stdio: ['pipe', 'pipe', 'pipe'],
@@ -226,7 +269,7 @@ export function acquireLock(repoDir) {
  */
 export function addFile(dir, filePath) {
   try {
-    execFileSync('git', ['add', filePath], { cwd: dir, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
+    execFileSync(['add', filePath], { cwd: dir, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
     return { ok: true };
   } catch (err) {
     return { ok: false, error: err.message };
@@ -241,7 +284,7 @@ export function addFile(dir, filePath) {
  */
 export function addAll(dir) {
   try {
-    execFileSync('git', ['add', '-A'], { cwd: dir, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
+    execFileSync(['add', '-A'], { cwd: dir, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] });
     return { ok: true };
   } catch (err) {
     return { ok: false, error: err.message };
@@ -262,7 +305,7 @@ export function addAll(dir) {
 export function addRemote(repoDir, remoteName, url, { overwrite = false } = {}) {
   let existing = null;
   try {
-    existing = execFileSync('git', ['remote', 'get-url', remoteName], {
+    existing = execFileSync(['remote', 'get-url', remoteName], {
       cwd: repoDir,
       encoding: 'utf8',
       timeout: 5000,
@@ -287,7 +330,7 @@ export function addRemote(repoDir, remoteName, url, { overwrite = false } = {})
   }
 
   if (existing) {
-    execFileSync('git', ['remote', 'set-url', remoteName, url], {
+    execFileSync(['remote', 'set-url', remoteName, url], {
       cwd: repoDir,
       timeout: 5000,
       stdio: ['pipe', 'pipe', 'pipe'],
@@ -295,7 +338,7 @@ export function addRemote(repoDir, remoteName, url, { overwrite = false } = {})
     return { ok: true, action: 'updated' };
   }
 
-  execFileSync('git', ['remote', 'add', remoteName, url], {
+  execFileSync(['remote', 'add', remoteName, url], {
     cwd: repoDir,
     timeout: 5000,
     stdio: ['pipe', 'pipe', 'pipe'],
@@ -315,7 +358,7 @@ export function addRemote(repoDir, remoteName, url, { overwrite = false } = {})
  */
 export function lsRemote(repoDir, remoteName, { timeoutMs = 5000 } = {}) {
   try {
-    const stdout = execFileSync('git', ['ls-remote', remoteName], {
+    const stdout = execFileSync(['ls-remote', remoteName], {
       cwd: repoDir,
       timeout: timeoutMs,
       env: { ...process.env, GIT_TERMINAL_PROMPT: '0' },
@@ -356,7 +399,7 @@ export function ensureUpstream(cwd, branch, remote = 'origin') {
   // Current upstream — empty string means none configured.
   let current = '';
   try {
-    current = execFileSync('git', ['rev-parse', '--abbrev-ref', '@{upstream}'], {
+    current = execFileSync(['rev-parse', '--abbrev-ref', '@{upstream}'], {
       cwd,
       encoding: 'utf8',
       stdio: ['pipe', 'pipe', 'pipe'],
@@ -371,7 +414,7 @@ export function ensureUpstream(cwd, branch, remote = 'origin') {
 
   // Verify the remote exists before trying to set upstream to it.
   try {
-    execFileSync('git', ['rev-parse', '--verify', remote], {
+    execFileSync(['rev-parse', '--verify', remote], {
       cwd,
       stdio: ['pipe', 'pipe', 'pipe'],
     });
@@ -380,7 +423,7 @@ export function ensureUpstream(cwd, branch, remote = 'origin') {
   }
 
   try {
-    execFileSync('git', ['branch', '--set-upstream-to', `${remote}/${branch}`], {
+    execFileSync(['branch', '--set-upstream-to', `${remote}/${branch}`], {
       cwd,
       stdio: ['pipe', 'pipe', 'pipe'],
     });

package/bizar-dash/src/server/memory-store.mjs

@@ -18,7 +18,7 @@
  */
 
 import { existsSync, readFileSync, writeFileSync, mkdirSync, readdirSync, statSync, lstatSync, unlinkSync } from 'node:fs';
-import { execFileSync } from 'node:child_process';
+import { execFileSync as _execFileSync } from 'node:child_process';
 import { join, dirname, relative, resolve as pathResolve, sep } from 'node:path';
 import { homedir } from 'node:os';
 import { parseFrontmatter, serializeFrontmatter } from './yaml.mjs';
@@ -28,6 +28,43 @@ import { atomicWriteJson, safeReadJSON, safeReadText } from '../../../cli/atomic
 import * as memoryGit from './memory-git.mjs';
 import { warn as loggerWarn } from './logger.mjs';
 
+/**
+ * Resolve the `git` binary path once at module load time.
+ *
+ * When the dashboard runs under systemd, PATH typically does not include
+ * /usr/bin — this helper finds git's absolute path so that all
+ * execFileSync calls work regardless of the process's environment.
+ *
+ * @returns {string} absolute path to git, or 'git' as a best-effort fallback
+ */
+function resolveGitBinary() {
+  try {
+    const out = _execFileSync('which', ['git'], {
+      encoding: 'utf8',
+      stdio: ['pipe', 'pipe', 'pipe'],
+    }).trim();
+    if (out) return out;
+  } catch { /* which not available or failed */ }
+
+  const fallbacks = ['/usr/bin/git', '/usr/local/bin/git', '/opt/homebrew/bin/git', '/opt/git/bin/git'];
+  for (const f of fallbacks) {
+    try {
+      _execFileSync(f, ['--version'], { stdio: ['pipe', 'pipe', 'pipe'] });
+      return f;
+    } catch { /* keep looking */ }
+  }
+
+  return 'git';
+}
+
+/** Cached absolute path to the git binary. */
+const GIT_BIN = resolveGitBinary();
+
+/** Wrapper that always uses the resolved GIT_BIN path. */
+function execFileSync(args, opts) {
+  return _execFileSync(GIT_BIN, args, opts);
+}
+
 const HOME = homedir();
 
 /**
@@ -99,9 +136,9 @@ export function ensureVaultExists() {
     mkdirSync(vault, { recursive: true, mode: 0o700 });
     if (!existsSync(join(vault, '.git'))) {
       try {
-        execFileSync('git', ['init'], { cwd: vault, stdio: 'pipe' });
-        execFileSync('git', ['config', 'user.email', 'bizar@localhost'], { cwd: vault, stdio: 'pipe' });
-        execFileSync('git', ['config', 'user.name', 'BizarHarness'], { cwd: vault, stdio: 'pipe' });
+        execFileSync(['init'], { cwd: vault, stdio: 'pipe' });
+        execFileSync(['config', 'user.email', 'bizar@localhost'], { cwd: vault, stdio: 'pipe' });
+        execFileSync(['config', 'user.name', 'BizarHarness'], { cwd: vault, stdio: 'pipe' });
       } catch (err) {
         loggerWarn('failed to init git in vault', { vault, err: err.message });
       }
@@ -318,7 +355,7 @@ export function initVault(projectRoot) {
     const isGit = existsSync(join(repoPath, '.git'));
     if (!isGit) {
       try {
-        execFileSync('git', ['init', '-b', 'main'], { cwd: repoPath, stdio: 'pipe' });
+        execFileSync(['init', '-b', 'main'], { cwd: repoPath, stdio: 'pipe' });
         created.push(`${repoPath}/.git/`);
       } catch (err) {
         return { ok: false, vaultRoot, created, error: `git init failed: ${err.message}` };
@@ -339,8 +376,8 @@ export function initVault(projectRoot) {
           'utf8');
         created.push(`${repoPath}/.gitignore`);
         try {
-          execFileSync('git', ['add', '.gitignore'], { cwd: repoPath });
-          execFileSync('git', ['commit', '-m', 'chore: initial .gitignore for memory repo'], {
+          execFileSync(['add', '.gitignore'], { cwd: repoPath });
+          execFileSync(['commit', '-m', 'chore: initial .gitignore for memory repo'], {
             cwd: repoPath,
             env: {
               ...process.env,

package/bizar-dash/src/web/components/SettingsNav.tsx

@@ -1,7 +1,11 @@
 // src/components/SettingsNav.tsx — v4.9.0 full-sidebar settings navigation.
 // Shown in the sidebar rail when settingsMode is active. All sections are
 // visible at once; clicking one scrolls the Settings view to that section.
-import { ChevronRight, Palette, Terminal, Cpu, RefreshCw, Gauge, ArrowLeft, LayoutGrid, type LucideIcon } from 'lucide-react';
+import {
+  ChevronRight, Palette, Terminal, Cpu, RefreshCw, Gauge, ArrowLeft,
+  LayoutGrid, Wifi, Bell, Shield, Users, Activity, FolderGit2,
+  type LucideIcon,
+} from 'lucide-react';
 import { cn } from '../lib/utils';
 
 export type SettingsSection = {
@@ -20,17 +24,26 @@ const SECTION_GROUPS: SettingsSectionGroup[] = [
     label: 'General',
     sections: [
       { id: 'theme', label: 'Theme', icon: Palette },
+      // Layout & General are both rendered by GeneralSection; one nav item covers both
       { id: 'layout', label: 'Layout', icon: LayoutGrid },
-      { id: 'general', label: 'General', icon: LayoutGrid },
     ],
   },
   {
     label: 'Core',
     sections: [
       { id: 'env-vars', label: 'Env Vars', icon: Terminal },
+      { id: 'network', label: 'Network', icon: Wifi },
+      { id: 'notifications', label: 'Notifications', icon: Bell },
+      { id: 'auth', label: 'Auth', icon: Shield },
       { id: 'system-llm', label: 'System LLM', icon: Cpu },
     ],
   },
+  {
+    label: 'Agents',
+    sections: [
+      { id: 'agents', label: 'Agents', icon: Users },
+    ],
+  },
   {
     label: 'Experience',
     sections: [
@@ -38,6 +51,13 @@ const SECTION_GROUPS: SettingsSectionGroup[] = [
       { id: 'headroom', label: 'Headroom', icon: Gauge },
     ],
   },
+  {
+    label: 'Data',
+    sections: [
+      { id: 'activity-log', label: 'Activity', icon: Activity },
+      { id: 'workspaces', label: 'Workspaces', icon: FolderGit2 },
+    ],
+  },
 ];
 
 export type SettingsNavProps = {
@@ -52,12 +72,12 @@ export function SettingsNav({ activeSection, onSectionChange, onExitSettings }:
       {/* Back button */}
       <button
         type="button"
-        className="settings-nav-back"
+        className="sidebar-tab"
         onClick={onExitSettings}
         aria-label="Exit settings"
       >
-        <ArrowLeft size={14} />
-        <span>Back</span>
+        <ArrowLeft size={14} aria-hidden />
+        <span className="sidebar-tab-label">Back</span>
       </button>
 
       <div className="settings-nav-divider" aria-hidden="true" />
@@ -73,14 +93,14 @@ export function SettingsNav({ activeSection, onSectionChange, onExitSettings }:
               <button
                 key={s.id}
                 type="button"
-                className={cn('settings-nav-item', active && 'settings-nav-item-active')}
+                className={cn('sidebar-tab', active && 'sidebar-tab-active')}
                 onClick={() => onSectionChange(active ? null : s.id)}
                 aria-current={active ? 'page' : undefined}
               >
-                {Icon && <Icon size={14} className="settings-nav-item-icon" aria-hidden />}
-                <span className="settings-nav-item-label">{s.label}</span>
+                {Icon && <Icon size={14} aria-hidden />}
+                <span className="sidebar-tab-label">{s.label}</span>
                 {active && (
-                  <ChevronRight size={12} className="settings-nav-item-chevron" aria-hidden />
+                  <ChevronRight size={12} aria-hidden />
                 )}
               </button>
             );

package/bizar-dash/src/web/styles/settings.css

@@ -617,28 +617,6 @@
   gap: 2px;
 }
 
-/* Back button */
-.settings-nav-back {
-  display: flex;
-  align-items: center;
-  gap: 6px;
-  padding: 6px 8px;
-  border-radius: var(--radius);
-  border: none;
-  background: transparent;
-  color: var(--text-dim);
-  font-size: 13px;
-  font-weight: 500;
-  cursor: pointer;
-  transition: background var(--motion-fast), color var(--motion-fast);
-  margin-bottom: var(--space-2);
-}
-
-.settings-nav-back:hover {
-  background: var(--bg-elev-2);
-  color: var(--text);
-}
-
 .settings-nav-divider {
   height: 1px;
   background: var(--border);
@@ -659,58 +637,6 @@
   padding: 4px 8px 6px;
 }
 
-/* Individual nav items */
-.settings-nav-item {
-  display: flex;
-  align-items: center;
-  gap: 8px;
-  padding: 6px 8px;
-  border-radius: var(--radius);
-  border: 1px solid transparent;
-  background: transparent;
-  color: var(--text-dim);
-  font-size: 13px;
-  font-weight: 500;
-  cursor: pointer;
-  text-align: left;
-  width: 100%;
-  transition: background var(--motion-fast), color var(--motion-fast), border-color var(--motion-fast);
-}
-
-.settings-nav-item:hover {
-  background: var(--bg-elev-2);
-  color: var(--text);
-}
-
-.settings-nav-item-active {
-  background: var(--accent-bg);
-  color: var(--accent-2);
-  border-color: var(--accent-border);
-}
-
-.settings-nav-item-active:hover {
-  background: var(--accent-bg);
-  color: var(--accent-2);
-}
-
-.settings-nav-item-icon {
-  flex-shrink: 0;
-  opacity: 0.7;
-}
-
-.settings-nav-item-active .settings-nav-item-icon {
-  opacity: 1;
-}
-
-.settings-nav-item-label {
-  flex: 1;
-}
-
-.settings-nav-item-chevron {
-  flex-shrink: 0;
-  opacity: 0.5;
-}
-
 /* ─────────────────────────────────────────────────────────────────────
    v5.4 — Mobile settings accordion (mobile-settings)
    ───────────────────────────────────────────────────────────────────── */

package/bizar-dash/src/web/views/Settings.tsx

@@ -288,7 +288,7 @@ function SettingsViewInner({ settings: initial, refreshSnapshot, settingsMode, s
       <div className={cn('settings-grid', activeSection && 'settings-grid-filtered')} data-active-section={activeSection || undefined}>
         {(showAll || sectionOf('theme')) && <div id="settings-theme"><ThemeSection {...sp} /></div>}
         {(showAll || sectionOf('updates')) && <div id="settings-updates"><UpdatesSection /></div>}
-        {(showAll || sectionOf('layout') || sectionOf('general')) && <div id="settings-general"><GeneralSection {...sp} autoSave={autoSave} /></div>}
+        {(showAll || sectionOf('layout')) && <div id="settings-general"><GeneralSection {...sp} autoSave={autoSave} /></div>}
         {(showAll || sectionOf('env-vars')) && <div id="settings-env-vars"><EnvVarsSection /></div>}
         {(showAll || sectionOf('network') || sectionOf('service') || sectionOf('tailscale')) && <div id="settings-network"><NetworkSection tailscale={tailscale} tailscaleDraft={tailscaleDraft} setTailscaleDraft={setTailscaleDraft} onTailscaleToggle={onTailscaleToggle} /></div>}
         {(showAll || sectionOf('network') || sectionOf('tailscale')) && <TailscaleSettings initialStatus={tailscale} />}

package/bizar-dash/tests/memory-git.test.mjs

@@ -15,6 +15,24 @@ const TEST_GIT = await import('../src/server/memory-git.mjs').then((m) => m);
 const GIT_INSTALLED = TEST_GIT.isGitInstalled();
 
 describe('memory-git', () => {
+  describe('resolveGitBinary', () => {
+    it('returns a non-empty string', () => {
+      const { resolveGitBinary } = TEST_GIT;
+      const bin = resolveGitBinary();
+      assert.ok(typeof bin === 'string' && bin.length > 0, `expected non-empty string, got: ${bin}`);
+    });
+
+    it('is a valid git command (absolute path or "git")', () => {
+      const { resolveGitBinary } = TEST_GIT;
+      const bin = resolveGitBinary();
+      // Must be either 'git' or an absolute path (starts with /)
+      assert.ok(
+        bin === 'git' || bin.startsWith('/'),
+        `expected 'git' or absolute path, got: ${bin}`,
+      );
+    });
+  });
+
   let bareRemote;
   let workingDir;
 

package/bizar-dash/tests/settings-layout.test.tsx

@@ -93,7 +93,7 @@ describe('Sidebar — settingsMode', () => {
       />,
     );
     const themeBtn = screen.getByRole('button', { name: /theme/i });
-    expect(themeBtn).toHaveClass('settings-nav-item-active');
+    expect(themeBtn).toHaveClass('sidebar-tab-active');
   });
 
   it('calls onSettingsSectionChange when a section is clicked', async () => {

package/bizar-dash/tests/settings-nav.test.tsx

@@ -51,7 +51,9 @@ describe('SettingsNav', () => {
     );
     expect(screen.getByText('General', { selector: '.settings-nav-group-label' })).toBeInTheDocument();
     expect(screen.getByText('Core', { selector: '.settings-nav-group-label' })).toBeInTheDocument();
+    expect(screen.getByText('Agents', { selector: '.settings-nav-group-label' })).toBeInTheDocument();
     expect(screen.getByText('Experience', { selector: '.settings-nav-group-label' })).toBeInTheDocument();
+    expect(screen.getByText('Data', { selector: '.settings-nav-group-label' })).toBeInTheDocument();
   });
 
   it('renders section items within each group', () => {
@@ -65,7 +67,6 @@ describe('SettingsNav', () => {
     // General group
     expect(screen.getByRole('button', { name: /theme/i })).toBeInTheDocument();
     expect(screen.getByRole('button', { name: /layout/i })).toBeInTheDocument();
-    expect(screen.getByRole('button', { name: /general/i })).toBeInTheDocument();
     // Core group
     expect(screen.getByRole('button', { name: /env vars/i })).toBeInTheDocument();
     expect(screen.getByRole('button', { name: /system llm/i })).toBeInTheDocument();
@@ -96,7 +97,7 @@ describe('SettingsNav', () => {
       />,
     );
     const themeBtn = screen.getByRole('button', { name: /theme/i });
-    expect(themeBtn).toHaveClass('settings-nav-item-active');
+    expect(themeBtn).toHaveClass('sidebar-tab-active');
   });
 
   it('toggles off the active section when clicking it again', async () => {

package/cli/doctor.mjs

@@ -222,7 +222,10 @@ async function checkProviderConfigSanity() {
   const cfg = JSON.parse(readFileSync(cfgPath, 'utf8'));
   const minimax = cfg.provider && cfg.provider.minimax;
   if (!minimax) {
-    throw new Error('provider.minimax block missing');
+    // Warn instead of throw — provision.mjs auto-adds this block on
+    // install/update, but users with an older pre-v5 opencode.json
+    // may not have it yet.
+    return 'warn: provider.minimax block missing (run `bizar update` to patch)';
   }
   const models = minimax.models || {};
   const saneNames = Object.entries(models).filter(([, m]) => {

package/cli/doctor.test.mjs

@@ -302,12 +302,14 @@ describe('runDoctor() with fixture HOME', () => {
     assert.equal(r.ok, true, r.message);
   });
 
-  test('provider-config-sanity fails without minimax block', async () => {
+  test('provider-config-sanity warns (not fails) without minimax block', async () => {
+    // v5.x: checkProviderConfigSanity warns instead of throwing when the
+    // provider.minimax block is missing, since provision.mjs auto-adds it.
     writeOpencodeConfig({ provider: {} });
     const result = await runDoctor({ silent: true });
     const r = findCheck(result, 'provider-config-sanity');
-    assert.equal(r.ok, false);
-    assert.match(r.message, /minimax/);
+    assert.equal(r.ok, true, 'should pass with warning, not throw');
+    assert.match(r.message, /warn.*minimax|minimax.*missing/i);
   });
 
   test('provider-config-sanity fails when models lack interleaved+reasoning', async () => {