AI instructions update (sync with upstream)

Author: softhack007

.coderabbit.yaml

@@ -17,6 +17,19 @@
 language: en-US
 
 reviews:
+  # generic review setting, see https://docs.coderabbit.ai/reference/configuration#reference
+  auto_apply_labels: true
+  # abort_on_close: false
+  high_level_summary: true
+  review_status: true
+  collapse_walkthrough: false
+  poem: false
+  # sequence_diagrams: false
+  auto_review:
+    enabled: true
+    ignore_title_keywords:
+      - WIP
+
   path_instructions:
     - path: "**/*.{cpp,h,hpp,ino}"
       instructions: >
@@ -39,7 +52,7 @@ reviews:
         Follow the web UI conventions documented in docs/web.instructions.md.
 
         Key rules: indent HTML and JavaScript with tabs, CSS with tabs or spaces.
-        Files here are built into wled00/html_*.h by tools/cdata.js — never
+        Files here are built into wled00/html_*.h and wled00/js_*.h by tools/cdata.js — never
         edit those generated headers directly.
 
     - path: "wled00/html_*.h"
@@ -48,6 +61,12 @@ reviews:
         They must never be manually edited or committed. Flag any PR that
         includes changes to these files.
 
+    - path: "wled00/js_*.h"
+      instructions: >
+        These files are auto-generated from wled00/data/ by tools/cdata.js.
+        They must never be manually edited or committed. Flag any PR that
+        includes changes to these files.
+
     - path: "usermods/**"
       instructions: >
         Usermods are community add-ons following the upstream WLED 0.15.x style.
@@ -82,3 +101,13 @@ reviews:
            AI-facing rules due to edits introduced in this PR.
         3. If new AI-facing rules were added without updating a related HUMAN_ONLY
            reference section, note this as a suggestion (not a required fix).
+
+  finishing_touches:
+    # Docstrings | Options for generating Docstrings for your PRs/MRs.
+    docstrings:
+      # Docstrings | Allow CodeRabbit to generate docstrings for PRs/MRs.
+      # default: true - disabled in WLED-MM: has caused confusion in the past
+      enabled: false
+    unit_tests:
+      # default: true - disabled in WLED-MM: we don't have a unit test framework, this option just confuses contributors
+      enabled: false

AGENTS.md

@@ -94,7 +94,7 @@ docs/                # Coding convention docs
 ### Error Handling
 - **No C++ exceptions** — some builds disable them
 - Use return codes (`false`, `-1`) and global flags (`errorFlag = ERR_LOW_MEM`)
-- Use early returns as guard clauses: `if (!enabled || strip.isUpdating()) return;`
+- Use early returns as guard clauses: `if (!enabled || (strip.isUpdating() && (millis() - last_time < MAX_USERMOD_DELAY))) return;`
 - Debug output: `DEBUG_PRINTF()` / `DEBUG_PRINTLN()` (compiled out unless `-D WLED_DEBUG`)
 
 ### Strings and Memory