@hasna/skills 0.9.13 → 0.9.14

package/bin/index.js

@@ -36868,7 +36868,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@hasna/skills",
-    version: "0.9.13",
+    version: "0.9.14",
     description: "Skills library for AI coding agents",
     type: "module",
     bin: {
@@ -50548,7 +50548,7 @@ function canonical2(value) {
 function inferencePreference(key, child) {
   if (key === "skipDangerousModePermissionPrompt") {
     need2(child.kind === "literal" && (child.value === "true" || child.value === "false"), "contains an invalid dialog acknowledgment");
-    return true;
+    return false;
   }
   if (key === "effortLevel") {
     need2(child.kind === "string" && PERSISTED_EFFORT_LEVELS.has(child.value), "contains an invalid persisted effort level");
@@ -51233,6 +51233,20 @@ function planAgentIntegration(options) {
   recheckRootAliases(aliases);
   return { dataDir: dataDir3, profileId, changes, nativeSkills, observedPolicy: { path: policyPath, before: previousPolicy }, discoveryBefore: discoveries, discoveryAfter, ...aliases.length ? { rootAliases: aliases } : {} };
 }
+function verifyCoordinatedDiscovery(binding, home, aliases) {
+  try {
+    if (binding.method !== "automatic" && binding.method !== "reviewed")
+      throw new Error("Invalid managed discovery method");
+    verifyAgentDiscovery(binding);
+    if (binding.method === "automatic") {
+      const current = resolveAgentDiscovery({ home, agent: binding.agent, canonical: (path) => canonicalAgentPath(path, aliases) });
+      if (JSON.stringify(current) !== JSON.stringify(binding))
+        throw new Error("Configured native discovery roots changed during Claude update");
+    }
+  } catch (error) {
+    throw new Error(`NATIVE_SKILL_DRIFT: ${error.message}`);
+  }
+}
 function atomicWrite2(path, content) {
   assertSafePath(path);
   mkdirSync5(dirname10(path), { recursive: true, mode: 448 });
@@ -51257,8 +51271,12 @@ function applyAgentIntegration(plan) {
   recheckRootAliases(aliases);
   for (const agent of plan.managedAgentChecks?.agents ?? [])
     assertManagedAgentBridge(agent, { home: plan.managedAgentChecks.home, dataDir: plan.dataDir, projectDir: plan.managedAgentChecks.home });
-  for (const binding of plan.discoveryBefore ?? [])
-    verifyAgentDiscovery(binding);
+  for (const binding of plan.discoveryBefore ?? []) {
+    if (plan.managedAgentChecks)
+      verifyCoordinatedDiscovery(binding, plan.managedAgentChecks.home, aliases);
+    else
+      verifyAgentDiscovery(binding);
+  }
   if (plan.observedPolicy && currentText(plan.observedPolicy.path) !== plan.observedPolicy.before)
     throw new Error("Agent policy changed after planning");
   for (const change of plan.changes)
@@ -51290,8 +51308,12 @@ function applyAgentIntegration(plan) {
       written.push({ path: change.path, before: change.before, after: after2 });
     }
     recheckRootAliases(aliases);
-    for (const binding of plan.discoveryAfter ?? [])
-      verifyAgentDiscovery(binding);
+    for (const binding of plan.discoveryAfter ?? []) {
+      if (plan.managedAgentChecks)
+        verifyCoordinatedDiscovery(binding, plan.managedAgentChecks.home, aliases);
+      else
+        verifyAgentDiscovery(binding);
+    }
     for (const agent of plan.managedAgentChecks?.agents ?? [])
       assertManagedAgentBridge(agent, { home: plan.managedAgentChecks.home, dataDir: plan.dataDir, projectDir: plan.managedAgentChecks.home });
     atomicWrite2(join20(backupRoot, "receipt.json"), JSON.stringify({ version: 1, changes: written.map((change) => ({ path: change.path, beforeHash: change.before === null ? null : sha(change.before), afterHash: sha(change.after) })), backups, ...aliases.length ? { rootAliases: aliases } : {} }) + `

package/bin/mcp.js

@@ -20019,7 +20019,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@hasna/skills",
-    version: "0.9.13",
+    version: "0.9.14",
     description: "Skills library for AI coding agents",
     type: "module",
     bin: {
@@ -47083,7 +47083,7 @@ function canonical2(value) {
 function inferencePreference(key, child) {
   if (key === "skipDangerousModePermissionPrompt") {
     need2(child.kind === "literal" && (child.value === "true" || child.value === "false"), "contains an invalid dialog acknowledgment");
-    return true;
+    return false;
   }
   if (key === "effortLevel") {
     need2(child.kind === "string" && PERSISTED_EFFORT_LEVELS.has(child.value), "contains an invalid persisted effort level");

package/bin/migrate.js

@@ -7,7 +7,7 @@ import { join as join5 } from "path";
 // package.json
 var package_default = {
   name: "@hasna/skills",
-  version: "0.9.13",
+  version: "0.9.14",
   description: "Skills library for AI coding agents",
   type: "module",
   bin: {

package/bin/server.js

@@ -24122,7 +24122,7 @@ var init_dist_es9 = __esm(() => {
 // package.json
 var package_default = {
   name: "@hasna/skills",
-  version: "0.9.13",
+  version: "0.9.14",
   description: "Skills library for AI coding agents",
   type: "module",
   bin: {

package/bin/worker.js

@@ -24125,7 +24125,7 @@ import { randomUUID as randomUUID6 } from "crypto";
 // package.json
 var package_default = {
   name: "@hasna/skills",
-  version: "0.9.13",
+  version: "0.9.14",
   description: "Skills library for AI coding agents",
   type: "module",
   bin: {

package/dist/index.d.ts

@@ -60,7 +60,7 @@ export type { SkillSelection, SkillProfile, ResolvedSkillSelection, ResolvedSkil
 export { buildSkillContext } from "./lib/skill-context.js";
 export { syncSelectionProfile } from "./lib/selection-resolver.js";
 export { inspectSkillSession, reconcileSkillSession, type SessionReconciliationInput, type SessionReconciliationOptions } from "./lib/session-reconciliation.js";
-export { planAgentIntegration, planClaudeStopHookUpdate, applyAgentIntegration, inventoryNativeSkills, archiveNativeSkills } from "./lib/agent-integration.js";
+export { planAgentIntegration, planClaudeStopHookUpdate, planClaudePreToolUseHookUpdate, planClaudeHookEventsUpdate, applyAgentIntegration, inventoryNativeSkills, archiveNativeSkills } from "./lib/agent-integration.js";
 export { captureDiscoveryDirectories, captureDiscoveryByteSources, captureDiscoveryPathSources, type DiscoveryDirectory, type DiscoverySource } from "./lib/agent-discovery.js";
 export { captureClaudeMarketplaceRegistry, captureClaudeMarketplaceRegistryV2 } from "./lib/claude-marketplace-registry.js";
 export { captureClaudeSettings, captureClaudeSettingsV2 } from "./lib/claude-settings-witness.js";
@@ -69,3 +69,4 @@ export { planPluginAdmission, admitPlugin, resolveAdmittedPlugin, pluginResolver
 export { captureManagedPluginRegistry, type ManagedPluginRegistrationWitness } from "./lib/plugin-discovery.js";
 export type { RemoteCreditCheckoutOptions, RemoteCreditCheckout, RemoteCreditCheckoutErrorCode } from "./lib/remote-credit-checkout.js";
 export { assertPortableAuthoringPath, type PortableAuthoringPathOptions } from "./lib/authoring-path.js";
+export { CLAUDE_COORDINATED_HOOK_EVENTS, type ClaudeCoordinatedHookEvent } from "./lib/claude-settings-witness.js";

package/dist/index.js

@@ -9610,7 +9610,7 @@ import { isAbsolute as isAbsolute11, join as join23, resolve as resolve11 } from
 // package.json
 var package_default = {
   name: "@hasna/skills",
-  version: "0.9.13",
+  version: "0.9.14",
   description: "Skills library for AI coding agents",
   type: "module",
   bin: {
@@ -12846,32 +12846,37 @@ function canonical2(value) {
     return { kind: "array", items: value.items.map(canonical2) };
   return value;
 }
-function assertClaudeStopHookReplacement(before, replacement) {
+var CLAUDE_COORDINATED_HOOK_EVENTS = ["PreToolUse", "PostToolUse", "Stop", "Notification", "SessionStart", "SessionEnd", "UserPromptSubmit", "SubagentStart"];
+function assertClaudeHookEventsReplacement(before, replacement, events) {
+  need2(Array.isArray(events) && events.length > 0 && events.length <= CLAUDE_COORDINATED_HOOK_EVENTS.length && new Set(events).size === events.length && events.every((event) => CLAUDE_COORDINATED_HOOK_EVENTS.includes(event)), "requires explicit supported hook events");
+  const label = events.join(",");
   function bounded(text2) {
-    need2(typeof text2 === "string" && Buffer.byteLength(text2) <= CLAUDE_SETTINGS_WITNESS_LIMITS.bytes, "Stop update exceeds its byte limit");
+    need2(typeof text2 === "string" && Buffer.byteLength(text2) <= CLAUDE_SETTINGS_WITNESS_LIMITS.bytes, `${label} update exceeds its byte limit`);
     return parse3(text2);
   }
   const original = bounded(before ?? "{}"), next = bounded(replacement);
   const hadHooks = original.entries.some(([key]) => key === "hooks");
-  function withoutStop(value) {
+  function withoutOwnedEvents(value) {
     const hooks = value.entries.find(([key]) => key === "hooks")?.[1];
     if (hooks !== undefined) {
-      need2(hooks.kind === "object", "Stop update requires an object of hooks");
-      const stop = hooks.entries.find(([key]) => key === "Stop")?.[1];
-      need2(stop === undefined || stop.kind === "array", "Stop update requires a Stop array");
-      hooks.entries = hooks.entries.filter(([key]) => key !== "Stop");
+      need2(hooks.kind === "object", `${label} update requires an object of hooks`);
+      for (const event of events) {
+        const selected = hooks.entries.find(([key]) => key === event)?.[1];
+        need2(selected === undefined || selected.kind === "array", `${event} update requires a ${event} array`);
+      }
+      hooks.entries = hooks.entries.filter(([key]) => !events.includes(key));
       if (!hadHooks && hooks.entries.length === 0)
         value.entries = value.entries.filter(([key]) => key !== "hooks");
     }
     return canonical2(value);
   }
-  need2(JSON.stringify(withoutStop(original)) === JSON.stringify(withoutStop(next)), "Stop update changed settings outside hooks.Stop");
+  need2(JSON.stringify(withoutOwnedEvents(original)) === JSON.stringify(withoutOwnedEvents(next)), `${label} update changed settings outside hooks.${label}`);
 }
 var PERSISTED_EFFORT_LEVELS = new Set(["low", "medium", "high", "xhigh"]);
 function inferencePreference(key, child) {
   if (key === "skipDangerousModePermissionPrompt") {
     need2(child.kind === "literal" && (child.value === "true" || child.value === "false"), "contains an invalid dialog acknowledgment");
-    return true;
+    return false;
   }
   if (key === "effortLevel") {
     need2(child.kind === "string" && PERSISTED_EFFORT_LEVELS.has(child.value), "contains an invalid persisted effort level");
@@ -13452,6 +13457,34 @@ function planAgentIntegration(options) {
   return { dataDir: dataDir2, profileId, changes, nativeSkills, observedPolicy: { path: policyPath, before: previousPolicy }, discoveryBefore: discoveries, discoveryAfter, ...aliases.length ? { rootAliases: aliases } : {} };
 }
 function planClaudeStopHookUpdate(options) {
+  return planClaudeHookEventsUpdate({ ...options, events: ["Stop"] });
+}
+function planClaudePreToolUseHookUpdate(options) {
+  return planClaudeHookEventsUpdate({ ...options, events: ["PreToolUse"] });
+}
+function claudeSettingsConsumers(home, path, discoveries, aliases) {
+  const canonical3 = (value) => canonicalAgentPath(value, aliases);
+  const containsSettings = (root) => {
+    const distance = relative4(canonical3(root), path);
+    return distance === "" || distance !== ".." && !distance.startsWith(`..${sep4}`) && !isAbsolute9(distance);
+  };
+  return discoveries.filter((binding) => binding.agent === "claude" || binding.agent === "opencode" || canonical3(join16(home, AGENT_ADAPTERS[binding.agent].config)) === path || binding.sources.some((source) => canonical3(source.path) === path) || binding.roots.some(containsSettings) || binding.directories?.some((directory) => containsSettings(directory.path))).map((binding) => binding.agent);
+}
+function verifyCoordinatedDiscovery(binding, home, aliases) {
+  try {
+    if (binding.method !== "automatic" && binding.method !== "reviewed")
+      throw new Error("Invalid managed discovery method");
+    verifyAgentDiscovery(binding);
+    if (binding.method === "automatic") {
+      const current = resolveAgentDiscovery({ home, agent: binding.agent, canonical: (path) => canonicalAgentPath(path, aliases) });
+      if (JSON.stringify(current) !== JSON.stringify(binding))
+        throw new Error("Configured native discovery roots changed during Claude update");
+    }
+  } catch (error) {
+    throw new Error(`NATIVE_SKILL_DRIFT: ${error.message}`);
+  }
+}
+function planClaudeHookEventsUpdate(options) {
   const home = resolve9(options.home ?? homedir2()), dataDir2 = options.dataDir ?? getDataDirReadOnly();
   const policyPath = join16(dataDir2, "agent-policy.json");
   assertSafePath(policyPath);
@@ -13469,31 +13502,42 @@ function planClaudeStopHookUpdate(options) {
   const before = lstatSync14(path, { throwIfNoEntry: false }) ? new TextDecoder("utf-8", { fatal: true, ignoreBOM: true }).decode(readNativeBytes(path, 1024 * 1024)) : null;
   if ((before === null ? "absent" : sha(before)) !== options.expectedSettingsSha256)
     throw new Error("Claude settings preimage changed");
-  assertClaudeStopHookReplacement(before, options.replacement);
+  assertClaudeHookEventsReplacement(before, options.replacement, options.events);
   const agents = bridge.agents;
   const discoveries = Object.entries(bridge.discovery).map(([agent, binding]) => {
-    if (!agents.includes(agent) || binding?.agent !== agent)
-      throw new Error("Invalid managed discovery owner");
+    if (!agents.includes(agent) || binding?.agent !== agent || typeof bridge.commands?.[agent] !== "string" || typeof bridge.profiles?.[agent] !== "string")
+      throw new Error("Invalid managed discovery owner or command/profile binding");
     return binding;
   });
-  if (discoveries.length !== agents.length)
-    throw new Error("Missing managed discovery coverage");
-  for (const agent of agents)
+  if (discoveries.length !== agents.length || !agents.includes("claude"))
+    throw new Error("Missing managed Claude discovery coverage");
+  for (const binding of discoveries)
+    verifyCoordinatedDiscovery(binding, home, aliases);
+  const consumers = claudeSettingsConsumers(home, path, discoveries, aliases);
+  for (const agent of consumers)
     assertManagedAgentBridge(agent, { home, dataDir: dataDir2, projectDir: home });
   const replacements = new Map(before === options.replacement ? [] : [[path, options.replacement]]);
-  const discoveryAfter = discoveries.map((binding) => rebindAgentDiscovery(binding, replacements));
+  const discoveryAfter = discoveries.map((binding) => {
+    const next = rebindAgentDiscovery(binding, replacements);
+    for (const [index, source] of binding.sources.entries()) {
+      if (source.path !== path && JSON.stringify(next.sources[index]) !== JSON.stringify(source)) {
+        throw new Error(`Unrelated native discovery input changed during Claude update: ${source.path}`);
+      }
+    }
+    return JSON.stringify(next) === JSON.stringify(binding) ? binding : next;
+  });
   const nextPolicy = { ...policy, bridge: { ...bridge, discovery: Object.fromEntries(discoveryAfter.map((binding) => [binding.agent, binding])) } };
   const changes = before === options.replacement ? [] : [{ path, before, after: options.replacement }];
   if (JSON.stringify(nextPolicy) !== JSON.stringify(policy))
     changes.push({ path: policyPath, before: snapshot.text, after: serializeManagedSkillPolicy(nextPolicy) });
   recheckRootAliases(aliases);
   const nativeSkills = inventoryNativeSkills(home, {
-    agents,
+    agents: consumers,
     projectDir: home,
     includeVendor: true,
-    guardHermes: agents.includes("hermes"),
+    guardHermes: consumers.includes("hermes"),
     allowRootAliases: aliases.length > 0,
-    agentRoots: discoveries.flatMap((binding) => binding.roots.map((path2) => ({ agent: binding.agent, path: path2 })))
+    agentRoots: discoveries.filter((binding) => consumers.includes(binding.agent)).flatMap((binding) => binding.roots.map((path2) => ({ agent: binding.agent, path: path2 })))
   });
   return {
     dataDir: dataDir2,
@@ -13504,7 +13548,7 @@ function planClaudeStopHookUpdate(options) {
     discoveryBefore: discoveries,
     discoveryAfter,
     rootAliases: aliases,
-    managedAgentChecks: { home, agents: [...agents] }
+    managedAgentChecks: { home, agents: consumers }
   };
 }
 function atomicWrite2(path, content) {
@@ -13531,8 +13575,12 @@ function applyAgentIntegration(plan) {
   recheckRootAliases(aliases);
   for (const agent of plan.managedAgentChecks?.agents ?? [])
     assertManagedAgentBridge(agent, { home: plan.managedAgentChecks.home, dataDir: plan.dataDir, projectDir: plan.managedAgentChecks.home });
-  for (const binding of plan.discoveryBefore ?? [])
-    verifyAgentDiscovery(binding);
+  for (const binding of plan.discoveryBefore ?? []) {
+    if (plan.managedAgentChecks)
+      verifyCoordinatedDiscovery(binding, plan.managedAgentChecks.home, aliases);
+    else
+      verifyAgentDiscovery(binding);
+  }
   if (plan.observedPolicy && currentText(plan.observedPolicy.path) !== plan.observedPolicy.before)
     throw new Error("Agent policy changed after planning");
   for (const change of plan.changes)
@@ -13564,8 +13612,12 @@ function applyAgentIntegration(plan) {
       written.push({ path: change.path, before: change.before, after });
     }
     recheckRootAliases(aliases);
-    for (const binding of plan.discoveryAfter ?? [])
-      verifyAgentDiscovery(binding);
+    for (const binding of plan.discoveryAfter ?? []) {
+      if (plan.managedAgentChecks)
+        verifyCoordinatedDiscovery(binding, plan.managedAgentChecks.home, aliases);
+      else
+        verifyAgentDiscovery(binding);
+    }
     for (const agent of plan.managedAgentChecks?.agents ?? [])
       assertManagedAgentBridge(agent, { home: plan.managedAgentChecks.home, dataDir: plan.dataDir, projectDir: plan.managedAgentChecks.home });
     atomicWrite2(join16(backupRoot, "receipt.json"), JSON.stringify({ version: 1, changes: written.map((change) => ({ path: change.path, beforeHash: change.before === null ? null : sha(change.before), afterHash: sha(change.after) })), backups, ...aliases.length ? { rootAliases: aliases } : {} }) + `
@@ -43658,6 +43710,8 @@ export {
   planSkillsS3SnapshotUpload,
   planPluginAdmission,
   planClaudeStopHookUpdate,
+  planClaudePreToolUseHookUpdate,
+  planClaudeHookEventsUpdate,
   planAgentIntegration,
   pinSkill,
   pinProjectSkill,
@@ -43861,6 +43915,7 @@ export {
   DEFAULT_EXPORT_DIR,
   ContentHashInputError,
   CONTENT_HASH_LIMITS,
+  CLAUDE_COORDINATED_HOOK_EVENTS,
   CATEGORIES,
   BASIC_SKILL_NAMES,
   ARTICLE_GENERATION_SLUG,

package/dist/lib/agent-integration.d.ts

@@ -1,6 +1,7 @@
 import { type AgentDiscoveryBinding, type ReviewedDiscoveryInputs } from "./agent-discovery.js";
 import { type IntegrationAgent } from "./agent-adapters.js";
 import type { CodexHookDiscoveryRecovery } from "./codex-hook-discovery-recovery.js";
+import { type ClaudeCoordinatedHookEvent } from "./claude-settings-witness.js";
 export type { IntegrationAgent } from "./agent-adapters.js";
 export type ContextHookEvent = "UserPromptSubmit" | "SessionStart" | "SubagentStart";
 export interface AgentRootAlias {
@@ -96,11 +97,18 @@ export declare function planAgentIntegration(options: {
  * every other settings change and owns discovery witnesses and guarded writes.
  * No policy means this installer has no Skills integration to update.
  */
-export declare function planClaudeStopHookUpdate(options: {
+export interface ClaudeHookUpdateOptions {
     home?: string;
     dataDir?: string;
     expectedSettingsSha256: string;
     replacement: string;
+}
+export declare function planClaudeStopHookUpdate(options: ClaudeHookUpdateOptions): AgentIntegrationPlan | null;
+/** Cooperating safety installers may replace only PreToolUse. The caller owns
+ * its exact hook authorization; Skills retains discovery and native guards. */
+export declare function planClaudePreToolUseHookUpdate(options: ClaudeHookUpdateOptions): AgentIntegrationPlan | null;
+export declare function planClaudeHookEventsUpdate(options: ClaudeHookUpdateOptions & {
+    events: readonly ClaudeCoordinatedHookEvent[];
 }): AgentIntegrationPlan | null;
 export declare function applyAgentIntegration(plan: AgentIntegrationPlan): {
     changed: string[];

package/dist/lib/claude-settings-witness.d.ts

@@ -9,12 +9,16 @@ export declare const CLAUDE_SETTINGS_WITNESS_LIMITS: Readonly<{
 export interface ClaudeSettingsWitnessBudget {
     remaining: number;
 }
-/** Cooperating installers may replace Stop only. Compare the strict syntax
+/** Cooperating installers may replace only their explicitly selected event. Compare the strict syntax
  * trees, not JSON.parse values or the preference-tolerant witness digest: all
  * other hooks, discovery settings, unknown values and number spellings remain
  * identical. Whitespace and object-key order have no configuration meaning.
  */
+export declare const CLAUDE_COORDINATED_HOOK_EVENTS: readonly ["PreToolUse", "PostToolUse", "Stop", "Notification", "SessionStart", "SessionEnd", "UserPromptSubmit", "SubagentStart"];
+export type ClaudeCoordinatedHookEvent = typeof CLAUDE_COORDINATED_HOOK_EVENTS[number];
+export declare function assertClaudeHookEventsReplacement(before: string | null, replacement: string, events: readonly ClaudeCoordinatedHookEvent[]): void;
 export declare function assertClaudeStopHookReplacement(before: string | null, replacement: string): void;
+export declare function assertClaudePreToolUseHookReplacement(before: string | null, replacement: string): void;
 /** Hash only reviewed installer-rendered settings after the disk preimage has been verified. */
 export declare function hashClaudeSettingsReplacement(text: string, budget: ClaudeSettingsWitnessBudget): string;
 /** Explicit capture never converts a stored legacy raw witness or changes settings. */

package/dist/sdk/index.js

@@ -32373,7 +32373,7 @@ var init_event_sink = __esm(() => {
 // package.json
 var package_default = {
   name: "@hasna/skills",
-  version: "0.9.13",
+  version: "0.9.14",
   description: "Skills library for AI coding agents",
   type: "module",
   bin: {
@@ -40126,32 +40126,37 @@ function canonical3(value) {
     return { kind: "array", items: value.items.map(canonical3) };
   return value;
 }
-function assertClaudeStopHookReplacement(before, replacement) {
+var CLAUDE_COORDINATED_HOOK_EVENTS = ["PreToolUse", "PostToolUse", "Stop", "Notification", "SessionStart", "SessionEnd", "UserPromptSubmit", "SubagentStart"];
+function assertClaudeHookEventsReplacement(before, replacement, events) {
+  need3(Array.isArray(events) && events.length > 0 && events.length <= CLAUDE_COORDINATED_HOOK_EVENTS.length && new Set(events).size === events.length && events.every((event) => CLAUDE_COORDINATED_HOOK_EVENTS.includes(event)), "requires explicit supported hook events");
+  const label = events.join(",");
   function bounded(text2) {
-    need3(typeof text2 === "string" && Buffer.byteLength(text2) <= CLAUDE_SETTINGS_WITNESS_LIMITS.bytes, "Stop update exceeds its byte limit");
+    need3(typeof text2 === "string" && Buffer.byteLength(text2) <= CLAUDE_SETTINGS_WITNESS_LIMITS.bytes, `${label} update exceeds its byte limit`);
     return parse6(text2);
   }
   const original = bounded(before ?? "{}"), next = bounded(replacement);
   const hadHooks = original.entries.some(([key]) => key === "hooks");
-  function withoutStop(value) {
+  function withoutOwnedEvents(value) {
     const hooks = value.entries.find(([key]) => key === "hooks")?.[1];
     if (hooks !== undefined) {
-      need3(hooks.kind === "object", "Stop update requires an object of hooks");
-      const stop = hooks.entries.find(([key]) => key === "Stop")?.[1];
-      need3(stop === undefined || stop.kind === "array", "Stop update requires a Stop array");
-      hooks.entries = hooks.entries.filter(([key]) => key !== "Stop");
+      need3(hooks.kind === "object", `${label} update requires an object of hooks`);
+      for (const event of events) {
+        const selected = hooks.entries.find(([key]) => key === event)?.[1];
+        need3(selected === undefined || selected.kind === "array", `${event} update requires a ${event} array`);
+      }
+      hooks.entries = hooks.entries.filter(([key]) => !events.includes(key));
       if (!hadHooks && hooks.entries.length === 0)
         value.entries = value.entries.filter(([key]) => key !== "hooks");
     }
     return canonical3(value);
   }
-  need3(JSON.stringify(withoutStop(original)) === JSON.stringify(withoutStop(next)), "Stop update changed settings outside hooks.Stop");
+  need3(JSON.stringify(withoutOwnedEvents(original)) === JSON.stringify(withoutOwnedEvents(next)), `${label} update changed settings outside hooks.${label}`);
 }
 var PERSISTED_EFFORT_LEVELS = new Set(["low", "medium", "high", "xhigh"]);
 function inferencePreference(key, child) {
   if (key === "skipDangerousModePermissionPrompt") {
     need3(child.kind === "literal" && (child.value === "true" || child.value === "false"), "contains an invalid dialog acknowledgment");
-    return true;
+    return false;
   }
   if (key === "effortLevel") {
     need3(child.kind === "string" && PERSISTED_EFFORT_LEVELS.has(child.value), "contains an invalid persisted effort level");
@@ -41301,6 +41306,34 @@ function planAgentIntegration(options) {
   return { dataDir: dataDir2, profileId, changes, nativeSkills, observedPolicy: { path: policyPath, before: previousPolicy }, discoveryBefore: discoveries, discoveryAfter, ...aliases.length ? { rootAliases: aliases } : {} };
 }
 function planClaudeStopHookUpdate(options) {
+  return planClaudeHookEventsUpdate({ ...options, events: ["Stop"] });
+}
+function planClaudePreToolUseHookUpdate(options) {
+  return planClaudeHookEventsUpdate({ ...options, events: ["PreToolUse"] });
+}
+function claudeSettingsConsumers(home, path, discoveries, aliases) {
+  const canonical4 = (value) => canonicalAgentPath(value, aliases);
+  const containsSettings = (root2) => {
+    const distance = relative3(canonical4(root2), path);
+    return distance === "" || distance !== ".." && !distance.startsWith(`..${sep4}`) && !isAbsolute15(distance);
+  };
+  return discoveries.filter((binding) => binding.agent === "claude" || binding.agent === "opencode" || canonical4(join21(home, AGENT_ADAPTERS[binding.agent].config)) === path || binding.sources.some((source) => canonical4(source.path) === path) || binding.roots.some(containsSettings) || binding.directories?.some((directory) => containsSettings(directory.path))).map((binding) => binding.agent);
+}
+function verifyCoordinatedDiscovery(binding, home, aliases) {
+  try {
+    if (binding.method !== "automatic" && binding.method !== "reviewed")
+      throw new Error("Invalid managed discovery method");
+    verifyAgentDiscovery(binding);
+    if (binding.method === "automatic") {
+      const current = resolveAgentDiscovery({ home, agent: binding.agent, canonical: (path) => canonicalAgentPath(path, aliases) });
+      if (JSON.stringify(current) !== JSON.stringify(binding))
+        throw new Error("Configured native discovery roots changed during Claude update");
+    }
+  } catch (error) {
+    throw new Error(`NATIVE_SKILL_DRIFT: ${error.message}`);
+  }
+}
+function planClaudeHookEventsUpdate(options) {
   const home = resolve14(options.home ?? homedir3()), dataDir2 = options.dataDir ?? getDataDirReadOnly();
   const policyPath = join21(dataDir2, "agent-policy.json");
   assertSafePath(policyPath);
@@ -41318,31 +41351,42 @@ function planClaudeStopHookUpdate(options) {
   const before = lstatSync18(path, { throwIfNoEntry: false }) ? new TextDecoder("utf-8", { fatal: true, ignoreBOM: true }).decode(readNativeBytes(path, 1024 * 1024)) : null;
   if ((before === null ? "absent" : sha2(before)) !== options.expectedSettingsSha256)
     throw new Error("Claude settings preimage changed");
-  assertClaudeStopHookReplacement(before, options.replacement);
+  assertClaudeHookEventsReplacement(before, options.replacement, options.events);
   const agents = bridge.agents;
   const discoveries = Object.entries(bridge.discovery).map(([agent, binding]) => {
-    if (!agents.includes(agent) || binding?.agent !== agent)
-      throw new Error("Invalid managed discovery owner");
+    if (!agents.includes(agent) || binding?.agent !== agent || typeof bridge.commands?.[agent] !== "string" || typeof bridge.profiles?.[agent] !== "string")
+      throw new Error("Invalid managed discovery owner or command/profile binding");
     return binding;
   });
-  if (discoveries.length !== agents.length)
-    throw new Error("Missing managed discovery coverage");
-  for (const agent of agents)
+  if (discoveries.length !== agents.length || !agents.includes("claude"))
+    throw new Error("Missing managed Claude discovery coverage");
+  for (const binding of discoveries)
+    verifyCoordinatedDiscovery(binding, home, aliases);
+  const consumers = claudeSettingsConsumers(home, path, discoveries, aliases);
+  for (const agent of consumers)
     assertManagedAgentBridge(agent, { home, dataDir: dataDir2, projectDir: home });
   const replacements = new Map(before === options.replacement ? [] : [[path, options.replacement]]);
-  const discoveryAfter = discoveries.map((binding) => rebindAgentDiscovery(binding, replacements));
+  const discoveryAfter = discoveries.map((binding) => {
+    const next = rebindAgentDiscovery(binding, replacements);
+    for (const [index, source] of binding.sources.entries()) {
+      if (source.path !== path && JSON.stringify(next.sources[index]) !== JSON.stringify(source)) {
+        throw new Error(`Unrelated native discovery input changed during Claude update: ${source.path}`);
+      }
+    }
+    return JSON.stringify(next) === JSON.stringify(binding) ? binding : next;
+  });
   const nextPolicy = { ...policy, bridge: { ...bridge, discovery: Object.fromEntries(discoveryAfter.map((binding) => [binding.agent, binding])) } };
   const changes = before === options.replacement ? [] : [{ path, before, after: options.replacement }];
   if (JSON.stringify(nextPolicy) !== JSON.stringify(policy))
     changes.push({ path: policyPath, before: snapshot2.text, after: serializeManagedSkillPolicy(nextPolicy) });
   recheckRootAliases(aliases);
   const nativeSkills = inventoryNativeSkills(home, {
-    agents,
+    agents: consumers,
     projectDir: home,
     includeVendor: true,
-    guardHermes: agents.includes("hermes"),
+    guardHermes: consumers.includes("hermes"),
     allowRootAliases: aliases.length > 0,
-    agentRoots: discoveries.flatMap((binding) => binding.roots.map((path2) => ({ agent: binding.agent, path: path2 })))
+    agentRoots: discoveries.filter((binding) => consumers.includes(binding.agent)).flatMap((binding) => binding.roots.map((path2) => ({ agent: binding.agent, path: path2 })))
   });
   return {
     dataDir: dataDir2,
@@ -41353,7 +41397,7 @@ function planClaudeStopHookUpdate(options) {
     discoveryBefore: discoveries,
     discoveryAfter,
     rootAliases: aliases,
-    managedAgentChecks: { home, agents: [...agents] }
+    managedAgentChecks: { home, agents: consumers }
   };
 }
 function atomicWrite2(path, content) {
@@ -41380,8 +41424,12 @@ function applyAgentIntegration(plan) {
   recheckRootAliases(aliases);
   for (const agent of plan.managedAgentChecks?.agents ?? [])
     assertManagedAgentBridge(agent, { home: plan.managedAgentChecks.home, dataDir: plan.dataDir, projectDir: plan.managedAgentChecks.home });
-  for (const binding of plan.discoveryBefore ?? [])
-    verifyAgentDiscovery(binding);
+  for (const binding of plan.discoveryBefore ?? []) {
+    if (plan.managedAgentChecks)
+      verifyCoordinatedDiscovery(binding, plan.managedAgentChecks.home, aliases);
+    else
+      verifyAgentDiscovery(binding);
+  }
   if (plan.observedPolicy && currentText(plan.observedPolicy.path) !== plan.observedPolicy.before)
     throw new Error("Agent policy changed after planning");
   for (const change of plan.changes)
@@ -41413,8 +41461,12 @@ function applyAgentIntegration(plan) {
       written.push({ path: change.path, before: change.before, after });
     }
     recheckRootAliases(aliases);
-    for (const binding of plan.discoveryAfter ?? [])
-      verifyAgentDiscovery(binding);
+    for (const binding of plan.discoveryAfter ?? []) {
+      if (plan.managedAgentChecks)
+        verifyCoordinatedDiscovery(binding, plan.managedAgentChecks.home, aliases);
+      else
+        verifyAgentDiscovery(binding);
+    }
     for (const agent of plan.managedAgentChecks?.agents ?? [])
       assertManagedAgentBridge(agent, { home: plan.managedAgentChecks.home, dataDir: plan.dataDir, projectDir: plan.managedAgentChecks.home });
     atomicWrite2(join21(backupRoot, "receipt.json"), JSON.stringify({ version: 1, changes: written.map((change) => ({ path: change.path, beforeHash: change.before === null ? null : sha2(change.before), afterHash: sha2(change.after) })), backups, ...aliases.length ? { rootAliases: aliases } : {} }) + `

package/docs/plugin-admission.md

@@ -294,9 +294,11 @@ addition to the v1 display/model selections, it tolerates the documented
 persisted effort values `low`, `medium`, `high`, and `xhigh`, both at the top
 level and in known built-in model entries whose only property is `effortLevel`.
 Custom model entries and entries with any other property remain fully bound.
-The boolean `skipDangerousModePermissionPrompt` is Claude's saved dialog
-acknowledgment; it does not select a permission mode. Permission settings,
-plugins, hooks, environment, ultracode and unknown configuration remain bound.
+The boolean `skipDangerousModePermissionPrompt` remains bound: Claude also uses
+this saved acceptance in bypass-mode and pending project MCP approval decisions.
+Permission settings, plugins, hooks, environment, ultracode and unknown
+configuration remain bound. Version 0.9.14 corrects the 0.9.13 normalization of
+this flag; an existing v2 review containing it must be explicitly reviewed again.
 Invalid recognized preference types are refused.
 
 `captureClaudeMarketplaceRegistryV2(path)` creates a
@@ -322,3 +324,25 @@ preference: its owning installer must coordinate a guarded settings/policy
 update, or it needs a fresh explicit review.
 
 The effort schema follows [Claude's model configuration documentation](https://code.claude.com/docs/en/model-config#adjust-effort-level).
+
+Cooperating installers can use `planClaudeHookEventsUpdate` with an explicit
+selection of supported Claude hook events. `planClaudePreToolUseHookUpdate` and
+`planClaudeStopHookUpdate` are single-event convenience wrappers. Supported events
+are `PreToolUse`, `PostToolUse`, `Stop`, `Notification`, `SessionStart`, `SessionEnd`
+`UserPromptSubmit` and `SubagentStart`. The transaction requires the exact settings SHA-256
+preimage and preserves every field outside the selected events, including other
+hooks. Apply the returned plan through `applyAgentIntegration`; this validates
+existing managed discovery and native guards, writes settings and policy with
+precondition checks, and retains recovery backups. An invalid policy or existing
+drift is a refusal, never permission to use an uncoordinated writer. A null plan
+means no managed Skills policy exists. These APIs do not authorize the hook's
+behavior or turn a configuration-changing hook into a preference.
+
+The transaction verifies every agent's discovery sources, directories and
+automatic discovery closure. Native runtime and trust checks cover Claude and
+the declared consumers of the settings path, including containing roots or
+directories and OpenCode's Claude compatibility surface. Unrelated agents keep
+their exact witnesses and trust state; they must still pass their own consumer
+checks before use. The plan's `managedAgentChecks.agents` and `nativeSkills`
+describe this scoped runtime check, not a fleet health result. A path-identity
+witness intersecting the write still refuses instead of adopting a new inode.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hasna/skills",
-  "version": "0.9.13",
+  "version": "0.9.14",
   "description": "Skills library for AI coding agents",
   "type": "module",
   "bin": {