Skip to content

v1.6.58: defer database resolution during package discovery - #241

Merged
roncodes merged 1 commit into
mainfrom
dev-v1.6.58
Aug 17, 2026
Merged

v1.6.58: defer database resolution during package discovery#241
roncodes merged 1 commit into
mainfrom
dev-v1.6.58

Conversation

@roncodes

Copy link
Copy Markdown
Member

Summary

  • resolve UserDeletionService only when fleetbase:user-delete executes
  • prevent Composer/Laravel package discovery from opening MySQL during image builds
  • bump Core API to v1.6.58 and add current release notes

Regression coverage

  • verifies the command resolves while the database-backed service binding throws
  • preserves dry-run, validation, confirmation, execution, and rollback behavior

Validation

  • vendor/bin/pest tests/Unit/Console/DeleteUserCommandTest.php (8 passed, 26 assertions)
  • composer test:unit (1,416 passed, 9,941 assertions)
  • composer test:lint (0 files need changes)
  • composer test:date-drift
  • root API php artisan package:discover --ansi with DB_HOST=127.0.0.1 DB_PORT=1

This fixes the v0.7.54 release image regression where composer dumpautoload failed with a MySQL connection-refused error before database services were available.

@codecov

codecov Bot commented Aug 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (0408548) to head (afe3164).

Additional details and impacted files
@@             Coverage Diff             @@
##                main      #241   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity      6730      6730           
===========================================
  Files            397       397           
  Lines          22447     22448    +1     
===========================================
+ Hits           22447     22448    +1     
Flag Coverage Δ
backend 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@roncodes
roncodes merged commit 941b59e into main Aug 17, 2026
10 checks passed
@roncodes
roncodes deleted the dev-v1.6.58 branch August 17, 2026 09:15
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