Skip to content

refactor(flags): migrate external skills flag#27685

Merged
nexxeln merged 1 commit into
devfrom
nxl/migrate-external-skills-flag
May 15, 2026
Merged

refactor(flags): migrate external skills flag#27685
nexxeln merged 1 commit into
devfrom
nxl/migrate-external-skills-flag

Conversation

@nexxeln

@nexxeln nexxeln commented May 15, 2026

Copy link
Copy Markdown
Member

summary

  • Add disableExternalSkills to RuntimeFlags.
  • Thread the runtime flag into skill discovery instead of reading the mutable Flag object.
  • Remove the obsolete core flag entry after confirming no remaining call sites.

testing

  • bun test test/skill/skill.test.ts
  • bun test test/effect/runtime-flags.test.ts
  • bun typecheck
  • pre-push hook: bun turbo typecheck

@nexxeln nexxeln merged commit 22cb039 into dev May 15, 2026
7 of 11 checks passed
@nexxeln nexxeln deleted the nxl/migrate-external-skills-flag branch May 15, 2026 07:54
rustybret pushed a commit to rustybret/opencode that referenced this pull request May 15, 2026
rustybret pushed a commit to rustybret/opencode that referenced this pull request May 15, 2026
rustybret pushed a commit to rustybret/opencode that referenced this pull request May 15, 2026
rustybret pushed a commit to rustybret/opencode that referenced this pull request May 15, 2026
rustybret pushed a commit to rustybret/opencode that referenced this pull request May 15, 2026
AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
avion23 pushed a commit to avion23/opencode that referenced this pull request Jun 10, 2026
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