chore(agent): trim mcp-manager dead exports and unused devDeps

.github/workflows/code-quality.yml

@@ -59,3 +59,26 @@ jobs:
 
       - name: Run Typecheck
         run: bun run typecheck
+
+  fallow:
+    name: runner / Fallow
+    runs-on: ubuntu-latest
+    defaults:
+      run:
+        working-directory: packages/browseros-agent
+    permissions:
+      contents: read
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v6
+        with:
+          persist-credentials: false
+
+      - name: Setup Bun
+        uses: oven-sh/setup-bun@v2
+
+      - name: Install dependencies
+        run: bun ci
+
+      - name: Run Fallow
+        run: bun fallow

packages/browseros-agent/.fallowrc.json

@@ -20,6 +20,8 @@
     "tw-animate-css",
     "@tailwindcss/typography",
     "pino-pretty",
+    "dotenv",
+    "picocolors",
     "@browseros/cdp-protocol",
     "@graphql-typed-document-node/core"
   ],

packages/browseros-agent/apps/agent/screens/mcp-settings/IntegrationsSection.tsx

@@ -16,7 +16,7 @@ import {
 } from './integrations-section.hooks'
 import { QuickSetupSection } from './QuickSetupSection'
 
-interface IntegrationsSectionProps {
+export interface IntegrationsSectionProps {
   serverUrl: string | null
 }
 

packages/browseros-agent/apps/agent/screens/mcp-settings/integrations-section.helpers.ts

@@ -24,7 +24,7 @@ export interface AgentPresentation {
   mark: FC<SVGProps<SVGSVGElement>>
 }
 
-export const AGENT_PRESENTATION: Record<string, AgentPresentation> = {
+const AGENT_PRESENTATION: Record<string, AgentPresentation> = {
   'claude-code': {
     label: 'Claude Code',
     installUrl: 'https://claude.ai/code',

packages/browseros-agent/apps/agent/screens/sidepanel/index/RemoteHermesBootPill.tsx

@@ -6,7 +6,7 @@ export interface RemoteHermesVmStatus {
   progress?: string
 }
 
-interface RemoteHermesBootPillProps {
+export interface RemoteHermesBootPillProps {
   vm: RemoteHermesVmStatus
 }
 

packages/browseros-agent/apps/server/src/lib/clients/remote-hermes/remote-hermes-client.ts

@@ -39,6 +39,7 @@ export interface PostTurnInput {
   modelId?: string | null
 }
 
+// fallow-ignore-next-line unused-type
 export interface PostTurnResult {
   taskId: string
 }

packages/browseros-agent/apps/server/src/lib/clients/remote-hermes/ws-bridge.ts

@@ -36,7 +36,7 @@ export interface WsBridgeDeps {
   resolveLocalMcpUrl(server: string): string | null
 }
 
-type SocketState = 'closed' | 'connecting' | 'open'
+export type SocketState = 'closed' | 'connecting' | 'open'
 
 const MODULE = 'remote-hermes'
 

packages/browseros-agent/apps/server/src/lib/mcp-manager/index.ts

@@ -5,23 +5,13 @@
  */
 
 export {
-  BROWSEROS_MCP_SERVER_NAME,
-  BROWSEROS_MCP_STDIO_SERVER_NAME,
-  getMcpManager,
   resetMcpManagerForTesting,
   setMcpManagerForTesting,
 } from './manager'
-export { type ReconcileUrlInput, reconcileUrl } from './reconcile'
+export { reconcileUrl } from './reconcile'
 export {
   humaniseInstallError,
   installInto,
   listAgents,
   uninstallFrom,
 } from './service'
-export type {
-  InstallAgentResult,
-  McpAgentId,
-  McpAgentRow,
-  ReconcileResult,
-  UninstallAgentResult,
-} from './types'

packages/browseros-agent/apps/server/src/lib/mcp-manager/service.ts

@@ -25,7 +25,6 @@ import {
 } from './manager'
 import type {
   InstallAgentResult,
-  McpAgentId,
   McpAgentRow,
   UninstallAgentResult,
 } from './types'
@@ -52,7 +51,7 @@ const STDIO_ONLY_AGENTS: ReadonlySet<string> = new Set(['codex'])
  * The two server-names BrowserOS manages in the manifest. Iterating
  * both is what `listAgents` + `reconcileUrl` need to do.
  */
-export const BROWSEROS_SERVER_NAMES: readonly string[] = [
+const BROWSEROS_SERVER_NAMES: readonly string[] = [
   BROWSEROS_MCP_SERVER_NAME,
   BROWSEROS_MCP_STDIO_SERVER_NAME,
 ]
@@ -63,7 +62,7 @@ interface AgentServerPlan {
 }
 
 /** Pick the server name + spec a given agent should be linked under. */
-export function planFor(agentId: string, currentUrl: string): AgentServerPlan {
+function planFor(agentId: string, currentUrl: string): AgentServerPlan {
   if (STDIO_ONLY_AGENTS.has(agentId)) {
     const spec: McpStdioSpec = {
       transport: 'stdio',
@@ -181,5 +180,3 @@ export function humaniseInstallError(err: unknown): {
   const message = err instanceof Error ? err.message : String(err)
   return { message, status: 500 }
 }
-
-export type McpAgentIdentifier = McpAgentId