didericis/bot-bottle
1
0
Fork 0
Code Issues 7 Pull Requests 15 Actions Packages Projects Releases Wiki Activity

Remove egress-block MCP tool and runtime route-mutation #202

Merged
didericis merged 1 commits from remove-egress-block-tool into main 2026-06-07 10:00:07 -04:00
Conversation 1 Commits 1 Files Changed 8
+63 -668
didericis-claude commented 2026-06-06 12:42:12 -04:00
Collaborator
Copy Link
Copy Source

Closes #198.

Summary

  • Removes the egress-block tool definition from supervise_server.py
  • Removes _merge_single_route, add_route, and apply_routes_change from egress_apply.py; keeps fetch_current_routes and validate_routes_content
  • Removes the egress-block proposal/approve/reject path from cli/supervise.py, including the add_route import and EgressApplyError from ApplyError
  • Removes TOOL_EGRESS_BLOCK from supervise.py TOOLS tuple and clears COMPONENT_FOR_TOOL (no tools write audit entries now)
  • Updates test_egress_apply.py, test_supervise_cli.py, test_supervise_server.py, and test_supervise.py

Merge rule(s)

Breaking — removes a public MCP tool. Agents using egress-block will receive "unknown tool" errors after this ships.

Closes #198. ## Summary - Removes the `egress-block` tool definition from `supervise_server.py` - Removes `_merge_single_route`, `add_route`, and `apply_routes_change` from `egress_apply.py`; keeps `fetch_current_routes` and `validate_routes_content` - Removes the egress-block proposal/approve/reject path from `cli/supervise.py`, including the `add_route` import and `EgressApplyError` from `ApplyError` - Removes `TOOL_EGRESS_BLOCK` from `supervise.py` TOOLS tuple and clears `COMPONENT_FOR_TOOL` (no tools write audit entries now) - Updates `test_egress_apply.py`, `test_supervise_cli.py`, `test_supervise_server.py`, and `test_supervise.py` ## Merge rule(s) Breaking — removes a public MCP tool. Agents using `egress-block` will receive "unknown tool" errors after this ships.
didericis added the Compat/Breaking label 2026-06-06 12:43:08 -04:00
didericis approved these changes 2026-06-06 12:43:49 -04:00
didericis force-pushed remove-egress-block-tool from c6080be4f4 to e43f364d94 2026-06-06 12:43:59 -04:00 Compare
didericis removed the Compat/Breaking label 2026-06-06 12:44:55 -04:00
didericis force-pushed remove-egress-block-tool from e43f364d94 to c12fc1c098 2026-06-06 16:20:01 -04:00 Compare
didericis force-pushed remove-egress-block-tool from c12fc1c098 to b07f7933ed 2026-06-07 09:50:51 -04:00 Compare
didericis added 1 commit 2026-06-07 09:56:46 -04:00
feat(supervise)!: remove egress-block MCP tool and runtime route-mutation
lint / lint (push) Successful in 1m39s
Details
test / unit (push) Successful in 40s
Details
test / integration (push) Successful in 1m1s
Details
test / unit (pull_request) Successful in 40s
Details
Update Quality Badges / update-badges (push) Successful in 1m45s
Details
test / integration (pull_request) Successful in 57s
Details
dc837a5400 
Drops `egress-block` from the supervise sidecar, removes
`_merge_single_route`, `add_route`, and `apply_routes_change` from
egress_apply.py, and strips the proposal/approve/reject flow for egress
from the supervise CLI. The list-egress-routes and capability-block tools
are unaffected. Tests updated throughout.

Closes #198
didericis force-pushed remove-egress-block-tool from b07f7933ed to dc837a5400 2026-06-07 09:56:46 -04:00 Compare
didericis merged commit dc837a5400 into main 2026-06-07 10:00:07 -04:00
didericis deleted branch remove-egress-block-tool 2026-06-07 10:00:08 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
didericis
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
didericis-claude (didericis (claude)) didericis-codex (didericis (codex)) didericis-gemma (gemma) didericis
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: didericis/bot-bottle#202
Delete Branch "remove-egress-block-tool"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?