Skip to content

fix(corpus): correct christianity source_language to grc (Koine)#59

Merged
F-e-u-e-r merged 1 commit into
mainfrom
fix/christianity-source-language
Jul 16, 2026
Merged

fix(corpus): correct christianity source_language to grc (Koine)#59
F-e-u-e-r merged 1 commit into
mainfrom
fix/christianity-source-language

Conversation

@F-e-u-e-r

Copy link
Copy Markdown
Owner

What

Cross-model review of #49 flagged (pre-existing, not introduced there): the two 和合本 NT records (約翰福音 1:1, 希伯來書 11:1) carry provenance.source_language: "el, he"el is modern Greek (Koine is grc), and he is a whole-edition description leak (an NT excerpt's source is not Hebrew).

Change

  • Both presentation.json copies: "el, he""grc" on those two records only (byte-identical copies preserved).
  • test_adr0008_christianity_canon_scope now pins source_language == "grc" per record.
  • CHANGELOG entry under a new ### Fixed heading.

Metadata only — provenance is not report-counted; rankings/counts/reports unchanged.

Validation

python3 -m pytest 443 passed / 1 skipped · ruff clean · sidecars cmp-identical.

🤖 Generated with Claude Code

The two 和合本 NT records carried provenance.source_language "el, he" —
the modern-Greek ISO code plus a whole-edition Hebrew leak (an NT
excerpt's source is not Hebrew). Cross-model review finding on #49
(pre-existing, not introduced there). Metadata only: provenance is not
report-counted; the curation test now pins grc per record; both sidecar
copies stay byte-identical.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@F-e-u-e-r
F-e-u-e-r merged commit 52861c5 into main Jul 16, 2026
4 checks passed
F-e-u-e-r pushed a commit that referenced this pull request Jul 16, 2026
…ction adjacency with #59)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@F-e-u-e-r
F-e-u-e-r deleted the fix/christianity-source-language branch July 16, 2026 10:58
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