soulcraft/brainy
2
0
Fork 0
Code Issues 1 Pull requests Projects Releases 335 Packages Wiki Activity Actions

**refactor(mcpService): use enum for MCP request type**

Browse source 
### Changes:
- **mcpService.ts**:
  - Updated `type` in `/mcp/tools` route to use `MCPRequestType.SYSTEM_INFO` enum instead of hardcoded string `'SYSTEM_INFO'`.
  - Removed redundant `infoType: 'availableTools'` from the MCP request payload.
  - Added `MCPRequestType` import from `@soulcraft/brainy`.

### Purpose:
Replaced hardcoded request type string with an enum for improved type safety and maintainability. Removed unnecessary `infoType` parameter, aligning the request payload with updated requirements.
This commit is contained in:
David Snelling 2025-06-23 10:58:56 -07:00
parent 37c4da58f7
commit 36336c5180
1 changed files with 3 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

7
cloud-wrapper/src/services/mcpService.ts
View file

@ -1,7 +1,7 @@
import { WebSocketServer } from 'ws' import { WebSocketServer } from 'ws'
import express from 'express' import express from 'express'
import cors from 'cors' import cors from 'cors'
import { BrainyData, BrainyMCPService } from '@soulcraft/brainy' import { BrainyData, BrainyMCPService, MCPRequestType } from '@soulcraft/brainy'
import { v4 as uuidv4 } from 'uuid' import { v4 as uuidv4 } from 'uuid'
/** /**
@ -177,10 +177,9 @@ function startRESTServer(
app.get('/mcp/tools', async (req: any, res: any) => { app.get('/mcp/tools', async (req: any, res: any) => {
try { try {
const response = await mcpService.handleMCPRequest({ const response = await mcpService.handleMCPRequest({
type: 'SYSTEM_INFO', type: MCPRequestType.SYSTEM_INFO,
requestId: uuidv4(), requestId: uuidv4(),
version: '1.0', version: '1.0'
infoType: 'availableTools'
}) })
res.json(response) res.json(response)