slicejs-web-framework 3.3.7 → 3.3.8

package/api/utils/publicEnvResolver.js

@@ -1,117 +1,123 @@
-import { readFileSync, existsSync } from 'node:fs';
-
-const PUBLIC_PREFIX = 'SLICE_PUBLIC_';
-const SUSPICIOUS_TERMS = ['SECRET', 'TOKEN', 'PASSWORD', 'PRIVATE', 'API_KEY', 'ACCESS_KEY', 'CREDENTIAL'];
-
-function parseEnvFile(envFilePath) {
-   if (!envFilePath || !existsSync(envFilePath)) {
-      return {};
-   }
-
-   const fileContent = readFileSync(envFilePath, 'utf8').replace(/^\uFEFF/, '');
-   const parsed = {};
-   const lines = fileContent.split(/\r?\n/);
-
-   for (const rawLine of lines) {
-      const line = rawLine.trim();
-
-      if (!line || line.startsWith('#')) {
-         continue;
-      }
-
-      const equalsIndex = line.indexOf('=');
-      if (equalsIndex === -1) {
-         continue;
-      }
-
-      let key = line.slice(0, equalsIndex).trim();
-      if (!key) {
-         continue;
-      }
-
-      if (key.startsWith('export ')) {
-         key = key.slice('export '.length).trim();
-      }
-
-      let value = line.slice(equalsIndex + 1).trim();
-
-      const quotedWithOptionalCommentMatch = value.match(/^(["'])(.*?)\1(?:\s+#.*)?$/);
-
-      if (quotedWithOptionalCommentMatch) {
-         value = quotedWithOptionalCommentMatch[2];
-      } else {
-         value = value.replace(/\s+#.*$/, '').trimEnd();
-      }
-
-      parsed[key] = value;
-   }
-
-   return parsed;
-}
-
-function warnSuspiciousKey(key, logger, warnedKeys) {
-   const upperKey = key.toUpperCase();
-   const isSuspicious = SUSPICIOUS_TERMS.some((term) => upperKey.includes(term));
-
-   if (isSuspicious && !warnedKeys.has(key) && logger && typeof logger.warn === 'function') {
-      logger.warn(`[slice-env] Suspicious public environment key detected: ${key}`);
-      warnedKeys.add(key);
-   }
-}
-
-function buildPublicPayload({ envFromFile, processEnv, logger }) {
-   const env = {};
-   const warnedKeys = new Set();
-
-   for (const [key, value] of Object.entries(envFromFile)) {
-      if (key.startsWith(PUBLIC_PREFIX)) {
-         env[key] = String(value ?? '');
-         warnSuspiciousKey(key, logger, warnedKeys);
-      }
-   }
-
-   for (const [key, value] of Object.entries(processEnv || {})) {
-      if (key.startsWith(PUBLIC_PREFIX)) {
-         env[key] = String(value ?? '');
-         warnSuspiciousKey(key, logger, warnedKeys);
-      }
-   }
-
-   return env;
-}
-
-export function resolvePublicEnv({ mode, envFilePath, processEnv = process.env, logger = console }) {
-   const envFromFile = parseEnvFile(envFilePath);
-   const env = buildPublicPayload({
-      envFromFile,
-      processEnv,
-      logger,
-   });
-
-   return {
-      mode,
-      env,
-   };
-}
-
-export function createPublicEnvProvider({ mode, envFilePath, processEnv = process.env, logger = console }) {
-   if (mode === 'production') {
-      let cachedPayload;
-
-      return {
-         getPayload() {
-            if (!cachedPayload) {
-               cachedPayload = resolvePublicEnv({ mode, envFilePath, processEnv, logger });
-            }
-
-            return cachedPayload;
-         },
-      };
-   }
-
-   return {
-      getPayload() {
-         return resolvePublicEnv({ mode, envFilePath, processEnv, logger });
-      },
-   };
-}
+import { readFileSync, existsSync } from 'node:fs';
+
+const PUBLIC_PREFIX = 'SLICE_PUBLIC_';
+const SUSPICIOUS_TERMS = ['SECRET', 'TOKEN', 'PASSWORD', 'PRIVATE', 'API_KEY', 'ACCESS_KEY', 'CREDENTIAL'];
+
+function parseEnvFile(envFilePath) {
+   if (!envFilePath || !existsSync(envFilePath)) {
+      return {};
+   }
+
+   let fileContent;
+   try {
+      fileContent = readFileSync(envFilePath, 'utf8').replace(/^\uFEFF/, '');
+   } catch (error) {
+      console.error(`Error reading env file ${envFilePath}:`, error);
+      return {};
+   }
+   const parsed = {};
+   const lines = fileContent.split(/\r?\n/);
+
+   for (const rawLine of lines) {
+      const line = rawLine.trim();
+
+      if (!line || line.startsWith('#')) {
+         continue;
+      }
+
+      const equalsIndex = line.indexOf('=');
+      if (equalsIndex === -1) {
+         continue;
+      }
+
+      let key = line.slice(0, equalsIndex).trim();
+      if (!key) {
+         continue;
+      }
+
+      if (key.startsWith('export ')) {
+         key = key.slice('export '.length).trim();
+      }
+
+      let value = line.slice(equalsIndex + 1).trim();
+
+      const quotedWithOptionalCommentMatch = value.match(/^(["'])(.*?)\1(?:\s+#.*)?$/);
+
+      if (quotedWithOptionalCommentMatch) {
+         value = quotedWithOptionalCommentMatch[2];
+      } else {
+         value = value.replace(/\s+#.*$/, '').trimEnd();
+      }
+
+      parsed[key] = value;
+   }
+
+   return parsed;
+}
+
+function warnSuspiciousKey(key, logger, warnedKeys) {
+   const upperKey = key.toUpperCase();
+   const isSuspicious = SUSPICIOUS_TERMS.some((term) => upperKey.includes(term));
+
+   if (isSuspicious && !warnedKeys.has(key) && logger && typeof logger.warn === 'function') {
+      logger.warn(`[slice-env] Suspicious public environment key detected: ${key}`);
+      warnedKeys.add(key);
+   }
+}
+
+function buildPublicPayload({ envFromFile, processEnv, logger }) {
+   const env = {};
+   const warnedKeys = new Set();
+
+   for (const [key, value] of Object.entries(envFromFile)) {
+      if (key.startsWith(PUBLIC_PREFIX)) {
+         env[key] = String(value ?? '');
+         warnSuspiciousKey(key, logger, warnedKeys);
+      }
+   }
+
+   for (const [key, value] of Object.entries(processEnv || {})) {
+      if (key.startsWith(PUBLIC_PREFIX)) {
+         env[key] = String(value ?? '');
+         warnSuspiciousKey(key, logger, warnedKeys);
+      }
+   }
+
+   return env;
+}
+
+export function resolvePublicEnv({ mode, envFilePath, processEnv = process.env, logger = console }) {
+   const envFromFile = parseEnvFile(envFilePath);
+   const env = buildPublicPayload({
+      envFromFile,
+      processEnv,
+      logger,
+   });
+
+   return {
+      mode,
+      env,
+   };
+}
+
+export function createPublicEnvProvider({ mode, envFilePath, processEnv = process.env, logger = console }) {
+   if (mode === 'production') {
+      let cachedPayload;
+
+      return {
+         getPayload() {
+            if (!cachedPayload) {
+               cachedPayload = resolvePublicEnv({ mode, envFilePath, processEnv, logger });
+            }
+
+            return cachedPayload;
+         },
+      };
+   }
+
+   return {
+      getPayload() {
+         return resolvePublicEnv({ mode, envFilePath, processEnv, logger });
+      },
+   };
+}

package/package.json

@@ -1,44 +1,44 @@
-{
-  "name": "slicejs-web-framework",
-  "version": "3.3.7",
-  "description": "",
-  "engines": {
-    "node": ">=20"
-  },
-  "scripts": {
-    "dev": "node ./node_modules/slicejs-cli/client.js dev",
-    "build": "node ./node_modules/slicejs-cli/client.js build",
-    "start": "node ./node_modules/slicejs-cli/client.js start",
-    "test": "echo \"Error: no test specified\" && exit 1",
-    "slice:init": "node ./node_modules/slicejs-cli/client.js init",
-    "slice:create": "node ./node_modules/slicejs-cli/client.js component create",
-    "slice:delete": "node ./node_modules/slicejs-cli/client.js component delete",
-    "slice:list": "node ./node_modules/slicejs-cli/client.js component list",
-    "slice:start": "node ./node_modules/slicejs-cli/client.js start",
-    "format": "prettier --write \"**/*.js\"",
-    "run": "node ./node_modules/slicejs-cli/client.js dev",
-    "slice:modify": "node ./node_modules/slicejs-cli/client.js modify"
-  },
-  "keywords": [],
-  "author": "",
-  "license": "ISC",
-  "type": "module",
-  "exports": {
-    ".": "./Slice/Slice.js",
-    "./api/framework/express.js": "./api/framework/express.js"
-  },
-  "types": "./types/index.d.ts",
-  "dependencies": {
-    "express": "^4.19.2"
-  },
-  "devDependencies": {
-    "prettier": "3.3.3"
-  },
-  "prettier": {
-    "trailingComma": "es5",
-    "tabWidth": 3,
-    "semi": true,
-    "singleQuote": true,
-    "printWidth": 120
-  }
-}
+{
+  "name": "slicejs-web-framework",
+  "version": "3.3.8",
+  "description": "",
+  "engines": {
+    "node": ">=20"
+  },
+  "scripts": {
+    "dev": "node ./node_modules/slicejs-cli/client.js dev",
+    "build": "node ./node_modules/slicejs-cli/client.js build",
+    "start": "node ./node_modules/slicejs-cli/client.js start",
+    "test": "echo \"Error: no test specified\" && exit 1",
+    "slice:init": "node ./node_modules/slicejs-cli/client.js init",
+    "slice:create": "node ./node_modules/slicejs-cli/client.js component create",
+    "slice:delete": "node ./node_modules/slicejs-cli/client.js component delete",
+    "slice:list": "node ./node_modules/slicejs-cli/client.js component list",
+    "slice:start": "node ./node_modules/slicejs-cli/client.js start",
+    "format": "prettier --write \"**/*.js\"",
+    "run": "node ./node_modules/slicejs-cli/client.js dev",
+    "slice:modify": "node ./node_modules/slicejs-cli/client.js modify"
+  },
+  "keywords": [],
+  "author": "",
+  "license": "ISC",
+  "type": "module",
+  "exports": {
+    ".": "./Slice/Slice.js",
+    "./api/framework/express.js": "./api/framework/express.js"
+  },
+  "types": "./types/index.d.ts",
+  "dependencies": {
+    "express": "^4.19.2"
+  },
+  "devDependencies": {
+    "prettier": "3.3.3"
+  },
+  "prettier": {
+    "trailingComma": "es5",
+    "tabWidth": 3,
+    "semi": true,
+    "singleQuote": true,
+    "printWidth": 120
+  }
+}