Files
jackwener__opencli/clis/paperreview/submit.ts
jakevin 12176de1a5 refactor: simplify core modules (#784)
* 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>
2026-04-05 03:35:12 +08:00

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,
});
},
});