Skip to content

code quality: remove dead code flagged by go vet and golangci-lint #707

Description

@wolf31o2

Goal

Remove dead code surfaced by go vet and golangci-lint, plus commented-out code blocks and stale TODO comments for items that have already shipped.

Changes

  • go vet ./... should be clean. Fix any warnings.
  • Remove commented-out code blocks. Do not remove example snippets in doc-style comments.
  • Remove functions and types flagged as unused by golangci-lint or staticcheck.
  • Audit TODO comments and remove any that reference work that has already shipped (cross-check against closed issues and recent commits).

Verification

go build ./...
go vet ./...
golangci-lint run
make test

Dependencies

Recommended after the slog and error message tasks since those touch many of the same call sites; landing this last avoids merge conflicts.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    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