Skip to content

fix(core): substituir uso de String.match() por RegExp.exec() em runF…#7

Merged
bob-reis merged 1 commit into
bob-reis:mainfrom
fonteeboa:core
Sep 20, 2025
Merged

fix(core): substituir uso de String.match() por RegExp.exec() em runF…#7
bob-reis merged 1 commit into
bob-reis:mainfrom
fonteeboa:core

Conversation

@fonteeboa

Copy link
Copy Markdown
Contributor

…astRecon e extractMeta

Descrição

  • Substitui o uso de String.match() por RegExp.exec() em funções core (runFastRecon e extractMeta), atendendo à regra SonarQube (typescript:S6594).
  • Ajusta normalização de status (404 → not_found, 403 → inconclusive, etc.), garantindo consistência em testes e classificações.
  • Impacto direto: melhora a robustez de parsing (canonical/og:url, fallback Instagram), evita falsos positivos em status de perfil e corrige testes quebrados.

Escopo da Mudança

  • Código da API/Engine
  • UI/UX (inclua screenshots/gifs)
  • Manifesto de sites (docs/sites.core.yaml)
  • Documentação (docs/*)

Checklist de Qualidade (Architect)

  • Lint e typecheck passam (pnpm lint, pnpm typecheck)
  • Testes unit/contract/e2e passam e cobertura mantida (≥ 85% no core)
  • Contrato SSE estável (site_start, site_result, progress, done)

Segurança (Agent Smith / Neo / Trinity)

  • Entrada validada (regex/allowlist por site), sem SSRF
  • Sem persistência de dados; logs sem PII
  • Semgrep e audit sem achados alto/crit
  • Nenhum segredo no código (Gitleaks)
  • Threat model revisado se superfícies mudaram

Notas de Implementação

  • Para telas de recuperação: 1 tentativa, sem seguir links, apenas parsing da página inicial
  • Identificadores mascarados rotulados corretamente

Outras Observações

  • Issues relacionadas: #123 (corrige falha em tests/status.spec.ts, tests/fast.spec.ts, tests/meta.spec.ts)
  • Screenshots/Anexos:
image Captura de tela 2025-09-19 162048 Captura de tela 2025-09-19 161920

@vercel

vercel Bot commented Sep 19, 2025

Copy link
Copy Markdown

@fonteeboa is attempting to deploy a commit to the Robertson Reis' projects Team on Vercel.

A member of the Team first needs to authorize it.

@bob-reis
bob-reis merged commit 95d7e98 into bob-reis:main Sep 20, 2025
11 of 12 checks passed
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.

2 participants