Skip to content

fix: harden managed skill installs#905

Open
Damin-Lee wants to merge 1 commit intogarrytan:mainfrom
Damin-Lee:fix/managed-skill-install-safety
Open

fix: harden managed skill installs#905
Damin-Lee wants to merge 1 commit intogarrytan:mainfrom
Damin-Lee:fix/managed-skill-install-safety

Conversation

@Damin-Lee
Copy link
Copy Markdown

Summary

  • stop mutating source SKILL.md files when relinking Claude installs by rendering managed copies instead
  • allow generated external-host output to live outside the source tree and update setup/relink tests accordingly
  • fix Bun 1.3 node-server bundling by externalizing @ngrok/ngrok and make uninstall only report SessionStart hook removal when it actually removed one

Verification

  • bun test
  • bun run build

@Damin-Lee
Copy link
Copy Markdown
Author

Mirrored this patch into my fork for local validation and manual merge flow: https://github.com/Damin-Lee/gstack/pull/1\n\nVerification on the patch branch:\n- bun test\n- bun run build\n\nOnce upstream review lands, I can switch the hac submodule pointer back to upstream main cleanly.

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.

1 participant