Skip to content

[BUG] Lint rules assessment dismiss linting if not part of pre-commit or agent hooks, also effects Code Smell Elimination scoring #474

@meyrevived

Description

@meyrevived

Bug Description

When scanning Konflux-ci/multi-platform-controller AgentReady score for the repo was falsly low:
AgentReady expects linting rules to exist only in its idea of deterministic enforcement, despite .golangci.yaml which configures 19 linters including gosec, staticcheck, errcheck, ginkgolinter, plus gofmt and goimports formatters. CI enforces all of these on every PR via go-ci.yaml. This IS deterministic enforcement — just not via git hooks.
Also effected is the Code Smell Elimination section, which also looks at the linting tool golangci-lint and not the actual linters it runs.

To Reproduce

  1. Clone Konflux-ci/multi-platform-controller
  2. Run AgentReady scan using the uvx command
  3. Review the results

Expected Behavior

Findings in Deterministic Enforcement reflect the existence of go-ci.yaml's Lint job.

Actual Behavior

scan report claims "No deterministic enforcement found"

assessment-20260526-105424.json

Environment

  • OS: Fedora CSB 42 (64 bit)
  • Version: v2.40.0
  • Python Version: 3.14.4

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions