Skip to content

chore: merge CLAUDE.md into .github/copilot-instructions.md#234

Open
herpaderpaldent wants to merge 11 commits into
4.xfrom
chore/merge-instructions
Open

chore: merge CLAUDE.md into .github/copilot-instructions.md#234
herpaderpaldent wants to merge 11 commits into
4.xfrom
chore/merge-instructions

Conversation

@herpaderpaldent
Copy link
Copy Markdown
Contributor

Consolidates two AI instruction files into one canonical location. Enriches copilot-instructions.md with all unique content from CLAUDE.md (version table, skills rules, Laravel Boost MCP tools, Inertia v3 notes, PHP style additions, Testing/Autonomy limits). CLAUDE.md deleted (was untracked).

herpaderpaldent and others added 11 commits April 25, 2026 01:16
…kage

Add return types, parameter types, and property types throughout all
source files in packages/web/src/ — controllers, middleware, resources,
actions, form requests, services, models, jobs, and helpers — without
changing any logic.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…pproval

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
… to ^8.3

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Rewrite bootstrap/app.php with L11 fluent Application builder
- Create bootstrap/providers.php
- Remove obsolete Http/Kernel, Console/Kernel
- Remove empty boilerplate middleware (TrustProxies, VerifyCsrfToken, etc.)
- Remove empty boilerplate providers (Auth, Event, Broadcast, Route)
- Update composer deps: symfony/process ^7.0, collision ^8.1, phpunit ^11, ignition ^3.0
- Add path repositories for seatplus/auth, eveapi, web to use local L11 packages
- Set merge-dev: false to avoid conflicting dev-dep versions across packages
- Require seatplus/web by dev branch (feat/web-type-coverage workspace)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
laravel-vite-plugin is ESM-only; without 'type':'module' in package.json
Node treats .js as CJS and require() of the plugin fails.
Renaming to .mjs forces ESM regardless of package.json type.

Also ignore .agentbridge/, .phpstorm.meta.php, /logs/, and local
package dirs (packages/auth, esi-client, eveapi) in root .gitignore.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ionsUsing

- Upgrade inertiajs/inertia-laravel from ^2.0 to ^3.0
- Upgrade @inertiajs/vue3 from ^1.0 to ^3.0 (JS side)
- Add axios as explicit dependency (no longer bundled with @inertiajs/vue3 v3)
- Remove custom Seatplus\Web\Exception\Handler and ExceptionHandler singleton override
- Register Inertia::handleExceptionsUsing() in WebServiceProvider::boot() using
  the v3 ExceptionResponse API with rootView, withSharedData, and usingMiddleware
- Fix missing trailing commas before setup() in 12 Options API Vue components
  (syntax error exposed by newer Babel parser in @inertiajs/vue3 v3)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
3.x-dev branch caused BindingResolutionException for SolutionProviderRepository.
Stable 2.12.0 fully supports Laravel 11 and properly registers the interface.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Adds the full set of applicable Spatie PHP guidelines as a permanent
## Code Style section so they are followed in all future refactorings.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Herd only activates Xdebug on-demand for HTTP requests. CLI pest
--coverage calls need the extension loaded explicitly via -d zend_extension.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add Root Application Stack version table
- Add AI Skills activation rules
- Add Laravel Boost MCP Tools section (search-docs, db-query, tinker)
- Add Conventions section (check siblings, descriptive names)
- Add Inertia v3 key changes to Web Package section
- Enrich Code Style with PHP-specific rules (TitleCase enums, PHPDoc shapes)
- Enrich Testing Conventions with PHPUnit rules and factory guidance
- Enrich Autonomy Limits (no docs, no deps, no new dirs, deployment note)
- CLAUDE.md deleted (was untracked local file, content now merged)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.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