Skip to content

[Mate] Add a per-skill status table to skills:install output - #2477

Open
wachterjohannes wants to merge 1 commit into
symfony:mainfrom
wachterjohannes:improve-skills-install-output
Open

wachterjohannes wants to merge 1 commit into
symfony:mainfrom
wachterjohannes:improve-skills-install-output

Conversation

@wachterjohannes

@wachterjohannes wachterjohannes commented Sep 1, 2026

Copy link
Copy Markdown
Member
Q A
Bug fix? no
New feature? yes
Docs? no
Issues n/a
License MIT

Problem

skills:install typically touches several skills in one run, but its output was a couple of terse, comma-joined sentences with nothing per-skill. skills:list already renders a rich per-skill table; this brings skills:install up to the same level.

Before

6 skills installed: mate-php-environment-check, mate-symfony-log-investigation, mate-symfony-profiler-debugging, mate-symfony-request-triage, mate-symfony-service-inspection, mate-system-information

After

Summary sentences are kept, and a per-skill table is rendered underneath using skills:list's columns plus a new Action column (installed/rebuilt/skipped/unchanged, or the --dry-run equivalents):

+----------------------------+-----------------------+--------------+---------+---------+---------+--------+-----------+
| Installed Name             | Original              | Package      | Enabled | Mode    | State   | Status | Action    |
+----------------------------+-----------------------+--------------+---------+---------+---------+--------+-----------+
| mate-php-environment-check | php-environment-check | vendor/pkg-a | yes     | managed | managed | ok     | installed |
+----------------------------+-----------------------+--------------+---------+---------+---------+--------+-----------+

Removed skills stay reported via the existing "Removed N skill(s)" sentence (no current status to show). --format=json/--format=toon added, matching skills:list.

Tests

vendor/bin/phpunit, phpstan analyse --debug, php-cs-fixer fix: all green (one pre-existing, unrelated environment failure noted in the PR history). New tests cover install/update/skip, the table, dry-run wording, and --format=json.

@carsonbot carsonbot added Feature New feature Mate Issues & PRs about the AI Mate component Status: Needs Review labels Sep 1, 2026
@wachterjohannes
wachterjohannes force-pushed the improve-skills-install-output branch from 045936d to dfaf7f2 Compare September 6, 2026 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature New feature Mate Issues & PRs about the AI Mate component Status: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants