Skip to content

chore: remove unreachable helpers#3769

Merged
Sayt-0 merged 1 commit into
mainfrom
chore/remove-dead-code
Jul 21, 2026
Merged

chore: remove unreachable helpers#3769
Sayt-0 merged 1 commit into
mainfrom
chore/remove-dead-code

Conversation

@Sayt-0

@Sayt-0 Sayt-0 commented Jul 21, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove five unreachable helpers left behind by earlier refactors.
  • Remove the unused toolset metadata abstraction superseded by Describer and DescribeToolSet.
  • Remove imports made obsolete by the cleanup.

Removed code

Area Removed
CLI Printer.PrintError
Server NewSourceLoader wrapper
Tools ToolsetMetadata and ToolsetIdentifier
Working directory workingdir.Default
TUI hit testing ExtractCoords

Static analysis

Metric Before After
deadcode -test findings 68 63

Validation

Check Result
go test ./... passed
go build ./... passed
go vet ./... passed
golangci-lint run --timeout=10m passed, 0 issues
git diff --check passed
Independent review passed, no findings

The full test suite was run without the workspace HTTP proxy variables, which otherwise intercept private-address requests used by the SSRF tests.

@Sayt-0
Sayt-0 requested a review from a team as a code owner July 21, 2026 11:38
@Sayt-0
Sayt-0 enabled auto-merge July 21, 2026 11:43
@Sayt-0
Sayt-0 merged commit eb1e6e5 into main Jul 21, 2026
14 checks passed
@Sayt-0
Sayt-0 deleted the chore/remove-dead-code branch July 21, 2026 11:53
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