@pixelbyte-software/pixcode 1.51.2 → 1.51.3

package/scripts/smoke/code-editor-vscode-engine.mjs

@@ -1,91 +1,91 @@
-import assert from 'node:assert/strict';
-import fs from 'node:fs';
-
-const read = (path) => fs.readFileSync(path, 'utf8');
-
-const packageJson = JSON.parse(read('package.json'));
-const surface = read('src/components/code-editor/view/subcomponents/CodeEditorSurface.tsx');
-const editor = read('src/components/code-editor/view/CodeEditor.tsx');
-const localMonaco = read('src/components/code-editor/utils/localMonaco.ts');
-
-const allDeps = {
-  ...(packageJson.dependencies ?? {}),
-  ...(packageJson.devDependencies ?? {}),
-};
-
-assert.ok(allDeps['@monaco-editor/react'], 'Code editor should depend on the React Monaco wrapper.');
-assert.ok(allDeps['monaco-editor'], 'Code editor should depend on the Monaco editor engine.');
-
-assert.match(
-  surface,
-  /const MonacoEditor = lazy[\s\S]*import\(['"]@monaco-editor\/react['"]\)/,
-  'Normal file editing should lazy-load Monaco instead of keeping a CodeMirror-only surface.',
-);
-
-assert.match(
-  surface,
-  /ensureLocalMonaco\(\)/,
-  'Monaco should be configured before mount so Linux/self-hosted installs do not depend on the CDN loader.',
-);
-
-assert.match(
-  localMonaco,
-  /LOCAL_MONACO_BASE_PATH\s*=\s*['"]\/vendor\/monaco-editor\/min\/vs['"]/,
-  "Code editor should load Monaco from Pixcode's same-origin vendor route.",
-);
-
-assert.match(
-  localMonaco,
-  /loader\.config\(\{\s*paths:\s*\{\s*vs:\s*LOCAL_MONACO_BASE_PATH/s,
-  'Code editor should point @monaco-editor/react at the local Monaco loader path.',
-);
-
-assert.doesNotMatch(
-  localMonaco,
-  /https:\/\/cdn\.jsdelivr\.net|unpkg\.com|from ['"]monaco-editor/,
-  'Local Monaco setup should not use a remote CDN or import the full monaco-editor barrel.',
-);
-
-assert.match(
-  read('server/index.js'),
-  /\/vendor\/monaco-editor\/min\/vs[\s\S]*express\.static/,
-  'Server should serve Monaco loader and workers from the same-origin vendor route.',
-);
-
-assert.match(
-  surface,
-  /onMount=\{handleMonacoMount\}/,
-  'Monaco should expose an onMount hook so Pixcode can wire VS Code-style key commands.',
-);
-
-assert.match(
-  surface,
-  /KeyMod\.CtrlCmd\s*\|\s*KeyCode\.KeyS/,
-  'Monaco should handle Ctrl/Cmd+S with the native editor command system.',
-);
-
-assert.match(
-  surface,
-  /selectOnLineNumbers:\s*true/,
-  'Monaco should support line-number click selection like VS Code.',
-);
-
-assert.match(
-  surface,
-  /automaticLayout:\s*true/,
-  'Monaco should relayout cleanly inside resizable workbench panes.',
-);
-
-assert.match(
-  editor,
-  /const useMonacoEditor = !\(/,
-  'CodeEditor should keep the existing CodeMirror diff fallback while using Monaco for regular files.',
-);
-
-assert.match(
-  editor,
-  /useMonacoEditor=\{useMonacoEditor\}/,
-  'CodeEditor should pass the Monaco-vs-diff decision into the editor surface.',
-);
-
-console.log('code editor VS Code engine smoke passed');
+import assert from 'node:assert/strict';
+import fs from 'node:fs';
+
+const read = (path) => fs.readFileSync(path, 'utf8');
+
+const packageJson = JSON.parse(read('package.json'));
+const surface = read('src/components/code-editor/view/subcomponents/CodeEditorSurface.tsx');
+const editor = read('src/components/code-editor/view/CodeEditor.tsx');
+const localMonaco = read('src/components/code-editor/utils/localMonaco.ts');
+
+const allDeps = {
+  ...(packageJson.dependencies ?? {}),
+  ...(packageJson.devDependencies ?? {}),
+};
+
+assert.ok(allDeps['@monaco-editor/react'], 'Code editor should depend on the React Monaco wrapper.');
+assert.ok(allDeps['monaco-editor'], 'Code editor should depend on the Monaco editor engine.');
+
+assert.match(
+  surface,
+  /const MonacoEditor = lazy[\s\S]*import\(['"]@monaco-editor\/react['"]\)/,
+  'Normal file editing should lazy-load Monaco instead of keeping a CodeMirror-only surface.',
+);
+
+assert.match(
+  surface,
+  /ensureLocalMonaco\(\)/,
+  'Monaco should be configured before mount so Linux/self-hosted installs do not depend on the CDN loader.',
+);
+
+assert.match(
+  localMonaco,
+  /LOCAL_MONACO_BASE_PATH\s*=\s*['"]\/vendor\/monaco-editor\/min\/vs['"]/,
+  "Code editor should load Monaco from Pixcode's same-origin vendor route.",
+);
+
+assert.match(
+  localMonaco,
+  /loader\.config\(\{\s*paths:\s*\{\s*vs:\s*LOCAL_MONACO_BASE_PATH/s,
+  'Code editor should point @monaco-editor/react at the local Monaco loader path.',
+);
+
+assert.doesNotMatch(
+  localMonaco,
+  /https:\/\/cdn\.jsdelivr\.net|unpkg\.com|from ['"]monaco-editor/,
+  'Local Monaco setup should not use a remote CDN or import the full monaco-editor barrel.',
+);
+
+assert.match(
+  read('server/index.js'),
+  /\/vendor\/monaco-editor\/min\/vs[\s\S]*express\.static/,
+  'Server should serve Monaco loader and workers from the same-origin vendor route.',
+);
+
+assert.match(
+  surface,
+  /onMount=\{handleMonacoMount\}/,
+  'Monaco should expose an onMount hook so Pixcode can wire VS Code-style key commands.',
+);
+
+assert.match(
+  surface,
+  /KeyMod\.CtrlCmd\s*\|\s*KeyCode\.KeyS/,
+  'Monaco should handle Ctrl/Cmd+S with the native editor command system.',
+);
+
+assert.match(
+  surface,
+  /selectOnLineNumbers:\s*true/,
+  'Monaco should support line-number click selection like VS Code.',
+);
+
+assert.match(
+  surface,
+  /automaticLayout:\s*true/,
+  'Monaco should relayout cleanly inside resizable workbench panes.',
+);
+
+assert.match(
+  editor,
+  /const useMonacoEditor = !\(/,
+  'CodeEditor should keep the existing CodeMirror diff fallback while using Monaco for regular files.',
+);
+
+assert.match(
+  editor,
+  /useMonacoEditor=\{useMonacoEditor\}/,
+  'CodeEditor should pass the Monaco-vs-diff decision into the editor surface.',
+);
+
+console.log('code editor VS Code engine smoke passed');

package/scripts/smoke/command-center-agent-writes.mjs

@@ -1,79 +1,79 @@
-#!/usr/bin/env node
-
-import assert from 'node:assert/strict';
-import { readFileSync } from 'node:fs';
-
-const changedFilesUtil = readFileSync('src/utils/changedFiles.ts', 'utf8');
-const changedFilesHook = readFileSync('src/hooks/useChangedFilesMonitor.ts', 'utf8');
-const mainContent = readFileSync('src/components/main-content/view/MainContent.tsx', 'utf8');
-const rail = readFileSync('src/components/main-content/view/subcomponents/ChangedFilesActivityRail.tsx', 'utf8');
-
-assert.ok(
-  changedFilesUtil.includes('extractChangedFilesFromMessage'),
-  'Changed-files utilities should extract Write/Edit/ApplyPatch tool events from realtime agent messages.',
-);
-
-assert.ok(
-  changedFilesUtil.includes('toolName') && changedFilesUtil.includes('toolInput'),
-  'Realtime extraction should understand normalized toolName/toolInput messages.',
-);
-
-assert.ok(
-  changedFilesUtil.includes('diffInfo') && changedFilesUtil.includes('old_string') && changedFilesUtil.includes('new_string'),
-  'Realtime agent writes should preserve diff payloads so the editor can highlight the changed content.',
-);
-
-assert.ok(
-  changedFilesHook.includes('latestMessage') && changedFilesHook.includes('extractChangedFilesFromMessage'),
-  'Changed-files monitor should ingest the latest realtime agent message in addition to polling git/filesystem status.',
-);
-
-assert.ok(
-  changedFilesHook.includes("ChangedFilesTrackingMode = 'local' | 'git'")
-    && changedFilesHook.includes('mode: trackingMode')
-    && changedFilesHook.includes("trackingMode !== 'local'"),
-  'Changed-files monitor should support separate Local changes and Git changes modes.',
-);
-
-assert.ok(
-  changedFilesHook.includes('mergeChangedFiles'),
-  'Changed-files monitor should merge direct agent writes with polled git/filesystem changes instead of replacing them.',
-);
-
-assert.ok(
-  changedFilesHook.includes("data.trackingMode === 'filesystem'")
-    && changedFilesHook.includes('mergeChangedFiles(polledChangedFilesRef.current, detectedChangedFiles)'),
-  'Filesystem tracking should keep detected files visible after the next empty poll instead of flashing for one interval.',
-);
-
-assert.ok(
-  mainContent.includes('useChangedFilesMonitor(selectedProject')
-    && mainContent.includes('latestMessage')
-    && mainContent.includes('changeTrackingMode'),
-  'MainContent should pass latestMessage into the changed-files monitor.',
-);
-
-assert.ok(
-  mainContent.includes('handleChangedFileOpen') && mainContent.includes('handleFileOpen(file.path, file.diffInfo'),
-  'Clicking a changed file should open the editor with diff context, not only focus the Files panel.',
-);
-
-assert.ok(
-  mainContent.includes('/api/git/file-with-diff') && mainContent.includes('old_string') && mainContent.includes('new_string'),
-  'Clicking a changed file without realtime diff metadata should hydrate editor diff context before focusing the changed chunk.',
-);
-
-assert.ok(
-  rail.includes('trackingMode: ChangedFilesTrackingMode')
-    && rail.includes('Local changes')
-    && rail.includes('Git changes')
-    && rail.includes('onTrackingModeChange'),
-  'Command Center rail should expose a visible switch between Local changes and Git changes.',
-);
-
-assert.ok(
-  rail.includes('onOpenFile: (file: ChangedFileEntry) => void') && rail.includes('onClick={() => onOpenFile(file)}'),
-  'ChangedFilesActivityRail should pass the full changed-file entry, including diffInfo, to the open handler.',
-);
-
-console.log('command center agent writes smoke passed');
+#!/usr/bin/env node
+
+import assert from 'node:assert/strict';
+import { readFileSync } from 'node:fs';
+
+const changedFilesUtil = readFileSync('src/utils/changedFiles.ts', 'utf8');
+const changedFilesHook = readFileSync('src/hooks/useChangedFilesMonitor.ts', 'utf8');
+const mainContent = readFileSync('src/components/main-content/view/MainContent.tsx', 'utf8');
+const rail = readFileSync('src/components/main-content/view/subcomponents/ChangedFilesActivityRail.tsx', 'utf8');
+
+assert.ok(
+  changedFilesUtil.includes('extractChangedFilesFromMessage'),
+  'Changed-files utilities should extract Write/Edit/ApplyPatch tool events from realtime agent messages.',
+);
+
+assert.ok(
+  changedFilesUtil.includes('toolName') && changedFilesUtil.includes('toolInput'),
+  'Realtime extraction should understand normalized toolName/toolInput messages.',
+);
+
+assert.ok(
+  changedFilesUtil.includes('diffInfo') && changedFilesUtil.includes('old_string') && changedFilesUtil.includes('new_string'),
+  'Realtime agent writes should preserve diff payloads so the editor can highlight the changed content.',
+);
+
+assert.ok(
+  changedFilesHook.includes('latestMessage') && changedFilesHook.includes('extractChangedFilesFromMessage'),
+  'Changed-files monitor should ingest the latest realtime agent message in addition to polling git/filesystem status.',
+);
+
+assert.ok(
+  changedFilesHook.includes("ChangedFilesTrackingMode = 'local' | 'git'")
+    && changedFilesHook.includes('mode: trackingMode')
+    && changedFilesHook.includes("trackingMode !== 'local'"),
+  'Changed-files monitor should support separate Local changes and Git changes modes.',
+);
+
+assert.ok(
+  changedFilesHook.includes('mergeChangedFiles'),
+  'Changed-files monitor should merge direct agent writes with polled git/filesystem changes instead of replacing them.',
+);
+
+assert.ok(
+  changedFilesHook.includes("data.trackingMode === 'filesystem'")
+    && changedFilesHook.includes('mergeChangedFiles(polledChangedFilesRef.current, detectedChangedFiles)'),
+  'Filesystem tracking should keep detected files visible after the next empty poll instead of flashing for one interval.',
+);
+
+assert.ok(
+  mainContent.includes('useChangedFilesMonitor(selectedProject')
+    && mainContent.includes('latestMessage')
+    && mainContent.includes('changeTrackingMode'),
+  'MainContent should pass latestMessage into the changed-files monitor.',
+);
+
+assert.ok(
+  mainContent.includes('handleChangedFileOpen') && mainContent.includes('handleFileOpen(file.path, file.diffInfo'),
+  'Clicking a changed file should open the editor with diff context, not only focus the Files panel.',
+);
+
+assert.ok(
+  mainContent.includes('/api/git/file-with-diff') && mainContent.includes('old_string') && mainContent.includes('new_string'),
+  'Clicking a changed file without realtime diff metadata should hydrate editor diff context before focusing the changed chunk.',
+);
+
+assert.ok(
+  rail.includes('trackingMode: ChangedFilesTrackingMode')
+    && rail.includes('Local changes')
+    && rail.includes('Git changes')
+    && rail.includes('onTrackingModeChange'),
+  'Command Center rail should expose a visible switch between Local changes and Git changes.',
+);
+
+assert.ok(
+  rail.includes('onOpenFile: (file: ChangedFileEntry) => void') && rail.includes('onClick={() => onOpenFile(file)}'),
+  'ChangedFilesActivityRail should pass the full changed-file entry, including diffInfo, to the open handler.',
+);
+
+console.log('command center agent writes smoke passed');

package/scripts/smoke/command-center-non-git.mjs

@@ -1,46 +1,46 @@
-#!/usr/bin/env node
-
-import assert from 'node:assert/strict';
-import { readFileSync } from 'node:fs';
-
-const gitRoute = readFileSync('server/routes/git.js', 'utf8');
-const changedFilesHook = readFileSync('src/hooks/useChangedFilesMonitor.ts', 'utf8');
-const mainContent = readFileSync('src/components/main-content/view/MainContent.tsx', 'utf8');
-const quickSettings = readFileSync('src/components/quick-settings-panel/view/QuickSettingsContent.tsx', 'utf8');
-
-assert.ok(
-  gitRoute.includes('isGitRepository: false'),
-  'Git status route should return a structured non-git status instead of a Git operation failed error.',
-);
-
-assert.ok(
-  gitRoute.includes('filesystemChangeSnapshots'),
-  'Git status route should maintain filesystem snapshots for local-only projects.',
-);
-
-assert.ok(
-  changedFilesHook.includes('isGitRepository?: boolean'),
-  'Changed-files monitor should understand git-backed and filesystem-backed status payloads.',
-);
-
-assert.ok(
-  !changedFilesHook.includes('setError(data.details || data.error'),
-  'Changed-files monitor should not surface the non-git fallback as a blocking error.',
-);
-
-assert.ok(
-  mainContent.includes('ChangedFilesActivityRail'),
-  'Main content should render changed-file activity beside chat/orchestration, not only in Quick Settings.',
-);
-
-assert.ok(
-  mainContent.includes('setFocusedChangedFilePath(latestDetectedFile.path)'),
-  'New changes should highlight the activity rail without automatically stealing the user into the Files panel.',
-);
-
-assert.ok(
-  quickSettings.includes('changedFilesSummary'),
-  'Quick Settings should keep a compact Command Center summary instead of being the main changed-files surface.',
-);
-
-console.log('command center non-git smoke passed');
+#!/usr/bin/env node
+
+import assert from 'node:assert/strict';
+import { readFileSync } from 'node:fs';
+
+const gitRoute = readFileSync('server/routes/git.js', 'utf8');
+const changedFilesHook = readFileSync('src/hooks/useChangedFilesMonitor.ts', 'utf8');
+const mainContent = readFileSync('src/components/main-content/view/MainContent.tsx', 'utf8');
+const quickSettings = readFileSync('src/components/quick-settings-panel/view/QuickSettingsContent.tsx', 'utf8');
+
+assert.ok(
+  gitRoute.includes('isGitRepository: false'),
+  'Git status route should return a structured non-git status instead of a Git operation failed error.',
+);
+
+assert.ok(
+  gitRoute.includes('filesystemChangeSnapshots'),
+  'Git status route should maintain filesystem snapshots for local-only projects.',
+);
+
+assert.ok(
+  changedFilesHook.includes('isGitRepository?: boolean'),
+  'Changed-files monitor should understand git-backed and filesystem-backed status payloads.',
+);
+
+assert.ok(
+  !changedFilesHook.includes('setError(data.details || data.error'),
+  'Changed-files monitor should not surface the non-git fallback as a blocking error.',
+);
+
+assert.ok(
+  mainContent.includes('ChangedFilesActivityRail'),
+  'Main content should render changed-file activity beside chat/orchestration, not only in Quick Settings.',
+);
+
+assert.ok(
+  mainContent.includes('setFocusedChangedFilePath(latestDetectedFile.path)'),
+  'New changes should highlight the activity rail without automatically stealing the user into the Files panel.',
+);
+
+assert.ok(
+  quickSettings.includes('changedFilesSummary'),
+  'Quick Settings should keep a compact Command Center summary instead of being the main changed-files surface.',
+);
+
+console.log('command center non-git smoke passed');

package/scripts/smoke/context-packet.mjs

@@ -1,43 +1,43 @@
-#!/usr/bin/env node
-
-import assert from 'node:assert/strict';
-import fs from 'node:fs';
-import path from 'node:path';
-
-const root = process.cwd();
-
-function read(relativePath) {
-  return fs.readFileSync(path.join(root, relativePath), 'utf8');
-}
-
-const contextSource = read('server/modules/orchestration/workflows/context-packet.ts');
-assert.match(contextSource, /PIXCODE_CONTEXT_PROTOCOL/, 'Context packets should declare a stable protocol id.');
-assert.match(contextSource, /pixcode\.context\.v1/, 'Context packets should use the v1 protocol id.');
-assert.match(contextSource, /originalUserRequest/, 'Context packets should preserve the original user request.');
-assert.match(contextSource, /project/, 'Context packets should include project metadata.');
-assert.match(contextSource, /task/, 'Context packets should include task metadata.');
-assert.match(contextSource, /constraints/, 'Context packets should include execution constraints.');
-assert.match(contextSource, /upstreamArtifacts/, 'Context packets should include upstream artifact context.');
-assert.match(contextSource, /runState/, 'Context packets should include current run state.');
-assert.match(contextSource, /compaction/, 'Context packets should expose compaction metadata.');
-assert.match(contextSource, /formatContextPacketForPrompt/, 'Context packets should be formatted for prompts.');
-
-const types = read('server/modules/orchestration/workflows/workflow.types.ts');
-assert.match(types, /WorkflowContextPacket/, 'Workflow types should expose context packets.');
-assert.match(types, /contextPacket\?: WorkflowContextPacket/, 'Node runs should persist context packets.');
-
-const runner = read('server/modules/orchestration/workflows/workflow-runner.ts');
-assert.match(runner, /buildWorkflowContextPacket/, 'Workflow runner should build context packets.');
-assert.match(runner, /formatContextPacketForPrompt/, 'Workflow runner should inject context packets into prompts.');
-assert.ok(
-  runner.indexOf('Original user request') < runner.indexOf('formatContextPacketForPrompt(contextPacket)')
-    && runner.indexOf('formatContextPacketForPrompt(contextPacket)') < runner.indexOf('workspaceContextPrompt(workspaceTarget)'),
-  'Original request must stay before structured context, and workspace context must remain derived context.',
-);
-
-const trace = read('server/modules/orchestration/workflows/workflow-trace.ts');
-assert.match(trace, /contextPacket/, 'Workflow trace should surface context packet metadata.');
-assert.match(trace, /workflow\.trace\.contextPacket/, 'Workflow trace should label context packet events.');
-assert.match(trace, /compaction/, 'Workflow trace should include context compaction metadata.');
-
-console.log('context packet smoke passed');
+#!/usr/bin/env node
+
+import assert from 'node:assert/strict';
+import fs from 'node:fs';
+import path from 'node:path';
+
+const root = process.cwd();
+
+function read(relativePath) {
+  return fs.readFileSync(path.join(root, relativePath), 'utf8');
+}
+
+const contextSource = read('server/modules/orchestration/workflows/context-packet.ts');
+assert.match(contextSource, /PIXCODE_CONTEXT_PROTOCOL/, 'Context packets should declare a stable protocol id.');
+assert.match(contextSource, /pixcode\.context\.v1/, 'Context packets should use the v1 protocol id.');
+assert.match(contextSource, /originalUserRequest/, 'Context packets should preserve the original user request.');
+assert.match(contextSource, /project/, 'Context packets should include project metadata.');
+assert.match(contextSource, /task/, 'Context packets should include task metadata.');
+assert.match(contextSource, /constraints/, 'Context packets should include execution constraints.');
+assert.match(contextSource, /upstreamArtifacts/, 'Context packets should include upstream artifact context.');
+assert.match(contextSource, /runState/, 'Context packets should include current run state.');
+assert.match(contextSource, /compaction/, 'Context packets should expose compaction metadata.');
+assert.match(contextSource, /formatContextPacketForPrompt/, 'Context packets should be formatted for prompts.');
+
+const types = read('server/modules/orchestration/workflows/workflow.types.ts');
+assert.match(types, /WorkflowContextPacket/, 'Workflow types should expose context packets.');
+assert.match(types, /contextPacket\?: WorkflowContextPacket/, 'Node runs should persist context packets.');
+
+const runner = read('server/modules/orchestration/workflows/workflow-runner.ts');
+assert.match(runner, /buildWorkflowContextPacket/, 'Workflow runner should build context packets.');
+assert.match(runner, /formatContextPacketForPrompt/, 'Workflow runner should inject context packets into prompts.');
+assert.ok(
+  runner.indexOf('Original user request') < runner.indexOf('formatContextPacketForPrompt(contextPacket)')
+    && runner.indexOf('formatContextPacketForPrompt(contextPacket)') < runner.indexOf('workspaceContextPrompt(workspaceTarget)'),
+  'Original request must stay before structured context, and workspace context must remain derived context.',
+);
+
+const trace = read('server/modules/orchestration/workflows/workflow-trace.ts');
+assert.match(trace, /contextPacket/, 'Workflow trace should surface context packet metadata.');
+assert.match(trace, /workflow\.trace\.contextPacket/, 'Workflow trace should label context packet events.');
+assert.match(trace, /compaction/, 'Workflow trace should include context compaction metadata.');
+
+console.log('context packet smoke passed');