fix functional-tests skill frontmatter + add skill-review CI

[fernandezbaptiste]

hey @MakFly, thanks for building superpowers-symfony. really like the breadth of Symfony skills you've put together in one place. Kudos on soon reaching 100 stars! I've just starred it.

ran your functional-tests skill through agent evals and spotted a quick fix:

• fixed allowed-tools from YAML array to string format (required for valid frontmatter parsing)

• expanded description with trigger terms like TDD, regression tests, bug reports, functional coverage

also added a lightweight GitHub Action that auto-reviews any skill.md changed in a PR (includes min permissions, uses a pinned action version, only posts a review comment).

this means that it gives you and your contributors an instant quality signal before you have to review yourself (no signup, no tokens needed).

these were easy changes to bring the skill in line with what performs well against Anthropic's best practices. honest disclosure, I work at tessl.io where we build tooling around this. not a pitch, just fixes that were straightforward to make! happy to answer any questions on the changes.

[MakFly]

Hi ! ,

I Check that today ! With all testing and more update !

Thanks !

[MakFly]

Hey @fernandezbaptiste, thanks for the PR and the kind words! 🙏

I've cherry-picked the useful part of your contribution — the enriched description with trigger terms — and applied
it directly on main (commit 7a1ea33). I also added WebTestCase which was missing from your version.

Regarding the two other changes, I'm not merging them for the following reasons:

allowed-tools format (array → string): Both YAML formats are valid and correctly parsed. All 43 skills in this
plugin consistently use the array format. Changing just one would break internal consistency. If we ever migrate to
string format, it'll be done across the board.

skill-review.yml workflow: I appreciate the thought, but I'd rather not add an external dependency
(tesslio/skill-review) with pull-requests: write access to the repo at this time. We already validate with
claude plugin validate . which covers frontmatter and schema validation.

Closing this PR since the description improvement has been integrated. Thanks again for taking the time to
contribute!

[fernandezbaptiste]

thanks for cherry-picking the description improvement @MakFly - glad it was useful! totally fair on the other two points.

curious - you mentioned you validate with claude plugin validate . for frontmatter and schema. can you share how you go about it?

if it's helpful, I'd be happy to keep contributing improvements to other skills as well and open more PRs where I see clear wins. you've got 43 skills there so happy to help improve more.

one thing to flag is that even after a skill is improved once, it can still drift over time as prompts evolve. longer-term value is having review + improvement as part of CI flows so regressions get caught early.

keen to hear how you would imagine this to be part of your CI approach - looking to get feedback on how to approach this in a secure way, so that it gives maintainers and their contributors an instant quality signal on every skill.md change, as well as concrete improvements when you need them.