Remove running state

Signed-off-by: gearnode <bryan@frimin.fr>

src/client.ts

@@ -19,7 +19,6 @@ export interface FetchExecutionRequest {
 }
 
 export type ExecutionState =
-  | "running"
   | "created"
   | "failed"
   | "succeed"

src/getExecution.ts

@@ -13,9 +13,8 @@ export async function getExecution(
   if (response)
     return Promise.resolve({
       ...response,
-      // TODO: bump a major and drop `running` state (invalid API state)
-      state: response.state === "started" ? "running" : response.state,
+      state: response.state,
     });
 
-  throw new APIError("execution not found", "");
+  throw new APIError("execution not found", "not found");
 }

src/index.ts

@@ -252,7 +252,7 @@ export const defer: Defer = (fn, options) => {
       console.log(`[defer.run][${fn.name}] defer ignore, no token found.`);
 
     const id = randomUUID();
-    __database.set(id, { id: id, state: "running" });
+    __database.set(id, { id: id, state: "started" });
     execLocally(id, fn, functionArguments);
     return { id };
   };
@@ -335,7 +335,7 @@ export const delay: DeferDelay = (deferFn, delay) => {
       console.log(`[defer.run][${fn.name}] defer ignore, no token found.`);
 
     const id = randomUUID();
-    __database.set(id, { id: id, state: "running" });
+    __database.set(id, { id: id, state: "started" });
     execLocally(id, fn, functionArguments);
     return { id };
   };
@@ -404,7 +404,7 @@ export const addMetadata: DeferAddMetadata = (deferFn, metadata) => {
       console.log(`[defer.run][${fn.name}] defer ignore, no token found.`);
 
     const id = randomUUID();
-    __database.set(id, { id: id, state: "running" });
+    __database.set(id, { id: id, state: "started" });
     execLocally(id, fn, functionArguments);
     return { id };
   };
@@ -456,7 +456,7 @@ export const awaitResult: DeferAwaitResult =
       response = await waitExecutionResult(__httpClient, { id: id });
     } else {
       const id = randomUUID();
-      __database.set(id, { id: id, state: "running" });
+      __database.set(id, { id: id, state: "started" });
       response = await execLocally(id, fn, functionArguments);
     }
 

src/next/useDeferRoute.ts

@@ -77,7 +77,7 @@ export const useDeferRoute = <
   return [
     execute,
     {
-      loading: !!status && ["started", "created", "running"].includes(status),
+      loading: !!status && ["started", "created"].includes(status),
       result,
       error,
     },

tests/awaitResult.spec.ts

@@ -31,7 +31,7 @@ describe("awaitResult(deferFn)", () => {
           })
           // second `/runs/:id` call
           .mockImplementationOnce(() => {
-            return { id: "1", state: "running" };
+            return { id: "1", state: "started" };
           })
           // third `/runs/:id` call
           .mockImplementationOnce(() => {