[godavid]

A diffet a GLM 5.3 írta (z.ai, OpenCode-on át), egy Claude-felügyelő ellenőrizte és fogadta el.

• extractZip: bejegyzésszám-korlát hiányzik (a 200 MB-os sapka nem köti a darabszámot) #2: az extractZip elutasítja a 2000-nél több bejegyzésű zipet (MAX_ENTRIES). Az ellenőrzés az fflate szűrőjében fut, még írás előtt. Teszt: 2001 bejegyzés hibát ad és üres célmappát hagy; pontosan 2000 mind kibomlik.
• Letöltött hivatalos iratok, meta.json és index.jsonl jogosultsága umask-ra hagyatkozik (0644/0755) #3: a levélmappák és kibontott mappák 0o700, a levélfájlok, a meta.json, a kibontott fájlok és az index.jsonl 0o600 jogosultsággal jönnek létre (src/extract.ts, src/tarhely/inbox.ts, src/store/index.ts). Új jogosultság-tesztek: test/inbox.test.ts, test/index-store.test.ts (alap 022-es umask mellett a forrásváltozás nélkül elbuknának).
• Allowlist-tesztek hiányoznak: nagybetű, záró pont, IDN/punycode, userinfo URL, IP-literál #4: új isAllowedHost-tesztek: nagybetűs host, záró pont, cirill homoglifa és punycode alakja, userinfo URL, IPv4 és IPv6.

Fixes #2
Fixes #3
Fixes #4

Tesztterv: 4 fókuszált tesztfájl, 58 teszt zöld helyben, Biome tiszta. A teljes kapu a PR CI-ja.

Nem kész: a #4 opcionális, launchBrowser-szintű route-tesztje.

Review nitek (nem javítjuk automatikusan):

• 2000-nél több bejegyzésű valódi levél tartósan a hibalistán marad, és minden futás újra letölti; megfontolandó a kibontás kihagyása figyelmeztetéssel.
• A mode csak új fájlra és mappára hat; régebbi telepítés mappái és index.jsonl-je 0o755/0o644 marad (chmodSync vagy README-jelzés kellene).
• src/extract.ts: a komment („semmi nem kerül a célmappába”) csak a szűrőkorlátokra igaz, írási hibánál félkész kibontás maradhat.

• model: claude-opus-5.5
  effort: high
  round: 1
  sha: 092cdaa
  verdict: clean
  context: fresh
  findings: important 0, nit 3, pre-existing 1
  cost: unknown

🤖 Generated with Claude Code

[godavid]

Review (claude-opus-5.5@high, round 1, fresh context, head 092cdaa)

Important

none

Nit

• src/tarhely/inbox.ts:159-160 + src/extract.ts:47-49: egy 2000-nél több bejegyzésű, de valódi levél most tartósan hibás lesz. extractZip dob, miután a levélfájl már a lemezen van. A levél a failures listába kerül (inbox.ts:112-116), nem kerül az index.jsonl-be, és minden futás újra letölti és újra elbukik. Ugyanez a minta már megvan a 200 MB-os és a gyanús útvonalas korlátnál, így ez a viselkedés tervezett. A 2000-es küszöb viszont új okot ad rá. Megfontolandó a kibontás kihagyása figyelmeztetéssel az egész levél elbuktatása helyett.
• src/tarhely/inbox.ts:156-157, src/store/index.ts:43-48: a mode csak az új fájlra és mappára hat. A korábbi verzió által létrehozott outDir, <mailbox.slug> mappa és index.jsonl 0o755/0o644 marad. Ugyanez igaz egy újrapróbált levél már létező fájljára. Az index.ts kommentje ezt kimondja, de a régi telepítéseken a szigorítás nem érvényesül. Ehhez egy chmodSync kellene, vagy a README-ben jelezni.
• src/extract.ts:41-42: a komment szerint hiba esetén „semmi nem kerül a célmappába”. Ez csak a szűrőben dobott korlátokra igaz. Ha a kiíró ciklus közben jön írási hiba, félkész kibontás marad. Pontosítani kellene: „ha valamelyik szűrő-korlát hibát dob”.

Pre-existing

• src/extract.ts:66-72: ha két bejegyzés ütközik, a kiíró ciklus félúton elhasal és részleges kibontást hagy. Két példa: a fájl és a/b bejegyzés, vagy kis- és nagybetűben eltérő nevek kis-nagybetű-érzéketlen fájlrendszeren. A hibát a diff nem hozta be, a ciklus az origin/main-en is így működik.

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

Pre-existing → #7

[godavid]

owner-ok — a tulajdonos kérése 2026-10-23-án, a GLM-kvóta prioritáslistája után, az irányító sessionben:

"hajtsd végre ezeket, subagent -ek felügyelnek, te csak összefogod"

Folytatási utasítás, 2026-10-23:

"futtasd te"