From 5e0a845a86b1f86226b915d7310e9fb23f25cb1b Mon Sep 17 00:00:00 2001 From: Fabro Date: Mon, 25 May 2026 01:12:13 +0000 Subject: [PATCH] fabro(01KSE2PAVXD56N4TWNK4T5H5VA): fixup (succeeded) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fabro-Run: 01KSE2PAVXD56N4TWNK4T5H5VA Fabro-Completed: 13 Fabro-Checkpoint: f120fc68abf014e98ef5e23345f03a995ae3aa3a ⚒️ Generated with [Fabro](https://fabro.sh) --- .../stage-insights-sidebar.test.tsx | 29 ++++--------------- 1 file changed, 5 insertions(+), 24 deletions(-) diff --git a/apps/fabro-web/app/components/stage-insights-sidebar.test.tsx b/apps/fabro-web/app/components/stage-insights-sidebar.test.tsx index 713f1b4bc..9bf0cf530 100644 --- a/apps/fabro-web/app/components/stage-insights-sidebar.test.tsx +++ b/apps/fabro-web/app/components/stage-insights-sidebar.test.tsx @@ -5,7 +5,6 @@ import { MemoryRouter } from "react-router"; import { AgentToolCategory, AgentSkillActivationSource, - PermissionLevel, StageContextWindowCategory, StageContextWindowCountMethod, StageContextWindowStaleness, @@ -148,17 +147,7 @@ describe("StageInsightsSidebar", () => { expect(dom).not.toContain("31%"); }); - test("renders permission badge for read-only", () => { - const dom = render(makeStage({ permission_level: PermissionLevel.READ_ONLY }), null); - expect(dom).toContain("Read-only"); - }); - - test("renders permission badge for full access", () => { - const dom = render(makeStage({ permission_level: PermissionLevel.FULL }), null); - expect(dom).toContain("Full access"); - }); - - test("renders projected agent tool names, descriptions, categories, and invoked state", () => { + test("renders projected agent tool names, descriptions as tooltips, and invoked state", () => { const dom = render( makeStage({ agent_tools: [ @@ -177,28 +166,20 @@ describe("StageInsightsSidebar", () => { invoked: false, }, ], - permission_level: PermissionLevel.FULL, }), null, ); expect(dom).toContain("1/2"); expect(dom).toContain("apply_patch"); + // Descriptions are surfaced as `title` tooltip props on each tool row. expect(dom).toContain("Apply a unified diff patch"); - expect(dom).toContain("write"); - expect(dom).toContain("used"); expect(dom).toContain("grep"); expect(dom).toContain("Search file contents"); - expect(dom).toContain("read"); - expect(dom).toContain("available"); - // Permission remains secondary compatibility metadata, not the source of - // the tool list. - expect(dom).toContain("Full access"); }); - test("legacy stages without agent tools keep permission fallback only", () => { - const dom = render(makeStage({ permission_level: PermissionLevel.READ_WRITE }), null); - expect(dom).toContain("Read/write"); + test("legacy stages without agent tools render no tool rows", () => { + const dom = render(makeStage(), null); expect(dom).not.toContain("apply_patch"); }); @@ -266,6 +247,6 @@ describe("StageInsightsSidebar", () => { const dom = render(undefined, null); // sidebar still renders even with no data expect(dom).toContain("Agent"); - expect(dom).toContain("Unknown"); + expect(dom).toContain("Context"); }); });