kimaki 0.19.0 → 0.20.0

package/src/message-formatting.ts

@@ -350,6 +350,54 @@ export async function getFileAttachments(
 
 const MAX_BASH_COMMAND_INLINE_LENGTH = 100
 
+/**
+ * Format the inline title for a bash tool part. Handles three cases:
+ * 1. Short single-line command → show full command
+ * 2. Long/multiline command with description → show description
+ * 3. Long/multiline command without description → truncate first line of command
+ *
+ * The description field was removed from the bash tool schema in newer opencode
+ * versions, so case 3 is the common path now. Without this fallback, long commands
+ * would render as just "┣ bash" with no context.
+ */
+export function formatBashToolTitle({
+  command,
+  description,
+  stateTitle,
+}: {
+  command: string
+  description?: string
+  stateTitle?: string
+}): string {
+  if (!command && !description && !stateTitle) return ''
+
+  const isSingleLine = !command.includes('\n')
+  // Find first non-empty line to handle commands with leading blank lines
+  const firstMeaningfulLine =
+    command
+      .split('\n')
+      .find((line) => line.trim().length > 0)
+      ?.trimStart() ?? ''
+
+  if (command && isSingleLine && command.length <= MAX_BASH_COMMAND_INLINE_LENGTH) {
+    return ` _${escapeInlineMarkdown(command)}_`
+  }
+  if (description) {
+    return ` _${escapeInlineMarkdown(description)}_`
+  }
+  if (firstMeaningfulLine.length > 0) {
+    const truncated =
+      firstMeaningfulLine.length > MAX_BASH_COMMAND_INLINE_LENGTH
+        ? firstMeaningfulLine.slice(0, MAX_BASH_COMMAND_INLINE_LENGTH) + '…'
+        : firstMeaningfulLine
+    return ` _${escapeInlineMarkdown(truncated)}_`
+  }
+  if (stateTitle) {
+    return ` _${escapeInlineMarkdown(stateTitle)}_`
+  }
+  return ''
+}
+
 export function getToolSummaryText(part: Part): string {
   if (part.type !== 'tool') return ''
 
@@ -568,13 +616,10 @@ export function formatPart(part: Part, prefix?: string): string {
       }
       const command = (part.state.input?.command as string) || ''
       const description = (part.state.input?.description as string) || ''
-      const isSingleLine = !command.includes('\n')
-      const toolTitle =
-        isSingleLine && command.length <= MAX_BASH_COMMAND_INLINE_LENGTH
-          ? ` _${escapeInlineMarkdown(command)}_`
-          : description
-            ? ` _${escapeInlineMarkdown(description)}_`
-            : ''
+      const toolTitle = formatBashToolTitle({
+        command,
+        description,
+      })
       return `┣ ${pfx}bash${toolTitle}`
     }
 
@@ -587,14 +632,12 @@ export function formatPart(part: Part, prefix?: string): string {
     } else if (part.tool === 'bash') {
       const command = (part.state.input?.command as string) || ''
       const description = (part.state.input?.description as string) || ''
-      const isSingleLine = !command.includes('\n')
-      if (isSingleLine && command.length <= MAX_BASH_COMMAND_INLINE_LENGTH) {
-        toolTitle = `_${escapeInlineMarkdown(command)}_`
-      } else if (description) {
-        toolTitle = `_${escapeInlineMarkdown(description)}_`
-      } else if (stateTitle) {
-        toolTitle = `_${escapeInlineMarkdown(stateTitle)}_`
-      }
+      const formatted = formatBashToolTitle({
+        command,
+        description,
+        stateTitle,
+      })
+      toolTitle = formatted.startsWith(' ') ? formatted.slice(1) : formatted
     } else if (stateTitle) {
       toolTitle = `_${escapeInlineMarkdown(stateTitle)}_`
     }

package/src/session-handler/thread-session-runtime.ts

@@ -365,23 +365,24 @@ function getTokenTotal(tokens: TokenUsage): number {
   )
 }
 
+/**
+ * Built-in read-only tools that are hidden in default verbosity mode.
+ * Any tool NOT in this list is considered "essential" and shown,
+ * which means custom tools, MCP tools, and plugin tools are visible by default.
+ */
+const HIDDEN_READONLY_TOOLS = [
+  'read',
+  'glob',
+  'grep',
+  'describe-media',
+  'todoread',
+]
+
 /** Check if a tool part is "essential" (shown in text-and-essential-tools mode). */
 export function isEssentialToolName(toolName: string): boolean {
-  const essentialTools = [
-    'edit',
-    'write',
-    'apply_patch',
-    'bash',
-    'webfetch',
-    'websearch',
-    'googlesearch',
-    'codesearch',
-    'task',
-    'todowrite',
-    'skill',
-  ]
-  // Also match any MCP tool that contains these names
-  return essentialTools.some((name) => {
+  // Hide known read-only built-in tools; show everything else
+  // (custom tools, MCP tools, plugin tools are visible by default)
+  return !HIDDEN_READONLY_TOOLS.some((name) => {
     return toolName === name || toolName.endsWith(`_${name}`)
   })
 }
@@ -559,6 +560,13 @@ export type IngressInput = {
   sessionStartSource?: { scheduleKind: 'at' | 'cron'; scheduledTaskId?: number }
   /** Optional guard for retries: skip enqueue when session has changed. */
   expectedSessionId?: string
+  /**
+   * When true, the message is added to the session context without triggering
+   * the AI agent loop. Used for messages that should be visible to the model
+   * on the next real turn but should not cause a response on their own
+   * (e.g. user-to-user replies in a thread).
+   */
+  noReply?: boolean
   /**
    * Lazy preprocessing callback. When set, the runtime serializes it via a
    * lightweight promise chain (preprocessChain) to resolve prompt/images/mode
@@ -2838,6 +2846,18 @@ export class ThreadSessionRuntime {
         return
       }
 
+      // Context-only messages (noReply) should not create a new session.
+      // If there is no existing session, silently skip.
+      if (input.noReply) {
+        const existingSessionId = this.state?.sessionId || await getThreadSession(this.thread.id) || undefined
+        if (!existingSessionId) {
+          logger.log(
+            `[INGRESS] Skipping noReply message for thread ${this.threadId}: no existing session`,
+          )
+          return
+        }
+      }
+
       // Helper: stop typing and drain queued local messages on error.
       const cleanupOnError = async (errorMessage: string) => {
         this.stopTyping()
@@ -3063,6 +3083,7 @@ export class ThreadSessionRuntime {
         ...(resolvedAgent ? { agent: resolvedAgent } : {}),
         ...(modelField ? { model: modelField } : {}),
         ...variantField,
+        ...(input.noReply ? { noReply: true } : {}),
       }
       const promptResult = await getClient().session.promptAsync(request)
         .catch((e) => new OpenCodeSdkError({ operation: 'session.promptAsync', cause: e }))
@@ -3082,7 +3103,10 @@ export class ThreadSessionRuntime {
         `[INGRESS] promptAsync accepted by opencode queue sessionId=${session.id} threadId=${this.threadId}`,
       )
 
-      this.markQueueDispatchBusy(session.id)
+      // noReply messages don't trigger the agent loop, so don't mark as busy
+      if (!input.noReply) {
+        this.markQueueDispatchBusy(session.id)
+      }
     })
 
     if (skippedBySessionGuard) {
@@ -3241,8 +3265,15 @@ export class ThreadSessionRuntime {
         // Route with the resolved mode through normal paths.
         // Await the enqueue so session state (ensureSession, setThreadSession)
         // is persisted before the next message's preprocessing reads it.
-        const enqueueResult =
-          resolvedInput.mode === 'local-queue' || resolvedInput.command
+        // noReply messages always go through the opencode path so the flag
+        // reaches promptAsync; local queue doesn't support noReply.
+        const enqueueResult = resolvedInput.noReply
+          ? await this.submitViaOpencodeQueue({
+              ...resolvedInput,
+              mode: 'opencode',
+              command: undefined,
+            })
+          : (resolvedInput.mode === 'local-queue' || resolvedInput.command)
             ? await this.enqueueViaLocalQueue(resolvedInput)
             : await this.submitViaOpencodeQueue(resolvedInput)
         resolveOuter(enqueueResult)

package/src/store.ts

@@ -110,6 +110,13 @@ export type KimakiState = {
   // Read by: cli-runner.ts run() before calling backgroundUpgradeKimaki().
   autoUpgradeEnabled: boolean
 
+  // When true, all new sessions from channel messages create git worktrees.
+  // Set once at startup from --worktrees CLI flag. The per-channel toggle
+  // (getChannelWorktreesEnabled) is checked separately; this is the global override.
+  // Changes: set once at startup.
+  // Read by: discord-bot.ts message handler, commands/agent.ts quick-agent with prompt.
+  useWorktrees: boolean
+
   // Whether background sync of external OpenCode sessions is enabled.
   // When true (default), sessions started from the OpenCode CLI or TUI
   // are mirrored into Discord threads so they can be browsed, searched,
@@ -172,6 +179,7 @@ export const store = createStore<KimakiState>(() => ({
   allowedMentions: ['users'],
   allowAllUsers: false,
   permissionTimeoutMs: 10 * 60 * 1000,
+  useWorktrees: false,
   autoUpgradeEnabled: true,
   syncEnabled: true,
   discordBaseUrl: 'https://discord.com',