ahmido
94877c9a66
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
29 lines
2.8 KiB
Markdown
29 lines
2.8 KiB
Markdown
# 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.
|