Skip to content

chore: severityRank duplicado entre internal/scanner e internal/cli #37

Closed

Description

@joaooncode

Problema

internal/scanner/scanner.go agora tem sua própria função não exportada severityRank, duplicando a que já existia em internal/cli/scan.go. As duas ordenam os mesmos quatro valores de Severity de forma idêntica.

Risco

As duas cópias podem divergir se níveis de severidade forem adicionados ou reordenados no futuro — alguém atualiza uma cópia e esquece a outra, quebrando silenciosamente a filtragem de severidade na CLI ou a lógica de piso/teto em scanSecrets em apenas um dos dois pontos.

Correção sugerida

Exportar uma única implementação (provavelmente a partir de internal/scanner, já que Severity é definido lá) e fazer internal/cli usá-la em vez de manter sua própria cópia.

Contexto

  • Encontrado durante o code review da branch feat/secret-scanning (commit 40a4708).
  • Arquivos: internal/scanner/scanner.go:242, internal/cli/scan.go (severityRank)

Activity

  1. added
    coreCore engine, scanning, and detectors
    choreTooling, CI/CD, and project setup
    on Sep 11, 2026
  2. changed the title [-]chore: severityRank duplicated between internal/scanner and internal/cli[/-] [+]chore: severityRank duplicado entre internal/scanner e internal/cli[/+] on Sep 11, 2026
  3. added a commit that references this issue on Sep 12, 2026
    ab96671
  4. joaooncode commented on Sep 12, 2026

    @joaooncode
    OwnerAuthor

    Corrigido no PR #44 (mesclado em main): a implementação de severityRank foi consolidada em um único scanner.SeverityRank exportado, usado tanto por internal/scanner quanto por internal/cli, com teste de unidade cobrindo os ranks.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    choreTooling, CI/CD, and project setupcoreCore engine, scanning, and detectorsgood first issueGood for newcomers

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions