feat: improve onboarding UX with auto-skip workflow review & persistent cloud LoRAs#766
feat: improve onboarding UX with auto-skip workflow review & persistent cloud LoRAs#766
Conversation
…w review dialog Cloud LoRA persistence: - Add shared persistent LoRA directory (/data/models/lora) for cloud mode - Cache sample/onboarding LoRAs to shared storage during session cleanup - Resolve and list endpoints check both session and shared LoRA dirs - Block deletion of sample LoRAs in cloud mode (403) - No-op locally (DAYDREAM_SCOPE_LORA_SHARED_DIR unset) Auto-skip review dialog: - Skip the workflow review dialog when all dependencies are resolved and there are no warnings, loading the workflow directly - Applies to both WorkflowImportDialog (onboarding + file import) and GraphWorkflowImportDialog (graph editor file import) Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]> Signed-off-by: Rafal Leszko <[email protected]>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🚀 fal.ai Preview Deployment
Livepeer Runner
TestingConnect to this preview deployment by running this on your branch: Livepeer mode: 🧪 E2E tests will run automatically against this deployment. |
✅ E2E Tests passed
Test ArtifactsCheck the workflow run for screenshots. |
Close the dialog before the async load begins so the review content doesn't flash back after the confirmation dialog dismisses. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]> Signed-off-by: Rafal Leszko <[email protected]>
Add read_only flag to LoRAFileInfo. LoRAs served from the shared persistent directory are marked read_only, and the delete button is hidden for them in the settings UI. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]> Signed-off-by: Rafal Leszko <[email protected]>
When a sample LoRA exists in both the session dir and the shared dir, it was listed from the session dir without read_only. Now session-dir LoRAs are also marked read_only if they exist in the shared dir. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]> Signed-off-by: Rafal Leszko <[email protected]>
…ated Sample LoRAs are already stored in the shared persistent directory (/data/models/lora) on the cloud volume, so there's no need to copy them from the session directory during cleanup. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]> Signed-off-by: Rafal Leszko <[email protected]>
Summary
Improves the onboarding experience with two changes:
/data/models/lora) so returning users don't re-download them each session. These LoRAs are marked as read-only and cannot be deleted.Changes
Frontend
WorkflowImportDialog,useGraphPersistence)read_onlyLoRAs in settingsread_onlyfield toLoRAFileInfoAPI typeBackend
DAYDREAM_SCOPE_LORA_SHARED_DIRenv var andget_shared_lora_dir()helperread_onlyTest plan