Skip to content

Conversation

@jdalton
Copy link
Contributor

@jdalton jdalton commented Nov 6, 2025

Summary

Collection of small cleanup tasks and fixes that improve code quality without changing functionality.

Changes

1. Remove Dead Code (237 lines removed)

  • src/utils/optional-models.mts - Planned feature for lazy-loading AI models, never integrated
  • src/utils/coana/spawn.mts - Superseded by spawnCoanaDlx in dlx/spawn.mts

2. Remove Deprecated Commands from Bucket List

  • Remove 'ask' and 'console' from command set (already removed from codebase)

3. Improve Error Badge Text Contrast

  • Change error badge text from red to white for better readability
  • Previous red-on-red had poor contrast

4. Use Exact Coana Version

  • Remove tilde range prefix (~) from Coana version
  • Ensures consistent version across installations

5. Remove Unused Dependencies

  • Remove ansi-regex and string-width from catalog dependencies
  • No longer referenced in codebase

Benefits

✅ Cleaner codebase - Remove unused code
✅ Better UX - Improved error message readability
✅ Predictable builds - Exact Coana version matching
✅ Smaller dependency tree - Remove unused packages

Test Plan

  • Type check passes
  • Security checks pass
  • Verify error messages display correctly
  • Verify Coana commands work with exact version

Impact

Low risk - All changes are either cleanup or minor improvements with no functional impact on core features.

Test User added 5 commits November 6, 2025 09:19
Remove unused files that were either planned features or superseded by
other implementations.

Removed files:
- src/utils/optional-models.mts: Planned feature for lazy-loading AI models
  via npm packages, never integrated into codebase
- src/utils/coana/spawn.mts: Superseded by spawnCoanaDlx in dlx/spawn.mts,
  no references found in codebase

Both files had TODO comments and were not being imported or used anywhere.
Remove 'ask' and 'console' from the command bucket list as these
commands have been deprecated or removed.
Change error badge text color from red to white for better readability
against the red background. The previous red-on-red combination had
poor contrast.
Remove tilde range prefix (~) from Coana version specification to use
exact version matching. This ensures consistent Coana CLI version across
installations and prevents unexpected version updates.
Remove ansi-regex and string-width from root package.json catalog
dependencies as they are no longer used.
@jdalton jdalton closed this Nov 8, 2025
@jdalton jdalton deleted the jdalton/cleanup-and-fixes branch November 8, 2025 15:59
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