Skip to content

Commit 701fd04

Browse files
committed
Enhancement: Clarify Memory Core healthcheck response for hybrid database strategy #7690
1 parent 59c3dba commit 701fd04

2 files changed

Lines changed: 13 additions & 1 deletion

File tree

ai/mcp/server/memory-core/openapi.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -514,6 +514,11 @@ components:
514514
managed:
515515
type: boolean
516516
example: true
517+
strategy:
518+
type: string
519+
enum: [managed, external, unknown]
520+
description: "Indicates how the ChromaDB process is managed (e.g., by this service or externally)."
521+
example: "managed"
517522
connection:
518523
type: object
519524
properties:

ai/mcp/server/memory-core/services/HealthService.mjs

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -265,10 +265,17 @@ class HealthService extends Base {
265265

266266
// If we made it here with no errors, report success
267267
if (payload.status === 'healthy') {
268-
payload.details.push('ChromaDB is running and all collections are accessible');
268+
if (payload.database.process.managed) {
269+
payload.details.push('Connected to a server-managed ChromaDB instance');
270+
} else {
271+
payload.details.push('Connected to an externally managed ChromaDB instance');
272+
}
269273
payload.details.push('All features are operational');
270274
}
271275

276+
// Add strategy for clarity
277+
payload.database.process.strategy = payload.database.process.managed ? 'managed' : 'external';
278+
272279
return payload;
273280
}
274281

0 commit comments

Comments
 (0)