Skip to content

fix(migrate): preserve shared openapi schemas#5

Merged
dxnter merged 3 commits into
mainfrom
fix/migrate-openapi-scan-path
Apr 27, 2026
Merged

fix(migrate): preserve shared openapi schemas#5
dxnter merged 3 commits into
mainfrom
fix/migrate-openapi-scan-path

Conversation

@dxnter
Copy link
Copy Markdown
Member

@dxnter dxnter commented Apr 27, 2026

Summary

Fixes chassis:migrate so migrated apps keep scanning chassis OpenAPI annotations when regenerating API schemas.

Changes

  • add a migration step that rewrites openapi:generate scripts in composer.json
  • append vendor/northwestern-sysdev/chassis/src/ to swagger-php scan paths when apps currently scan only app/
  • add regression coverage for the OpenAPI script rewrite

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

📊 Test Results

    6 files  ± 0    192 suites  +6   15s ⏱️ -1s
  283 tests + 4    283 ✅ + 4  0 💤 ±0  0 ❌ ±0 
1 698 runs  +24  1 698 ✅ +24  0 💤 ±0  0 ❌ ±0 

Results for commit a506d24. ± Comparison against base commit eb0eda6.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

📊 Code Coverage Report

Metric Base Current Difference
Overall Coverage 🟢 100% 🟢 100% ⚪ 0%
Coverage by File
File Base Coverage New Coverage Difference
Attributes/AutoSeed.php 🟢 100% 🟢 100% ⚪ 0%
Attributes/AutomaticallyOrdered.php 🟢 100% 🟢 100% ⚪ 0%
Attributes/ValidatesConfig.php 🟢 100% 🟢 100% ⚪ 0%
Console/Commands/AutoSeedListCommand.php 🟢 100% 🟢 100% ⚪ 0%
Console/Commands/RestoreLocalEnvironmentFilesCommand.php 🟢 100% 🟢 100% ⚪ 0%
Console/Commands/WakeDatabaseCommand.php 🟢 100% 🟢 100% ⚪ 0%
Database/ValueObjects/SchemaFileCollection.php 🟢 100% 🟢 100% ⚪ 0%
Database/ValueObjects/SchemaSnapshot.php 🟢 100% 🟢 100% ⚪ 0%
Database/ValueObjects/SnapshotListItem.php 🟢 100% 🟢 100% ⚪ 0%
Enums/ApiRequestFailure.php 🟢 100% 🟢 100% ⚪ 0%
Exceptions/ProblemDetailsRenderer.php 🟢 100% 🟢 100% ⚪ 0%
Exceptions/SentryExceptionHandler.php 🟢 100% 🟢 100% ⚪ 0%
Http/Middleware/AuthenticatesAccessTokens.php 🟢 100% 🟢 100% ⚪ 0%
Http/Middleware/EnsureFeatureEnabled.php 🟢 100% 🟢 100% ⚪ 0%
Http/Middleware/EnvironmentLockdown.php 🟢 100% 🟢 100% ⚪ 0%
Http/Middleware/LogsApiRequests.php 🟢 100% 🟢 100% ⚪ 0%
Http/Responses/ProblemDetails.php 🟢 100% 🟢 100% ⚪ 0%
Models/Concerns/Auditable.php 🟢 100% 🟢 100% ⚪ 0%
Models/Concerns/HasAutomaticOrdering.php 🟢 100% 🟢 100% ⚪ 0%
Models/Scopes/AutomaticallyOrderedScope.php 🟢 100% 🟢 100% ⚪ 0%
Rules/ValidIpOrCidrRule.php 🟢 100% 🟢 100% ⚪ 0%
Seeding/Concerns/AuditsSeederChanges.php 🟢 100% 🟢 100% ⚪ 0%
Seeding/Concerns/CleansUpOrphans.php 🟢 100% 🟢 100% ⚪ 0%
Seeding/Concerns/PerformsIdempotentUpserts.php 🟢 100% 🟢 100% ⚪ 0%
Seeding/IdempotentSeeder.php 🟢 100% 🟢 100% ⚪ 0%
Seeding/IdempotentSeederResolver.php 🟢 100% 🟢 100% ⚪ 0%
Seeding/ValueObjects/SeederInfo.php 🟢 100% 🟢 100% ⚪ 0%
Services/ConfigValidatorResolver.php 🟢 100% 🟢 100% ⚪ 0%
Services/DateTimeFormatter.php 🟢 100% 🟢 100% ⚪ 0%

@dxnter dxnter merged commit 7ec9044 into main Apr 27, 2026
11 checks passed
@dxnter dxnter deleted the fix/migrate-openapi-scan-path branch April 27, 2026 17:22
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