Skip to content

Harden browser response boundaries - #467

Open
parsakhaz wants to merge 1 commit into
mainfrom
local-403-react-doctor-4i
Open

Harden browser response boundaries#467
parsakhaz wants to merge 1 commit into
mainfrom
local-403-react-doctor-4i

Conversation

@parsakhaz

Copy link
Copy Markdown
Member

Summary

  • isolate the web fallback Discord window from its opener
  • branch on remote-daemon HTTP status before decoding success or error payloads
  • separate notebook JSON text formatting from the component's sanitized HTML render paths

Part of #403.

React Doctor

  • current-main baseline: 0 errors, 317 warnings, 317 total
  • after: 0 errors, 314 warnings, 314 total
  • removes window-open-without-noopener, no-fetch-response-used-without-status-check, and the false-positive unsafe-json-in-html finding without replacements

Validation

  • pnpm lint
  • pnpm typecheck
  • pnpm --filter frontend test (245 tests)
  • pnpm build (including signed universal DMG/ZIP packaging)
  • pnpm dlx [email protected] frontend --no-score --no-supply-chain --no-dead-code --json

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@github-actions

Copy link
Copy Markdown

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit de82aac.

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