@tencent-rtc/trtc-agent-skills 0.1.3 → 0.1.5

package/bin/cli.js

@@ -5,9 +5,9 @@
 /**
  * @tencent-rtc/trtc-agent-skills installer
  *
- * Installs the TRTC AI Integration skill suite (6 cross-referencing skills:
- * trtc + trtc-onboarding/docs/topic/search/apply) plus the shared
- * knowledge-base into your IDE's skills directory, and wires up the
+ * Installs the TRTC AI Integration skill suite (cross-referencing skills:
+ * trtc + trtc-onboarding/docs/topic/search/apply + trtc-ai-service) plus the
+ * shared knowledge-base into your IDE's skills directory, and wires up the
  * `tencent-rtc-skill-tool` MCP server (used for prompt / runtime telemetry).
  *
  * IMPORTANT — why skills are copied as SIBLING DIRECTORIES:
@@ -50,15 +50,33 @@ const SKILLS_SRC  = path.join(PKG_ROOT, "skills");
 const KB_SRC      = path.join(PKG_ROOT, "knowledge-base");
 const HOOKS_SRC   = path.join(PKG_ROOT, "hooks");
 
-// The 6 skills that make up the suite. Order is cosmetic; `trtc` is the entry.
-const SKILL_NAMES = [
+// Dynamically discover all skills under SKILLS_SRC. Each skill must be a
+// directory containing a SKILL.md entry point. `trtc` is always listed first;
+// the rest are sorted alphabetically. This avoids the stale-hardcoded-list
+// problem — adding a new skill directory is enough to get it installed.
+//
+// Security: only skills in SKILL_ALLOWLIST are installed. This prevents
+// draft / fork / debug directories from being silently picked up.
+const SKILL_ALLOWLIST = new Set([
   "trtc",
-  "trtc-onboarding",
   "trtc-docs",
-  "trtc-topic",
-  "trtc-search",
-  "trtc-apply",
-];
+  "trtc-conference",
+  "trtc-ai-service",
+  "trtc-chat",
+]);
+
+function getSkillNames() {
+  return fs.readdirSync(SKILLS_SRC, { withFileTypes: true })
+    .filter(entry => entry.isDirectory())
+    .map(entry => entry.name)
+    .filter(name => SKILL_ALLOWLIST.has(name))
+    .filter(name => fs.existsSync(path.join(SKILLS_SRC, name, "SKILL.md")))
+    .sort((a, b) => {
+      if (a === "trtc") return -1;
+      if (b === "trtc") return 1;
+      return a.localeCompare(b);
+    });
+}
 
 // IDE skill-install targets (project-level). Each IDE reads skills from a
 // different directory, but the layout inside is identical: one dir per skill.
@@ -94,9 +112,9 @@ const MCP_SERVER_ENTRY = "@tencent-rtc/skill-tool@latest";
 //     The original hooks.json uses ${CLAUDE_PLUGIN_ROOT} / ${CODEBUDDY_PLUGIN_ROOT}
 //     placeholders that get expanded by the IDE in plugin mode; in npx mode we
 //     materialize them to absolute paths under the IDE's settings dir.
-//   cursor: hooks-cursor.json is rewritten + merged into ~/.cursor/hooks.json
-//     (USER-LEVEL — Cursor doesn't load project-level hooks). The
-//     cursor-adapter.py is copied to <root>/.cursor/hooks/ and its hardcoded
+//   cursor: hooks-cursor.json is rewritten + merged into <root>/.cursor/hooks.json
+//     (project-level). cursor-adapter.py is copied to
+//     <root>/.cursor/hooks/trtc-agent-skills/ and its hardcoded
 //     $HOME/.cursor/plugins/local/... reference is rewritten to the actual path.
 const HOOKS_TARGETS = {
   claude: {
@@ -119,22 +137,64 @@ const HOOKS_TARGETS = {
     hooksDir:        ".codex/hooks",
     // Codex loads hooks from <repo>/.codex/hooks.json (or ~/.codex/hooks.json)
     // — NOT from .agents/settings.json. See https://developers.openai.com/codex/hooks
+    //
+    // Codex CLI ≥0.135 parses hooks.json with a strict serde schema that rejects
+    // unknown top-level fields ("unknown field `__trtc_agent_skills__`, expected
+    // `hooks`"). We therefore mark codex as `strictSchema: true` so the merge
+    // logic skips the marker injection (uninstall identifies our entries by
+    // hook command path substrings instead — see OWNED_COMMAND_HINTS).
     settingsFile:    ".codex/hooks.json",
     sourceConfig:    "hooks.json",
     rootPlaceholder: "${CLAUDE_PLUGIN_ROOT}",
     rootRewrite:     ".codex",
     fallbackPlaceholder: "${CODEBUDDY_PLUGIN_ROOT}",
+    strictSchema:    true,
   },
   cursor: {
-    hooksDir:        ".cursor/hooks",
-    // ⚠ user-level — Cursor only loads ~/.cursor/hooks.json, not project-level.
-    settingsFile:    path.join(os.homedir(), ".cursor", "hooks.json"),
+    // Namespace under .cursor/hooks/trtc-agent-skills/ so we never collide
+    // with another skill's hooks/ contents. cursor-adapter.py auto-detects
+    // PLUGIN_ROOT by walking up to the nearest dir containing skills/, so
+    // this nested location still resolves correctly.
+    hooksDir:        ".cursor/hooks/trtc-agent-skills",
+    hooksFiles:      ["cursor-adapter.py"],
+    settingsFile:    ".cursor/hooks.json",
     sourceConfig:    "hooks-cursor.json",
     // The hardcoded path string we need to rewrite in hooks-cursor.json.
     cursorAdapterPlaceholder: "$HOME/.cursor/plugins/local/trtc-agent-skills/hooks/cursor-adapter.py",
   },
 };
 
+// For IDEs whose hook config schema rejects unknown fields (codex), we cannot
+// embed our `__trtc_agent_skills__` ownership markers. Instead, uninstall
+// detects "our" hook entries by checking whether any command string contains
+// one of these path-segment hints — every guardrail script we ship lives under
+// `skills/<skill>/hooks/` or `skills/<skill>/guardrails/`, and the cursor
+// adapter under our namespaced hooks subdir.
+const OWNED_COMMAND_HINTS = [
+  "/skills/trtc/hooks/",
+  "/skills/trtc/room-builder/guardrails/",
+  "/skills/trtc-topic/guardrails/",
+  "/skills/trtc-apply/guardrails/",
+  "/skills/trtc-conference/hooks/",
+  "/hooks/trtc-agent-skills/cursor-adapter.py",
+];
+
+function isOwnedHookEntry(entry) {
+  if (!entry || typeof entry !== "object") return false;
+  if (entry.__trtc_agent_skills__) return true;
+  // Cursor-style: { command: "...", ... }
+  if (typeof entry.command === "string" && OWNED_COMMAND_HINTS.some(h => entry.command.includes(h))) {
+    return true;
+  }
+  // Claude/Codex-style: { matcher?, hooks: [{ command, ... }] }
+  if (Array.isArray(entry.hooks)) {
+    return entry.hooks.some(h => h && typeof h === "object"
+      && typeof h.command === "string"
+      && OWNED_COMMAND_HINTS.some(hint => h.command.includes(hint)));
+  }
+  return false;
+}
+
 // AI instruction files distribution per IDE.
 //   - root-md  : project-root markdown files (CLAUDE.md / AGENTS.md / CODEBUDDY.md).
 //                If the file already exists, our content is wrapped in HTML
@@ -290,13 +350,20 @@ function printHelp() {
 }
 
 function listSkills() {
+  const descriptions = {
+    "trtc":             "Entry router — detects product/platform, routes to sub-skills",
+    "trtc-ai-service":  "AI customer service scenarios (TRTC Conversational AI)",
+    "trtc-onboarding":  "Get-started / integration / troubleshooting flow",
+    "trtc-docs":        "Docs & error-code lookup",
+    "trtc-topic":       "Step-by-step scenario walkthrough",
+    "trtc-search":      "Internal slice lookup (AI-facing)",
+    "trtc-apply":       "Internal compile/integration quality gate",
+  };
   console.log(`\n  ${c.bold("Skills shipped in this package:")}\n`);
-  console.log(`  ${c.cyan("trtc/")}            ${c.dim("Entry router — detects product/platform, routes to sub-skills")}`);
-  console.log(`  ${c.cyan("trtc-onboarding/")} ${c.dim("Get-started / integration / troubleshooting flow")}`);
-  console.log(`  ${c.cyan("trtc-docs/")}       ${c.dim("Docs & error-code lookup")}`);
-  console.log(`  ${c.cyan("trtc-topic/")}      ${c.dim("Step-by-step scenario walkthrough")}`);
-  console.log(`  ${c.cyan("trtc-search/")}     ${c.dim("Internal slice lookup (AI-facing)")}`);
-  console.log(`  ${c.cyan("trtc-apply/")}      ${c.dim("Internal compile/integration quality gate")}`);
+  for (const name of getSkillNames()) {
+    const desc = descriptions[name] || "";
+    console.log(`  ${c.cyan(name + "/")}` + (desc ? ` ${c.dim(desc)}` : ""));
+  }
   console.log("");
 }
 
@@ -304,7 +371,7 @@ function listSkills() {
 function cleanSkills(skillsRootAbs) {
   if (!fs.existsSync(skillsRootAbs)) return 0;
   let wiped = 0;
-  for (const name of SKILL_NAMES) {
+  for (const name of getSkillNames()) {
     const target = path.join(skillsRootAbs, name);
     if (fs.existsSync(target)) { rmrf(target); wiped++; }
   }
@@ -345,9 +412,9 @@ function cleanAiInstructions(ideList, resolvedRoot) {
   }
 }
 
-// Strip our hook entries from each IDE's settings.json. We tag entries with
-// __trtc_agent_skills__ so we can filter precisely without disturbing the
-// user's own hook entries (relevant for cursor's user-level hooks.json).
+// Strip our hook entries from each IDE's settings file. We tag entries with
+// __trtc_agent_skills__ where the IDE schema allows (claude/codebuddy/cursor),
+// and fall back to command-path matching for strict-schema IDEs (codex).
 function cleanHooksSettings(ideList, resolvedRoot) {
   for (const ide of ideList) {
     const target = HOOKS_TARGETS[ide];
@@ -369,23 +436,35 @@ function cleanHooksSettings(ideList, resolvedRoot) {
       for (const event of Object.keys(settings.hooks)) {
         const val = settings.hooks[event];
         if (Array.isArray(val)) {
-          settings.hooks[event] = val.filter(e => !(e && typeof e === "object" && e.__trtc_agent_skills__));
+          settings.hooks[event] = val.filter(e => !isOwnedHookEntry(e));
           if (settings.hooks[event].length === 0) delete settings.hooks[event];
+        } else if (val && typeof val === "object" && Array.isArray(val.hooks)) {
+          // Some IDEs nest hooks under a single object per event instead of
+          // an array. Filter the inner hooks list.
+          val.hooks = val.hooks.filter(h => !isOwnedHookEntry(h));
+          if (val.hooks.length === 0) delete settings.hooks[event];
         } else {
-          // Non-array (claude/codebuddy style) — we own the whole event under our install.
-          delete settings.hooks[event];
+          // Unknown shape — leave it alone rather than risk corrupting it.
         }
       }
       if (Object.keys(settings.hooks).length === 0) delete settings.hooks;
     }
 
+    // For strict-schema codex, if we cleared everything, remove the file so
+    // codex doesn't see a stale empty file.
+    const onlyHadOurState = !settings.hooks && Object.keys(settings).length === 0;
+    if (onlyHadOurState) {
+      rmrf(settingsPath);
+      continue;
+    }
+
     fs.writeFileSync(settingsPath, JSON.stringify(settings, null, 2) + "\n", "utf8");
   }
 }
 
 function installSkills(skillsRootAbs) {
   ensureDir(skillsRootAbs);
-  for (const name of SKILL_NAMES) {
+  for (const name of getSkillNames()) {
     const src = path.join(SKILLS_SRC, name);
     if (fs.existsSync(src)) {
       copyRecursive(src, path.join(skillsRootAbs, name));
@@ -409,7 +488,14 @@ function copyKnowledgeBase(skillsRootAbs) {
 // to the plugin install root. In npx mode there's no plugin root, so we
 // materialize those placeholders to absolute paths pointing at the IDE's
 // settings dir (where we put .{ide}/skills/, .{ide}/hooks/, etc).
-function rewriteHooksContent(content, target, ideAbsRoot) {
+//
+// `hooksDestAbs` is the absolute path the hooks/ source dir was actually
+// copied to (e.g. <root>/.cursor/hooks/trtc-agent-skills for cursor, or
+// <root>/.claude/hooks for claude). We use it to resolve cursor-adapter.py
+// rather than reconstructing it from `ideAbsRoot + "hooks"`, because cursor
+// nests its hooks one level deeper (under trtc-agent-skills/) for namespace
+// isolation — see HOOKS_TARGETS.cursor.hooksDir.
+function rewriteHooksContent(content, target, ideAbsRoot, hooksDestAbs) {
   let out = content;
   if (target.rootPlaceholder) {
     // Replace BOTH ${CLAUDE_PLUGIN_ROOT} and ${CODEBUDDY_PLUGIN_ROOT} — the
@@ -433,7 +519,12 @@ function rewriteHooksContent(content, target, ideAbsRoot) {
     // We need the resulting JSON string to evaluate to a shell-quoted path so
     // project paths with spaces don't break shell parsing — that means
     // emitting `\"<abs>\"` (JSON-escaped quotes) into the string.
-    const cursorAdapterAbs = path.join(ideAbsRoot, "hooks", "cursor-adapter.py");
+    //
+    // Use hooksDestAbs (the actual copy destination) — NOT ideAbsRoot+"hooks"
+    // — because cursor's hooksDir is namespaced as
+    // .cursor/hooks/trtc-agent-skills, so the script lives one level deeper
+    // than the .cursor/hooks/ that ideAbsRoot+"hooks" would point at.
+    const cursorAdapterAbs = path.join(hooksDestAbs, "cursor-adapter.py");
     const replacement = `\\"${cursorAdapterAbs}\\"`;
     out = out.split(target.cursorAdapterPlaceholder).join(replacement);
   }
@@ -456,12 +547,12 @@ function copyHooksDir(target, resolvedRoot) {
 // ~/.cursor/hooks.json we merge per-event arrays so a previously-installed
 // project's adapter path gets replaced by ours but the user's own hook
 // entries (if any) are preserved.
-function mergeHooksConfig(target, resolvedRoot, ideAbsRoot) {
+function mergeHooksConfig(target, resolvedRoot, ideAbsRoot, hooksDestAbs) {
   const srcPath = path.join(HOOKS_SRC, target.sourceConfig);
   if (!fs.existsSync(srcPath)) return null;
 
   const rawSrc = fs.readFileSync(srcPath, "utf8");
-  const rewritten = rewriteHooksContent(rawSrc, target, ideAbsRoot);
+  const rewritten = rewriteHooksContent(rawSrc, target, ideAbsRoot, hooksDestAbs);
   let parsed;
   try { parsed = JSON.parse(rewritten); }
   catch (err) {
@@ -487,10 +578,16 @@ function mergeHooksConfig(target, resolvedRoot, ideAbsRoot) {
   const incomingHooks = parsed.hooks || {};
   if (!existing.hooks || typeof existing.hooks !== "object") existing.hooks = {};
 
-  // Marker used inside user-level cursor hooks.json to identify our entries
-  // when multiple projects install. Tagged on each individual hook entry so
-  // a future uninstall can filter precisely.
+  // For strict-schema IDEs (codex) we MUST NOT embed any ownership marker —
+  // codex CLI ≥0.135 rejects the whole file with
+  //   "unknown field `__trtc_agent_skills__`, expected `hooks`"
+  // and skips all hooks. Identify our entries on uninstall via command-path
+  // hints (see isOwnedHookEntry) instead.
+  const useMarker = !target.strictSchema;
+
+  // Marker to identify our entries so a future uninstall can filter precisely.
   const tagged = (entry) => {
+    if (!useMarker) return entry;
     if (entry && typeof entry === "object") {
       return Object.assign({}, entry, { __trtc_agent_skills__: true });
     }
@@ -501,24 +598,32 @@ function mergeHooksConfig(target, resolvedRoot, ideAbsRoot) {
     if (Array.isArray(eventValue)) {
       // Cursor format: hooks.<event> = [{command: ...}, ...]
       const stripped = (existing.hooks[eventName] || [])
-        .filter(e => !(e && typeof e === "object" && e.__trtc_agent_skills__));
+        .filter(e => !isOwnedHookEntry(e));
       existing.hooks[eventName] = stripped.concat(eventValue.map(tagged));
     } else if (Array.isArray(existing.hooks[eventName])) {
       // existing is array (cursor-style), incoming is non-array (claude-style):
       // overwrite — this combination shouldn't happen in practice.
       existing.hooks[eventName] = eventValue;
     } else {
-      // Claude/Codebuddy format: hooks.<event> = [{matcher, hooks: [...]}, ...]
-      // The bundled hooks.json IS the only source for this key; just replace.
+      // Claude/Codebuddy/Codex format: hooks.<event> = [{matcher, hooks:[...]}, ...]
+      // For strict-schema codex, the bundled hooks.json IS the only source for
+      // this key and we own the file; just replace.
       existing.hooks[eventName] = eventValue;
     }
   }
 
   // Top-level marker so a future uninstall can detect our presence quickly.
-  existing.__trtc_agent_skills__ = {
-    version: PKG_VERSION,
-    hookEvents: Object.keys(incomingHooks),
-  };
+  // Skip for strict-schema IDEs (codex) — see useMarker above.
+  if (useMarker) {
+    existing.__trtc_agent_skills__ = {
+      version: PKG_VERSION,
+      hookEvents: Object.keys(incomingHooks),
+    };
+  } else if (existing.__trtc_agent_skills__) {
+    // Defensive: if a previous (buggy) install left this field behind, clean
+    // it up so codex doesn't keep failing schema validation.
+    delete existing.__trtc_agent_skills__;
+  }
 
   // Preserve / propagate top-level keys that the IDE expects (e.g. cursor
   // requires `"version": 1` at the root of ~/.cursor/hooks.json or it rejects
@@ -543,7 +648,7 @@ function installHooks(ideList, resolvedRoot) {
     const hooksDest = copyHooksDir(target, resolvedRoot);
     console.log(c.green("    ✓ ") + `${ide} hooks → ${hooksDest}/`);
 
-    const merged = mergeHooksConfig(target, resolvedRoot, ideAbsRoot);
+    const merged = mergeHooksConfig(target, resolvedRoot, ideAbsRoot, hooksDest);
     if (merged) {
       const isUserLevel = path.isAbsolute(target.settingsFile);
       const prefix = isUserLevel ? c.yellow("    ⚠ ") : c.green("    ✓ ");
@@ -812,7 +917,7 @@ function main() {
     }
 
     installSkills(skillsRootAbs);
-    for (const name of SKILL_NAMES) console.log(c.green("    ✓ ") + name + "/");
+    for (const name of getSkillNames()) console.log(c.green("    ✓ ") + name + "/");
 
     const kbDest = copyKnowledgeBase(skillsRootAbs);
     console.log(c.green("    ✓ ") + "knowledge-base/ " + c.dim("→ " + kbDest));

package/hooks/cursor-adapter.py

@@ -25,8 +25,8 @@ Cursor event mapping (see hooks-cursor.json):
   sessionStart        -> trtc-prepare-ui
   beforeReadFile      -> gate-slice-read
   preToolUse          -> gate-slice-write       (filtered to Write/Edit inside)
-  afterFileEdit       -> verify-ui-post-write, verify-slice-must
-  stop                -> stop-apply-evidence, trtc-verify-ui, verify-apply-project
+  afterFileEdit       -> verify-ui-post-write
+  stop                -> stop-apply-evidence, trtc-verify-ui
                          (Claude Code's `Stop` event fires per agent loop
                           iteration end. Cursor's `stop` event has the same
                           semantic ("Called when the agent loop ends") and
@@ -42,10 +42,9 @@ Cursor event mapping (see hooks-cursor.json):
                           firing. Empirically (Cursor 3.3.8) `stop` does
                           fire reliably; the original "stop never fires"
                           observation was caused by hooks not being loaded
-                          at all, not by the `stop` event itself. See
-                          install instructions in the README for the
-                          documented user-level `~/.cursor/hooks.json`
-                          install path — plugin-level hooks declared in
+                          at all, not by the `stop` event itself. npx
+                          installs to project-level `.cursor/hooks.json`;
+                          plugin-level hooks declared in
                           .cursor-plugin/plugin.json are NOT loaded by
                           current Cursor versions.)
 
@@ -62,7 +61,24 @@ import sys
 from pathlib import Path
 
 ADAPTER_DIR = Path(__file__).resolve().parent
-PLUGIN_ROOT = ADAPTER_DIR.parent  # plugin install root
+
+
+def _find_plugin_root(start: Path) -> Path:
+    # Walk up until we find a directory that contains skills/. This makes the
+    # adapter location-independent: it works whether it lives at
+    # <plugin>/hooks/cursor-adapter.py (Cursor plugin install + the original
+    # npx layout) or under a namespaced subdir like
+    # <plugin>/hooks/trtc-agent-skills/cursor-adapter.py (current npx layout,
+    # chosen so multiple skill packages can co-exist under .cursor/hooks/).
+    # Falls back to the old `.parent` behaviour if no `skills/` is found, so
+    # we never regress an existing install.
+    for candidate in (start, *start.parents):
+        if (candidate / "skills").is_dir():
+            return candidate
+    return start.parent
+
+
+PLUGIN_ROOT = _find_plugin_root(ADAPTER_DIR)
 
 
 # Optional debug logging — only writes when TRTC_HOOK_DEBUG_LOG is set to a
@@ -103,23 +119,20 @@ def _probe_log(key, payload) -> None:
 
 # Dispatch table: dispatch_key -> relative script path under PLUGIN_ROOT.
 DISPATCH = {
-    "trtc-prepare-ui":      "skills/trtc/room-builder/guardrails/trtc_prepare_ui.py",
-    "gate-slice-read":      "skills/trtc-topic/guardrails/gate_slice_read.py",
-    "gate-slice-write":     "skills/trtc-topic/guardrails/gate_slice_write.py",
-    "verify-ui-post-write": "skills/trtc/room-builder/guardrails/verify_ui_post_write.sh",
-    "verify-slice-must":    "skills/trtc-apply/guardrails/verify_slice_must_rules.py",
-    "stop-apply-evidence":  "skills/trtc-topic/guardrails/stop_require_apply_evidence.py",
-    "trtc-verify-ui":       "skills/trtc/room-builder/guardrails/trtc_verify_ui.py",
-    "verify-apply-project": "skills/trtc-apply/guardrails/verify_apply_project.py",
+    "gate-slice-read":                        "skills/trtc/hooks/gate_slice_read.py",
+    "topic-phase-gate":                       "skills/trtc/hooks/topic_phase_gate.py",
+    "gate-slice-write":                       "skills/trtc/hooks/gate_slice_write.py",
+    "stop-apply-evidence":                    "skills/trtc/hooks/stop_require_apply_evidence.py",
+    "pretooluse-require-business-decisions":  "skills/trtc-conference/hooks/pretooluse_require_business_decisions.py",
 }
 
 # Dispatch keys whose underlying scripts read JSON from stdin.
 # Other keys are CLI-only and will receive empty stdin.
 STDIN_KEYS = {
     "gate-slice-read",
+    "topic-phase-gate",
     "gate-slice-write",
-    "verify-ui-post-write",
-    "verify-slice-must",
+    "pretooluse-require-business-decisions",
 }
 
 
@@ -164,7 +177,21 @@ def _translate_payload(key: str, cursor: dict) -> dict:
         tool_input = cursor.get("tool_input") or cursor.get("toolInput") or {}
         return {"tool_name": tool_name, "tool_input": tool_input}
 
-    if key in ("verify-ui-post-write", "verify-slice-must"):
+    if key == "topic-phase-gate":
+        tool_name = cursor.get("tool_name") or cursor.get("toolName")
+        if tool_name not in ("Write", "Edit"):
+            return {}
+        tool_input = cursor.get("tool_input") or cursor.get("toolInput") or {}
+        return {"tool_name": tool_name, "tool_input": tool_input}
+
+    if key == "pretooluse-require-business-decisions":
+        tool_name = cursor.get("tool_name") or cursor.get("toolName")
+        if tool_name not in ("Write", "Edit"):
+            return {}
+        tool_input = cursor.get("tool_input") or cursor.get("toolInput") or {}
+        return {"tool_name": tool_name, "tool_input": tool_input}
+
+    if key in ("verify-ui-post-write",):
         # Cursor's afterFileEdit payload: {file_path, edits}
         file_path = cursor.get("file_path") or cursor.get("filePath")
         return {"tool_name": "Edit", "tool_input": {"file_path": file_path}}

package/hooks/hooks-cursor.json

@@ -1,38 +1,25 @@
 {
   "version": 1,
   "hooks": {
-    "sessionStart": [
-      {
-        "command": "python3 $HOME/.cursor/plugins/local/trtc-agent-skills/hooks/cursor-adapter.py trtc-prepare-ui"
-      }
-    ],
-    "beforeReadFile": [
+      "beforeReadFile": [
       {
         "command": "python3 $HOME/.cursor/plugins/local/trtc-agent-skills/hooks/cursor-adapter.py gate-slice-read"
       }
     ],
     "preToolUse": [
       {
-        "command": "python3 $HOME/.cursor/plugins/local/trtc-agent-skills/hooks/cursor-adapter.py gate-slice-write"
-      }
-    ],
-    "afterFileEdit": [
+        "command": "python3 $HOME/.cursor/plugins/local/trtc-agent-skills/hooks/cursor-adapter.py topic-phase-gate"
+      },
       {
-        "command": "python3 $HOME/.cursor/plugins/local/trtc-agent-skills/hooks/cursor-adapter.py verify-ui-post-write"
+        "command": "python3 $HOME/.cursor/plugins/local/trtc-agent-skills/hooks/cursor-adapter.py gate-slice-write"
       },
       {
-        "command": "python3 $HOME/.cursor/plugins/local/trtc-agent-skills/hooks/cursor-adapter.py verify-slice-must"
+        "command": "python3 $HOME/.cursor/plugins/local/trtc-agent-skills/hooks/cursor-adapter.py pretooluse-require-business-decisions"
       }
     ],
     "stop": [
       {
         "command": "python3 $HOME/.cursor/plugins/local/trtc-agent-skills/hooks/cursor-adapter.py stop-apply-evidence"
-      },
-      {
-        "command": "python3 $HOME/.cursor/plugins/local/trtc-agent-skills/hooks/cursor-adapter.py trtc-verify-ui"
-      },
-      {
-        "command": "python3 $HOME/.cursor/plugins/local/trtc-agent-skills/hooks/cursor-adapter.py verify-apply-project"
       }
     ]
   }

package/hooks/hooks.json

@@ -1,22 +1,12 @@
 {
   "hooks": {
-    "SessionStart": [
-      {
-        "hooks": [
-          {
-            "type": "command",
-            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/room-builder/guardrails/trtc_prepare_ui.py\" ] || python3 \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/room-builder/guardrails/trtc_prepare_ui.py\""
-          }
-        ]
-      }
-    ],
     "PreToolUse": [
       {
         "matcher": "Read",
         "hooks": [
           {
             "type": "command",
-            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc-topic/guardrails/gate_slice_read.py\" ] || python3 \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc-topic/guardrails/gate_slice_read.py\""
+            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/hooks/gate_slice_read.py\" ] || python3 \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/hooks/gate_slice_read.py\""
           }
         ]
       },
@@ -25,22 +15,15 @@
         "hooks": [
           {
             "type": "command",
-            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc-topic/guardrails/gate_slice_write.py\" ] || python3 \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc-topic/guardrails/gate_slice_write.py\""
-          }
-        ]
-      }
-    ],
-    "PostToolUse": [
-      {
-        "matcher": "Write|Edit",
-        "hooks": [
+            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/hooks/topic_phase_gate.py\" ] || python3 \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/hooks/topic_phase_gate.py\""
+          },
           {
             "type": "command",
-            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/room-builder/guardrails/verify_ui_post_write.sh\" ] || bash \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/room-builder/guardrails/verify_ui_post_write.sh\""
+            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/hooks/gate_slice_write.py\" ] || python3 \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/hooks/gate_slice_write.py\""
           },
           {
             "type": "command",
-            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc-apply/guardrails/verify_slice_must_rules.py\" ] || python3 \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc-apply/guardrails/verify_slice_must_rules.py\""
+            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc-conference/hooks/pretooluse_require_business_decisions.py\" ] || python3 \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc-conference/hooks/pretooluse_require_business_decisions.py\""
           }
         ]
       }
@@ -50,15 +33,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc-topic/guardrails/stop_require_apply_evidence.py\" ] || python3 \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc-topic/guardrails/stop_require_apply_evidence.py\""
-          },
-          {
-            "type": "command",
-            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/room-builder/guardrails/trtc_verify_ui.py\" ] || python3 \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/room-builder/guardrails/trtc_verify_ui.py\""
-          },
-          {
-            "type": "command",
-            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc-apply/guardrails/verify_apply_project.py\" ] || python3 \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc-apply/guardrails/verify_apply_project.py\""
+            "command": "[ ! -f \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/hooks/stop_require_apply_evidence.py\" ] || python3 \"${CLAUDE_PLUGIN_ROOT:-${CODEBUDDY_PLUGIN_ROOT}}/skills/trtc/hooks/stop_require_apply_evidence.py\""
           }
         ]
       }