Piccolo CLI registers user app automatically

[pelid]

The documentation promises that the user app will not connect automatically: docs. But Piccolo CLI ignores this and registers the application anyway.

# piccolo_conf.py
APP_REGISTRY = AppRegistry(apps=[])

Console output:

$ piccolo
...
user create
Create a new user. 

user list
List existing users. 

user change_password
Change a user's password. 

user change_permissions
Change a user's permissions. 
...

The problem is that my project uses another similar application instead of the standard user application. When Piccolo CLI is launched, confusion begins in the names of commands and applications.

[sinisaos]

@pelid It might be a bit strange (wrong position) in the documentation, but the user app is enabled by default (if you commented out that line, you no longer see the user app in the terminal). If you have your own user app, it might be easiest (most practical) to write custom commands like this. The targ commands are just Python functions and you can specify whatever you want and register the command in piccolo_app.py like this. I hope that makes sense.

[pelid]

I have already written my own set of commands for custom user app ydmb and now I have two sets of similar commands in piccolo CLI help text:

# piccolo
...

user create
Create a new user. 

user list
List existing users. 

user change_password
Change a user's password. 

user change_permissions
Change a user's permissions. 

ydmb create_admin_account
Create a new admin account. If `non_interactive` is True, `username` and `password` must be provided.

ydmb list_admin_accounts
List admin accounts. By default shows only `active=true`.

ydmb change_password
Change admin account password. By default searches only `active=true` accounts.

It's very confusing:

• user create vs ydmb create_admin_account
• user change_password vs ydmb change_password

I want to remove all user app commands from piccolo CLI, but I can't do this without rewriting a piece of the library. Looks like a bug / missing feature.

[sinisaos]

Maybe we should remove the user app from the included apps because, as already stated in the authentication documentation, we need to register the user app to use it?