Skip to content

Remove rhiza welcome command#391

Merged
tschm merged 2 commits intomainfrom
copilot/remove-rhiza-welcome-command
Mar 8, 2026
Merged

Remove rhiza welcome command#391
tschm merged 2 commits intomainfrom
copilot/remove-rhiza-welcome-command

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 8, 2026

rhiza welcome was a dedicated command that only printed a static ASCII-bordered banner — information already covered by rhiza --help. Deleted the command and all associated code.

Changes

  • src/rhiza/commands/welcome.py — deleted entirely
  • src/rhiza/cli.py — removed welcome_cmd import and the @app.command() welcome() function
  • tests/test_cli_commands.py — removed TestWelcomeCommand class and its three test methods
Original prompt

Summary

Remove the rhiza welcome command as a simplification. It is a 62-line file that does nothing except print() a static ASCII-bordered welcome message — information that duplicates rhiza --help. A dedicated command for a static text banner is unnecessary complexity.

Changes required

1. Delete src/rhiza/commands/welcome.py

Delete the file entirely. It contains a single function welcome() that only prints a hardcoded multi-line string.

2. Update src/rhiza/cli.py

  • Remove the import on line 23:
    from rhiza.commands.welcome import welcome as welcome_cmd
  • Remove the entire welcome CLI command (lines 388–398):
    @app.command()
    def welcome() -> None:
        r"""Display a friendly welcome message and explain what Rhiza is.
    
        Shows a welcome message, explains Rhiza's purpose, key features,
        and provides guidance on getting started with the tool.
    
        Examples:
            rhiza welcome
        """
        welcome_cmd()

3. Update tests/test_cli_commands.py

Remove the entire TestWelcomeCommand class and its three test methods:

  • test_welcome_command
  • test_welcome_command_function_coverage
  • test_welcome_cli_wrapper_coverage

These tests exclusively test the welcome command and welcome.py module being deleted.

Notes

  • No other files import from rhiza.commands.welcome — only cli.py and the test file.
  • The commands/__init__.py does not import welcome, so no change is needed there.
  • This is a pure deletion/cleanup — no replacement behaviour is needed. Users wanting an overview of the tool should use rhiza --help.

This pull request was created from Copilot chat.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: tschm <2046079+tschm@users.noreply.github.com>
@tschm tschm marked this pull request as ready for review March 8, 2026 08:05
Copilot AI changed the title [WIP] Remove rhiza welcome command for simplification Remove rhiza welcome command Mar 8, 2026
Copilot finished work on behalf of tschm March 8, 2026 08:05
@tschm tschm merged commit 2610132 into main Mar 8, 2026
16 checks passed
@tschm tschm deleted the copilot/remove-rhiza-welcome-command branch March 8, 2026 08:06
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