@bahulam/code 0.1.7 → 0.1.8

package/src/terminal/repl.mjs

@@ -22,7 +22,7 @@ import { execSync as _execSync } from 'node:child_process';
 import { Writable as _WritableStream } from 'node:stream';
 import { c, progressBar, spinner, inPlace, renderMarkdown, renderDiff, formatElapsed, formatCost, stripAnsi } from './ansi.mjs';
 import { calculateCost, formatCostValue, formatTokens, costToCredits, formatCredits } from '../core/pricing.mjs';
-import { TarangStreamClient, EVENT_TYPES } from '../core/stream-client.mjs';
+import { BahulamStreamClient, EVENT_TYPES } from '../core/stream-client.mjs';
 import { AgentHistoryTurnBuilder } from '../core/agent-history.mjs';
 import { JsonlWriter } from '../core/jsonl-writer.mjs';
 import { tapSseEvent, registerBroadcaster } from '../daemon/event-tap.mjs';
@@ -53,7 +53,7 @@ import {
   MODES as V_MODES,
 } from '../state/verbosity.mjs';
 import { persistProjectArtifacts } from '../core/project-artifacts.mjs';
-import { TarangAuth } from '../auth/tarang-auth.mjs';
+import { BahulamAuth } from '../auth/bahulam-auth.mjs';
 import { ApprovalManager } from '../core/approval.mjs';
 import * as telemetry from '../telemetry/index.mjs';
 import { resolveBackendUrl } from '../core/backend-url.mjs';
@@ -1586,8 +1586,8 @@ function foldSubAgentToolResult(data = {}) {
     fold.entries.push(entry);
   }
   const durationMs = data?.duration_ms ?? (data?.duration_s != null ? data.duration_s * 1000 : null);
-  const summary = summarizeResult(tool, data);
   entry.args = entry.args && Object.keys(entry.args).length ? entry.args : args;
+  const summary = summarizeResult(tool, data, entry.args);
   entry.result = data;
   entry.durationMs = durationMs;
   entry.outcome = summary.text || '';
@@ -2358,7 +2358,7 @@ function renderEvent(event) {
       break;
 
     // Live steering ack events (PRD-081 §5.2). renderEvent is called from
-    // the SSE loop (see `for await ... jsonlWriter.writeKeplerEvent(event)`
+    // the SSE loop (see `for await ... jsonlWriter.writeBahulamEvent(event)`
     // in the /execute path) so every event already lands in the transcript.
     // Here we only render the user-visible surface — no direct jsonlWriter
     // calls (it's out of scope in this top-level dispatcher anyway).
@@ -3776,7 +3776,7 @@ export async function startTerminalRepl() {
   safeCwd(); // prime the cache in repl-utils.mjs for later recovery
 
   const cliArgs = parseArgs(process.argv.slice(2));
-  const auth = new TarangAuth();
+  const auth = new BahulamAuth();
 
   // Projects are registered and indexed on demand through get_project_overview.
   // CheckpointManager records per-file snapshots before edits so /undo works.
@@ -4123,7 +4123,7 @@ export async function startTerminalRepl() {
       && richHistory.summary
       && Number(richHistory.summaryCoveredMessageCount) > Number(richHistory.summaryCheckpointMessageCount || 0)
     ) {
-      jsonlWriter.writeKeplerEvent({
+      jsonlWriter.writeBahulamEvent({
         type: 'resume_summary',
         data: {
           session_id: sessionId,
@@ -4138,7 +4138,7 @@ export async function startTerminalRepl() {
         },
       });
     }
-    jsonlWriter.writeKeplerEvent({
+    jsonlWriter.writeBahulamEvent({
       type: 'resume_context',
       data: {
         session_id: sessionId,
@@ -4424,7 +4424,7 @@ export async function startTerminalRepl() {
   // Proxy stream: swallows writes when the dock owns the input row so
   // readline's echoes and _refreshLine cursor moves can't fight the dock's
   // absolute cursor placement (PRD-081 §5.1 — cursor race). When the dock
-  // is not mounted (plain-mode fallback, KEPLER_FIXED_INPUT=0, non-TTY),
+  // is not mounted (plain-mode fallback, BAHULAM_FIXED_INPUT=0, non-TTY),
   // writes pass through so history navigation / backspace still redraw.
   const readlineOutputProxy = new _WritableStream({
     write(chunk, _enc, cb) {
@@ -4471,7 +4471,7 @@ export async function startTerminalRepl() {
     // 3 rows fits comfortably in the dock's reservation without leaking
     // past the safety row.
     if (isInputDockMounted()) return 3;
-    const raw = process.env.KEPLER_PROMPT_BOTTOM_PADDING ?? '5';
+    const raw = process.env.BAHULAM_PROMPT_BOTTOM_PADDING ?? '5';
     const n = Number.parseInt(raw, 10);
     if (!Number.isFinite(n) || n <= 0) return 0;
     return Math.min(8, n);
@@ -4744,14 +4744,14 @@ export async function startTerminalRepl() {
   //   1. If the terminal supports bracketed paste, we hold flushing until we
   //      see the ESC[201~ end marker — reliable regardless of paste latency.
   //   2. Otherwise we fall back to a short timer that merges bursts arriving
-  //      within KEPLER_PASTE_FLUSH_MS.
+  //      within BAHULAM_PASTE_FLUSH_MS.
   let _lineInFlight = false;
   const _queuedLines = [];
   let _pasteLines = [];
   let _pasteFlushTimer = null;
 
   function pasteFlushDelayMs() {
-    const raw = Number.parseInt(process.env.KEPLER_PASTE_FLUSH_MS || '35', 10);
+    const raw = Number.parseInt(process.env.BAHULAM_PASTE_FLUSH_MS || '35', 10);
     return Number.isFinite(raw) && raw >= 0 ? Math.min(250, raw) : 35;
   }
 
@@ -4897,7 +4897,7 @@ export async function startTerminalRepl() {
     // Create or reuse stream client — sessionId persists across turns.
     // The same client also owns the authenticated vision-analysis preflight.
     if (!streamClient || streamClient.baseUrl !== creds.backendUrl || streamClient.token !== creds.token) {
-      streamClient = new TarangStreamClient({
+      streamClient = new BahulamStreamClient({
         baseUrl: creds.backendUrl,
         token: creds.token,
         toolExecutor,
@@ -4921,7 +4921,7 @@ export async function startTerminalRepl() {
           `  ${c.brand('◇')} ${c.dim(`attached ${docPrep.documents.length} document${docPrep.documents.length === 1 ? '' : 's'}:`)} ` +
           `${docPrep.documents.map(documentSummaryLine).join(c.dim(' · '))}\n`
         );
-        jsonlWriter.writeKeplerEvent({
+        jsonlWriter.writeBahulamEvent({
           type: 'attachments',
           data: { documents: docPrep.documents.map(publicDocumentMetadata) },
         });
@@ -4937,7 +4937,7 @@ export async function startTerminalRepl() {
       });
       if (prepared.attachments.length) {
         process.stderr.write(`  ${c.brand('◇')} ${c.dim(`attached ${prepared.attachments.length} image${prepared.attachments.length === 1 ? '' : 's'}:`)} ${prepared.attachments.map(attachmentSummaryLine).join(c.dim(' · '))}\n`);
-        jsonlWriter.writeKeplerEvent({
+        jsonlWriter.writeBahulamEvent({
           type: 'attachments',
           data: { attachments: prepared.attachments.map(publicAttachmentMetadata) },
         });
@@ -4954,7 +4954,7 @@ export async function startTerminalRepl() {
           });
           process.stderr.write(`\r${' '.repeat(80)}\r`);
           process.stderr.write(`  ${c.green('✓')} ${c.dim(`Vision analysis completed for ${prepared.attachments.length} image${prepared.attachments.length === 1 ? '' : 's'}`)}${analysis.model ? c.dim(` · ${analysis.model}`) : ''}\n`);
-          jsonlWriter.writeKeplerEvent({
+          jsonlWriter.writeBahulamEvent({
             type: 'vision_analysis',
             data: {
               model: analysis.model || '',
@@ -5140,7 +5140,7 @@ export async function startTerminalRepl() {
       // Persist the local record regardless of outcome so the transcript
       // reflects what the user typed. Delivered/queued follow-ups will
       // get their SSE ack events written separately by the event handler.
-      jsonlWriter.writeKeplerEvent({
+      jsonlWriter.writeBahulamEvent({
         type: 'user_intervention',
         data: {
           instruction,
@@ -5511,7 +5511,7 @@ export async function startTerminalRepl() {
         _turnIterable = client.execute(input, execContext, session.agentHistory);
       }
       for await (const event of _turnIterable) {
-        jsonlWriter.writeKeplerEvent(event);
+        jsonlWriter.writeBahulamEvent(event);
         // . daemon event log. Env-var gated (off by default) —
         // when BAHULAM_DAEMON_EVENTLOG=1, mirror each SSE frame that maps
         // to a first-class type into ~/.bahulam/sessions/<id>/events.jsonl.

package/src/terminal/tool-display.mjs

@@ -90,7 +90,7 @@ function currentWorkingDirectory() {
 //                              handles the display width from here)
 //
 // The point: keep the basename + parent visible even when the full path
-// is long, so users don't see "Editing /Users/sree/Sites/Tarang-Orca/
+// is long, so users don't see "Editing /Users/sree/Sites/.../
 // codekepler-ba…" with the actual filename lost off the right edge.
 function shortPath(filePath, cwd = currentWorkingDirectory()) {
   const value = String(filePath || '');

package/src/tools/agent.mjs

@@ -87,10 +87,10 @@ export const AgentTool = {
             settings: {
                 stream: false,
                 stagnationDetection: !['0', 'false', 'no', 'off'].includes(
-                    (process.env.KEPLER_STAGNATION_DETECTION ?? '0').toLowerCase(),
+                    (process.env.BAHULAM_STAGNATION_DETECTION ?? '0').toLowerCase(),
                 ),
                 stagnationThreshold: Number.parseInt(
-                    process.env.KEPLER_STAGNATION_THRESHOLD
+                    process.env.BAHULAM_STAGNATION_THRESHOLD
                     ?? '3',
                     10,
                 ),

package/src/tools/analyze-image.mjs

@@ -5,7 +5,7 @@
  * sends the same bounded attachment shape used by the preflight vision path to
  * the authenticated backend `/api/vision/analyze` endpoint.
  */
-import { TarangAuth } from '../auth/tarang-auth.mjs';
+import { BahulamAuth } from '../auth/bahulam-auth.mjs';
 import {
   loadImageAttachment,
   publicAttachmentMetadata,
@@ -88,7 +88,7 @@ export const AnalyzeImageTool = {
       };
     }
 
-    const creds = new TarangAuth().loadCredentials();
+    const creds = new BahulamAuth().loadCredentials();
     if (!creds.backendUrl || !creds.token) {
       return {
         success: false,

package/src/tools/generate-image.mjs

@@ -1,5 +1,5 @@
 /**
- * generate_image — text-to-image generation for Kepler Code.
+ * generate_image — text-to-image generation for Bahulam Code.
  *
  * Uses the OpenRouter-compatible image API shape (`POST /v1/images`) and saves
  * generated bytes to local files so the agent can use the assets in code.
@@ -7,7 +7,7 @@
 import fs from 'node:fs/promises';
 import path from 'node:path';
 
-import { TarangAuth } from '../auth/tarang-auth.mjs';
+import { BahulamAuth } from '../auth/bahulam-auth.mjs';
 
 const DEFAULT_IMAGE_GENERATION_MODEL = 'google/gemini-3-pro-image';
 const MAX_GENERATED_IMAGE_BYTES = 10 * 1024 * 1024;
@@ -97,7 +97,7 @@ export const GenerateImageTool = {
             output_path: optionalString(input.output_path),
             filename: optionalString(input.filename),
         };
-        const creds = new TarangAuth().loadCredentials();
+        const creds = new BahulamAuth().loadCredentials();
         const apiKey = gatewayApiKey(creds);
         let result;
         if (creds.backendUrl && creds.token) {
@@ -241,7 +241,7 @@ async function callImageApi({ apiKey, model, prompt, n, aspect_ratio, resolution
                 Authorization: `Bearer ${apiKey}`,
                 'Content-Type': 'application/json',
                 'X-Product': process.env.BAHULAM_PRODUCT || 'kepler',
-                'X-Bahulam-Client': 'kepler-code-image-generation',
+                'X-Bahulam-Client': 'bahulam-code-image-generation',
             },
             body: JSON.stringify(body),
         });

package/src/tools/project-overview.mjs

@@ -322,8 +322,8 @@ function _readIfExists(dir, filename, maxChars = 8000) {
     } catch { return ''; }
 }
 
-function _scanSkills(keplerDir) {
-    const skillsDir = path.join(keplerDir, 'skills');
+function _scanSkills(bahulamDir) {
+    const skillsDir = path.join(bahulamDir, 'skills');
     if (!fs.existsSync(skillsDir)) return [];
     try {
         return fs.readdirSync(skillsDir, { withFileTypes: true })
@@ -402,15 +402,15 @@ export class ProjectRegistry {
     // .bahulam/KEPLER.md, goal/plan/style, skills, AGENTS.md, etc. changes.
     _attachLiveContext(resource, root) {
         // Resolver — prefers .bahulam/, falls back to .kepler/ for legacy projects.
-        const keplerDir = projectConfigDir(root);
+        const bahulamDir = projectConfigDir(root);
         resource.environment = detectEnvironment();
-        resource.project_context = _readIfExists(keplerDir, 'KEPLER.md', 10000) ||
+        resource.project_context = _readIfExists(bahulamDir, 'KEPLER.md', 10000) ||
             _readIfExists(root, 'KEPLER.md', 10000) ||
-            _readIfExists(keplerDir, 'project.md', 8000);
-        resource.style = _readIfExists(keplerDir, 'style.md', 4000);
-        resource.goal = _readIfExists(keplerDir, 'goal.md', 2000);
-        resource.plan = _readIfExists(keplerDir, 'plan.md', 6000);
-        resource.skills_index = _scanSkills(keplerDir);
+            _readIfExists(bahulamDir, 'project.md', 8000);
+        resource.style = _readIfExists(bahulamDir, 'style.md', 4000);
+        resource.goal = _readIfExists(bahulamDir, 'goal.md', 2000);
+        resource.plan = _readIfExists(bahulamDir, 'plan.md', 6000);
+        resource.skills_index = _scanSkills(bahulamDir);
 
         if (!resource.project_context) {
             for (const name of ['.bahulam.md', 'AGENTS.md', 'CLAUDE.md']) {
@@ -426,7 +426,7 @@ export class ProjectRegistry {
             throw new Error('get_project_overview requires a project path');
         }
 
-        // LLM sometimes passes shell-escaped paths ("Tarang\ Orca") or paths
+        // LLM sometimes passes shell-escaped paths ("Bahulam") or paths
         // beginning with "~". Normalize defensively so the tool does not bounce
         // back a "not found" error on a path that's correct apart from quoting.
         rawPath = normalizePathInput(rawPath);
@@ -600,7 +600,7 @@ export class ProjectRegistry {
         }
 
         // LLM frequently passes shell-quoted paths copied from a terminal,
-        // e.g. "Tarang\ Orca/src/app/\(kepler\)/page.tsx". Normalize here so
+        // e.g. "Bahulam/src/app/\(kepler\)/page.tsx". Normalize here so
         // every tool benefits, not just get_project_overview.
         rawPath = normalizePathInput(rawPath);
 
@@ -633,7 +633,7 @@ export class ProjectRegistry {
         let containingScratchRoot = containingProject ? null : findScratchRoot(candidate);
 
         // Two reasons to try the unescaped variant:
-        //   (1) candidate is outside every project root (literal "Tarang\ Orca"
+        //   (1) candidate is outside every project root (literal "Bahulam"
         //       does not contain a real project), or
         //   (2) candidate is inside a root but does not exist on disk because
         //       a path segment like "\(kepler\)" only resolves once unescaped.

package/src/ui/formatter.mjs

@@ -1,5 +1,5 @@
 /**
- * Formatter — Clean terminal output for Tarang events.
+ * Formatter — Clean terminal output for Bahulam events.
  *
  * Style reference: Claude Code screenshot
  *   ◐ Cyan spinner + cyan text for working/thinking
@@ -135,7 +135,7 @@ export class EventFormatter {
         if (!this._lastBlock) return;
         if (compactSame && this._lastBlock === nextBlock) return;
 
-        const mode = String(process.env.KEPLER_BLOCK_SEPARATOR || 'space').toLowerCase();
+        const mode = String(process.env.BAHULAM_BLOCK_SEPARATOR || 'space').toLowerCase();
         if (mode === 'off' || mode === 'none') return;
         if (mode === 'dotted' || mode === 'dots') {
             const cols = Math.max(24, process.stderr.columns || process.stdout.columns || 80);

package/src/ui/input-dock.mjs

@@ -23,15 +23,15 @@
  *        ...                                                  ← up to MAX_INPUT_ROWS
  *                                                              ← spacer
  *   ══════════════════════════════════════════════════════   ← bottom rule (colored)
- *      deepseek-chat-v3 · Tarang-Orca ⎇ main · turn 4         ← meta row
+ *      deepseek-chat-v3 · Bahulam ⎇ main · turn 4         ← meta row
  *      [Enter] send · [/] commands …                          ← tips row
  *   (one blank row at very bottom for cursor safety)
  *
  * Config:
- *   BAHULAM_INPUT_ROWS_MAX / KEPLER_INPUT_ROWS_MAX   1..12  hard cap on input row growth (default 6)
- *   BAHULAM_TTY_MODE=stable / KEPLER_TTY_MODE=stable        scrollback-safe transcript, no fixed dock
- *   BAHULAM_PLAIN=1 / KEPLER_PLAIN=1                        deterministic no-ANSI output for automation
- *   BAHULAM_FIXED_INPUT=0 / KEPLER_FIXED_INPUT=0             disable dock entirely (fallback readline)
+ *   BAHULAM_INPUT_ROWS_MAX   1..12  hard cap on input row growth (default 6)
+ *   BAHULAM_TTY_MODE=stable         scrollback-safe transcript, no fixed dock
+ *   BAHULAM_PLAIN=1                 deterministic no-ANSI output for automation
+ *   BAHULAM_FIXED_INPUT=0           disable dock entirely (fallback readline)
  */
 
 import { paint, width as visibleWidth } from './palette.mjs';
@@ -214,7 +214,7 @@ function inputTextBudget() {
 function resolveMaxInputRows(requested) {
   const raw = requested != null
     ? requested
-    : (process.env.BAHULAM_INPUT_ROWS_MAX || process.env.KEPLER_INPUT_ROWS_MAX);
+    : process.env.BAHULAM_INPUT_ROWS_MAX;
   const n = Number.parseInt(String(raw), 10);
   if (!Number.isFinite(n)) return DEFAULT_MAX_INPUT_ROWS;
   return Math.max(MIN_INPUT_ROWS, Math.min(MAX_INPUT_ROWS_CAP, n));
@@ -530,7 +530,7 @@ export function mountInputDock({
   const t = term();
   if (!t.isTTY || t.plain) return false;
   if (t.ttyMode !== 'rich' || t.fixedInput === false) return false;
-  if (process.env.BAHULAM_FIXED_INPUT === '0' || process.env.KEPLER_FIXED_INPUT === '0') return false;
+  if (process.env.BAHULAM_FIXED_INPUT === '0') return false;
   if (mounted) return true;
 
   inputRowsMax = resolveMaxInputRows(requestedMax);

package/src/ui/spinner.mjs

@@ -16,7 +16,7 @@
  *
  * Behavior:
  *   - 120ms per frame.
- *   - Suppressed entirely when stdout is not a TTY or when KEPLER_PLAIN=1.
+ *   - Suppressed entirely when stdout is not a TTY or when BAHULAM_PLAIN=1.
  *     The `start`/`stop` API is still safe to call (no-op).
  *   - Color follows the current orbit (orchestrated by the caller via the
  *     palette token). Default: brand.primary.

package/src/ui/term.mjs

@@ -34,9 +34,7 @@ function readEnv() {
     NO_COLOR: env.NO_COLOR,
     FORCE_COLOR: env.FORCE_COLOR,
     BAHULAM_PLAIN: env.BAHULAM_PLAIN,
-    KEPLER_PLAIN: env.KEPLER_PLAIN,
     BAHULAM_TTY_MODE: env.BAHULAM_TTY_MODE,
-    KEPLER_TTY_MODE: env.KEPLER_TTY_MODE,
     COLORTERM: (env.COLORTERM || '').toLowerCase(),
     TERM: (env.TERM || '').toLowerCase(),
     TERM_PROGRAM: env.TERM_PROGRAM || '',
@@ -48,7 +46,6 @@ function detectColorLevel(env, isTTY) {
   // Hard opt-out (https://no-color.org). Honored even on TTYs.
   if (env.NO_COLOR !== undefined && env.NO_COLOR !== '') return 'none';
   if (env.BAHULAM_PLAIN === '1') return 'none';
-  if (env.KEPLER_PLAIN === '1') return 'none';
 
   // Hard opt-in. FORCE_COLOR=1|2|3 maps to ansi16|ansi256|truecolor.
   // FORCE_COLOR with no value or =true falls through to detection.
@@ -84,7 +81,6 @@ function detectColorLevel(env, isTTY) {
 
 function detectUnicode(env) {
   if (env.BAHULAM_PLAIN === '1') return false;
-  if (env.KEPLER_PLAIN === '1') return false;
   // Most modern terminals on macOS/Linux handle UTF-8.
   // Windows ConEmu / older terminals are the main holdouts; conservative
   // fallback when LANG and LC_* are missing or explicitly POSIX.
@@ -95,8 +91,8 @@ function detectUnicode(env) {
 }
 
 function detectTtyMode(env) {
-  const raw = String(env.BAHULAM_TTY_MODE || env.KEPLER_TTY_MODE || '').trim().toLowerCase();
-  if (env.BAHULAM_PLAIN === '1' || env.KEPLER_PLAIN === '1') return 'plain';
+  const raw = String(env.BAHULAM_TTY_MODE || '').trim().toLowerCase();
+  if (env.BAHULAM_PLAIN === '1') return 'plain';
   if (raw === 'stable' || raw === 'simple' || raw === 'static' || raw === 'transcript') return 'stable';
   return 'rich';
 }

package/src/ui/tool-card.mjs

@@ -78,12 +78,15 @@ const paintShellAdapter = {
  *
  * @returns {{ text: string, tone: 'success'|'warn'|'danger'|'dim' }}
  */
-export function summarizeResult(tool, data) {
+export function summarizeResult(tool, data, args = {}) {
   if (!data) return { text: '', tone: 'dim' };
 
   if (data._blocked) {
     return { text: firstOutputLine(data) || 'blocked', tone: 'danger' };
   }
+  if (isNoChangeEditResult(tool, data)) {
+    return { text: 'no changes', tone: 'warn' };
+  }
   if (data._observation_timeout) {
     const ms = data._observation_timeout_ms;
     const duration = typeof ms === 'number' ? formatDuration(ms) : '';
@@ -111,7 +114,8 @@ export function summarizeResult(tool, data) {
 
   switch (tool) {
     case 'read_file': {
-      const lines = data._total_lines || lineCount(data.output || data.output_preview);
+      const lines = readFileLineCount(data, args);
+      if (lines == null) return { text: 'read', tone: 'success' };
       return { text: `${lines} line${lines === 1 ? '' : 's'}`, tone: 'success' };
     }
     case 'read_files':
@@ -136,6 +140,9 @@ export function summarizeResult(tool, data) {
     case 'write_file':
     case 'write_project': {
       const delta = diffDelta(data);
+      if (tool === 'edit_file' && delta === '+0 −0') {
+        return { text: 'no changes', tone: 'warn' };
+      }
       if (delta) return { text: delta, tone: 'success' };
       return { text: 'updated', tone: 'success' };
     }
@@ -448,6 +455,64 @@ function lineCount(s) {
   return String(s).split('\n').filter(Boolean).length;
 }
 
+function physicalLineCount(text) {
+  const value = String(text ?? '').replace(/\r\n?/g, '\n');
+  if (!value) return 0;
+  const lines = value.split('\n');
+  if (lines[lines.length - 1] === '') lines.pop();
+  return lines.length;
+}
+
+function readFileLineCount(data = {}, args = {}) {
+  const explicit = explicitReadLineCount(data);
+  if (explicit != null) return explicit;
+
+  const text = firstReadTextPayload(data);
+  if (text != null) return physicalLineCount(text);
+
+  const range = requestedRangeLineCount(args || data?.args || {});
+  if (range != null) return range;
+
+  return null;
+}
+
+function explicitReadLineCount(data = {}) {
+  if (!data || typeof data !== 'object') return null;
+  for (const key of ['_total_lines', 'line_count', 'lines_count', 'total_lines', 'lineCount', 'totalLines']) {
+    const value = data[key];
+    if (value == null || value === '') continue;
+    const number = Number(value);
+    if (Number.isFinite(number) && number >= 0) return Math.floor(number);
+  }
+  if (Array.isArray(data.lines)) return data.lines.length;
+  if (data.result && typeof data.result === 'object' && data.result !== data) {
+    return explicitReadLineCount(data.result);
+  }
+  return null;
+}
+
+function firstReadTextPayload(data = {}) {
+  if (!data || typeof data !== 'object') return null;
+  for (const key of ['content', 'text', 'output', 'output_preview', 'message']) {
+    if (typeof data[key] === 'string') return data[key];
+  }
+  if (data.result && typeof data.result === 'object' && data.result !== data) {
+    return firstReadTextPayload(data.result);
+  }
+  return null;
+}
+
+function requestedRangeLineCount(args = {}) {
+  const start = Number(args.start_line ?? args.startLine);
+  const end = Number(args.end_line ?? args.endLine);
+  if (Number.isFinite(start) && Number.isFinite(end) && start > 0 && end >= start) {
+    return Math.floor(end - start + 1);
+  }
+  const limit = Number(args.limit ?? args.max_lines ?? args.maxLines);
+  if (Number.isFinite(limit) && limit >= 0) return Math.floor(limit);
+  return null;
+}
+
 function countMatches(data) {
   if (typeof data?.match_count === 'number') return data.match_count;
   return lineCount(data?.output);
@@ -474,6 +539,12 @@ function diffDelta(data) {
   return `+${a} −${r}`;
 }
 
+function isNoChangeEditResult(tool, data = {}) {
+  if (tool !== 'edit_file') return false;
+  if (data._no_change || data.no_change) return true;
+  return data.success !== false && data.lines_added === 0 && data.lines_removed === 0;
+}
+
 function tone(text, t) {
   switch (t) {
     case 'success': return paint.state.success(text);
@@ -577,7 +648,7 @@ export function formatCard({ tool, args, result, durationMs, indent, columns, cw
   const cols = columns || term().columns || 120;
   const head = formatCardHead(tool, args, { indent, columns: cols, cwd });
 
-  const summary = summarizeResult(tool, result);
+  const summary = summarizeResult(tool, result, args);
   const duration = formatDuration(durationMs ?? result?.duration_ms ?? (result?.duration_s != null ? result.duration_s * 1000 : null));
 
   if (!summary.text && !duration) return head;