FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add ChoiceBySchemeDocumentLoader for scheme-based document loading by anatoly-scherbakov · Pull Request #302 · digitalbazaar/pyld · GitHub

/ pyld Public

Add ChoiceBySchemeDocumentLoader for scheme-based document loading - #302

Merged
anatoly-scherbakov merged 11 commits into
masterfrom
choice-by-scheme-document-loader
Aug 19, 2026
ConversationCommits (11)ChecksFiles changed
Merged

Add ChoiceBySchemeDocumentLoader for scheme-based document loading#302
anatoly-scherbakov merged 11 commits into
masterfrom
choice-by-scheme-document-loader

Conversation

anatoly-scherbakov commented Jul 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add ChoiceBySchemeDocumentLoader, which dispatches to per-scheme loaders so a local file: document can resolve a remote https: @context
  • Export the loader, cover dispatch/fallback/errors with tests, and document it with an offline composed-loader example
  • Add an example_data macro and tabbed sourced JSON-LD on the ChoiceByScheme and File loader pages

Stacked on #301 (file-document-loader). Closes #300.

Test plan

  • pytest tests/test_choice_by_scheme_document_loader.py
  • make test
  • make lint
  • make docs-build
  • Confirm docs page tabs show Example + sourced JSON-LD for ChoiceByScheme and File loaders

github-actions Bot commented Jul 26, 2026 •
edited
Loading

Copy link
Copy Markdown

Coverage Report
FileStmtsMissCoverMissing
lib/pyld
   canon.py218498%27, 180, 565–566
   context_resolver.py103892%78, 137, 151, 197, 203–205, 229
   iri_resolver.py141398%311–312, 322
   jsonld.py251716693%258–262, 311, 348, 353, 363, 380–395, 431–432, 474, 482, 502, 510–511, 619–624, 704–705, 720–721, 781–786, 811–812, 827–828, 838–839, 865, 906, 916, 922–926, 937–938, 981, 988, 1053, 1076, 1098–1100, 1110–1113, 1128, 1151, 1163–1166, 1262, 1274–1277, 1297, 1326, 1359, 1363, 1400, 1443, 1462–1465, 1478, 1527, 1630–1637, 1664–1666, 1692–1695, 1871, 1909, 2215, 2346, 2355–2362, 2456, 2483, 2507, 2515, 3106, 3151, 3170, 3235, 3288, 3291, 3354, 3439, 3473, 3481, 3759, 3764, 3766, 3951–3955, 4066, 4108, 4171, 4406, 4694, 4826, 4860, 4889–4892, 4918–4919, 5102, 5166, 5193, 5267, 5269, 5287, 5385, 5434, 5666, 5668, 5707, 5709, 5718, 5860, 5988, 6075, 6195–6209, 6271, 6273, 6341, 6554, 6560–6564, 6603, 6606, 6696–6697, 6706
lib/pyld/documentloader
   aiohttp.py712269%29–39, 68, 74, 86, 98–106, 118–121, 138, 153–156
   file.py36197%49
   requests.py52590%57, 71, 83–91
   scheme_directed.py15193%38
TOTAL348121094% 

Tests Skipped Failures Errors Time
1734 37 💤 0 ❌ 0 🔥 2m 57s ⏱️

This was referenced Jul 26, 2026
Merged
Open
anatoly-scherbakov force-pushed the choice-by-scheme-document-loader branch from e306a03 to 59fa58c Compare August 10, 2026 18:28
mielvds approved these changes Aug 19, 2026
View reviewed changes
Comment thread lib/pyld/documentloader/choice_by_scheme.py Outdated
Base automatically changed from file-document-loader to master August 19, 2026 18:56
anatoly-scherbakov and others added 11 commits August 20, 2026 00:02
[#300]: Add SchemeDirectedDocumentLoader for scheme-based dispatch
b7c3736 
Co-authored-by: Cursor <cursoragent@cursor.com>
[#300]: Export SchemeDirectedDocumentLoader from pyld
ffa1c6b 
Co-authored-by: Cursor <cursoragent@cursor.com>
[#300]: Add tests for SchemeDirectedDocumentLoader
975f35b 
Co-authored-by: Cursor <cursoragent@cursor.com>
[#300]: Add sample person JSON-LD with remote @context
53e56e6 
Co-authored-by: Cursor <cursoragent@cursor.com>
[#300]: Add SchemeDirectedDocumentLoader docs example
18797db 
Co-authored-by: Cursor <cursoragent@cursor.com>
[#300]: Add example_data mkdocs macro for sourced example files
56aa0d4 
Co-authored-by: Cursor <cursoragent@cursor.com>
[#300]: Show sourced JSON-LD beside FileDocumentLoader examples
89be072 
Co-authored-by: Cursor <cursoragent@cursor.com>
[#300]: Document SchemeDirectedDocumentLoader
6f6586d 
Co-authored-by: Cursor <cursoragent@cursor.com>
[#300]: Link SchemeDirectedDocumentLoader on document loaders index
80cec29 
Co-authored-by: Cursor <cursoragent@cursor.com>
[#300]: Note SchemeDirectedDocumentLoader in the unreleased changelog
add904a 
Co-authored-by: Cursor <cursoragent@cursor.com>
[#300]: Require composed document loaders to accept only `DocumentLoa…
ea83877 
…der` instances

Co-authored-by: Cursor <cursoragent@cursor.com>
anatoly-scherbakov force-pushed the choice-by-scheme-document-loader branch from 59fa58c to ea83877 Compare August 19, 2026 20:02
anatoly-scherbakov linked an issue Aug 19, 2026 that may be closed by this pull request
Add ChoiceBySchemeDocumentLoader for scheme-based document loading #300
Closed
anatoly-scherbakov merged commit d2cd732 into master Aug 19, 2026
12 of 20 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

mielvds mielvds approved these changes
BigBlueHat Awaiting requested review from BigBlueHat

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v3.3.0 (CLI)

Development

Successfully merging this pull request may close these issues.

Add ChoiceBySchemeDocumentLoader for scheme-based document loading

2 participants

Back | FazBrowse Home | New Git URL