@peter.naydenov/notice 2.4.2 → 2.4.4

package/graphify-out/manifest.json

@@ -0,0 +1,17 @@
+{
+  "rollup.config.js": 1707045102.6095035,
+  "types/main.d.ts": 1775124398.1611881,
+  "types/src/main.d.ts": 1775124370.093825,
+  "test/01-notice.js": 1760816734.0085897,
+  "coverage/lcov-report/block-navigation.js": 1760816753.1686575,
+  "coverage/lcov-report/prettify.js": 1760816753.170141,
+  "coverage/lcov-report/sorter.js": 1760816753.1693764,
+  "src/main.js": 1760821018.106246,
+  "CODE_OF_CONDUCT.md": 1702927582.1023386,
+  "Changelog.md": 1775124762.301444,
+  "README.md": 1710493604.3297157,
+  "SECURITY.md": 1702927582.1027253,
+  "notice-desk.png": 1702927582.103096,
+  "coverage/lcov-report/favicon.png": 1760816753.168918,
+  "coverage/lcov-report/sort-arrow-sprite.png": 1760816753.1690521
+}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@peter.naydenov/notice",
   "description": "Event emmiter - NOTICE",
-  "version": "2.4.2",
+  "version": "2.4.4",
   "license": "MIT",
   "author": "Peter Naydenov",
   "main": "./src/main.js",
@@ -18,8 +18,8 @@
     "./src/*": "./src/*"
   },
   "scripts": {
-    "test": "mocha test",
-    "cover": "c8 mocha",
+    "test": "vitest run",
+    "cover": "vitest run --coverage",
     "build": "rollup -c",
     "postbuild": "tsc -p tsconfig.json"
   },
@@ -28,31 +28,13 @@
     "url": "git+https://github.com/PeterNaydenov/notice"
   },
   "devDependencies": {
-    "@rollup/plugin-commonjs": "^29.0.2",
+    "@rollup/plugin-commonjs": "^29.0.3",
     "@rollup/plugin-node-resolve": "^16.0.3",
     "@rollup/plugin-terser": "^1.0.0",
-    "c8": "^11.0.0",
-    "chai": "6.2.2",
-    "mocha": "11.7.5",
-    "rollup": "^4.60.1",
-    "typescript": "^6.0.2"
-  },
-  "overrides": {
-    "diff": "^8.0.4",
-    "serialize-javascript": "^7.0.5"
-  },
-  "c8": {
-    "include": [
-      "src/**/*.js"
-    ],
-    "exclude": [
-      "node_modules",
-      "test"
-    ],
-    "reporter": [
-      "lcov",
-      "text-summary"
-    ]
+    "@vitest/coverage-v8": "^4.1.10",
+    "rollup": "^4.62.2",
+    "typescript": "^6.0.3",
+    "vitest": "^4.1.10"
   },
   "keywords": [
     "events",
@@ -60,5 +42,8 @@
     "publisher",
     "subscriber",
     "pubsub"
-  ]
+  ],
+  "allowScripts": {
+    "fsevents@2.3.2": true
+  }
 }

package/src/main.js

@@ -1,9 +1,9 @@
 "use strict"
 function notice () {
     
-                    let 
-                          scroll     = {'*':[]}  // General events with their subscribers
-                        , scrollOnce = {}  // Single events with their subscribers
+                    let
+                          scroll     = Object.assign ( Object.create(null), {'*':[]} )  // General events with their subscribers. Null prototype - event names like '__proto__' are safe
+                        , scrollOnce = Object.create ( null )  // Single events with their subscribers
                         , ignore     = new Set ()  // Ignore event names ( general and single )
                         , debugFlag  = false 
                         , debugHeader = ''
@@ -42,20 +42,21 @@ function notice () {
                             if ( fx ) {   // fx is optional
                                     if ( scroll[e]     )  scroll[e]     = scroll[e].filter     ( fn => fn !== fx )
                                     if ( scrollOnce[e] )  scrollOnce[e] = scrollOnce[e].filter ( fn => fn !== fx )
-                                    if ( scroll[e] && scroll[e].length         === 0 )   delete scroll[e]
-                                    if ( scrollOnce[e] && scrollOnce[e].length === 0 )   delete scroll[e]
+                                    if ( e !== '*' && scroll[e] && scroll[e].length === 0 )   delete scroll[e]   // scroll['*'] must always exist - 'emit' relies on it
+                                    if ( scrollOnce[e] && scrollOnce[e].length === 0 )   delete scrollOnce[e]
                                     return
                                 }
                             if ( scrollOnce[e] )   delete scrollOnce[e]
-                            if ( scroll[e]     )   delete scroll[e]
+                            if ( e === '*'     )   scroll['*'] = []   // scroll['*'] must always exist - 'emit' relies on it
+                            else if ( scroll[e] )   delete scroll[e]
                         } // off func.
                     /**
                      * Resets all event-related data structures.
                      * Clears all general and single event subscriptions, as well as the ignore list.
                      */
                     function reset () {
-                            scroll     = {'*':[]}
-                            scrollOnce = {}
+                            scroll     = Object.assign ( Object.create(null), {'*':[]} )
+                            scrollOnce = Object.create ( null )
                             ignore     = new Set ()
                         } // reset func.
                     /**
@@ -80,7 +81,7 @@ function notice () {
                     function emit () {
                             const [ e, ...args ] = arguments
                             if ( debugFlag ) {  
-                                        console.log ( `${debugHeader} Event "${e}" was triggered.`)
+                                        console.log ( `${debugHeader} Event "${String(e)}" was triggered.`)   // String() - event names can be Symbols
                                         if ( args.length > 0 ) {
                                             console.log ( 'Arguments:')
                                             console.log ( ...args )
@@ -105,14 +106,16 @@ function notice () {
                                 } // exeCallback func.
 
                             if ( e === '*' ) {   // The wildcard '*' doesn't work for 'once' events
-                                        let evNames = Object.keys ( scroll )
+                                        let evNames = Reflect.ownKeys ( scroll )   // Reflect.ownKeys - event names can be Symbols
                                         evNames.forEach ( name => exeCallback(name)   )
                                         return
                                 }
                             if ( scrollOnce[e] ) {
                                         if ( ignore.has(e) )   return
-                                        scrollOnce[e].forEach ( fn => fn(...args)   )
-                                        delete scrollOnce[e]
+                                        const onceFns = scrollOnce[e]
+                                        delete scrollOnce[e]   // Delete before the calls, so handlers can re-register with 'once'
+                                        onceFns.forEach ( fn => fn(...args)   )
+                                        if ( !scroll[e] )   scroll['*'].forEach ( fn => fn(e,...args)  )   // Notify wildcard listeners; if regular subscribers exist, 'exeCallback' will do it
                                 }
                             if ( scroll[e]     ) { 
                                         exeCallback ( e )
@@ -139,9 +142,9 @@ function notice () {
                      */
                     function stop ( e ) {
                             if ( e === '*' ) {
-                                        const 
-                                              evNames     = Object.keys ( scroll )
-                                            , evOnceNames = Object.keys ( scrollOnce )
+                                        const
+                                              evNames     = Reflect.ownKeys ( scroll )   // Reflect.ownKeys - event names can be Symbols
+                                            , evOnceNames = Reflect.ownKeys ( scrollOnce )
                                             ;
                                         ignore = new Set ([ ...evOnceNames, ...evNames ])
                                         return

package/vitest.config.js

@@ -0,0 +1,12 @@
+import { defineConfig } from 'vitest/config'
+
+export default defineConfig({
+  test: {
+    coverage: {
+      provider: 'v8',
+      include: ['src/**/*.js'],
+      exclude: ['node_modules', 'test', 'coverage'],
+      reporter: ['lcov', 'text-summary'],
+    },
+  },
+})