fix: effect run result type (#182)

paritytech · Aug 9, 2022 · 52ea7d1 · 52ea7d1
1 parent b114a23
commit 52ea7d1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/effect/sys/Atom.ts b/effect/sys/Atom.ts
@@ -1,7 +1,6 @@
 import { E_, Effect, Resolved, T_, ValCollection } from "./Effect.ts";
 import { key } from "./key.ts";
-import { RunContext } from "./run.ts";
-import { run } from "./run.ts";
+import { run, RunContext, RunResult } from "./run.ts";
 
 export function atom<N extends string, A extends unknown[], R>(
   fqn: N,
@@ -24,7 +23,7 @@ export class Atom<N extends string, A extends unknown[], R>
     super(fqn);
   }
 
-  run() {
+  run(): RunResult<this> {
     return run(this);
   }
 }

diff --git a/effect/sys/run.ts b/effect/sys/run.ts
@@ -6,8 +6,8 @@ import { key } from "./key.ts";
 export interface RunContext {
   run: Run;
 }
-
-export type Run = <Root extends AnyAtom>(root: Root) => Promise<T_<Root> | E_<Root>>;
+export type RunResult<Root extends AnyAtom> = Promise<T_<Root> | E_<Root>>;
+export type Run = <Root extends AnyAtom>(root: Root) => RunResult<Root>;
 
 export const { run } = (new class Runtime implements RunContext {
   #cache = new Map<string, Promise<unknown>>(); // TODO: set max size / use LRU