Split CLI entrypoint out (yargs breaks under jest)

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Author: ronshapiro

precomputed/ai_additions/Menachot 77a.json

@@ -0,0 +1,22 @@
+{
+  "Rashi on Menachot 77a:1:1": {
+    "hebrew": "<strong class=\"dibur-hamatchil\">עשרה היו לחמץ עשרון לחלה</strong> - די' חלות היו חמץ, ובמצה ג' מינין וכל אחד היה של י' חלות, נמצא במצה שלשים חלות של י' עשרונות, נמצאו שלש עשרונות ושליש עשרון לכל מין ומין, וג' חלות לעשרון, שמכל עשרון היה עושה שלשה חלות של מצה, דשלשים חלות לא היו אלא י' עשרון. ומה ששנינו עד כאן היא במדה מדברית, אבל במדה ירושלמית שהיו ה' סאין, שהן שלשים קבין, דסאה ששה קבין, ט\"ו קבין לחמץ וט\"ו למצה:",
+    "english": "<strong class=\"dibur-hamatchil\">Ten were for leavened bread, a tenth for each loaf</strong> - That the ten loaves were leavened bread. And for matzah there were three kinds, and each kind consisted of ten loaves; thus for matzah there were thirty loaves, from ten tenths [of an ephah]. So there were three and a third tenths for each and every kind, and three loaves per tenth - for from each tenth he would make three loaves of matzah, since the thirty loaves came from only ten tenths. And what we have learned so far is according to the wilderness measure. But according to the Jerusalem measure, [the total dough] was five se'ah, which are thirty kav (a se'ah being six kav) - fifteen kav for the leavened bread and fifteen for the unleavened bread."
+  },
+  "Rashi on Menachot 77a:2:1": {
+    "english": "<b>One and a half <i>kav</i></b>: He would allot this [amount] for [each] loaf, for from the fifteen <i>kav</i> he would make ten loaves."
+  },
+  "Rashi on Menachot 77a:2:2": {
+    "hebrew": "<strong class=\"dibur-hamatchil\">ושתי חלות לקב</strong> - דמקב אחד היה עושה שתי חלות, שהרי מט\"ו קבין היה עושה שלשים חלות של מצה:",
+    "english": "<b>And two loaves per <i>kav</i></b>: For from one <i>kav</i> he would make two loaves, since from fifteen <i>kav</i> he would make thirty loaves of matza."
+  },
+  "Rashi on Menachot 77a:3:1": {
+    "english": "<strong>From where are these matters</strong> - that an ephah is three <i>se’a</i>."
+  },
+  "Rashi on Menachot 77a:3:2": {
+    "english": "<strong class=\"dibur-hamatchil\">ONE MEASURE</strong> - The ephah and the bat are the same measure."
+  },
+  "Rashi on Menachot 77a:4:1": {
+    "english": "The bat represents the tenth-part taken from a ḥomer, for a ḥomer is thirty se’a, and the verse states that the ephah and the bat are of the same measure."
+  }
+}

precomputed/rsi_state/generation_records/rashi_tosafot_translation/Menachot 77a.json

@@ -0,0 +1,62 @@
+{
+  "Rashi on Menachot 77a:1:1": {
+    "dependsOn": [],
+    "generatedAt": "2026-08-10T13:07:25.524Z",
+    "model": "claude-haiku-4-5-20251001",
+    "promptVersion": "v1",
+    "sourceRefs": [
+      "Rashi on Menachot 77a:1:1"
+    ],
+    "sourceText": "<strong class=\"dibur-hamatchil\">עשרה היו לחמץ עשרון לחלה</strong> - די' חלות היו חמץ ובמצה ג' מינין וכל אחד היה של י' חלות נמצא במצה שלשים חלות של י' עשרונות נמצאו שלש עשרונות ושליש עשרון לכל מין ומין וג' חלות לעשרון שמכל עשרון היה עושה שלשה חלות של מצה דשלשים חלות לא היו אלא י' עשרון ומה ששנינו עד כאן היא במדה מדברית אבל במדה ירושלמית שהיו ה' סאין שהן שלשים קבין דסאה ששה קבין ט\"ו קבין לחמץ וט\"ו למצה:"
+  },
+  "Rashi on Menachot 77a:2:1": {
+    "dependsOn": [],
+    "generatedAt": "2026-08-10T13:08:28.026Z",
+    "model": "claude-haiku-4-5-20251001",
+    "promptVersion": "v1",
+    "sourceRefs": [
+      "Rashi on Menachot 77a:2:1"
+    ],
+    "sourceText": "<strong class=\"dibur-hamatchil\">קב ומחצה</strong> - היה נותן לחלה דמט\"ו קבין היה עושה י' חלות:"
+  },
+  "Rashi on Menachot 77a:2:2": {
+    "dependsOn": [],
+    "generatedAt": "2026-08-10T13:10:01.795Z",
+    "model": "claude-haiku-4-5-20251001",
+    "promptVersion": "v1",
+    "sourceRefs": [
+      "Rashi on Menachot 77a:2:2"
+    ],
+    "sourceText": "<strong class=\"dibur-hamatchil\">ושתי חלות לקב</strong> - דמקב אחד היה עושה שתי חלות שהרי מט\"ו קבין היה עושה שלשים חלות של מצה:"
+  },
+  "Rashi on Menachot 77a:3:1": {
+    "dependsOn": [],
+    "generatedAt": "2026-08-10T13:11:03.205Z",
+    "model": "claude-haiku-4-5-20251001",
+    "promptVersion": "v1",
+    "sourceRefs": [
+      "Rashi on Menachot 77a:3:1"
+    ],
+    "sourceText": "<strong class=\"dibur-hamatchil\">מנהני מילי</strong> - דאיפה שלש סאין:"
+  },
+  "Rashi on Menachot 77a:3:2": {
+    "dependsOn": [],
+    "generatedAt": "2026-08-10T13:11:41.268Z",
+    "model": "claude-sonnet-5",
+    "promptVersion": "v1",
+    "sourceRefs": [
+      "Rashi on Menachot 77a:3:2"
+    ],
+    "sourceText": "<strong class=\"dibur-hamatchil\">תוכן אחד</strong> - מדה אחת הן:"
+  },
+  "Rashi on Menachot 77a:4:1": {
+    "dependsOn": [],
+    "generatedAt": "2026-08-10T13:13:32.200Z",
+    "model": "claude-haiku-4-5-20251001",
+    "promptVersion": "v1",
+    "sourceRefs": [
+      "Rashi on Menachot 77a:4:1"
+    ],
+    "sourceText": "<strong class=\"dibur-hamatchil\">לשאת <span class=\"parenthesized\">(את)</span> מעשר החומר הבת</strong> - והבת היא הוא נטילת מעשר על החומר דחומר הוא ל' סאין ואמר קרא דאיפה ובת תוכן אחד הוא:"
+  }
+}

rsi_orchestrator/README.md

@@ -26,17 +26,19 @@ billing.
   boundaries live in the repo rather than pre-assembling context itself, so the model decides how
   much it needs to read — then runs a bounded self-critique pass (generate → critique → at most
   one retry with feedback → give up) before writing to `precomputed/ai_additions/<Book Page>.json`
-  and recording a generation record. Run with `npx ts-node rsi_orchestrator/rashi_tosafot_translation.ts <CanonicalBookName>`
-  (e.g. `Zevachim`) — requires `cached_outputs/api_request_handler/` to be populated for that book
-  first (`npx ts-node cache_all_api_requests.ts`).
+  and recording a generation record. Requires `cached_outputs/api_request_handler/` to be
+  populated for that book first (`npx ts-node cache_all_api_requests.ts`).
+- `rashi_tosafot_translation_cli.ts` — the CLI entrypoint for the above, kept in a separate file
+  because `yargs` is ESM-only and breaks under jest; the core module stays importable by its test
+  file this way. `--section`/`--limit` bound a run to one page / a handful of candidates.
 - `precomputed/rsi_state/triage_log.json` (created on first run) — tracks which issue numbers have
   already been triaged, so re-running doesn't re-comment on the same issue.
 
 ## Running manually
 
 ```sh
 npx ts-node rsi_orchestrator/triage_suggestions.ts
-npx ts-node rsi_orchestrator/rashi_tosafot_translation.ts Zevachim
+npx ts-node rsi_orchestrator/rashi_tosafot_translation_cli.ts Zevachim --section 2a --limit 2
 ```
 
 Requires the `gh` CLI authenticated with access to `ronshapiro/talmud.page` (already true on this

rsi_orchestrator/rashi_tosafot_translation.ts

@@ -1,9 +1,9 @@
 import * as fs from "fs";
 import {Amud, ApiComment} from "../apiTypes";
-import {Book, books} from "../books";
+import {Book} from "../books";
 import {cachedOutputFilePath} from "../cached_outputs";
 import {readUtf8} from "../files";
-import {Edit, writeAiEdit} from "../precomputed/ai_edits";
+import {Edit} from "../precomputed/ai_edits";
 import {readGenerationRecord, upsertGenerationRecord} from "../precomputed/rsi_state/generation_record";
 import {checkTextStaleness, DEFAULT_STALENESS_THRESHOLDS} from "../precomputed/rsi_state/staleness";
 import {toFlatArray} from "../sefariaTextType";
@@ -198,7 +198,7 @@ async function critiqueViaClaude(
   return parseJsonResponse<CritiqueVerdict>(result.text);
 }
 
-async function generateAndRecord(
+export async function generateAndRecord(
   candidate: TranslationCandidate,
 ): Promise<GeneratedEdit | undefined> {
   return generateWithSelfCritique(candidate, {
@@ -207,7 +207,7 @@ async function generateAndRecord(
   });
 }
 
-function recordGenerationForCandidate(
+export function recordGenerationForCandidate(
   candidate: TranslationCandidate, generated: GeneratedEdit,
 ): void {
   upsertGenerationRecord(TASK_TYPE, candidate.page, candidate.ref, {
@@ -219,31 +219,3 @@ function recordGenerationForCandidate(
     dependsOn: [],
   });
 }
-
-async function main(): Promise<void> {
-  const bookName = process.argv[2];
-  if (!bookName || !books.byCanonicalName[bookName]) {
-    console.error("Usage: ts-node rashi_tosafot_translation.ts <CanonicalBookName> [limit]");
-    process.exitCode = 1;
-    return;
-  }
-  const book = books.byCanonicalName[bookName];
-  const limit = process.argv[3] ? parseInt(process.argv[3], 10) : undefined;
-  await translateRashiTosafotComments({
-    listCandidates: () => {
-      const candidates = listCandidatesForBook(book);
-      return limit ? candidates.slice(0, limit) : candidates;
-    },
-    isFresh: isFreshTranslation,
-    generate: generateAndRecord,
-    writeEdit: (candidate, edit) => writeAiEdit(candidate.page, candidate.ref, edit),
-    recordGeneration: recordGenerationForCandidate,
-  });
-}
-
-if (require.main === module) {
-  main().catch(e => {
-    console.error(e);
-    process.exitCode = 1;
-  });
-}

rsi_orchestrator/rashi_tosafot_translation_cli.ts

@@ -0,0 +1,50 @@
+import * as yargs from "yargs";
+import {hideBin} from "yargs/helpers";
+import {books} from "../books";
+import {writeAiEdit} from "../precomputed/ai_edits";
+import {
+  generateAndRecord,
+  isFreshTranslation,
+  listCandidatesForBook,
+  recordGenerationForCandidate,
+  translateRashiTosafotComments,
+} from "./rashi_tosafot_translation";
+
+/**
+ * CLI entrypoint, kept separate from rashi_tosafot_translation.ts because `yargs` ships ESM-only
+ * and breaks under jest — this file is never imported by a test, so it's fine for it to depend on
+ * yargs; the core module (which the test file does import) stays jest-safe.
+ */
+async function main(): Promise<void> {
+  const FLAGS = yargs(hideBin(process.argv))
+    .options({
+      section: {type: "string", describe: 'e.g. "77" (matches 77a/77b) or "77a" (exact)'},
+      limit: {type: "number", describe: "cap the number of candidates processed"},
+    })
+    .parseSync();
+  const bookName = FLAGS._[0] as string | undefined;
+  if (!bookName || !books.byCanonicalName[bookName]) {
+    console.error(
+      "Usage: ts-node rashi_tosafot_translation_cli.ts <CanonicalBookName> "
+      + "[--section 77] [--limit 2]");
+    process.exitCode = 1;
+    return;
+  }
+  const book = books.byCanonicalName[bookName];
+  await translateRashiTosafotComments({
+    listCandidates: () => {
+      let candidates = listCandidatesForBook(book);
+      if (FLAGS.section) candidates = candidates.filter(c => c.section.startsWith(FLAGS.section!));
+      return FLAGS.limit ? candidates.slice(0, FLAGS.limit) : candidates;
+    },
+    isFresh: isFreshTranslation,
+    generate: generateAndRecord,
+    writeEdit: (candidate, edit) => writeAiEdit(candidate.page, candidate.ref, edit),
+    recordGeneration: recordGenerationForCandidate,
+  });
+}
+
+main().catch(e => {
+  console.error(e);
+  process.exitCode = 1;
+});