session-flash 1.0.3 → 1.0.5

package/README.md

@@ -35,6 +35,8 @@ app.use(session({
 
 app.use(flash());
 
+
+
 ```
 
 With the `flash` middleware in place, all requests will have a `req.flash()` function

package/lib/index.cjs

@@ -0,0 +1,65 @@
+'use strict';
+
+const { format } = require('util');
+
+/**
+ * Expose `flash()` function on requests.
+ *
+ * @param {Object} options
+ * @return {Function}
+ * @api public
+ */
+module.exports = function flash(options = {}) {
+  const safe = options.unsafe === undefined ? true : !options.unsafe;
+
+  return function flashMiddleware(req, res, next) {
+    if (req.flash && safe) return next();
+    req.flash = _flash;
+    next();
+  };
+};
+
+/**
+ * Queue or retrieve flash messages.
+ *
+ * @param {String} type
+ * @param {String|Array} msg
+ * @return {Array|Object|Number}
+ * @api public
+ */
+function _flash(type, msg) {
+  if (!this.session) {
+    throw new Error('req.flash() requires sessions');
+  }
+
+  const flashStore = (this.session.flash ||= {});
+
+  // SET
+  if (type && msg !== undefined) {
+    // Support util.format
+    if (arguments.length > 2) {
+      msg = format.apply(null, Array.prototype.slice.call(arguments, 1));
+    }
+
+    // Support array of messages
+    if (Array.isArray(msg)) {
+      msg.forEach(val => {
+        (flashStore[type] ||= []).push(val);
+      });
+      return flashStore[type].length;
+    }
+
+    return (flashStore[type] ||= []).push(msg);
+  }
+
+  // GET by type
+  if (type) {
+    const messages = flashStore[type] || [];
+    delete flashStore[type];
+    return messages;
+  }
+
+  // GET all
+  this.session.flash = {};
+  return flashStore;
+}

package/lib/index.mjs

@@ -0,0 +1,65 @@
+'use strict';
+
+import { format } from 'util';
+
+/**
+ * Expose `flash()` function on requests.
+ *
+ * @param {Object} options
+ * @return {Function}
+ * @api public
+ */
+export default function flash(options = {}) {
+  const safe = options.unsafe === undefined ? true : !options.unsafe;
+
+  return function flashMiddleware(req, res, next) {
+    if (req.flash && safe) return next();
+    req.flash = _flash;
+    next();
+  };
+};
+
+/**
+ * Queue or retrieve flash messages.
+ *
+ * @param {String} type
+ * @param {String|Array} msg
+ * @return {Array|Object|Number}
+ * @api public
+ */
+function _flash(type, msg) {
+  if (!this.session) {
+    throw new Error('req.flash() requires sessions');
+  }
+
+  const flashStore = (this.session.flash ||= {});
+
+  // SET
+  if (type && msg !== undefined) {
+    // Support util.format
+    if (arguments.length > 2) {
+      msg = format.apply(null, Array.prototype.slice.call(arguments, 1));
+    }
+
+    // Support array of messages
+    if (Array.isArray(msg)) {
+      msg.forEach(val => {
+        (flashStore[type] ||= []).push(val);
+      });
+      return flashStore[type].length;
+    }
+
+    return (flashStore[type] ||= []).push(msg);
+  }
+
+  // GET by type
+  if (type) {
+    const messages = flashStore[type] || [];
+    delete flashStore[type];
+    return messages;
+  }
+
+  // GET all
+  this.session.flash = {};
+  return flashStore;
+}

package/package.json

@@ -1,11 +1,14 @@
 {
   "name": "session-flash",
-  "version": "1.0.3",
+  "version": "1.0.5",
   "description": "Flash message middleware for express.",
-  "main": "./lib/index.js",
+  "main": "./lib/index.cjs",
+  "module": "./lib/index.mjs",
   "types": "./lib/index.d.ts",
   "exports": {
-    "require": "./lib/index.js"
+    "require": "./lib/index.cjs",
+    "import": "./lib/index.mjs",
+    "types": "./lib/index.d.ts"
   },
   "keywords": [
     "session-flash",
@@ -20,9 +23,7 @@
   "license": "MIT",
   "dependencies": {},
   "devDependencies": {
-    "@types/node": "^25.0.3",
-    "typescript": "^5.9.3",
-    "vows": "0.8.x"
+    "vows": "^0.8.3"
   },
   "scripts": {
     "test": "NODE_PATH=lib node_modules/.bin/vows test/*-test.js",

package/lib/index.js

@@ -1,66 +0,0 @@
-/**
- * Simple session flash middleware
- * Requires express-session to be set up first
- */
-
-module.exports = function sessionFlash(options) {
-    options = options || {};
-    const unsafe = options.unsafe === true;
-
-    return function (req, res, next) {
-        if (!req.session) {
-            throw new Error("session-flash requires express-session");
-        }
-
-        if (!req.session.flash) {
-            req.session.flash = {};
-        }
-
-        /**
-         * flash()
-         * flash(type)
-         * flash(type, message)
-         * flash(type, format, ...args)
-         */
-        req.flash = function (type, message) {
-            // flash() → return all messages
-            if (arguments.length === 0) {
-                const flash = req.session.flash;
-                req.session.flash = {};
-                return flash;
-            }
-
-            // flash(type) → return messages for type
-            if (arguments.length === 1) {
-                const messages = req.session.flash[type] || [];
-                delete req.session.flash[type];
-                return messages;
-            }
-
-            // flash(type, message | format, ...args)
-            let msg;
-
-            if (typeof message === "string" && arguments.length > 2) {
-                // format string with args
-                const args = Array.prototype.slice.call(arguments, 2);
-                msg = message.replace(/%s/g, () => args.shift());
-            } else {
-                msg = message;
-            }
-
-            if (!Array.isArray(req.session.flash[type])) {
-                req.session.flash[type] = [];
-            }
-
-            if (Array.isArray(msg)) {
-                req.session.flash[type].push.apply(req.session.flash[type], msg);
-                return msg.length;
-            } else {
-                req.session.flash[type].push(msg);
-                return 1;
-            }
-        };
-
-        next();
-    };
-};