Skip to content

fix: trust boundary followup — TOML key typo + missing meta commands#625

Merged
pszymkowiak merged 1 commit intodevelopfrom
fix/trust-boundary-followup
Mar 16, 2026
Merged

fix: trust boundary followup — TOML key typo + missing meta commands#625
pszymkowiak merged 1 commit intodevelopfrom
fix/trust-boundary-followup

Conversation

@pszymkowiak
Copy link
Copy Markdown
Collaborator

Summary

  • Fix [filter.[filters. in print_risk_summary() — was counting wrong TOML key (singular vs plural)
  • Add trust, untrust, session, rewrite to RTK_META_COMMANDS — these were falling through to raw execution on parse error instead of showing Clap help
  • Update test_meta_commands_reject_bad_flags to skip rewrite/session (trailing_var_arg, like proxy)

Follows up on PR #623 (SA-2025-RTK-002 trust boundary).

Test plan

  • cargo test trust — 10 tests pass
  • cargo test test_meta_commands — pass
  • cargo test — 932 pass, 0 fail

- Fix `[filter.` → `[filters.` in risk summary (matches actual TOML DSL format)
- Add trust, untrust, session, rewrite to RTK_META_COMMANDS array
- Update test to skip trailing_var_arg commands (rewrite, session)

Follows up on PR #623 (SA-2025-RTK-002 trust boundary).

Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
@pszymkowiak pszymkowiak merged commit 8d8e188 into develop Mar 16, 2026
8 of 9 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.

1 participant