claw: fix skill dispatch to only execute registered skills

Previously, dispatch_command tried ai_skill_try_command for any
unrecognized /command. When the skill didn't exist, it returned
CLAW_ERROR and fell through to "Unknown command", but
ai_skill_try_command internally called ai_skill_execute which
could produce confusing output before failing.

Now dispatch_command explicitly checks ai_skill_find() first.
If the skill exists, it executes and returns (even on failure).
If the skill does not exist, it falls through to "Unknown command"
without attempting execution. This ensures only NVS-persisted or
built-in skills are treated as commands.

Signed-off-by: Chao Liu <chao.liu.zevorn@gmail.com>

Author: zevorn

platform/common/espressif/esp_shell.c

@@ -444,17 +444,24 @@ static void dispatch_command(char *line)
 #ifdef CONFIG_RTCLAW_SKILL_ENABLE
     /* Try dynamic skill as last resort */
     {
-        char *skill_reply = claw_malloc(SKILL_REPLY_SIZE);
-        if (skill_reply && ai_skill_try_command(argv[0], argc, argv,
-                                                skill_reply,
-                                                SKILL_REPLY_SIZE)
-                == CLAW_OK) {
-            printf("\n" CLR_GREEN "rt-claw> " CLR_RESET "%s\n",
-                   skill_reply);
-            claw_free(skill_reply);
+        const char *sname = argv[0];
+        if (sname[0] == '/') {
+            sname++;
+        }
+        if (ai_skill_find(sname)) {
+            char *skill_reply = claw_malloc(SKILL_REPLY_SIZE);
+            if (skill_reply) {
+                if (ai_skill_try_command(argv[0], argc, argv,
+                                         skill_reply,
+                                         SKILL_REPLY_SIZE)
+                        == CLAW_OK) {
+                    printf("\n" CLR_GREEN "rt-claw> " CLR_RESET
+                           "%s\n", skill_reply);
+                }
+                claw_free(skill_reply);
+            }
             return;
         }
-        claw_free(skill_reply);
     }
 #endif
 

platform/linux/shell.c

@@ -302,17 +302,24 @@ static void dispatch_command(char *line)
 
 #ifdef CONFIG_RTCLAW_SKILL_ENABLE
     {
-        char *skill_reply = claw_malloc(SKILL_REPLY_SIZE);
-        if (skill_reply &&
-            ai_skill_try_command(argv[0], argc, argv,
-                                skill_reply,
-                                SKILL_REPLY_SIZE) == CLAW_OK) {
-            printf("\n" CLR_GREEN "rt-claw> " CLR_RESET
-                   "%s\n", skill_reply);
-            claw_free(skill_reply);
+        const char *sname = argv[0];
+        if (sname[0] == '/') {
+            sname++;
+        }
+        if (ai_skill_find(sname)) {
+            char *skill_reply = claw_malloc(SKILL_REPLY_SIZE);
+            if (skill_reply) {
+                if (ai_skill_try_command(argv[0], argc, argv,
+                                         skill_reply,
+                                         SKILL_REPLY_SIZE)
+                        == CLAW_OK) {
+                    printf("\n" CLR_GREEN "rt-claw> " CLR_RESET
+                           "%s\n", skill_reply);
+                }
+                claw_free(skill_reply);
+            }
             return;
         }
-        claw_free(skill_reply);
     }
 #endif
 

platform/zynq-a9/shell.c

@@ -425,19 +425,25 @@ static void dispatch_command(char *line)
     }
 
 #ifdef CONFIG_RTCLAW_SKILL_ENABLE
-    /* Try dynamic skill as last resort */
     {
-        char *skill_reply = claw_malloc(SKILL_REPLY_SIZE);
-        if (skill_reply && ai_skill_try_command(argv[0], argc, argv,
-                                                skill_reply,
-                                                SKILL_REPLY_SIZE)
-                == CLAW_OK) {
-            printf("\n" CLR_GREEN "rt-claw> " CLR_RESET "%s\n",
-                   skill_reply);
-            claw_free(skill_reply);
+        const char *sname = argv[0];
+        if (sname[0] == '/') {
+            sname++;
+        }
+        if (ai_skill_find(sname)) {
+            char *skill_reply = claw_malloc(SKILL_REPLY_SIZE);
+            if (skill_reply) {
+                if (ai_skill_try_command(argv[0], argc, argv,
+                                         skill_reply,
+                                         SKILL_REPLY_SIZE)
+                        == CLAW_OK) {
+                    printf("\n" CLR_GREEN "rt-claw> " CLR_RESET
+                           "%s\n", skill_reply);
+                }
+                claw_free(skill_reply);
+            }
             return;
         }
-        claw_free(skill_reply);
     }
 #endif