✅ Snyk checks have passed. No issues have been found so far.

Status	Scan Engine	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Greptile Summary

This PR wraps Popover.Content inside a Popover.Portal in SecretPathInput to fix a Safari-specific bug where the popover was incorrectly positioned due to CSS stacking/overflow constraints on ancestor elements. Rendering into a portal appends the popover to document.body, bypassing those constraints. A secondary improvement changes overflow-y-scroll to overflow-y-auto so the scrollbar only appears when content actually overflows.

• The portal wrapping is the correct Radix UI pattern for fixing browser-specific positioning issues.
• Changing overflow-y-scroll → overflow-y-auto avoids an always-visible scrollbar track when there are few suggestions.
• The pre-existing relative top-2 class on Popover.Content is effectively a no-op since Radix UI overrides positioning via inline styles, but this is unchanged from before and has no functional impact.
• No security, logic, or API-breaking issues were identified.

Confidence Score: 5/5

• This PR is safe to merge — it is a minimal, well-scoped frontend-only fix with no security, logic, or breaking change concerns.
• The change is confined to a single UI component, uses a well-established Radix UI pattern (portal rendering) to address a known cross-browser positioning issue, and introduces no new logic or risk surface.
• No files require special attention.

Important Files Changed

_{Last reviewed commit: "fix: use portal in s..."}