chore(server): address effect review feedback

t3-code[bot] committed 3 months ago

Commit dbd81891 parent b5627231 file · +7 −5Browse files

Unified · Split

           }
           // A completed assistant message only settles the turn once the
           // session is no longer running it. Providers may emit several
-          // assistant messages per turn, and the turn must stay unsettled
+          // assistant messages per turn (commentary between tool calls), and
-          // until the provider reports turn end.
+          // the turn must stay unsettled until the provider reports turn end
+          // (projected as thread.session-set leaving the "running" status).
           const session = yield* projectionThreadSessionRepository.getByThreadId({
             threadId: event.payload.threadId,
           });
           Effect.provideService(Path.Path, path),
           Effect.provideService(ServerConfig, serverConfig),
           Effect.asVoid,
-          Effect.catchTag("SqlError", (sqlError) =>
+          Effect.catchTags({
-            Effect.fail(toPersistenceSqlError("ProjectionPipeline.projectEvent:query")(sqlError)),
+            SqlError: (sqlError) =>
-          ),
+              Effect.fail(toPersistenceSqlError("ProjectionPipeline.projectEvent:query")(sqlError)),
+          }),
         );
     };