Go to file
ahmido 060a82a1ed feat/048-backup-restore-ui-graph-safety (#55)
Feature 048: Backup/Restore UI Graph-Safety (Phase 1)

Dieses PR entfernt Microsoft Graph Calls aus UI-Renderpfaden (Filament/Livewire mount/render/options/typeahead/labels) in den kritischen Backup/Restore Screens und fügt Fail-Hard Guard Tests hinzu, die regressionssicher verhindern, dass UI-Rendering wieder Graph aufruft.

⸻

Motivation

Backup/Restore UI wurde teilweise “fragil”, weil UI-Komponenten (z.B. Group Typeahead/Option Labels) Graph/Entra direkt beim Rendern triggern konnten. Das führt zu:
	•	langsamen/unstabilen Seiten (429/Timeout/Permissions)
	•	schwer reproduzierbaren UI-Fehlern im MSP-Scale
	•	unnötiger Kopplung von “Page render” an Graph-Verfügbarkeit

Ziel: UI muss DB-only rendern; Graph darf nur in Jobs/Run-Execution stattfinden.

⸻

Scope / Changes

1) Restore Wizard: Entfernt Graph-Typeahead & Label-Resolution
	•	Group Mapping ist jetzt DB-only:
	•	manuelle GUID Eingabe / Skip
	•	GUID Validation
	•	Helper Text, wo die Object ID zu finden ist
	•	Keine Graph calls mehr in options() / getOptionLabelUsing() / typeahead beim Rendern.

2) Fail-Hard Guard Tests (Graph-Safety)
	•	Neue Test-Infrastruktur: FailHardGraphClient (GraphClientInterface darf nicht aufgerufen werden)
	•	Guard Tests als Pest Feature Tests (HTTP GET):
	•	Backup Sets Index rendert mit fail-hard Graph client
	•	Restore Wizard Route rendert mit fail-hard Graph client
	•	Assertions:
	•	200 OK
	•	plus stable UI marker string
	•	Masking/Fallback Format ist deterministisch: Unresolved (…<last8>)

3) Spec/Plan/Tasks/Checklist
	•	Spec 048 aktualisiert, Tasks abgehakt
	•	requirements.md Checklist Gate: PASS

⸻

Out of Scope / Non-Goals
	•	Kein Umbau der “Execution”-Actions zu Jobs (Capture snapshot, Restore rerun, Dry-Run execution etc.) → eigener Folge-Spec (Phase 2).
	•	Keine Entra Group Name Resolution (separates Group-Inventory/Cache Feature).
	•	Keine neuen Tabellen/Migrations in Phase 1.

⸻

How to verify (manual)

Mit absichtlich kaputtem Tenant/Auth (Graph failt):
	1.	Öffne Backups & Restore → Backup Sets
 muss laden (UI render DB-only)
	2.	Öffne Restore Runs → Create Restore Run (Wizard)
 muss laden, kein Group-Typeahead mehr
	3.	Starte eine Restore Operation
 darf fehlschlagen (Graph kaputt) – wichtig ist: Render bleibt stabil, Run zeigt Fehler sauber pro Item.

⸻

Tests / Validation

Executed:
	•	./vendor/bin/pint --dirty 
	•	./vendor/bin/sail artisan test tests/Feature/Filament/BackupSetGraphSafetyTest.php tests/Feature/Filament/RestoreWizardGraphSafetyTest.php 
	•	(optional) Combined targeted suite 

⸻

Notes
	•	This PR intentionally focuses on UI Graph-Safety only.
	•	Any future reintroduction of Graph search/typeahead in UI must go through contracts first and be executed asynchronously, never in UI render paths.

Co-authored-by: Ahmed Darrazi <ahmeddarrazi@adsmac.local>
Reviewed-on: #55
2026-01-11 00:14:35 +00:00
.codex/prompts dev-merges/c709b36 (#3) 2025-12-21 23:15:12 +00:00
.gemini feat: merge 001-filament-json 2025-12-14 20:23:18 +01:00
.gitea chore: add PR and issue templates (#1) 2025-12-14 21:39:47 +00:00
.github feat/047-inventory-foundations-nodes (#51) 2026-01-10 20:47:29 +00:00
.specify docs: deprecate .specify specs and update references (#53) 2026-01-10 21:40:46 +00:00
app feat/048-backup-restore-ui-graph-safety (#55) 2026-01-11 00:14:35 +00:00
bootstrap Initial commit from Specify template 2025-12-10 22:27:21 +01:00
config feat/040-inventory-core (#43) 2026-01-07 14:54:24 +00:00
database feat/042-inventory-dependencies-graph (#49) 2026-01-10 00:20:14 +00:00
docs feat/004-assignments-scope-tags (#4) 2025-12-23 21:49:58 +00:00
drizzle feat: TenantPilot v1 - Complete implementation (Phases 1-12) 2025-12-12 02:27:54 +01:00
public feat: merge 001-filament-json 2025-12-14 20:23:18 +01:00
resources feat/047-inventory-foundations-nodes (#51) 2026-01-10 20:47:29 +00:00
routes feat/032-backup-scheduling-mvp (#34) 2026-01-05 04:22:13 +00:00
spechistory feat: TenantPilot v1 - Complete implementation (Phases 1-12) 2025-12-12 02:27:54 +01:00
specs feat/048-backup-restore-ui-graph-safety (#55) 2026-01-11 00:14:35 +00:00
storage feat: TenantPilot v1 - Complete implementation (Phases 1-12) 2025-12-12 02:27:54 +01:00
tests feat/048-backup-restore-ui-graph-safety (#55) 2026-01-11 00:14:35 +00:00
.dockerignore feat: merge 001-filament-json 2025-12-14 20:23:18 +01:00
.editorconfig Initial commit from Specify template 2025-12-10 22:27:21 +01:00
.env.example Initial commit from Specify template 2025-12-10 22:27:21 +01:00
.gitattributes Initial commit from Specify template 2025-12-10 22:27:21 +01:00
.gitignore feat/032-backup-scheduling-mvp (#34) 2026-01-05 04:22:13 +00:00
.npmignore feat: merge 001-filament-json 2025-12-14 20:23:18 +01:00
.prettierignore feat: merge 001-filament-json 2025-12-14 20:23:18 +01:00
Agents.md docs: deprecate .specify specs and update references (#53) 2026-01-10 21:40:46 +00:00
artisan Initial commit from Specify template 2025-12-10 22:27:21 +01:00
boost.json feat: TenantPilot v1 - Complete implementation (Phases 1-12) 2025-12-12 02:27:54 +01:00
composer.json 013-scripts-management (#19) 2026-01-01 22:02:30 +00:00
composer.lock 013-scripts-management (#19) 2026-01-01 22:02:30 +00:00
docker-compose.yml fix/sail-node-modules-volume (#46) 2026-01-08 01:19:54 +00:00
drizzle.config.ts feat: TenantPilot v1 - Complete implementation (Phases 1-12) 2025-12-12 02:27:54 +01:00
GEMINI.md docs: deprecate .specify specs and update references (#53) 2026-01-10 21:40:46 +00:00
package-lock.json dev-merges/c709b36 (#3) 2025-12-21 23:15:12 +00:00
package.json feat: TenantPilot v1 - Complete implementation (Phases 1-12) 2025-12-12 02:27:54 +01:00
phpunit.xml feat/012-windows-update-rings (#18) 2026-01-01 10:44:17 +00:00
PROJECT_SUMMARY.md chore/docs-constitution-v1.1.0 (#40) 2026-01-07 13:43:21 +00:00
README.md feat/032-backup-scheduling-mvp (#36) 2026-01-07 01:12:12 +00:00
vite.config.js dev-merges/c709b36 (#3) 2025-12-21 23:15:12 +00:00

Laravel Logo

Build Status Total Downloads Latest Stable Version License

TenantPilot setup

  • Local dev (Sail-first):
    • Start stack: ./vendor/bin/sail up -d
    • Init DB: ./vendor/bin/sail artisan migrate --seed
    • Tests: ./vendor/bin/sail artisan test
    • Policy sync: ./vendor/bin/sail artisan intune:sync-policies
  • Filament admin: /admin (seed user test@example.com, set password via factory or artisan tinker).
  • Microsoft Graph (Intune) env vars:
    • GRAPH_TENANT_ID
    • GRAPH_CLIENT_ID
    • GRAPH_CLIENT_SECRET
    • GRAPH_SCOPE (default https://graph.microsoft.com/.default)
    • Without these, the NullGraphClient runs in dry mode (no Graph calls).
    • Required API Permissions: See docs/PERMISSIONS.md for complete list
      • Missing permissions? Scope tags will show as "Unknown (ID: X)" - add DeviceManagementRBAC.Read.All
  • Deployment (Dokploy, staging → production):
    • Containerized deploy; ensure Postgres + Redis are provisioned (see docker-compose.yml for local baseline).
    • Run migrations on staging first, validate backup/restore flows, then promote to production.
    • Ensure queue workers are running for jobs (e.g., policy sync) after deploy.
    • Keep secrets/env in Dokploy, never in code.

Bulk operations (Feature 005)

  • Bulk actions are available in Filament resource tables (Policies, Policy Versions, Backup Sets, Restore Runs).
  • Destructive operations require type-to-confirm at higher thresholds (e.g. DELETE).
  • Long-running bulk ops are queued; the bottom-right progress widget polls for active runs.

Troubleshooting

  • Progress stuck on “Queued…” usually means the queue worker is not running (or not processing the queue you expect).
    • Prefer using the Sail/Docker worker (see docker-compose.yml) rather than starting an additional local php artisan queue:work.
    • Check worker status/logs: ./vendor/bin/sail ps and ./vendor/bin/sail logs -f queue.
  • Exit code 137 for queue:work typically means the process was killed (often OOM). Increase Docker memory/limits or run the worker inside the container.

Configuration

  • TENANTPILOT_BULK_CHUNK_SIZE (default 10): job refresh/progress chunk size.
  • TENANTPILOT_BULK_POLL_INTERVAL_SECONDS (default 3): Livewire polling interval for the progress widget (clamped to 110s).

Intune RBAC Onboarding Wizard

  • Entry point: Tenant detail in Filament (Setup Intune RBAC in the ⋯ ActionGroup). Visible only for active tenants with app_client_id.
  • Flow (synchronous, delegated):
    1. Configure Role (default Policy/Profile Manager), Scope (global or scope group), Group mode (create default TenantPilot-Intune-RBAC or pick existing security-enabled group). Review planned changes.
    2. Delegated admin login (short-lived token, not stored in DB/cache).
    3. Execute: resolve service principal, ensure/validate security group, ensure membership, ensure/create/patch Intune role assignment; persists IDs on tenant for idempotency; no queue.
    4. Post-verify: forces fresh token, runs canary reads (deviceConfigurations/deviceCompliancePolicies; CA canary only if feature enabled), updates health and warnings (scope-limited, CA disabled, manual assignment required).
  • Safety/notes: least-privilege default, idempotent reruns, “already exists” treated as success. If service principal missing, run Admin consent first. Scope-limited setups may yield partial inventory/restore; warnings are surfaced in UI and health panel.

Graph Contract Registry & Drift Guard

  • Registry: config/graph_contracts.php defines per-type contracts (resource paths, allowed $select/$expand, @odata.type family, create/update methods, id field, hydration).
  • Client behavior:
    • Sanitizes $select/$expand to allowed fields; logs warnings on trim.
    • Derived @odata.type values within the family are accepted for preview/restore routing.
    • Capability fallback: on 400s related to select/expand, retries without those clauses and surfaces warnings.
  • Drift check: php artisan graph:contract:check [--tenant=] runs lightweight probes against contract endpoints to detect capability/shape issues; useful in staging/CI (prod optional).
  • If Graph returns capability errors, TenantPilot downgrades safely, records warnings/audit entries, and avoids breaking preview/restore flows.

Policy Settings Display

  • Policy detail pages render normalized settings instead of raw JSON:
    • OMA-URI/custom policies → path/value table
    • Settings Catalog → flattened key/value entries
    • Standard objects → labeled key/value view with metadata filtered
  • Version detail pages show both pretty-printed JSON and normalized settings.
  • Warnings surface malformed snapshots or @odata.type mismatches before restore.

Policy JSON Viewer (Feature 002)

  • Location: Policy View pages (/admin/policies/{record})
  • Capability: Pretty-printed JSON snapshot viewer with copy-to-clipboard
  • Settings Catalog Enhancement: Dual-view tabs (Settings table + JSON viewer) for Settings Catalog policies
  • Features:
    • Copy JSON to clipboard with success message
    • Large payload detection (>500 KB) with warning badge and auto-collapse
    • Dark mode support integrated with Filament design system
    • Browser native search (Cmd+F / Ctrl+F) for finding specific keys or values
    • Scrollable container with max height to prevent page overflow
  • Usage: See specs/002-filament-json/quickstart.md for detailed examples and configuration
  • Performance: Optimized for payloads up to 1 MB; auto-collapse improves initial render for large snapshots

About Laravel

Laravel is a web application framework with expressive, elegant syntax. We believe development must be an enjoyable and creative experience to be truly fulfilling. Laravel takes the pain out of development by easing common tasks used in many web projects, such as:

Laravel is accessible, powerful, and provides tools required for large, robust applications.

Learning Laravel

Laravel has the most extensive and thorough documentation and video tutorial library of all modern web application frameworks, making it a breeze to get started with the framework. You can also check out Laravel Learn, where you will be guided through building a modern Laravel application.

If you don't feel like reading, Laracasts can help. Laracasts contains thousands of video tutorials on a range of topics including Laravel, modern PHP, unit testing, and JavaScript. Boost your skills by digging into our comprehensive video library.

Laravel Sponsors

We would like to extend our thanks to the following sponsors for funding Laravel development. If you are interested in becoming a sponsor, please visit the Laravel Partners program.

Premium Partners

Contributing

Thank you for considering contributing to the Laravel framework! The contribution guide can be found in the Laravel documentation.

Code of Conduct

In order to ensure that the Laravel community is welcoming to all, please review and abide by the Code of Conduct.

Security Vulnerabilities

If you discover a security vulnerability within Laravel, please send an e-mail to Taylor Otwell via taylor@laravel.com. All security vulnerabilities will be promptly addressed.

License

The Laravel framework is open-sourced software licensed under the MIT license.