refactor: delete orphan modules

TA
5h ago
+5-626

Proposed changes (including videos or screenshots)

Deletes modules in apps/meteor that nothing imports. Each was found with an esbuild reachability graph from the client and server entry points (plus every spec, story and test file as an extra entry), then confirmed by grepping the repo for the exported symbol.

Deleted, unreachable from any entry point:

  • client/lib/utils/callWithErrorHandling.ts, client/lib/utils/timeAgo.ts
  • server/lib/cloud/{checkUserHasCloudLogin,finishOAuthAuthorization,getOAuthAuthorizationUrl,getWorkspaceKey,startRegisterWorkspace}.ts
  • server/lib/http/call.ts, server/lib/integrations/lib/compileIntegrationScript.ts
  • server/lib/users/{getStatusText,gravatarDeprecation}.ts, server/lib/utils/functions/isDocker.ts
  • ee/server/sdk/types/index.ts (type barrel; callers import from the source files)

Narrowed:

  • engagementDashboard/dataView/colors.ts was only used by LegendSymbol.stories.tsx; inlined into the story.
  • OutboundMessageWizard/forms/index.ts re-exported runtime modules but its two importers only needed SubmitPayload; replaced by forms/SubmitPayload.ts.

Issue(s)

Steps to test or reproduce

No behavior change. Typecheck and unit tests cover it.

Further comments

Left out on purpose:

  • ee/server/patches/fetchContactHistory.ts is also never imported, but it is a missing wire-up rather than dead code; it is fixed in fix/contact-history-source-filter.
  • server/lib/cloud/userLogout.ts is only reached from its own test; removing it with its test is a separate change.

🤖 Generated with Claude Code

Task: ARCH-2585

Summary by CodeRabbit

  • Changes
    • Several legacy helper functions have been removed, including utilities for cloud workspace registration and authorization, date formatting, and server-side HTTP requests.
    • No changes to end-user behavior are specified in this update.