Fix linter warnings in command package

Vasilii Iakliushinrequested to merge
duo-edit-20260107-094508 into main

Summary

This MR resolves all linter warnings reported in the command package and related files.

Changes

internal/command/command.go

  • Added package comment describing the command package purpose
  • Added comments for exported types: Command, LogMetadata, LogData
  • Added comment for CheckForVersionFlag function
  • Fixed Setup function comment to follow Go conventions (removed parentheses)
  • Added comment for NewLogData function
  • Fixed errcheck warning by explicitly ignoring closer.Close() error with _ = closer.Close()

internal/command/readwriter/readwriter.go

  • Added package comment
  • Added comment for ReadWriter type

internal/command/receivepack/gitalycall_test.go

  • Renamed struct field keyId to keyID to follow Go naming conventions
  • Fixed testifylint warning by reversing expected/actual values in require.Equal

internal/command/lfsauthenticate/lfsauthenticate.go

  • Fixed errcheck warning by handling fmt.Fprintf error return value

internal/command/githttp/pull.go and push.go

  • Extracted duplicate SSH request handling code into a shared executeSSHRequest helper function
  • This resolves the dupl linter warning about duplicate code blocks

Merge request reports