@polderlabs/bizar 5.5.0 → 5.5.2

package/bizar-dash/tests/views/Marketplace.test.tsx

@@ -0,0 +1,200 @@
+/**
+ * tests/views/Marketplace.test.tsx
+ *
+ * v6.x — Marketplace view tests. Verifies the new UX:
+ *   - Loading state shows a "Fetching registry from <url>" message.
+ *   - Success state surfaces the registry source URL.
+ *   - Empty state shows a friendly "No plugins published yet" message.
+ *   - Refresh button re-fetches the registry.
+ *   - Category filter works (still works after the rewrite).
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { render, screen, waitFor } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
+import React from 'react';
+import { ToastProvider } from '../../src/web/components/Toast';
+import { ModalProvider } from '../../src/web/components/Modal';
+import { Marketplace } from '../../src/web/views/Marketplace';
+import { api, ApiError } from '../../src/web/lib/api';
+
+vi.mock('../../src/web/lib/api', () => {
+  const get = vi.fn();
+  const post = vi.fn();
+  return {
+    api: { get, put: vi.fn(), post, del: vi.fn(), setToken: vi.fn(), probeAuthStatus: vi.fn() },
+    ApiError: class ApiError extends Error {
+      status: number;
+      data: unknown;
+      constructor(status: number, message: string, data: unknown = null) {
+        super(message);
+        this.status = status;
+        this.data = data;
+      }
+    },
+  };
+});
+
+const Harness = () => (
+  <ToastProvider>
+    <ModalProvider>
+      <Marketplace />
+    </ModalProvider>
+  </ToastProvider>
+);
+
+const samplePlugins = [
+  {
+    id: 'plugin-a',
+    name: 'Plugin A',
+    version: '1.0.0',
+    author: 'Alice',
+    description: 'First test plugin',
+    category: 'Utilities',
+    tags: ['tool'],
+    permissions: [],
+  },
+  {
+    id: 'plugin-b',
+    name: 'Plugin B',
+    version: '0.2.1',
+    author: 'Bob',
+    description: 'Second test plugin',
+    category: 'Integrations',
+    tags: ['github'],
+    permissions: ['net'],
+  },
+];
+
+describe('Marketplace view', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it('shows a loading state while the registry is being fetched', () => {
+    (api.get as ReturnType<typeof vi.fn>).mockReturnValue(new Promise(() => {})); // never resolves
+    render(<Harness />);
+    expect(screen.getByTestId('marketplace-loading')).toBeInTheDocument();
+    // Both the subtitle ("Fetching registry from the community server…")
+    // and the spinner paragraph mention "Fetching registry"; assert the
+    // spinner paragraph specifically.
+    const loading = screen.getByTestId('marketplace-loading');
+    expect(loading.textContent?.toLowerCase()).toContain('fetching');
+  });
+
+  it('surfaces the registry source URL after a successful fetch', async () => {
+    (api.get as ReturnType<typeof vi.fn>).mockResolvedValue({
+      plugins: samplePlugins,
+      registry: { source: 'https://github.com/DrB0rk/bizar-mods', updatedAt: null },
+    });
+
+    render(<Harness />);
+
+    await waitFor(() => {
+      const source = screen.getByTestId('marketplace-source');
+      expect(source.textContent).toContain('https://github.com/DrB0rk/bizar-mods');
+    });
+
+    // Plugin count pill
+    const count = screen.getByTestId('marketplace-count');
+    expect(count.textContent).toMatch(/2 plugins/);
+  });
+
+  it('shows a "cached" notice when the server reports cached data', async () => {
+    (api.get as ReturnType<typeof vi.fn>).mockResolvedValue({
+      plugins: samplePlugins,
+      registry: { source: 'https://github.com/DrB0rk/bizar-mods', updatedAt: null },
+      cached: true,
+      cacheTimestamp: Date.now() - 1000 * 60 * 30, // 30 min ago
+    });
+
+    render(<Harness />);
+
+    await waitFor(() => {
+      expect(screen.getByTestId('marketplace-cache-notice')).toBeInTheDocument();
+    });
+    expect(screen.getByTestId('marketplace-cache-notice').textContent?.toLowerCase()).toContain('cached');
+  });
+
+  it('renders a friendly empty state when the registry has zero plugins', async () => {
+    (api.get as ReturnType<typeof vi.fn>).mockResolvedValue({
+      plugins: [],
+      registry: { source: 'https://github.com/DrB0rk/bizar-mods', updatedAt: null },
+    });
+
+    render(<Harness />);
+
+    await waitFor(() => {
+      expect(screen.getByText(/no plugins published yet/i)).toBeInTheDocument();
+    });
+  });
+
+  it('renders a registry-unreachable error state on failure', async () => {
+    (api.get as ReturnType<typeof vi.fn>).mockRejectedValue(
+      new ApiError(502, 'all registry URLs failed', { registryUrl: 'https://github.com/DrB0rk/bizar-mods' }),
+    );
+
+    render(<Harness />);
+
+    await waitFor(() => {
+      expect(screen.getByText(/registry unreachable/i)).toBeInTheDocument();
+    });
+    // URL surfaced in error state
+    expect(screen.getByText(/github\.com\/DrB0rk\/bizar-mods/i)).toBeInTheDocument();
+  });
+
+  it('re-fetches the registry when the Refresh button is clicked', async () => {
+    (api.get as ReturnType<typeof vi.fn>).mockResolvedValue({
+      plugins: samplePlugins,
+      registry: { source: 'https://github.com/DrB0rk/bizar-mods', updatedAt: null },
+    });
+
+    const user = userEvent.setup();
+    render(<Harness />);
+
+    await waitFor(() => {
+      expect(screen.getByTestId('marketplace-source')).toBeInTheDocument();
+    });
+
+    (api.get as ReturnType<typeof vi.fn>).mockClear();
+    await user.click(screen.getByTestId('marketplace-refresh'));
+    await waitFor(() => {
+      expect(api.get).toHaveBeenCalled();
+    });
+  });
+
+  it('filters plugins by category when a chip is clicked', async () => {
+    (api.get as ReturnType<typeof vi.fn>).mockResolvedValue({
+      plugins: samplePlugins,
+      registry: { source: 'https://github.com/DrB0rk/bizar-mods', updatedAt: null },
+    });
+
+    const user = userEvent.setup();
+    render(<Harness />);
+
+    await waitFor(() => {
+      expect(screen.getByTestId('marketplace-grid')).toBeInTheDocument();
+    });
+
+    // Initially both plugins visible.
+    expect(screen.getByText('Plugin A')).toBeInTheDocument();
+    expect(screen.getByText('Plugin B')).toBeInTheDocument();
+
+    // Click the Integrations chip — only Plugin B should remain.
+    await user.click(screen.getByTestId('marketplace-category-integrations'));
+    await waitFor(() => {
+      expect(screen.queryByText('Plugin A')).not.toBeInTheDocument();
+      expect(screen.getByText('Plugin B')).toBeInTheDocument();
+    });
+
+    // Reset filter via "All" chip — both visible again.
+    await user.click(screen.getByTestId('marketplace-category-all'));
+    await waitFor(() => {
+      expect(screen.getByText('Plugin A')).toBeInTheDocument();
+      expect(screen.getByText('Plugin B')).toBeInTheDocument();
+    });
+  });
+});

package/bizar-dash/tests/views/Memory.test.tsx

@@ -0,0 +1,274 @@
+/**
+ * tests/views/Memory.test.tsx
+ *
+ * v6.x — Memory UI tests focused on the vault path loader fix.
+ * Before this fix, the ConfigPanel and MemorySection hardcoded the vault
+ * path to '' and rendered "(loading…)" forever. After the fix, they fetch
+ * /api/memory/status and render the actual path or a "not initialised"
+ * placeholder, plus an Initialise button.
+ *
+ * v6.x also adds tests for the Save Path button calling
+ * POST /memory/config/vault (instead of just showing a message).
+ */
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { render, screen, waitFor } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
+import React from 'react';
+import { ToastProvider } from '../../src/web/components/Toast';
+import { ModalProvider } from '../../src/web/components/Modal';
+import { MemoryOverview } from '../../src/web/views/memory/MemoryOverview';
+import { ConfigPanel } from '../../src/web/views/memory/ConfigPanel';
+import { MemorySection } from '../../src/web/views/settings/MemorySection';
+import { api } from '../../src/web/lib/api';
+
+vi.mock('../../src/web/lib/api', () => {
+  const get = vi.fn();
+  return {
+    api: {
+      get,
+      put: vi.fn(),
+      post: vi.fn(),
+      del: vi.fn(),
+      setToken: vi.fn(),
+      probeAuthStatus: vi.fn(),
+    },
+    ApiError: class ApiError extends Error {
+      status: number;
+      constructor(status: number, message: string) {
+        super(message);
+        this.status = status;
+      }
+    },
+  };
+});
+
+const Harness = ({ children }: { children: React.ReactNode }) => (
+  <ToastProvider>
+    <ModalProvider>{children}</ModalProvider>
+  </ToastProvider>
+);
+
+describe('MemoryOverview vault path', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it('renders the resolved vault path from /memory/status', async () => {
+    (api.get as ReturnType<typeof vi.fn>).mockImplementation((url: string) => {
+      if (url === '/memory/status') return Promise.resolve({
+        initialized: true,
+        mode: 'local-only',
+        vaultRoot: '/Users/test/.bizar_memory',
+        projectVaultRoot: '/Users/test/.bizar_memory',
+        noteCount: 5,
+        gitClean: true,
+        branch: 'main',
+      });
+      if (url === '/memory/health') return Promise.resolve({
+        score: 90,
+        status: 'healthy',
+        checks: [],
+        message: 'all good',
+      });
+      if (url === '/memory/lightrag/stats') return Promise.resolve({
+        running: false, pid: null, host: '127.0.0.1', port: 9621, indexedApprox: 0, queryCountLast24h: 0, lastReindexAt: null,
+      });
+      if (url === '/memory/storage') return Promise.resolve({ total: 1024, breakdown: [] });
+      return Promise.resolve(null);
+    });
+
+    render(
+      <Harness>
+        <MemoryOverview refreshKey={0} onRefresh={vi.fn()} setActiveSubPanel={vi.fn()} />
+      </Harness>,
+    );
+
+    await waitFor(() => {
+      const el = screen.getByTestId('memory-overview-vault-path');
+      expect(el.textContent).toContain('/Users/test/.bizar_memory');
+    });
+  });
+
+  it('shows "not initialised" when status reports initialized=false', async () => {
+    (api.get as ReturnType<typeof vi.fn>).mockImplementation((url: string) => {
+      if (url === '/memory/status') return Promise.resolve({
+        initialized: false, mode: null, projectId: null, vaultRoot: null, projectVaultRoot: null,
+      });
+      if (url === '/memory/health') return Promise.resolve({
+        score: 0, status: 'unconfigured', checks: [], message: 'not initialised',
+      });
+      if (url === '/memory/lightrag/stats') return Promise.resolve({
+        running: false, pid: null, host: '127.0.0.1', port: 9621, indexedApprox: 0, queryCountLast24h: 0, lastReindexAt: null,
+      });
+      if (url === '/memory/storage') return Promise.resolve({ total: 0, breakdown: [] });
+      return Promise.resolve(null);
+    });
+
+    render(
+      <Harness>
+        <MemoryOverview refreshKey={0} onRefresh={vi.fn()} setActiveSubPanel={vi.fn()} />
+      </Harness>,
+    );
+
+    await waitFor(() => {
+      const el = screen.getByTestId('memory-overview-vault-path');
+      expect(el.textContent?.toLowerCase()).toContain('not initialised');
+    });
+  });
+});
+
+describe('ConfigPanel vault path', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it('does not render "(loading…)" forever — shows actual vault path', async () => {
+    (api.get as ReturnType<typeof vi.fn>).mockImplementation((url: string) => {
+      if (url === '/memory/config/global') return Promise.resolve({
+        config: { git: { remoteUrl: '' } },
+      });
+      if (url === '/memory/status') return Promise.resolve({
+        initialized: true, vaultRoot: '/Users/test/.bizar_memory', mode: 'local-only',
+      });
+      if (url === '/memory/git/status') return Promise.resolve({ ok: false });
+      return Promise.resolve(null);
+    });
+
+    render(
+      <Harness>
+        <ConfigPanel refreshKey={0} />
+      </Harness>,
+    );
+
+    await waitFor(() => {
+      const el = screen.getByTestId('memory-current-vault-path');
+      expect(el.textContent).toContain('/Users/test/.bizar_memory');
+    });
+
+    // Sanity: the legacy "(loading…)" string is no longer rendered.
+    expect(screen.queryByText('(loading…)')).not.toBeInTheDocument();
+  });
+
+  it('shows an "Initialise default" button when not initialised', async () => {
+    (api.get as ReturnType<typeof vi.fn>).mockImplementation((url: string) => {
+      if (url === '/memory/config/global') return Promise.resolve({ config: {} });
+      if (url === '/memory/status') return Promise.resolve({ initialized: false, vaultRoot: null });
+      return Promise.resolve(null);
+    });
+
+    render(
+      <Harness>
+        <ConfigPanel refreshKey={0} />
+      </Harness>,
+    );
+
+    await waitFor(() => {
+      expect(screen.getByTestId('memory-init-vault')).toBeInTheDocument();
+    });
+  });
+
+  it('triggers /memory/init when "Initialise default" is clicked', async () => {
+    const post = vi.fn().mockResolvedValue({ ok: true });
+    (api.get as ReturnType<typeof vi.fn>).mockImplementation((url: string) => {
+      if (url === '/memory/config/global') return Promise.resolve({ config: {} });
+      if (url === '/memory/status') return Promise.resolve({ initialized: false, vaultRoot: null });
+      return Promise.resolve(null);
+    });
+    (api.post as ReturnType<typeof vi.fn>) = post;
+
+    const user = userEvent.setup();
+    render(
+      <Harness>
+        <ConfigPanel refreshKey={0} />
+      </Harness>,
+    );
+
+    await waitFor(() => {
+      expect(screen.getByTestId('memory-init-vault')).toBeInTheDocument();
+    });
+    await user.click(screen.getByTestId('memory-init-vault'));
+    expect(post).toHaveBeenCalledWith('/memory/init', expect.anything());
+  });
+
+  // v6.x — Save Path now calls POST /memory/config/vault
+  it('calls POST /memory/config/vault when Save path is clicked (already initialised)', async () => {
+    const post = vi.fn().mockResolvedValue({ ok: true, vaultRoot: '/new/path/.bizar_memory' });
+    (api.get as ReturnType<typeof vi.fn>).mockImplementation((url: string) => {
+      if (url === '/memory/config/global') return Promise.resolve({ config: { git: { remoteUrl: '' } } });
+      if (url === '/memory/status') return Promise.resolve({
+        initialized: true, vaultRoot: '/old/path/.bizar_memory', mode: 'managed',
+      });
+      if (url === '/memory/git/status') return Promise.resolve({ ok: true, clean: true, branch: 'main' });
+      return Promise.resolve(null);
+    });
+    (api.post as ReturnType<typeof vi.fn>) = post;
+
+    const user = userEvent.setup();
+    render(
+      <Harness>
+        <ConfigPanel refreshKey={0} />
+      </Harness>,
+    );
+
+    await waitFor(() => {
+      expect(screen.getByTestId('memory-save-vault-path')).toBeInTheDocument();
+    });
+
+    // Change the vault path input
+    const input = screen.getByTestId('memory-vault-path-input');
+    await user.clear(input);
+    await user.type(input, '/new/path/.bizar_memory');
+
+    await user.click(screen.getByTestId('memory-save-vault-path'));
+
+    // v6.x: should call /memory/config/vault (not just show a message)
+    expect(post).toHaveBeenCalledWith('/memory/config/vault', { vaultRoot: '/new/path/.bizar_memory' });
+  });
+});
+
+describe('MemorySection (settings) vault path', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it('renders the vault path from /memory/status (was hardcoded to "")', async () => {
+    (api.get as ReturnType<typeof vi.fn>).mockImplementation((url: string) => {
+      if (url === '/memory/config/global') return Promise.resolve({ config: { git: { remoteUrl: '' } } });
+      if (url === '/memory/status') return Promise.resolve({
+        initialized: true, vaultRoot: '/home/test/.bizar_memory', mode: 'local-only',
+      });
+      return Promise.resolve(null);
+    });
+
+    render(
+      <Harness>
+        <MemorySection />
+      </Harness>,
+    );
+
+    await waitFor(() => {
+      const el = screen.getByTestId('settings-memory-vault-path');
+      expect(el.textContent).toContain('/home/test/.bizar_memory');
+    });
+  });
+
+  it('shows an Initialise button when the vault is not initialised', async () => {
+    (api.get as ReturnType<typeof vi.fn>).mockImplementation((url: string) => {
+      if (url === '/memory/config/global') return Promise.resolve({ config: {} });
+      if (url === '/memory/status') return Promise.resolve({ initialized: false, vaultRoot: null });
+      return Promise.resolve(null);
+    });
+
+    render(
+      <Harness>
+        <MemorySection />
+      </Harness>,
+    );
+
+    await waitFor(() => {
+      const el = screen.getByTestId('settings-memory-vault-path');
+      expect(el.textContent?.toLowerCase()).toContain('not initialised');
+      expect(screen.getByRole('button', { name: /initialise/i })).toBeInTheDocument();
+    });
+  });
+});

package/bizar-dash/tests/views/sidebar.test.tsx

@@ -0,0 +1,116 @@
+/**
+ * tests/views/sidebar.test.tsx
+ *
+ * v6.x — Sidebar tests. Each settings section is its own top-level tab;
+ * there is no longer a `settingsMode` toggle or sub-nav render path.
+ * The Sidebar always renders the regular `.sidebar-tab` rail.
+ */
+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, Bot } from 'lucide-react';
+
+const TABS: TabDef[] = [
+  { id: 'overview', label: 'Overview', icon: LayoutDashboard },
+  { id: 'chat', label: 'Chat', icon: MessageSquare },
+  { id: 'agents', label: 'Agents', icon: Bot },
+];
+
+const SETTINGS_TABS: TabDef[] = [
+  { id: 'settings-theme', label: 'Theme', icon: LayoutDashboard },
+  { id: 'settings-general', label: 'General', icon: MessageSquare },
+];
+
+describe('Sidebar', () => {
+  const onTabChange = vi.fn();
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it('renders the regular tab rail', () => {
+    render(
+      <Sidebar
+        tabs={TABS}
+        activeTab="overview"
+        onTabChange={onTabChange}
+      />,
+    );
+    expect(screen.getByRole('tablist')).toBeInTheDocument();
+    expect(screen.getByRole('tab', { name: /overview/i })).toBeInTheDocument();
+    expect(screen.getByRole('tab', { name: /chat/i })).toBeInTheDocument();
+    expect(screen.getByRole('tab', { name: /agents/i })).toBeInTheDocument();
+  });
+
+  it('does not render the legacy SettingsNav back button', () => {
+    render(
+      <Sidebar
+        tabs={TABS}
+        activeTab="overview"
+        onTabChange={onTabChange}
+      />,
+    );
+    expect(screen.queryByRole('button', { name: /exit settings/i })).not.toBeInTheDocument();
+    expect(screen.queryByRole('button', { name: /back/i })).not.toBeInTheDocument();
+  });
+
+  it('highlights the active tab', () => {
+    render(
+      <Sidebar
+        tabs={TABS}
+        activeTab="chat"
+        onTabChange={onTabChange}
+      />,
+    );
+    const chatTab = screen.getByRole('tab', { name: /chat/i });
+    expect(chatTab).toHaveClass('sidebar-tab-active');
+  });
+
+  it('calls onTabChange when a tab is clicked', async () => {
+    const user = userEvent.setup();
+    render(
+      <Sidebar
+        tabs={TABS}
+        activeTab="overview"
+        onTabChange={onTabChange}
+      />,
+    );
+    await user.click(screen.getByRole('tab', { name: /chat/i }));
+    expect(onTabChange).toHaveBeenCalledWith('chat');
+  });
+
+  it('renders settings sections as their own tabs', () => {
+    render(
+      <Sidebar
+        tabs={[...TABS, ...SETTINGS_TABS]}
+        activeTab="settings-theme"
+        onTabChange={onTabChange}
+      />,
+    );
+    expect(screen.getByRole('tab', { name: /theme/i })).toBeInTheDocument();
+    expect(screen.getByRole('tab', { name: /general/i })).toBeInTheDocument();
+    const themeTab = screen.getByRole('tab', { name: /theme/i });
+    expect(themeTab).toHaveClass('sidebar-tab-active');
+  });
+
+  it('renders mod views under a "Mods" divider', () => {
+    const modTab: TabDef & { isMod?: boolean } = {
+      id: 'graphify:web',
+      label: 'Graph',
+      icon: LayoutDashboard,
+      isMod: true,
+    };
+    render(
+      <Sidebar
+        tabs={[...TABS, modTab]}
+        activeTab="overview"
+        onTabChange={onTabChange}
+      />,
+    );
+    expect(screen.getByText('Mods')).toBeInTheDocument();
+    expect(screen.getByRole('tab', { name: /graph/i })).toBeInTheDocument();
+  });
+});

package/bizar-dash/tests/views/topbar.test.tsx

@@ -0,0 +1,87 @@
+/**
+ * tests/views/topbar.test.tsx
+ *
+ * v4.9.0 / v5.5.0 — Topbar tests. Verifies that:
+ *   - The Settings tab appears as a single entry in the tab rail.
+ *   - Clicking Settings invokes onTabChange with 'settings'.
+ *   - The Settings tab shows the settings-mode-indicator when active.
+ *   - All other tabs work normally.
+ */
+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 { Topbar, TABS } from '../../src/web/components/Topbar';
+
+describe('Topbar', () => {
+  const onTabChange = vi.fn();
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  const renderTopbar = (activeTab: string) =>
+    render(
+      <Topbar
+        activeTab={activeTab}
+        onTabChange={onTabChange}
+        wsStatus="connected"
+        version="v4.9.0"
+        activeProject={null}
+        projects={[]}
+        onProjectChange={vi.fn()}
+        onProjectsRefresh={vi.fn()}
+        onOpenSearch={vi.fn()}
+      />,
+    );
+
+  it('renders the Settings tab as a single entry', () => {
+    renderTopbar('overview');
+    expect(screen.getByRole('tab', { name: /settings/i })).toBeInTheDocument();
+  });
+
+  it('renders all main tabs', () => {
+    renderTopbar('overview');
+    expect(screen.getByRole('tab', { name: /overview/i })).toBeInTheDocument();
+    expect(screen.getByRole('tab', { name: /chat/i })).toBeInTheDocument();
+    expect(screen.getByRole('tab', { name: /agents/i })).toBeInTheDocument();
+    expect(screen.getByRole('tab', { name: /memory/i })).toBeInTheDocument();
+    expect(screen.getByRole('tab', { name: /settings/i })).toBeInTheDocument();
+  });
+
+  it('exposes the Settings tab id in TABS', () => {
+    const ids = TABS.map((t) => t.id);
+    expect(ids).toContain('settings');
+    // Settings should NOT have a prefix (it's a single entry, not per-section)
+    expect(ids).not.toContain('settings-theme');
+    expect(ids).not.toContain('settings-general');
+  });
+
+  it('calls onTabChange with "settings" when the Settings tab is clicked', async () => {
+    const user = userEvent.setup();
+    renderTopbar('overview');
+    await user.click(screen.getByRole('tab', { name: /settings/i }));
+    expect(onTabChange).toHaveBeenCalledWith('settings');
+  });
+
+  it('calls onTabChange with "overview" when the Overview tab is clicked', async () => {
+    const user = userEvent.setup();
+    renderTopbar('settings');
+    await user.click(screen.getByRole('tab', { name: /overview/i }));
+    expect(onTabChange).toHaveBeenCalledWith('overview');
+  });
+
+  it('highlights the active tab with `tab-active`', () => {
+    renderTopbar('settings');
+    const settingsTab = screen.getByRole('tab', { name: /settings/i });
+    expect(settingsTab).toHaveClass('tab-active');
+  });
+
+  it('does not render settings section tabs (they are in the sidebar when in settings mode)', () => {
+    renderTopbar('overview');
+    // These section tabs should NOT exist in the topbar
+    expect(screen.queryByRole('tab', { name: /theme/i })).not.toBeInTheDocument();
+    expect(screen.queryByRole('tab', { name: /general/i })).not.toBeInTheDocument();
+    expect(screen.queryByRole('tab', { name: /layout/i })).not.toBeInTheDocument();
+  });
+});

package/cli/commands/marketplace.mjs

@@ -29,7 +29,7 @@ export function showMarketplaceHelp() {
   Description:
     The marketplace is the public registry of plugins hosted at
     ${process.env.BIZAR_REGISTRY_URL ||
-      'https://raw.githubusercontent.com/DrB0rk/bizar-plugins/main/registry.json'}.
+      'https://raw.githubusercontent.com/DrB0rk/bizar-mods/main/registry.json'}.
 
     This command is a thin alias for the \`plugin\` command — run
     \`bizar plugin --help\` for the full surface (config, update, invoke, ...).