Skip to content

refactor: simplify cmd/metis imports and remove unused code - #13

Merged
meaningforger merged 2 commits into
mainfrom
codex/simplify-metis-imports
Sep 26, 2026
Merged

meaningforger merged 2 commits into
mainfrom
codex/simplify-metis-imports

Conversation

@meaningforger

@meaningforger meaningforger commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Use package names directly where imports do not collide; keep only the MCP SDK and DuckDB fixture aliases needed for disambiguation.
  • Remove unused request logging and a metrics route helper that production no longer calls.
  • Delete the obsolete cmd/metis metrics test file and test the actual metrics route in app/hosting.

Validation

  • go test ./cmd/metis ./app/hosting
  • CGO_ENABLED=1 go test -tags duckdb ./cmd/metis ./app/hosting
  • go run golang.org/x/tools/cmd/deadcode -test ./cmd/metis (no cmd/metis findings)
  • CGO_ENABLED=1 go run golang.org/x/tools/cmd/deadcode -tags duckdb -test ./cmd/metis (no cmd/metis findings)
  • git diff --check

@meaningforger meaningforger changed the title refactor: simplify cmd/metis import aliases refactor: simplify cmd/metis imports and remove unused code Sep 26, 2026
@meaningforger
meaningforger merged commit 7ca75f1 into main Sep 26, 2026
4 checks passed
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.

1 participant