Fix four silent validation failures, add PHP 8.5 support and an optional Laravel provider

.github/workflows/dependency-review.yml

@@ -15,6 +15,6 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: 'Checkout Repository'
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
       - name: 'Dependency Review'
         uses: actions/dependency-review-action@v2

.github/workflows/fix-php-code-style-issues.yml

@@ -8,14 +8,14 @@ jobs:
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           ref: ${{ github.head_ref }}
 
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '8.4'
+          php-version: '8.5'
           tools: composer
 
       - name: Install dependencies
@@ -27,4 +27,4 @@ jobs:
       - name: Commit changes
         uses: stefanzweifel/git-auto-commit-action@v4
         with:
-          commit_message: Fix styling to adhere to PSR12 standards.
+          commit_message: Fix styling to adhere to PSR12 standards.

.github/workflows/phpunit.yml

@@ -7,15 +7,19 @@ on:
   pull_request:
 
 jobs:
+  # The default dependency set: laravel/framework ^12 || ^13 is installed, so
+  # the provider's boot() is exercised against a real Illuminate application.
   test:
+    name: PHP ${{ matrix.php-version }} - locked deps
     runs-on: ubuntu-latest
     strategy:
+      fail-fast: false
       matrix:
-        php-version: ['8.3', '8.4']
+        php-version: ['8.3', '8.4', '8.5']
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
 
       - name: Set up PHP
         uses: shivammathur/setup-php@v2
@@ -25,7 +29,71 @@ jobs:
           coverage: xdebug
 
       - name: Install dependencies
-        run: composer install --prefer-dist --no-progress --no-suggest
+        run: composer install --prefer-dist --no-progress --no-interaction
 
       - name: Run PHPUnit
-        run: vendor/bin/phpunit --testdox
+        run: vendor/bin/phpunit --testdox
+
+  # Every Laravel major the package claims to support, resolved from scratch.
+  #
+  # This job answers "does the suite pass against this dependency set", so it
+  # runs without a coverage driver. phpunit.xml sets failOnWarning, and PHPUnit
+  # warns when a coverage driver is missing, so --no-coverage is required here
+  # rather than optional -- without it the job fails on the warning alone while
+  # every test passes.
+  #
+  # Laravel 10 and 11 use the split illuminate/* packages deliberately: every
+  # laravel/framework release in those ranges is withheld by Composer's
+  # security-advisory policy, and the alternative would be disabling that check.
+  # The advisories do not apply to the split packages, so those majors are still
+  # exercised here -- they just cannot supply a real Application, and the
+  # boot() tests skip themselves accordingly.
+  laravel-majors:
+    name: Laravel ${{ matrix.laravel }} - PHP ${{ matrix.php-version }}
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        php-version: ['8.3', '8.5']
+        laravel: ['10', '11', '12', '13']
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Set up PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php-version }}
+          extensions: zip, intl
+          coverage: none
+
+      - name: Pin the Laravel major
+        run: |
+          # laravel/framework replaces the illuminate/* split packages, so the
+          # two cannot both be constrained at once. Drop it for the majors whose
+          # framework releases are advisory-blocked, and pin it for the rest.
+          if [ "${{ matrix.laravel }}" -le 11 ]; then
+            composer remove --dev laravel/framework --no-update --no-interaction
+          else
+            composer require --dev "laravel/framework:^${{ matrix.laravel }}.0" \
+              --no-update --no-interaction
+          fi
+
+          for pkg in support validation filesystem translation config; do
+            composer require --dev "illuminate/${pkg}:^${{ matrix.laravel }}.0" \
+              --no-update --no-interaction
+          done
+
+      - name: Resolve dependencies
+        run: |
+          composer update --with-all-dependencies \
+            --prefer-dist --no-progress --no-interaction
+
+      - name: Show what actually resolved
+        # illuminate/support is not separately installed when laravel/framework
+        # is present -- the framework replaces it -- so list whichever is there.
+        run: composer show | grep -E '^(laravel/framework|illuminate/support) ' || true
+
+      - name: Run PHPUnit
+        run: vendor/bin/phpunit --testdox --no-coverage

.gitignore

@@ -5,3 +5,4 @@ clover.xml
 /data-validation
 *.cache
 .phpunit.result.cache
+docs/

Dockerfile

@@ -1,10 +1,13 @@
-FROM php:8.4-fpm
+ARG PHP_VERSION=8.5
+FROM php:${PHP_VERSION}-fpm
 
 RUN apt-get update \
     && apt-get -y install libzip-dev zlib1g-dev git zip unzip libicu-dev \
-    && apt-get clean; rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/* /usr/share/doc/* \
+    && apt-get clean \
+    && rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/* /usr/share/doc/*
 
-RUN docker-php-ext-configure zip && docker-php-ext-install zip intl
+RUN docker-php-ext-configure zip \
+    && docker-php-ext-install zip intl
 
 # Install Xdebug only
 RUN pecl install xdebug \
@@ -24,4 +27,4 @@ RUN chown -R www-data:www-data /var/www
 
 # Expose port 9000 and start php-fpm server
 EXPOSE 9000
-CMD ["php-fpm"]
+CMD ["php-fpm"]

README.md

@@ -33,7 +33,24 @@ composer require yorcreative/data-validation
 ```
 
 **Requirements**:
-- PHP 8.3 or 8.4
+- PHP 8.3, 8.4, or 8.5
+
+### Optional Laravel Integration
+
+The core library has **zero runtime framework dependencies**. When the package is installed inside a Laravel 10, 11, 12, or 13 application, an optional service provider (`YorCreative\DataValidation\Laravel\DataValidationServiceProvider`) is auto-discovered and binds:
+
+- A shared `DataValidationConfig` singleton, hydrated from `config/data-validation.php`
+- A `yorcreative.data-validation` factory closure that returns a `Validator` pre-configured from the container
+
+Publish the config file to customize cache limits and chunk size:
+
+```bash
+php artisan vendor:publish --tag=data-validation-config
+```
+
+Set `chunk_size` to `null` if you want the validator to fall back to its dynamic wildcard chunking heuristic instead of a fixed override.
+
+Outside of Laravel the provider class is never loaded — the library remains fully framework-agnostic.
 
 ## Basic Usage
 
@@ -191,6 +208,8 @@ if ($validator->fails()) {
 }
 ```
 
+`fails()` and `passes()` trigger validation on first use, so they can be used directly when you do not want to call `validate()` yourself.
+
 ## Extending with Custom Rules
 
 For applications requiring bespoke validation logic, implement custom rules via `ValidationRuleInterface`:
@@ -311,4 +330,4 @@ Report issues or suggest features on the [GitHub repository](https://github.com/
 
 ## License
 
-DataValidation is licensed under the MIT License. See the [LICENSE](https://github.com/yorcreative/data-validation/blob/main/LICENSE) file for details.
+DataValidation is licensed under the MIT License. See the [LICENSE](https://github.com/yorcreative/data-validation/blob/main/LICENSE) file for details.

composer.json

@@ -19,26 +19,31 @@
     "psr-12",
     "psr-4",
     "security",
-    "open-source  ",
+    "open-source",
     "no-dependencies",
     "testing",
     "enterprise",
     "validator",
     "benchmarks"
   ],
   "require": {
-    "php": "^8.3|^8.4"
+    "php": "^8.3 || ^8.4 || ^8.5"
   },
   "require-dev": {
-    "orchestra/testbench": "^9.0|^10.0",
     "phpunit/phpunit": "^10.0|^11.5.3",
     "squizlabs/php_codesniffer": "^3.8.0",
     "friendsofphp/php-cs-fixer": "^3.44.2",
     "mockery/mockery": "^1.0",
-    "illuminate/validation": "^10.0 || ^11.0",
-    "symfony/translation": "^6.0 || ^7.0",
-    "illuminate/filesystem": "^10.0 || ^11.0",
-    "illuminate/translation": "^10.0 || ^11.0"
+    "illuminate/support": "^10.0 || ^11.0 || ^12.0 || ^13.0",
+    "illuminate/validation": "^10.0 || ^11.0 || ^12.0 || ^13.0",
+    "symfony/translation": "^6.0 || ^7.0 || ^8.0",
+    "illuminate/filesystem": "^10.0 || ^11.0 || ^12.0 || ^13.0",
+    "illuminate/translation": "^10.0 || ^11.0 || ^12.0 || ^13.0",
+    "illuminate/config": "^10.0 || ^11.0 || ^12.0 || ^13.0",
+    "laravel/framework": "^12.0 || ^13.0"
+  },
+  "suggest": {
+    "illuminate/support": "Required only when using the optional Laravel ServiceProvider (^10.0 || ^11.0 || ^12.0 || ^13.0)."
   },
   "autoload": {
     "psr-4": {
@@ -53,7 +58,7 @@
   },
   "extra": {
     "laravel": {
-      "providers": ["YorCreative\\DataValidation\\DataValidationServiceProvider"]
+      "providers": ["YorCreative\\DataValidation\\Laravel\\DataValidationServiceProvider"]
     }
   },
   "scripts": {
@@ -66,6 +71,9 @@
     "coverage-html": "php ./vendor/bin/phpunit --testdox --coverage-html data-validation"
   },
   "config": {
-    "process-timeout": 0
+    "process-timeout": 0,
+    "platform": {
+      "php": "8.3.0"
+    }
   }
-}
+}