Repository navigation
fix(pipeline): anomáliánál a már commitolt mai állapot is az indexbe kerül (#38) - #46
Merged
Merged
Conversation
…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>
Owner
Author
|
Friss Claude-review (claude-opus-5.5@high, kontextus: csak a diff és az AGENTS.md szabályai).
verdict: clean |
Owner
Author
|
Friss Codex-review (codex-gpt-6.1-sol@high,
verdict: clean |
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. |
Owner
Author
|
verdict: clean |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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. Azanomaliakmost az anomália dátumát is tárolja.packages/pipeline/src/health.ts: két tiszta döntés-függvény aregebbiAllapot()mintájára:bekerultAllapot()éshelyreallitasKell().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
pnpm --filter @gitjog/pipeline test: 110 sikeres, 4 kihagyva (golden hash, lemez-cache nélkül)pnpm --filter @gitjog/pipeline exec tsc --noEmit: tisztaindex-ellenorzes: nem futott; élesben a napi deltaIndex-integritáslépése ellenőrziNitek (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).anomaliakaz ELSŐ anomália dátumát tárolja növekvő dátumsorrendben, csak a kód és a docstring őrzi.effort: high
round: 1
sha: 305dccc
verdict: clean
context: fresh
findings: important 0, nit 2, pre-existing 2
cost: 88k tokens
duration: 1m37s
effort: high
round: 1
sha: 305dccc
verdict: clean
context: fresh
findings: important 0, nit 0, pre-existing 0
cost: 73k tokens
duration: 1m10s
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