@learnpack/learnpack 5.0.340 → 5.0.341

package/lib/utils/api.js

@@ -460,8 +460,17 @@ const createRigoPackage = async (token, slug, config) => {
     }
 };
 let technologiesCache = [];
+/** Strip .env comments (e.g. "token # comment") so the token is sent without spaces. */
+function sanitizeToken(token) {
+    if (!token)
+        return "";
+    const trimmed = token.trim();
+    const beforeComment = trimmed.split(/\s+#/)[0].trim();
+    return beforeComment;
+}
 const fetchTechnologies = async () => {
-    const BREATHECODE_PERMANENT_TOKEN = process.env.BREATHECODE_PERMANENT_TOKEN;
+    const rawToken = process.env.BREATHECODE_PERMANENT_TOKEN;
+    const BREATHECODE_PERMANENT_TOKEN = sanitizeToken(rawToken);
     const LANGS = ["en", "es"];
     if (!BREATHECODE_PERMANENT_TOKEN) {
         throw new Error("BREATHECODE_PERMANENT_TOKEN is not defined in environment variables");

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@learnpack/learnpack",
   "description": "Seamlessly build, sell and/or take interactive & auto-graded tutorials, start learning now or build a new tutorial to your audience.",
-  "version": "5.0.340",
+  "version": "5.0.341",
   "author": "Alejandro Sanchez @alesanchezr",
   "contributors": [
     {

package/src/commands/serve.ts

@@ -1184,7 +1184,8 @@ class ServeCommand extends SessionCommand {
       process.env.GCP_BUCKET_NAME || "learnpack-packages"
     )
 
-    const host = process.env.HOST
+    const rawHost = process.env.HOST || ""
+    const host = rawHost.trim().split(/\s+#/)[0].trim()
 
     if (!host) {
       console.log(
@@ -1196,15 +1197,31 @@ class ServeCommand extends SessionCommand {
     }
 
     // Initialize Redis (official client)
-    const redisUrl = process.env.REDIS_URL || "redis://localhost:6379"
+    let redisUrl = (process.env.REDIS_URL || "redis://localhost:6379").trim()
+    // Strip surrounding quotes if present (e.g. from .env)
+    if (
+      (redisUrl.startsWith('"') && redisUrl.endsWith('"')) ||
+      (redisUrl.startsWith("'") && redisUrl.endsWith("'"))
+    ) {
+      redisUrl = redisUrl.slice(1, -1)
+    }
+
+    const isPlaceholderRedisUrl =
+      redisUrl.includes("your_production_password") ||
+      redisUrl.includes("your-redis-instance.cloud")
 
     if (
       process.env.NODE_ENV === "production" &&
-      redisUrl === "redis://localhost:6379"
+      (redisUrl === "redis://localhost:6379" || isPlaceholderRedisUrl)
     ) {
       console.error("❌ REDIS_URL not configured for production environment!")
       console.warn("⚠️  History features will be unavailable")
       this.redis = null
+    } else if (isPlaceholderRedisUrl) {
+      console.warn(
+        "⚠️  REDIS_URL looks like a placeholder; skipping Redis. History features (undo/redo) will be unavailable"
+      )
+      this.redis = null
     } else {
       try {
         const useTLS = redisUrl.startsWith("rediss://")
@@ -1262,7 +1279,9 @@ class ServeCommand extends SessionCommand {
     const server = http.createServer(app)
     initSocketIO(server)
 
-    const PORT = process.env.PORT || 3000
+    const rawPort = process.env.PORT || "3000"
+    const PORT =
+      parseInt(String(rawPort).trim().split(/\s+#/)[0].trim(), 10) || 3000
 
     const distPath = path.resolve(__dirname, "../creatorDist")