@cmodjs/core 0.1.6 → 0.1.8

package/src/runtime/lifecycle.ts

@@ -6,7 +6,7 @@ import { formatExit, listed, messageOf } from '../utils/text.js'
 import type { Claude } from './claude.js'
 import { beforeDeadline } from './deadline.js'
 import { answerCall, dependencyCalls, notInstalled } from './dependencies.js'
-import { classicHook, userSkillHook, type RoutedEvent } from './hooks.js'
+import { classicHook, permissionEvents, userSkillHook, type RoutedEvent } from './hooks.js'
 import { createRouter, type Router, type RouterNext } from './router.js'
 import { createState } from './state.js'
 import { toolCalls } from './tool-calls.js'
@@ -36,6 +36,7 @@ type ModRuntime = {
   readonly router: Router
   readonly progress: Progress
   readonly dataFolder: string
+  readonly checksPermissions: () => boolean
 }
 
 type ActiveMod<State extends object> = {
@@ -120,7 +121,7 @@ async function cmodVersion(claude: Claude): Promise<string | undefined> {
   return result.stdout.trim().split(/\s+/).at(-1)
 }
 
-export function createLifecycle<State extends object>(definition: ModDefinition<State>): Lifecycle<State> {
+export function createLifecycle<State extends object>(definition: ModDefinition<State>, checksPermissions: () => boolean = () => true): Lifecycle<State> {
   let router: Router = createRouter()
   let phase: Phase = 'starting'
   let runtime: Pick<ModRuntime, 'claude' | 'progress'> | undefined
@@ -136,7 +137,7 @@ export function createLifecycle<State extends object>(definition: ModDefinition<
   const startSettled = new Promise<void>((resolve) => (settleStart = resolve))
 
   const claude = () => {
-    if (runtime === undefined) throw new Error(`${definition.name}: the lifecycle has not started. connect(on, mod) starts it at session.start.`)
+    if (runtime === undefined) throw new Error(`${definition.name}: the lifecycle has not started. registerMod(addHook, mod) starts it at session.start.`)
     return runtime.claude
   }
 
@@ -171,7 +172,7 @@ export function createLifecycle<State extends object>(definition: ModDefinition<
 
   const record = () => {
     recording ??= recordThroughCmod()
-      .catch((error: unknown) => claude().ui.log(`${definition.name} has no CMod record yet: ${messageOf(error)}`, { to: 'debug' }))
+      .catch((error: unknown) => claude().ui.log(`${definition.name} has no cmod record yet: ${messageOf(error)}`, { to: 'debug' }))
       .finally(() => {
         recording = undefined
       })
@@ -180,7 +181,7 @@ export function createLifecycle<State extends object>(definition: ModDefinition<
   const activate = async () => {
     if (runtime === undefined || plugin === undefined) return
     const activeRouter = createRouter()
-    const active = await createMod(definition, { ...runtime, router: activeRouter, dataFolder: dataFolder(plugin.store, plugin.name) }).catch((error: unknown) => {
+    const active = await createMod(definition, { ...runtime, router: activeRouter, dataFolder: dataFolder(plugin.store, plugin.name), checksPermissions }).catch((error: unknown) => {
       fail(messageOf(error), 'Fix it, then run /reload-plugins.')
       throw error
     })
@@ -226,7 +227,7 @@ export function createLifecycle<State extends object>(definition: ModDefinition<
     if (plugin === undefined) return
     if (plugin.name === cmodPluginName) return bootstrap(plugin.root)
     phase = 'installing'
-    if ((await cmodVersion(claude())) === undefined) return waitForCMod()
+    if ((await cmodVersion(claude())) === undefined) return waitForcmod()
     endLine()
     const progress = showLine()
     let outcome: RunnerEvent | undefined
@@ -242,9 +243,9 @@ export function createLifecycle<State extends object>(definition: ModDefinition<
     fail(reason, `Fix the cause, then run: cmod install ${definition.name}`)
   }
 
-  const waitForCMod = () => {
+  const waitForcmod = () => {
     phase = 'waiting'
-    showLine().wait('Waiting for CMod')
+    showLine().wait('Waiting for cmod')
     let check: Promise<void> | undefined
     const stop = () => {
       timer.cancel()
@@ -263,7 +264,7 @@ export function createLifecycle<State extends object>(definition: ModDefinition<
           report(error)
         })
     })
-    const longWait = claude().clock.after(cmodWaitMs, () => showLine().wait("Still waiting for CMod to download cmod. See CMod's own line."))
+    const longWait = claude().clock.after(cmodWaitMs, () => showLine().wait("Still waiting for cmod to download cmod. See cmod's own line."))
   }
 
   const bootstrap = async (root: string) => {
@@ -447,6 +448,10 @@ async function createMod<State extends object>(definition: ModDefinition<State>,
   }
   await failsAs('its state did not load', () => modState.load())
   await failsAs('its setup function threw', () => definition.setup(mod))
+  const permissionHooks = permissionEvents.filter((event) => router.has(event))
+  if (permissionHooks.length > 0 && !runtime.checksPermissions()) {
+    throw new Error(`it decides permissions on ${listed(permissionHooks)}, so hooks/register.ts must call registerPermissionCheck(addHook) after registerMod`)
+  }
   await failsAs('its open panes did not load', () => area.restorePanes())
   if (hookEvents.length > 0) added.push(`${hookEvents.length === 1 ? 'a hook' : 'hooks'} on ${listed(hookEvents)}`)
   return { mod, added }

package/src/runtime/router.ts

@@ -7,6 +7,7 @@ export type RoutedHook<N extends RoutedEvent> = (e: Frozen<Args<N>>, next: Route
 
 export type Router = {
   add<N extends RoutedEvent>(event: N, hook: RoutedHook<N>): void
+  has(event: RoutedEvent): boolean
   dispatch<N extends RoutedEvent>(event: N, e: Frozen<Args<N>>, next: RouterNext<N>): Promise<EventResult<N>>
 }
 
@@ -21,6 +22,7 @@ export function createRouter(): Router {
       chain.push(hook as unknown as ErasedHook)
       hooks.set(event, chain)
     },
+    has: (event) => hooks.has(event),
     dispatch<N extends RoutedEvent>(event: N, e: Frozen<Args<N>>, next: RouterNext<N>): Promise<EventResult<N>> {
       const chain = hooks.get(event)
       const call = async (index: number, current: unknown): Promise<unknown> => {

package/src/runtime/tool-calls.ts

@@ -34,6 +34,11 @@ export function toolCalls(claude: Claude, router: Router): ToolCalls {
     return next(e)
   })
   router.add('tool.call', async (e, next) => {
+    cwds.set(e.tool_use_id, await claude.session.cwd())
+    for (const oldest of cwds.keys()) {
+      if (cwds.size <= keptCalls) break
+      cwds.delete(oldest)
+    }
     if (e.agentId === undefined) return next(e)
     agentIds.set(e.tool_use_id, e.agentId)
     try {
@@ -42,14 +47,6 @@ export function toolCalls(claude: Claude, router: Router): ToolCalls {
       agentIds.delete(e.tool_use_id)
     }
   })
-  router.add('classic.PreToolUse', async (e, next) => {
-    cwds.set(e.tool_use_id, await claude.session.cwd())
-    for (const oldest of cwds.keys()) {
-      if (cwds.size <= keptCalls) break
-      cwds.delete(oldest)
-    }
-    return next(e)
-  })
   for (const event of ['classic.PostToolUse', 'classic.PostToolUseFailure'] as const) {
     router.add(event, async (e, next) => {
       try {

package/vendor.js

@@ -179,9 +179,7 @@ var require_utils = __commonJS((exports) => {
       const platform = navigator.platform.toLowerCase();
       return platform === "win32" || platform === "windows";
     }
-    if (typeof process !== "undefined" && process.platform) {
-      return process.platform === "win32";
-    }
+    if (false) {}
     return false;
   };
   exports.removeBackslashes = (str) => {

package/connect.d.ts

@@ -1,4 +0,0 @@
-import type { On } from 'claude-code';
-import type { ModDefinition } from './mod.js';
-export declare function connect<State extends object>(on: On, definition: ModDefinition<State>): void;
-//# sourceMappingURL=connect.d.ts.map

package/connect.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"connect.d.ts","sourceRoot":"","sources":["src/connect.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAoD,EAAE,EAAE,MAAM,aAAa,CAAA;AACvF,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,UAAU,CAAA;AAwE7C,wBAAgB,OAAO,CAAC,KAAK,SAAS,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,UAAU,EAAE,aAAa,CAAC,KAAK,CAAC,GAAG,IAAI,CA8B5F"}