Skip to content

chore(api): drop unwired execute predicates and vault live check - #90

Draft
tomdmeredith wants to merge 1 commit into
mainfrom
cursor/delete-unwired-execute-predicates-9947
Draft

tomdmeredith wants to merge 1 commit into
mainfrom
cursor/delete-unwired-execute-predicates-9947

Conversation

@tomdmeredith

@tomdmeredith tomdmeredith commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Deletes dead direct-execute predicates and the Agent Vault live token check. None of these symbols have callers, including tests. Request behavior is unchanged: the dispatcher in capability_execute.py still owns the capability frozensets, and the ensure_* allow-gates stay.

  • Route files: remove *_CAPABILITY_IDS and is_*_capability from actions, crm, db, warehouse, deployment, storage, and support execute modules.
  • Connection registries: remove unused boolean twins repository_is_allowed, object_type_is_allowed, and dataset_is_allowed. Keep ensure_repository_allowed, ensure_object_type_allowed, ensure_dataset_allowed, and table_is_allowed.
  • agent_vault.py: remove verify_live and the unused httpx import. Callers keep using validate_format.

Test plan

  • Relevant API tests: 228 passed. Covered Agent Vault, the three connection registries, the seven direct-execute routes, and test_capability_execute.py.
  • The two test_agent_vault.py failures (requires_token, requires_provider, KeyError: 'detail') also fail on main and do not call the deleted helpers.
  • Dispatcher frozensets in packages/api/routes/capability_execute.py are untouched.
Open in Web Open in Cursor 

These helpers have no callers. Dispatcher capability sets and ensure_* gates stay in place.

Co-authored-by: tomdmeredith <tomdmeredith@users.noreply.github.com>
@vercel

vercel Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
rhumb Ready Ready Preview Sep 30, 2026 11:30am UTC

Request Review

This branch was successfully deployed

1 active deployment
Preview — c6e7f128 Deployed Sep 30, 2026 by vercel[bot]
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