Skip to content

delta: a terjedelem-őr a lemezen lévő szöveghez mér, nem a legutolsó állapothoz #47

Open

Description

@godavid

Forrás: a #46 Claude-review-ja, Pre-existing (blame de8f4a63).

packages/pipeline/src/delta.ts (~244, const regi = await readFile(szovegUt…)): a terjedelem-őr a lemezen lévő szoveg.md-hez hasonlítja az új állapotot. Ha ugyanazon a napon előbb egy késve bekerülő, régebbi állapot (d1 < legutóbbi ismert) íródik ki, a következő mai állapotot (d2) a d1 régi szövegéhez méri, nem a legutóbbi ismert állapothoz. Ez hamis anomáliát okozhat, vagy elrejthet egy valódit.

Javaslat: a viszonyítási alap a dátum szerint közvetlenül megelőző ismert állapot szövege legyen (a commitjából), ne a lemez aktuális tartalma.

Activity

  1. godavid commented on Nov 13, 2026

    @godavid
    OwnerAuthor

    Goal
    A terjedelem-őr minden új időállapotot a dátum szerinti helyes előzményhez hasonlítson.

    Evidence
    packages/pipeline/src/delta.ts:212 dátum szerint rendezi az eseményeket; :253 a lemezen lévő szoveg.md-t olvassa; :264 annak hosszát ellenőrzi; :284 az új szöveget kiírja. A legutóbbi ismert állapot HEAD-helyreállítása csak :301 után történik.

    Root cause
    A lemezen lévő munkapéldány egy korábban feldolgozott, késve bekerülő régi állapot is lehet. Ez nem azonos az új állapot dátum szerinti közvetlen előzményével.

    Files
    packages/pipeline/src/delta.ts
    packages/pipeline/src/health.ts
    packages/pipeline/test/health.test.ts
    Szükség esetén egy fókuszált delta-teszt a packages/pipeline/test könyvtárban.

    Non-goals
    Nincs history-átírás, élő adatjavítás, workflow-változás vagy terjedelemküszöb-módosítás. A HEAD-helyreállítás és az anomáliás állapot kihagyása marad.

    Acceptance check
    Kényszerített eseménysor: korábban ismert későbbi állapot mellett előbb késve bekerülő régi állapot, majd mai állapot. A teszt bizonyítsa, hogy az őr a dátum szerinti közvetlen előzmény szövegét használja, és a korábbi lemezállapot sem hamis riasztást, sem valódi anomália elfedését nem okozza. Fedje le a már ismert SHA-ból olvasott és az ugyanabban a futásban bekerült előzményt, valamint az előzmény nélküli esetet. A regresszióteszt előtte bukjon, utána menjen át. Élő adat-repót ne módosítson.

    Risk tier
    Várható risk:medium; a pipeline adatminőségi őrt érint.

    Siblings
    magyar-jog#9 HEAD- és indexintegritási tételei kapcsolódnak, de ezt a hibát nem oldják meg.

  2. added
    brief-ready[status] A scoped fix brief is in the issue; start with: fix <n>
    bug[type] Something does not work as intended
    manual[source] Opened by a person or an owner-lane session
    on Nov 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    brief-ready[status] A scoped fix brief is in the issue; start with: fix <n>bug[type] Something does not work as intendedmanual[source] Opened by a person or an owner-lane session

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions