Skip to content

fix(reviewer): add deterministic integrity gate - #31

Open
marcelokarval wants to merge 1 commit into
sandeco:mainfrom
marcelokarval:fix/reviewer-deterministic-gate
Open

fix(reviewer): add deterministic integrity gate#31
marcelokarval wants to merge 1 commit into
sandeco:mainfrom
marcelokarval:fix/reviewer-deterministic-gate

Conversation

@marcelokarval

Copy link
Copy Markdown

Resumo

Adiciona um gate determinístico e read-only à conclusão do Reviewer para impedir que inconsistências objetivas sejam persistidas no checkpoint.

O gate valida:

  • referências Q-XX-NN órfãs;
  • divergências de severidade 🔴/🟡, inclusive com marcador antes ou depois do ID;
  • conjunto de bloqueantes versus perguntas 🔴;
  • artefatos obrigatórios por doc_level;
  • cross-review-result.md somente quando a revisão cruzada ocorreu;
  • denominadores publicados de arquivos Markdown e linhas.

Integração

  • distribui agents/reversa-reviewer/scripts/verify-discovery-review.py no pacote;
  • torna IDs canônicos Q-<UNIT>-<NN> explícitos no template;
  • bloqueia o checkpoint reviewer da fase revisao até ok=true;
  • permite waiver somente com aprovação humana explícita e finding_ids individuais presentes no resultado;
  • persiste um recibo auditável no checkpoint;
  • executa o conjunto no npm run verify e no workflow existente.

Verificação

  • npm run verify: PASS
    • eixo de invocação: PASS
    • 18 testes do gate: PASS
    • transporte do installer: PASS
  • python3 -m py_compile: PASS
  • git diff --check: PASS
  • npm pack --dry-run --json: validador incluído; nenhum bytecode incluído
  • revisão independente final: ACCEPTED

Evidência real

A reprodução read-only contra um corpus grande de engenharia reversa retornou exit 1 esperado e 25 findings com 25 IDs únicos:

  • 14 conflitos de severidade;
  • 6 referências órfãs;
  • 2 artefatos obrigatórios ausentes;
  • 1 divergência de bloqueantes;
  • 1 divergência de contagem de Markdown;
  • 1 divergência de contagem de linhas.

O corpus usado como evidence case não integra este PR e não foi modificado.

Fixes #30

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(reviewer): validar referências, severidades, denominadores e artefatos obrigatórios

1 participant