fix(rules): remove Shell.ActionMode.ALL from hallucinated-API list

Shell.ActionMode.ALL is a valid enum value — GNOME Shell's own js/ui/main.js
calls wm.allowKeybinding(..., Shell.ActionMode.ALL) directly. Flagging it as
a hallucinated API produced false-positive FAILs on caffeine and forge during
field testing (R-SLOP-08 / R-SLOP-26).

Changes:
- R-SLOP-08: remove Shell.ActionMode.ALL from the pattern; update message
- R-SLOP-26: remove entirely (rule was exclusively about Shell.ActionMode.ALL)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: Solomon

rules/patterns.yaml

@@ -641,10 +641,10 @@
 # LLMs frequently call methods that don't exist in GJS/GNOME Shell.
 
 - id: R-SLOP-08
-  pattern: "\\b(Meta\\.Screen|Meta\\.Cursor|Shell\\.ActionMode\\.ALL|Shell\\.WindowTracker\\.get_default\\(\\)\\.get_active_window)"
+  pattern: "\\b(Meta\\.Screen|Meta\\.Cursor|Shell\\.WindowTracker\\.get_default\\(\\)\\.get_active_window)"
   scope: ["*.js"]
   severity: advisory
-  message: "Likely hallucinated API — Meta.Screen, Meta.Cursor, Shell.ActionMode.ALL do not exist"
+  message: "Likely hallucinated API — Meta.Screen and Meta.Cursor do not exist"
   category: ai-slop
   fix: "Check the GJS API docs: https://gjs-docs.gnome.org. Use Meta.Display, global.display, etc."
 
@@ -707,14 +707,6 @@
   category: ai-slop
   fix: "Remove the call. Extension lifecycle is managed by GNOME Shell, not by the extension itself."
 
-- id: R-SLOP-26
-  pattern: "\\bShell\\.ActionMode\\.ALL\\b"
-  scope: ["*.js"]
-  severity: advisory
-  message: "Shell.ActionMode.ALL does not exist; use Shell.ActionMode.NORMAL or combine specific modes"
-  category: ai-slop
-  fix: "Use Shell.ActionMode.NORMAL for most keybindings, or Shell.ActionMode.OVERVIEW | Shell.ActionMode.NORMAL for multi-mode"
-
 # --- Code quality (advisory/blocking) ---
 # Module-scope GObject construction, unnecessary patterns, code smell detection.