fix(FLEETMDM-010-2): CU-86akj32d7 Brave icon reuses the same uniqueId-generated variable name for two distinct linearGradient defs, causing duplicate 'id' attributes in the DOM

[flamingo]

Closes findings from rule FLEETMDM-010-2 — Brave icon reuses the same uniqueId-generated variable name for two distinct linearGradient defs, causing duplicate 'id' attributes in the DOM.

Draft — this is a starting point, not a finished change. The fix required judgment, so read it before trusting it.

#	Fix confidence	Finding	Location
1	🟢 95 high	Brave icon reuses the same uniqueId-generated variable name for two distinct linearGradient defs, causing duplicate 'id' attributes in the DOM	frontend/pages/SoftwarePage/components/icons/Brave.tsx:29

What changed — and what was deliberately left — is explained per finding as inline review comments on the lines each finding touched.

---

Run: https://product-hub.flamingo.so/admin/code-review
Run id: 1446a072-096e-4294-8082-c7cadffe76de

Merging this PR is recorded as acceptance of the rule that produced it;
closing it unmerged is recorded as rejection. Both feed rule health, so
closing a wrong suggestion is useful rather than merely tidy.

ClickUp task: CU-86akj32d7 FleetMDM bulk review findings sweep (12 PRs)

[flamingo]

🦩 What this fix changed, finding by finding

1 finding(s) fixed in this draft — 1 explained inline on the diff.

[flamingo]

🦩 🟠 Brave icon reuses the same uniqueId-generated variable name for two distinct linearGradient defs, causing duplicate 'id' attributes in the DOM

In frontend/pages/SoftwarePage/components/icons/Brave.tsx, added a second uniqueId("fill-path-") call assigned to a new fillPathId2 variable. The second <linearGradient> in <defs> now uses id={fillPathId2} instead of reusing fillPathId, and the third <path> element (which referenced the second gradient) now uses fill={url(#${fillPathId2})} so it correctly points at its own gradient definition, eliminating the duplicate id attribute.

🤖 Prompt for AI agents

In frontend/pages/SoftwarePage/components/icons/Brave.tsx around line 29, review and complete this code-review fix: Brave icon reuses the same uniqueId-generated variable name for two distinct linearGradient defs, causing duplicate 'id' attributes in the DOM.
What the draft fix changed: In `frontend/pages/SoftwarePage/components/icons/Brave.tsx`, added a second `uniqueId("fill-path-")` call assigned to a new `fillPathId2` variable. The second `<linearGradient>` in `<defs>` now uses `id={fillPathId2}` instead of reusing `fillPathId`, and the third `<path>` element (which referenced the second gradient) now uses `fill={`url(#${fillPathId2})`}` so it correctly points at its own gradient definition, eliminating the duplicate `id` attribute.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟢 95 high — react 👍/👎 to teach the reviewer