Skip to content

Harden debug localStorage parsing - #21

Open
albertoblancas123 wants to merge 1 commit into
Liberdus:mainfrom
albertoblancas123:codex/harden-debug-storage-20260709
Open

Harden debug localStorage parsing#21
albertoblancas123 wants to merge 1 commit into
Liberdus:mainfrom
albertoblancas123:codex/harden-debug-storage-20260709

Conversation

@albertoblancas123

Copy link
Copy Markdown

Refs #3

This is a small code-improvement bounty submission focused on app robustness.

What changed:

  • Safely handle missing, inaccessible, or malformed localStorage.debug values in isDebugEnabled.
  • Clear malformed debug settings so a stale bad value does not keep breaking future page loads.
  • Add Playwright coverage that starts the app with invalid debug settings and verifies the wallet UI still initializes.

Validation:

  • node --check js/config.js
  • node --check tests/e2e/wallet.spec.js
  • Attempted pnpm exec playwright test tests/e2e/wallet.spec.js --project=chromium; local environment blocked Playwright worker spawn with spawn EPERM before browser startup.

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