Skill import from URLs (generic, not GitHub-only) #948

New issue

Closed

opened 2026-06-18 10:57:46 +02:00 by sleepy · 1 comment

sleepy commented

2026-06-18 10:57:46 +02:00

Owner

Overview

Allow importing SKILL.md bundles from arbitrary URLs. The upstream implementation targets GitHub raw URLs, but the core feature is generic URL import — works with any URL serving a .md file.

Upstream References

PR #2576: Import SKILL.md bundles from public GitHub URLs
Upstream commit: b448119 feat(skills): import SKILL.md bundles from public GitHub URLs (#2576)
Local paths: ../odysseus_upstream/routes/skills_routes.py
Tests: ../odysseus_upstream/tests/test_skill_importer.py

Implementation

Add URL import endpoint to skills routes
Accept arbitrary URL (not GitHub-specific — works with any .md source)
Validate SKILL.md frontmatter (name, description required)
SSRF-guarded fetch (validate URL scheme, no internal hosts)
Store imported skill alongside existing ones
UI: add URL import field in skills management
Port tests/test_skill_importer.py
Adapt URL to work with local Forgejo raw file URLs: https://git.kokoham.com/sleepy/<repo>/raw/branch/path/to/SKILL.md

Priority

Medium — enables sharing skills across instances.

## Overview Allow importing SKILL.md bundles from arbitrary URLs. The upstream implementation targets GitHub raw URLs, but the core feature is generic URL import — works with any URL serving a `.md` file. ## Upstream References - **PR #2576**: Import SKILL.md bundles from public GitHub URLs - **Upstream commit**: `b448119 feat(skills): import SKILL.md bundles from public GitHub URLs (#2576)` - **Local paths**: `../odysseus_upstream/routes/skills_routes.py` - **Tests**: `../odysseus_upstream/tests/test_skill_importer.py` ## Implementation 1. Add URL import endpoint to skills routes 2. Accept arbitrary URL (not GitHub-specific — works with any `.md` source) 3. Validate SKILL.md frontmatter (name, description required) 4. SSRF-guarded fetch (validate URL scheme, no internal hosts) 5. Store imported skill alongside existing ones 6. UI: add URL import field in skills management 7. Port `tests/test_skill_importer.py` 8. Adapt URL to work with local Forgejo raw file URLs: `https://git.kokoham.com/sleepy/<repo>/raw/branch/path/to/SKILL.md` ## Priority Medium — enables sharing skills across instances.

sleepy referenced this issue from a commit

2026-06-18 18:19:41 +02:00

feat(skills): import SKILL.md from arbitrary URLs

sleepy referenced this issue from a pull request that will close it,

2026-06-18 18:41:33 +02:00

feat: skill import from URLs (generic, not GitHub-only) (#948) #959

sleepy referenced this issue from a commit

2026-06-18 18:51:49 +02:00

feat(skills): import SKILL.md from arbitrary URLs

sleepy referenced this issue from a commit

2026-06-18 18:51:49 +02:00

fix: address PR review feedback for #948 skill import URLs

sleepy referenced this issue from a commit

2026-06-18 18:56:41 +02:00

Merge PR #959: skill import from URLs (#948)

sleepy closed this issue

2026-06-18 18:57:23 +02:00

sleepy commented

2026-06-18 18:57:23 +02:00

Author

Owner

Merged via squash to dev (7df4ec4). 6 files, +1174 lines. SSRF-guarded URL fetcher with scheme whitelist, public-IP DNS check, redirect validation, Content-Type filtering, 128KB cap. 50 tests passing. PR review: 1 round (scope violation, broken redirect, dead imports, missing Content-Type validation → all fixed).

Merged via squash to dev (7df4ec4). 6 files, +1174 lines. SSRF-guarded URL fetcher with scheme whitelist, public-IP DNS check, redirect validation, Content-Type filtering, 128KB cap. 50 tests passing. PR review: 1 round (scope violation, broken redirect, dead imports, missing Content-Type validation → all fixed).