@polderlabs/bizar 4.9.0 → 5.0.1

package/bizar-dash/tests/schedules-templates.test.mjs

@@ -0,0 +1,144 @@
+/**
+ * bizar-dash/tests/schedules-templates.test.mjs
+ *
+ * Verify schedule template loading and from-template endpoint.
+ */
+import { describe, it } from 'node:test';
+import assert from 'node:assert';
+import { readFileSync, existsSync, readdirSync } from 'node:fs';
+import { join, dirname } from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const __dirname = fileURLToPath(new URL('.', import.meta.url));
+const PROJECT_ROOT = join(__dirname, '..', '..');
+const TEMPLATES_DIR = join(PROJECT_ROOT, 'templates', 'schedules');
+
+describe('Schedule templates', () => {
+  it('templates/schedules directory exists', () => {
+    assert.ok(existsSync(TEMPLATES_DIR), 'Missing templates/schedules directory');
+  });
+
+  it('at least 4 template files exist', () => {
+    const files = readdirSync(TEMPLATES_DIR).filter((f) => f.endsWith('.json'));
+    assert.ok(files.length >= 4, `Expected >=4 templates, found ${files.length}: ${files.join(', ')}`);
+  });
+
+  it('each template is valid JSON with required fields', () => {
+    const files = readdirSync(TEMPLATES_DIR).filter((f) => f.endsWith('.json'));
+    for (const file of files) {
+      const content = readFileSync(join(TEMPLATES_DIR, file), 'utf8');
+      const template = JSON.parse(content);
+      assert.ok(template.id, `${file}: missing id`);
+      assert.ok(template.name, `${file}: missing name`);
+      assert.ok(template.description, `${file}: missing description`);
+      assert.ok(template.type, `${file}: missing type`);
+      assert.ok(template.action, `${file}: missing action`);
+      assert.strictEqual(template.source, undefined, `${file}: should NOT have source field (added at load time)`);
+    }
+  });
+
+  it('daily-backup.json is a valid cron schedule template', () => {
+    const content = readFileSync(join(TEMPLATES_DIR, 'daily-backup.json'), 'utf8');
+    const t = JSON.parse(content);
+    assert.strictEqual(t.type, 'cron');
+    assert.strictEqual(t.schedule, '0 3 * * *');
+    assert.strictEqual(t.action.type, 'command');
+    assert.ok(t.action.target.includes('bizar backup'));
+  });
+
+  it('weekly-digest.json is a valid agent template', () => {
+    const content = readFileSync(join(TEMPLATES_DIR, 'weekly-digest.json'), 'utf8');
+    const t = JSON.parse(content);
+    assert.strictEqual(t.type, 'cron');
+    assert.strictEqual(t.action.type, 'agent');
+    assert.ok(t.action.prompt);
+  });
+
+  it('hourly-health-check.json is a valid interval template', () => {
+    const content = readFileSync(join(TEMPLATES_DIR, 'hourly-health-check.json'), 'utf8');
+    const t = JSON.parse(content);
+    assert.strictEqual(t.type, 'interval');
+    assert.strictEqual(t.action.type, 'command');
+  });
+
+  it('webhook-on-push.json has webhook action', () => {
+    const content = readFileSync(join(TEMPLATES_DIR, 'webhook-on-push.json'), 'utf8');
+    const t = JSON.parse(content);
+    assert.strictEqual(t.action.type, 'webhook');
+    assert.ok(t.action.target);
+  });
+});
+
+describe('loadTemplates (unit)', () => {
+  // Re-implement loadTemplates here to test in isolation
+  function loadTemplates() {
+    if (!existsSync(TEMPLATES_DIR)) return [];
+    return readdirSync(TEMPLATES_DIR)
+      .filter((f) => f.endsWith('.json'))
+      .map((f) => {
+        const content = JSON.parse(readFileSync(join(TEMPLATES_DIR, f), 'utf8'));
+        return { ...content, source: 'template', templateFile: f };
+      });
+  }
+
+  it('returns array with source=template and templateFile set', () => {
+    const templates = loadTemplates();
+    assert.ok(Array.isArray(templates));
+    assert.ok(templates.length > 0);
+    for (const t of templates) {
+      assert.strictEqual(t.source, 'template', 'each template should have source=template');
+      assert.ok(t.templateFile.endsWith('.json'), 'templateFile should be a .json file');
+    }
+  });
+
+  it('missing templates dir returns empty array', () => {
+    // Use a path that definitely doesn't exist
+    const fakeDir = join(TEMPLATES_DIR, '..', 'schedules-does-not-exist');
+    function loadFromFake() {
+      if (!existsSync(fakeDir)) return [];
+      return readdirSync(fakeDir).filter((f) => f.endsWith('.json'));
+    }
+    const result = loadFromFake();
+    assert.deepStrictEqual(result, []);
+  });
+});
+
+describe('from-template endpoint contract', () => {
+  it('template payload can be used to create a valid schedule payload', () => {
+    const content = readFileSync(join(TEMPLATES_DIR, 'daily-backup.json'), 'utf8');
+    const template = JSON.parse(content);
+
+    // Simulate what the route does
+    const schedulePayload = {
+      name: template.name,
+      type: template.type,
+      schedule: template.schedule,
+      timezone: template.timezone || 'UTC',
+      action: template.action,
+      enabled: true,
+    };
+
+    assert.ok(schedulePayload.name);
+    assert.ok(['cron', 'interval', 'once'].includes(schedulePayload.type));
+    assert.ok(schedulePayload.schedule);
+    assert.ok(schedulePayload.action);
+    assert.strictEqual(schedulePayload.enabled, true);
+  });
+
+  it('custom name can override template name', () => {
+    const content = readFileSync(join(TEMPLATES_DIR, 'daily-backup.json'), 'utf8');
+    const template = JSON.parse(content);
+    const customName = 'My Custom Backup';
+
+    const schedulePayload = {
+      name: customName || template.name,
+      type: template.type,
+      schedule: template.schedule,
+      timezone: template.timezone || 'UTC',
+      action: template.action,
+      enabled: true,
+    };
+
+    assert.strictEqual(schedulePayload.name, customName);
+  });
+});

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

@@ -0,0 +1,129 @@
+/**
+ * tests/settings-layout.test.tsx
+ *
+ * v4.9.0 — Integration tests for the settings sidebar layout mode.
+ * Verifies that when settingsMode is active, the sidebar shows SettingsNav
+ * and clicking sections updates the active section.
+ */
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { render, screen } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
+import React from 'react';
+import { Sidebar } from '../src/web/components/Sidebar';
+import type { TabDef } from '../src/web/components/Topbar';
+import { LayoutDashboard, MessageSquare, Sliders } from 'lucide-react';
+
+const TABS: TabDef[] = [
+  { id: 'overview', label: 'Overview', icon: LayoutDashboard },
+  { id: 'chat', label: 'Chat', icon: MessageSquare },
+  { id: 'settings', label: 'Settings', icon: Sliders },
+];
+
+describe('Sidebar — settingsMode', () => {
+  const onTabChange = vi.fn();
+  const onSettingsSectionChange = vi.fn();
+  const onExitSettings = vi.fn();
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it('renders normal tab rail when settingsMode is false', () => {
+    render(
+      <Sidebar
+        tabs={TABS}
+        activeTab="overview"
+        onTabChange={onTabChange}
+        settingsMode={false}
+      />,
+    );
+    expect(screen.getByRole('tablist')).toBeInTheDocument();
+    expect(screen.getByRole('tab', { name: /overview/i })).toBeInTheDocument();
+    expect(screen.getByRole('tab', { name: /chat/i })).toBeInTheDocument();
+  });
+
+  it('renders SettingsNav instead of tab rail when settingsMode is true', () => {
+    render(
+      <Sidebar
+        tabs={TABS}
+        activeTab="settings"
+        onTabChange={onTabChange}
+        settingsMode={true}
+        settingsActiveSection={null}
+        onSettingsSectionChange={onSettingsSectionChange}
+        onExitSettings={onExitSettings}
+      />,
+    );
+    // Should NOT show the normal tab rail
+    expect(screen.queryByRole('tablist')).not.toBeInTheDocument();
+    // Should show the back button
+    expect(screen.getByRole('button', { name: /back/i })).toBeInTheDocument();
+    // Should show section groups (use selector to avoid duplicate text matches)
+    expect(screen.getByText('General', { selector: '.settings-nav-group-label' })).toBeInTheDocument();
+    expect(screen.getByText('Core', { selector: '.settings-nav-group-label' })).toBeInTheDocument();
+  });
+
+  it('shows Back button that calls onExitSettings', async () => {
+    const user = userEvent.setup();
+    render(
+      <Sidebar
+        tabs={TABS}
+        activeTab="settings"
+        onTabChange={onTabChange}
+        settingsMode={true}
+        settingsActiveSection={null}
+        onSettingsSectionChange={onSettingsSectionChange}
+        onExitSettings={onExitSettings}
+      />,
+    );
+    await user.click(screen.getByText('Back'));
+    expect(onExitSettings).toHaveBeenCalledTimes(1);
+  });
+
+  it('highlights the active section when settingsActiveSection is set', () => {
+    render(
+      <Sidebar
+        tabs={TABS}
+        activeTab="settings"
+        onTabChange={onTabChange}
+        settingsMode={true}
+        settingsActiveSection="theme"
+        onSettingsSectionChange={onSettingsSectionChange}
+        onExitSettings={onExitSettings}
+      />,
+    );
+    const themeBtn = screen.getByRole('button', { name: /theme/i });
+    expect(themeBtn).toHaveClass('settings-nav-item-active');
+  });
+
+  it('calls onSettingsSectionChange when a section is clicked', async () => {
+    const user = userEvent.setup();
+    render(
+      <Sidebar
+        tabs={TABS}
+        activeTab="settings"
+        onTabChange={onTabChange}
+        settingsMode={true}
+        settingsActiveSection={null}
+        onSettingsSectionChange={onSettingsSectionChange}
+        onExitSettings={onExitSettings}
+      />,
+    );
+    await user.click(screen.getByRole('button', { name: /memory/i }));
+    expect(onSettingsSectionChange).toHaveBeenCalledWith('memory');
+  });
+
+  it('calls onTabChange when a normal sidebar tab is clicked', async () => {
+    const user = userEvent.setup();
+    render(
+      <Sidebar
+        tabs={TABS}
+        activeTab="overview"
+        onTabChange={onTabChange}
+        settingsMode={false}
+      />,
+    );
+    await user.click(screen.getByRole('tab', { name: /chat/i }));
+    expect(onTabChange).toHaveBeenCalledWith('chat');
+  });
+});

package/bizar-dash/tests/settings-mode-wiring.test.tsx

@@ -0,0 +1,151 @@
+/**
+ * tests/settings-mode-wiring.test.tsx
+ *
+ * v4.9.0 — Wiring tests for settings mode.
+ * Verifies:
+ * 1. Topbar Settings tab calls onTabChange('settings') when clicked
+ * 2. Sidebar renders SettingsNav when settingsMode is true
+ * 3. Sidebar renders normal tab rail when settingsMode is false
+ */
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { render, screen } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
+import React from 'react';
+import { Sidebar } from '../src/web/components/Sidebar';
+import { Topbar, TABS } from '../src/web/components/Topbar';
+import type { TabDef } from '../src/web/components/Topbar';
+
+// Minimal tabs for testing
+const TEST_TABS: TabDef[] = TABS.slice(0, 4);
+
+describe('Settings mode wiring', () => {
+  describe('Topbar', () => {
+    it('calls onTabChange with "settings" when the Settings tab is clicked', async () => {
+      const onTabChange = vi.fn();
+      const user = userEvent.setup();
+
+      render(
+        <Topbar
+          activeTab="overview"
+          onTabChange={onTabChange}
+          wsStatus="connected"
+          version="v4.9.0"
+          activeProject={null}
+          projects={[]}
+          onProjectChange={vi.fn()}
+          onProjectsRefresh={vi.fn()}
+          onOpenSearch={vi.fn()}
+        />,
+      );
+
+      // Find and click the Settings tab
+      const settingsTab = screen.getByRole('tab', { name: /settings/i });
+      await user.click(settingsTab);
+
+      expect(onTabChange).toHaveBeenCalledWith('settings');
+    });
+
+    it('calls onTabChange with "overview" when the Overview tab is clicked', async () => {
+      const onTabChange = vi.fn();
+      const user = userEvent.setup();
+
+      render(
+        <Topbar
+          activeTab="settings"
+          onTabChange={onTabChange}
+          wsStatus="connected"
+          version="v4.9.0"
+          activeProject={null}
+          projects={[]}
+          onProjectChange={vi.fn()}
+          onProjectsRefresh={vi.fn()}
+          onOpenSearch={vi.fn()}
+        />,
+      );
+
+      const overviewTab = screen.getByRole('tab', { name: /overview/i });
+      await user.click(overviewTab);
+
+      expect(onTabChange).toHaveBeenCalledWith('overview');
+    });
+  });
+
+  describe('Sidebar', () => {
+    const onTabChange = vi.fn();
+    const onSettingsSectionChange = vi.fn();
+    const onExitSettings = vi.fn();
+
+    beforeEach(() => {
+      vi.clearAllMocks();
+    });
+
+    it('renders SettingsNav when settingsMode is true', () => {
+      render(
+        <Sidebar
+          tabs={TEST_TABS}
+          activeTab="settings"
+          onTabChange={onTabChange}
+          settingsMode={true}
+          settingsActiveSection={null}
+          onSettingsSectionChange={onSettingsSectionChange}
+          onExitSettings={onExitSettings}
+        />,
+      );
+
+      // SettingsNav should be visible (renders the back button)
+      expect(screen.getByRole('button', { name: /exit settings/i })).toBeInTheDocument();
+      // Normal tab rail should NOT be visible
+      expect(screen.queryByRole('tablist')).not.toBeInTheDocument();
+    });
+
+    it('renders normal tab rail when settingsMode is false', () => {
+      render(
+        <Sidebar
+          tabs={TEST_TABS}
+          activeTab="overview"
+          onTabChange={onTabChange}
+          settingsMode={false}
+        />,
+      );
+
+      // Tab rail should be visible
+      expect(screen.getByRole('tablist')).toBeInTheDocument();
+      // SettingsNav should NOT be visible
+      expect(screen.queryByRole('button', { name: /back/i })).not.toBeInTheDocument();
+    });
+
+    it('exits settings mode when back button is clicked', async () => {
+      const user = userEvent.setup();
+      render(
+        <Sidebar
+          tabs={TEST_TABS}
+          activeTab="settings"
+          onTabChange={onTabChange}
+          settingsMode={true}
+          settingsActiveSection={null}
+          onSettingsSectionChange={onSettingsSectionChange}
+          onExitSettings={onExitSettings}
+        />,
+      );
+
+      await user.click(screen.getByRole('button', { name: /exit settings/i }));
+      expect(onExitSettings).toHaveBeenCalledTimes(1);
+    });
+
+    it('calls onTabChange when a tab is clicked while in settingsMode=false', async () => {
+      const user = userEvent.setup();
+      render(
+        <Sidebar
+          tabs={TEST_TABS}
+          activeTab="overview"
+          onTabChange={onTabChange}
+          settingsMode={false}
+        />,
+      );
+
+      // Click the Chat tab
+      await user.click(screen.getByRole('tab', { name: /chat/i }));
+      expect(onTabChange).toHaveBeenCalledWith('chat');
+    });
+  });
+});

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

@@ -0,0 +1,126 @@
+/**
+ * tests/settings-nav.test.tsx
+ *
+ * v4.9.0 — Component tests for SettingsNav sidebar component.
+ */
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { render, screen } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
+import React from 'react';
+import { SettingsNav } from '../src/web/components/SettingsNav';
+
+describe('SettingsNav', () => {
+  const onSectionChange = vi.fn();
+  const onExitSettings = vi.fn();
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it('renders the back button', () => {
+    render(
+      <SettingsNav
+        activeSection={null}
+        onSectionChange={onSectionChange}
+        onExitSettings={onExitSettings}
+      />,
+    );
+    expect(screen.getByRole('button', { name: /back/i })).toBeInTheDocument();
+  });
+
+  it('calls onExitSettings when back button is clicked', async () => {
+    const user = userEvent.setup();
+    render(
+      <SettingsNav
+        activeSection={null}
+        onSectionChange={onSectionChange}
+        onExitSettings={onExitSettings}
+      />,
+    );
+    await user.click(screen.getByText('Back'));
+    expect(onExitSettings).toHaveBeenCalledTimes(1);
+  });
+
+  it('renders section groups with group-label class', () => {
+    render(
+      <SettingsNav
+        activeSection={null}
+        onSectionChange={onSectionChange}
+        onExitSettings={onExitSettings}
+      />,
+    );
+    expect(screen.getByText('General', { selector: '.settings-nav-group-label' })).toBeInTheDocument();
+    expect(screen.getByText('Core', { 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', () => {
+    render(
+      <SettingsNav
+        activeSection={null}
+        onSectionChange={onSectionChange}
+        onExitSettings={onExitSettings}
+      />,
+    );
+    expect(screen.getByRole('button', { name: /theme/i })).toBeInTheDocument();
+    expect(screen.getByRole('button', { name: /env vars/i })).toBeInTheDocument();
+    expect(screen.getByRole('button', { name: /memory/i })).toBeInTheDocument();
+    expect(screen.getByRole('button', { name: /system llm/i })).toBeInTheDocument();
+    expect(screen.getByRole('button', { name: /updates/i })).toBeInTheDocument();
+    expect(screen.getByRole('button', { name: /skills/i })).toBeInTheDocument();
+    expect(screen.getByRole('button', { name: /headroom/i })).toBeInTheDocument();
+    expect(screen.getByRole('button', { name: /backup/i })).toBeInTheDocument();
+  });
+
+  it('calls onSectionChange with correct id when a section is clicked', async () => {
+    const user = userEvent.setup();
+    render(
+      <SettingsNav
+        activeSection={null}
+        onSectionChange={onSectionChange}
+        onExitSettings={onExitSettings}
+      />,
+    );
+    await user.click(screen.getByRole('button', { name: /theme/i }));
+    expect(onSectionChange).toHaveBeenCalledWith('theme');
+  });
+
+  it('highlights the active section', () => {
+    render(
+      <SettingsNav
+        activeSection="theme"
+        onSectionChange={onSectionChange}
+        onExitSettings={onExitSettings}
+      />,
+    );
+    const themeBtn = screen.getByRole('button', { name: /theme/i });
+    expect(themeBtn).toHaveClass('settings-nav-item-active');
+  });
+
+  it('toggles off the active section when clicking it again', async () => {
+    const user = userEvent.setup();
+    render(
+      <SettingsNav
+        activeSection="theme"
+        onSectionChange={onSectionChange}
+        onExitSettings={onExitSettings}
+      />,
+    );
+    await user.click(screen.getByRole('button', { name: /theme/i }));
+    expect(onSectionChange).toHaveBeenCalledWith(null); // toggles off
+  });
+
+  it('renders chevron on the active section', () => {
+    render(
+      <SettingsNav
+        activeSection="theme"
+        onSectionChange={onSectionChange}
+        onExitSettings={onExitSettings}
+      />,
+    );
+    const themeBtn = screen.getByRole('button', { name: /theme/i });
+    // The chevron is inside the button as an SVG (aria-hidden), so we just verify the button exists
+    expect(themeBtn).toBeInTheDocument();
+  });
+});

package/bizar-dash/tests/users.test.mjs

@@ -0,0 +1,108 @@
+/**
+ * tests/users.test.mjs
+ *
+ * v5.0.0 — Tests for the users route handlers.
+ */
+import { describe, it, beforeEach, afterEach, after } from 'node:test';
+import assert from 'node:assert';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+import {
+  mkdirSync,
+  rmSync,
+} from 'node:fs';
+
+// These are tested via the REST API in workspaces.test.mjs (user-related functions)
+// This file tests the users store functions directly.
+
+const TEST_ID = `bizar-user-test-${Date.now()}-${Math.random().toString(36).slice(2)}`;
+const TEST_HOME = join(tmpdir(), TEST_ID);
+const BIZAR_LOCAL = join(TEST_HOME, '.local', 'share', 'bizar');
+const WORKSPACES_ROOT = join(BIZAR_LOCAL, 'workspaces');
+
+const originalHomedir = process.env.HOME;
+process.env.HOME = TEST_HOME;
+
+describe('users', () => {
+  let cleanupDirs = [];
+
+  beforeEach(() => {
+    mkdirSync(BIZAR_LOCAL, { recursive: true });
+    cleanupDirs.push(TEST_HOME);
+  });
+
+  afterEach(() => {
+    for (const dir of cleanupDirs) {
+      try { rmSync(dir, { recursive: true, force: true }); } catch { /* ignore */ }
+    }
+    cleanupDirs = [];
+  });
+
+  after(() => {
+    process.env.HOME = originalHomedir;
+  });
+
+  // Users are managed through the workspaces module (index.json users array)
+  // This test verifies user CRUD through workspaces
+
+  describe('user CRUD via workspaces module', () => {
+    it('getOrCreateUser creates user and persists to index', async () => {
+      const ws = await import('../src/server/workspaces.mjs');
+      const { user, created } = await ws.getOrCreateUser('test@example.com', 'Test User');
+
+      assert.ok(created);
+      assert.ok(user.id.startsWith('usr_'));
+      assert.strictEqual(user.email, 'test@example.com');
+      assert.strictEqual(user.name, 'Test User');
+
+      // Verify persisted
+      const found = await ws.getUser(user.id);
+      assert.ok(found);
+      assert.strictEqual(found.email, 'test@example.com');
+    });
+
+    it('getUser returns user by id', async () => {
+      const ws = await import('../src/server/workspaces.mjs');
+      const { user } = await ws.getOrCreateUser('findme@example.com', 'Find Me');
+
+      const found = await ws.getUser(user.id);
+      assert.ok(found);
+      assert.strictEqual(found.id, user.id);
+      assert.strictEqual(found.name, 'Find Me');
+    });
+
+    it('getUser returns null for non-existent user', async () => {
+      const ws = await import('../src/server/workspaces.mjs');
+      const found = await ws.getUser('usr_nonexistent');
+      assert.strictEqual(found, null);
+    });
+
+    it('updateUser updates name and email', async () => {
+      const ws = await import('../src/server/workspaces.mjs');
+      const { user } = await ws.getOrCreateUser('original@example.com', 'Original Name');
+
+      const updated = await ws.updateUser(user.id, {
+        name: 'Updated Name',
+        email: 'updated@example.com',
+      });
+
+      assert.strictEqual(updated.name, 'Updated Name');
+      assert.strictEqual(updated.email, 'updated@example.com');
+
+      // Verify persisted
+      const found = await ws.getUser(user.id);
+      assert.strictEqual(found.name, 'Updated Name');
+      assert.strictEqual(found.email, 'updated@example.com');
+    });
+
+    it('email matching is case-insensitive', async () => {
+      const ws = await import('../src/server/workspaces.mjs');
+      const { user } = await ws.getOrCreateUser('case@test.com', 'Case Test');
+
+      // Try to get or create with different case
+      const found = await ws.getOrCreateUser('CASE@TEST.COM', 'Different Case');
+      assert.strictEqual(found.created, false);
+      assert.strictEqual(found.user.id, user.id);
+    });
+  });
+});