Skip to content

Require MCP authorization in rust_verify - #5

Merged
kiranandcode merged 2 commits into
mainfrom
codex/direct-verus-mcp-gate
Sep 4, 2026
Merged

Require MCP authorization in rust_verify#5
kiranandcode merged 2 commits into
mainfrom
codex/direct-verus-mcp-gate

Conversation

@kiranandcode

Copy link
Copy Markdown
Collaborator

Summary

  • reject direct rust_verify invocation unless --mcp is present
  • consume the authorization flag before normal Verus argument parsing
  • preserve authorized cargo-verus, wrapper, trait-check, vstd-build, and test paths
  • update direct-invocation documentation

Tests

  • cargo test -p rust_verify --bin rust_verify
  • cargo test -p cargo-verus
  • cargo test -p vstd_build
  • cargo clippy -p rust_verify -p verus -p cargo-verus -p vstd_build --all-targets -- -D warnings
  • cargo fmt --all -- --check
  • manual rust_verify --version rejection and rust_verify --mcp --version acceptance

Assisted-by: Codex:GPT-5

@yiyunliu
yiyunliu self-requested a review September 4, 2026 14:37

@yiyunliu yiyunliu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm. will merge once the CI passes

@kiranandcode
kiranandcode merged commit 034d4d6 into main Sep 4, 2026
22 checks passed
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.

2 participants