Skip to content

fix(claw-app): drop the audit timeline labels and recolor the rows - #2416

Merged
Dani Akash (DaniAkash) merged 1 commit into
mainfrom
fix/audit-remove-risk-labels
Aug 21, 2026
Merged

fix(claw-app): drop the audit timeline labels and recolor the rows#2416
Dani Akash (DaniAkash) merged 1 commit into
mainfrom
fix/audit-remove-risk-labels

Conversation

@DaniAkash

Copy link
Copy Markdown
Contributor

What

The audit timeline in BrowserOS neo labeled act / evaluate / run / download rows as High risk and highlighted them in warning-amber. That characterization is inaccurate, so this removes the label and recolors those rows.

Changes

  • Remove the "High risk" badge from timeline rows.
  • Recolor the emphasized rows from amber to the theme accent (blue) via the primary token; error rows keep their red highlight.
  • The same tool set still auto-expands on load; the internal name is now NOTABLE_TOOLS / isNotable instead of the risk framing.

Verification

claw-app typecheck clean, Biome clean, Timeline tests pass (11).

The audit timeline tagged act/evaluate/run/download rows as "High risk", which
is inaccurate, and highlighted them in warning-amber. Remove the label and switch
the row emphasis to the theme accent (blue); errors keep their red highlight. The
same tool set still auto-expands on load, now named for what it is (notable
actions) rather than risk.
@github-actions github-actions Bot added the fix label Aug 21, 2026
@github-actions

Copy link
Copy Markdown
Contributor

✅ Tests passed: 392/392

Ran 1 of 15 suites (14 not affected by this change).

Suite Passed Failed Skipped
server-agent n/a n/a not affected
server-api n/a n/a not affected
server-tools n/a n/a not affected
server-browser n/a n/a not affected
server-integration n/a n/a not affected
server-lib n/a n/a not affected
server-root n/a n/a not affected
agent n/a n/a not affected
claw-app 392/392 0 0
claw-onboard n/a n/a not affected
build n/a n/a not affected
release n/a n/a not affected
claw-server-rust n/a n/a not affected
claw-server-rust-quality n/a n/a not affected
claw-mcp n/a n/a not affected

View workflow run

@greptile-apps

greptile-apps Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR removes inaccurate “High risk” labeling from notable audit timeline actions while preserving their auto-expand behavior.

  • Renames the internal tool classification from risk-oriented to notable-action terminology.
  • Recolors notable rows with the theme’s primary accent while retaining red error styling.
  • Removes the badge and updates the row grid to match the remaining four fields.
  • Updates timeline test descriptions and comments to reflect the new terminology.

Confidence Score: 5/5

The PR appears safe to merge with no actionable correctness, security, or quality issues identified.

The renamed classification preserves the existing auto-expansion set, the primary accent utilities are supported by the app theme, red error styling retains precedence, and the revised grid matches its four remaining children.

Important Files Changed

Filename Overview
packages/browseros-agent/apps/claw-app/components/audit/Timeline.tsx Removes risk labeling, applies valid primary-token styling, and correctly adjusts the grid for the removed badge.
packages/browseros-agent/apps/claw-app/components/audit/Timeline.test.tsx Updates test names and comments without changing the existing behavioral coverage.

Reviews (1): Last reviewed commit: "fix(claw-app): drop the audit timeline r..." | Re-trigger Greptile

@DaniAkash Dani Akash (DaniAkash) changed the title fix(claw-app): drop the audit timeline risk labels and recolor the rows fix(claw-app): drop the audit timeline labels and recolor the rows Aug 21, 2026
@DaniAkash
Dani Akash (DaniAkash) merged commit e635a80 into main Aug 21, 2026
15 checks passed
@DaniAkash
Dani Akash (DaniAkash) deleted the fix/audit-remove-risk-labels branch August 21, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant