fix(i18n): corrige mojibake de pontuação Windows-1252 remanescente (— • – ∞)#400
Merged
Conversation
… • – ∞) A validação exaustiva do repositório revelou que a primeira correção de mojibake (reversão Latin-1) deixou passar a família de símbolos cp1252 (em-dash —, bullet •, en-dash –, infinito ∞), pois esses envolvem o byte € (U+20AC), não codificável em Latin-1. Aplicada reversão cp1252 seletiva: 19 arquivos, 58 runs. Atinge telas reais (resumo de faixa de preço em filtros, validação de volume, tabela de fornecedores, simulador, login). 0 mojibake remanescente. Sem caractere de substituição introduzido. https://claude.ai/code/session_01HjiGVkF3Df9GiFjDbfxDYn
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (19)
WalkthroughPR corrige encoding de caracteres Unicode corrompidos em toda a base. Substitui em dashes (—), bullets (•), multiplicadores (×), setas (→, ↳), acentos e emojis por equivalentes Unicode corretos em UI de componentes, hooks de estado, comentários e normalização de parsing de dimensões. ChangesCorreção de Caracteres Unicode
Sequence Diagram(s)Não aplicável: mudanças são correções de encoding em strings estáticas e não introduzem novos fluxos ou interações entre componentes. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~4 minutes ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR performs a follow-up i18n cleanup to fix remaining Windows-1252/UTF-8 mojibake artifacts in user-facing UI strings and developer comments, restoring intended punctuation/symbols (e.g., — • – ∞ × ≥ → ↳) across the app.
Changes:
- Replaced mojibake sequences in UI text (filters, pricing simulator, kit volume validation, supplier/admin screens) with correct Unicode characters.
- Cleaned up assorted comments/labels affected by mojibake in auth, voice hooks, and pricing components.
- Adjusted dimension-string normalization to use the correct multiplication sign character.
Reviewed changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/pages/auth/Auth.tsx | Fixes mojibake in auth UI copy, placeholder bullets, and developer-facing comments. |
| src/lib/personalization/transformers.ts | Fixes mojibake in section header comment. |
| src/lib/kit-builder/volume-calculator.ts | Fixes mojibake in headers/docs and updates dimension separator normalization. |
| src/lib/kit-builder/price-calculator.ts | Fixes mojibake in headers and breakdown label arrow glyph. |
| src/hooks/voice/processTranscript.ts | Fixes mojibake in doc comment punctuation. |
| src/hooks/voice/logVoiceCommand.ts | Fixes mojibake in doc/comments punctuation. |
| src/components/pricing/simulator/TechniqueSelector.tsx | Fixes mojibake bullet separators in technique/location metadata. |
| src/components/pricing/simulator/QuantityAndResult.tsx | Fixes mojibake multiplication sign and emoji/title rendering. |
| src/components/pricing/simulator/ProductSearch.tsx | Fixes mojibake bullets in product metadata display. |
| src/components/pricing/simulator/MultiEngravingResult.tsx | Fixes mojibake multiplication sign in totals breakdown. |
| src/components/pricing/simulator/EngravingList.tsx | Fixes mojibake multiplication sign in size badge formatting. |
| src/components/pricing/QuantityPriceCalculator.tsx | Fixes mojibake in header comment punctuation. |
| src/components/pricing/calculator/TechniqueMultiSelector.tsx | Fixes mojibake in header comment punctuation. |
| src/components/filters/filter-panel/useFilterPanelState.ts | Fixes mojibake in price/stock summary formatting (–, ∞, ≥). |
| src/components/admin/suppliers-manager/SupplierTable.tsx | Fixes mojibake em-dash placeholders. |
| src/components/admin/products/kit-components/VolumeValidation.tsx | Fixes mojibake in status badge, dimensions display (×), bullets, and warning text. |
| src/components/admin/products/kit-components/PrintAreaForm.tsx | Fixes mojibake em-dash joiner in preview string. |
| src/components/admin/products/kit-components/api.ts | Fixes mojibake in section divider comment glyphs. |
| src/components/admin/products/bulk-import/StepPreview.tsx | Fixes mojibake em-dashes and placeholder values in bulk import preview. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| .replace(/\s+/g, '') | ||
| .replace(/\u00d7/g, 'x') | ||
| .replace(/×/g, 'x') | ||
| .replace(/×/g, 'x') |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Correção complementar de mojibake (família de pontuação Windows-1252)
A validação exaustiva do repositório (a pedido) revelou que a primeira correção de mojibake (#294, reversão Latin-1) deixou passar a família de símbolos cp1252 — porque eles envolvem o byte
€(U+20AC), que não é codificável em Latin-1, então o reversor seletivo os pulava silenciosamente.Símbolos corrigidos
â€"→—(em-dash)•→•(bullet)â€"→–(en-dash)∞→∞(infinito)Reversão cp1252 seletiva
0mojibake remanescente (verificado comgit grep -P "Ã[\x80-\xBF]|â€|âˆ").�) introduzido. Diff balanceado (53/53), só conteúdo de strings/comentários.Telas reais afetadas (agora corretas)
R$X–Y/∞—•••••••••e comentáriosTest plan
— • – ∞corretamentehttps://claude.ai/code/session_01HjiGVkF3Df9GiFjDbfxDYn
Generated by Claude Code
Summary by cubic
Fixes remaining Windows-1252 mojibake by replacing broken punctuation with correct Unicode across the app. Cleans up visible UI text without changing behavior.
Written for commit a7d7807. Summary will update on new commits. Review in cubic
Summary by CodeRabbit
Bug Fixes