chore: important note on not support async schema for env vars

Signed-off-by: tunnckoCore <5038030+tunnckoCore@users.noreply.github.com>

Author: tunnckoCore

.rules/zagora.md

@@ -137,6 +137,8 @@ const sum = mathAgent(5, 10); // { ok: true, data: 15 }
 
 Define schemas for type-safe environment variables with Zod, Valibot, or any Standard Schema V1 compliant library
 
+**Important: Providing async schema for env variables is not supported, at least for now.**
+
 ```typescript
 const safeApi = zagora()
   .env(z.object({
@@ -159,6 +161,7 @@ const sum = safeApi(5, 10); // { ok: true, data: 15 + PORT }
 **Important notes:**
 - When `disableOptions` is enabled (eg. `true`) then handler WILL NOT have access to type-safe env vars.
 - When `autoCallable` is enabled (eg. `true`) make sure to provide the runtime env vars as second argument to the `.env(schema, processEnvOrImportMetaEnv)` method.
+- Async schema validation is not supported, for now
 
 ## Error Handling
 

AGENTS.md

@@ -137,6 +137,8 @@ const sum = mathAgent(5, 10); // { ok: true, data: 15 }
 
 Define schemas for type-safe environment variables with Zod, Valibot, or any Standard Schema V1 compliant library
 
+**Important: Providing async schema for env variables is not supported, at least for now.**
+
 ```typescript
 const safeApi = zagora()
   .env(z.object({
@@ -159,6 +161,7 @@ const sum = safeApi(5, 10); // { ok: true, data: 15 + PORT }
 **Important notes:**
 - When `disableOptions` is enabled (eg. `true`) then handler WILL NOT have access to type-safe env vars.
 - When `autoCallable` is enabled (eg. `true`) make sure to provide the runtime env vars as second argument to the `.env(schema, processEnvOrImportMetaEnv)` method.
+- Async schema validation is not supported, for now
 
 ## Error Handling
 

README.md

@@ -653,6 +653,8 @@ You can also provide the cache through `.callable({ cache })`. That is useful, i
 
 You can provide the runtime env vars (either `process.env` or `import.meta.env`) through the second argument of `.env(schema, envs)` or at later stage through the `.callable({ env })` call. Either way, they will be validated. The parsed variables will be accessible through the handler's `options` object.
 
+**Important: Providing async schema for env variables is not supported, at least for now.**
+
 ```ts
 const zaWithEnv = zagora()
   .env(z.object({
@@ -672,6 +674,9 @@ Keep in mind that if you have `autoCallable: true` enabled in the instance, then
 
 Also important to note that when `disableOptions` you will loose access to the `env` vars, as well `context` and `errors` which is normal behavior.
 
+**Important: Providing async schema for env variables is not supported, at least for now.**
+
+
 [Back to top](#table-of-contents)
 
 ### Handler Options Object

src/utils.ts

@@ -6,128 +6,6 @@ export function getCacheHash(data: string) {
   return nodeCrypto.createHash("sha256").update(data).digest("hex");
 }
 
-// TEST: with expect-type
-// export function createProcedure<TKindNames, TEnv, TNewContext, TContext>({
-//   zagora,
-//   env,
-//   context,
-// }: {
-//   zagora: ZagoraDef;
-//   env: TEnv;
-//   context: TNewContext;
-// }) {
-//   const disableOptions = zagora.disableOptions ?? false;
-//   const handlerFn = zagora.handler;
-//   const errorsMap = zagora.errorsMap;
-//   const inputSchema = zagora.inputSchema as TInputSchema;
-//   const outputSchema = zagora.outputSchema as TOutputSchema;
-//   const cacheAdapter = zagora.cacheAdapter;
-//   const envVarsMapSchema = zagora.envVarsMapSchema;
-//   const baseEnvVars = zagora.envVars;
-
-//   const mergedEnvVars = baseEnvVars ? deepMerge(baseEnvVars, env) : env;
-
-//   return (...args: unknown[]) => {
-//     const envVars = validateInputOutputOrEnv(
-//       "env",
-//       envVarsMapSchema,
-//       mergedEnvVars,
-//     );
-
-//     if (envVars instanceof Promise) {
-//       return createInternalError(
-//         "Environment Variables cannot have async schema validation",
-//       );
-//     }
-//     if (!envVars.ok) {
-//       return envVars;
-//     }
-
-//     const mergedContext = context
-//       ? deepMerge(zagora.initialContext, context)
-//       : zagora.initialContext;
-
-//     const errors = zagora.errorsMap
-//       ? createErrorHelpers(zagora.errorsMap as any)
-//       : undefined;
-//     const options = {
-//       errors: errors,
-//       context: mergedContext,
-//       env: envVars.data,
-//     } as Prettify<
-//       ResolveHandlerOptions<
-//         Prettify<TNewContext & TContext>,
-//         TErrorsMap,
-//         TEnvVarsMap
-//       >
-//     >;
-
-//     const schemaAny = inputSchema as any;
-//     const isTuple =
-//       schemaAny?._def?.type === "tuple" || schemaAny?.type === "tuple";
-
-//     const processor = (mode: "input" | "output", schema: any, data: any) => {
-//       if (schema) {
-//         return validateInputOutputOrEnv(mode, schema, data);
-//       }
-//       return createResult(data, null, false);
-//     };
-
-//     const processInput = (inputData: any) => {
-//       // NOTE: isTuple is safe/enough here cuz it's based on the inputSchema,
-//       // thus if inputSchema is not defined, then it would be isTuple=false too.
-//       const handlerArgs = isTuple
-//         ? handleTupleDefaults(inputSchema, inputData as any)
-//         : [inputData];
-
-//       const executionArgs = disableOptions
-//         ? handlerArgs
-//         : [options, ...handlerArgs];
-
-//       const state = processHandler(handlerFn, executionArgs, cacheAdapter, {
-//         inputSchema,
-//         outputSchema,
-//         envVarsMapSchema,
-//         errorsMap,
-//       });
-
-//       const handleState = (st: any) => {
-//         if (st.ok) {
-//           return processor("output", outputSchema, st.data);
-//         }
-
-//         const { isAsync, handlerFailed, ...rest } = st;
-
-//         // if handler failed, then we need to validate the error if errorShema,
-//         // otherwise we can passthrough the Result object whatever it is.
-//         return handlerFailed
-//           ? validateError<TKindNames>(errorsMap, st.error, isAsync)
-//           : rest;
-//       };
-
-//       if (state instanceof Promise) {
-//         return state.then((st) => handleState(st));
-//       }
-
-//       return handleState(state);
-//     };
-
-//     const inputArgs = isTuple ? args : args[0];
-
-//     const inputResult = inputSchema
-//       ? validateInputOutputOrEnv("input", inputSchema, inputArgs)
-//       : ({ ok: true, data: inputArgs } as const);
-
-//     if (inputResult instanceof Promise) {
-//       return inputResult.then((resultObj) =>
-//         resultObj.error ? resultObj : processInput(resultObj.data),
-//       );
-//     }
-
-//     return inputResult.error ? inputResult : processInput(inputResult.data);
-//   };
-// }
-
 // TEST: with expect-type
 export function validateInputOutputOrEnv(
   mode: "input" | "output" | "env",