@xlameiro/env-typegen 0.1.5 → 0.1.7

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 0.1.7
+
+### Patch Changes
+
+- 97738bd: ## Fuzzy Dancers Find — env-typegen QA deficiency fixes (D1-D12)
+
+## 0.1.6
+
+### Patch Changes
+
+- 38a33eb: ## Fuzzy Dancers Find — env-typegen QA deficiency fixes (D1-D12)
+
 ## 0.1.5
 
 ### Patch Changes

package/README.md

@@ -1,9 +1,14 @@
 # env-typegen
 
 > From `.env.example` to typed outputs and contract-based environment governance.
+>
+> If this package saves debugging time for your team, consider starring the repository.
 
 [![npm version](https://badge.fury.io/js/%40xlameiro%2Fenv-typegen.svg)](https://npmjs.com/package/@xlameiro/env-typegen)
+[![npm downloads](https://img.shields.io/npm/dm/@xlameiro/env-typegen)](https://npmjs.com/package/@xlameiro/env-typegen)
 [![CI](https://github.com/xlameiro/env-typegen/actions/workflows/ci.yml/badge.svg)](https://github.com/xlameiro/env-typegen/actions/workflows/ci.yml)
+[![GitHub stars](https://img.shields.io/github/stars/xlameiro/env-typegen?style=social)](https://github.com/xlameiro/env-typegen/stargazers)
+[![Maintainer](https://img.shields.io/badge/maintainer-xlameiro-0ea5e9)](https://github.com/xlameiro)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
 
 ## What this package does
@@ -134,6 +139,13 @@ Start with `check`. Add `diff` or `doctor` as your pipeline maturity grows.
 - Package docs index: [`/packages/env-typegen/docs`](./docs)
 - Changelog: [`CHANGELOG.md`](./CHANGELOG.md)
 
+## Trust signals
+
+- Maintained by [@xlameiro](https://github.com/xlameiro)
+- Security policy: [`SECURITY.md`](../../SECURITY.md)
+- Contribution guide: [`CONTRIBUTING.md`](../../CONTRIBUTING.md)
+- Published releases: [npm package page](https://www.npmjs.com/package/@xlameiro/env-typegen)
+
 ## Status
 
 `env-typegen` is actively maintained and published on npm.

package/dist/cli.js

@@ -388,7 +388,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"
   },
   {
@@ -557,6 +559,27 @@ function buildParsedVar(params, commentBlock, options) {
   }
   return parsedVar;
 }
+function deduplicateVars(vars) {
+  const seenKeys = /* @__PURE__ */ new Set();
+  const deduped = [];
+  const warnings = [];
+  for (let i = vars.length - 1; i >= 0; i--) {
+    const variable = vars[i];
+    if (variable === void 0) continue;
+    if (seenKeys.has(variable.key)) {
+      warnings.push({
+        code: "ENV_DUPLICATE_KEY",
+        message: `Duplicate key "${variable.key}" at line ${variable.lineNumber} \u2014 last occurrence wins.`,
+        line: variable.lineNumber,
+        key: variable.key
+      });
+    } else {
+      seenKeys.add(variable.key);
+      deduped.unshift(variable);
+    }
+  }
+  return { deduped, warnings };
+}
 function parseEnvFileContent(content, filePath, options) {
   const lines = content.split("\n");
   const vars = [];
@@ -599,7 +622,13 @@ function parseEnvFileContent(content, filePath, options) {
     );
     commentBlock = [];
   }
-  return { filePath, vars, groups };
+  const { deduped, warnings } = deduplicateVars(vars);
+  return {
+    filePath,
+    vars: deduped,
+    groups,
+    ...warnings.length > 0 && { warnings }
+  };
 }
 function parseEnvFile(filePath) {
   const content = readFileSync(filePath, "utf8");
@@ -615,7 +644,8 @@ async function readEnvFile(filePath) {
     return await readFile(resolved, "utf8");
   } catch (err) {
     if (err instanceof Error && err.code === "ENOENT") {
-      throw new Error(`File not found: ${filePath}`);
+      const displayPath = path4.isAbsolute(filePath) ? filePath : `${filePath} (resolved: ${resolved})`;
+      throw new Error(`File not found: ${displayPath}`);
     }
     throw err;
   }
@@ -646,6 +676,9 @@ var { green, red, yellow } = import_picocolors.default;
 function log(message) {
   console.log(message);
 }
+function warn(message) {
+  console.warn(yellow(`\u26A0 ${message}`));
+}
 function error(message) {
   console.error(red(`\u2716 ${message}`));
 }
@@ -655,7 +688,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 = path5.extname(base);
+      const noExt2 = ext2.length > 0 ? base.slice(0, -ext2.length) : base;
+      return `${noExt2}.d.ts`;
+    }
+    return base;
+  }
   const ext = path5.extname(base);
   const noExt = ext.length > 0 ? base.slice(0, -ext.length) : base;
   const baseExt = ext.length > 0 ? ext : ".ts";
@@ -665,7 +705,8 @@ function deriveOutputPath(base, generator, isSingle) {
 function deriveOutputBaseForInput(output, inputPath) {
   const dir = path5.dirname(output);
   const ext = path5.extname(output);
-  const stem = path5.basename(inputPath, path5.extname(inputPath));
+  const rawBasename = path5.basename(inputPath);
+  const stem = rawBasename.startsWith(".") ? rawBasename.slice(1).replaceAll(".", "-") : path5.basename(inputPath, path5.extname(inputPath));
   return path5.join(dir, `${stem}${ext}`);
 }
 function buildOutput(generator, parsed) {
@@ -702,6 +743,12 @@ async function persistOutput(params) {
     success(`Generated ${outputPath}`);
   }
 }
+function emitParserWarnings(parsed) {
+  if (parsed.warnings === void 0) return;
+  for (const w of parsed.warnings) {
+    warn(`[${w.code}] ${w.message}`);
+  }
+}
 async function runGenerate(options) {
   const {
     input,
@@ -724,6 +771,7 @@ async function runGenerate(options) {
       inputPath,
       inferenceRules2 === void 0 ? void 0 : { inferenceRules: inferenceRules2 }
     );
+    emitParserWarnings(parsed);
     for (const generator of generators) {
       let generated = buildOutput(generator, parsed);
       if (shouldFormat && !dryRun) {
@@ -821,7 +869,8 @@ async function loadCloudSource(options) {
     raw = await readFile2(resolvedPath, "utf8");
   } catch (err) {
     if (err instanceof Error && err.code === "ENOENT") {
-      throw new Error(`File not found: ${options.filePath}`);
+      const displayPath = path6.isAbsolute(options.filePath) ? options.filePath : `${options.filePath} (resolved: ${resolvedPath})`;
+      throw new Error(`File not found: ${displayPath}`);
     }
     throw err;
   }
@@ -834,7 +883,7 @@ import { existsSync as existsSync2 } from "fs";
 import path7 from "path";
 import { pathToFileURL as pathToFileURL2 } from "url";
 var SECRET_KEY_RE = /(SECRET|TOKEN|PASSWORD|PRIVATE|API_KEY|ACCESS_KEY|CLIENT_SECRET)/i;
-var CONTRACT_FILE_NAMES = ["env.contract.ts", "env.contract.mjs", "env.contract.js"];
+var CONTRACT_FILE_NAMES = ["env.contract.mjs", "env.contract.js", "env.contract.ts"];
 function isRecord2(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
@@ -958,6 +1007,12 @@ async function loadValidationContract(options) {
   const discoveredContractPath = findDefaultContractPath(cwd);
   const resolvedContractPath = contractPath === void 0 ? discoveredContractPath : path7.resolve(cwd, contractPath);
   if (resolvedContractPath !== void 0 && existsSync2(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 = pathToFileURL2(resolvedContractPath).href;
     const moduleValue = await import(moduleUrl);
     const candidate = moduleValue.default ?? moduleValue.contract;
@@ -1216,7 +1271,7 @@ function isClientSecret(variable, key) {
 function checkContractVariable(key, variable, context) {
   const { options, issues } = context;
   const value = options.values[key];
-  const hasValue = value !== void 0;
+  const hasValue = value !== void 0 && (value !== "" || !variable.required);
   if (variable.required && !hasValue) {
     issues.push(
       createIssue({
@@ -1318,7 +1373,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({
@@ -1489,6 +1548,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_;
@@ -1660,7 +1720,8 @@ async function loadCommandConfig(configPath) {
   }
   const resolvedPath = path11.resolve(configPath);
   if (!existsSync4(resolvedPath)) {
-    throw new Error(`Config file not found: ${configPath}`);
+    const displayPath = path11.isAbsolute(configPath) ? configPath : `${configPath} (resolved: ${resolvedPath})`;
+    throw new Error(`Config file not found: ${displayPath}`);
   }
   const configDir = path11.dirname(resolvedPath);
   const moduleValue = await import(pathToFileURL4(resolvedPath).href);
@@ -2088,7 +2149,8 @@ async function runValidationSubcommand(subcommand, argv) {
 }
 async function loadExplicitConfig(configPath, userPath) {
   if (!existsSync6(configPath)) {
-    throw new Error(`Config file not found: ${userPath}`);
+    const displayPath = path13.isAbsolute(userPath) ? userPath : `${userPath} (resolved: ${configPath})`;
+    throw new Error(`Config file not found: ${displayPath}`);
   }
   const configDir = path13.dirname(configPath);
   const mod = await import(pathToFileURL5(configPath).href);
@@ -2096,6 +2158,9 @@ async function loadExplicitConfig(configPath, userPath) {
   return rawConfig ? applyConfigPaths2(rawConfig, configDir) : void 0;
 }
 async function runCli(argv = process.argv.slice(2)) {
+  if (argv[0] === "generate") {
+    argv = argv.slice(1);
+  }
   const maybeSubcommand = argv[0];
   if (maybeSubcommand !== void 0 && VALIDATION_SUBCOMMANDS.has(maybeSubcommand)) {
     await runValidationSubcommand(maybeSubcommand, argv.slice(1));