Skip to content

Fix syntax error in AppInfoTool packages array - #9

Open
wit3 wants to merge 1 commit into
NativePHP:mainfrom
wit3:fix/app-info-tool-syntax-error
Open

wit3 wants to merge 1 commit into
NativePHP:mainfrom
wit3:fix/app-info-tool-syntax-error

Conversation

@wit3

@wit3 wit3 commented Sep 25, 2026

Copy link
Copy Markdown

Problem

app/Mcp/Tools/AppInfoTool.php line 29 has a stray like this after the nativephp/mobile entry:

'nativephp/mobile' => InstalledVersions::getPrettyVersion('nativephp/mobile'), like this

PHP fails to parse the class (ParseError: syntax error, unexpected identifier "this", expecting "]"). The effects:

  • POST /mcp/superstack returns 500 on a fresh laravel new --using=nativephp/superstack app.
  • tests/Feature/McpServerTest.php fails.

Fix

Remove the stray text. php -l now passes.

🤖 Generated with Claude Code

A stray "like this" after the nativephp/mobile entry makes the class fail
to parse, so the MCP server returns 500 and McpServerTest fails on fresh
installs.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant