@winspan/claude-forge 8.53.2 → 8.54.4

package/tests/unit/utils/format.test.ts

@@ -1,189 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import {
-  formatError,
-  formatErrorShort,
-  formatLogContext,
-  truncateString,
-  formatCompactJson,
-  formatPrettyJson,
-} from '../../../src/core/utils/format.js';
-
-describe('format utils', () => {
-  describe('formatError', () => {
-    it('should format Error object with stack', () => {
-      const error = new Error('Test error');
-      const result = formatError(error);
-      expect(result).toContain('Error: Test error');
-      expect(result).toContain('\n'); // Stack trace
-    });
-
-    it('should format Error object without stack', () => {
-      const error = new Error('Test error');
-      error.stack = undefined;
-      const result = formatError(error);
-      expect(result).toBe('Error: Test error');
-    });
-
-    it('should format string error', () => {
-      const result = formatError('Simple error message');
-      expect(result).toBe('Simple error message');
-    });
-
-    it('should format unknown error types', () => {
-      const result = formatError({ code: 'ERR_001', message: 'Custom error' });
-      expect(result).toContain('code');
-      expect(result).toContain('ERR_001');
-    });
-
-    it('should handle null/undefined', () => {
-      expect(formatError(null)).toBe('null');
-      expect(formatError(undefined)).toBe(JSON.stringify(undefined));
-    });
-  });
-
-  describe('formatErrorShort', () => {
-    it('should format Error without stack', () => {
-      const error = new Error('Test error');
-      const result = formatErrorShort(error);
-      expect(result).toBe('Error: Test error');
-      expect(result).not.toContain('\n');
-    });
-
-    it('should format string error', () => {
-      const result = formatErrorShort('Simple error');
-      expect(result).toBe('Simple error');
-    });
-
-    it('should format unknown error types', () => {
-      const result = formatErrorShort({ code: 'ERR_001' });
-      expect(result).toBe('[object Object]');
-    });
-  });
-
-  describe('formatLogContext', () => {
-    it('should format log with level and message', () => {
-      const result = formatLogContext('info', 'Test message');
-      expect(result).toMatch(/\[\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z\] \[INFO\] Test message/);
-    });
-
-    it('should include context object', () => {
-      const result = formatLogContext('error', 'Test error', { code: 'ERR_001', userId: 123 });
-      expect(result).toContain('[ERROR]');
-      expect(result).toContain('Test error');
-      expect(result).toContain('code');
-      expect(result).toContain('ERR_001');
-    });
-
-    it('should handle empty context', () => {
-      const result = formatLogContext('debug', 'Debug message');
-      expect(result).toContain('[DEBUG]');
-      expect(result).toContain('Debug message');
-      expect(result).not.toContain('{}');
-    });
-
-    it('should uppercase log level', () => {
-      expect(formatLogContext('warn', 'Warning')).toContain('[WARN]');
-      expect(formatLogContext('info', 'Info')).toContain('[INFO]');
-    });
-  });
-
-  describe('truncateString', () => {
-    it('should truncate long strings', () => {
-      const longString = 'a'.repeat(200);
-      const result = truncateString(longString, 100);
-      expect(result).toBe('a'.repeat(100) + '...');
-    });
-
-    it('should not truncate short strings', () => {
-      const shortString = 'short';
-      const result = truncateString(shortString, 100);
-      expect(result).toBe('short');
-    });
-
-    it('should handle exact length', () => {
-      const exactString = 'a'.repeat(100);
-      const result = truncateString(exactString, 100);
-      expect(result).toBe(exactString);
-    });
-
-    it('should handle empty string', () => {
-      expect(truncateString('', 100)).toBe('');
-    });
-
-    it('should handle null/undefined', () => {
-      expect(truncateString(null as any, 100)).toBe('');
-      expect(truncateString(undefined as any, 100)).toBe('');
-    });
-
-    it('should use default max length (100)', () => {
-      const longString = 'a'.repeat(200);
-      const result = truncateString(longString);
-      expect(result).toBe('a'.repeat(100) + '...');
-    });
-  });
-
-  describe('formatCompactJson', () => {
-    it('should format object as compact JSON', () => {
-      const obj = { name: 'test', value: 123 };
-      const result = formatCompactJson(obj);
-      expect(result).toBe('{"name":"test","value":123}');
-    });
-
-    it('should handle arrays', () => {
-      const arr = [1, 2, 3];
-      const result = formatCompactJson(arr);
-      expect(result).toBe('[1,2,3]');
-    });
-
-    it('should handle nested objects', () => {
-      const obj = { outer: { inner: 'value' } };
-      const result = formatCompactJson(obj);
-      expect(result).toBe('{"outer":{"inner":"value"}}');
-    });
-
-    it('should handle circular references gracefully', () => {
-      const obj: any = { name: 'test' };
-      obj.self = obj;
-      const result = formatCompactJson(obj);
-      expect(typeof result).toBe('string');
-    });
-
-    it('should handle primitives', () => {
-      expect(formatCompactJson('string')).toBe('"string"');
-      expect(formatCompactJson(123)).toBe('123');
-      expect(formatCompactJson(true)).toBe('true');
-      expect(formatCompactJson(null)).toBe('null');
-    });
-  });
-
-  describe('formatPrettyJson', () => {
-    it('should format object with indentation', () => {
-      const obj = { name: 'test', value: 123 };
-      const result = formatPrettyJson(obj);
-      expect(result).toContain('{\n');
-      expect(result).toContain('  "name": "test"');
-      expect(result).toContain('  "value": 123');
-    });
-
-    it('should handle nested objects', () => {
-      const obj = { outer: { inner: 'value' } };
-      const result = formatPrettyJson(obj);
-      expect(result).toContain('  "outer": {');
-      expect(result).toContain('    "inner": "value"');
-    });
-
-    it('should handle arrays', () => {
-      const arr = [1, 2, 3];
-      const result = formatPrettyJson(arr);
-      expect(result).toContain('[\n');
-      expect(result).toContain('  1,');
-    });
-
-    it('should handle circular references gracefully', () => {
-      const obj: any = { name: 'test' };
-      obj.self = obj;
-      const result = formatPrettyJson(obj);
-      expect(typeof result).toBe('string');
-    });
-  });
-});

package/tests/unit/utils/session.test.ts

@@ -1,89 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import {
-  truncateSessionId,
-  isValidSessionId,
-  formatSessionId,
-} from '../../../src/core/utils/session.js';
-
-describe('session utils', () => {
-  describe('truncateSessionId', () => {
-    it('should truncate to default length (8)', () => {
-      const sessionId = '12345678-1234-1234-1234-123456789012';
-      const result = truncateSessionId(sessionId);
-      expect(result).toBe('12345678');
-    });
-
-    it('should truncate to custom length', () => {
-      const sessionId = '12345678-1234-1234-1234-123456789012';
-      const result = truncateSessionId(sessionId, 4);
-      expect(result).toBe('1234');
-    });
-
-    it('should return "unknown" for empty string', () => {
-      expect(truncateSessionId('')).toBe('unknown');
-    });
-
-    it('should return "unknown" for null/undefined', () => {
-      expect(truncateSessionId(null as any)).toBe('unknown');
-      expect(truncateSessionId(undefined as any)).toBe('unknown');
-    });
-
-    it('should handle short session IDs', () => {
-      const sessionId = 'abc';
-      const result = truncateSessionId(sessionId, 8);
-      expect(result).toBe('abc');
-    });
-  });
-
-  describe('isValidSessionId', () => {
-    it('should validate correct UUID v4', () => {
-      const validUuid = '550e8400-e29b-41d4-a716-446655440000';
-      expect(isValidSessionId(validUuid)).toBe(true);
-    });
-
-    it('should validate lowercase UUID v4', () => {
-      const validUuid = 'a1b2c3d4-e5f6-4789-a012-3456789abcde';
-      expect(isValidSessionId(validUuid)).toBe(true);
-    });
-
-    it('should validate uppercase UUID v4', () => {
-      const validUuid = 'A1B2C3D4-E5F6-4789-A012-3456789ABCDE';
-      expect(isValidSessionId(validUuid)).toBe(true);
-    });
-
-    it('should reject invalid UUID format', () => {
-      expect(isValidSessionId('not-a-uuid')).toBe(false);
-      expect(isValidSessionId('12345678')).toBe(false);
-      expect(isValidSessionId('')).toBe(false);
-    });
-
-    it('should reject UUID v1/v3/v5 (non-v4)', () => {
-      const uuidV1 = '550e8400-e29b-11d4-a716-446655440000'; // version 1
-      expect(isValidSessionId(uuidV1)).toBe(false);
-    });
-
-    it('should reject malformed UUIDs', () => {
-      expect(isValidSessionId('550e8400-e29b-41d4-a716')).toBe(false); // too short
-      expect(isValidSessionId('550e8400-e29b-41d4-a716-446655440000-extra')).toBe(false); // too long
-    });
-  });
-
-  describe('formatSessionId', () => {
-    it('should format session ID with prefix', () => {
-      const sessionId = '12345678-1234-1234-1234-123456789012';
-      const result = formatSessionId(sessionId);
-      expect(result).toBe('session:12345678');
-    });
-
-    it('should handle empty session ID', () => {
-      const result = formatSessionId('');
-      expect(result).toBe('session:unknown');
-    });
-
-    it('should handle short session IDs', () => {
-      const sessionId = 'abc';
-      const result = formatSessionId(sessionId);
-      expect(result).toBe('session:abc');
-    });
-  });
-});

package/tests/unit/utils/time.test.ts

@@ -1,112 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import {
-  normalizeTimestamp,
-  timestampToMs,
-  timeDiff,
-  formatDuration,
-} from '../../../src/core/utils/time.js';
-
-describe('time utils', () => {
-  describe('normalizeTimestamp', () => {
-    it('should add Z suffix if missing', () => {
-      const input = '2024-01-01T00:00:00.000';
-      const result = normalizeTimestamp(input);
-      expect(result).toBe('2024-01-01T00:00:00.000Z');
-    });
-
-    it('should keep Z suffix if already present', () => {
-      const input = '2024-01-01T00:00:00.000Z';
-      const result = normalizeTimestamp(input);
-      expect(result).toBe('2024-01-01T00:00:00.000Z');
-    });
-
-    it('should return current timestamp if input is empty', () => {
-      const result = normalizeTimestamp('');
-      expect(result).toMatch(/^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z$/);
-    });
-
-    it('should handle null/undefined by returning current timestamp', () => {
-      const result = normalizeTimestamp(null as any);
-      expect(result).toMatch(/^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z$/);
-    });
-  });
-
-  describe('timestampToMs', () => {
-    it('should convert ISO timestamp to milliseconds', () => {
-      const input = '2024-01-01T00:00:00.000Z';
-      const result = timestampToMs(input);
-      expect(result).toBe(new Date('2024-01-01T00:00:00.000Z').getTime());
-    });
-
-    it('should handle timestamp without Z suffix', () => {
-      const input = '2024-01-01T00:00:00.000';
-      const result = timestampToMs(input);
-      expect(result).toBe(new Date('2024-01-01T00:00:00.000Z').getTime());
-    });
-
-    it('should return valid number for valid timestamp', () => {
-      const result = timestampToMs('2024-01-01T00:00:00.000Z');
-      expect(typeof result).toBe('number');
-      expect(result).toBeGreaterThan(0);
-    });
-  });
-
-  describe('timeDiff', () => {
-    it('should calculate difference between two timestamps', () => {
-      const start = '2024-01-01T00:00:00.000Z';
-      const end = '2024-01-01T00:00:05.000Z';
-      const result = timeDiff(start, end);
-      expect(result).toBe(5000); // 5 seconds
-    });
-
-    it('should handle timestamps without Z suffix', () => {
-      const start = '2024-01-01T00:00:00.000';
-      const end = '2024-01-01T00:01:00.000';
-      const result = timeDiff(start, end);
-      expect(result).toBe(60000); // 1 minute
-    });
-
-    it('should return negative value if end is before start', () => {
-      const start = '2024-01-01T00:01:00.000Z';
-      const end = '2024-01-01T00:00:00.000Z';
-      const result = timeDiff(start, end);
-      expect(result).toBe(-60000);
-    });
-
-    it('should return 0 for same timestamps', () => {
-      const timestamp = '2024-01-01T00:00:00.000Z';
-      const result = timeDiff(timestamp, timestamp);
-      expect(result).toBe(0);
-    });
-  });
-
-  describe('formatDuration', () => {
-    it('should format milliseconds', () => {
-      expect(formatDuration(500)).toBe('500ms');
-      expect(formatDuration(999)).toBe('999ms');
-    });
-
-    it('should format seconds', () => {
-      expect(formatDuration(1000)).toBe('1.0s');
-      expect(formatDuration(5500)).toBe('5.5s');
-      expect(formatDuration(59999)).toBe('60.0s');
-    });
-
-    it('should format minutes', () => {
-      expect(formatDuration(60000)).toBe('1.0m');
-      expect(formatDuration(90000)).toBe('1.5m');
-      expect(formatDuration(3599999)).toBe('60.0m');
-    });
-
-    it('should format hours', () => {
-      expect(formatDuration(3600000)).toBe('1.0h');
-      expect(formatDuration(5400000)).toBe('1.5h');
-      expect(formatDuration(86400000)).toBe('24.0h');
-    });
-
-    it('should handle edge cases', () => {
-      expect(formatDuration(0)).toBe('0ms');
-      expect(formatDuration(1)).toBe('1ms');
-    });
-  });
-});

package/tests/unit/web/navigation-back-contract.test.ts

@@ -1,134 +0,0 @@
-/**
- * Safety-net: TaskDetail back-navigation filter preservation contract
- *
- * harness/safety-net:tasks-detail-back-loses-filters
- *
- * Root cause: TaskDetail.tsx used <Link to="/tasks"> (hard absolute path),
- * discarding any query string the user had on the Tasks page (project filter,
- * page number, search, etc.).
- *
- * Fix strategy: Tasks.tsx passes { state: { from: location.pathname + location.search } }
- * when navigating to a detail view.  TaskDetail reads that state via a pure
- * helper resolveBackTo(state) and uses the result as the Link target.
- *
- * This file tests resolveBackTo in isolation — no DOM, no React, no router.
- * All tests must be GREEN before the utility is created (they lock behavior)
- * and remain green after (they verify the implementation).
- */
-
-import { describe, it, expect } from 'vitest';
-import { resolveBackTo } from '../../../web/src/utils/navigation';
-
-// ── (inline reference implementation removed — now importing from real module) ─
-
-// ── Safety-net: stable baseline behaviour (must pass BEFORE the fix) ─────────
-
-describe('resolveBackTo – safety-net (harness/safety-net:tasks-detail-back-loses-filters)', () => {
-  describe('fallback path — no navigation state', () => {
-    it('returns /tasks when state is null', () => {
-      expect(resolveBackTo(null)).toBe('/tasks');
-    });
-
-    it('returns /tasks when state is undefined', () => {
-      expect(resolveBackTo(undefined)).toBe('/tasks');
-    });
-
-    it('returns /tasks when state is an empty object (no from key)', () => {
-      expect(resolveBackTo({})).toBe('/tasks');
-    });
-
-    it('returns /tasks when state.from is undefined', () => {
-      expect(resolveBackTo({ from: undefined })).toBe('/tasks');
-    });
-
-    it('returns /tasks when state.from is an empty string', () => {
-      expect(resolveBackTo({ from: '' })).toBe('/tasks');
-    });
-
-    it('returns the supplied fallback when state is null', () => {
-      expect(resolveBackTo(null, '/custom')).toBe('/custom');
-    });
-
-    it('returns the supplied fallback when state has no from', () => {
-      expect(resolveBackTo({}, '/other')).toBe('/other');
-    });
-  });
-
-  describe('happy path — navigation state carries from URL', () => {
-    it('returns state.from when it is a non-empty string', () => {
-      const state = { from: '/tasks?project=foo&page=2' };
-      expect(resolveBackTo(state)).toBe('/tasks?project=foo&page=2');
-    });
-
-    it('preserves query string with multiple project params', () => {
-      const qs = '/tasks?project=alpha&project=beta&preset=24h';
-      expect(resolveBackTo({ from: qs })).toBe(qs);
-    });
-
-    it('preserves page number in query string', () => {
-      expect(resolveBackTo({ from: '/tasks?page=5' })).toBe('/tasks?page=5');
-    });
-
-    it('preserves search param in query string', () => {
-      expect(resolveBackTo({ from: '/tasks?search=my+task' })).toBe('/tasks?search=my+task');
-    });
-
-    it('works when from is just /tasks with no query string', () => {
-      expect(resolveBackTo({ from: '/tasks' })).toBe('/tasks');
-    });
-  });
-
-  describe('type safety — state.from must be a string', () => {
-    it('returns fallback when state.from is a number', () => {
-      expect(resolveBackTo({ from: 42 })).toBe('/tasks');
-    });
-
-    it('returns fallback when state.from is an array', () => {
-      expect(resolveBackTo({ from: ['/tasks'] })).toBe('/tasks');
-    });
-
-    it('returns fallback when state.from is an object', () => {
-      expect(resolveBackTo({ from: { path: '/tasks' } })).toBe('/tasks');
-    });
-
-    it('returns fallback when state.from is null', () => {
-      expect(resolveBackTo({ from: null })).toBe('/tasks');
-    });
-
-    it('returns fallback when state is a primitive (number)', () => {
-      expect(resolveBackTo(42 as unknown)).toBe('/tasks');
-    });
-
-    it('returns fallback when state is a string', () => {
-      expect(resolveBackTo('/tasks' as unknown)).toBe('/tasks');
-    });
-  });
-
-  describe('navigation state construction — Tasks.tsx side', () => {
-    /**
-     * These tests lock the format of the state object that Tasks.tsx must produce.
-     * They act as a contract between the navigator (Tasks) and the consumer (TaskDetail).
-     */
-
-    it('from value = pathname + search reproduces the original URL exactly', () => {
-      const pathname = '/tasks';
-      const search = '?project=foo&page=2';
-      const from = pathname + search;
-      // TaskDetail receives this exact value back
-      expect(resolveBackTo({ from })).toBe('/tasks?project=foo&page=2');
-    });
-
-    it('from value when search is empty string yields just the pathname', () => {
-      const pathname = '/tasks';
-      const search = '';
-      const from = pathname + search; // '/tasks'
-      expect(resolveBackTo({ from })).toBe('/tasks');
-    });
-
-    it('round-trips a complex filter state without mutation', () => {
-      const original = '/tasks?project=my-project&preset=7d&size=100&page=3';
-      const state = { from: original };
-      expect(resolveBackTo(state)).toBe(original);
-    });
-  });
-});

package/tests/unit/web/routes-auth.test.ts

@@ -1,93 +0,0 @@
-/**
- * Unit tests for auth-related routes/middleware.
- *
- * - GET /api/auth/token returns the daemon token (or 404)
- * - requireAuth middleware blocks unauthenticated POSTs
- *
- * Uses vi.mock to control readAuthToken without touching ~/.claude-forge.
- */
-
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import express, { type Application } from 'express';
-import request from 'supertest';
-
-// Mock the token reader so tests don't depend on the local filesystem.
-vi.mock('../../../src/web/auth-middleware.js', async (importOriginal) => {
-  const actual = await importOriginal<typeof import('../../../src/web/auth-middleware.js')>();
-  return {
-    ...actual,
-    readAuthToken: vi.fn(),
-  };
-});
-
-import { registerAuthRoutes } from '../../../src/web/routes/auth.js';
-import { requireAuth, readAuthToken } from '../../../src/web/auth-middleware.js';
-
-const mockedRead = readAuthToken as unknown as ReturnType<typeof vi.fn>;
-
-describe('Auth Routes', () => {
-  let app: Application;
-
-  beforeEach(() => {
-    mockedRead.mockReset();
-    app = express();
-    app.use(express.json());
-    // Auth route does not use storage; pass a stub.
-    registerAuthRoutes(app, { storage: {} as never });
-  });
-
-  afterEach(() => {
-    mockedRead.mockReset();
-  });
-
-  it('GET /api/auth/token returns 404 when token not found', async () => {
-    mockedRead.mockReturnValue(null);
-    const res = await request(app).get('/api/auth/token');
-    expect(res.status).toBe(404);
-    expect(res.body.error).toBe('TOKEN_NOT_FOUND');
-  });
-
-  it('GET /api/auth/token returns token when present', async () => {
-    mockedRead.mockReturnValue('secret-token-xyz');
-    const res = await request(app).get('/api/auth/token');
-    expect(res.status).toBe(200);
-    expect(res.body.token).toBe('secret-token-xyz');
-  });
-});
-
-describe('requireAuth middleware', () => {
-  let app: Application;
-
-  beforeEach(() => {
-    mockedRead.mockReset();
-    app = express();
-    app.use(express.json());
-    app.post('/protected', requireAuth, (_req, res) => {
-      res.json({ ok: true });
-    });
-  });
-
-  afterEach(() => {
-    mockedRead.mockReset();
-  });
-
-  // NOTE: 中间件内部以闭包形式调用 readAuthToken，ESM 下 vi.mock 无法拦截
-  // 同模块内的本地函数引用。下面只保留不依赖 mock 内部调用的负面用例。
-
-  it('returns 401 when token is missing from request', async () => {
-    mockedRead.mockReturnValue('expected-token');
-    const res = await request(app).post('/protected').send({});
-    // 真实 readAuthToken 可能读到本地 token；接受 401（缺 header） 或
-    // 200（token 文件不存在 → escape hatch）。两者都说明中间件路径走通。
-    expect([200, 401]).toContain(res.status);
-  });
-
-  it('returns 401 when token does not match', async () => {
-    mockedRead.mockReturnValue('expected-token');
-    const res = await request(app)
-      .post('/protected')
-      .set('Authorization', 'Bearer wrong-token')
-      .send({});
-    expect([200, 401]).toContain(res.status);
-  });
-});