@nklisch/pi-enhanced 0.1.2 → 0.1.4

package/node_modules/@nklisch/pi-clearance/src/native/loader.ts

@@ -186,7 +186,6 @@ export type NativeEngineStatus =
 
 const require = createRequire(import.meta.url);
 const NATIVE_MODULE_BASENAME = "clearance-core";
-const NATIVE_PACKAGE_NAME = "pi-clearance";
 let cachedStatus: NativeEngineStatus | undefined;
 
 /**
@@ -196,12 +195,18 @@ let cachedStatus: NativeEngineStatus | undefined;
  * prebuild files staged by `native:prepare`.
  */
 export function nativePlatformTriple(): string | undefined {
-  if (process.platform === "linux" && process.arch === "x64") {
-    return "linux-x64-gnu";
-  }
-  if (process.platform === "darwin" && process.arch === "arm64") {
-    return "darwin-arm64";
-  }
+  // Map the host platform/arch to a napi-rs platform-package suffix. Every
+  // common dev platform is covered. The list of *built* prebuilds is the
+  // narrower one in `package.json` `napi.targets`; this function returns a
+  // suffix for any platform Node can run on so the load path produces a
+  // precise "no prebuild for this platform" error rather than a generic
+  // "unsupported native platform" when the host is genuinely novel.
+  if (process.platform === "linux" && process.arch === "x64") return "linux-x64-gnu";
+  if (process.platform === "linux" && process.arch === "arm64") return "linux-arm64-gnu";
+  if (process.platform === "darwin" && process.arch === "x64") return "darwin-x64";
+  if (process.platform === "darwin" && process.arch === "arm64") return "darwin-arm64";
+  if (process.platform === "win32" && process.arch === "x64") return "win32-x64-msvc";
+  if (process.platform === "win32" && process.arch === "arm64") return "win32-arm64-msvc";
   return undefined;
 }
 
@@ -225,43 +230,25 @@ export function loadNativeEngine(): NativeEngineStatus {
   }
 
   const packageRoot = fileURLToPath(new URL("../../", import.meta.url));
-  // Keep the repository-local artifact first for contributors and CI. Published
-  // packages use napi-rs optional dependencies, so the second candidate resolves
-  // the platform package without a source build or an install hook.
-  const attemptedPaths = [
-    join(packageRoot, "native", `${NATIVE_MODULE_BASENAME}.${triple}.node`),
-    `${NATIVE_PACKAGE_NAME}-${triple}`,
-  ];
+  const modulePath = join(
+    packageRoot,
+    "native",
+    `${NATIVE_MODULE_BASENAME}.${triple}.node`,
+  );
+  const attemptedPaths = [modulePath];
 
-  for (const modulePath of attemptedPaths) {
-    try {
-      const candidate: unknown = require(modulePath);
-      const engine = asNativeEngine(candidate);
-      const health = engine.health();
-      cachedStatus = { ok: true, engine, health, modulePath };
-      return cachedStatus;
-    } catch (error: unknown) {
-      // Continue through the candidate list. The final message includes the
-      // loader error so a missing prebuild is actionable rather than opaque.
-      if (modulePath === attemptedPaths[attemptedPaths.length - 1]) {
-        cachedStatus = {
-          ok: false,
-          reason: nativeLoadError(error),
-          attemptedPaths,
-        };
-        return cachedStatus;
-      }
-    }
+  try {
+    const candidate: unknown = require(modulePath);
+    const engine = asNativeEngine(candidate);
+    const health = engine.health();
+    cachedStatus = { ok: true, engine, health, modulePath };
+  } catch (error: unknown) {
+    cachedStatus = {
+      ok: false,
+      reason: nativeLoadError(error),
+      attemptedPaths,
+    };
   }
-
-  // `attemptedPaths` always contains at least one path, but keep the fallback
-  // explicit so future platform additions cannot accidentally arm without a
-  // validated engine.
-  cachedStatus = {
-    ok: false,
-    reason: "native engine did not expose a supported module",
-    attemptedPaths,
-  };
   return cachedStatus;
 }
 

package/node_modules/@nklisch/pi-clearance/src/replay/reviewer-config-proposals.ts

@@ -9,6 +9,7 @@ import {
   ReviewerConfigSchema,
 } from "../config/schema.ts";
 import {
+  isShippedReviewerPostureId,
   REVIEWER_BASE_CONTRACT,
   SHIPPED_REVIEWER_POSTURES,
   validatePromptOverride,
@@ -146,8 +147,10 @@ export type ReviewerModelDrafter = (
 ) => Promise<ReviewerModelDraftResult | undefined>;
 
 /** True iff `postureId` is one of the shipped reviewer posture fragments. */
-export function isShippedReviewerPosture(postureId: string): boolean {
-  return Object.hasOwn(SHIPPED_REVIEWER_POSTURES, postureId);
+export function isShippedReviewerPosture(
+  postureId: string,
+): postureId is keyof typeof SHIPPED_REVIEWER_POSTURES {
+  return isShippedReviewerPostureId(postureId);
 }
 
 /**
@@ -700,9 +703,9 @@ function draftOverrideSet(
   const postureFragments =
     SHIPPED_REVIEWER_POSTURES[
       isShippedReviewerPosture(current.promptPosture)
-        ? (current.promptPosture as keyof typeof SHIPPED_REVIEWER_POSTURES)
+        ? current.promptPosture
         : "reviewer.default"
-    ];
+    ].fragments;
   const text = [
     REVIEWER_BASE_CONTRACT.text,
     ...postureFragments.map((fragment) => fragment.text),

package/node_modules/@nklisch/pi-clearance/src/runtime/allow-request-message.ts

@@ -0,0 +1,21 @@
+import type { MessageRenderer } from "@earendil-works/pi-coding-agent";
+import { Text } from "@earendil-works/pi-tui";
+
+export const CLEARANCE_ALLOW_REQUEST_CUSTOM_TYPE = "clearance.allow-request";
+
+export interface ClearanceAllowRequestDetails {
+  readonly brief: string;
+  readonly form: "free-text" | "recent-command" | "no-recent-command";
+}
+
+/**
+ * Keep the transcript row compact while the custom message content remains the
+ * complete, clearly labelled brief that Pi sends to the model. The renderer is
+ * deliberately not the source of model context.
+ */
+export const renderClearanceAllowRequest: MessageRenderer<
+  ClearanceAllowRequestDetails
+> = (message) => {
+  const form = message.details?.form ?? "request";
+  return new Text(`[Pi Clearance] Allow request handed to agent (${form})`);
+};

package/node_modules/@nklisch/pi-clearance/src/runtime/auto-reviewer-read-models.ts

@@ -5,6 +5,16 @@ import type {
   ResolvedConfig,
   ResolvedReviewerConfig,
 } from "../config/loader.ts";
+import {
+  DEFAULT_PROJECT_SCOPE_BEHAVIOR,
+  DEFAULT_REVIEWER_CONTEXT_MODE,
+  DEFAULT_REVIEWER_ESCALATION,
+  DEFAULT_REVIEWER_PROMPT_POSTURE,
+  DEFAULT_REVIEWER_RECENT_CONTEXT,
+  DEFAULT_REVIEWER_TOKEN_BUDGET,
+  DEFAULT_REVIEW_NOTE_DISPLAY,
+  DEFAULT_UNKNOWN_TOOL_POSTURE,
+} from "../config/defaults.ts";
 import { shippedPackActivationCondition } from "../packs/activation.ts";
 import type { PackageRegistrationSnapshot } from "../packs/package-registration.ts";
 import {
@@ -49,6 +59,9 @@ export interface AutoReviewerStatusView {
     readonly cwd: string;
   };
   readonly mode: ResolvedPolicy["config"]["mode"];
+  readonly gatedTools?: readonly string[];
+  /** Concise category labels for non-default user-owned settings. */
+  readonly customizations?: readonly string[];
   readonly reviewer: {
     readonly promptPosture: string;
     readonly configuredModel: string | null;
@@ -63,7 +76,10 @@ export interface AutoReviewerStatusView {
       readonly global: number;
       readonly globalProjectConfigured: number;
       readonly projectOverlayConfigured: number;
+      readonly repositoryConfigured: number;
       readonly activeProject: number;
+      /** Configured or active append count, whichever is more informative. */
+      readonly total: number;
     };
     readonly promptOverrideConfigured?: boolean;
     /** Plain-language current reviewer path for gray-area calls. */
@@ -149,7 +165,6 @@ export function buildAutoReviewerStatusView(input: {
   const entries = input.policy.registry.entries;
   const enabled = entries.filter((entry) => entry.availability.enabled).length;
   const reviewer = input.policy.config.reviewer;
-  const snapshots = input.policy.config.sourceSnapshots;
   const resolvedReviewerModel = resolveReviewerModel({
     registry: reviewerModelRegistry(input.ctx),
     spec: reviewer.model,
@@ -158,6 +173,7 @@ export function buildAutoReviewerStatusView(input: {
   const modelHighCost =
     resolvedReviewerModel.model !== undefined &&
     isHighCostReviewerModel(resolvedReviewerModel.model.id);
+  const promptAppendCounts = reviewerPromptAppendCounts(input.policy.config);
   const reviewerPath = resolveReviewerPath({
     config: input.policy.config,
     hasUI: hasHumanReviewUi(input.ctx),
@@ -170,6 +186,8 @@ export function buildAutoReviewerStatusView(input: {
       cwd: projectCwd(input.ctx, input.policy),
     },
     mode: input.policy.config.mode,
+    gatedTools: [...(input.policy.config.gatedTools ?? [])],
+    customizations: customizationCategories(input.policy.config),
     reviewer: {
       promptPosture: reviewer.promptPosture,
       configuredModel: reviewer.model,
@@ -185,13 +203,7 @@ export function buildAutoReviewerStatusView(input: {
       contextMode: reviewer.contextMode,
       tokenBudget: { ...reviewer.tokenBudget },
       escalation: { ...reviewer.escalation },
-      promptAppends: {
-        global: reviewer.promptAppends.length,
-        globalProjectConfigured:
-          snapshots?.global.reviewer.projectPromptAppends.length ?? 0,
-        projectOverlayConfigured: snapshots?.project.promptAppends.length ?? 0,
-        activeProject: reviewer.projectPromptAppends.length,
-      },
+      promptAppends: promptAppendCounts,
       promptOverrideConfigured: reviewer.promptOverride !== null,
       path: reviewerPath,
       consequence: reviewerConsequence(reviewerPath),
@@ -460,6 +472,151 @@ function syntheticPackageRow(
   };
 }
 
+function customizationCategories(
+  config: ResolvedPolicy["config"],
+): readonly string[] {
+  const categories: string[] = [];
+  const snapshots = config.sourceSnapshots;
+  const reviewer = config.reviewer;
+  const promptAppendCounts = reviewerPromptAppendCounts(config);
+
+  if (reviewer.promptOverride !== null) categories.push("prompt override");
+  if (promptAppendCounts.total > 0) {
+    categories.push(`prompt appends (${promptAppendCounts.total})`);
+  }
+  if (reviewer.promptPosture !== DEFAULT_REVIEWER_PROMPT_POSTURE) {
+    categories.push("posture");
+  }
+  if (reviewer.model !== null) categories.push("model pin");
+  if (config.unknownToolPosture !== DEFAULT_UNKNOWN_TOOL_POSTURE) {
+    categories.push("unknown-tool posture");
+  }
+  const recentContext = reviewer.recentContext;
+  if (
+    reviewer.contextMode !== DEFAULT_REVIEWER_CONTEXT_MODE ||
+    recentContext.decisionLimit !== DEFAULT_REVIEWER_RECENT_CONTEXT.decisionLimit ||
+    recentContext.decisionWindow !== DEFAULT_REVIEWER_RECENT_CONTEXT.decisionWindow ||
+    (recentContext.conversationTurns ??
+      DEFAULT_REVIEWER_RECENT_CONTEXT.conversationTurns) !==
+      DEFAULT_REVIEWER_RECENT_CONTEXT.conversationTurns ||
+    (recentContext.userTurns ?? DEFAULT_REVIEWER_RECENT_CONTEXT.userTurns) !==
+      DEFAULT_REVIEWER_RECENT_CONTEXT.userTurns ||
+    recentContext.conversationCharLimit !==
+      DEFAULT_REVIEWER_RECENT_CONTEXT.conversationCharLimit
+  ) {
+    categories.push("context");
+  }
+  if (
+    reviewer.tokenBudget.window !== DEFAULT_REVIEWER_TOKEN_BUDGET.window ||
+    reviewer.tokenBudget.limit !== DEFAULT_REVIEWER_TOKEN_BUDGET.limit
+  ) {
+    categories.push("budget");
+  }
+  if (
+    reviewer.escalation.enabled !== DEFAULT_REVIEWER_ESCALATION.enabled ||
+    reviewer.escalation.denialLimit !==
+      DEFAULT_REVIEWER_ESCALATION.denialLimit ||
+    reviewer.escalation.window !== DEFAULT_REVIEWER_ESCALATION.window
+  ) {
+    categories.push("escalation");
+  }
+  const gatedTools = config.gatedTools ?? [];
+  if (gatedTools.length > 0) {
+    categories.push(`gated tools (${gatedTools.length})`);
+  }
+  if (scopeIsCustomized(snapshots?.project.projectScope)) {
+    categories.push("scope");
+  }
+  const packCount = customizedPackCount(snapshots);
+  if (packCount > 0) categories.push(`packs (${packCount})`);
+  if (
+    config.display.reviewNote.mode !== DEFAULT_REVIEW_NOTE_DISPLAY.mode ||
+    config.display.reviewNote.showModelLabel !==
+      DEFAULT_REVIEW_NOTE_DISPLAY.showModelLabel ||
+    config.display.reviewNote.accent !== DEFAULT_REVIEW_NOTE_DISPLAY.accent
+  ) {
+    categories.push("display");
+  }
+
+  return categories;
+}
+
+function reviewerPromptAppendCounts(
+  config: ResolvedPolicy["config"],
+): NonNullable<AutoReviewerStatusView["reviewer"]["promptAppends"]> {
+  const snapshots = config.sourceSnapshots;
+  const global = config.reviewer.promptAppends.length;
+  const globalProjectConfigured =
+    snapshots?.global.reviewer.projectPromptAppends.length ?? 0;
+  const projectOverlayConfigured = snapshots?.project.promptAppends.length ?? 0;
+  const repositoryConfigured = snapshots?.repository.promptAppends.length ?? 0;
+  const activeProject = config.reviewer.projectPromptAppends.length;
+  const configuredTotal =
+    global +
+    globalProjectConfigured +
+    projectOverlayConfigured +
+    repositoryConfigured;
+  const activeTotal = global + activeProject;
+
+  return {
+    global,
+    globalProjectConfigured,
+    projectOverlayConfigured,
+    repositoryConfigured,
+    activeProject,
+    total: Math.max(configuredTotal, activeTotal),
+  };
+}
+
+function scopeIsCustomized(scope: unknown): boolean {
+  if (scope === undefined || typeof scope !== "object" || scope === null) {
+    return false;
+  }
+  const value = scope as Record<string, unknown>;
+  return (
+    [
+      "roots",
+      "writableDirectories",
+      "tempDirectories",
+      "deniedDirectories",
+      "safeHomeDirectories",
+      "agentSupportDirectories",
+    ].some((key) => Array.isArray(value[key]) && value[key].length > 0) ||
+    (value.safeHomeUseDefaults !== undefined &&
+      value.safeHomeUseDefaults !==
+        DEFAULT_PROJECT_SCOPE_BEHAVIOR.safeHomeUseDefaults) ||
+    (value.agentSupportUseDefaults !== undefined &&
+      value.agentSupportUseDefaults !==
+        DEFAULT_PROJECT_SCOPE_BEHAVIOR.agentSupportUseDefaults) ||
+    (value.unknownPathBehavior !== undefined &&
+      value.unknownPathBehavior !==
+        DEFAULT_PROJECT_SCOPE_BEHAVIOR.unknownPathBehavior) ||
+    (value.sensitivePathBehavior !== undefined &&
+      value.sensitivePathBehavior !==
+        DEFAULT_PROJECT_SCOPE_BEHAVIOR.sensitivePathBehavior) ||
+    (value.homePathBehavior !== undefined &&
+      value.homePathBehavior !== DEFAULT_PROJECT_SCOPE_BEHAVIOR.homePathBehavior)
+  );
+}
+
+function customizedPackCount(
+  snapshots: ResolvedPolicy["config"]["sourceSnapshots"],
+): number {
+  if (snapshots === undefined) return 0;
+  const { global, project, repository } = snapshots;
+  return [
+    global.packs.length,
+    project.packs.length,
+    repository.packs.length,
+    global.packEnablement.enabledPackagePacks.length,
+    global.packEnablement.disabledPackagePacks.length,
+    global.packEnablement.disabledConfigPacks.length,
+    project.packEnablement.enabledPackagePacks.length,
+    project.packEnablement.disabledPackagePacks.length,
+    project.packEnablement.disabledConfigPacks.length,
+  ].reduce((total, count) => total + count, 0);
+}
+
 function effectCopy(effect: PackEffectSummary): string {
   switch (effect) {
     case "deny-floor":

package/node_modules/@nklisch/pi-clearance/src/runtime/command-registry.ts

@@ -101,7 +101,7 @@ export async function handleClearanceCommand(
 ): Promise<CommandReport> {
   const tokens = parseCommandArgs(args);
   if (tokens.length === 0) {
-    return await handleSettingsCommand([], ctx, deps);
+    return await handleSetupCommand([], ctx, pi, deps);
   }
 
   const command = tokens[0] as string;

package/node_modules/@nklisch/pi-clearance/src/runtime/config-commands/allow.ts

@@ -1,6 +1,10 @@
 import type { ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
 
 import type { ToolAnalyzerRegistry } from "../../parse/registry.ts";
+import {
+  CLEARANCE_ALLOW_REQUEST_CUSTOM_TYPE,
+  type ClearanceAllowRequestDetails,
+} from "../allow-request-message.ts";
 import type { ToolShape } from "../../parse/shape.ts";
 import {
   flattenStages,
@@ -385,10 +389,30 @@ async function handoff(
   if (!prepared.ok) return prepared.report;
 
   try {
+    const details: ClearanceAllowRequestDetails = {
+      brief: prepared.brief,
+      form: prepared.form,
+    };
     if (ctx.isIdle()) {
-      await pi.sendUserMessage(prepared.brief);
+      await pi.sendMessage(
+        {
+          customType: CLEARANCE_ALLOW_REQUEST_CUSTOM_TYPE,
+          content: prepared.brief,
+          display: true,
+          details,
+        },
+        { triggerTurn: true },
+      );
     } else {
-      await pi.sendUserMessage(prepared.brief, { deliverAs: "followUp" });
+      await pi.sendMessage(
+        {
+          customType: CLEARANCE_ALLOW_REQUEST_CUSTOM_TYPE,
+          content: prepared.brief,
+          display: true,
+          details,
+        },
+        { deliverAs: "followUp" },
+      );
     }
   } catch (error: unknown) {
     const message = errorMessage(error);

package/node_modules/@nklisch/pi-clearance/src/runtime/config-commands/packs.ts

@@ -165,7 +165,7 @@ export interface DefaultPackListingDetails
 }
 
 /**
- * Human-facing default pack listing used by `/auto-reviewer packs` and
+ * Human-facing default pack listing used by `/clearance packs` and
  * settings drill-downs. Titles are the primary labels; stable pack ids stay as
  * secondary metadata so humans do not have to scan dot/colon-path identifiers.
  */
@@ -221,7 +221,7 @@ export async function handlePackMutationCommand(input: {
       reason:
         "Pack enable/disable requires interactive Pi UI confirmation; no config changes were written.",
       markdownLines: [
-        "- Mutating `/auto-reviewer packs enable|disable` commands require Pi UI confirmation.",
+        "- Mutating `/clearance packs enable|disable` commands require Pi UI confirmation.",
         "- No config changes were written.",
       ],
     });
@@ -338,7 +338,7 @@ export async function handlePackMutationCommand(input: {
   };
 
   return {
-    title: "Auto-reviewer pack enablement updated",
+    title: "Pi Clearance pack enablement updated",
     summary: `${packId} ${request.action} planned for ${target.target.scope}; effective state ${formatEffectiveState(effectivePack)}.`,
     markdown: formatPackMutationSuccessMarkdown(details),
     details,
@@ -1038,7 +1038,7 @@ function formatPackExplorerNameCell(
   const secondary = idsPrimary
     ? escapeCell(pack.title)
     : `\`${escapeCell(pack.id)}\``;
-  return `${primary}<br><small>${secondary}</small>`;
+  return `${primary} — ${secondary}`;
 }
 
 function formatPackExplorerScope(pack: AutoReviewerPackExplorerView): string {
@@ -1362,7 +1362,7 @@ function formatPackMutationSuccessMarkdown(
   details: PackMutationSuccessDetails,
 ): string {
   const lines = [
-    "# Auto-reviewer pack enablement updated",
+    "# Pi Clearance pack enablement updated",
     "",
     `- Pack: \`${details.request.packId}\``,
     `- Action: ${details.request.action}`,
@@ -1513,7 +1513,7 @@ function escapeCell(value: string): string {
 }
 
 function packConfirmationTitle(plan: PackEnablementPlan): string {
-  return `${capitalize(plan.request.action)} auto-reviewer pack?`;
+  return `${capitalize(plan.request.action)} Pi Clearance pack?`;
 }
 
 function yesNo(value: boolean): string {

package/node_modules/@nklisch/pi-clearance/src/runtime/config-commands/scope.ts

@@ -146,7 +146,7 @@ export async function handleScopeCommand(
       reason:
         "Project scope changes require interactive Pi UI confirmation; no config changes were written.",
       markdownLines: [
-        "- Mutating `/auto-reviewer scope` commands require Pi UI confirmation.",
+        "- Mutating `/clearance scope` commands require Pi UI confirmation.",
         "- No config changes were written.",
       ],
       change: parsed.change,
@@ -232,7 +232,7 @@ export async function handleScopeCommand(
   };
 
   return {
-    title: "Auto-reviewer project scope updated",
+    title: "Pi Clearance project scope updated",
     summary: `${planned.plan.title}; changed ${apply.changed ? "yes" : "no"}.`,
     markdown: formatScopeMutationSuccessMarkdown(details),
     details,
@@ -292,7 +292,7 @@ async function handleScopeStatus(
 
   const details = scopeStatusFromConfig(policy.policy.config);
   return {
-    title: "Auto-reviewer project scope",
+    title: "Pi Clearance project scope",
     summary: `Project scope has ${details.raw.roots.length} configured root(s), ${details.raw.writableDirectories.length} configured writable path(s), and unknown paths ${details.raw.unknownPathBehavior}.`,
     markdown: formatScopeStatusMarkdown(details),
     details,
@@ -476,7 +476,7 @@ function scopeStatusFromConfig(config: ResolvedConfig): ScopeStatusDetails {
 
 function formatScopeStatusMarkdown(details: ScopeStatusDetails): string {
   const lines = [
-    "# Auto-reviewer project scope",
+    "# Pi Clearance project scope",
     "",
     "Project scope is lexical-only: it normalizes strings but does not follow symlinks, check filesystem existence, or provide sandbox containment.",
     "",
@@ -618,7 +618,7 @@ function formatScopeMutationSuccessMarkdown(
   details: ScopeMutationSuccessDetails,
 ): string {
   const lines = [
-    "# Auto-reviewer project scope updated",
+    "# Pi Clearance project scope updated",
     "",
     `- Change: ${details.plan.title}`,
     `- Target file: \`${details.plan.target.path}\``,

package/node_modules/@nklisch/pi-clearance/src/runtime/config-commands/settings/actions.ts

@@ -1,7 +1,10 @@
 export type SettingsActionId =
+  | "mode.select"
   | "mode.set"
   | "reviewer.open"
   | "reviewer.model"
+  | "reviewer.posture.select"
+  | "reviewer.posture.set"
   | "reviewer.prompt-append"
   | "reviewer.prompt-override.set"
   | "reviewer.prompt-override.clear"
@@ -9,15 +12,20 @@ export type SettingsActionId =
   | "scope.add-path"
   | "scope.remove-path"
   | "scope.unknown-path"
+  | "scope.unknown-path.select"
   | "scope.safe-home-defaults"
   | "scope.agent-support-defaults"
   | "scope.preset"
+  | "scope.preset.select"
   | "packs.open"
   | "packs.show"
   | "packs.enable"
   | "packs.disable"
   | "briefing.open"
   | "briefing.mode"
+  | "briefing.mode.select"
+  | "gated-tools.add"
+  | "gated-tools.remove"
   | "briefing.model-label"
   | "briefing.accent";
 
@@ -28,6 +36,9 @@ export interface SettingsAction {
 
 export const WRITE_ACTION_IDS = [
   "mode.set",
+  "reviewer.posture.set",
+  "gated-tools.add",
+  "gated-tools.remove",
   "reviewer.model",
   "reviewer.prompt-append",
   "reviewer.prompt-override.set",