smashimo/oh-my-pi
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor(coding-agent): replaced ctx/Σ labels with icon and cleaner cost separator

Browse Source 
- Removed "ctx" suffix and cumulative Σ-token display from status lines.
- Replaced "N tools" text with tool count + extensionTool icon.
- Changed cost separator to ` . ` to visually distinguish it from dim stats.
- Added test asserting new format and absence of old labels.
This commit is contained in:
can1357
2026-05-30 06:38:09 +02:00
parent 304a9346e9
commit 377ed34e08
3 changed files with 48 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/coding-agent/test/task/render-nested-live.test.ts
+26 -5
View File
@@ -3,12 +3,8 @@ import { resetSettingsForTest, Settings } from "@oh-my-pi/pi-coding-agent/config
import { getThemeByName, setThemeInstance } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
import type { AgentProgress, SingleResult, TaskToolDetails } from "@oh-my-pi/pi-coding-agent/task";
import { taskToolRenderer } from "@oh-my-pi/pi-coding-agent/task/render";
import { formatNumber } from "@oh-my-pi/pi-utils";
// Defends the live-rendering contract for the `task` tool: while a Level-1
// subagent is still mid-flight, any nested `task` activity it has produced
// (already-completed sub-calls in `extractedToolData.task`, plus the in-flight
// snapshot in `inflightTaskDetails`) MUST surface in the parent's streaming
// output — same way it surfaces in the finished result.
describe("task renderer: nested live rendering", () => {
beforeAll(async () => {
resetSettingsForTest();
@@ -22,6 +18,12 @@ describe("task renderer: nested live rendering", () => {
resetSettingsForTest();
});
// Defends the live-rendering contract for the `task` tool: while a Level-1
// subagent is still mid-flight, any nested `task` activity it has produced
// (already-completed sub-calls in `extractedToolData.task`, plus the in-flight
// snapshot in `inflightTaskDetails`) MUST surface in the parent's streaming
// output — same way it surfaces in the finished result.
function makeRunningProgress(overrides: Partial<AgentProgress>): AgentProgress {
return {
index: 0,
@@ -181,4 +183,23 @@ describe("task renderer: nested live rendering", () => {
// Completed entries are emitted before the in-flight snapshot.
expect(epsilonIdx).toBeLessThan(zetaIdx);
});
it("formats running progress stats with tool icon, context window, and cost", async () => {
const theme = (await getThemeByName("dark"))!;
const text = await render(
makeRunningProgress({
toolCount: 19,
contextTokens: 58_000,
contextWindow: 272_000,
cost: 2.1,
durationMs: 0,
}),
);
const expectedStats = `${formatNumber(19)} ${theme.icon.extensionTool} · ${formatNumber(58_000)}/${formatNumber(272_000)} . $2.10`;
expect(text).toContain(expectedStats);
expect(text).not.toContain("tools");
expect(text).not.toContain("ctx");
expect(text).not.toContain("Σ");
});
});