Skip to content

feat(authors): import author biographies into author_bio - #108

Merged
kdroidFilter merged 1 commit into
masterfrom
feat/author-bios
Oct 8, 2026
Merged

kdroidFilter merged 1 commit into
masterfrom
feat/author-bios

Conversation

@kdroidFilter

Copy link
Copy Markdown
Owner

What

  • Schema: author_bio(authorId, summary, bodyMd, confidence, licenseId); tracked by the patch tables and the logical content hash.
  • :generator-common:importAuthorBios (AuthorBioFile parser + ImportAuthorBiosCli): downloads the main archive of kdroidFilter/seforim-author-bio (now public), or reads -PauthorBioDir=. Joins by author name (the id's natural key) ignoring LRM/RLM marks, stores the תקציר summary, the markdown body (sections, zayit:// links, sources) and the confidence, pointing at the CC-BY-SA license. Unmatched biographies are logged and skipped.
  • generateSeforimDb runs it after generateHavroutaLinks and before buildCatalog.

Verification

  • ./gradlew :generator-common:jvmTest :sefariasqlite:jvmTest :dao:jvmTest :delta-updater:jvmTest
  • On a copy of the current production DB: 384/384 biographies imported (one needed the bidi-mark normalization: ישראל ליפשיץ).

Notes

  • The download path could not be exercised locally (TLS interception on the dev machine); CI has direct access.

🤖 Generated with Claude Code

New :generator-common:importAuthorBios step, run by generateSeforimDb once
every author exists (after Otzaria, before the catalog). It reads the
kdroidFilter/seforim-author-bio biographies (main branch archive, or
-PauthorBioDir= for a local checkout), joins each one to its author by
name ignoring bidi marks, and stores the תקציר summary, the full markdown
body and the confidence, under the CC-BY-SA license row. Biographies
without a matching author are reported and skipped.

Signed-off-by: Elie Gambache <elyahou.hadass@gmail.com>
@kdroidFilter
kdroidFilter merged commit ecdf041 into master Oct 8, 2026
1 check passed
@kdroidFilter
kdroidFilter deleted the feat/author-bios branch October 8, 2026 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant