Skip to content

fix(web): add read() via Jina Reader + skill install fallback#238

Merged
Panniantong merged 1 commit intomainfrom
fix/web-read-fallback
Mar 31, 2026
Merged

fix(web): add read() via Jina Reader + skill install fallback#238
Panniantong merged 1 commit intomainfrom
fix/web-read-fallback

Conversation

@Panniantong
Copy link
Copy Markdown
Owner

Summary

  • WebChannel.read(): reads any URL via Jina Reader, returns Markdown
  • _install_skill(): fallback from importlib.resources to Path(__file__) for editable installs
  • UTF-8 encoding on all skill file operations

Inspired by #215, implemented with Jina Reader (not raw requests).

Closes #215

Test plan

  • 77 tests passing

- WebChannel.read(): reads any URL via Jina Reader (r.jina.ai), returns Markdown
- _install_skill(): add fallback from importlib.resources to Path(__file__) for editable installs
- Explicit UTF-8 encoding on all file read/write operations

Inspired by PR #215, implemented correctly (Jina instead of raw requests).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Panniantong Panniantong merged commit 0f9cfe6 into main Mar 31, 2026
8 checks passed
@Panniantong Panniantong deleted the fix/web-read-fallback branch March 31, 2026 10:31
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