mirror of
https://github.com/jackwener/OpenCLI.git
synced 2026-09-14 18:25:42 +08:00
12176de1a5
* refactor: simplify core modules — remove root shims, consolidate error classification, streamline cascade/interceptor, clean up synthesize 1. Remove root-level shim files (errors.ts, logger.ts, registry.ts, types.ts, utils.ts, launcher.ts) — update all ~840 adapter imports to reference src/ directly 2. Consolidate interceptor: reuse shared DISGUISE_FN in tap interceptor instead of reimplementing 3. Unify error classification: single ClassifiedError type with icon/exitCode/hint lookup table, eliminating duplicated pattern matching between resolveExitCode and renderError 4. Simplify cascade probe: replace repetitive switch cases with PROBE_OPTIONS lookup map 12. Clean up synthesize.ts: remove deprecated snake_case field aliases (recommended_args, recommended_columns, recommendedColumnsLegacy) and unnecessary constant aliases Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * docs: update import paths in contributor docs and skill templates Update all documentation and skill files to reference src/ directly, matching the shim removal in the previous commit. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: update new Gemini adapter imports to use src/ paths Fix imports in newly added deep-research adapter files that were still referencing the deleted root shim files. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: update xiaohongshu tests for /search_result/ URL change Tests now expect /search_result/<id> for bare note IDs (matching the note-helpers.ts change from PR #774) and updated empty-shell hint assertion. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: update LessWrong and hupu adapter imports to use src/ paths Fix imports in newly merged LessWrong and hupu/mentions adapter files that were still referencing the deleted root shim files. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * test(xueqiu): mock logger via src path --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
69 lines
2.5 KiB
TypeScript
69 lines
2.5 KiB
TypeScript
import * as fs from 'node:fs/promises';
|
|
import * as os from 'node:os';
|
|
import * as path from 'node:path';
|
|
import { beforeEach, describe, expect, it, vi } from 'vitest';
|
|
import { CliError } from '../../src/errors.js';
|
|
import {
|
|
MAX_PDF_BYTES,
|
|
buildReviewUrl,
|
|
parseYesNo,
|
|
readPdfFile,
|
|
requestJson,
|
|
validateHelpfulness,
|
|
} from './utils.js';
|
|
|
|
describe('paperreview utils', () => {
|
|
beforeEach(() => {
|
|
vi.restoreAllMocks();
|
|
});
|
|
|
|
it('builds review URLs from the token', () => {
|
|
expect(buildReviewUrl('tok 123')).toBe('https://paperreview.ai/review?token=tok%20123');
|
|
});
|
|
|
|
it('parses yes/no flags', () => {
|
|
expect(parseYesNo('yes', 'critical-error')).toBe(true);
|
|
expect(parseYesNo('NO', 'critical-error')).toBe(false);
|
|
});
|
|
|
|
it('rejects invalid yes/no flags with CliError', () => {
|
|
expect(() => parseYesNo('maybe', 'critical-error')).toThrow(CliError);
|
|
expect(() => parseYesNo('maybe', 'critical-error')).toThrow('"critical-error" must be either "yes" or "no".');
|
|
});
|
|
|
|
it('validates helpfulness scores', () => {
|
|
expect(validateHelpfulness(5)).toBe(5);
|
|
expect(() => validateHelpfulness(0)).toThrow(CliError);
|
|
});
|
|
|
|
it('reads a valid PDF file and returns metadata', async () => {
|
|
const tempDir = await fs.mkdtemp(path.join(os.tmpdir(), 'paperreview-'));
|
|
const pdfPath = path.join(tempDir, 'sample.pdf');
|
|
const pdfBytes = Buffer.concat([Buffer.from('%PDF-1.4\n'), Buffer.alloc(256, 1)]);
|
|
await fs.writeFile(pdfPath, pdfBytes);
|
|
|
|
const result = await readPdfFile(pdfPath);
|
|
|
|
expect(result.fileName).toBe('sample.pdf');
|
|
expect(result.resolvedPath).toBe(pdfPath);
|
|
expect(result.sizeBytes).toBe(pdfBytes.length);
|
|
expect(result.buffer.equals(pdfBytes)).toBe(true);
|
|
});
|
|
|
|
it('rejects PDFs larger than the paperreview.ai size limit', async () => {
|
|
const tempDir = await fs.mkdtemp(path.join(os.tmpdir(), 'paperreview-'));
|
|
const pdfPath = path.join(tempDir, 'large.pdf');
|
|
await fs.writeFile(pdfPath, Buffer.alloc(MAX_PDF_BYTES + 1, 1));
|
|
|
|
await expect(readPdfFile(pdfPath)).rejects.toThrow(CliError);
|
|
await expect(readPdfFile(pdfPath)).rejects.toThrow('The PDF is larger than paperreview.ai\'s 10MB limit.');
|
|
});
|
|
|
|
it('normalizes fetch failures into CliError', async () => {
|
|
vi.stubGlobal('fetch', vi.fn().mockRejectedValue(new Error('socket hang up')));
|
|
|
|
await expect(requestJson('/api/review/token')).rejects.toThrow(CliError);
|
|
await expect(requestJson('/api/review/token')).rejects.toThrow('Unable to reach paperreview.ai');
|
|
});
|
|
});
|