Skip to content

Feature/built in tools#61

Merged
Gerome-Elassaad merged 3 commits into
mainfrom
feature/built-in-tools
Dec 30, 2025
Merged

Feature/built in tools#61
Gerome-Elassaad merged 3 commits into
mainfrom
feature/built-in-tools

Conversation

@Gerome-Elassaad

@Gerome-Elassaad Gerome-Elassaad commented Dec 30, 2025

Copy link
Copy Markdown
Member

Key Design Decisions

  1. Followed MCP Pattern: Used MCPService as reference for architecture consistency
  2. Server/Client Split: Separated tools by execution environment (server vs WebContainer)
  3. Gradual Rollout: Enabled 3 server-side tools first, leaving client-side for future PR
  4. Tool Registry: Centralized tool definitions for easy management
  5. No Breaking Changes: All existing functionality preserved, new tools are additive
  6. Comprehensive Prompts: Updated both prompt variants to guide AI tool usage

@Gerome-Elassaad Gerome-Elassaad self-assigned this Dec 30, 2025
@Gerome-Elassaad Gerome-Elassaad added the feature Requested feature or new feature label Dec 30, 2025
@Gerome-Elassaad Gerome-Elassaad linked an issue Dec 30, 2025 that may be closed by this pull request
add polish & integration #47
Open
@Gerome-Elassaad Gerome-Elassaad removed a link to an issue Dec 30, 2025
add polish & integration #47
Open
@Gerome-Elassaad Gerome-Elassaad merged commit cb81160 into main Dec 30, 2025
2 checks passed
@Gerome-Elassaad Gerome-Elassaad deleted the feature/built-in-tools branch January 3, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@Gerome-Elassaad Gerome-Elassaad

Labels

feature Requested feature or new feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Gerome-Elassaad