Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions src/everything/resources/subscriptions.ts
Original file line number Diff line number Diff line change
Expand Up @@ -166,3 +166,22 @@ export const stopSimulatedResourceUpdates = (sessionId?: string) => {
subsUpdateIntervals.delete(sessionId);
}
};

/**
* Removes a session from every URI's subscriber set, dropping any URI entry
* that ends up with no remaining subscribers.
*
* A session that disconnects without explicitly unsubscribing otherwise stays
* in `subscriptions` for the life of the process. Call this from the
* transport's `cleanup(sessionId)` when a session ends.
*
* @param {string} [sessionId]
*/
export const removeSubscriber = (sessionId?: string) => {
for (const [uri, subscribers] of subscriptions) {
subscribers.delete(sessionId);
if (subscribers.size === 0) {
subscriptions.delete(uri);
}
}
};
3 changes: 3 additions & 0 deletions src/everything/server/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import {
import {
setSubscriptionHandlers,
stopSimulatedResourceUpdates,
removeSubscriber,
} from "../resources/subscriptions.js";
import { registerConditionalTools, registerTools } from "../tools/index.js";
import { registerResources, readInstructions } from "../resources/index.js";
Expand Down Expand Up @@ -110,6 +111,8 @@ export const createServer: () => ServerFactoryResponse = () => {
// Stop any simulated logging or resource updates that may have been initiated.
stopSimulatedLogging(sessionId);
stopSimulatedResourceUpdates(sessionId);
// Drop this session from any resource subscriptions it left open.
removeSubscriber(sessionId);
// Clean up task store timers
taskStore.cleanup();
if (initializeTimeout) clearTimeout(initializeTimeout);
Expand Down