Refresh time-balanced shard timings for Core and Browser suites - #134
Conversation
The committed shards.json only covered 28 test classes from PR #102, so every sharded CI run warned it was out of date and fell back to stale balancing. Regenerated via --update-shards across both suites: 204 classes with fresh timings. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HVdCTd166nSKuhBsnKePM9
composer audit reported 19 advisories across 10 packages (laravel/framework, guzzle, several symfony components). All fixes were patch/minor bumps within existing constraints, so this is a lock-only update; composer audit is now clean. Livewire 4.3 raised the default payload.max_components from 20 to 200, which broke the lazy-load isolation test's hardcoded 25-file setup. The test now derives its file count from the config value so it keeps exceeding the limit it exists to exercise. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HVdCTd166nSKuhBsnKePM9
Composer flags nunomaduro/pao as abandoned in favor of laravel/pao. The swap also replaces its transitive shipfastlabs/agent-detector with the maintained laravel/agent-detector, so composer audit no longer reports any abandoned packages. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HVdCTd166nSKuhBsnKePM9
npm audit reported two high-severity advisories in ws (uninitialized memory disclosure, memory-exhaustion DoS). Lock-only update via npm audit fix plus minor in-range bumps; audit is now clean and Vitest suite and CSS build pass. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HVdCTd166nSKuhBsnKePM9
|
Caution Review failedAn error occurred during the review process. Please try again later. 📝 WalkthroughWalkthroughSwapped the ChangesDependency and test maintenance
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Poem A rabbit hopped through composer's lock, Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
The committed shards.json only covered 28 test classes from PR #102,
so every sharded CI run warned it was out of date and fell back to
stale balancing. Regenerated via --update-shards across both suites:
204 classes with fresh timings.
Co-Authored-By: Claude Fable 5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01HVdCTd166nSKuhBsnKePM9
Summary by CodeRabbit