PHP 8.4 autofixes (nullsafe logger, container share API, path … #62

ksandell · 2025-08-19T17:07:28Z

…separators, typed returns, route methods fix, helpers env unquote)

…ss PHP 8.18.4

…docs across core files

src/Application.php

src/Core/Controller.php

src/Core/HookManager.php

src/Core/RouteGroup.php

src/Factories/Http/ServerRequestFactory.php

@var

- Added proper type declarations to all class properties across the codebase - Updated property docblocks with proper @var annotations - Fixed property types in Core, Cache, Database, Helpers, Factories, and Classes - Improved code quality and IDE support with strict typing - All properties now have explicit types: string, int, bool, array, etc. - Enhanced type safety and code documentation

- Completely rewrote Configuration.md with JSON-based config structure - Completely rewrote Routing.md with SPIN-specific routing patterns - Completely rewrote Middleware.md with actual SPIN middleware structure - Completely rewrote Security.md with SPIN security patterns - All docs now based on spin-skeleton examples instead of generic patterns

- Replace outdated PHP array examples with correct JSON configuration - Update project structure to match spin-skeleton layout (src/app, src/public) - Fix routing examples to show actual SPIN JSON route configuration - Update controller examples to use SPIN's handleGET() pattern - Fix middleware examples to use SPIN's initialize() and handle() methods - Update web server configuration paths to use src/public - Add missing security features (JWT, Rate Limiting) - Reorganize documentation links to match actual available docs

- Fix PSR-7 compatibility issues in helper functions - Replace Laravel-style methods with proper PSR-7 Request methods - Fix queryParam, postParam, and cookieParam functions - Fix MIME content type function return types - Add comprehensive test suite for all global helper functions - Test coverage for env, app, config, container, logger functions - Test coverage for request parameter functions (query, post, cookie) - Test coverage for response functions (redirect, response, responseJson, etc.) - Test coverage for utility functions (getClientIp, generateRefId, etc.) - Resolve conflicts with built-in PHP mime_content_type function

- Add missing test configuration files (config-unittest.json, config-dev.json) - Set test environment to 'unittest' in bootstrap.php - Fix cache configuration to prevent 'Cache class does not exist' errors - Fix generateRefId test timing issue - Ensure proper test environment setup for GitHub Actions

- Fix withHeader() method calls to handle both array and string values - Ensure proper type casting for header values to prevent PSR-7 v2 compatibility issues - Fix applies to redirect(), response(), and other response helper functions - Resolves type mismatch errors with GuzzleHttp 7.x and PSR-7 v2

ksandell added 3 commits August 19, 2025 19:05

chore: PHP 8.4 autofixes (nullsafe logger, container share API, path …

c19571c

…separators, typed returns, route methods fix, helpers env unquote)

test: skip Redis tests when Redis is not available; stabilize CI acro…

853c5cd

…ss PHP 8.18.4

docs: remove duplicate docblocks; standardize @inheritdoc and method …

51a2fdb

…docs across core files

ksandell marked this pull request as draft August 20, 2025 11:52

ksandell and others added 2 commits August 20, 2025 18:20

docblock additions/corrections

ad9f15f

Merge remote-tracking branch 'origin/develop' into cursor-autofixes

1fa2530