Skip to content

fix(pipeline): anomáliánál a már commitolt mai állapot is az indexbe kerül (#38) - #46

Merged
godavid merged 1 commit into
mainfrom
local/delta-anomalia-38
Nov 9, 2026
Merged

godavid merged 1 commit into
mainfrom
local/delta-anomalia-38

Conversation

@godavid

@godavid godavid commented Nov 9, 2026 •

Copy link
Copy Markdown
Owner
  • packages/pipeline/src/delta.ts: ha egy jogszabály egyik mai állapota terjedelem-anomáliára fut, a már commitolt, korábbi dátumú mai állapotai is bekerülnek az indexbe és a felvételi naplóba, a HEAD-helyreállítás pedig rájuk is fut. Eddig a slug minden mai eseménye kimaradt, így a HEAD régebbi szöveget mutathatott, miközben az index változatlan volt. Az anomaliak most az anomália dátumát is tárolja.
  • packages/pipeline/src/health.ts: két tiszta döntés-függvény a regebbiAllapot() mintájára: bekerultAllapot() és helyreallitasKell().
  • packages/pipeline/test/health.test.ts: 7 új teszt; a régi szabállyal kettő elbukik.
  • AGENTS.md, docs/uzemeltetes.md: a „Rossz adat nem kerül az adat-repóba” sor és a parser-hiba szakasz a valós viselkedést írja.

Closes #38

Elfogadás

  • egységteszt: d1 átmegy, d2 anomália → bekerült = [d1]; d1 < legutóbbi ismert → helyreállítás fut
  • pnpm --filter @gitjog/pipeline test: 110 sikeres, 4 kihagyva (golden hash, lemez-cache nélkül)
  • pnpm --filter @gitjog/pipeline exec tsc --noEmit: tiszta
  • helyi delta adat-repo-klónon + index-ellenorzes: nem futott; élesben a napi delta Index-integritás lépése ellenőrzi

Nitek (nem javítva)

  • delta.ts:401-409: a riasztás szövege még „az alábbiak kimaradtak”; a slug korábbi mai állapotai már bekerülhetnek, és a sor nem írja ki a kimaradt állapot dátumát (a.datum).
  • A tesztek csak a két tiszta függvényt fedik; azt az előfeltételt, hogy az anomaliak az ELSŐ anomália dátumát tárolja növekvő dátumsorrendben, csak a kód és a docstring őrzi.
  • model: claude-opus-5.5
    effort: high
    round: 1
    sha: 305dccc
    verdict: clean
    context: fresh
    findings: important 0, nit 2, pre-existing 2
    cost: 88k tokens
    duration: 1m37s
  • model: codex-gpt-6.1-sol
    effort: high
    round: 1
    sha: 305dccc
    verdict: clean
    context: fresh
    findings: important 0, nit 0, pre-existing 0
    cost: 73k tokens
    duration: 1m10s
  • model: gemini-4-argon
    effort: medium
    round: 1
    sha: 305dccc
    verdict: clean
    context: fresh
    findings: important 0, nit 0, pre-existing 0
    cost: unknown
    duration: 6m29s

🤖 Generated with Claude Code

…indexbe és a HEAD-helyreállításba kerül (#38)

A bekerült események szűrése eddig a slug szintű anomália-halmazból
számolt, így az anomália előtti, már commitolt mai állapot kimaradt az
indexből, a felvételi naplóból és a HEAD-helyreállításból. Az anomália
mostantól a napját is rögzíti; a bekerülést a bekerultAllapot(), a
helyreállítás szükségességét a helyreallitasKell() dönti el (tesztelve).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@godavid

godavid commented Nov 9, 2026

Copy link
Copy Markdown
Owner Author

Friss Claude-review (claude-opus-5.5@high, kontextus: csak a diff és az AGENTS.md szabályai).

  • Nit: delta.ts:401-409 – a riasztás szövege („az alábbiak kimaradtak”) már pontatlan, és nem írja ki a kimaradt állapot dátumát.
  • Nit: a tesztek csak a két tiszta függvényt fedik; az „első anomália dátuma, növekvő sorrend” előfeltételt csak a kód őrzi.
  • Pre-existing (parkolva: delta: a terjedelem-őr a lemezen lévő szöveghez mér, nem a legutolsó állapothoz #47): a terjedelem-őr a lemezen lévő szöveghez mér (delta.ts:244).
  • Ellenőrizve: dátum szerinti növekvő feldolgozás; slugonként egy esemény dátumonként (crawl.ts napiGyoztesek()); csak két nem dobó kihagyás van; az anomaliaOk nem ír az anomaliak-ba; az ismertNyers csak az anomália-ellenőrzés után változik; a shaTerkepParse() megtalálja a d1 SHA-ját; health.test.ts 36/36.

verdict: clean
important 0, nit 2, pre-existing 2

@godavid

godavid commented Nov 9, 2026

Copy link
Copy Markdown
Owner Author

Friss Codex-review (codex-gpt-6.1-sol@high, codex exec, kontextus: csak a diff és az AGENTS.md).

  • A ciklus növekvő dátumsorrendben dolgozik, az anomáliát az írás előtt rögzíti, így a bekerultAllapot() pontosan az elfogadott állapotokat választja (delta.ts:212).
  • A korábbi elfogadott állapotok az ismertNyers-ben maradnak, a slug eljut az index- és napló-SHA-kereséshez (delta.ts:342).
  • A helyreallitasKell() megtartja a korábbi max-dátum összevetést (delta.ts:307).

verdict: clean
important 0, nit 0, pre-existing 0

@godavid godavid added risk:medium reviewed:claude-opus-5.5@high claude-opus-5.5 review at high effort reviewed:codex-gpt-6.1-sol@high Codex GPT 6.1 Sol review at high effort review-round:1 owner-ok labels Nov 9, 2026
@godavid

godavid commented Nov 9, 2026

Copy link
Copy Markdown
Owner Author

owner-ok: a tulajdonos döntése 2026-11-09-én a /decide körben: „#38 delta-javítás (Ajánlott)” – két családos (Claude + Codex) review után landol.

@godavid
godavid enabled auto-merge (squash) November 9, 2026 08:30
@godavid
godavid merged commit c0eb318 into main Nov 9, 2026
16 checks passed
@godavid
godavid deleted the local/delta-anomalia-38 branch November 9, 2026 14:25
@godavid godavid added the review:in-progress Review in progress label Nov 11, 2026
@godavid

godavid commented Nov 11, 2026

Copy link
Copy Markdown
Owner Author

gemini-4-argon@medium · round 1 · full review (ffa4dad...305dccc)

verdict: clean
important 0, nit 0, pre-existing 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

delta: terjedelem-anomáliánál a HEAD régebbi szöveget mutathat, az index változatlan

1 participant