Skip to content

fix(emitter): split at CHROME_BODY_SLOT before filling marks (finding #6, 49.8→97.8) - #136

Merged
ABB65 merged 1 commit into
mainfrom
fix/emitter-fill-order
Aug 25, 2026
Merged

fix(emitter): split at CHROME_BODY_SLOT before filling marks (finding #6, 49.8→97.8)#136
ABB65 merged 1 commit into
mainfrom
fix/emitter-fill-order

Conversation

@ABB65

@ABB65 ABB65 commented Aug 24, 2026

Copy link
Copy Markdown
Member

Ne

Bulgu #6 (dört satırlık bağımsız yeniden üretimle kanıtlanmış, ölçülen bedel 49.8 → 97.8): üretilen layout, markları doldurduktan SONRA işaretçide ayırıyordu — fillMarks'ın @@…@@ deseni <!--@@body@@--> içindeki @@body@@'yi yiyor, geriye <!----> kalıyor, split(BODY_SLOT) hiçbir şey bulamıyor ve gövde sessizce düşüyordu.

Düzeltme (öneri 1): üretilen fill.ts'e composeBody eklendi — önce işaretçide ayır, sonra her yarıyı doldur:

chromeBody.split(BODY_SLOT).map((part) => fillMarks(part, marks)).join(content)

Layout artık composeBody(chrome.body, marks, content) çağırıyor. Sözleşme sabiti (CHROME_BODY_SLOT) değişmedi — bu bir uygulama-sırası hatasıydı.

Test

  • Yeni regresyon testi hatayı davranışsal olarak belgeler: doldur-sonra-ayır sırasının gövdeyi düşürdüğünü (<!----> kalıntısıyla) ve ayır-sonra-doldur sırasının koruduğunu kanıtlar
  • Üretilen kod doğrulamaları: layout composeBody kullanır, fillMarks(chrome.body içermez; fill.ts'te split filling'den önce gelir
  • lint 0/0 · tsc · 8/8 paket build+test. Changeset: patch.

🤖 Generated with Claude Code

https://claude.ai/code/session_01NjgQYCFqwZjJt97yioyjnS

)

fillMarks @@…@@ deseni CHROME_BODY_SLOT içindeki @@Body@@yi yiyordu; <!---->
kalıyor, split boş dönüyor, gövde sessizce düşüyordu (ölçülen: 49.8→97.8).
Sözleşme sabiti değişmedi; uygulama-sırası hatası. Regresyon testi davranışsal.
@netlify

netlify Bot commented Aug 24, 2026

Copy link
Copy Markdown

Deploy Preview for contentrain-ai ready!

Name Link
🔨 Latest commit 8d5b154
🔍 Latest deploy log https://app.netlify.com/projects/contentrain-ai/deploys/6a8cd848153dc500086ff00e
😎 Deploy Preview https://deploy-preview-136--contentrain-ai.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ABB65
ABB65 merged commit 15db4a8 into main Aug 25, 2026
6 checks passed
@ABB65
ABB65 deleted the fix/emitter-fill-order branch August 25, 2026 08:19
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant