Skip to content

refactor: update PUBLIC_SKILLS_REPO to OpenHands/extensions#2085

Open
neubig wants to merge 2 commits intomainfrom
refactor/extensions-rename
Open

refactor: update PUBLIC_SKILLS_REPO to OpenHands/extensions#2085
neubig wants to merge 2 commits intomainfrom
refactor/extensions-rename

Conversation

@neubig
Copy link
Contributor

@neubig neubig commented Feb 15, 2026

Summary

Update all references from OpenHands/skills to OpenHands/extensions to support the repository rename.

Changes

  • Update PUBLIC_SKILLS_REPO constant in skill.py from https://github.com/OpenHands/skills to https://github.com/OpenHands/extensions
  • Update agent_context.py load_public_skills description
  • Update skills_router.py and skills_service.py comments
  • Update examples and documentation
  • Update tests

Files Changed

File Change
openhands-sdk/openhands/sdk/context/skills/skill.py PUBLIC_SKILLS_REPO URL constant
openhands-sdk/openhands/sdk/context/agent_context.py Description text
openhands-agent-server/openhands/agent_server/skills_router.py Comments
openhands-agent-server/openhands/agent_server/skills_service.py Comments
examples/01_standalone_sdk/03_activate_skill.py Comments
examples/03_github_workflows/02_pr_review/README.md Documentation URLs
examples/03_github_workflows/02_pr_review/agent_script.py Comments
tests/sdk/context/skill/test_load_public_skills.py Test URLs

Merge Order

⚠️ Important: This PR should be merged AFTER the skills repository is renamed to OpenHands/extensions.

GitHub redirects will handle the transition period, so there's no risk of breakage:

  • Before rename: The URL https://github.com/OpenHands/extensions won't work, but tests will fail clearly
  • After rename: GitHub automatically redirects OpenHands/skillsOpenHands/extensions

Related PRs

Testing

Tests should pass once the repository is renamed. Before the rename, tests referencing the new URL will fail (expected behavior).

@neubig can click here to continue refining the PR


Agent Server images for this PR

GHCR package: https://github.com/OpenHands/agent-sdk/pkgs/container/agent-server

Variants & Base Images

Variant Architectures Base Image Docs / Tags
java amd64, arm64 eclipse-temurin:17-jdk Link
python amd64, arm64 nikolaik/python-nodejs:python3.12-nodejs22 Link
golang amd64, arm64 golang:1.21-bookworm Link

Pull (multi-arch manifest)

# Each variant is a multi-arch manifest supporting both amd64 and arm64
docker pull ghcr.io/openhands/agent-server:9675f55-python

Run

docker run -it --rm \
  -p 8000:8000 \
  --name agent-server-9675f55-python \
  ghcr.io/openhands/agent-server:9675f55-python

All tags pushed for this build

ghcr.io/openhands/agent-server:9675f55-golang-amd64
ghcr.io/openhands/agent-server:9675f55-golang_tag_1.21-bookworm-amd64
ghcr.io/openhands/agent-server:9675f55-golang-arm64
ghcr.io/openhands/agent-server:9675f55-golang_tag_1.21-bookworm-arm64
ghcr.io/openhands/agent-server:9675f55-java-amd64
ghcr.io/openhands/agent-server:9675f55-eclipse-temurin_tag_17-jdk-amd64
ghcr.io/openhands/agent-server:9675f55-java-arm64
ghcr.io/openhands/agent-server:9675f55-eclipse-temurin_tag_17-jdk-arm64
ghcr.io/openhands/agent-server:9675f55-python-amd64
ghcr.io/openhands/agent-server:9675f55-nikolaik_s_python-nodejs_tag_python3.12-nodejs22-amd64
ghcr.io/openhands/agent-server:9675f55-python-arm64
ghcr.io/openhands/agent-server:9675f55-nikolaik_s_python-nodejs_tag_python3.12-nodejs22-arm64
ghcr.io/openhands/agent-server:9675f55-golang
ghcr.io/openhands/agent-server:9675f55-java
ghcr.io/openhands/agent-server:9675f55-python

About Multi-Architecture Support

  • Each variant tag (e.g., 9675f55-python) is a multi-arch manifest supporting both amd64 and arm64
  • Docker automatically pulls the correct architecture for your platform
  • Individual architecture tags (e.g., 9675f55-python-amd64) are also available if needed

Update all references from OpenHands/skills to OpenHands/extensions
to support the repository rename.

Changes:
- Update PUBLIC_SKILLS_REPO constant in skill.py
- Update documentation and examples
- Update agent_context.py load_public_skills description
- Update skills_router.py and skills_service.py comments
- Update tests

This change should be merged AFTER the skills repo is renamed to
OpenHands/extensions. GitHub redirects will handle the transition
period.

Related PRs:
- OpenHands/skills#50: Prepare for rename to OpenHands/extensions
- OpenHands/docs: Update documentation URLs (pending)

Co-authored-by: openhands <openhands@all-hands.dev>
neubig pushed a commit to OpenHands/docs that referenced this pull request Feb 15, 2026
Update all documentation references from OpenHands/skills to
OpenHands/extensions to support the repository rename.

Updated files:
- overview/skills.mdx
- overview/skills/public.mdx
- overview/skills/keyword.mdx
- overview/skills/repo.mdx
- sdk/guides/skill.mdx
- sdk/guides/github-workflows/pr-review.mdx
- openhands/usage/use-cases/code-review.mdx

Related PRs:
- OpenHands/skills#50: Prepare for rename to OpenHands/extensions
- OpenHands/software-agent-sdk#2085: Update PUBLIC_SKILLS_REPO URL

Co-authored-by: openhands <openhands@all-hands.dev>
neubig pushed a commit to OpenHands/OpenHands-CLI that referenced this pull request Feb 15, 2026
Update test comment reference from OpenHands/skills to OpenHands/extensions
to match the repository rename.

Related PRs:
- OpenHands/skills#50: Prepare for rename to OpenHands/extensions
- OpenHands/software-agent-sdk#2085: Update PUBLIC_SKILLS_REPO URL
- OpenHands/docs#337: Update documentation URLs

Co-authored-by: openhands <openhands@all-hands.dev>
@github-actions
Copy link
Contributor

github-actions bot commented Feb 15, 2026

Coverage

Coverage Report •
FileStmtsMissCoverMissing
openhands-agent-server/openhands/agent_server
   skills_service.py1543974%118, 152, 155–158, 161–163, 166–167, 170–174, 177–181, 183, 187–188, 190, 201–205, 342–343, 350–351, 355–357, 367–368
openhands-sdk/openhands/sdk/context
   agent_context.py1371092%126–127, 148–149, 273–275, 304, 327, 333
openhands-sdk/openhands/sdk/context/skills
   skill.py3502194%90–91, 246–247, 433–439, 442, 531–534, 751–752, 876, 883–884
TOTAL18964522272% 

The change to PUBLIC_SKILLS_REPO URL description is detected as an API
change by Griffe. Bump minor version to satisfy API breakage check.

Co-authored-by: openhands <openhands@all-hands.dev>
@neubig neubig added the review-this This label triggers a PR review by OpenHands label Feb 15, 2026 — with OpenHands AI
Copy link
Collaborator

@all-hands-bot all-hands-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! The refactoring is comprehensive and consistent across all references.

Quick question: Is the MINOR bump (1.11.4 → 1.12.0) intentional? Since this is just a URL update (with GitHub redirects handling backward compat), I'd expect a PATCH bump (1.11.5). But maybe there are other changes planned for 1.12.0 or you're treating the repo rename as significant enough for MINOR? 🤔

The code changes themselves look great - ready to merge after the skills→extensions repo rename is complete! ✨

@neubig neubig marked this pull request as ready for review February 16, 2026 02:42
Copy link
Collaborator

@all-hands-bot all-hands-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Clean and complete refactoring. 🚀

All references updated consistently across code, docs, examples, and tests. Backward compatibility maintained via GitHub redirects.

The MINOR bump (1.11.4 → 1.12.0) was already questioned by @all-hands-bot - I'd also expect PATCH for a URL update, but it's not a blocker. If there's a reason for MINOR (maybe treating this as a "user-facing change" since the description field changed?), that's fine.

Ready to merge once the OpenHands/skills repo is renamed! ✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-this This label triggers a PR review by OpenHands

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants