Fixed
Project:
Drupal core
Version:
main
Component:
meetings
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
1 Jul 2026 at 13:29 UTC
Updated:
1 Jul 2026 at 14:14 UTC
Jump to comment: Most recent
Hello, everyone. Welcome to the ✨ CLI in Core Initiative ✨ Meeting! 📅 This meeting:
➤ Usually happens weekly either at Wednesday 14:00 UTC.
➤ Will use Slack threads to continue discussions.
➤ Please comment on the roll call thread throughout the day.
| mradcliffe | Hi, Matthew here (mradcliffe). I put on shuffle for all my music as I was porting D12 modules, and as I typed this a new song started and it is “Somebody’s Watching Me” by Rockwell from a Halloween Party Monster Mix. |
| bircher | bircher - Stäffisburg by Patent Ochsner |
| mradcliffe | I haven't made progress on issue summary updates. |
| mradcliffe | I need to create a meeting notes issue from last week. |
| mradcliffe | https://www.drupal.org/project/drupal/issues/3602919 |
| mradcliffe | -#3594184: CLI cannot discover commands in subdirectories that have a period in the name |
| mradcliffe | A proposal for closing the feature request[#3588144], what are your thoughts @ifrik? |
| ifrik | What are the reasons proposing to close it? |
| mradcliffe | @dww mentioned some concerns over exposing sensitive CLI commands to the UI. |
| ifrik | Hmm... which would that be? Because I was thinking of only exposing them on those admin pages where you can do the same action in the GUI as in the CLI |
| ifrik | So, if you can't do the action on the GUI, then you won't see the CLI command |
| mradcliffe | Oh, I didn't realize moshe already closed it. Feel free to reply re-open. |
| mradcliffe | comment #5 has the list of things. |
| ifrik | My whole point was to make it easier for sitebuilders and site admins to start using (more) cli commands, not to run CLI commands in the UI. But maybe I was not clear about what I was suggesting |
| mradcliffe | https://www.drupal.org/project/drupal/issues/3597406 |
| bircher | @ifrik you mean we add an optional "route(s)" argument to the attribute or another attribute and then the ui has a small block that tells you "some of the things here can be done via cli" like the help module? |
| ifrik | yes! It only needs to be the information that an cli commands exist. No need to make it do anything in the GUI (edited) |
| ifrik | I've been using drush for years and I still miss the fact that there is a drush command for a module so I do it in the GUI. And some tasks are just faster in the CLI - like indexing your search - so sitebuilders benefit from knowing about the commands. |
| ifrik | @mradcliffe I've commented on[#3588144]#comment-16647206 and added a mockup because the concerns seem to address functionality that I didn't propose. Is it okay to reopen it? |
| mradcliffe |
| mradcliffe | @Gábor Hojtsy (he/him) asked about a Drush porting guide and I think either a page on the API documentation or a page underneath the Initiative page? |
| mradcliffe | @bircher described a bit of the process in the discussion in dww in #cli-in-core from last week. |
| bircher | yes so the most "difficult" is to convert a class with many methods (one command per method) into many classes for one command each. Then one can extend the command for dr to a new class where drush loads the classes and one easily supports both |
| mradcliffe | Hmm, I thought discovery would support AsCommand on the method as well as the class |
| bircher | maybe, but the location is different for both anyway |
| bircher | do we have tests for that? or examples? |
| mradcliffe | No example or tests for that. |
| Gábor Hojtsy (he/him) | The change record explains one class many commands! |
| mradcliffe | AsCommand attribute is only targeting classes |
| mradcliffe | Symfony 8.1 adds support for method based commands |
| mradcliffe | but not in Symfony 8.0 |
| Gábor Hojtsy (he/him) | That is important to elaborate on the change record too then ;) |
Participants:
mradcliffe, bircher, ifrik, Gábor Hojtsy
Comments
Comment #2
smustgrave commentedChecking slack threads appear to be captured, credited the users.