שיוניים נוספים במפרשים, והוספת תמיכה במפרש ריק#10
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces support for non-contiguous positions (intentional gaps) in default commentators by utilizing a "-" sentinel in default_commentators.json. The database queries and repository methods have been updated to handle commentators as (commentatorBookId, position) pairs instead of simple ID lists, and unit tests have been added to verify this position resolution logic. The feedback suggests wrapping the delete and insert operations in setDefaultCommentatorsForBook within a database transaction to ensure atomicity and improve performance.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
תסביר יותר, בבקשה. |
|
@palmoni5 |
|
@Y-PLONI |
|
הסתדת? אם זה חשוב לך אוריד שוב את המאגר למחשב המקומי ואבצע ריבייס |
|
אשמח.
בתאריך יום ד׳, 17 ביוני 2026 ב-14:23 מאת palmoni5 <
***@***.***>:
… *palmoni5* left a comment (Otzaria/SeforimLibrary#10)
<#10 (comment)>
הסתדת? אם זה חשוב לך אוריד שוב את המאגר למחשב המקומי ואבצע ריבייס
—
Reply to this email directly, view it on GitHub
<#10?email_source=notifications&email_token=A3Z3BBGKDCJWYPTCCFJBUIT5AJ5U5A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINZSHEZTONJQHA32M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-4729375087>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A3Z3BBEJRTQMERK7J57TZ3L5AJ5U5AVCNFSNUABGKJSXA33TNF2G64TZHMYTCMZVGE2TIMBRGI5US43TOVSTWNBVHA3DCNRUGA3TBILWAI>
.
Triage notifications, keep track of coding agent tasks and review pull
requests on the go with GitHub Mobile for iOS
<https://github.com/notifications/mobile/ios/A3Z3BBANZMHNVDHLJUHDPJD5AJ5U5A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINZSHEZTONJQHA32M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJKTGN5XXIZLSL5UW64Y>
and Android
<https://github.com/notifications/mobile/android/A3Z3BBG2R44LIKBA2GOPIO35AJ5U5A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINZSHEZTONJQHA32M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLTGN5XXIZLSL5QW4ZDSN5UWI>.
Download it today!
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
@Y-PLONI |
|
@Y-PLONI |
Y-PLONI
force-pushed
the
master
branch
2 times, most recently
from
b6c6aed to
9b576c5
Compare
|
@Y-PLONI |
|
@Y-PLONI |
…cha + Shaar HaTziyun - Remove Mishnah Berurah and Shaar HaTziyun from Shulchan Aruch (OC) default commentators. - Add 'משנה ברורה' as a base book whose default commentators are Biur Halacha and Shaar HaTziyun.
…sitions A default-commentator list may leave a gap with the "-" sentinel, e.g. ["א", "-", "ב"] keeps "ב" at position 2 (consumers decide how to interpret the gap). - position now reflects the slot index (non-contiguous when a slot is skipped) instead of a running counter. Missing/duplicate commentators are still packed (back-compat). - setter takes explicit (id, position); getter returns positions too (symmetric API). - Extracted resolvePositionedCommentators with unit tests (gap/missing/duplicate/self).
Rambam: halachot without Magid Mishneh use "-" between Kesef Mishneh and Hasagot HaRaavad, so Hasagot keeps position 2 and Magid's slot (1) stays empty — consistent placement across all halachot. Yerushalmi: tractates without Korban HaEdah / Shayarei Korban use "-" so Mareh HaPanim keeps position 2. Pure data change relying on the existing "-" sentinel; trailing "-" trimmed.
ערכי alt-struct של Sefaria משתמשים לעיתים באיות שם שאינו הראשי (למשל
"Messilat Yesharim" מול הראשי "Mesillat Yesharim", מופיע ב-titleVariants).
שם ספר לא מזוהה נכשל בהתאמה הקנונית ונופל ל-tail fallback ("13"), שמתנגש
עם segment באותו מספר בהקדמה - וכך כל פרק במסילת ישרים נותב להקדמה.
הפתרון: זיהוי prefix של וריאנט שם מוכר (מ-titleAliasKeys) ושכתובו לשם
הראשי לפני הפתרון, כך שההתאמה הקנונית פוגעת בפרק הנכון.
נוסף טסט titleVariantWholeRefShouldResolveToChapterNotIntroduction.
…llision Selecting two columns (commentatorBookId, position) made SQLDelight generate a result class named SelectByBookId, colliding with the one from TocQueries.sq. The collision pushed both classes into per-query subpackages (db.tocQueries / db.defaultCommentatorQueries), so the flat io.github.kdroidfilter.seforimlibrary.db.SelectByBookId referenced by ModelExtensions.toModel() no longer resolved, breaking :dao:compileKotlinJvm with overload-resolution-ambiguity errors across every toModel() call. Rename the query to selectByBookIdWithPosition so its result class is SelectByBookIdWithPosition, freeing the flat SelectByBookId name for toc. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The new Mesillat Yesharim alt-toc test built a BookPayload without the required heShortDesc parameter (added upstream), breaking :sefariasqlite:compileTestKotlinJvm. Pass heShortDesc = null to match the other BookPayload constructions in this file. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
ממצאים
מה ה־PR כן פתר
|
@palmoni5 |
The default-commentator API exposed (commentatorBookId, position) as a generic Pair in its public surface. Introduce a named DefaultCommentatorPosition model (mirroring LineTocMapping) and thread it through getDefaultCommentatorsForBook / setDefaultCommentatorsForBook, resolvePositionedCommentators, and callers/tests. Documents the gap contract on the type: non-contiguous positions keep a stable column in fixed-slot page layouts; honoring position is opt-in - a consumer that ignores it renders commentators consecutively with no gap and is unaffected.
|
@Y-PLONI |
2 participants
@Y-PLONI
שים לב! יש פה שינויי לוגיקה לתמיכה במפרש ריק. אני לא בטוח שכדאי למזג לתוך הPR הפתוח ללא מתייאש.
זה לא השינוי עליו דיברנו בצ'אט, הוא עוד מחכה בצד
ההגיון מאחורי השינוי: אם יש מפרש מסויים שלא קיים על כל הספר, למשל מגיד משנה שלא קיים על כל הרמב"ם, אז המפרש הבא יופיע במקומו הראוי ולא במקום הספר החסר, לשמירה על תפיסת עין המשתמש.
(לבקשת @איש-שלום)