Skip to content

refactor: remove figlet and not used code#164

Merged
sjinks merged 2 commits intomainfrom
remove/figlet
Dec 28, 2025
Merged

refactor: remove figlet and not used code#164
sjinks merged 2 commits intomainfrom
remove/figlet

Conversation

@sjinks
Copy link
Copy Markdown
Member

@sjinks sjinks commented Dec 28, 2025

This pull request removes all usage of the figlet library and related ASCII art fonts from the CLI output, simplifying message formatting to plain colored text. It also eliminates several features and commands related to secret toggles and release channels, along with the associated code paths and tests. The main goal is to streamline the CLI codebase and reduce dependencies.

Dependency and Feature Removal:

  • Removed the figlet dependency from package.json and deleted all code that generated ASCII art using figlet fonts. All stylized text outputs now use plain colored text via chalk. [1] [2]
  • Removed all features and CLI options related to secret toggles and release channel management, including their handlers, user configuration updates, and output formatting. [1] [2]

Codebase Simplification:

  • Refactored the art.js module to remove the niceFont helper and update all output messages to use simple colored text instead of ASCII art. [1] [2] [3] [4] [5] [6] [7] [8]
  • Cleaned up the CLI implementation in cli.js by removing references to secret toggle files and related legacy logic. [1] [2]

Test Cleanup:

  • Removed test cases related to the removed makeArt functionality in cli.spec.js.

@sjinks sjinks self-assigned this Dec 28, 2025
@sjinks sjinks merged commit bb79795 into main Dec 28, 2025
18 checks passed
@sjinks sjinks deleted the remove/figlet branch December 28, 2025 01:38
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