@toddzheng024/dscode-bundle 0.7.6 → 0.7.8

package/plugins/session-metrics/view.mjs

@@ -18,6 +18,8 @@ export function summarize(rows, events = [], corrupt = false) {
   const history = [];
   for (const event of events) {
     if (event.type === 'request/header') route = event.data.header.config;
+    // Ledger rows cover everything from the first row's time on, so only older events are
+    // backfilled. `continue`, not `break`: the loop must not depend on event ordering.
     if (event.time >= first) continue;
     if (event.type === 'assistant/message' || event.type === 'compaction/summary' && event.data.llmStreamCall) {
       const r = event.type === 'compaction/summary' ? event.data : route;
@@ -34,7 +36,7 @@ export function summarize(rows, events = [], corrupt = false) {
     const u = row.usage;
     if (!u || !Number.isFinite(u.inputTokens) || !Number.isFinite(u.outputTokens)) { cacheUnknown = true; continue; }
     const total = u.inputTokens + (u.cacheReadTokens ?? 0) + (u.cacheWriteTokens ?? 0);
-    // pi-ai (OpenRouter) reports cache reads only when there are some: a missing count is zero, not unknown.
+    // OpenRouter reports cache reads only when there are some: a missing count is zero, not unknown.
     input += total; hit += u.cacheReadTokens ?? 0;
   }
   return { cost, unknown, calls, pending, cache: input > 0 && !cacheUnknown ? Math.min(100, hit / input * 100) : null };
@@ -82,14 +84,22 @@ export function formatFooter(metrics, context, columns = 80, rates, locale = 'en
   for (const char of floor) { if (displayWidth(clipped + char) > columns) break; clipped += char; }
   return clipped;
 }
+/** Per-events memo: the status line renders up to once a second, and summarize/average are O(events). */
+const footerCache = new WeakMap();
 export function footerFor(id, stats, columns, header = '', locale = 'en') {
   try {
     const data = id ? source?.(id) : undefined;
     const ledger = id && process.env.DSH_HOME ? readMetrics(process.env.DSH_HOME, id) : { rows: [], corrupt: false };
-    const summary = summarize(ledger.rows, data?.events ?? [], ledger.corrupt);
+    const events = data?.events ?? [];
+    // Identity alone is not enough: a session event list may be appended to in place.
+    const hit = events.length > 0 ? footerCache.get(events) : undefined;
+    const tail = events.at(-1)?.time;
+    const fresh = hit !== undefined && hit.key === ledger.rows && hit.length === events.length && hit.tail === tail;
+    const summary = fresh ? hit.summary : summarize(ledger.rows, events, ledger.corrupt);
     const used = data?.used;
     const capacity = data?.capacity ?? stats.contextWindow;
-    const average = sessionAverageTps(data?.events ?? []);
+    const average = fresh ? hit.average : sessionAverageTps(events);
+    if (events.length > 0 && !fresh) footerCache.set(events, { key: ledger.rows, length: events.length, tail, summary, average });
     return formatFooter(summary, Number.isFinite(used) && capacity > 0 ? used / capacity * 100 : undefined, columns, { current: data?.currentTps, average }, locale, header);
   } catch { return formatFooter({ cost: 0, unknown: true, cache: null }, undefined, columns, { current: null, average: null }, locale, header); }
 }

package/plugins/tui-tools/index.mjs

@@ -62,7 +62,8 @@ export async function findConflicts(cwd, configs, winners, env = process.env) {
 
 export function apply(ctx) {
   const diagnosticsHome = process.env.DSH_HOME ?? process.env.DSCODE_HOME;
-  if (diagnosticsHome) ctx.logger.exporter({ levels: { default: 2 }, export: message => recordRuntimeLog(diagnosticsHome, message) });
+  // A minimal composition (and the unit fixtures) may mount no logger; diagnostics are best-effort.
+  if (diagnosticsHome) ctx.logger?.exporter?.({ levels: { default: 2 }, export: message => recordRuntimeLog(diagnosticsHome, message) });
   const entries = agent => [
     ...(ctx.get('loader')?.entries() ?? []),
     ...(agent?.ctx ? standingMountFor(agent.ctx)?.tree.entries() ?? [] : []),

package/plugins/ultra/policy.mjs

@@ -16,22 +16,6 @@ export function flashRequest(options, messages) {
   return copy;
 }
 
-/**
- * The same shaping for pi-ai routes (OpenRouter), on harness-format options:
- * Ultra sends max and adds the collaboration policy. Delegation tools are offered
- * at every effort (the shell policy keeps delegation rare below Ultra); workflow
- * and ralph never are. Returns a copy; the logged input is never mutated.
- */
-export function piAiRequest(options) {
-  const ultra = options.reasoningEffort === 'ultra';
-  const next = { ...options, ...(options.tools ? { tools: options.tools.filter(tool => tool.name !== 'workflow' && tool.name !== 'ralph') } : {}), ...(ultra ? { reasoningEffort: 'max' } : {}) };
-  if (!ultra || options.purpose || !options.tools?.some(t => t.name === 'subagent' || t.name === 'subagent_fork')) return next;
-  if (typeof next.system === 'string') return { ...next, system: next.system + '\n\n' + ULTRA_POLICY };
-  const [first, ...rest] = next.messages ?? [];
-  if (first?.role === 'system' && Array.isArray(first.content)) return { ...next, messages: [{ ...first, content: [...first.content, { type: 'text', text: '\n\n' + ULTRA_POLICY }] }, ...rest] };
-  return { ...next, messages: [{ role: 'system', content: [{ type: 'text', text: ULTRA_POLICY }] }, ...(next.messages ?? [])] };
-}
-
 export function ultraRequest(options, messages) {
   if (options.reasoningEffort !== 'ultra' || options.purpose || !options.tools?.some(t => t.name === 'subagent' || t.name === 'subagent_fork')) return messages;
   const copy = messages.map(m => ({ ...m }));

package/presets/dscode/agent.cordis.yml

@@ -28,10 +28,21 @@
     prefix: >-
       You are a coding agent powered by the {{model}} model, working through a persistent shell.
       Reply in the language the user writes in.
+      Keep the user informed during tool-driven work with ordinary assistant text, visible even when verbose mode is off. Reasoning blocks and tool descriptions do not count as progress updates.
+      Before the first tool call, briefly state what you will check or change. For a simple answer without tools, answer directly.
+      During sustained work, give a concise progress update roughly every minute when you have control, and when a significant finding, blocker or change of direction occurs. If a long tool call prevents an update, summarize its result when control returns; do not claim to speak while a tool is blocking.
+      Use one or two sentences explaining what you learned and what the next step will resolve. Report observable facts and decisions, not private reasoning, raw tool payloads or a narration of every command. Avoid repetitive filler and invented progress.
+      Continue working after each update without asking permission for already-authorized steps. Progress text is not the final answer; finish with a self-contained result and any remaining gaps.
       Before changing code, read the relevant code and any project instructions; reuse existing functions and patterns instead of adding new machinery.
       Make routine judgment calls yourself and ask only when different answers would lead to materially different work.
       Deliver the whole requested scope; if part of it is blocked, finish the rest and say what was left out and why.
       Verify changes by running the relevant checks, and report outcomes faithfully: say when a check fails, when a step was skipped, and when something could not be verified.
+      Before editing, identify the requested scope and observable acceptance checks. Keep them stable unless the user changes the task or evidence shows a necessary dependency.
+      Once the requested behavior and required checks pass, move to delivery. Do not turn a finished task into optional optimization, refactoring or speculative hardening.
+      Treat review findings as claims to verify against the actual code and requirements, not automatic instructions. Fix confirmed defects affecting this task; report unrelated opportunities separately.
+      After a fix, rerun affected checks. Repeat broader checks only when changed behavior or a failure invalidates the earlier result; do not rerun passing suites just to feel more certain.
+      If a check suddenly becomes much slower, inspect the changed workload and the existing output before rerunning it. Keep test fixtures small and independent of production-sized limits.
+      Respect the automatic review budget. Reaching it is not proof of correctness: finish necessary fixes and focused checks, report unresolved issues honestly, and continue already-authorized delivery steps when their requirements are met.
       Keep the final reply concise, lead with the outcome, and never claim work you did not do.
 
 - id: agent-instructions
@@ -180,7 +191,7 @@
     toolResultPruner: true
   config:
     - id: compaction-basic
-      name: '@deepseek-ai/dsh-compaction-basic'
+      name: '@toddzheng024/dscode-bundle/compaction-basic'
 
     - id: command-compact
       name: '@deepseek-ai/dsh-command-compact'