Repository navigation
Point list-forms description at list-entries - #3621
Conversation
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
|
Important Review skippedBot user detected. To trigger a single review, invoke the
|
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| PHP | Oct 26, 2026 4:26p.m. | Review ↗ | |
| JavaScript | Oct 26, 2026 4:26p.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
There was a problem hiding this comment.
Approve: one-string change, correct against Lite.
list-entriesis registered on Lite (FrmAbilitiesEntriesController, listed intest_FrmAbilitiesContract.php);create-entryis Pro only (only mentioned inFrmMcpAbilityRegistry). The description now names only Lite abilities, as the issue (formidable-pro#6858) asks.list-fieldsandget-formkept.list-entriestakesform_idas ID or form_key, so "form_key or id" still reads true.- No test asserts this string (grepped
tests/).php -lclean. - Psalm is red here but also fails on master at the base commit (a0b9742); this PR touches no code path.
Not run: no live before/after. The "bug" is a wrong word in a description string, checked by reading source, not by running an MCP client. Local PHPUnit not run; CI skipped the heavy jobs for this bot-authored PR.
Fixes Strategy11/formidable-pro#6858
Broken: the
list-formsability description told callers to use the form key forcreate-entry, which Lite doesn't register (Pro only).Changed: description now names
list-entriesinstead;list-fieldsandget-formkept.Verified:
php -lclean. No test asserts this string (grep oftests/), so none added. Single-string change, no CI-behavior impact.🤖 Generated with Claude Code