Skip to content

fix: remove unarchive, get_features, and clarify make_call scope#18

Merged
maharshi-smallest merged 1 commit into
mainfrom
fix/tool-cleanup
Apr 27, 2026
Merged

fix: remove unarchive, get_features, and clarify make_call scope#18
maharshi-smallest merged 1 commit into
mainfrom
fix/tool-cleanup

Conversation

@maharshi-smallest

Copy link
Copy Markdown
Collaborator

Summary

  • delete_agent: Removed unarchive parameter — unarchiving is not supported, users directed to app.smallest.ai
  • get_features: Removed tool registration entirely
  • make_call: Updated description to clarify only outbound telephony is supported; webcall/chat should use app.smallest.ai

Test plan

  • Verify delete_agent no longer accepts unarchive param
  • Verify get_features tool is no longer listed
  • Verify make_call description reflects outbound-only support

🤖 Generated with Claude Code

…ll scope

- delete_agent: remove unarchive option, direct users to app.smallest.ai
- get_features: remove tool registration (not needed for MCP users)
- make_call: clarify only outbound telephony is supported

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@maharshi-smallest maharshi-smallest merged commit 9dd7f95 into main Apr 27, 2026
1 check passed
@entelligence-ai-pr-reviews

Copy link
Copy Markdown

EntelligenceAI PR Summary

This PR restricts and clarifies tool capabilities across the MCP server toolset.

  • src/tools/delete-agent.ts: Dropped unarchive parameter and ?on=false query string; delete_agent is now archive-only with updated messaging pointing restores to the web app
  • src/tools/index.ts: Removed registerGetFeatures import and registration, deprecating the 'get features' tool entirely
  • src/tools/make-call.ts: Clarified make_call tool description to explicitly limit support to telephony outbound calls, redirecting webcall/chat use cases to app.smallest.ai

Confidence Score: 5/5 - Safe to Merge

Safe to merge — this PR cleanly restricts tool capabilities by removing the unarchive parameter from delete-agent.ts, deprecating registerGetFeatures in index.ts, and clarifying the make_call scope in make-call.ts. The changes are additive in the conservative direction (reducing surface area rather than expanding it), and no review comments were generated across any of the three changed files. The PR achieves its stated goal of scoping down and clarifying tool behavior without introducing new logic paths or runtime risks.

Key Findings:

  • Removal of the unarchive parameter and ?on=false query string from delete-agent.ts simplifies the archive-only path and eliminates a potentially confusing dual-mode operation, with no new logic introduced.
  • Dropping registerGetFeatures from index.ts is a clean deprecation — removing an import and registration call with zero residual references that could cause runtime errors.
  • The make_call description clarification in make-call.ts is a documentation-only change scoped to telephony outbound calls, carrying no runtime or logic impact.
  • All three changed files received full review coverage and generated zero comments, indicating no identified issues across logic, security, or correctness dimensions.
Files requiring special attention
  • src/tools/delete-agent.ts
  • src/tools/index.ts
  • src/tools/make-call.ts

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