Applied the decision-first diagnostic surface IA contract to EnvironmentDiagnostics and SupportDiagnostics bundles. Added recommended_first_check and separated technical metadata as per Spec 373. Co-authored-by: Ahmed Darrazi <ahmed.darrazi@live.de> Reviewed-on: #444
2.8 KiB
2.8 KiB
Affected Files
Status: complete after implementation, targeted tests, browser smoke, and artifact review.
| File | Touch | Risk | Verification |
|---|---|---|---|
apps/platform/app/Filament/Pages/EnvironmentDiagnostics.php |
Added page-local diagnosticSummary() derived from existing public page state. |
low; action safety had to remain unchanged | TenantDiagnosticsRepairsTest, source review |
apps/platform/resources/views/filament/pages/environment-diagnostics.blade.php |
Replaced flat cards with one leading summary plus lower blocker detail sections. | medium UI hierarchy risk | TenantDiagnosticsRepairsTest, browser screenshot 001-environment-diagnostics-after.png |
apps/platform/app/Support/SupportDiagnostics/SupportDiagnosticBundleBuilder.php |
Added derived recommended_first_check using existing sections/references. |
low; no persistence or provider calls | SupportDiagnostics filter, builder unit test |
apps/platform/resources/views/filament/modals/support-diagnostic-bundle.blade.php |
Rendered recommended first check before lower support sections. | medium modal hierarchy/redaction risk | Support diagnostics modal tests, browser screenshot 002-support-diagnostics-after-or-blocked.png |
apps/platform/tests/Feature/Filament/TenantDiagnosticsRepairsTest.php |
Added no-action, duplicate, both-blocker, confirmation, and fail-hard Graph assertions. | low test-only | focused file run passed |
apps/platform/tests/Feature/SupportDiagnostics/TenantSupportDiagnosticActionTest.php |
Added modal hierarchy and fail-hard Graph assertions. | low test-only | SupportDiagnostics filter passed |
apps/platform/tests/Feature/SupportDiagnostics/OperationRunSupportDiagnosticActionTest.php |
Added OperationRun modal first-check assertions and fail-hard Graph guard. | low test-only | SupportDiagnostics filter passed |
apps/platform/tests/Unit/Support/SupportDiagnostics/SupportDiagnosticBundleBuilderTest.php |
Added fallback and untranslated provider reason recommended-first-check coverage. | low test-only | focused unit test passed |
docs/ui-ux-enterprise-audit/page-reports/ui-012-environment-diagnostics.md |
Created dedicated UI-012 Environment Diagnostics report. | low docs/artifact | manual review, git diff --check |
docs/ui-ux-enterprise-audit/route-inventory.md |
Linked UI-012 to the new dedicated report. | low docs/artifact | git diff --check |
specs/373-diagnostic-surface-separation/artifacts/* |
Updated implementation, browser, safety, validation, and screenshot evidence. | low | artifact review |
Out Of Scope Side Effects
- No migrations.
- No packages.
- No env vars.
- No queues, scheduler, or storage changes.
- No Graph/provider HTTP calls during render.
- No panel provider registration changes.
- No global search changes.
- No Provider Connections or Required Permissions reimplementation.