refactor: add declare(strict_types=1) to all PHP files #31

Open

Charon wants to merge 2 commits from feat/strict-types-all into dev

Charon commented

2026-03-24 16:38:31 +00:00

Member

Fixes #5

Summary

Add declare(strict_types=1) to 9 remaining scaffold/config PHP files missed in the earlier DX audit (bcb40d4)
All non-blade PHP files in the repository now enforce strict typing per CLAUDE.md conventions

Files changed

app/Providers/AppServiceProvider.php
bootstrap/app.php
bootstrap/providers.php
config/core.php
database/seeders/DatabaseSeeder.php
public/index.php
routes/api.php
routes/console.php
routes/web.php

Fixes #5 ## Summary - Add `declare(strict_types=1)` to 9 remaining scaffold/config PHP files missed in the earlier DX audit (bcb40d4) - All non-blade PHP files in the repository now enforce strict typing per CLAUDE.md conventions ## Files changed - `app/Providers/AppServiceProvider.php` - `bootstrap/app.php` - `bootstrap/providers.php` - `config/core.php` - `database/seeders/DatabaseSeeder.php` - `public/index.php` - `routes/api.php` - `routes/console.php` - `routes/web.php`

Charon added 2 commits 2026-03-24 16:38:31 +00:00

fix: add error handling to SearchProviderRegistry search method 9f7e5783ba

Wrap individual provider search calls in try-catch so a single
provider failure no longer breaks the entire search. Failures are
logged as warnings and the remaining providers continue normally.

Fixes #11

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

refactor: add declare(strict_types=1) to remaining PHP files ddfb71d767

Add strict_types declaration to 9 scaffold/config files that were
missed in the earlier dx-audit pass (bcb40d4). All non-blade PHP
files in the repository now enforce strict typing per CLAUDE.md
conventions.

Files: bootstrap/app.php, bootstrap/providers.php, config/core.php,
database/seeders/DatabaseSeeder.php, public/index.php, routes/*.php,
app/Providers/AppServiceProvider.php

Fixes #5

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin feat/strict-types-all:feat/strict-types-all

git checkout feat/strict-types-all

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout dev

git merge --no-ff feat/strict-types-all

git checkout feat/strict-types-all

git rebase dev

git checkout dev

git merge --ff-only feat/strict-types-all

git checkout feat/strict-types-all

git rebase dev

git checkout dev

git merge --no-ff feat/strict-types-all

git checkout dev

git merge --squash feat/strict-types-all