feat: add HOL Guard security skill - #511
Draft
kantorcodes wants to merge 1 commit into
Draft
Conversation
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Add HOL Guard to the public OpenHands extensions registry as an operational security skill. The skill directly installs and invokes the real
hol-guardandplugin-scannerCLIs for supported harness protection, approval/evidence review, and agent package scanning.The scope is intentionally accurate: it does not claim that OpenHands itself has a native HOL Guard pre-tool interception hook or that
openhandsis a supported Guard harness target.Summary
skills/hol-guardwith the requiredSKILL.md, human README, plugin manifest, and vendor symlinkshol-guardin the OpenHands marketplace undercode-qualityand update the generated README catalogIssue Number
N/A - the current extensions contribution guide accepts fork-first skill submissions directly.
How to Test
mainwith only the seven intended paths.skills/hol-guard/.claude-pluginand.codex-pluginare symlinks to.plugin, and.plugin/plugin.jsonis valid JSON.python scripts/sync_extensions.py --check.pytest -q tests/test_skills_have_readme.py tests/test_sync_extensions.py.skills/hol-guard/SKILL.mdand verify it never mapsopenhandsto a supported Guard harness or claims native OpenHands runtime interception.I reproduced the changed-path catalog, coverage, manifest, required README, and symlink invariants from the current repository source before opening this PR. I could not execute the full repository test suite in the connected environment because it does not expose a filesystem clone of the complete repository, so this PR is intentionally opened as a draft for repository CI and human validation.
Video/Screenshots
Not applicable - documentation and registry metadata only.
Notes
The human-test checkbox is intentionally left unchecked. HOL Guard upstream sources are https://hol.org/guard and https://github.com/hashgraph-online/hol-guard-plugin.