@pixelbyte-software/pixcode 1.51.2 → 1.51.3

package/scripts/smoke/provider-models-opencode-live.mjs

@@ -1,66 +1,66 @@
-#!/usr/bin/env node
-
-import assert from 'node:assert/strict';
-import { mkdtemp } from 'node:fs/promises';
-import os from 'node:os';
-import path from 'node:path';
-
-const tempHome = await mkdtemp(path.join(os.tmpdir(), 'pixcode-provider-models-'));
-process.env.HOME = tempHome;
-process.env.OPENCODE_MODELS_URL = 'https://models.dev/test-api.json';
-
-const calls = [];
-globalThis.fetch = async (url) => {
-  calls.push(String(url));
-  return {
-    ok: true,
-    status: 200,
-    async json() {
-      return {
-        opencode: {
-          name: 'OpenCode Zen',
-          models: {
-            'big-pickle': {
-              name: 'Big Pickle',
-              cost: { input: 0, output: 0 },
-              limit: { context: 128000 },
-            },
-            'fresh-free': {
-              name: 'Fresh Free',
-              cost: { input: 0, output: 0 },
-              limit: { context: 64000 },
-            },
-          },
-        },
-      };
-    },
-  };
-};
-
-const { getProviderModels } = await import('../../server/services/provider-models.js');
-
-const staleStatic = [
-  { value: 'opencode/hy3-preview-free', label: 'Stale Hy3 Preview' },
-  { value: 'opencode/ling-2.6-flash-free', label: 'Stale Ling 2.6 Flash' },
-];
-
-const refreshed = await getProviderModels('opencode', {
-  forceRefresh: true,
-  staticList: staleStatic,
-});
-const refreshedValues = refreshed.models.map((model) => model.value);
-assert.equal(calls.length, 1, 'force refresh should fetch the OpenCode live catalog once');
-assert.ok(refreshedValues.includes('opencode/fresh-free'), 'live OpenCode models should be returned');
-assert.ok(!refreshedValues.includes('opencode/hy3-preview-free'), 'stale static OpenCode models must not be merged into a successful live catalog');
-assert.ok(!refreshedValues.includes('opencode/ling-2.6-flash-free'), 'stale static OpenCode models must not survive a successful live refresh');
-
-const cached = await getProviderModels('opencode', {
-  staticList: staleStatic,
-});
-const cachedValues = cached.models.map((model) => model.value);
-assert.equal(calls.length, 1, 'fresh cache should be used without another network request');
-assert.ok(cachedValues.includes('opencode/fresh-free'), 'cached live OpenCode models should be returned');
-assert.ok(!cachedValues.includes('opencode/hy3-preview-free'), 'cached live OpenCode catalog must stay free of stale static models');
-assert.ok(!cachedValues.includes('opencode/ling-2.6-flash-free'), 'cached live OpenCode catalog must stay free of stale static models');
-
-console.log('provider OpenCode live model catalog smoke passed');
+#!/usr/bin/env node
+
+import assert from 'node:assert/strict';
+import { mkdtemp } from 'node:fs/promises';
+import os from 'node:os';
+import path from 'node:path';
+
+const tempHome = await mkdtemp(path.join(os.tmpdir(), 'pixcode-provider-models-'));
+process.env.HOME = tempHome;
+process.env.OPENCODE_MODELS_URL = 'https://models.dev/test-api.json';
+
+const calls = [];
+globalThis.fetch = async (url) => {
+  calls.push(String(url));
+  return {
+    ok: true,
+    status: 200,
+    async json() {
+      return {
+        opencode: {
+          name: 'OpenCode Zen',
+          models: {
+            'big-pickle': {
+              name: 'Big Pickle',
+              cost: { input: 0, output: 0 },
+              limit: { context: 128000 },
+            },
+            'fresh-free': {
+              name: 'Fresh Free',
+              cost: { input: 0, output: 0 },
+              limit: { context: 64000 },
+            },
+          },
+        },
+      };
+    },
+  };
+};
+
+const { getProviderModels } = await import('../../server/services/provider-models.js');
+
+const staleStatic = [
+  { value: 'opencode/hy3-preview-free', label: 'Stale Hy3 Preview' },
+  { value: 'opencode/ling-2.6-flash-free', label: 'Stale Ling 2.6 Flash' },
+];
+
+const refreshed = await getProviderModels('opencode', {
+  forceRefresh: true,
+  staticList: staleStatic,
+});
+const refreshedValues = refreshed.models.map((model) => model.value);
+assert.equal(calls.length, 1, 'force refresh should fetch the OpenCode live catalog once');
+assert.ok(refreshedValues.includes('opencode/fresh-free'), 'live OpenCode models should be returned');
+assert.ok(!refreshedValues.includes('opencode/hy3-preview-free'), 'stale static OpenCode models must not be merged into a successful live catalog');
+assert.ok(!refreshedValues.includes('opencode/ling-2.6-flash-free'), 'stale static OpenCode models must not survive a successful live refresh');
+
+const cached = await getProviderModels('opencode', {
+  staticList: staleStatic,
+});
+const cachedValues = cached.models.map((model) => model.value);
+assert.equal(calls.length, 1, 'fresh cache should be used without another network request');
+assert.ok(cachedValues.includes('opencode/fresh-free'), 'cached live OpenCode models should be returned');
+assert.ok(!cachedValues.includes('opencode/hy3-preview-free'), 'cached live OpenCode catalog must stay free of stale static models');
+assert.ok(!cachedValues.includes('opencode/ling-2.6-flash-free'), 'cached live OpenCode catalog must stay free of stale static models');
+
+console.log('provider OpenCode live model catalog smoke passed');

package/scripts/smoke/provider-rest-api.mjs

@@ -1,124 +1,124 @@
-#!/usr/bin/env node
-
-import process from 'node:process';
-
-const apiUrl = (process.env.PIXCODE_API_URL || 'http://127.0.0.1:3001').replace(/\/$/, '');
-const apiKey = process.env.PIXCODE_API_KEY || process.env.PIXCODE_AGENT_API_KEY || '';
-const providers = (process.env.PIXCODE_PROVIDERS || 'claude,cursor,codex,gemini,qwen,opencode')
-  .split(',')
-  .map((provider) => provider.trim())
-  .filter(Boolean);
-const projectPath = process.env.PIXCODE_PROJECT_PATH || process.cwd();
-const message = process.env.PIXCODE_SMOKE_MESSAGE
-  || 'Reply with exactly "pixcode-smoke-ok". Do not edit files.';
-const timeoutMs = Number(process.env.PIXCODE_SMOKE_TIMEOUT_MS || 180000);
-const modelMap = parseModelMap(process.env.PIXCODE_PROVIDER_MODELS || '{}');
-
-if (!apiKey) {
-  console.error('PIXCODE_API_KEY is required. Create one in Settings -> API Keys and rerun.');
-  process.exit(2);
-}
-
-const results = [];
-
-for (const provider of providers) {
-  const startedAt = Date.now();
-  const controller = new AbortController();
-  const timer = setTimeout(() => controller.abort(), timeoutMs);
-
-  try {
-    const body = {
-      provider,
-      projectPath,
-      message,
-      stream: false,
-      cleanup: false,
-    };
-    if (modelMap[provider]) {
-      body.model = modelMap[provider];
-    }
-
-    const response = await fetch(`${apiUrl}/api/agent`, {
-      method: 'POST',
-      headers: {
-        Authorization: `Bearer ${apiKey}`,
-        'Content-Type': 'application/json',
-      },
-      body: JSON.stringify(body),
-      signal: controller.signal,
-    });
-
-    const text = await response.text();
-    const payload = safeJson(text);
-    const assistantText = extractAssistantText(payload);
-    const ok = response.ok && payload?.success === true && assistantText.trim().length > 0;
-
-    results.push({
-      provider,
-      status: ok ? 'ok' : 'provider-error',
-      httpStatus: response.status,
-      success: Boolean(payload?.success),
-      durationMs: Date.now() - startedAt,
-      sessionId: payload?.sessionId || null,
-      assistantPreview: assistantText.trim().slice(0, 240),
-      error: ok ? null : (payload?.error || payload?.rawError || text.slice(0, 500)),
-    });
-  } catch (error) {
-    results.push({
-      provider,
-      status: error?.name === 'AbortError' ? 'timeout' : 'transport-error',
-      httpStatus: null,
-      success: false,
-      durationMs: Date.now() - startedAt,
-      sessionId: null,
-      assistantPreview: '',
-      error: error?.message || String(error),
-    });
-  } finally {
-    clearTimeout(timer);
-  }
-}
-
-console.log(JSON.stringify({ apiUrl, projectPath, results }, null, 2));
-process.exit(results.every((result) => result.status === 'ok') ? 0 : 1);
-
-function parseModelMap(raw) {
-  try {
-    const parsed = JSON.parse(raw);
-    return parsed && typeof parsed === 'object' && !Array.isArray(parsed) ? parsed : {};
-  } catch {
-    return {};
-  }
-}
-
-function safeJson(text) {
-  try {
-    return JSON.parse(text);
-  } catch {
-    return null;
-  }
-}
-
-function extractAssistantText(payload) {
-  if (!payload || !Array.isArray(payload.messages)) return '';
-
-  const chunks = [];
-  for (const messageItem of payload.messages) {
-    if (messageItem?.type !== 'assistant') continue;
-    const content = messageItem.message?.content;
-    if (typeof content === 'string') {
-      chunks.push(content);
-      continue;
-    }
-    if (Array.isArray(content)) {
-      for (const part of content) {
-        if (typeof part === 'string') {
-          chunks.push(part);
-        } else if (part?.type === 'text' && typeof part.text === 'string') {
-          chunks.push(part.text);
-        }
-      }
-    }
-  }
-  return chunks.join('');
-}
+#!/usr/bin/env node
+
+import process from 'node:process';
+
+const apiUrl = (process.env.PIXCODE_API_URL || 'http://127.0.0.1:3001').replace(/\/$/, '');
+const apiKey = process.env.PIXCODE_API_KEY || process.env.PIXCODE_AGENT_API_KEY || '';
+const providers = (process.env.PIXCODE_PROVIDERS || 'claude,cursor,codex,gemini,qwen,opencode')
+  .split(',')
+  .map((provider) => provider.trim())
+  .filter(Boolean);
+const projectPath = process.env.PIXCODE_PROJECT_PATH || process.cwd();
+const message = process.env.PIXCODE_SMOKE_MESSAGE
+  || 'Reply with exactly "pixcode-smoke-ok". Do not edit files.';
+const timeoutMs = Number(process.env.PIXCODE_SMOKE_TIMEOUT_MS || 180000);
+const modelMap = parseModelMap(process.env.PIXCODE_PROVIDER_MODELS || '{}');
+
+if (!apiKey) {
+  console.error('PIXCODE_API_KEY is required. Create one in Settings -> API Keys and rerun.');
+  process.exit(2);
+}
+
+const results = [];
+
+for (const provider of providers) {
+  const startedAt = Date.now();
+  const controller = new AbortController();
+  const timer = setTimeout(() => controller.abort(), timeoutMs);
+
+  try {
+    const body = {
+      provider,
+      projectPath,
+      message,
+      stream: false,
+      cleanup: false,
+    };
+    if (modelMap[provider]) {
+      body.model = modelMap[provider];
+    }
+
+    const response = await fetch(`${apiUrl}/api/agent`, {
+      method: 'POST',
+      headers: {
+        Authorization: `Bearer ${apiKey}`,
+        'Content-Type': 'application/json',
+      },
+      body: JSON.stringify(body),
+      signal: controller.signal,
+    });
+
+    const text = await response.text();
+    const payload = safeJson(text);
+    const assistantText = extractAssistantText(payload);
+    const ok = response.ok && payload?.success === true && assistantText.trim().length > 0;
+
+    results.push({
+      provider,
+      status: ok ? 'ok' : 'provider-error',
+      httpStatus: response.status,
+      success: Boolean(payload?.success),
+      durationMs: Date.now() - startedAt,
+      sessionId: payload?.sessionId || null,
+      assistantPreview: assistantText.trim().slice(0, 240),
+      error: ok ? null : (payload?.error || payload?.rawError || text.slice(0, 500)),
+    });
+  } catch (error) {
+    results.push({
+      provider,
+      status: error?.name === 'AbortError' ? 'timeout' : 'transport-error',
+      httpStatus: null,
+      success: false,
+      durationMs: Date.now() - startedAt,
+      sessionId: null,
+      assistantPreview: '',
+      error: error?.message || String(error),
+    });
+  } finally {
+    clearTimeout(timer);
+  }
+}
+
+console.log(JSON.stringify({ apiUrl, projectPath, results }, null, 2));
+process.exit(results.every((result) => result.status === 'ok') ? 0 : 1);
+
+function parseModelMap(raw) {
+  try {
+    const parsed = JSON.parse(raw);
+    return parsed && typeof parsed === 'object' && !Array.isArray(parsed) ? parsed : {};
+  } catch {
+    return {};
+  }
+}
+
+function safeJson(text) {
+  try {
+    return JSON.parse(text);
+  } catch {
+    return null;
+  }
+}
+
+function extractAssistantText(payload) {
+  if (!payload || !Array.isArray(payload.messages)) return '';
+
+  const chunks = [];
+  for (const messageItem of payload.messages) {
+    if (messageItem?.type !== 'assistant') continue;
+    const content = messageItem.message?.content;
+    if (typeof content === 'string') {
+      chunks.push(content);
+      continue;
+    }
+    if (Array.isArray(content)) {
+      for (const part of content) {
+        if (typeof part === 'string') {
+          chunks.push(part);
+        } else if (part?.type === 'text' && typeof part.text === 'string') {
+          chunks.push(part.text);
+        }
+      }
+    }
+  }
+  return chunks.join('');
+}

package/scripts/smoke/provider-selection-status.mjs

@@ -1,52 +1,52 @@
-import assert from 'node:assert/strict';
-import fs from 'node:fs';
-
-const read = (path) => fs.readFileSync(path, 'utf8');
-
-const providerPicker = read('src/components/chat/view/subcomponents/ProviderSelectionEmptyState.tsx');
-const providerStatusHook = read('src/components/provider-auth/hooks/useProviderAuthStatus.ts');
-const geminiAuth = read('server/modules/providers/list/gemini/gemini-auth.provider.ts');
-
-assert.match(
-  providerPicker,
-  /const isChecking = !status \|\| status\.loading;/,
-  'Provider cards should only show Checking while a request is actually loading.',
-);
-
-assert.doesNotMatch(
-  providerPicker,
-  /const isUnknown = !status \|\| status\.loading \|\| status\.installed === null;/,
-  'Completed unknown/error provider status must not be rendered as Checking forever.',
-);
-
-assert.match(
-  providerPicker,
-  /providerSelection\.statusUnavailable/,
-  'Provider cards should render a user-facing unavailable state after status errors.',
-);
-
-assert.match(
-  providerStatusHook,
-  /PROVIDER_AUTH_STATUS_TIMEOUT_MS/,
-  'Provider auth status requests should have a frontend timeout.',
-);
-
-assert.match(
-  providerStatusHook,
-  /AbortController/,
-  'Provider auth status fetches should abort instead of leaving cards loading forever.',
-);
-
-assert.match(
-  geminiAuth,
-  /GEMINI_TOKEN_INFO_TIMEOUT_MS/,
-  'Gemini tokeninfo lookup should have a backend timeout.',
-);
-
-assert.match(
-  geminiAuth,
-  /signal: controller\.signal/,
-  'Gemini tokeninfo fetch should use an abort signal.',
-);
-
-console.log('provider selection status smoke passed');
+import assert from 'node:assert/strict';
+import fs from 'node:fs';
+
+const read = (path) => fs.readFileSync(path, 'utf8');
+
+const providerPicker = read('src/components/chat/view/subcomponents/ProviderSelectionEmptyState.tsx');
+const providerStatusHook = read('src/components/provider-auth/hooks/useProviderAuthStatus.ts');
+const geminiAuth = read('server/modules/providers/list/gemini/gemini-auth.provider.ts');
+
+assert.match(
+  providerPicker,
+  /const isChecking = !status \|\| status\.loading;/,
+  'Provider cards should only show Checking while a request is actually loading.',
+);
+
+assert.doesNotMatch(
+  providerPicker,
+  /const isUnknown = !status \|\| status\.loading \|\| status\.installed === null;/,
+  'Completed unknown/error provider status must not be rendered as Checking forever.',
+);
+
+assert.match(
+  providerPicker,
+  /providerSelection\.statusUnavailable/,
+  'Provider cards should render a user-facing unavailable state after status errors.',
+);
+
+assert.match(
+  providerStatusHook,
+  /PROVIDER_AUTH_STATUS_TIMEOUT_MS/,
+  'Provider auth status requests should have a frontend timeout.',
+);
+
+assert.match(
+  providerStatusHook,
+  /AbortController/,
+  'Provider auth status fetches should abort instead of leaving cards loading forever.',
+);
+
+assert.match(
+  geminiAuth,
+  /GEMINI_TOKEN_INFO_TIMEOUT_MS/,
+  'Gemini tokeninfo lookup should have a backend timeout.',
+);
+
+assert.match(
+  geminiAuth,
+  /signal: controller\.signal/,
+  'Gemini tokeninfo fetch should use an abort signal.',
+);
+
+console.log('provider selection status smoke passed');

package/scripts/smoke/run-state-refresh.mjs

@@ -1,52 +1,52 @@
-#!/usr/bin/env node
-
-import assert from 'node:assert/strict';
-import { existsSync, readFileSync } from 'node:fs';
-
-const refreshUtilPath = 'src/utils/runStateRefresh.ts';
-assert.ok(existsSync(refreshUtilPath), 'Run state refresh utility should exist.');
-
-const refreshUtil = readFileSync(refreshUtilPath, 'utf8');
-const chatRealtime = readFileSync('src/components/chat/hooks/useChatRealtimeHandlers.ts', 'utf8');
-const chatSession = readFileSync('src/components/chat/hooks/useChatSessionState.ts', 'utf8');
-const chatInterface = readFileSync('src/components/chat/view/ChatInterface.tsx', 'utf8');
-const changedFilesHook = readFileSync('src/hooks/useChangedFilesMonitor.ts', 'utf8');
-const orchestrationPage = readFileSync('src/components/orchestration/OrchestrationPage.tsx', 'utf8');
-const runPanel = readFileSync('src/components/orchestration/workflows/WorkflowRunPanel.tsx', 'utf8');
-
-assert.ok(
-  refreshUtil.includes('PIXCODE_RUN_STATE_REFRESH_EVENT') && refreshUtil.includes('dispatchRunStateRefresh'),
-  'Run state refresh utility should expose a stable browser event and dispatcher.',
-);
-
-assert.ok(
-  chatRealtime.includes('onSessionSettled') && chatRealtime.includes('dispatchRunStateRefresh'),
-  'Chat realtime handlers should dispatch and callback on completion/failure so persisted messages are rehydrated.',
-);
-
-assert.ok(
-  chatSession.includes('refreshActiveSessionMessages') && chatSession.includes('refreshActiveSessionMessages,'),
-  'Chat session state should expose its canonical server refresh path to realtime handlers.',
-);
-
-assert.ok(
-  chatInterface.includes('handleSessionSettled') && chatInterface.includes('refreshActiveSessionMessages'),
-  'Chat interface should refresh the active session when a run settles.',
-);
-
-assert.ok(
-  changedFilesHook.includes('PIXCODE_RUN_STATE_REFRESH_EVENT') && changedFilesHook.includes("run-state"),
-  'Changed-files monitor should refresh on canonical run-state events.',
-);
-
-assert.ok(
-  orchestrationPage.includes('mergeRunSnapshot') && orchestrationPage.includes('dispatchRunStateRefresh'),
-  'Orchestration page should merge run snapshots and dispatch terminal refresh events.',
-);
-
-assert.ok(
-  runPanel.includes('onRunSnapshot') && runPanel.includes('onRunSnapshot?.(nextRun)'),
-  'Workflow run panel should push run snapshots back to the parent list.',
-);
-
-console.log('run state refresh smoke passed');
+#!/usr/bin/env node
+
+import assert from 'node:assert/strict';
+import { existsSync, readFileSync } from 'node:fs';
+
+const refreshUtilPath = 'src/utils/runStateRefresh.ts';
+assert.ok(existsSync(refreshUtilPath), 'Run state refresh utility should exist.');
+
+const refreshUtil = readFileSync(refreshUtilPath, 'utf8');
+const chatRealtime = readFileSync('src/components/chat/hooks/useChatRealtimeHandlers.ts', 'utf8');
+const chatSession = readFileSync('src/components/chat/hooks/useChatSessionState.ts', 'utf8');
+const chatInterface = readFileSync('src/components/chat/view/ChatInterface.tsx', 'utf8');
+const changedFilesHook = readFileSync('src/hooks/useChangedFilesMonitor.ts', 'utf8');
+const orchestrationPage = readFileSync('src/components/orchestration/OrchestrationPage.tsx', 'utf8');
+const runPanel = readFileSync('src/components/orchestration/workflows/WorkflowRunPanel.tsx', 'utf8');
+
+assert.ok(
+  refreshUtil.includes('PIXCODE_RUN_STATE_REFRESH_EVENT') && refreshUtil.includes('dispatchRunStateRefresh'),
+  'Run state refresh utility should expose a stable browser event and dispatcher.',
+);
+
+assert.ok(
+  chatRealtime.includes('onSessionSettled') && chatRealtime.includes('dispatchRunStateRefresh'),
+  'Chat realtime handlers should dispatch and callback on completion/failure so persisted messages are rehydrated.',
+);
+
+assert.ok(
+  chatSession.includes('refreshActiveSessionMessages') && chatSession.includes('refreshActiveSessionMessages,'),
+  'Chat session state should expose its canonical server refresh path to realtime handlers.',
+);
+
+assert.ok(
+  chatInterface.includes('handleSessionSettled') && chatInterface.includes('refreshActiveSessionMessages'),
+  'Chat interface should refresh the active session when a run settles.',
+);
+
+assert.ok(
+  changedFilesHook.includes('PIXCODE_RUN_STATE_REFRESH_EVENT') && changedFilesHook.includes("run-state"),
+  'Changed-files monitor should refresh on canonical run-state events.',
+);
+
+assert.ok(
+  orchestrationPage.includes('mergeRunSnapshot') && orchestrationPage.includes('dispatchRunStateRefresh'),
+  'Orchestration page should merge run snapshots and dispatch terminal refresh events.',
+);
+
+assert.ok(
+  runPanel.includes('onRunSnapshot') && runPanel.includes('onRunSnapshot?.(nextRun)'),
+  'Workflow run panel should push run snapshots back to the parent list.',
+);
+
+console.log('run state refresh smoke passed');