Remove duplicate Patrol scoped-run scope chip

This commit is contained in:
rcourtman
2026-03-25 22:11:50 +00:00
parent bb6571fd20
commit 551b5eef8a
3 changed files with 6 additions and 1 deletions
@@ -423,6 +423,10 @@ dedicated `frontend-modern/src/features/patrol/patrolInvestigationContextModel.t
owner, so the Patrol hook composes one canonical payload-to-summary derivation
instead of rebuilding recent-change, correlation, and governed-resource count
copy inline.
That same normalization applies to supporting effort strips inside the expanded
run card. Once the run presenter is already carrying canonical coverage copy,
secondary chips must not reintroduce a raw `Scoped to N resources` variant that
re-opens the same ambiguity.
The Patrol page's run-history tab label is now also tightened to `Runs`, while
the underlying run-history panel remains canonical for snapshot filtering and
tool-call inspection. That copy change is intentional: run history is support
@@ -391,7 +391,7 @@ export function RunHistoryEntry(props: RunHistoryEntryProps) {
{((run.input_tokens || 0) + (run.output_tokens || 0)).toLocaleString()} tokens
</span>
</Show>
<Show when={run.type === 'scoped'}>
<Show when={run.type === 'scoped' && !coverageSummary}>
<span class="inline-flex items-center gap-1 px-1.5 py-0.5 rounded bg-blue-50 text-blue-600 dark:bg-blue-900 dark:text-blue-400 text-[10px] font-medium">
{formatScope(run) || 'Scoped'}
</span>
@@ -139,6 +139,7 @@ describe('RunHistoryEntry', () => {
expect(screen.getByText('• Checked 1 of 2 scoped resources')).toBeInTheDocument();
expect(screen.getByText('Resources checked (1 of 2 scoped)')).toBeInTheDocument();
expect(screen.queryByText('• 1 resources')).not.toBeInTheDocument();
expect(screen.queryByText('Scoped to 2 resources')).not.toBeInTheDocument();
expect(
screen.getByText((_, element) =>
element?.tagName === 'P' &&