Skip to content

fix(skills): require user confirmation before skill updates - #3295

Open
santhiprakash wants to merge 1 commit into
heygen-com:mainfrom
santhiprakash:fix/skill-update-user-confirmation
Open

fix(skills): require user confirmation before skill updates#3295
santhiprakash wants to merge 1 commit into
heygen-com:mainfrom
santhiprakash:fix/skill-update-user-confirmation

Conversation

@santhiprakash

Copy link
Copy Markdown
Contributor

What

Replace the "run silently, don't ask" skill-freshness directive with explicit user-confirmation guidance in ten workflow SKILL.md files, and regenerate skills-manifest.json.

Why

Fixes the trust-model issue in #2613: published skills were instructing agents to run npx hyperframes skills update without surfacing it to the user. skills/general-video/SKILL.md already used safer wording; this aligns the remaining workflow skills.

How

One-line wording change per affected skill:

  • Before: run silently, don't ask
  • After: confirm with the user before running

No CLI or runtime behavior changes.

Test plan

  • bun packages/cli/scripts/gen-skills-manifest.ts --check — manifest in sync
  • rg "run silently, don't ask" skills -g SKILL.md — zero matches
  • rg "confirm with the user before running" skills -g SKILL.md — 10 matches (workflow skills)
  • CI: Skills manifest + docs checks

Notes

Fixes #2613 (silent self-update portion only; shell-injection and SRI items remain open in the issue).

Replace "run silently, don't ask" with explicit confirmation guidance
in ten workflow SKILL.md files so agents do not auto-run npx updates
without the user. Regenerate skills-manifest.json.

Refs heygen-com#2613
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.

Skills instruct agents to silently self-update via npx with no user confirmation; one script has an unreachable shell-injection shape

1 participant