refactor(epicenter): enhance type safety by excluding internal properties in iterActions and adding type imports in createServer

Author: braden-w

packages/epicenter/src/core/epicenter/client.shared.ts

@@ -95,10 +95,11 @@ export function* iterActions<TWorkspaces extends readonly AnyWorkspaceConfig[]>(
 	for (const [workspaceId, workspaceClient] of Object.entries(
 		workspaceClients,
 	)) {
-		// Extract all exports (excluding cleanup methods from the workspace interface)
+		// Extract all exports (excluding cleanup methods and internal properties from the workspace interface)
 		const {
 			destroy: _workspaceDestroy,
 			[Symbol.asyncDispose]: _workspaceAsyncDispose,
+			$ydoc: _$ydoc,
 			...workspaceExports
 		} = workspaceClient as WorkspaceClient<WorkspaceExports>;
 

packages/epicenter/src/server/server.ts

@@ -2,13 +2,15 @@ import { openapi } from '@elysiajs/openapi';
 import { Elysia } from 'elysia';
 // import { mcp } from 'elysia-mcp';
 import { Err, isResult, Ok } from 'wellcrafted/result';
+import type { WorkspaceExports } from '../core/actions';
 import {
 	createEpicenterClient,
 	type EpicenterConfig,
 	iterActions,
 } from '../core/epicenter';
 import type {
 	AnyWorkspaceConfig,
+	WorkspaceClient,
 	WorkspacesToClients,
 } from '../core/workspace';
 import { createSyncPlugin } from './sync';
@@ -105,8 +107,11 @@ export async function createServer<
 			createSyncPlugin({
 				getDoc: (room) => {
 					// Room name is the workspace ID
-					const workspace =
-						client[room as keyof WorkspacesToClients<TWorkspaces>];
+					// Type assertion needed because TypeScript can't prove the generic
+					// WorkspacesToClients mapping resolves to WorkspaceClient
+					const workspace = client[
+						room as keyof WorkspacesToClients<TWorkspaces>
+					] as WorkspaceClient<WorkspaceExports> | undefined;
 					return workspace?.$ydoc;
 				},
 			}),