Skip to content

fix(discordsh): deduplicate DroidProvider/OverlayShell + audit fixes#7903

Merged
h0lybyte merged 2 commits intodevfrom
trunk/discordsh-1773262848
Mar 11, 2026
Merged

fix(discordsh): deduplicate DroidProvider/OverlayShell + audit fixes#7903
h0lybyte merged 2 commits intodevfrom
trunk/discordsh-1773262848

Conversation

@h0lybyte
Copy link
Member

Summary

Related

Closes #7712

Test plan

  • Verify DroidProvider initializes once (check React DevTools for single provider tree)
  • Confirm overlay toasts, modals, and tooltips still render on all pages
  • Auth flow (login/logout) still works end-to-end
  • Run e2e tests before merge

Closes #7712 item 1 — DroidProvider and OverlayShell were rendered in
both Footer.astro and the unused Layout.astro. Consolidate into
NavBar.astro (present on every page via Starlight Header override)
so React initializes these providers exactly once. Also fixes the
Footer copy missing i18nPath and dataPath config.
@github-actions
Copy link
Contributor

github-actions bot commented Mar 11, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

…nnect, Redirect component

- Add loading="lazy" and decoding="async" to server card images
- Replace onMouseEnter/onMouseLeave JS hover with CSS-only .server-card:hover
- Add hCaptcha preconnect links to Starlight head config
- Fix nanostore subscription leak in NavBar (unsub on astro:before-swap)
- Create reusable Redirect.astro in @kbve/astro with meta refresh + JS fallback
- Use Redirect component for auth/index page
- Add @kbve/astro/components/* tsconfig path for .astro subpath imports

Ref #7712
@h0lybyte h0lybyte merged commit 6646871 into dev Mar 11, 2026
5 checks passed
@h0lybyte h0lybyte deleted the trunk/discordsh-1773262848 branch March 11, 2026 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant