Skip to content

Commit 3f8d8fe

Browse files
chore(lint): disable returnEffectInGen
1 parent 86ee140 commit 3f8d8fe

6 files changed

Lines changed: 1 addition & 5 deletions

File tree

‎apps/server/src/observability/EventLoopMonitor.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,6 @@ const makeNodeSampler = Effect.gen(function* () {
7575
let usage = process.resourceUsage();
7676
let sampledAt = NodePerfHooks.performance.now();
7777

78-
// @effect-diagnostics-next-line returnEffectInGen:off - the read effect is the result.
7978
return Effect.sync(() => {
8079
const nextElu = NodePerfHooks.performance.eventLoopUtilization();
8180
const nextUsage = process.resourceUsage();

‎apps/server/src/orchestration-v2/LiveStreamBudget.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,6 @@ export const makeLiveStreamBudget = Effect.fn("makeLiveStreamBudget")(function*
172172
}),
173173
Effect.forkScoped,
174174
);
175-
// @effect-diagnostics-next-line returnEffectInGen:off - Stream.fromPull needs the pull effect as its result.
176175
return Effect.gen(function* () {
177176
// RpcServer requests the next batch after ACK; dequeueing does not end delivery.
178177
release(inFlight);

‎apps/server/src/orchestration-v2/ShellStream.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -384,7 +384,6 @@ export function composeShellStreamWithEnrichment<A, E, R, A2, E2, R2, A3, E3, R3
384384
let previous: Deferred.Deferred<void> | undefined;
385385
// Stream.merge pulls ahead after handing off each batch. Wait for the next
386386
// downstream pull so shell delivery keeps that batch charged until its RPC ACK.
387-
// @effect-diagnostics-next-line returnEffectInGen:off - Stream.fromPull needs the pull effect as its result.
388387
return Effect.gen(function* () {
389388
if (previous) yield* Deferred.succeed(previous, undefined);
390389
const next = yield* Queue.take(output);

‎apps/server/src/ws.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3366,7 +3366,6 @@ export const layer = Layer.unwrap(
33663366
),
33673367
Effect.forkScoped,
33683368
);
3369-
// @effect-diagnostics-next-line returnEffectInGen:off
33703369
return httpEffect;
33713370
}).pipe(
33723371
Effect.provide(

‎infra/relay/src/hooks/HookInboxObject.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,6 @@ export const layer = HookInboxObject.make(
7575
Effect.gen(function* () {
7676
const state = yield* Cloudflare.DurableObjectState;
7777
// Only the per-object inner Effect may access storage.
78-
// @effect-diagnostics-next-line returnEffectInGen:off
7978
return Effect.gen(function* () {
8079
const layerSql = SqliteClient.layer({ storage: state.raw.storage });
8180
const inboxId = state.raw.id.name ?? state.raw.id.toString();

‎tsconfig.base.json‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@
4242
"flatMapIgnoredParamToAndThen": "error",
4343
"mapSomeToAsSome": "error",
4444
"unnecessaryPipeChain": "error",
45+
"returnEffectInGen": "off",
4546
"effectFnOpportunity": "suggestion",
4647
"anyUnknownInErrorContext": "error",
4748
"unsafeEffectTypeAssertion": "error",

0 commit comments

Comments
 (0)