Skip to content

feat(corpus): tag Quran records with corpus family#50

Merged
F-e-u-e-r merged 2 commits into
mainfrom
corpus/phase2-islam-quran
Jul 15, 2026
Merged

feat(corpus): tag Quran records with corpus family#50
F-e-u-e-r merged 2 commits into
mainfrom
corpus/phase2-islam-quran

Conversation

@F-e-u-e-r

@F-e-u-e-r F-e-u-e-r commented Jul 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

Summary

  • Add corpus_family=quran to the five curated Ma Jian Qur'an published-translation records in both presentation sidecars.
  • Add a curation regression test covering the Qur'an corpus-family decision and explicitly pinning no canon_scope/textual_witness/edition-backed assurance for these records.
  • Document the ADR 0008 Phase 1 backfill metadata-only change in CHANGELOG without regenerating reports.

Review

  • No blocking issues found in the local code review.
  • Confirmed both presentation.json copies are byte-identical.
  • Confirmed committed retrieval benchmark reports remain reproducible.

Tests

  • python3 -m pytest
  • python3 -m pytest tests/test_retrieval_benchmark.py::RetrievalBenchmarkTest::test_committed_threshold_t2_is_reproducible tests/test_retrieval_benchmark.py::RetrievalBenchmarkTest::test_committed_bm25_is_reproducible tests/test_retrieval_benchmark.py::RetrievalBenchmarkTest::test_committed_bm25_threshold_reports_are_reproducible tests/test_retrieval_benchmark.py::RetrievalBenchmarkTest::test_committed_baseline_report_is_reproducible
  • python3 -m ruff check .

@F-e-u-e-r F-e-u-e-r mentioned this pull request Jul 6, 2026
Merged
@claude
Merge origin/main into corpus/phase2-islam-quran (resolve adjacency; …
c68cc40 
…fix stale prose per review)

Keeps both sides (main first) in CHANGELOG.md and tests/test_corpus_curation.py.
Review-driven fixes folded in per the round-A dispositions: Phase 1 backfill
naming; the CHANGELOG count wording now states 12 lexical / 13 bm25 (set by the
Christianity backfill; corpus_family is not report-counted); the stale 'no
sidecar disclosure' prose about the 6th Qur'an record now points at the #51
interpretation_only decision, and the test additionally pins that the untagged
record IS that cue (assertIs interpretation_only True).

Verified on the merged tree: 443 passed / 1 skipped, ruff clean, fixtures OK,
sidecars byte-identical, baseline + bm25 reports regenerate byte-identically.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@F-e-u-e-r
F-e-u-e-r merged commit 55bc844 into main Jul 15, 2026
4 checks passed
@F-e-u-e-r
F-e-u-e-r deleted the corpus/phase2-islam-quran branch July 15, 2026 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@F-e-u-e-r