Frontmatter
| id | 7868 |
| title | Implement consistent error handling for Memory Core MCP tools |
| state | Closed |
| labels | enhancementai |
| assignees | tobiu |
| createdAt | Nov 22, 2025, 10:12 PM |
| updatedAt | Nov 22, 2025, 10:20 PM |
| githubUrl | https://github.com/neomjs/neo/issues/7868 |
| author | tobiu |
| commentsCount | 0 |
| parentIssue | null |
| subIssues | [] |
| subIssuesCompleted | 0 |
| subIssuesTotal | 0 |
| contentTrust | |
| projected | |
| quarantined | 0 |
| signals | [] |
| blockedBy | [] |
| blocking | [] |
| closedAt | Nov 22, 2025, 10:20 PM |
The following tools in
ai/mcp/server/memory-core/services/toolService.mjslack top-leveltry-catchblocks to ensure a consistent error shape for the MCP server:Services to Update:
MemoryService.mjs:addMemory,listMemories,queryMemoriesSummaryService.mjs:deleteAllSummaries,listSummaries,querySummariesDatabaseService.mjs:exportDatabase,importDatabaseDatabaseLifecycleService.mjs:startDatabase,stopDatabaseRequired Error Shape:
{ error : 'Description of the error', message: error.message, code : 'ERROR_CODE' // e.g., MEMORY_ADD_ERROR, DATABASE_EXPORT_ERROR }Note:
SessionService.summarizeSessionsalready implements this pattern.HealthService.healthcheckhandles errors internally to return a health status, but should be reviewed to ensure it doesn't throw raw exceptions.Task: Wrap the logic of the listed methods in
try-catchblocks and return the standardized error object on failure.