fix(archives): restore .7z content read via BytesIOFactory (#1250 #1248)

[FabioLeitao]

Summary

• Replace nonexistent `SevenZipFile.read()` with documented `extract(targets=..., factory=BytesIOFactory)` ([P1][bug] Leitura de conteudo dentro de .7z sempre falha (AttributeError) - nao e questao de senha, e API do py7zr incompativel #1250 / [bug] .7z sempre falha com AttributeError: 'SevenZipFile' object has no attribute 'read' mesmo com py7zr instalado #1248).
• Collect desired members after filters, then one extract pass (solid-archive friendly) + `reset()` after `list()`.
• Password-required path again surfaces as `encrypted_no_password`, not `archive_read_error`/`AttributeError`.

Closes #1250
Closes #1248

Test plan

• `./scripts/quick-test.sh --path tests/test_archives_scan_failures.py` (round-trip PII, multi+filters, encrypted)
• `./scripts/check-all.sh --enforced`
• CI green on PR
• Bugbot review (solid extract + password path)

Made with Cursor

[cursor]

Stale comment

[FabioLeitao]

Auditoria externa read-only (A.I.I.D.C.O.B.P.P.) — confirmação independente do achado do Bugbot sobre a interação com o budget do #1233.

Verificado contra o diff atual deste PR (commit 707549ed, branch do .7z em core/archives.py):

Confirmo o achado (HIGH) do Bugbot. No loop for member in archive.list():, a chamada a _after_examine(...) (hook do budget agregado introduzido no #1233 / merge #1243) ainda dispara return no meio da coleta. Como esse return acontece antes do extract(targets=desired, ...), se o budget estoura no membro N, todos os membros já enfileirados em desired (1..N-1) nunca são extraídos — o .7z perde tudo que já havia sido selecionado antes do teto.

Divergência de formato confirmada: nos branches ZIP/TAR, quando o budget estoura no loop, os membros lidos antes do estouro já foram materializados/yielded — o scan preserva o trabalho parcial. O novo branch 7z, com return antes do extract, descarta o trabalho parcial. Comportamento divergente entre formatos no mesmo cenário de choke do #1233 — regressão de paridade, como o Bugbot apontou.

Direção do fix (concordo com o Bugbot): trocar o return no loop de coleta por break sai da coleta mas segue para o extract(desired), processando os membros que couberam antes do estouro (o membro que estourou o budget não entra em desired). Isso restaura paridade com ZIP/TAR.

Nota de processo: este vetor (fix do .7z × budget agregado do #1233) não foi capturado nem pela auditoria de fonte interna nem por esta auditoria externa olhando o diff isoladamente — apareceu só no Bugbot, porque é um bug de interação entre o fix novo e um choke preexistente. Recomenda-se aplicar o break neste mesmo PR (correção do próprio fix, não mudança nova) e re-rodar o Bugbot antes do merge — a alteração toca justamente o caminho do budget, que merece re-verificação, não merge direto.

(Auditoria read-only: não commito, não mergeio, não aprovo — apenas registro a verificação. Merge é decisão do operador, após Bugbot verde na segunda passada.)