@pixelbyte-software/pixcode 1.54.7 → 1.54.8

package/scripts/smoke/notification-center.mjs

@@ -1,87 +1,87 @@
-#!/usr/bin/env node
-
-import assert from 'node:assert/strict';
-import { readFileSync } from 'node:fs';
-
-const orchestrator = readFileSync('server/services/notification-orchestrator.js', 'utf8');
-const server = readFileSync('server/index.js', 'utf8');
-const workflowRoutes = readFileSync('server/modules/orchestration/workflows/workflow.routes.ts', 'utf8');
-const workflowRunner = readFileSync('server/modules/orchestration/workflows/workflow-runner.ts', 'utf8');
-const appContent = readFileSync('src/components/app/AppContent.tsx', 'utf8');
-const notificationCenter = readFileSync('src/components/notifications/InAppNotificationCenter.tsx', 'utf8');
-const localNotifications = readFileSync('src/utils/localNotifications.ts', 'utf8');
-const settingsTypes = readFileSync('src/components/settings/types/types.ts', 'utf8');
-
-assert.ok(
-  orchestrator.includes('setNotificationWebSocketServer'),
-  'Notification orchestrator should accept the shared WebSocket server for in-app delivery.',
-);
-
-assert.ok(
-  orchestrator.includes('broadcastInAppNotification'),
-  'Notification orchestrator should broadcast in-app notification events.',
-);
-
-assert.ok(
-  orchestrator.includes("type: 'notification:event'"),
-  'In-app notifications should use a stable websocket event type.',
-);
-
-assert.ok(
-  !orchestrator.includes('id: event.dedupeKey ||'),
-  'Notification payload ids should not reuse the dedupe key forever; repeated completed runs for the same session need fresh local notifications.',
-);
-
-assert.ok(
-  orchestrator.includes('event.createdAt') && orchestrator.includes('dedupeKey'),
-  'Notification payloads should preserve dedupe metadata while still carrying a per-event timestamp.',
-);
-
-assert.ok(
-  server.includes('setNotificationWebSocketServer(wss)'),
-  'Server should attach the WebSocket server to notification orchestrator.',
-);
-
-assert.ok(
-  server.includes('ws.userId = request?.user?.id ?? request?.user?.userId ?? null'),
-  'Chat WebSocket connections should carry user id for targeted notification delivery.',
-);
-
-assert.ok(
-  workflowRoutes.includes('readRequestUserId') && workflowRoutes.includes('userId: readRequestUserId(req)'),
-  'Orchestration run creation should carry the authenticated user id into workflow metadata.',
-);
-
-assert.ok(
-  workflowRunner.includes('notifyWorkflowRunFinished')
-    && workflowRunner.includes('notifyRunStopped')
-    && workflowRunner.includes('notifyRunFailed'),
-  'Workflow runner should notify users when orchestration runs complete or fail.',
-);
-
-assert.ok(
-  appContent.includes('<InAppNotificationCenter latestMessage={latestMessage} />'),
-  'App shell should mount the in-app notification center.',
-);
-
-assert.ok(
-  notificationCenter.includes("message.type !== 'notification:event'"),
-  'Notification center should listen only to notification:event payloads.',
-);
-
-assert.ok(
-  notificationCenter.includes('notifyLocalEventOnce'),
-  'Notification center should attempt browser/local notification fallback when supported.',
-);
-
-assert.ok(
-  localNotifications.includes('notifyLocalEventOnce'),
-  'Local notification utilities should support event-specific browser notifications.',
-);
-
-assert.ok(
-  settingsTypes.includes('telegram: boolean') && settingsTypes.includes('desktop: boolean'),
-  'Notification preferences should model Telegram and desktop/native channels alongside in-app and web push.',
-);
-
-console.log('notification center smoke passed');
+#!/usr/bin/env node
+
+import assert from 'node:assert/strict';
+import { readFileSync } from 'node:fs';
+
+const orchestrator = readFileSync('server/services/notification-orchestrator.js', 'utf8');
+const server = readFileSync('server/index.js', 'utf8');
+const workflowRoutes = readFileSync('server/modules/orchestration/workflows/workflow.routes.ts', 'utf8');
+const workflowRunner = readFileSync('server/modules/orchestration/workflows/workflow-runner.ts', 'utf8');
+const appContent = readFileSync('src/components/app/AppContent.tsx', 'utf8');
+const notificationCenter = readFileSync('src/components/notifications/InAppNotificationCenter.tsx', 'utf8');
+const localNotifications = readFileSync('src/utils/localNotifications.ts', 'utf8');
+const settingsTypes = readFileSync('src/components/settings/types/types.ts', 'utf8');
+
+assert.ok(
+  orchestrator.includes('setNotificationWebSocketServer'),
+  'Notification orchestrator should accept the shared WebSocket server for in-app delivery.',
+);
+
+assert.ok(
+  orchestrator.includes('broadcastInAppNotification'),
+  'Notification orchestrator should broadcast in-app notification events.',
+);
+
+assert.ok(
+  orchestrator.includes("type: 'notification:event'"),
+  'In-app notifications should use a stable websocket event type.',
+);
+
+assert.ok(
+  !orchestrator.includes('id: event.dedupeKey ||'),
+  'Notification payload ids should not reuse the dedupe key forever; repeated completed runs for the same session need fresh local notifications.',
+);
+
+assert.ok(
+  orchestrator.includes('event.createdAt') && orchestrator.includes('dedupeKey'),
+  'Notification payloads should preserve dedupe metadata while still carrying a per-event timestamp.',
+);
+
+assert.ok(
+  server.includes('setNotificationWebSocketServer(wss)'),
+  'Server should attach the WebSocket server to notification orchestrator.',
+);
+
+assert.ok(
+  server.includes('ws.userId = request?.user?.id ?? request?.user?.userId ?? null'),
+  'Chat WebSocket connections should carry user id for targeted notification delivery.',
+);
+
+assert.ok(
+  workflowRoutes.includes('readRequestUserId') && workflowRoutes.includes('userId: readRequestUserId(req)'),
+  'Orchestration run creation should carry the authenticated user id into workflow metadata.',
+);
+
+assert.ok(
+  workflowRunner.includes('notifyWorkflowRunFinished')
+    && workflowRunner.includes('notifyRunStopped')
+    && workflowRunner.includes('notifyRunFailed'),
+  'Workflow runner should notify users when orchestration runs complete or fail.',
+);
+
+assert.ok(
+  appContent.includes('<InAppNotificationCenter latestMessage={latestMessage} />'),
+  'App shell should mount the in-app notification center.',
+);
+
+assert.ok(
+  notificationCenter.includes("message.type !== 'notification:event'"),
+  'Notification center should listen only to notification:event payloads.',
+);
+
+assert.ok(
+  notificationCenter.includes('notifyLocalEventOnce'),
+  'Notification center should attempt browser/local notification fallback when supported.',
+);
+
+assert.ok(
+  localNotifications.includes('notifyLocalEventOnce'),
+  'Local notification utilities should support event-specific browser notifications.',
+);
+
+assert.ok(
+  settingsTypes.includes('telegram: boolean') && settingsTypes.includes('desktop: boolean'),
+  'Notification preferences should model Telegram and desktop/native channels alongside in-app and web push.',
+);
+
+console.log('notification center smoke passed');

package/scripts/smoke/notification-inapp-preference.mjs

@@ -1,23 +1,23 @@
-#!/usr/bin/env node
-
-import assert from 'node:assert/strict';
-import { readFileSync } from 'node:fs';
-
-const notificationCenter = readFileSync('src/components/notifications/InAppNotificationCenter.tsx', 'utf8');
-
-assert.ok(
-  notificationCenter.includes('inAppEnabledByPreference'),
-  'In-app notification center should explicitly read notificationPreferences.channels.inApp.',
-);
-
-assert.ok(
-  notificationCenter.includes('if (!inAppEnabled)'),
-  'In-app notification center should skip storing/opening alerts when the in-app channel is disabled.',
-);
-
-assert.ok(
-  notificationCenter.indexOf('notifyLocalEventOnce') < notificationCenter.indexOf('if (!inAppEnabled)'),
-  'Desktop/browser notification dispatch should remain independent from the in-app channel guard.',
-);
-
-console.log('notification in-app preference smoke passed');
+#!/usr/bin/env node
+
+import assert from 'node:assert/strict';
+import { readFileSync } from 'node:fs';
+
+const notificationCenter = readFileSync('src/components/notifications/InAppNotificationCenter.tsx', 'utf8');
+
+assert.ok(
+  notificationCenter.includes('inAppEnabledByPreference'),
+  'In-app notification center should explicitly read notificationPreferences.channels.inApp.',
+);
+
+assert.ok(
+  notificationCenter.includes('if (!inAppEnabled)'),
+  'In-app notification center should skip storing/opening alerts when the in-app channel is disabled.',
+);
+
+assert.ok(
+  notificationCenter.indexOf('notifyLocalEventOnce') < notificationCenter.indexOf('if (!inAppEnabled)'),
+  'Desktop/browser notification dispatch should remain independent from the in-app channel guard.',
+);
+
+console.log('notification in-app preference smoke passed');

package/scripts/smoke/notification-taxonomy.mjs

@@ -1,58 +1,58 @@
-import assert from 'node:assert/strict';
-
-import {
-  NOTIFICATION_EVENT_TYPES,
-  createNotificationEventContract,
-  getNotificationPreferenceKey,
-  normalizeNotificationEvent,
-} from '../../server/services/notification-taxonomy.js';
-
-const requiredEvents = [
-  'CHAT_DONE',
-  'ORCHESTRATION_DONE',
-  'APPROVAL_NEEDED',
-  'ERROR',
-  'TEST_FAILED',
-  'LIVE_VIEW_FAILED',
-];
-
-for (const name of requiredEvents) {
-  assert.equal(typeof NOTIFICATION_EVENT_TYPES[name], 'string', `${name} must be a public event type`);
-}
-
-const approval = normalizeNotificationEvent({
-  provider: 'claude',
-  sessionId: 'session-1',
-  kind: 'action_required',
-  code: 'permission.required',
-  severity: 'warning',
-});
-assert.equal(approval.eventType, NOTIFICATION_EVENT_TYPES.APPROVAL_NEEDED);
-assert.equal(getNotificationPreferenceKey(approval), 'actionRequired');
-
-const chatDone = normalizeNotificationEvent({
-  provider: 'codex',
-  sessionId: 'session-2',
-  eventType: NOTIFICATION_EVENT_TYPES.CHAT_DONE,
-  code: 'run.stopped',
-});
-assert.equal(chatDone.kind, 'stop');
-assert.equal(chatDone.severity, 'info');
-assert.equal(getNotificationPreferenceKey(chatDone), 'stop');
-
-const liveViewFailed = normalizeNotificationEvent({
-  provider: 'system',
-  sessionId: 'preview-1',
-  eventType: NOTIFICATION_EVENT_TYPES.LIVE_VIEW_FAILED,
-  code: 'live_view.failed',
-});
-assert.equal(liveViewFailed.kind, 'error');
-assert.equal(liveViewFailed.severity, 'error');
-assert.equal(getNotificationPreferenceKey(liveViewFailed), 'error');
-
-const publicContract = createNotificationEventContract(liveViewFailed);
-assert.equal(publicContract.eventType, NOTIFICATION_EVENT_TYPES.LIVE_VIEW_FAILED);
-assert.equal(publicContract.preferenceKey, 'error');
-assert.equal(publicContract.category, 'live_view');
-
-console.log('notification taxonomy smoke passed');
+import assert from 'node:assert/strict';
+
+import {
+  NOTIFICATION_EVENT_TYPES,
+  createNotificationEventContract,
+  getNotificationPreferenceKey,
+  normalizeNotificationEvent,
+} from '../../server/services/notification-taxonomy.js';
+
+const requiredEvents = [
+  'CHAT_DONE',
+  'ORCHESTRATION_DONE',
+  'APPROVAL_NEEDED',
+  'ERROR',
+  'TEST_FAILED',
+  'LIVE_VIEW_FAILED',
+];
+
+for (const name of requiredEvents) {
+  assert.equal(typeof NOTIFICATION_EVENT_TYPES[name], 'string', `${name} must be a public event type`);
+}
+
+const approval = normalizeNotificationEvent({
+  provider: 'claude',
+  sessionId: 'session-1',
+  kind: 'action_required',
+  code: 'permission.required',
+  severity: 'warning',
+});
+assert.equal(approval.eventType, NOTIFICATION_EVENT_TYPES.APPROVAL_NEEDED);
+assert.equal(getNotificationPreferenceKey(approval), 'actionRequired');
+
+const chatDone = normalizeNotificationEvent({
+  provider: 'codex',
+  sessionId: 'session-2',
+  eventType: NOTIFICATION_EVENT_TYPES.CHAT_DONE,
+  code: 'run.stopped',
+});
+assert.equal(chatDone.kind, 'stop');
+assert.equal(chatDone.severity, 'info');
+assert.equal(getNotificationPreferenceKey(chatDone), 'stop');
+
+const liveViewFailed = normalizeNotificationEvent({
+  provider: 'system',
+  sessionId: 'preview-1',
+  eventType: NOTIFICATION_EVENT_TYPES.LIVE_VIEW_FAILED,
+  code: 'live_view.failed',
+});
+assert.equal(liveViewFailed.kind, 'error');
+assert.equal(liveViewFailed.severity, 'error');
+assert.equal(getNotificationPreferenceKey(liveViewFailed), 'error');
+
+const publicContract = createNotificationEventContract(liveViewFailed);
+assert.equal(publicContract.eventType, NOTIFICATION_EVENT_TYPES.LIVE_VIEW_FAILED);
+assert.equal(publicContract.preferenceKey, 'error');
+assert.equal(publicContract.category, 'live_view');
+
+console.log('notification taxonomy smoke passed');