fix: address Codex bot review comments for PR MoonshotAI#2995

- Recheck busy state after loading skills in /skill command
- Add idle-only availability to /skill command to block during active turns
- Implement left/right arrow key pagination in skill selector
- Count each skill only once per group subtree in skill selector

Author: mb

apps/kimi-code/src/tui/commands/dispatch.ts

@@ -653,6 +653,16 @@ async function handleSkillCommand(
     return;
   }
 
+  // Recheck busy state after loading skills (P1: Block /skill while a turn is active)
+  const busyCheckAfterLoad = slashCommandBusyReason({
+    isStreaming: host.state.appState.streamingPhase !== 'idle',
+    isCompacting: host.state.appState.isCompacting,
+  });
+  if (busyCheckAfterLoad !== undefined) {
+    host.showError(slashBusyMessage('skill', busyCheckAfterLoad));
+    return;
+  }
+
   const activatableSkills = skills.filter(isUserActivatableSkill);
   const trimmedArgs = args.trim();
 

apps/kimi-code/src/tui/commands/registry.ts

@@ -211,6 +211,7 @@ export const BUILTIN_SLASH_COMMANDS = [
     aliases: ['skills'],
     description: 'Select skill from hierarchical group selector',
     priority: 90,
+    availability: 'idle-only',
   },
   {
     name: 'btw',

apps/kimi-code/src/tui/components/dialogs/skill-selector.ts

@@ -47,11 +47,21 @@ export type SkillSelectorItem =
     };
 
 function countSkillsInTree(node: SkillGroupNode): number {
-  let count = node.skills.length;
-  for (const child of node.childGroups) {
-    count += countSkillsInTree(child);
+  const seen = new Set<string>();
+  function countNode(node: SkillGroupNode): number {
+    let count = 0;
+    for (const skill of node.skills) {
+      if (!seen.has(skill.name)) {
+        seen.add(skill.name);
+        count++;
+      }
+    }
+    for (const child of node.childGroups) {
+      count += countNode(child);
+    }
+    return count;
   }
-  return count;
+  return countNode(node);
 }
 
 function collectDescendantSkills(
@@ -187,6 +197,16 @@ export class SkillSelectorComponent extends Container implements Focusable {
       return;
     }
 
+    // Handle left/right arrow keys for pagination (P2: Implement advertised left/right paging keys)
+    if (matchesKey(data, Key.left)) {
+      this.list.pageUp();
+      return;
+    }
+    if (matchesKey(data, Key.right)) {
+      this.list.pageDown();
+      return;
+    }
+
     const isSpace = matchesKey(data, Key.space) || printableChar(data) === ' ';
     if (matchesKey(data, Key.enter) || (isSpace && this.opts.searchable !== true)) {
       const selected = this.list.selected();