## Summary - harden nested Filament and Livewire tenant-context handling across the backup schedule operation runs relation manager, managed-environment triage arrival continuity, the backup set policy picker table, and the Operate Hub shell - add architecture, feature, and browser coverage for nested Filament tenant-context continuity and restore-run resource behavior - add the Spec 334 artifacts (`spec.md`, `plan.md`, `tasks.md`, and the requirements checklist) ## Testing - Not run as part of this commit/push/PR workflow Co-authored-by: Ahmed Darrazi <ahmed.darrazi@live.de> Reviewed-on: #395
2.0 KiB
2.0 KiB
Spec 334 Requirements Checklist
- Purpose: Preparation-quality validation for Spec 334 before runtime implementation.
- Created: 2026-05-24
- Feature: spec.md
Content Quality
- Spec is focused on a concrete operator-visible defect class (nested context loss) and does not drift into a tenancy rewrite.
- Spec Candidate Check is filled and justifies why this is Core Enterprise work (correctness + blocker removal).
- Non-goals explicitly exclude routing/panel/RBAC rewrites and broad repo sweeps.
- Acceptance criteria are testable and bounded to confirmed surfaces.
- Preparation artifacts contain no runtime implementation code changes.
Repo Truth
- Confirmed scope file paths were verified as existing:
apps/platform/app/Livewire/BackupSetPolicyPickerTable.phpapps/platform/app/Filament/Resources/RestoreRunResource.phpapps/platform/app/Filament/Concerns/ResolvesPanelTenantContext.phpapps/platform/app/Filament/Resources/BackupScheduleResource/RelationManagers/BackupScheduleOperationRunsRelationManager.phpapps/platform/app/Filament/Widgets/ManagedEnvironment/ManagedEnvironmentTriageArrivalContinuity.php
- Repo-truth deviation is documented: user draft references Spec 332 WIP, but
platform-devhas nospecs/332-*directory today.
Security / Isolation / RBAC
- Contract explicitly treats UI visibility as non-authorization and requires mutation-time rechecks.
- Fail-closed posture is explicit: no broadening, no silent tenant override, no cross-scope selection/attachment.
- Unsafe model-derived
Filament::setTenant(...)switching is explicitly guarded.
Test And Smoke Readiness
- Plan includes the intended lane mix (Feature/Livewire + architecture guard; browser smoke only where user-visible bugs exist).
- Tasks include a guardrail-first step and explicit validation commands.
- Browser smoke scenarios are explicitly listed for the two originally user-visible bugs.