Skip to content

fix(skill): add missing parameters#50

Merged
esifea merged 2 commits into
CryptoLabInc:mainfrom
esifea:fix/esifea/fix-activate
Apr 6, 2026
Merged

fix(skill): add missing parameters#50
esifea merged 2 commits into
CryptoLabInc:mainfrom
esifea:fix/esifea/fix-activate

Conversation

@esifea

@esifea esifea commented Apr 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • What changed: Added missing parameters eval_mode and query_encryption in the activate skill
  • Why: Error occurred on /rune:activate
  • Scope: commands/claude/activate.md

Validation

  • Tests run (or explain why not):
  • Docs updated (if behavior/setup changed)

Cross-Agent Invariants

  • scripts/bootstrap-mcp.sh remains the single source of truth for runtime prep (venv/deps/self-heal)
  • No agent-specific script duplicates bootstrap/setup logic
  • Agent-specific scripts remain thin adapters (registration/wiring only)
  • Codex-only commands (codex mcp ...) are clearly separated from cross-agent/common instructions
  • Claude/Gemini/OpenAI instructions do not include Codex-only commands
  • SKILL.md, commands/rune/*.toml, and AGENT_INTEGRATION.md stay consistent on boundaries

Notes for Reviewers

  • Risk areas:
  • Backward compatibility impact:
  • Follow-up work (if any):

@esifea esifea self-assigned this Apr 2, 2026
@esifea
esifea requested a review from sunchuljung April 2, 2026 02:44
@esifea esifea linked an issue Apr 2, 2026 that may be closed by this pull request
Comment thread commands/claude/activate.md Outdated
Comment thread commands/claude/activate.md Outdated

@sunchuljung sunchuljung left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@esifea
esifea merged commit 88307f5 into CryptoLabInc:main Apr 6, 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.

/rune:activate shows 'missing parameters' error in progress

2 participants