Skip to content

chore(commands): rewrite /rune:* slash commands for v0.4 Go binary#122

Merged
couragehong merged 1 commit into
feat/go-migrationfrom
couragehong/chore/v04-commands-rewrite
May 8, 2026
Merged

chore(commands): rewrite /rune:* slash commands for v0.4 Go binary#122
couragehong merged 1 commit into
feat/go-migrationfrom
couragehong/chore/v04-commands-rewrite

Conversation

@couragehong
Copy link
Copy Markdown
Contributor

Sweeps v0.3 Python-era leftovers from the 9 commands/claude/*.md files the plugin manifest exposes as user-facing slash commands. The Python MCP server, .venv, install.sh, and mcp__plugin_rune_envector__ tool prefix are all gone in v0.4 — keeping their references in the docs sent users to dead paths.

Changes per file

activate.md (largest rewrite):

  • drop steps 4 (.venv check + auto-install.sh), 5a (Python TCP probe
    • Python token validation via VaultClient import), 5b (.venv sanity check)
  • drop "restart Claude Code as fallback" — Task Rune Benchmark Framework including: #28 (fix(lifecycle): re-spawn boot loop on dormant reload #117) made Manager.Retrigger re-spawn the boot loop on dormant, so reload is sufficient
  • reshape Steps 5–8 around mcp__envector__{reload_pipelines, diagnostics, vault_status} — the MCP tools the v0.4 server already exposes for this purpose
  • allowed-tools narrowed accordingly (no python3, no scripts/, no curl, no find)

capture.md, recall.md, delete.md, history.md:

  • mcp__plugin_rune_envector__* → mcp__envector__* (server name in .claude-plugin/plugin.json is "envector"; the "plugin_rune_" prefix is from an earlier plugin layout that no longer exists)
  • drop Bash(python3:*) from allowed-tools (capture/recall)

reset.md:

  • drop Bash(python3:*) from allowed-tools

deactivate.md:

configure.md:

status.md:

  • mcp__rune__diagnostics / mcp__rune__vault_status → mcp__envector__*
  • drop the Infrastructure section's "Python venv" line — diagnostics already covers what's actually meaningful in v0.4
  • drop Bash(python3:) and Bash(scripts/) from allowed-tools

Sweeps v0.3 Python-era leftovers from the 9 commands/claude/*.md files
the plugin manifest exposes as user-facing slash commands. The Python
MCP server, .venv, install.sh, and mcp__plugin_rune_envector__ tool
prefix are all gone in v0.4 — keeping their references in the docs
sent users to dead paths.

Changes per file
----------------

activate.md (largest rewrite):
  - drop steps 4 (.venv check + auto-install.sh), 5a (Python TCP probe
    + Python token validation via VaultClient import), 5b (.venv
    sanity check)
  - drop "restart Claude Code as fallback" — Task #28 (#117) made
    Manager.Retrigger re-spawn the boot loop on dormant, so reload
    is sufficient
  - reshape Steps 5–8 around mcp__envector__{reload_pipelines,
    diagnostics, vault_status} — the MCP tools the v0.4 server already
    exposes for this purpose
  - allowed-tools narrowed accordingly (no python3, no scripts/, no
    curl, no find)

capture.md, recall.md, delete.md, history.md:
  - mcp__plugin_rune_envector__* → mcp__envector__* (server name in
    .claude-plugin/plugin.json is "envector"; the "plugin_rune_" prefix
    is from an earlier plugin layout that no longer exists)
  - drop Bash(python3:*) from allowed-tools (capture/recall)

reset.md:
  - drop Bash(python3:*) from allowed-tools

deactivate.md:
  - drop "MCP pipelines will remain live until session restart" — same
    Task #28 reason as activate.md

configure.md:
  - drop the "Known issue: ReloadPipelines doesn't yet re-spawn
    RunBootLoop ... restart Claude Code" block (Task #28 fix landed)

status.md:
  - mcp__rune__diagnostics / mcp__rune__vault_status → mcp__envector__*
  - drop the Infrastructure section's "Python venv" line — diagnostics
    already covers what's actually meaningful in v0.4
  - drop Bash(python3:*) and Bash(scripts/*) from allowed-tools

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@couragehong couragehong self-assigned this May 8, 2026
@couragehong couragehong merged commit 818d587 into feat/go-migration May 8, 2026
1 check passed
@couragehong couragehong deleted the couragehong/chore/v04-commands-rewrite branch May 8, 2026 00:16
couragehong added a commit that referenced this pull request May 8, 2026
The Go rune-mcp at internal/* + cmd/rune-mcp/ has reached parity (PR
#102 + #110 + #117) and end-to-end verification (#118, #122, #124),
which means the Python tree is now dead weight. Carrying both
implementations is actively misleading: a fresh contributor following
the in-repo install instructions would still land in mcp/ + agents/
and try to set up a venv that no longer ships, and parity audits keep
re-discovering the Python source instead of treating the Go side as
the source of truth.

Removed
-------

  agents/common/        12 files  (config, embedding, llm, schemas)
  agents/retriever/      4 files  (query_processor, searcher, synthesizer)
  agents/scribe/        12 files  (detector, llm_extractor, handlers, server)
  agents/tests/         16 files  (pytest suite)
  agents/__init__.py
  agents/README.md            (Python agents intro — gone with the impl)
  agents/SLACK_SETUP.md       (Slack notifier setup for Python scribe)
  mcp/                  18 files  (Python adapter + server + tests)
  requirements.txt            (root Python dependency list)
  scripts/migrate_embeddings.py (one-off Python migration helper)

  Total: 67 files, 17,815 lines.

Kept (intentional)
------------------

  agents/claude/{scribe,retriever}.md   referenced by .claude-plugin/
                                        plugin.json — agent prompts that
                                        the runtime loads
  agents/codex/scribe.md                Codex-side agent prompt
  agents/gemini/{scribe,retriever}.md   Gemini-side agent prompts
  benchmark/                            deferred (separate decision —
                                        rewrite in Go vs delete entirely)
  docs/v04/spec/python-mapping.md       parity blueprint that maps Python
                                        source to Go destinations; useful
                                        as a historical record post-deletion
  docs/migration/*.md                   migration plan + audit trail —
                                        intentional history
  scripts/bootstrap-mcp.sh and other    referenced by gemini-extension.json;
    Python-era shell scripts            removal blocked on Gemini support
                                        decision (separate PR)

Not in scope
------------

  .github/workflows/pr-tests.yml + pr-comment.yml — Python pytest CI;
    handled by PR #125 (ci-drop-python).
  README.md, CLAUDE.md, SKILL.md, AGENT_INTEGRATION.md, GEMINI.md,
    CONTRIBUTING.md — top-level docs still describe the v0.3 install
    flow; rewrite scheduled separately so this commit stays focused
    on code deletion.

Verification
------------

  go build ./...   passes
  go vet ./...     passes
  go test ./...    full suite passes (no test referenced deleted paths)
  grep across remaining .{go,md,json,sh,toml,yml,yaml} for the deleted
    paths returned zero hits — no dangling references.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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