Skip to content

Add Skill Inspector companion skill - #253

Merged
rng1995 merged 1 commit into
NVIDIA:mainfrom
Dxboy266:codex/add-skill-inspector
Aug 12, 2026
Merged

Add Skill Inspector companion skill#253
rng1995 merged 1 commit into
NVIDIA:mainfrom
Dxboy266:codex/add-skill-inspector

Conversation

@Dxboy266

@Dxboy266 Dxboy266 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds skills/skill-inspector/SKILL.md, a companion Agent Skill that uses SkillSpector as the static evidence layer before an agent performs source-aware semantic review.

The skill workflow:

  • runs skillspector scan --no-llm --format json
  • reads the target skill's SKILL.md, executable scripts, MCP config/tool metadata, and finding locations
  • combines static findings with source context into APPROVE / CAUTION / REJECT

Why

SkillSpector scores and findings are strong evidence, but some sensitive capabilities are expected for certain skills. This companion skill gives agent runtimes a repeatable install-before-review workflow that does not require configuring an LLM provider inside SkillSpector.

Files

  • skills/skill-inspector/SKILL.md

Verification

Validated the PR shape with only SKILL.md present:

tmpdir=$(mktemp -d)
cp SKILL.md "$tmpdir/SKILL.md"
skillspector scan "$tmpdir" --no-llm --format json --output /tmp/skill-inspector-pr-shape.json

Result: 0 / LOW / SAFE, no issues.

@rng1995 rng1995 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[Automated SkillSpector Review]

Approved. The companion skill keeps the target untrusted, runs deterministic analysis first, prohibits executing target scripts or silently installing dependencies, and requires source-aware review of sensitive findings before an APPROVE/CAUTION/REJECT verdict. The docs-only change is coherent and its manual scan shape is appropriate.

Non-blocking: use a per-run mktemp report path instead of the predictable shared /tmp/skill-inspector-report.json.

@rng1995

rng1995 commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Fix the CI issue @Dxboy266

Signed-off-by: 青衫 <dx.boy266@gmail.com>
@Dxboy266
Dxboy266 force-pushed the codex/add-skill-inspector branch from c1bde4d to 352a40a Compare July 17, 2026 06:49
@Dxboy266
Dxboy266 marked this pull request as draft July 28, 2026 09:18
@Dxboy266
Dxboy266 marked this pull request as ready for review July 28, 2026 09:18
@Dxboy266

Copy link
Copy Markdown
Contributor Author

Hi maintainers 👋 The review is already approved (thanks @rng1995!) and the branch is mergeable with no conflicts. The only blocker is that the CI workflow is still awaiting maintainer approval to run — no checks have executed yet. Could someone with write access approve and trigger the workflow so we can get this merged? Appreciate it!

@rng1995
rng1995 merged commit 64c6299 into NVIDIA:main Aug 12, 2026
5 checks passed
@rng1995

rng1995 commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Thanks @Dxboy266 for adding this new skill ! Super-useful

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.

2 participants