Skip to content

[codex] Remove unused Rust helpers#17146

Merged
pakrym-oai merged 3 commits intomainfrom
pakrym/remove-unused-functions
Apr 14, 2026
Merged

[codex] Remove unused Rust helpers#17146
pakrym-oai merged 3 commits intomainfrom
pakrym/remove-unused-functions

Conversation

@pakrym-oai
Copy link
Copy Markdown
Collaborator

Summary

Removes high-confidence unused Rust helper functions and exports across codex-tui, codex-shell-command, and utility crates.

The cleanup includes dead TUI helper methods, unused path/string/elapsed/fuzzy-match utilities, an unused Windows PowerShell lookup helper, and the unused terminal palette version counter. This keeps the remaining public surface smaller without changing behavior.

Validation

  • just fmt
  • cargo test -p codex-tui -p codex-shell-command -p codex-utils-elapsed -p codex-utils-fuzzy-match -p codex-utils-string -p codex-utils-path
  • just fix -p codex-tui -p codex-shell-command -p codex-utils-elapsed -p codex-utils-fuzzy-match -p codex-utils-string -p codex-utils-path
  • git diff --check

@pakrym-oai pakrym-oai force-pushed the pakrym/remove-unused-functions branch from 5124b3d to 6b76c90 Compare April 8, 2026 20:21
@pakrym-oai pakrym-oai marked this pull request as ready for review April 13, 2026 23:33
@pakrym-oai pakrym-oai enabled auto-merge (squash) April 13, 2026 23:33
@pakrym-oai pakrym-oai disabled auto-merge April 14, 2026 01:26
@pakrym-oai pakrym-oai merged commit 0c8f317 into main Apr 14, 2026
20 of 22 checks passed
@pakrym-oai pakrym-oai deleted the pakrym/remove-unused-functions branch April 14, 2026 01:27
@github-actions github-actions bot locked and limited conversation to collaborators Apr 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants