עזרה | הערות על תוסף תיקון קוראים

[מיכאלוש]

הרב @פלמנמוני היקר התוסף ממש מיוחד כמה הערות קטנות:

• דבר ראשון ברוב התיקון קוראים הטור ניקוד בימין והטור סתם בשמאל

• בגופן סתם תגדיר ברירת מחדל את קלאסי (זה מה שיש בוינדוס)
  נ.ב. עכשיו הוא לא מוצג עקב שגיאת תחביר כנראה במהלך בשינוי שם שעשיתי לו...

• הייתי מאפשר להוסיף ולהציג מגופני המערכת גם בגופן ניקוד וגם בגופן סתם

• עליות יותר מתאים ראשון שני שלישי או כהן לוי שלישי מאשר עליה א

• כדאי להוסיף אופציה להסתרת שם השם

• הקווים בין השורות הייתי שוקל להסיר או לפחות לאפשר בהגדרות להסיר

• אולי כדאי להוסיף יתרות וחסרות בצד כמו שמקובל בתיקון סופרים שת (שורה תמימה) יא (יתר אחת) חא (חסר אחת ) וכו'

• הבאג היחיד שנתקלתי זה בקריאות שלא טוען את התוכן כל השאר זה רק הצעות לשיפור שחלק גדול מהם אני אשם...

[פלמנמוני]

@מיכאלוש כתב:

דבר ראשון ברוב התיקון קוראים הטור ניקוד בימין והטור סתם בשמאל

זה תלוי ע"א או ע"ב

@מיכאלוש כתב:

בגופן סתם תגדיר ברירת מחדל את קלאסי (זה מה שיש בוינדוס)

לא נגעתי בהגדרה זו, שאתה הגדרת...

@מיכאלוש כתב:

עליות יותר מתאים ראשון שני שלישי או כהן לוי שלישי מאשר עליה א

ככה זה מגיע ממערכת אוצריא

[נתנאל_26]

דבר ראשון זה נראה מדהים למרות שאני לא מצליח לראות כלום

יש לי איזה בעיה גם בו וגם בשנים מקרא שמשום מה הטקסט לא נטען לי זה הקנסול בתיקון קוראים

app.js:122 Plugin Booting... {plugin: {…}, app: {…}, theme: {…}, permissions: Array(8)}
navigation.js:522 Failed to load book בראשית {code: 'error.internal', message: 'Exception: Book not found: בראשית'}
fetchFullBook @ navigation.js:522
await in fetchFullBook
(anonymous) @ navigation.js:543
loadAndProcessAll @ navigation.js:543
navigateToCurrentParasha @ navigation.js:808
populateParashaSelect @ navigation.js:256
populateBookSelect @ navigation.js:241
initNavigation @ navigation.js:48
(anonymous) @ app.js:133
await in (anonymous)
w @ VM376:12
_boot @ VM360:23
(anonymous) @ VM376:28
(anonymous) @ VM376:29
navigation.js:544 [tikkun] loaded bereshit: 0 chars
navigation.js:522 Failed to load book שמות {code: 'error.internal', message: 'Exception: Book not found: שמות'}
fetchFullBook @ navigation.js:522
await in fetchFullBook
(anonymous) @ navigation.js:543
loadAndProcessAll @ navigation.js:543
navigateToCurrentParasha @ navigation.js:808
populateParashaSelect @ navigation.js:256
populateBookSelect @ navigation.js:241
initNavigation @ navigation.js:48
(anonymous) @ app.js:133
await in (anonymous)
w @ VM376:12
_boot @ VM360:23
(anonymous) @ VM376:28
(anonymous) @ VM376:29
navigation.js:544 [tikkun] loaded shemot: 0 chars
navigation.js:522 Failed to load book ויקרא {code: 'error.internal', message: 'Exception: Book not found: ויקרא'}
fetchFullBook @ navigation.js:522
await in fetchFullBook
(anonymous) @ navigation.js:543
loadAndProcessAll @ navigation.js:543
navigateToCurrentParasha @ navigation.js:808
populateParashaSelect @ navigation.js:256
populateBookSelect @ navigation.js:241
initNavigation @ navigation.js:48
(anonymous) @ app.js:133
await in (anonymous)
w @ VM376:12
_boot @ VM360:23
(anonymous) @ VM376:28
(anonymous) @ VM376:29
navigation.js:544 [tikkun] loaded vayikra: 0 chars
navigation.js:522 Failed to load book במדבר {code: 'error.internal', message: 'Exception: Book not found: במדבר'}
fetchFullBook @ navigation.js:522
await in fetchFullBook
(anonymous) @ navigation.js:543
loadAndProcessAll @ navigation.js:543
navigateToCurrentParasha @ navigation.js:808
populateParashaSelect @ navigation.js:256
populateBookSelect @ navigation.js:241
initNavigation @ navigation.js:48
(anonymous) @ app.js:133
await in (anonymous)
w @ VM376:12
_boot @ VM360:23
(anonymous) @ VM376:28
(anonymous) @ VM376:29
navigation.js:544 [tikkun] loaded bamidbar: 0 chars
navigation.js:522 Failed to load book דברים {code: 'error.internal', message: 'Exception: Book not found: דברים'}
fetchFullBook @ navigation.js:522
await in fetchFullBook
(anonymous) @ navigation.js:543
loadAndProcessAll @ navigation.js:543
navigateToCurrentParasha @ navigation.js:808
populateParashaSelect @ navigation.js:256
populateBookSelect @ navigation.js:241
initNavigation @ navigation.js:48
(anonymous) @ app.js:133
await in (anonymous)
w @ VM376:12
_boot @ VM360:23
(anonymous) @ VM376:28
(anonymous) @ VM376:29
navigation.js:544 [tikkun] loaded devarim: 0 chars
navigation.js:522 Failed to load book בראשית {code: 'error.internal', message: 'Exception: Book not found: בראשית'}
fetchFullBook @ navigation.js:522
await in fetchFullBook
(anonymous) @ navigation.js:543
loadAndProcessAll @ navigation.js:543
navigateToCurrentParasha @ navigation.js:808
populateParashaSelect @ navigation.js:256
populateBookSelect @ navigation.js:241
updateUIForSection @ navigation.js:114
initNavigation @ navigation.js:50
(anonymous) @ app.js:133
await in (anonymous)
w @ VM376:12
_boot @ VM360:23
(anonymous) @ VM376:28
(anonymous) @ VM376:29
navigation.js:544 [tikkun] loaded bereshit: 0 chars
navigation.js:522 Failed to load book שמות {code: 'error.internal', message: 'Exception: Book not found: שמות'}
fetchFullBook @ navigation.js:522
await in fetchFullBook
(anonymous) @ navigation.js:543
loadAndProcessAll @ navigation.js:543
navigateToCurrentParasha @ navigation.js:808
populateParashaSelect @ navigation.js:256
populateBookSelect @ navigation.js:241
updateUIForSection @ navigation.js:114
initNavigation @ navigation.js:50
(anonymous) @ app.js:133
await in (anonymous)
w @ VM376:12
_boot @ VM360:23
(anonymous) @ VM376:28
(anonymous) @ VM376:29
navigation.js:544 [tikkun] loaded shemot: 0 chars
navigation.js:522 Failed to load book ויקרא {code: 'error.internal', message: 'Exception: Book not found: ויקרא'}
fetchFullBook @ navigation.js:522
await in fetchFullBook
(anonymous) @ navigation.js:543
loadAndProcessAll @ navigation.js:543
navigateToCurrentParasha @ navigation.js:808
populateParashaSelect @ navigation.js:256
populateBookSelect @ navigation.js:241
updateUIForSection @ navigation.js:114
initNavigation @ navigation.js:50
(anonymous) @ app.js:133
await in (anonymous)
w @ VM376:12
_boot @ VM360:23
(anonymous) @ VM376:28
(anonymous) @ VM376:29
navigation.js:544 [tikkun] loaded vayikra: 0 chars
navigation.js:522 Failed to load book במדבר {code: 'error.internal', message: 'Exception: Book not found: במדבר'}
fetchFullBook @ navigation.js:522
await in fetchFullBook
(anonymous) @ navigation.js:543
loadAndProcessAll @ navigation.js:543
navigateToCurrentParasha @ navigation.js:808
populateParashaSelect @ navigation.js:256
populateBookSelect @ navigation.js:241
updateUIForSection @ navigation.js:114
initNavigation @ navigation.js:50
(anonymous) @ app.js:133
await in (anonymous)
w @ VM376:12
_boot @ VM360:23
(anonymous) @ VM376:28
(anonymous) @ VM376:29
navigation.js:544 [tikkun] loaded bamidbar: 0 chars
navigation.js:522 Failed to load book דברים {code: 'error.internal', message: 'Exception: Book not found: דברים'}
fetchFullBook @ navigation.js:522
await in fetchFullBook
(anonymous) @ navigation.js:543
loadAndProcessAll @ navigation.js:543
navigateToCurrentParasha @ navigation.js:808
populateParashaSelect @ navigation.js:256
populateBookSelect @ navigation.js:241
updateUIForSection @ navigation.js:114
initNavigation @ navigation.js:50
(anonymous) @ app.js:133
await in (anonymous)
w @ VM376:12
_boot @ VM360:23
(anonymous) @ VM376:28
(anonymous) @ VM376:29
navigation.js:544 [tikkun] loaded devarim: 0 chars
navigation.js:558 [tikkun] book starts: {bereshit: 0, shemot: 1, vayikra: 2, bamidbar: 3, devarim: 4}
navigation.js:565 [tikkun] lines: 16, pages: ~1
navigation.js:590 [tikkun] done in 0s
navigation.js:558 [tikkun] book starts: {bereshit: 0, shemot: 1, vayikra: 2, bamidbar: 3, devarim: 4}
navigation.js:565 [tikkun] lines: 16, pages: ~1
navigation.js:590 [tikkun] done in 0s
navigation.js:771 Parasha בראשית not found in tokens
findPageForParasha @ navigation.js:771
navigateToCurrentParasha @ navigation.js:812
await in navigateToCurrentParasha
populateParashaSelect @ navigation.js:256
populateBookSelect @ navigation.js:241
updateUIForSection @ navigation.js:114
initNavigation @ navigation.js:50
(anonymous) @ app.js:133
await in (anonymous)
w @ VM376:12
_boot @ VM360:23
(anonymous) @ VM376:28
(anonymous) @ VM376:29

ועוד משהו קטן הדך נטען לפני העיצוב מה שיוצר קפיצה קטנה בעיצוב

[בן יוסף]

@מיכאלוש כתב:

הבאג היחיד שנתקלתי זה בקריאות שלא טוען את התוכן כל השאר זה רק הצעות לשיפור שחלק גדול מהם אני אשם...

@פלמנמוני
גא"מ לבאג זה - אבל את הקריאה הראשונה של שבת ראש חודש חנוכה והקריאה האחרונה תשעה באב - שחרית הוא כן טוען ומביא.

עריכה, עכשיו רק תשעה באב נטען ולא שבת חנוכה

[מיכאלוש]

@פלמנמוני כתב:

לא נגעתי בהגדרה זו, שאתה הגדרת...

אני יודע אבל הקוד כבר אצלך...
לכן גם לקחתי אחריות והסברתי שיש באג בהצגה של קלאסי משינוי שם שעשיתי לו בהתחלה הוא נקרא חסידי ולכן הוא היה בסוף אבל לא השגתי גופן חסידי לשימוש מסחרי אז החלפתי לקלאסי מה שיש בווינדוס וכנראה במהלך השינוי שם לא שינתי בכל המקומות

@פלמנמוני כתב:

זה תלוי ע"א או ע"ב

אני רגיל למהדורת בלכתך בדרך שם זה תמיד כמו שכתבתי

אגב יש אצלי באג בטעינת חלק מהפרשות נגיד פרשת בהעלותך לא טוען אצלי

בהזדמנות כדאי לסדר שבהדפסה זה יצא יפה כמו שזה מוצג עכשיו ולא יתחרבש קצת

@פלמנמוני כתב:

ככה זה מגיע ממערכת אוצריא

מפטיר אין באוצריא ?

[אוצר]

גם אצלי לא עולה התוכן ואני לא יודע מה הבעיה.

[פלמנמוני]

@מיכאלוש כתב:

מפטיר אין באוצריא ?

לא

[נתנאל_26]

נתנאל_26 כתב:

יש לי איזה בעיה גם בו וגם בשנים מקרא שמשום מה הטקסט לא נטען לי זה הקנסול בתיקון קוראים

בקשתי מאפוס 4.7 שיבדוק לי למה זה קורה אצלי והוא תיקן לי קבצים בקוד של אוצריא וזה הסתדר אבל אני לא הבנתי למה זה רק אצל חלק מהמשתמשים
בכל אופן הבנתי קטנה והטענה שלו בספוילר

Spoiler

תיאור הבעיה
הקריאה library.getBookContent({ bookId: 'X' }) נכשלת עם השגיאה:


Exception: Book not found: X
גם כאשר הספר X קיים בקטלוג — library.findBooks({ query: 'X' }) מחזיר אותו עם נתיב הקטגוריה הנכון. לדוגמה אצלי קרה ב"במדבר" וב"בראשית" — findBooks הצליח, getBookContent נכשל.

למי זה קורה
לא לכולם — וזו הסיבה שהבאג נשאר תחת הרדאר. המשתמשים המושפעים:

מי שיש לו רק seforim.db ללא קבצי טקסט נפרדים בדיסק (זה אני וכמה משתמשים נוספים)
ולא פתחו את הספר ידנית בקורא של אוצריא במהלך הסשן (אחרת ה-cache _bookToProvider היה מיושב מקודם)
מי שעדיין משתמש בגרסה ישנה של אוצריא עם קבצי txt — לא מבחין בבעיה.

שורש הבעיה (טכני)
ב-lib/plugins/bridge/plugin_bridge_adapter.dart, ב-case 'getBookContent':


final rawText = await DataRepository.instance.getBookText(bookId);
הקריאה מעבירה רק את שם הספר — לא categoryId ולא fileType. ב-file_system_data_provider.dart:343-348, זה מתרגם ל:


final text = await _providerManager.getBookText(
  title,
  categoryId: categoryId,        // null
  fileType: fileType ?? 'txt',   // ← נקבע ל-'txt' תמיד!
);
וב-book_composite_key.dart:79:


return fileType == normalizeFileType(otherFileType);  // דרישת התאמה מדויקת
המשמעות: ב-seforim.db הספרים מאוחסנים עם metadata של fileType (כפי שמוגדר ב-catalog). אבל ה-Plugin API מתעלם מ-metadata הזה ומקבע fileType='txt'. בשבילי, כאשר הפורמט בקטלוג אינו txt, ההתאמה נכשלת → "Book not found" — אפילו שהספר 100% קיים.

לעומת זאת, library.getBookToc עובד מצוין כי הוא מאתר קודם את ה-TextBook מהקטלוג ומשתמש ב-book.tableOfContents שמעביר את book.fileType הנכון מתוך ה-metadata.

הצעת תיקון
library.getBookContent צריך לעשות אותו דבר כמו library.getBookToc — לאתר את ה-TextBook מהקטלוג קודם, ואז להשתמש ב-TextBookRepository.getBookContent(book) שמכיר את הפורמט מה-metadata:


case 'getBookContent':
  final bookId = (args['bookId'] ?? args['title']) as String?;
  if (bookId == null) throw Exception('bookId required');
  // אתר את ה-TextBook מהקטלוג כדי לקבל categoryId/fileType נכונים
  final cataloged = library.getAllBooks()
      .cast<dynamic>()
      .firstWhere((b) => b?.title == bookId, orElse: () => null);
  final String rawText;
  if (cataloged is TextBook) {
    rawText = await TextBookRepository(
      fileSystem: FileSystemData.instance,
    ).getBookContent(cataloged);
  } else {
    rawText = await DataRepository.instance.getBookText(bookId);
  }
  // המשך הקוד המקורי לחיתוך לפי limit/offset/section...
תיקון מינימלי, לא שובר תאימות, עקבי עם getBookToc. תיקנתי אצלי מקומית — הכל עובד מצוין, כולל כל הספרים שעד עכשיו לא נטענו דרך תוספים.

[אוצר]

אתה יכול להביא גם לנו את המתוקן

[נתנאל_26]

@אוצר כתב:

אתה יכול להביא גם לנו את המתוקן

זה קבצי קוד של אוצריא לא של התוסף אז רק אם אתה מריץ את אוצריא במצב דיבוג זה יעזור לך

[יום חדש מתחיל]

@נתנאל_26 כתב:

@אוצר כתב:

אתה יכול להביא גם לנו את המתוקן

זה קבצי קוד של אוצריא לא של התוסף אז רק אם אתה מריץ את אוצריא במצב דיבוג זה יעזור לך

למה שלא תשלח PR?

[נתנאל_26]

@יום-חדש-מתחיל
אני לא מספיק סומך על AI שאני יעבור שכל מה שהוא עושה זה מאה אחוז
בכל אופן פלמנמוני תיקן את ה אצלו ב"ה הוא יעלה את זה בקרוב

[איש בנימין]

הגלילה לא עובדת לי.
וכן א"א לבחור פרשה/חומש/עליה בחלונית המיועדת לכך.

[דנדי]

@פלמנמוני מהמם!!!
רמה גבוהה!

נראה לי שבמסגרת הבדיקות שאתה עושה לתוסף שעולה לחנות כלומר הבדיקה אוטומטית צריך להגביל את התוספים שאינם נראים בסגנון של אוצריא

איך עשית את זה? אולי תתן את הניסוח המדוייק לתת לAI לעשות את זה בדיוק בסגנון של אוצריא

[פלמנמוני]

@דנדי כתב:

נראה לי שבמסגרת הבדיקות שאתה עושה לתוסף שעולה לחנות כלומר הבדיקה אוטומטית צריך להגביל את התוספים שאינם נראים בסגנון של אוצריא

אגב, יש בדיקות כאלו, אבל זה לא חוסם העלאה, אלא רק א"א לשים את התגית של מראה תואם לאוצריא, ולהפך - מה שתואם מקבל את זה אוטומטית, ודוקא התוסף הזה לא קיבל את זה, בשל צורה מסויימת שהייתי צריך להשתמש בגופן...

[דנדי]

@פלמנמוני זה תואם לאוצריא? לא נראה לי...

[פלמנמוני]

@דנדי זה היה לפני הבדיקות האוטומטיות

[דנדי]

@פלמנמוני אז נראה לי שבעינן תקוני...
זה פשוט מפריע לי בעין מאוד... זה נראה לא מקצועי
אגב יש הרבה כאלה כמו שנים מקרא ואפי' וורד לאוצריא, למה הוא בוחר אוטומטית את הפונט דיויד?

[פלמנמוני]

עלה כעת גרסה 1, עם שיפורים רבים, לע"ע אני מפסיק לפתח את זה, מי שרוצה מוזמן לפתוח PR במאגר הגיטהאב

[צבי דורש ציון]

@פלמנמוני היקר, ממש יפה!
אף על פי שאתה לא מוסיף כרגע, אני מבקש אולי יהיה מישהו שירצה להוסיף
כדאי שייפתח ברירת מחדל אוטומטי על פרשת השבוע או הקריאה הקרואבה,
אלא אם כן מגדירים אחרת (למקרה שמישהו רוצה להכין קריאה רחוקה יותר ולא רוצה שכל פעם יעלם המיקום שאחז בו, מצוי מאד במועדים וכו')
דבר נוסף, נראה לי שחובה קיצור מקלדת למעבר לעמוד הבא