resplite 1.5.2 → 1.5.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "resplite",
-  "version": "1.5.2",
+  "version": "1.5.4",
   "description": "A RESP2 server with practical Redis compatibility, backed by SQLite",
   "type": "module",
   "main": "src/index.js",

package/src/commands/registry.js

@@ -289,14 +289,21 @@ export function dispatch(engine, argv, context) {
     return { error: 'ERR wrong number of arguments' };
   }
   const cmd = (Buffer.isBuffer(argv[0]) ? argv[0].toString('utf8') : String(argv[0])).toUpperCase();
+  const args = argv.slice(1);
+  const argvStrings = argv.map((b) => (Buffer.isBuffer(b) ? b.toString('utf8') : String(b)));
   const policy = compileCommandPolicy(context?.commandPolicy);
   const commandResolution = resolveIncomingCommand(cmd, policy);
   if (commandResolution.blocked) {
+    context?.onUnknownCommand?.({
+      command: cmd,
+      argsCount: args.length,
+      argv: argvStrings ?? [cmd],
+      clientAddress: context?.clientAddress ?? '',
+      connectionId: context?.connectionId ?? 0,
+    });
     return { error: unsupported() };
   }
   const resolvedCommand = commandResolution.resolvedCommand;
-  const args = argv.slice(1);
-  const argvStrings = argv.map((b) => (Buffer.isBuffer(b) ? b.toString('utf8') : String(b)));
   if (context) {
     context.getCommandNames = () => listVisibleCommandNames(policy);
     context.resolveCommandForIntrospection = (name) => {

package/test/integration/embed.test.js

@@ -142,6 +142,33 @@ describe('createRESPlite', () => {
     assert.ok(sub.clientAddress.length > 0);
   });
 
+  it('onUnknownCommand hook is also called for disabled commands', async () => {
+    const unknownCalls = [];
+    const srv = await createRESPlite({
+      commandPolicy: {
+        disabled: ['MONITOR'],
+      },
+      hooks: {
+        onUnknownCommand(payload) {
+          unknownCalls.push(payload);
+        },
+      },
+    });
+    const client = await redisClient(srv.port);
+    try {
+      await client.sendCommand(['MONITOR']);
+      assert.fail('expected error');
+    } catch (e) {
+      assert.ok(e.message.includes('not supported'), e.message);
+    }
+    await client.quit();
+    await srv.close();
+
+    assert.equal(unknownCalls.length, 1);
+    assert.equal(unknownCalls[0].command, 'MONITOR');
+    assert.equal(unknownCalls[0].argsCount, 0);
+  });
+
   it('onCommandError hook is called when command returns or throws error', async () => {
     const errorCalls = [];
     const srv = await createRESPlite({