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>
120 lines
3.6 KiB
TypeScript
120 lines
3.6 KiB
TypeScript
import { cli, Strategy } from '../../src/registry.js';
|
|
import { CliError } from '../../src/errors.js';
|
|
import {
|
|
PAPERREVIEW_DOMAIN,
|
|
ensureApiSuccess,
|
|
ensureSuccess,
|
|
normalizeVenue,
|
|
readPdfFile,
|
|
requestJson,
|
|
summarizeSubmission,
|
|
uploadPresignedPdf,
|
|
} from './utils.js';
|
|
|
|
cli({
|
|
site: 'paperreview',
|
|
name: 'submit',
|
|
description: 'Submit a PDF to paperreview.ai for review',
|
|
domain: PAPERREVIEW_DOMAIN,
|
|
strategy: Strategy.PUBLIC,
|
|
browser: false,
|
|
timeoutSeconds: 120,
|
|
args: [
|
|
{ name: 'pdf', positional: true, required: true, help: 'Path to the paper PDF' },
|
|
{ name: 'email', required: true, help: 'Email address for the submission' },
|
|
{ name: 'venue', help: 'Optional target venue such as ICLR or NeurIPS' },
|
|
{ name: 'dry-run', type: 'bool', default: false, help: 'Validate the input and stop before remote submission' },
|
|
{ name: 'prepare-only', type: 'bool', default: false, help: 'Request an upload slot but stop before uploading the PDF' },
|
|
],
|
|
columns: ['status', 'file', 'email', 'venue', 'token', 'review_url', 'message'],
|
|
footerExtra: (kwargs) => {
|
|
if (kwargs['dry-run'] === true) return 'dry run only';
|
|
if (kwargs['prepare-only'] === true) return 'prepared only';
|
|
return undefined;
|
|
},
|
|
func: async (_page, kwargs) => {
|
|
const pdfFile = await readPdfFile(kwargs.pdf);
|
|
const email = String(kwargs.email ?? '').trim();
|
|
const venue = normalizeVenue(kwargs.venue);
|
|
const dryRun = kwargs['dry-run'] === true;
|
|
const prepareOnly = kwargs['prepare-only'] === true;
|
|
|
|
if (!email) {
|
|
throw new CliError('ARGUMENT', 'An email address is required.', 'Pass --email <address>');
|
|
}
|
|
|
|
if (dryRun) {
|
|
return summarizeSubmission({
|
|
pdfFile,
|
|
email,
|
|
venue,
|
|
message: 'Input validation passed. No remote request was sent.',
|
|
dryRun: true,
|
|
});
|
|
}
|
|
|
|
const { response: uploadUrlResponse, payload: uploadUrlPayload } = await requestJson('/api/get-upload-url', {
|
|
method: 'POST',
|
|
headers: { 'Content-Type': 'application/json' },
|
|
body: JSON.stringify({
|
|
filename: pdfFile.fileName,
|
|
venue,
|
|
}),
|
|
});
|
|
ensureSuccess(
|
|
uploadUrlResponse,
|
|
uploadUrlPayload,
|
|
'Failed to request an upload URL.',
|
|
'Try again in a moment',
|
|
);
|
|
ensureApiSuccess(
|
|
uploadUrlPayload,
|
|
'paperreview.ai did not return a usable upload URL.',
|
|
'Try again in a moment',
|
|
);
|
|
|
|
if (prepareOnly) {
|
|
return summarizeSubmission({
|
|
pdfFile,
|
|
email,
|
|
venue,
|
|
message: 'Upload slot prepared. The PDF was not uploaded and no submission was confirmed.',
|
|
s3Key: uploadUrlPayload.s3_key,
|
|
status: 'prepared',
|
|
});
|
|
}
|
|
|
|
await uploadPresignedPdf(uploadUrlPayload.presigned_url, pdfFile, uploadUrlPayload);
|
|
|
|
const confirmForm = new FormData();
|
|
confirmForm.append('s3_key', uploadUrlPayload.s3_key);
|
|
confirmForm.append('venue', venue);
|
|
confirmForm.append('email', email);
|
|
|
|
const { response: confirmResponse, payload: confirmPayload } = await requestJson('/api/confirm-upload', {
|
|
method: 'POST',
|
|
body: confirmForm,
|
|
});
|
|
ensureSuccess(
|
|
confirmResponse,
|
|
confirmPayload,
|
|
'Failed to confirm the upload with paperreview.ai.',
|
|
'Try again in a moment',
|
|
);
|
|
ensureApiSuccess(
|
|
confirmPayload,
|
|
'paperreview.ai did not confirm the submission.',
|
|
'Try again in a moment',
|
|
);
|
|
|
|
return summarizeSubmission({
|
|
pdfFile,
|
|
email,
|
|
venue,
|
|
token: confirmPayload.token,
|
|
message: confirmPayload.message,
|
|
s3Key: uploadUrlPayload.s3_key,
|
|
});
|
|
},
|
|
});
|