-
Notifications
You must be signed in to change notification settings - Fork 4
Required changes #32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Required changes #32
Conversation
axif0
commented
Sep 5, 2025
- Update README and enhance CLI error messages
- add command handlers for improved functionality
…r improved functionality
- Update version output to 1.0 - Fix JSON command status codes and outputs - Improve credential and auth error messages - Fix argument parsing errors for invalid commands - Clean up table formatting and output consistency - Remove debug artifacts and TOFIX comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code quality is better, no nested lambda.
Readme Usage section should be proof read, most commands are misspelled
README.md
Outdated
|
|
||
| ## Available Commands | ||
| - `file (f)` - Manage individual file operations and metadata | ||
| - `background-activities` - Monitor and manage long-running background processes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is background, not "background_activities" ! Run help and check spelling of each command carefully!
cbrain_cli/handlers.py
Outdated
| if target == "all": | ||
| print("Project Unswitch 'all' not yet implemented as of Aug 2025") | ||
| else: | ||
| print(f"Error: Invalid target '{target}'. Only 'all' is supported.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it is should be just 'unswitch' without any arguments.