## Summary
- centralize tenant operability into a lane-aware, actor-aware policy boundary
- align selector eligibility, administrative discoverability, remembered context, tenant-bound routes, and canonical run viewers
- add focused Pest coverage plus Spec 148 artifacts and final polish task completion
## Validation
- `vendor/bin/sail artisan test --compact tests/Unit/Tenants/TenantOperabilityServiceTest.php tests/Unit/Tenants/TenantOperabilityOutcomeTest.php tests/Feature/Workspaces/ChooseTenantPageTest.php tests/Feature/Workspaces/SelectTenantControllerTest.php tests/Feature/TenantRBAC/ArchivedTenantRouteAccessTest.php tests/Feature/TenantRBAC/TenantRouteDenyAsNotFoundTest.php tests/Feature/Operations/TenantlessOperationRunViewerTest.php tests/Feature/OpsUx/OperateHubShellTest.php tests/Feature/Rbac/TenantLifecycleActionVisibilityTest.php tests/Feature/TenantRBAC/TenantSwitcherScopeTest.php tests/Feature/Rbac/TenantResourceAuthorizationTest.php tests/Feature/Filament/ManagedTenantsLandingLifecycleTest.php tests/Feature/Filament/TenantGlobalSearchLifecycleScopeTest.php tests/Feature/Onboarding/OnboardingDraftLifecycleTest.php tests/Feature/Onboarding/OnboardingDraftAuthorizationTest.php`
- `vendor/bin/sail bin pint --dirty --format agent`
- manual browser smoke checks on `/admin/choose-tenant`, `/admin/tenants`, `/admin/onboarding`, `/admin/onboarding/{draft}`, and `/admin/operations/{run}`
## Filament / platform notes
- Livewire v4 compliance preserved
- panel provider registration unchanged in `bootstrap/providers.php`
- Tenant resource global search remains backed by existing view/edit pages and is now separated from active-only selector eligibility
- destructive actions remain action closures with confirmation and authorization enforcement
- no asset pipeline changes and no new `filament:assets` deployment requirement
Co-authored-by: Ahmed Darrazi <ahmed.darrazi@live.de>
Reviewed-on: #177
140 lines
4.5 KiB
PHP
140 lines
4.5 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace App\Policies;
|
|
|
|
use App\Models\Tenant;
|
|
use App\Models\TenantOnboardingSession;
|
|
use App\Models\User;
|
|
use App\Models\Workspace;
|
|
use App\Services\Auth\WorkspaceCapabilityResolver;
|
|
use App\Services\Tenants\TenantOperabilityService;
|
|
use App\Support\Auth\Capabilities;
|
|
use App\Support\Tenants\TenantInteractionLane;
|
|
use App\Support\Tenants\TenantOperabilityQuestion;
|
|
use App\Support\Workspaces\WorkspaceContext;
|
|
use Illuminate\Auth\Access\Response;
|
|
use Illuminate\Support\Facades\Gate;
|
|
|
|
class TenantOnboardingSessionPolicy
|
|
{
|
|
public function viewAny(User $user): bool|Response
|
|
{
|
|
$workspace = $this->currentWorkspace($user);
|
|
|
|
if (! $workspace instanceof Workspace) {
|
|
return Response::denyAsNotFound();
|
|
}
|
|
|
|
return $this->authorizeForWorkspace($user, $workspace, Capabilities::WORKSPACE_MANAGED_TENANT_ONBOARD);
|
|
}
|
|
|
|
public function view(User $user, TenantOnboardingSession $tenantOnboardingSession): bool|Response
|
|
{
|
|
return $this->authorizeForDraft(
|
|
user: $user,
|
|
tenantOnboardingSession: $tenantOnboardingSession,
|
|
capability: Capabilities::WORKSPACE_MANAGED_TENANT_ONBOARD,
|
|
);
|
|
}
|
|
|
|
public function update(User $user, TenantOnboardingSession $tenantOnboardingSession): bool|Response
|
|
{
|
|
return $this->authorizeForDraft(
|
|
user: $user,
|
|
tenantOnboardingSession: $tenantOnboardingSession,
|
|
capability: Capabilities::WORKSPACE_MANAGED_TENANT_ONBOARD,
|
|
);
|
|
}
|
|
|
|
public function cancel(User $user, TenantOnboardingSession $tenantOnboardingSession): bool|Response
|
|
{
|
|
return $this->authorizeForDraft(
|
|
user: $user,
|
|
tenantOnboardingSession: $tenantOnboardingSession,
|
|
capability: Capabilities::WORKSPACE_MANAGED_TENANT_ONBOARD_CANCEL,
|
|
);
|
|
}
|
|
|
|
private function currentWorkspace(User $user): ?Workspace
|
|
{
|
|
$workspaceId = app(WorkspaceContext::class)->currentWorkspaceId(request());
|
|
|
|
if (! is_int($workspaceId)) {
|
|
return null;
|
|
}
|
|
|
|
$workspace = Workspace::query()->whereKey($workspaceId)->first();
|
|
|
|
if (! $workspace instanceof Workspace) {
|
|
return null;
|
|
}
|
|
|
|
/** @var WorkspaceCapabilityResolver $resolver */
|
|
$resolver = app(WorkspaceCapabilityResolver::class);
|
|
|
|
if (! $resolver->isMember($user, $workspace)) {
|
|
return null;
|
|
}
|
|
|
|
return $workspace;
|
|
}
|
|
|
|
private function authorizeForDraft(
|
|
User $user,
|
|
TenantOnboardingSession $tenantOnboardingSession,
|
|
string $capability,
|
|
): bool|Response {
|
|
$workspace = $this->currentWorkspace($user);
|
|
|
|
if (! $workspace instanceof Workspace) {
|
|
return Response::denyAsNotFound();
|
|
}
|
|
|
|
if ((int) $tenantOnboardingSession->workspace_id !== (int) $workspace->getKey()) {
|
|
return Response::denyAsNotFound();
|
|
}
|
|
|
|
$tenant = $tenantOnboardingSession->tenant;
|
|
|
|
if ($tenant instanceof Tenant) {
|
|
$viewability = app(TenantOperabilityService::class)->outcomeFor(
|
|
tenant: $tenant,
|
|
question: TenantOperabilityQuestion::TenantBoundViewability,
|
|
actor: $user,
|
|
workspaceId: (int) $workspace->getKey(),
|
|
lane: TenantInteractionLane::AdministrativeManagement,
|
|
);
|
|
|
|
if (! $viewability->allowed) {
|
|
return Response::denyAsNotFound();
|
|
}
|
|
}
|
|
|
|
return $this->authorizeForWorkspace($user, $workspace, $capability);
|
|
}
|
|
|
|
private function authorizeForWorkspace(User $user, Workspace $workspace, string $capability): bool|Response
|
|
{
|
|
/** @var WorkspaceCapabilityResolver $resolver */
|
|
$resolver = app(WorkspaceCapabilityResolver::class);
|
|
|
|
if (! $resolver->isMember($user, $workspace)) {
|
|
return Response::denyAsNotFound();
|
|
}
|
|
|
|
return Gate::forUser($user)->allows($capability, $workspace)
|
|
? Response::allow()
|
|
: Response::deny($this->forbiddenCapabilityMessage($capability));
|
|
}
|
|
|
|
private function forbiddenCapabilityMessage(string $capability): string
|
|
{
|
|
return match ($capability) {
|
|
Capabilities::WORKSPACE_MANAGED_TENANT_ONBOARD_CANCEL => 'You do not have permission to cancel this onboarding draft.',
|
|
default => 'You do not have permission to continue this onboarding draft.',
|
|
};
|
|
}
|
|
}
|