fix(critical): _skillAction scoping — move to module scope

_skillAction was defined inside the /forge handler closure but called
from validateAndDeploy at module scope. Every skill approval (both
/forge approve and forge_approve_skill agent tool) would crash with
ReferenceError. Moved to module scope where bold/mono are available.

Author: Ace Autonomous

index.ts

@@ -17,6 +17,10 @@ import { analyzePatterns } from "./src/pattern/analyze.js";
 import { buildHierarchicalSkillIndex } from "./src/skill/index.js";
 import { notify } from "./src/notify.js";
 import { bold, mono, metric, compose, actions as fmtActions } from "./src/notify-format.js";
+
+// Formatting helper for action notifications — module scope so validateAndDeploy can use it
+const _skillAction = (icon: string, verb: string, name: string): string =>
+  `${icon} ${bold(verb)}  ${mono(name)}`;
 import { resetLlmRateLimit } from "./src/skill/llm-generator.js";
 import {
   recordActivation,
@@ -578,10 +582,6 @@ function buildPlugin() {
           const sub = spaceIdx === -1 ? raw.toLowerCase() : raw.slice(0, spaceIdx).toLowerCase();
           const subArgs = spaceIdx === -1 ? "" : raw.slice(spaceIdx + 1).trim();
 
-          // Formatting helper for action notifications (uses notify-format.ts)
-          const _skillAction = (icon: string, verb: string, name: string) =>
-            `${icon} ${bold(verb)}  ${mono(name)}`;
-
           switch (sub) {
             // ── No subcommand: dashboard ──
             case "":