Skip to content

chore: sync dev with main#1969

Merged
8 commits merged into
devfrom
main
May 12, 2026
Merged

chore: sync dev with main#1969
8 commits merged into
devfrom
main

Conversation

@chip-peanut-bot

Copy link
Copy Markdown
Contributor

Branch Sync

main is 3 commits ahead of dev.

Commits to sync:

• fix: prevent translation extension crashes across all browsers
• fix: address review — window global for HMR, dev-mode logging, no-cleanup comment
• Merge pull request #1964 from peanutprotocol/fix/translation-mutation-handler


Auto-generated by Chip 🐿️

kushagrasarathe and others added 3 commits May 11, 2026 14:46
the old mutation observer only handled google translate's data-translated
attribute. brave translate, safari, and others use different DOM
manipulation that was never caught, causing removeChild crashes during
react reconciliation.

patches Node.prototype.removeChild and insertBefore to gracefully handle
nodes moved by any translation extension — the standard react ecosystem
fix (react/react#11538).
…-handler

fix: prevent translation extension crashes across all browsers
@vercel

vercel Bot commented May 12, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
peanut-wallet Ready Ready Preview, Comment May 12, 2026 3:38pm

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

Code-analysis diff

Painscore total: 5739.63 → 5737.93 (-1.7)
Findings: -1 net (+4 new, -5 resolved)

🆕 New findings (4)

  • low any-usage — src/hooks/useTranslationMutationHandler.ts — 2 any annotation(s)
  • low missing-return-type — src/components/Global/TranslationSafeWrapper.tsx:5 — TranslationSafeWrapper: exported fn missing return type annotation
  • low nextjs-use-client-overuse — src/components/Global/TranslationSafeWrapper.tsx:1 — 'use client' but no hooks/handlers/browser API
  • low missing-return-type — src/hooks/useTranslationMutationHandler.ts:12 — useTranslationMutationHandler: exported fn missing return type annotation

✅ Resolved (5)

  • src/hooks/useTranslationMutationHandler.ts — CC 19, MI 69.32, SLOC 50
  • src/hooks/useTranslationMutationHandler.ts:4 — useTranslationMutationHandler: MDD 17.7 (uses across many lines from declarations)
  • src/hooks/useTranslationMutationHandler.ts:8 — : MDD 10.2 (uses across many lines from declarations)
  • src/components/Global/TranslationSafeWrapper.tsx:8 — TranslationSafeWrapper: exported fn missing return type annotation
  • src/hooks/useTranslationMutationHandler.ts:4 — useTranslationMutationHandler: exported fn missing return type annotation

📈 Painscore deltas (top movers)

File Before After Δ
src/hooks/useTranslationMutationHandler.ts 7.6 6.3 -1.3

@github-actions

Copy link
Copy Markdown
Contributor

🧪 UI test report — ✅ all green

Suites

  • unit: 951 ran, 0 failed, 0 skipped, 11.9s

📊 Coverage (unit)

metric %
statements 45.0%
branches 24.1%
functions 24.2%
lines 44.7%
⏱ 10 slowest test cases
time test
0.2s src/app/actions/__tests__/api-headers.test.ts › should include Content-Type in updateUserById
0.2s src/app/actions/__tests__/api-headers-extended.test.ts › should not include apiKey in updateUserById body
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid 9-digit US account
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle invalid ETH address (missing 0x prefix)
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle too long for US account
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid ETH address in lowercase
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle maximum length (17 digits) US account
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid ETH address with surrounding spaces
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid German IBAN
0.1s src/app/(mobile-ui)/qr-pay/__tests__/qr-pay-states.test.tsx › Perk claimed shows shake class + go home button
📍 Inline annotations are in the **Unit test report** check above. Coverage artifact: `coverage-unit`. Generated by `.github/workflows/tests.yml`.

@kushagrasarathe kushagrasarathe closed this pull request by merging all changes into dev in ae19f30 May 12, 2026
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.

2 participants