Skip to content

fix: epub kapağı doğru seçilsin + dalı main'deki ingest koduyla hizala - #23

Merged
dpentx merged 1 commit into
book/knh-15from
claude/knh-15-cover-fix
Sep 28, 2026
Merged

dpentx merged 1 commit into
book/knh-15from
claude/knh-15-cover-fix

Conversation

@dpentx

@dpentx dpentx commented Sep 28, 2026

Copy link
Copy Markdown
Owner

Ciltlenen epub'ta kapak olarak rastgele bir ek illüstrasyon (Insert6.jpg) seçiliyordu, ayrıca aynı dosya epub'a iki kez yazılıyordu (duplicate zip girdisi).

Sebep: extract_cover_image sadece ITEM_IMAGE tipine bakıyordu; gerçek kapak (Images/Cover.jpg) EPUB3'te ITEM_COVER tipiyle okunduğu için aday bile değildi, o yüzden "ilk resim"e düşülüyordu.

Fix: kapak sırayla ITEM_COVER → EPUB2 <meta name="cover"> → adında "cover" geçen resim → spine'ın ilk sayfasının referans verdiği resim ile bulunuyor; hiçbiri yoksa yanlış resim yerine kapaksız bırakılıyor. Kapak artık sadece set_cover() ile bir kez ekleniyor.

Dal hizalama: book/knh-15 PR #21/#22'den önce açıldığı için eski (epub gönderen, 413 verecek) web_ingest.py/convert.py kodunu taşıyordu. web_ingest.py, lib/cover.py, requirements.txt main ile birebir aynı içeriğe getirildi (completion PR'ında çakışma çıkmasın).

Test: gerçek knh-15, knh-14, knh-10, pg2147 orijinalleriyle ciltlenip doğru kapak, duplicate yok, kırık resim referansı yok doğrulandı.

…i; dalı main'deki txt/kapak ingest koduyla hizala
@vercel

vercel Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
eptran-trasnlate Ignored Ignored Preview Sep 28, 2026 11:07am UTC

@dpentx
dpentx merged commit d7192ae into book/knh-15 Sep 28, 2026
2 checks passed
@dpentx
dpentx deleted the claude/knh-15-cover-fix branch September 29, 2026 19:08
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