Skip to content

deprecate insights module#270

Open
rootflo-hardik wants to merge 1 commit intodevelopfrom
CU-86d2hrmgw-deprecate-insights-module
Open

deprecate insights module#270
rootflo-hardik wants to merge 1 commit intodevelopfrom
CU-86d2hrmgw-deprecate-insights-module

Conversation

@rootflo-hardik
Copy link
Copy Markdown
Contributor

@rootflo-hardik rootflo-hardik commented Apr 4, 2026

Summary by CodeRabbit

Release Notes

  • Removed Features
    • Removed the Insights module, including dynamic query execution API, parsed data object management endpoints (fetch, update, media retrieval), alert generation, and insights signal analysis capabilities.
    • Removed BigQuery and Redshift database connectors; related cloud provider integrations and usage metrics tracking no longer available.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0d391bcd-3586-4772-9822-9baa35ebd7f5

📥 Commits

Reviewing files that changed from the base of the PR and between da41526 and addb00a.

⛔ Files ignored due to path filters (1)
  • wavefront/server/uv.lock is excluded by !**/*.lock
📒 Files selected for processing (24)
  • wavefront/server/apps/floware/floware/di/application_container.py
  • wavefront/server/apps/floware/floware/server.py
  • wavefront/server/apps/floware/pyproject.toml
  • wavefront/server/docker/floware.Dockerfile
  • wavefront/server/modules/insights_module/insights_module/controllers/dynamic_query_controller.py
  • wavefront/server/modules/insights_module/insights_module/controllers/pdo_controller.py
  • wavefront/server/modules/insights_module/insights_module/controllers/router.py
  • wavefront/server/modules/insights_module/insights_module/db/bigquery_connector.py
  • wavefront/server/modules/insights_module/insights_module/db/redshift_connector.py
  • wavefront/server/modules/insights_module/insights_module/insights_container.py
  • wavefront/server/modules/insights_module/insights_module/models/dymanic_query.py
  • wavefront/server/modules/insights_module/insights_module/models/insights_signal.py
  • wavefront/server/modules/insights_module/insights_module/models/insights_signal_query.py
  • wavefront/server/modules/insights_module/insights_module/models/lead_signal_query.py
  • wavefront/server/modules/insights_module/insights_module/models/leads_aggreegate.py
  • wavefront/server/modules/insights_module/insights_module/repository/pvo_repository.py
  • wavefront/server/modules/insights_module/insights_module/service/dynamic_query_service.py
  • wavefront/server/modules/insights_module/insights_module/service/insights_service.py
  • wavefront/server/modules/insights_module/insights_module/service/pdo_service.py
  • wavefront/server/modules/insights_module/insights_module/service/usage_metric_service.py
  • wavefront/server/modules/insights_module/insights_module/utils/helper.py
  • wavefront/server/modules/insights_module/pyproject.toml
  • wavefront/server/modules/insights_module/tests/conftest.py
  • wavefront/server/modules/insights_module/tests/test_pvo_controller.py
💤 Files with no reviewable changes (24)
  • wavefront/server/docker/floware.Dockerfile
  • wavefront/server/apps/floware/pyproject.toml
  • wavefront/server/apps/floware/floware/di/application_container.py
  • wavefront/server/modules/insights_module/pyproject.toml
  • wavefront/server/modules/insights_module/insights_module/controllers/router.py
  • wavefront/server/apps/floware/floware/server.py
  • wavefront/server/modules/insights_module/insights_module/models/leads_aggreegate.py
  • wavefront/server/modules/insights_module/insights_module/utils/helper.py
  • wavefront/server/modules/insights_module/insights_module/insights_container.py
  • wavefront/server/modules/insights_module/tests/test_pvo_controller.py
  • wavefront/server/modules/insights_module/insights_module/models/lead_signal_query.py
  • wavefront/server/modules/insights_module/insights_module/service/usage_metric_service.py
  • wavefront/server/modules/insights_module/insights_module/controllers/dynamic_query_controller.py
  • wavefront/server/modules/insights_module/insights_module/db/bigquery_connector.py
  • wavefront/server/modules/insights_module/tests/conftest.py
  • wavefront/server/modules/insights_module/insights_module/models/insights_signal_query.py
  • wavefront/server/modules/insights_module/insights_module/service/dynamic_query_service.py
  • wavefront/server/modules/insights_module/insights_module/models/dymanic_query.py
  • wavefront/server/modules/insights_module/insights_module/db/redshift_connector.py
  • wavefront/server/modules/insights_module/insights_module/service/insights_service.py
  • wavefront/server/modules/insights_module/insights_module/controllers/pdo_controller.py
  • wavefront/server/modules/insights_module/insights_module/models/insights_signal.py
  • wavefront/server/modules/insights_module/insights_module/repository/pvo_repository.py
  • wavefront/server/modules/insights_module/insights_module/service/pdo_service.py

📝 Walkthrough

Walkthrough

This PR removes the entire insights-module from the floware application, including all dependency injection declarations, API controllers, business services, data repository layer, database connectors, data models, testing infrastructure, and project configuration files.

Changes

Cohort / File(s) Summary
Dependency Injection & Application Setup
wavefront/server/apps/floware/floware/di/application_container.py, wavefront/server/apps/floware/floware/server.py, wavefront/server/modules/insights_module/insights_module/insights_container.py
Removed insights_service and pvo_repository DI provider declarations; eliminated InsightsContainer instantiation and insights router wiring from server initialization; deleted entire DI container configuration for insights module including cloud provider selection logic.
API Controllers & Routing
wavefront/server/modules/insights_module/insights_module/controllers/dynamic_query_controller.py, wavefront/server/modules/insights_module/insights_module/controllers/pdo_controller.py, wavefront/server/modules/insights_module/insights_module/controllers/router.py
Removed three controller modules exposing endpoints for dynamic query execution, parsed data object management, and audio/transcript retrieval; eliminated router composition that registered these controllers under /v1/insights prefix.
Business Logic Services
wavefront/server/modules/insights_module/insights_module/service/dynamic_query_service.py, wavefront/server/modules/insights_module/insights_module/service/insights_service.py, wavefront/server/modules/insights_module/insights_module/service/pdo_service.py, wavefront/server/modules/insights_module/insights_module/service/usage_metric_service.py
Deleted four service classes handling dynamic query execution, alert/insights extraction, parsed data object operations, and usage metrics collection.
Data Access Layer
wavefront/server/modules/insights_module/insights_module/repository/pvo_repository.py
Removed repository class providing YAML-based query loading, SQL query execution, data caching, and usage metric aggregation against Redshift data store.
Data Models
wavefront/server/modules/insights_module/insights_module/models/dymanic_query.py, wavefront/server/modules/insights_module/insights_module/models/insights_signal.py, wavefront/server/modules/insights_module/insights_module/models/insights_signal_query.py, wavefront/server/modules/insights_module/insights_module/models/lead_signal_query.py, wavefront/server/modules/insights_module/insights_module/models/leads_aggreegate.py
Deleted dataclass definitions for dynamic queries, alerts, insights, signal queries, and lead aggregations; removed YAML parsing/conversion utilities.
Database Connectors
wavefront/server/modules/insights_module/insights_module/db/bigquery_connector.py, wavefront/server/modules/insights_module/insights_module/db/redshift_connector.py
Removed BigQuery and Redshift database connectors with parameterized query execution, connection pooling, and read-only transaction enforcement.
Utilities & Helpers
wavefront/server/modules/insights_module/insights_module/utils/helper.py
Deleted OData filter generation utility function.
Testing Infrastructure
wavefront/server/modules/insights_module/tests/conftest.py, wavefront/server/modules/insights_module/tests/test_pvo_controller.py
Removed test configuration fixtures (DI container setup, test database, auth/middleware) and all controller integration tests.
Project & Deployment Configuration
wavefront/server/apps/floware/pyproject.toml, wavefront/server/modules/insights_module/pyproject.toml, wavefront/server/docker/floware.Dockerfile
Removed insights-module from floware dependencies and workspace sources; deleted insights-module pyproject.toml with all runtime/dev dependencies and build configuration; removed insights-module directory copy from Docker build process.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~65 minutes

Possibly related PRs

Poem

🐰 A module once bustled with queries and alerts,
Dynamic insights and parsed data works,
But now it hops away, removed with great care,
Leaving the codebase cleaner and spare!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'deprecate insights module' directly and clearly summarizes the main change—the comprehensive removal/deprecation of the insights module from the codebase.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch CU-86d2hrmgw-deprecate-insights-module

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.

vishnurk6247
vishnurk6247 previously approved these changes Apr 4, 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