Skip to content

Remove redundant client wrappers and unused code - #862

Draft
btsouth wants to merge 1 commit into
mainfrom
codex/cleanup-redundant-code
Draft

Remove redundant client wrappers and unused code#862
btsouth wants to merge 1 commit into
mainfrom
codex/cleanup-redundant-code

Conversation

@btsouth

@btsouth btsouth commented Sep 5, 2026

Copy link
Copy Markdown
Owner

Inline 21 client-path forwarding functions, remove an unused Separator component, and drop three tests that only checked documentation or another test's source text. Platform-specific resolvers and behavioral tests remain.

Validation: client tests, packaging tests, TypeScript, and formatting checks.

Note

Remove redundant client path wrappers and unused code

  • Replaces 22 dedicated client path helper functions in clients.rs with inline closures that call the shared config-path resolver, moving per-client path comments into the registry entries
  • Deletes the unused Separator UI component from separator.tsx
  • Removes stale or obsolete tests: a SECURITY.md regression test in security_doc.rs, Homebrew cask and Arch installer assertion blocks in homebrew-cask.test.ts and linux-packaging.test.ts
  • Risk: removal of the SECURITY.md and packaging doc tests means those docs are no longer guarded by CI; stale content can reappear unnoticed

Macroscope summarized 1ef78b9.

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