@pixelbyte-software/pixcode 1.54.9 → 1.54.11

package/server/modules/orchestration/a2a/validator.ts

@@ -1,113 +1,113 @@
-// server/modules/orchestration/a2a/validator.ts
-// Hand-written validators for incoming A2A payloads.
-// We deliberately avoid adding a new dep (zod, ajv) for the
-// foundation; a follow-on plan can swap to a schema lib if needed.
-//
-// All path strings use JSONPath-style "$" as the document root so
-// callers can map errors to wire-payload locations consistently.
-
-import type { AgentCard, DataPart, FilePart, Message, Part, SubmitTaskInput, TextPart } from '@/modules/orchestration/a2a/types.js';
-
-export class A2AValidationError extends Error {
-  constructor(message: string, public readonly path: string) {
-    super(`${path}: ${message}`);
-    this.name = 'A2AValidationError';
-  }
-}
-
-function assertNonEmptyString(value: unknown, path: string): asserts value is string {
-  if (typeof value !== 'string' || value.length === 0) {
-    throw new A2AValidationError('expected non-empty string', path);
-  }
-}
-
-function assertOptionalNonEmptyString(value: unknown, path: string): void {
-  if (value === undefined) {
-    return;
-  }
-  assertNonEmptyString(value, path);
-}
-
-function assertPlainObject(value: unknown, path: string): void {
-  if (!value || typeof value !== 'object' || Array.isArray(value)) {
-    throw new A2AValidationError('expected plain object', path);
-  }
-}
-
-function assertPart(value: unknown, path: string): asserts value is Part {
-  if (!value || typeof value !== 'object') {
-    throw new A2AValidationError('expected object', path);
-  }
-  const part = value as { kind?: unknown };
-  if (part.kind === 'text') {
-    assertNonEmptyString((part as Partial<TextPart>).text, `${path}.text`);
-    return;
-  }
-  if (part.kind === 'file') {
-    assertNonEmptyString((part as Partial<FilePart>).name, `${path}.name`);
-    return;
-  }
-  if (part.kind === 'data') {
-    const data = (part as Partial<DataPart>).data;
-    if (!data || typeof data !== 'object' || Array.isArray(data)) {
-      throw new A2AValidationError('data must be a plain object', `${path}.data`);
-    }
-    return;
-  }
-  throw new A2AValidationError('part.kind must be text|file|data', `${path}.kind`);
-}
-
-export function assertMessage(value: unknown, path = '$'): asserts value is Message {
-  if (!value || typeof value !== 'object') {
-    throw new A2AValidationError('expected object', path);
-  }
-  const m = value as { messageId?: unknown; role?: unknown; parts?: unknown; taskId?: unknown };
-  assertNonEmptyString(m.messageId, `${path}.messageId`);
-  if (m.role !== 'user' && m.role !== 'agent') {
-    throw new A2AValidationError('role must be user|agent', `${path}.role`);
-  }
-  if (!Array.isArray(m.parts) || m.parts.length === 0) {
-    throw new A2AValidationError('parts must be non-empty array', `${path}.parts`);
-  }
-  assertOptionalNonEmptyString(m.taskId, `${path}.taskId`);
-  m.parts.forEach((p, i) => assertPart(p, `${path}.parts[${i}]`));
-}
-
-export function assertSubmitTaskInput(value: unknown): asserts value is SubmitTaskInput {
-  if (!value || typeof value !== 'object') {
-    throw new A2AValidationError('expected object', '$');
-  }
-  const v = value as {
-    message?: unknown;
-    adapterId?: unknown;
-    contextId?: unknown;
-    metadata?: unknown;
-  };
-  assertMessage(v.message, '$.message');
-  assertNonEmptyString(v.adapterId, '$.adapterId');
-  assertOptionalNonEmptyString(v.contextId, '$.contextId');
-  if (v.metadata !== undefined) {
-    assertPlainObject(v.metadata, '$.metadata');
-  }
-}
-
-export function assertAgentCard(value: unknown): asserts value is AgentCard {
-  if (!value || typeof value !== 'object') {
-    throw new A2AValidationError('expected object', '$');
-  }
-  const card = value as Partial<AgentCard>;
-  assertNonEmptyString(card.name, '$.name');
-  assertNonEmptyString(card.description, '$.description');
-  assertNonEmptyString(card.url, '$.url');
-  assertNonEmptyString(card.version, '$.version');
-  if (!Array.isArray(card.capabilities)) {
-    throw new A2AValidationError('capabilities must be array', '$.capabilities');
-  }
-  if (!Array.isArray(card.skills)) {
-    throw new A2AValidationError('skills must be array', '$.skills');
-  }
-  card.skills.forEach((s, i) => {
-    assertNonEmptyString(s.id, `$.skills[${i}].id`);
-    assertNonEmptyString(s.description, `$.skills[${i}].description`);
-  });
-}
+// server/modules/orchestration/a2a/validator.ts
+// Hand-written validators for incoming A2A payloads.
+// We deliberately avoid adding a new dep (zod, ajv) for the
+// foundation; a follow-on plan can swap to a schema lib if needed.
+//
+// All path strings use JSONPath-style "$" as the document root so
+// callers can map errors to wire-payload locations consistently.
+
+import type { AgentCard, DataPart, FilePart, Message, Part, SubmitTaskInput, TextPart } from '@/modules/orchestration/a2a/types.js';
+
+export class A2AValidationError extends Error {
+  constructor(message: string, public readonly path: string) {
+    super(`${path}: ${message}`);
+    this.name = 'A2AValidationError';
+  }
+}
+
+function assertNonEmptyString(value: unknown, path: string): asserts value is string {
+  if (typeof value !== 'string' || value.length === 0) {
+    throw new A2AValidationError('expected non-empty string', path);
+  }
+}
+
+function assertOptionalNonEmptyString(value: unknown, path: string): void {
+  if (value === undefined) {
+    return;
+  }
+  assertNonEmptyString(value, path);
+}
+
+function assertPlainObject(value: unknown, path: string): void {
+  if (!value || typeof value !== 'object' || Array.isArray(value)) {
+    throw new A2AValidationError('expected plain object', path);
+  }
+}
+
+function assertPart(value: unknown, path: string): asserts value is Part {
+  if (!value || typeof value !== 'object') {
+    throw new A2AValidationError('expected object', path);
+  }
+  const part = value as { kind?: unknown };
+  if (part.kind === 'text') {
+    assertNonEmptyString((part as Partial<TextPart>).text, `${path}.text`);
+    return;
+  }
+  if (part.kind === 'file') {
+    assertNonEmptyString((part as Partial<FilePart>).name, `${path}.name`);
+    return;
+  }
+  if (part.kind === 'data') {
+    const data = (part as Partial<DataPart>).data;
+    if (!data || typeof data !== 'object' || Array.isArray(data)) {
+      throw new A2AValidationError('data must be a plain object', `${path}.data`);
+    }
+    return;
+  }
+  throw new A2AValidationError('part.kind must be text|file|data', `${path}.kind`);
+}
+
+export function assertMessage(value: unknown, path = '$'): asserts value is Message {
+  if (!value || typeof value !== 'object') {
+    throw new A2AValidationError('expected object', path);
+  }
+  const m = value as { messageId?: unknown; role?: unknown; parts?: unknown; taskId?: unknown };
+  assertNonEmptyString(m.messageId, `${path}.messageId`);
+  if (m.role !== 'user' && m.role !== 'agent') {
+    throw new A2AValidationError('role must be user|agent', `${path}.role`);
+  }
+  if (!Array.isArray(m.parts) || m.parts.length === 0) {
+    throw new A2AValidationError('parts must be non-empty array', `${path}.parts`);
+  }
+  assertOptionalNonEmptyString(m.taskId, `${path}.taskId`);
+  m.parts.forEach((p, i) => assertPart(p, `${path}.parts[${i}]`));
+}
+
+export function assertSubmitTaskInput(value: unknown): asserts value is SubmitTaskInput {
+  if (!value || typeof value !== 'object') {
+    throw new A2AValidationError('expected object', '$');
+  }
+  const v = value as {
+    message?: unknown;
+    adapterId?: unknown;
+    contextId?: unknown;
+    metadata?: unknown;
+  };
+  assertMessage(v.message, '$.message');
+  assertNonEmptyString(v.adapterId, '$.adapterId');
+  assertOptionalNonEmptyString(v.contextId, '$.contextId');
+  if (v.metadata !== undefined) {
+    assertPlainObject(v.metadata, '$.metadata');
+  }
+}
+
+export function assertAgentCard(value: unknown): asserts value is AgentCard {
+  if (!value || typeof value !== 'object') {
+    throw new A2AValidationError('expected object', '$');
+  }
+  const card = value as Partial<AgentCard>;
+  assertNonEmptyString(card.name, '$.name');
+  assertNonEmptyString(card.description, '$.description');
+  assertNonEmptyString(card.url, '$.url');
+  assertNonEmptyString(card.version, '$.version');
+  if (!Array.isArray(card.capabilities)) {
+    throw new A2AValidationError('capabilities must be array', '$.capabilities');
+  }
+  if (!Array.isArray(card.skills)) {
+    throw new A2AValidationError('skills must be array', '$.skills');
+  }
+  card.skills.forEach((s, i) => {
+    assertNonEmptyString(s.id, `$.skills[${i}].id`);
+    assertNonEmptyString(s.description, `$.skills[${i}].description`);
+  });
+}

package/server/modules/orchestration/index.ts

@@ -1,99 +1,99 @@
-// server/modules/orchestration/index.ts
-// Public surface for the orchestration module.
-// All cross-module consumers must import from here per
-// eslint.config.js boundaries rules.
-
-export { adapterRegistry } from './a2a/adapter-registry.js';
-export { ClaudeCodeA2AAdapter } from './a2a/adapters/claude-code.adapter.js';
-export { CodexA2AAdapter } from './a2a/adapters/codex.adapter.js';
-export { CursorA2AAdapter } from './a2a/adapters/cursor.adapter.js';
-export { GeminiA2AAdapter } from './a2a/adapters/gemini.adapter.js';
-export { QwenA2AAdapter } from './a2a/adapters/qwen.adapter.js';
-export { OpenCodeA2AAdapter } from './a2a/adapters/opencode.adapter.js';
-export { JsonEventA2AAdapter } from './a2a/adapters/json-event.adapter.js';
-export type {
-  AdapterContext,
-  TaskHandle,
-} from './a2a/adapters/abstract-a2a.adapter.js';
-export { AbstractA2AAdapter } from './a2a/adapters/abstract-a2a.adapter.js';
-export { a2aBus } from './a2a/bus.js';
-export { portWatcher } from './preview/port-watcher.js';
-export { createPreviewProxyRouter } from './preview/preview-proxy.js';
-export {
-  DEFAULT_PERMISSION_POLICY,
-  PERMISSION_CAPABILITIES,
-  PERMISSION_POLICY_MODES,
-  PIXCODE_PERMISSION_POLICY_PROTOCOL,
-  evaluatePermissionRequest,
-  normalizePermissionPolicy,
-  resolvePermissionPolicyFromMetadata,
-} from './security/permission-policy.js';
-export { createOrchestrationTaskRouter } from './tasks/orchestration-task.routes.js';
-export { orchestrationTaskService } from './tasks/orchestration-task.service.js';
-export { createWorkflowRouter } from './workflows/workflow.routes.js';
-export {
-  listPendingApprovals,
-  resolvePermissionApproval,
-} from './workflows/approval-queue.js';
-export {
-  PIXCODE_WORKFLOW_TEMPLATE_PROTOCOL,
-  applyWorkflowTemplateToMetadata,
-  builtInWorkflowTemplates,
-  getWorkflowTemplate,
-} from './workflows/workflow-templates.js';
-export { workflowRunner } from './workflows/workflow-runner.js';
-export { workflowStore } from './workflows/workflow-store.js';
-export { workspaceManager } from './workspace/workspace-manager.js';
-export type {
-  AgentCard,
-  Artifact,
-  ArtifactType,
-  BusEvent,
-  Message,
-  Part,
-  SubmitTaskInput,
-  Task,
-  TaskSummary,
-  TaskError,
-  TaskState,
-} from './a2a/types.js';
-export type {
-  PortEvent,
-  PreviewArtifactData,
-} from './preview/types.js';
-export type {
-  PermissionApprovalRequest,
-  PermissionCapability,
-  PermissionDecision,
-  PermissionPolicy,
-  PermissionPolicyContext,
-  PermissionPolicyEvent,
-  PermissionPolicyMode,
-} from './security/permission-policy.js';
-export type {
-  CreateOrchestrationTaskInput,
-  DispatchOrchestrationTaskInput,
-  OrchestrationTask,
-  OrchestrationTaskState,
-} from './tasks/orchestration-task.types.js';
-export type {
-  WorkflowTemplate,
-  WorkflowTemplateAgentSlot,
-} from './workflows/workflow-templates.js';
-export type {
-  Workflow,
-  WorkflowNode,
-  WorkflowNodeRun,
-  WorkflowNodeStatus,
-  WorkflowRun,
-  WorkflowRunStatus,
-  WorkflowTraceEvent,
-} from './workflows/workflow.types.js';
-export type {
-  ExecResult,
-  WorkspaceHandle,
-  WorkspaceKind,
-  WorkspaceMetadata,
-  WorkspaceRequest,
-} from './workspace/types.js';
-export { WorkspaceError } from './workspace/types.js';
+// server/modules/orchestration/index.ts
+// Public surface for the orchestration module.
+// All cross-module consumers must import from here per
+// eslint.config.js boundaries rules.
+
+export { adapterRegistry } from './a2a/adapter-registry.js';
+export { ClaudeCodeA2AAdapter } from './a2a/adapters/claude-code.adapter.js';
+export { CodexA2AAdapter } from './a2a/adapters/codex.adapter.js';
+export { CursorA2AAdapter } from './a2a/adapters/cursor.adapter.js';
+export { GeminiA2AAdapter } from './a2a/adapters/gemini.adapter.js';
+export { QwenA2AAdapter } from './a2a/adapters/qwen.adapter.js';
+export { OpenCodeA2AAdapter } from './a2a/adapters/opencode.adapter.js';
+export { JsonEventA2AAdapter } from './a2a/adapters/json-event.adapter.js';
+export type {
+  AdapterContext,
+  TaskHandle,
+} from './a2a/adapters/abstract-a2a.adapter.js';
+export { AbstractA2AAdapter } from './a2a/adapters/abstract-a2a.adapter.js';
+export { a2aBus } from './a2a/bus.js';
+export { portWatcher } from './preview/port-watcher.js';
+export { createPreviewProxyRouter } from './preview/preview-proxy.js';
+export {
+  DEFAULT_PERMISSION_POLICY,
+  PERMISSION_CAPABILITIES,
+  PERMISSION_POLICY_MODES,
+  PIXCODE_PERMISSION_POLICY_PROTOCOL,
+  evaluatePermissionRequest,
+  normalizePermissionPolicy,
+  resolvePermissionPolicyFromMetadata,
+} from './security/permission-policy.js';
+export { createOrchestrationTaskRouter } from './tasks/orchestration-task.routes.js';
+export { orchestrationTaskService } from './tasks/orchestration-task.service.js';
+export { createWorkflowRouter } from './workflows/workflow.routes.js';
+export {
+  listPendingApprovals,
+  resolvePermissionApproval,
+} from './workflows/approval-queue.js';
+export {
+  PIXCODE_WORKFLOW_TEMPLATE_PROTOCOL,
+  applyWorkflowTemplateToMetadata,
+  builtInWorkflowTemplates,
+  getWorkflowTemplate,
+} from './workflows/workflow-templates.js';
+export { workflowRunner } from './workflows/workflow-runner.js';
+export { workflowStore } from './workflows/workflow-store.js';
+export { workspaceManager } from './workspace/workspace-manager.js';
+export type {
+  AgentCard,
+  Artifact,
+  ArtifactType,
+  BusEvent,
+  Message,
+  Part,
+  SubmitTaskInput,
+  Task,
+  TaskSummary,
+  TaskError,
+  TaskState,
+} from './a2a/types.js';
+export type {
+  PortEvent,
+  PreviewArtifactData,
+} from './preview/types.js';
+export type {
+  PermissionApprovalRequest,
+  PermissionCapability,
+  PermissionDecision,
+  PermissionPolicy,
+  PermissionPolicyContext,
+  PermissionPolicyEvent,
+  PermissionPolicyMode,
+} from './security/permission-policy.js';
+export type {
+  CreateOrchestrationTaskInput,
+  DispatchOrchestrationTaskInput,
+  OrchestrationTask,
+  OrchestrationTaskState,
+} from './tasks/orchestration-task.types.js';
+export type {
+  WorkflowTemplate,
+  WorkflowTemplateAgentSlot,
+} from './workflows/workflow-templates.js';
+export type {
+  Workflow,
+  WorkflowNode,
+  WorkflowNodeRun,
+  WorkflowNodeStatus,
+  WorkflowRun,
+  WorkflowRunStatus,
+  WorkflowTraceEvent,
+} from './workflows/workflow.types.js';
+export type {
+  ExecResult,
+  WorkspaceHandle,
+  WorkspaceKind,
+  WorkspaceMetadata,
+  WorkspaceRequest,
+} from './workspace/types.js';
+export { WorkspaceError } from './workspace/types.js';

package/server/modules/orchestration/preview/port-watcher.ts

@@ -1,112 +1,112 @@
-import { execFile } from 'node:child_process';
-import { promisify } from 'node:util';
-
-import type { PortEvent } from '@/modules/orchestration/preview/types.js';
-import type { WorkspaceHandle } from '@/modules/orchestration/workspace/types.js';
-
-const execFileAsync = promisify(execFile);
-
-interface WatchOptions {
-  taskId: string;
-  workspace: WorkspaceHandle;
-  intervalMs?: number;
-  onPort(event: PortEvent): void;
-}
-
-interface ListeningPort {
-  port: number;
-  host: string;
-  processName?: string;
-}
-
-const knownPreviewPorts = new Map<number, PortEvent>();
-
-function parseAddress(address: string): { host: string; port: number } | null {
-  const normalized = address.trim();
-  const idx = normalized.lastIndexOf(':');
-  if (idx === -1) return null;
-  const port = Number.parseInt(normalized.slice(idx + 1), 10);
-  if (!Number.isFinite(port) || port <= 0) return null;
-  const host = normalized.slice(0, idx).replace(/^\[|\]$/g, '') || '127.0.0.1';
-  return { host: host === '*' || host === '0.0.0.0' ? '127.0.0.1' : host, port };
-}
-
-function parseSs(output: string): ListeningPort[] {
-  const ports: ListeningPort[] = [];
-  for (const line of output.split('\n')) {
-    if (!line.includes('LISTEN')) continue;
-    const parts = line.trim().split(/\s+/);
-    const address = parts[3] ?? parts[4];
-    const parsed = address ? parseAddress(address) : null;
-    if (!parsed) continue;
-    const processMatch = line.match(/users:\(\("([^"]+)"/);
-    ports.push({
-      ...parsed,
-      processName: processMatch?.[1],
-    });
-  }
-  return ports;
-}
-
-async function readListeningPorts(): Promise<ListeningPort[]> {
-  try {
-    const { stdout } = await execFileAsync('ss', ['-ltnp'], {
-      maxBuffer: 1024 * 1024,
-    });
-    return parseSs(String(stdout));
-  } catch {
-    return [];
-  }
-}
-
-export function getKnownPreviewPort(port: number): PortEvent | undefined {
-  return knownPreviewPorts.get(port);
-}
-
-export class PortWatcher {
-  watch(options: WatchOptions): () => void {
-    const {
-      taskId,
-      workspace,
-      intervalMs = 1000,
-      onPort,
-    } = options;
-    const seen = new Set<number>();
-    let stopped = false;
-    let timer: NodeJS.Timeout | undefined;
-    let initialized = false;
-
-    const tick = async () => {
-      if (stopped) return;
-      const ports = await readListeningPorts();
-      for (const port of ports) {
-        if (seen.has(port.port)) continue;
-        seen.add(port.port);
-        if (!initialized) continue;
-        const event: PortEvent = {
-          taskId,
-          workspaceId: workspace.id,
-          port: port.port,
-          host: port.host,
-          url: `http://${port.host}:${port.port}`,
-          processName: port.processName,
-          confidence: 'low',
-          detectedAt: Date.now(),
-        };
-        knownPreviewPorts.set(port.port, event);
-        onPort(event);
-      }
-      initialized = true;
-      timer = setTimeout(tick, intervalMs);
-    };
-
-    void tick();
-
-    return () => {
-      stopped = true;
-      if (timer) clearTimeout(timer);
-    };
-  }
-}
-
-export const portWatcher = new PortWatcher();
+import { execFile } from 'node:child_process';
+import { promisify } from 'node:util';
+
+import type { PortEvent } from '@/modules/orchestration/preview/types.js';
+import type { WorkspaceHandle } from '@/modules/orchestration/workspace/types.js';
+
+const execFileAsync = promisify(execFile);
+
+interface WatchOptions {
+  taskId: string;
+  workspace: WorkspaceHandle;
+  intervalMs?: number;
+  onPort(event: PortEvent): void;
+}
+
+interface ListeningPort {
+  port: number;
+  host: string;
+  processName?: string;
+}
+
+const knownPreviewPorts = new Map<number, PortEvent>();
+
+function parseAddress(address: string): { host: string; port: number } | null {
+  const normalized = address.trim();
+  const idx = normalized.lastIndexOf(':');
+  if (idx === -1) return null;
+  const port = Number.parseInt(normalized.slice(idx + 1), 10);
+  if (!Number.isFinite(port) || port <= 0) return null;
+  const host = normalized.slice(0, idx).replace(/^\[|\]$/g, '') || '127.0.0.1';
+  return { host: host === '*' || host === '0.0.0.0' ? '127.0.0.1' : host, port };
+}
+
+function parseSs(output: string): ListeningPort[] {
+  const ports: ListeningPort[] = [];
+  for (const line of output.split('\n')) {
+    if (!line.includes('LISTEN')) continue;
+    const parts = line.trim().split(/\s+/);
+    const address = parts[3] ?? parts[4];
+    const parsed = address ? parseAddress(address) : null;
+    if (!parsed) continue;
+    const processMatch = line.match(/users:\(\("([^"]+)"/);
+    ports.push({
+      ...parsed,
+      processName: processMatch?.[1],
+    });
+  }
+  return ports;
+}
+
+async function readListeningPorts(): Promise<ListeningPort[]> {
+  try {
+    const { stdout } = await execFileAsync('ss', ['-ltnp'], {
+      maxBuffer: 1024 * 1024,
+    });
+    return parseSs(String(stdout));
+  } catch {
+    return [];
+  }
+}
+
+export function getKnownPreviewPort(port: number): PortEvent | undefined {
+  return knownPreviewPorts.get(port);
+}
+
+export class PortWatcher {
+  watch(options: WatchOptions): () => void {
+    const {
+      taskId,
+      workspace,
+      intervalMs = 1000,
+      onPort,
+    } = options;
+    const seen = new Set<number>();
+    let stopped = false;
+    let timer: NodeJS.Timeout | undefined;
+    let initialized = false;
+
+    const tick = async () => {
+      if (stopped) return;
+      const ports = await readListeningPorts();
+      for (const port of ports) {
+        if (seen.has(port.port)) continue;
+        seen.add(port.port);
+        if (!initialized) continue;
+        const event: PortEvent = {
+          taskId,
+          workspaceId: workspace.id,
+          port: port.port,
+          host: port.host,
+          url: `http://${port.host}:${port.port}`,
+          processName: port.processName,
+          confidence: 'low',
+          detectedAt: Date.now(),
+        };
+        knownPreviewPorts.set(port.port, event);
+        onPort(event);
+      }
+      initialized = true;
+      timer = setTimeout(tick, intervalMs);
+    };
+
+    void tick();
+
+    return () => {
+      stopped = true;
+      if (timer) clearTimeout(timer);
+    };
+  }
+}
+
+export const portWatcher = new PortWatcher();