@xlameiro/env-typegen 0.1.4 → 0.1.6

package/dist/index.d.cts

@@ -461,9 +461,12 @@ type EnvContract$1 = {
 
 /**
  * Contract file names searched in order when calling {@link loadContract}.
- * Mirrors the search strategy used by `loadConfig` in `config.ts`.
+ * BUG-01: .mjs and .js are searched before .ts — TypeScript files cannot be
+ * loaded by Node.js import() at runtime without a loader like tsx or ts-node.
+ * Discovering a .mjs file first prevents a confusing ERR_UNKNOWN_FILE_EXTENSION
+ * failure when both .ts and .mjs files coexist in the same directory.
  */
-declare const CONTRACT_FILE_NAMES: readonly ["env.contract.ts", "env.contract.mjs", "env.contract.js"];
+declare const CONTRACT_FILE_NAMES: readonly ["env.contract.mjs", "env.contract.js", "env.contract.ts"];
 /**
  * Type-safe contract factory. Returns the contract object unchanged — exists
  * purely for IDE autocompletion and compile-time validation of the contract shape.

package/dist/index.d.ts

@@ -461,9 +461,12 @@ type EnvContract$1 = {
 
 /**
  * Contract file names searched in order when calling {@link loadContract}.
- * Mirrors the search strategy used by `loadConfig` in `config.ts`.
+ * BUG-01: .mjs and .js are searched before .ts — TypeScript files cannot be
+ * loaded by Node.js import() at runtime without a loader like tsx or ts-node.
+ * Discovering a .mjs file first prevents a confusing ERR_UNKNOWN_FILE_EXTENSION
+ * failure when both .ts and .mjs files coexist in the same directory.
  */
-declare const CONTRACT_FILE_NAMES: readonly ["env.contract.ts", "env.contract.mjs", "env.contract.js"];
+declare const CONTRACT_FILE_NAMES: readonly ["env.contract.mjs", "env.contract.js", "env.contract.ts"];
 /**
  * Type-safe contract factory. Returns the contract object unchanged — exists
  * purely for IDE autocompletion and compile-time validation of the contract shape.

package/dist/index.js

@@ -3,7 +3,7 @@ import path6 from 'path';
 import { pathToFileURL } from 'url';
 import { readFile, mkdir, writeFile } from 'fs/promises';
 import { format } from 'prettier';
-import { green, red, yellow } from 'picocolors';
+import pc from 'picocolors';
 import { parseArgs } from 'util';
 
 // src/parser/comment-parser.ts
@@ -157,7 +157,9 @@ var inferenceRules = [
   {
     id: "P10_url_scheme",
     priority: 10,
-    match: (_key, value) => /^[a-zA-Z][a-zA-Z0-9+.-]*:\/\//.test(value),
+    // BUG-02: comma-separated URL lists (e.g. ALLOWED_ORIGINS) must NOT be inferred as
+    // a single URL — they don't pass z.string().url() or new URL() validation at runtime.
+    match: (_key, value) => !value.includes(",") && /^[a-zA-Z][a-zA-Z0-9+.-]*:\/\//.test(value),
     type: "url"
   },
   {
@@ -285,7 +287,16 @@ function parseEnvFileContent(content, filePath, options) {
     );
     commentBlock = [];
   }
-  return { filePath, vars, groups };
+  const seenKeys = /* @__PURE__ */ new Set();
+  const deduped = [];
+  for (let i = vars.length - 1; i >= 0; i--) {
+    const variable = vars[i];
+    if (variable !== void 0 && !seenKeys.has(variable.key)) {
+      seenKeys.add(variable.key);
+      deduped.unshift(variable);
+    }
+  }
+  return { filePath, vars: deduped, groups };
 }
 function parseEnvFile(filePath) {
   const content = readFileSync(filePath, "utf8");
@@ -486,9 +497,9 @@ function generateT3Env(parsed) {
   return lines.join("\n") + "\n";
 }
 var CONTRACT_FILE_NAMES = [
-  "env.contract.ts",
   "env.contract.mjs",
-  "env.contract.js"
+  "env.contract.js",
+  "env.contract.ts"
 ];
 function defineContract(contract) {
   return contract;
@@ -497,6 +508,12 @@ async function loadContract(cwd = process.cwd()) {
   for (const name of CONTRACT_FILE_NAMES) {
     const filePath = path6.resolve(cwd, name);
     if (existsSync(filePath)) {
+      if (filePath.endsWith(".ts")) {
+        throw new Error(
+          `Contract file ${filePath} is a TypeScript file and cannot be loaded at runtime by Node.js.
+Rename it to ${filePath.replace(/\.ts$/, ".mjs")} and use ESM syntax (export default ...), or run via \`tsx\` / \`ts-node\` as a loader.`
+        );
+      }
       const fileUrl = pathToFileURL(filePath).href;
       const mod = await import(fileUrl);
       return mod.default;
@@ -537,7 +554,15 @@ env-typegen.config.mjs for runtime loading.`
   return void 0;
 }
 async function readEnvFile(filePath) {
-  return readFile(path6.resolve(filePath), "utf8");
+  const resolved = path6.resolve(filePath);
+  try {
+    return await readFile(resolved, "utf8");
+  } catch (err) {
+    if (err instanceof Error && err.code === "ENOENT") {
+      throw new Error(`File not found: ${filePath}`);
+    }
+    throw err;
+  }
 }
 async function writeOutput(filePath, content) {
   const resolved = path6.resolve(filePath);
@@ -555,6 +580,7 @@ async function formatOutput(content, parser = "typescript") {
     return content;
   }
 }
+var { green, red, yellow } = pc;
 function log(message) {
   console.log(message);
 }
@@ -570,7 +596,14 @@ function success(message) {
 
 // src/pipeline.ts
 function deriveOutputPath(base, generator, isSingle) {
-  if (isSingle) return base;
+  if (isSingle) {
+    if (generator === "declaration" && !base.endsWith(".d.ts")) {
+      const ext2 = path6.extname(base);
+      const noExt2 = ext2.length > 0 ? base.slice(0, -ext2.length) : base;
+      return `${noExt2}.d.ts`;
+    }
+    return base;
+  }
   const ext = path6.extname(base);
   const noExt = ext.length > 0 ? base.slice(0, -ext.length) : base;
   const baseExt = ext.length > 0 ? ext : ".ts";
@@ -608,10 +641,6 @@ async function persistOutput(params) {
   }
   if (dryRun) {
     if (!silent) {
-      if (!isSingle) {
-        console.log(`// --- ${generator}: ${outputPath} ---`);
-      }
-      console.log(generated);
       success(`Dry run: would write ${outputPath}`);
     }
     return;
@@ -719,7 +748,8 @@ function checkInvalidType(variable, entry, environment) {
       break;
     }
     case "boolean": {
-      if (value !== "true" && value !== "false") {
+      const lower = value.toLowerCase();
+      if (!["true", "false", "1", "0", "yes", "no"].includes(lower)) {
         return {
           code: "ENV_INVALID_TYPE",
           key: variable.key,
@@ -842,7 +872,8 @@ function validateContract(parsed, contract, opts = {}) {
   const parsedByKey = new Map(parsed.vars.map((v) => [v.key, v]));
   for (const entry of contract.vars) {
     if (!entry.required) continue;
-    if (parsedByKey.has(entry.name)) continue;
+    const existing = parsedByKey.get(entry.name);
+    if (existing !== void 0 && existing.rawValue !== "") continue;
     issues.push({
       code: "ENV_MISSING",
       key: entry.name,
@@ -1023,12 +1054,20 @@ function parseProviderPayload(provider, value) {
 }
 async function loadCloudSource(options) {
   const resolvedPath = path6.resolve(options.filePath);
-  const raw = await readFile(resolvedPath, "utf8");
+  let raw;
+  try {
+    raw = await readFile(resolvedPath, "utf8");
+  } catch (err) {
+    if (err instanceof Error && err.code === "ENOENT") {
+      throw new Error(`File not found: ${options.filePath}`);
+    }
+    throw err;
+  }
   const parsed = JSON.parse(raw);
   return parseProviderPayload(options.provider, parsed);
 }
 var SECRET_KEY_RE = /(SECRET|TOKEN|PASSWORD|PRIVATE|API_KEY|ACCESS_KEY|CLIENT_SECRET)/i;
-var CONTRACT_FILE_NAMES2 = ["env.contract.ts", "env.contract.mjs", "env.contract.js"];
+var CONTRACT_FILE_NAMES2 = ["env.contract.mjs", "env.contract.js", "env.contract.ts"];
 function isRecord2(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
@@ -1152,6 +1191,12 @@ async function loadValidationContract(options) {
   const discoveredContractPath = findDefaultContractPath(cwd);
   const resolvedContractPath = contractPath === void 0 ? discoveredContractPath : path6.resolve(cwd, contractPath);
   if (resolvedContractPath !== void 0 && existsSync(resolvedContractPath)) {
+    if (resolvedContractPath.endsWith(".ts")) {
+      throw new Error(
+        `Contract file ${resolvedContractPath} is a TypeScript file and cannot be loaded at runtime by Node.js.
+Rename it to ${resolvedContractPath.replace(/\.ts$/, ".mjs")} and use ESM syntax (export default ...), or run via \`tsx\` / \`ts-node\` as a loader.`
+      );
+    }
     const moduleUrl = pathToFileURL(resolvedContractPath).href;
     const moduleValue = await import(moduleUrl);
     const candidate = moduleValue.default ?? moduleValue.contract;
@@ -1186,6 +1231,9 @@ function isPlugin(value) {
 }
 async function loadPluginFromPath(pluginPath, cwd) {
   const resolvedPath = path6.resolve(cwd, pluginPath);
+  if (!existsSync(resolvedPath)) {
+    throw new Error(`Plugin not found: ${pluginPath}`);
+  }
   const moduleValue = await import(pathToFileURL(resolvedPath).href);
   const candidate = moduleValue.default ?? moduleValue.plugin;
   if (isPlugin(candidate)) return candidate;
@@ -1402,7 +1450,7 @@ function isClientSecret(variable, key) {
 function checkContractVariable(key, variable, context) {
   const { options, issues } = context;
   const value = options.values[key];
-  const hasValue = value !== void 0 && value.trim().length > 0;
+  const hasValue = value !== void 0 && (value !== "" || !variable.required);
   if (variable.required && !hasValue) {
     issues.push(
       createIssue({
@@ -1504,7 +1552,11 @@ function diffTypeConflicts(key, present, context) {
   for (const entry of present) {
     typeBySource.set(entry.sourceName, detectReceivedType(entry.value ?? ""));
   }
-  if (new Set(typeBySource.values()).size <= 1) return;
+  const knownTypes = /* @__PURE__ */ new Set();
+  for (const t of typeBySource.values()) {
+    if (t !== "unknown") knownTypes.add(t);
+  }
+  if (knownTypes.size <= 1) return;
   for (const [sourceName, detectedType] of typeBySource.entries()) {
     issues.push(
       createIssue({
@@ -1583,12 +1635,8 @@ function diffEnvironmentSources(options) {
       sourceName,
       value: options.sources[sourceName]?.[key]
     }));
-    const present = valuesBySource.filter(
-      (entry) => entry.value !== void 0 && entry.value !== ""
-    );
-    const missing = valuesBySource.filter(
-      (entry) => entry.value === void 0 || entry.value === ""
-    );
+    const present = valuesBySource.filter((entry) => entry.value !== void 0);
+    const missing = valuesBySource.filter((entry) => entry.value === void 0);
     if (present.length === 0 && variable?.required === true) {
       for (const entry of missing) {
         issues.push(
@@ -1675,6 +1723,7 @@ async function loadEnvSource(options) {
     return parseEnvSourceContent(content);
   } catch (error_) {
     if (options.allowMissing === true && error_ instanceof Error && "code" in error_ && error_.code === "ENOENT") {
+      warn(`Target file not found: ${options.filePath} \u2014 treating as empty`);
       return {};
     }
     throw error_;
@@ -1841,6 +1890,9 @@ async function loadCommandConfig(configPath) {
     return loadConfig(process.cwd());
   }
   const resolvedPath = path6.resolve(configPath);
+  if (!existsSync(resolvedPath)) {
+    throw new Error(`Config file not found: ${configPath}`);
+  }
   const configDir = path6.dirname(resolvedPath);
   const moduleValue = await import(pathToFileURL(resolvedPath).href);
   if (moduleValue.default === void 0) return void 0;