Address comments

Author: minggangw

lib/parameter_event_handler.js

@@ -514,13 +514,22 @@ class ParameterEventHandler {
    * @private
    */
   #resolvePath(nodePath) {
-    if (nodePath.startsWith('/')) {
-      return nodePath;
+    // Absolute node paths are already rooted. Relative names are resolved
+    // against the handler node namespace before building the content filter.
+    const unresolvedPath = nodePath.startsWith('/')
+      ? nodePath
+      : `${this.#node.namespace().replace(/\/+$/, '')}/${nodePath}`;
+
+    // Collapse repeated separators for inputs like '/ns//node/' or 'nested//node'.
+    const resolvedPath = unresolvedPath.replace(/\/+/g, '/');
+
+    // Preserve the root namespace as '/' and strip trailing slashes everywhere
+    // else so the filter matches the canonical ParameterEvent.node format.
+    if (resolvedPath === '/') {
+      return resolvedPath;
     }
 
-    const nodeNamespace = this.#node.namespace().replace(/\/+$/, '');
-    const resolvedPath = `${nodeNamespace}/${nodePath}`.replace(/\/+/g, '/');
-    return resolvedPath.startsWith('/') ? resolvedPath : `/${resolvedPath}`;
+    return resolvedPath.replace(/\/+$/, '');
   }
 
   /**

test/test-parameter-event-handler.js

@@ -360,6 +360,34 @@ describe('ParameterEventHandler tests', function () {
       });
     });
 
+    it('should normalize repeated and trailing slashes in node names', function () {
+      let lastFilter;
+      const subscription = {
+        setContentFilter: (filter) => {
+          lastFilter = filter;
+          return true;
+        },
+        clearContentFilter: () => true,
+        hasContentFilter: () => true,
+      };
+
+      handler = new rclnodejs.ParameterEventHandler(
+        createFakeHandlerNode(subscription)
+      );
+
+      assert.strictEqual(
+        handler.configureNodesFilter([
+          '/test_ns//remote_node/',
+          'nested//node/',
+        ]),
+        true
+      );
+      assert.deepStrictEqual(lastFilter, {
+        expression: 'node = %0 OR node = %1',
+        parameters: ["'/test_ns/remote_node'", "'/test_ns/nested/node'"],
+      });
+    });
+
     it('should clear the content filter when nodeNames is omitted', function () {
       let hasFilter = true;
       const subscription = {