[agent/claude:opus] DX audit and fix (Laravel/PHP package). 1) Review CLAUDE.md ... #4

Closed

Virgil wants to merge 0 commits from agent/dx-audit-and-fix--laravel-php-package into dev

Virgil commented

2026-03-17 09:04:09 +00:00

Member

Task

DX audit and fix (Laravel/PHP package). 1) Review CLAUDE.md — update any outdated commands. 2) Run composer test if available — report results. 3) Check code follows PSR-12 via Laravel Pint. Run ./vendor/bin/pint --test if available. 4) Check declare(strict_types=1) in all PHP files. 5) Commit fixes.

Agent: claude:opus
Commits: 1
Branch: agent/dx-audit-and-fix--laravel-php-package

Auto-created by core-agent dispatch system.
Co-Authored-By: Virgil virgil@lethean.io

## Task DX audit and fix (Laravel/PHP package). 1) Review CLAUDE.md — update any outdated commands. 2) Run `composer test` if available — report results. 3) Check code follows PSR-12 via Laravel Pint. Run `./vendor/bin/pint --test` if available. 4) Check declare(strict_types=1) in all PHP files. 5) Commit fixes. **Agent:** claude:opus **Commits:** 1 **Branch:** `agent/dx-audit-and-fix--laravel-php-package` --- Auto-created by core-agent dispatch system. Co-Authored-By: Virgil <virgil@lethean.io>

Virgil added 1 commit 2026-03-17 09:04:09 +00:00

fix(dx): code style fixes, strict_types, and test repair

CI / PHP 8.3 (pull_request) Successful in 2m32s

Details

CI / PHP 8.4 (pull_request) Successful in 2m17s

Details

208cb93c95

- Remove non-existent src/Core/Service/ from CLAUDE.md L1 packages list
- Fix LifecycleEventsTest: remove dependency on McpToolHandler interface
  (lives in core-mcp, not needed since McpToolsRegistering stores class
  name strings)
- Run Laravel Pint to fix PSR-12 violations across all source and test files
- Add missing declare(strict_types=1) to 18 PHP files (tests, seeders,
  Layout.php, GenerateServiceOgImages.php)

Co-Authored-By: Virgil <virgil@lethean.io>

Virgil commented

2026-03-17 09:04:09 +00:00

Author

Member

Verification Failed

Command: go test ./...

go: inconsistent vendoring in /Users/snider/Code/host-uk/core/.core/workspace/php-1773731850/src:
	forge.lthn.ai/core/cli@v0.3.5: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	forge.lthn.ai/core/go-i18n@v0.1.5: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	forge.lthn.ai/core/go-io@v0.1.5: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	github.com/dunglas/frankenphp@v1.12.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	github.com/stretchr/testify@v1.11.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	gopkg.in/yaml.v3@v3.0.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	forge.lthn.ai/core/go@v0.3.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	forge.lthn.ai/core/go-inference@v0.1.4: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	forge.lthn.ai/core/go-log@v0.0.4: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	github.com/MauriceGit/skiplist@v0.0.0-20211105230623-77f5c8d3e145: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	github.com/RoaringBitmap/roaring/v2@v2.15.0: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	github.com/aymanbagabas/go-osc52/v2@v2.0.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	github.com/beorn7/perks@v1.0.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	github.com/bits-and-blooms/bitset@v1.24.4: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	github.com/cespare/xxhash/v2@v2.3.0: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	github.com/charmbracelet/bubbletea@v1.3.10: is explicitly required in...

Exit code: 1

PR will not be auto-merged.

## Verification Failed **Command:** `go test ./...` ``` go: inconsistent vendoring in /Users/snider/Code/host-uk/core/.core/workspace/php-1773731850/src: forge.lthn.ai/core/cli@v0.3.5: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt forge.lthn.ai/core/go-i18n@v0.1.5: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt forge.lthn.ai/core/go-io@v0.1.5: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/dunglas/frankenphp@v1.12.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/stretchr/testify@v1.11.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt gopkg.in/yaml.v3@v3.0.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt forge.lthn.ai/core/go@v0.3.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt forge.lthn.ai/core/go-inference@v0.1.4: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt forge.lthn.ai/core/go-log@v0.0.4: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/MauriceGit/skiplist@v0.0.0-20211105230623-77f5c8d3e145: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/RoaringBitmap/roaring/v2@v2.15.0: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/aymanbagabas/go-osc52/v2@v2.0.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/beorn7/perks@v1.0.1: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/bits-and-blooms/bitset@v1.24.4: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/cespare/xxhash/v2@v2.3.0: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt github.com/charmbracelet/bubbletea@v1.3.10: is explicitly required in... ``` **Exit code:** 1 PR will not be auto-merged.