mirror of
https://github.com/simstudioai/sim.git
synced 2026-08-31 01:11:53 +08:00
Waleed
167fcb483b
* refactor(orchestration): remove seven unreferenced perform* entry points Each had exactly one declaration, a barrel re-export, and no caller anywhere in the repo — no route, no application use case, no tool handler, no test. Their Params/Result interfaces went with them where nothing else consumed them; PerformCredentialResult, PerformUpdateWorkflowParams and PerformUpdateWorkflowResult stay, since live functions still use them. Removed: performDeleteCredential, performGetWorkspaceFileShare, performUpsertWorkspaceFileShare, performMoveRenameWorkspaceFile, performUpdateTableDescription, performUpdateWorkflow, performUpdateWorkspaceFileContent. * test: drop assertions that cannot fail Four tests asserted nothing about the code under test: - app/api/copilot/methods/route.test.ts was the directory's only file — it asserted expect(true).toBe(true) against a route that does not exist. - tools/index.test.ts carried a block self-documented as existing "to maintain test count". - mcp/storage/memory-cache.test.ts closed a delete-a-missing-key case with expect(true).toBe(true); it now asserts the call resolves without throwing. - realtime/src/index.test.ts checked typeof roomManager.x === 'function' and typeof process.on === 'function', both of which tsc already proves. Removing the realtime cases leaves a real gap: index.ts registers uncaughtException, unhandledRejection, SIGINT and SIGTERM handlers with no coverage. Better to have that gap visible than papered over by a test that would pass with the handlers deleted.