Repository navigation
fix(api,szoveg): pont nélküli §, naptári dátum-ellenőrzés, rövid cache - #30
Conversation
datum, since, until, tol and ig accepted shape-valid but non-existent days (2026-13-01, 2026-02-30, 2026-10-00) and picked a time state by string comparison. A shared ervenyesDatum() round-trip check now backs the REST/MCP zod schema and the API-level checks. Adds a node:test runner for apps/web (test files excluded from the Next typecheck). Fixes #28 Co-Authored-By: GLM 5.3 (z.ai) <noreply@z.ai> Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Only a datum before today (UTC) is final and keeps s-maxage 86400; today or later gets 900 s like valtozasok, so the daily delta shows up within minutes. Fixes #27 Co-Authored-By: GLM 5.3 (z.ai) <noreply@z.ai> Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
szam() required the dot, so "18 §" returned 404 on heading-less laws while hivatkozas.ts already treats the dot as optional. Fixes #29 Co-Authored-By: GLM 5.3 (z.ai) <noreply@z.ai> Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Review — claude-opus-5.5 @ high, round 1, fresh context, head
Ellenőrizve, rendben: a verdict: clean |
|
owner-ok — landolási engedély a tulajdonostól (2026-10-23, az orchestráló sessionben, a GLM-kvóta prioritáslistája után):
|
CodeQL js/polynomial-redos flagged szam(): the optional dot made /\.?\s*§\s*$/ quadratic on inputs with many spaces. Strip the trailing § and dot with string operations instead; a 100k-space input now runs in linear time. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Review (claude-opus-5.5@high, round 1, fresh context, head d85942b) The new commit d85942b fixes the CodeQL Ran: verdict: clean |
A diffet a GLM 5.3 (z.ai, OpenCode-on át) írta, egy Claude-felügyelő ellenőrizte és fogadta el. Fixes #29, fixes #28, fixes #27. A #26 nyitva marad (tulajdonosi döntés).
szam()(packages/szoveg/src/paragrafus.ts) a pontot opcionálisnak veszi, így a „18 §" ugyanúgy működik, mint a „18. §". Új vitest-eset a „18 §", „18.§" és „18/A §" alakra a meglévő heading nélküli mintán.ervenyesDatum()(apps/web/lib/api/datum.ts: alak + UTC oda-vissza alakítás). Használja aszakasz.ts(datum), avaltozasok.ts(since/until), adiff.ts(tol/ig) és a közös zod-séma (semak.ts,.refine) a REST-hez és az MCP-hez. A már nem használtDATUM_MINTAkikerült akozos.ts-ből.szakaszCacheMp()(apps/web/lib/api/cache-ido.ts): múltbeli datum 86400 s, mai vagy jövőbeli 900 s, datum nélkül 3600 s; bekötve a/api/v1/szakaszroute-ba, a hibás komment javítva. Korlát: egy múltbeli datum, amelynek időállapotát a delta későn rögzíti, legfeljebb 24 órán át elavult lehet — az issue javasolt megoldása is ezt fogadja el.apps/web-ben:testscript (node --test "lib/**/*.test.ts", új függőség nélkül); atsconfig.jsonkizárja a**/*.test.ts-t, hogy a Next típusellenőrzése ne bukjon a.ts-kiterjesztésű importokon. CI-ben ma nincs tesztjob, ezek kézzel vagypnpm -r test-tel futnak.Elfogadási feltételek: mindhárom issue viselkedése teljesül, mindegyikhez fókuszált teszt.
Tesztterv (
dev-budgetalatt, helyben):apps/web:node --test lib/api/datum.test.ts lib/api/cache-ido.test.ts→ 7/7 zöldpackages/szoveg:vitest run test/paragrafus.test.ts→ 5/5 zöldReview-nitek (nem javítjuk automatikusan):
apps/web/lib/api/semak.ts:35: adatumséma a.regex()és azervenyesDatummiatt kétszer ellenőrzi az alakot; rossz alakú értékre a 400-as üzenet két hibát fűz össze. Elég lenne a.refine(ervenyesDatum, …).apps/web/lib/api/cache-ido.ts:8: amaUTC-dátum, nem budapesti; éjfél után 1-2 óráig a helyi tegnapi datum is 900 s-ot kap. Ez a biztonságos irány, egy komment jelezhetné, hogy szándékos.effort: high
round: 1
sha: 439a12a
verdict: clean
context: fresh
findings: important 0, nit 2, pre-existing 0
cost: unknown
🤖 Generated with Claude Code