Make all ACL categories explicit in JSON files (#2887)

Resolves #417

---------

Signed-off-by: Daniil Kashapov <[email protected]>

Author: dvkashapov

src/commands.def

@@ -134,16 +134,13 @@ following keys. To be safe, assume all of them are optional.
     if type is "pure-token". If type is anything else, then `"token"` indicates
     the argument is preceded by an extra (fixed string) argument.
 
-Implicit ACL categories
+ACL categories
 -----------------------
 
-The ACL categories specified as `"acl_categories"` are not the ones actually used.
-The effective ACL categories are affected also by command flags.
-
-The logic for this can be found in the function `setImplicitACLCategories()` in
-`server.c`. Here are the rules (unless they have changed since this
-documentation was written):
+The ACL categories specified as `"acl_categories"` are the ones that are actually used.
+**Note:** commands categories should follow specific rules that are checked in `utils/generate-command-code.py`
 
+**ACL Category Rules:**
 * Command flag WRITE implies ACL category WRITE.
 * Command flag READONLY and not ACL category SCRIPTING implies ACL category READ.
   "Exclude scripting commands from the RO category."
@@ -153,9 +150,6 @@ documentation was written):
 * Command flag BLOCKING implies ACL category BLOCKING.
 * Not ACL category FAST implies ACL category SLOW. "If it's not fast, it's slow."
 
-There's an issue about explicitly listing all categories, removing this
-discrepancy: https://github.com/valkey-io/valkey/issues/417
-
 Key specs
 ---------
 

src/commands/README.md

@@ -37,6 +37,9 @@
                 "type": "string",
                 "optional": true
             }
+        ],
+        "acl_categories": [
+            "SLOW"
         ]
     }
 }

src/commands/acl-cat.json

@@ -15,9 +15,9 @@
             "SENTINEL"
         ],
         "command_tips": [
-          "REQUEST_POLICY:ALL_NODES",
-          "RESPONSE_POLICY:ALL_SUCCEEDED"
-        ],        
+            "REQUEST_POLICY:ALL_NODES",
+            "RESPONSE_POLICY:ALL_SUCCEEDED"
+        ],
         "reply_schema": {
             "type": "integer",
             "description": "The number of users that were deleted."
@@ -28,6 +28,11 @@
                 "type": "string",
                 "multiple": true
             }
+        ],
+        "acl_categories": [
+            "ADMIN",
+            "DANGEROUS",
+            "SLOW"
         ]
     }
 }

src/commands/acl-deluser.json

@@ -41,6 +41,11 @@
                 "optional": true,
                 "multiple": true
             }
+        ],
+        "acl_categories": [
+            "ADMIN",
+            "DANGEROUS",
+            "SLOW"
         ]
     }
 }

src/commands/acl-dryrun.json

@@ -23,6 +23,9 @@
                 "type": "integer",
                 "optional": true
             }
+        ],
+        "acl_categories": [
+            "SLOW"
         ]
     }
 }

src/commands/acl-genpass.json

@@ -86,6 +86,11 @@
                     "type": "null"
                 }
             ]
-        }
+        },
+        "acl_categories": [
+            "ADMIN",
+            "DANGEROUS",
+            "SLOW"
+        ]
     }
 }

src/commands/acl-getuser.json

@@ -18,6 +18,9 @@
             "items": {
                 "type": "string"
             }
-        }
+        },
+        "acl_categories": [
+            "SLOW"
+        ]
     }
 }

src/commands/acl-help.json

@@ -20,6 +20,11 @@
             "items": {
                 "type": "string"
             }
-        }
+        },
+        "acl_categories": [
+            "ADMIN",
+            "DANGEROUS",
+            "SLOW"
+        ]
     }
 }

src/commands/acl-list.json

@@ -16,6 +16,11 @@
         ],
         "reply_schema": {
             "const": "OK"
-        }
+        },
+        "acl_categories": [
+            "ADMIN",
+            "DANGEROUS",
+            "SLOW"
+        ]
     }
 }