Skip to content

Fix constrained deny-list tool declarations - #40

Merged
trotsky1997 merged 1 commit into
mainfrom
agent/fix-22-constrained-deny-tools
Jun 2, 2026
Merged

Fix constrained deny-list tool declarations#40
trotsky1997 merged 1 commit into
mainfrom
agent/fix-22-constrained-deny-tools

Conversation

@trotsky1997

Copy link
Copy Markdown
Owner

Fixes #22.

Summary

  • keep constrained allow-list declarations fail-closed so they do not expose the whole tool
  • make constrained deny-list declarations fail closed by blocking the matching head tool name
  • add a slash-runner regression test for disallowed-tools: Bash(git push:*)

Testing

  • cargo fmt --check
  • git diff --check -- src/session.rs tests/session_runner_slash.rs
  • attempted cargo test --test session_runner_slash constrained, but local dependency fetch for https://github.com/Dicklesworthstone/opentui_rust timed out/disconnected with fetch-pack: unexpected disconnect while reading sideband packet

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sorry @trotsky1997, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@trotsky1997
trotsky1997 merged commit c80e409 into main Jun 2, 2026
6 checks passed
@trotsky1997
trotsky1997 deleted the agent/fix-22-constrained-deny-tools branch June 2, 2026 09:45
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.

Tighten deny-list handling for constrained tool declarations

1 participant