docs(release-audit): Telegram Android capture pass (2026-07-20) - #54
Merged
Conversation
All 13 listing variants captured on Android (incl. price_drop_ping, the Desktop gap) render content byte-faithful. One cosmetic non-blocking anomaly: the 💶 buyer-total row soft-wraps to two lines at the operator's Android font size (clean reflow, no markup break) — documented as a limitation, not BLOCKED; a one-line fix would be a post-v1.0 design ticket (FR22-locked row). §2 for the new glyphs holds by inspection (text carries the signal). Still open: §3 keyboard-lifecycle + a real in-place edit, both needing a live event. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GCRu65gaarXToBMgxJhFep
📝 WalkthroughWalkthroughThe audit summary updates the v0.4.4 capture status, documents Android results for all 13 listing variants and live Corsair evidence, records emoji wrapping and color-blind guidance, and lists remaining lifecycle and in-place edit checks. ChangesTelegram Android audit
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/release-audits/v1.0/SUMMARY.md`:
- Around line 149-150: Update the Desktop status cell for price_drop_ping to use
an allowed checklist marker: leave it blank while retaining the Android-only
explanation in the note, or define — in the documented capture-status vocabulary
before using it.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 4dfbc1fd-eb80-4820-8277-3906c6e0e945
📒 Files selected for processing (1)
docs/release-audits/v1.0/SUMMARY.md
Comment on lines
+149
to
150
| | `price_drop_ping` | — | not in this batch; captured on Android instead (below) | | ||
|
|
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Use a defined capture-status marker for the Desktop gap.
The checklist below permits only ✓, !, or blank, but this cell introduces — without defining it. Either leave the Desktop cell blank and explain the Android-only capture in the note, or add — to the documented status vocabulary.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/release-audits/v1.0/SUMMARY.md` around lines 149 - 150, Update the
Desktop status cell for price_drop_ping to use an allowed checklist marker:
leave it blank while retaining the Android-only explanation in the note, or
define — in the documented capture-status vocabulary before using it.
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
1 participant
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.
Registra el pase de captura on-device en Telegram Android contra v0.4.4 (11 capturas, las 13 variantes de listing + las reales del Corsair).
Contenido byte-fiel en todas — escapes resueltos, negrita/cursiva, emoji intactos, banners de edición encima del cuerpo, teclados correctos,
price_drop_pingen texto plano. Con esto la columna Android queda completa y se cubre la celda deprice_drop_pingque faltaba en Desktop.Anomalía
!cosmética, NO bloqueante: la fila 💶 hace soft-wrap a dos líneas en Android con el tamaño de fuente del operador (… + 4,82/Protección = 63,32 €). Es el reflow normal de líneas largas de Telegram — la cabecera de precio y la línea de Confidence también hacen wrap —, no una ruptura de markup: texto idéntico al de referencia, sin escape colado ni emoji corrupto, legible. Falla el invariante literal "💶 en una línea" pero no cumple ningún criterio de bloqueo. Veredicto: limitación documentada, no BLOCKED. Si se quisiera una línea, sería un ticket de diseño post-v1.0 (acortar la fila), no un parche de auditoría (formato FR22-locked).§2 colorblind (glifos nuevos): 💶/📉 son informativos, no portan señal; el par 🔴/🟢 son los tokens de severidad ya auditados y la palabra del banner (
RESERVADO/Disponible/Bajada) lleva el estado. Sin dependencia de color nueva → el contrato UX-DR22 se sostiene por inspección.Sigue abierto (ambos clientes ya capturados): el vistazo §3 al ciclo de teclado y una edición real in-place — los dos requieren un evento en vivo, no un emit.
Solo docs.
🤖 Generated with Claude Code
https://claude.ai/code/session_01GCRu65gaarXToBMgxJhFep
Summary by CodeRabbit