claude-code-session-manager 0.61.0 → 0.62.1

package/src/main/lib/__tests__/browserAgentActions.test.cjs

@@ -1,76 +0,0 @@
-/**
- * browserAgentActions.test.cjs — unit tests for the pure action-mapping
- * logic behind the on-demand browser-agent API (PRD 535).
- *
- * Run: timeout 120 node --test src/main/lib/__tests__/browserAgentActions.test.cjs
- */
-
-'use strict';
-
-const { test } = require('node:test');
-const assert = require('node:assert/strict');
-const {
-  parseKeyCombo,
-  computeScrollDelta,
-  computeScreenshotScale,
-  AGENT_SCREENSHOT_MAX_DIM,
-} = require('../browserAgentActions.cjs');
-
-test('parseKeyCombo: plain special key', () => {
-  assert.deepEqual(parseKeyCombo('Return'), { keyCode: 'Enter', modifiers: [] });
-});
-
-test('parseKeyCombo: single modifier + letter', () => {
-  assert.deepEqual(parseKeyCombo('ctrl+s'), { keyCode: 's', modifiers: ['control'] });
-});
-
-test('parseKeyCombo: multiple modifiers', () => {
-  assert.deepEqual(parseKeyCombo('ctrl+shift+p'), { keyCode: 'p', modifiers: ['control', 'shift'] });
-});
-
-test('parseKeyCombo: cmd/meta/option aliases normalize', () => {
-  assert.deepEqual(parseKeyCombo('cmd+option+Escape'), { keyCode: 'Escape', modifiers: ['meta', 'alt'] });
-});
-
-test('parseKeyCombo: unknown modifier throws', () => {
-  assert.throws(() => parseKeyCombo('foo+a'), /unknown modifier/);
-});
-
-test('parseKeyCombo: empty string throws', () => {
-  assert.throws(() => parseKeyCombo(''), /empty key/);
-});
-
-test('computeScrollDelta: up/down/left/right map to signed deltas', () => {
-  assert.deepEqual(computeScrollDelta({ scroll_direction: 'up', scroll_amount: 2 }), { deltaX: 0, deltaY: 200 });
-  assert.deepEqual(computeScrollDelta({ scroll_direction: 'down', scroll_amount: 2 }), { deltaX: 0, deltaY: -200 });
-  assert.deepEqual(computeScrollDelta({ scroll_direction: 'left', scroll_amount: 1 }), { deltaX: 100, deltaY: 0 });
-  assert.deepEqual(computeScrollDelta({ scroll_direction: 'right', scroll_amount: 1 }), { deltaX: -100, deltaY: 0 });
-});
-
-test('computeScrollDelta: defaults scroll_amount to 1 when not finite', () => {
-  assert.deepEqual(computeScrollDelta({ scroll_direction: 'up' }), { deltaX: 0, deltaY: 100 });
-});
-
-test('computeScrollDelta: unsupported direction throws', () => {
-  assert.throws(() => computeScrollDelta({ scroll_direction: 'diagonal' }), /unsupported scroll_direction/);
-});
-
-test('computeScreenshotScale: leaves small images untouched', () => {
-  const result = computeScreenshotScale(800, 600);
-  assert.equal(result.scale, 1);
-  assert.equal(result.width, 800);
-  assert.equal(result.height, 600);
-});
-
-test('computeScreenshotScale: downscales long edge to the fixed max', () => {
-  const result = computeScreenshotScale(2560, 1440);
-  assert.equal(result.width, AGENT_SCREENSHOT_MAX_DIM);
-  assert.ok(result.scale < 1);
-  assert.equal(result.height, Math.round(1440 * result.scale));
-});
-
-test('computeScreenshotScale: portrait image scales on height', () => {
-  const result = computeScreenshotScale(900, 3000);
-  assert.equal(result.height, AGENT_SCREENSHOT_MAX_DIM);
-  assert.ok(result.width < 900);
-});

package/src/main/lib/browserAgentActions.cjs

@@ -1,114 +0,0 @@
-/**
- * browserAgentActions.cjs — pure mapping logic for the on-demand single-
- * action browser-agent API (PRD 535). Extracted out of browserView.cjs so
- * the action -> input-event/scale mapping can be unit-tested without
- * booting Electron — mirrors how lib/schedulerBatch.cjs was pulled out of
- * scheduler.cjs for the same reason. Anything that actually touches a live
- * WebContentsView (sendInputEvent, capturePage, executeJavaScript) stays in
- * browserView.cjs; this module only computes the values passed to those
- * calls.
- */
-
-'use strict';
-
-// Anthropic's computer_use tool scrolls in "clicks"; one click here maps to
-// this many CSS pixels of wheel delta.
-const SCROLL_UNIT_PX = 100;
-
-// Anthropic-tool-use-style key names -> Electron's sendInputEvent keyCode
-// strings. Anything not in this table is passed through as-is (covers plain
-// single characters like "a" or "s").
-const SPECIAL_KEY_MAP = {
-  return: 'Enter',
-  enter: 'Enter',
-  tab: 'Tab',
-  escape: 'Escape',
-  esc: 'Escape',
-  backspace: 'Backspace',
-  delete: 'Delete',
-  space: ' ',
-  up: 'Up',
-  down: 'Down',
-  left: 'Left',
-  right: 'Right',
-  page_up: 'PageUp',
-  pageup: 'PageUp',
-  page_down: 'PageDown',
-  pagedown: 'PageDown',
-  home: 'Home',
-  end: 'End',
-};
-
-const MODIFIER_MAP = {
-  ctrl: 'control',
-  control: 'control',
-  shift: 'shift',
-  alt: 'alt',
-  option: 'alt',
-  cmd: 'meta',
-  meta: 'meta',
-  super: 'meta',
-};
-
-// Parses a computer-use-style key combo ("ctrl+s", "Return", "a") into the
-// { keyCode, modifiers } shape Electron's webContents.sendInputEvent wants.
-function parseKeyCombo(keyStr) {
-  const parts = String(keyStr || '')
-    .split('+')
-    .map((p) => p.trim())
-    .filter(Boolean);
-  if (!parts.length) throw new Error('empty key');
-  const last = parts.pop();
-  const modifiers = parts.map((p) => {
-    const mapped = MODIFIER_MAP[p.toLowerCase()];
-    if (!mapped) throw new Error(`unknown modifier: ${p}`);
-    return mapped;
-  });
-  const lowerLast = last.toLowerCase();
-  const keyCode = SPECIAL_KEY_MAP[lowerLast] || last;
-  return { keyCode, modifiers };
-}
-
-// Maps a computer-use-style scroll action to a mouseWheel delta.
-function computeScrollDelta({ scroll_direction, scroll_amount }) {
-  const amount = Number.isFinite(scroll_amount) ? scroll_amount : 1;
-  switch (scroll_direction) {
-    case 'up':
-      return { deltaX: 0, deltaY: amount * SCROLL_UNIT_PX };
-    case 'down':
-      return { deltaX: 0, deltaY: -amount * SCROLL_UNIT_PX };
-    case 'left':
-      return { deltaX: amount * SCROLL_UNIT_PX, deltaY: 0 };
-    case 'right':
-      return { deltaX: -amount * SCROLL_UNIT_PX, deltaY: 0 };
-    default:
-      throw new Error(`unsupported scroll_direction: ${scroll_direction}`);
-  }
-}
-
-// Fixed max long-edge a screenshot is resized to before being returned.
-// Anthropic's computer-use API silently downscales oversized images
-// server-side, which breaks coordinate mapping unless caller and model agree
-// on one fixed logical resolution up front — this is that fixed point.
-const AGENT_SCREENSHOT_MAX_DIM = 1280;
-
-// Given a captured screenshot's real pixel size, returns the target size to
-// resize to plus the scale factor a caller must divide a clicked coordinate
-// on the resized image by to recover the real-page coordinate.
-function computeScreenshotScale(width, height) {
-  const longEdge = Math.max(width, height);
-  const scale = longEdge > AGENT_SCREENSHOT_MAX_DIM ? AGENT_SCREENSHOT_MAX_DIM / longEdge : 1;
-  return {
-    scale,
-    width: Math.max(1, Math.round(width * scale)),
-    height: Math.max(1, Math.round(height * scale)),
-  };
-}
-
-module.exports = {
-  parseKeyCombo,
-  computeScrollDelta,
-  computeScreenshotScale,
-  AGENT_SCREENSHOT_MAX_DIM,
-  SCROLL_UNIT_PX,
-};

package/src/main/lib/e2eStateMachine.cjs

@@ -1,39 +0,0 @@
-/**
- * Pure E2E session state machine for the web-remote relay.
- * No Electron, no I/O — importable in unit tests.
- *
- * State transitions:
- *   idle        → pending_sas   : successful deriveSessionKey + deriveSas
- *   idle        → failed        : crypto derivation error
- *   pending_sas → authenticated : user confirms SAS
- *   pending_sas → failed        : deriveSas threw after sessionKey succeeded
- *   any         → idle          : disconnect / reset
- */
-
-/** @returns {{ state: string, sessionKey: Buffer|null, pendingSas: string|null }} */
-function makeState(state = 'idle', sessionKey = null, pendingSas = null) {
-  return { state, sessionKey, pendingSas };
-}
-
-/**
- * Attempt to confirm the SAS.  Pure — does not mutate; returns the next state.
- * @param {{ state: string, sessionKey: Buffer|null, pendingSas: string|null }} e2eState
- * @returns {{ ok: boolean, error?: string, next: { state: string, sessionKey: Buffer|null, pendingSas: string|null } }}
- */
-function confirmSas(e2eState) {
-  if (e2eState.state !== 'pending_sas') {
-    const errorMap = {
-      idle: 'no_e2e_session',
-      failed: 'e2e_failed',
-      authenticated: 'already_authenticated',
-    };
-    const error = errorMap[e2eState.state] ?? 'unexpected_state';
-    return { ok: false, error, next: e2eState };
-  }
-  return {
-    ok: true,
-    next: makeState('authenticated', e2eState.sessionKey, null),
-  };
-}
-
-module.exports = { makeState, confirmSas };