Skip to content

chore: flo-ai upgrade, ui fixes and env changes#186

Merged
vizsatiz merged 7 commits intodevelopfrom
feat/use-latest-flo-ai
Dec 15, 2025
Merged

chore: flo-ai upgrade, ui fixes and env changes#186
vizsatiz merged 7 commits intodevelopfrom
feat/use-latest-flo-ai

Conversation

@vishnurk6247
Copy link
Member

@vishnurk6247 vishnurk6247 commented Dec 15, 2025

Summary by CodeRabbit

  • New Features

    • Added "Coming soon" banner for Model Inference feature
  • UI/UX Updates

    • Updated date display format on application cards
    • Added alpha badge indicator to Model Inference navigation item
    • Refined error message handling across the application
  • Chores

    • Environment configuration and dependency updates

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Dec 15, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request introduces configuration updates for development infrastructure, refactors notification handling across multiple pages, modifies the app deletion API to support conditional deployment deletion, updates the app interface with a deployment type field, and removes user-facing error notifications from several workflows. Additionally, flo-ai dependencies are updated across multiple server modules.

Changes

Cohort / File(s) Summary
Configuration & Infrastructure
.gitignore, docker-compose.dev.yml
Added .env to gitignore; introduced new docker-compose development file with PostgreSQL (pgvector), Redis services, and bridge network configuration.
Client-side Base Configuration
wavefront/client/server.cjs
Updated BASE_URL from /wavefront to /floconsole and added dynamic connectSrc CSP directive based on BASE_URL origin for enhanced security policy.
App Service API
wavefront/client/src/api/app-service.ts, wavefront/client/src/types/app.ts, wavefront/client/src/pages/apps/index.tsx
Enhanced deleteApp method to accept deleteDeployment boolean parameter; added deployment_type field to App interface; updated deletion call to pass conditional deployment deletion flag.
Notification System Cleanup
wavefront/client/src/pages/apps/create.tsx, wavefront/client/src/pages/apps/[appId]/agents/[id].tsx, wavefront/client/src/pages/apps/[appId]/workflows/[id].tsx
Removed user-facing error notifications while preserving console logging; reduced useNotifyStore destructuring to only notifySuccess.
UI Components & Pages
wavefront/client/src/components/AppCard.tsx, wavefront/client/src/pages/apps/layout.tsx, wavefront/client/src/pages/apps/[appId]/model-inference/index.tsx
Updated date display format and fallback logic; added alpha badge to Model Inference navigation item; introduced "Coming soon" alert banner on model inference page.
Server Dependencies
wavefront/server/modules/agents_module/pyproject.toml, wavefront/server/modules/knowledge_base_module/pyproject.toml, wavefront/server/modules/tools_module/pyproject.toml
Updated flo-ai dependency from 1.1.0-rc5 to 1.1.0-rc6 across three server modules.
Plugin Module Error Handling
wavefront/server/modules/plugins_module/plugins_module/controllers/message_processor_controller.py, wavefront/server/modules/plugins_module/plugins_module/services/message_processor_service.py
Simplified error response format in controller; replaced raise_for_status() with explicit status code check and custom error propagation in service.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

  • Attention areas:
    • Verify the conditional deployment deletion logic in index.tsx matches intended behavior
    • Review error handling changes in message processor service to ensure proper exception propagation
    • Confirm CSP directive changes in server.cjs are correct for the new BASE_URL
    • Validate that removing error notifications doesn't impact user experience or debugging workflows

Possibly related PRs

Suggested reviewers

  • vizsatiz

Poem

🐰 A hop through configs and alerts so fine,
Deployments dance when parameters align,
Notifications fade like morning dew,
New badges glow for features coming soon, it's true!
With PostgreSQL and Redis in the bay,
Development blooms in a Docker-y way! 🌱

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/use-latest-flo-ai

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f85a2bf and a353103.

⛔ Files ignored due to path filters (1)
  • wavefront/server/uv.lock is excluded by !**/*.lock
📒 Files selected for processing (17)
  • .gitignore (1 hunks)
  • docker-compose.dev.yml (1 hunks)
  • wavefront/client/server.cjs (3 hunks)
  • wavefront/client/src/api/app-service.ts (1 hunks)
  • wavefront/client/src/components/AppCard.tsx (1 hunks)
  • wavefront/client/src/pages/apps/[appId]/agents/[id].tsx (2 hunks)
  • wavefront/client/src/pages/apps/[appId]/model-inference/index.tsx (2 hunks)
  • wavefront/client/src/pages/apps/[appId]/workflows/[id].tsx (0 hunks)
  • wavefront/client/src/pages/apps/create.tsx (2 hunks)
  • wavefront/client/src/pages/apps/index.tsx (1 hunks)
  • wavefront/client/src/pages/apps/layout.tsx (2 hunks)
  • wavefront/client/src/types/app.ts (1 hunks)
  • wavefront/server/modules/agents_module/pyproject.toml (1 hunks)
  • wavefront/server/modules/knowledge_base_module/pyproject.toml (1 hunks)
  • wavefront/server/modules/plugins_module/plugins_module/controllers/message_processor_controller.py (1 hunks)
  • wavefront/server/modules/plugins_module/plugins_module/services/message_processor_service.py (1 hunks)
  • wavefront/server/modules/tools_module/pyproject.toml (1 hunks)

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@vizsatiz vizsatiz merged commit a6b5ac2 into develop Dec 15, 2025
7 of 8 checks passed
@vizsatiz vizsatiz deleted the feat/use-latest-flo-ai branch December 15, 2025 06:28
@coderabbitai coderabbitai bot mentioned this pull request Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants