Skip to content

Conversation

akshaydeo
Copy link
Contributor

@akshaydeo akshaydeo commented Sep 20, 2025

Summary

Added a new Observability section to the UI with support for OpenTelemetry, Maxim, Datadog, and New Relic integrations. Also refactored HTTP middleware handling to improve type safety and error handling.

Changes

  • Added new Observability page with dedicated views for different monitoring platforms
  • Created form components for configuring OTEL and Maxim integrations
  • Refactored HTTP middleware to use a dedicated BifrostHTTPMiddleware type
  • Enhanced middleware chain to properly handle early response termination
  • Fixed duplicate cleanup path in Makefile
  • Added enterprise fallback components for governance and cluster views
  • Reorganized sidebar navigation with a dedicated Enterprise section

Type of change

  • Bug fix
  • Feature
  • Refactor
  • Documentation
  • Chore/CI

Affected areas

  • Core (Go)
  • Transports (HTTP)
  • Providers/Integrations
  • Plugins
  • UI (Next.js)
  • Docs

How to test

Test the new Observability UI and middleware changes:

# Core/Transports
go version
go test ./...

# UI
cd ui
pnpm i || npm i
pnpm dev || npm run dev

Navigate to the new Observability page to verify the UI components render correctly.

Screenshots/Recordings

N/A

Breaking changes

  • Yes
  • No

Related issues

N/A

Security considerations

No significant security implications. The middleware refactoring improves request handling but doesn't change authentication or authorization logic.

Checklist

  • I read docs/contributing/README.md and followed the guidelines
  • I added/updated tests where appropriate
  • I updated documentation where needed
  • I verified builds succeed (Go and UI)
  • I verified the CI pipeline passes locally if applicable

Copy link
Contributor

coderabbitai bot commented Sep 20, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (7)
  • main
  • beta
  • feat/*
  • feature/*
  • fix/*
  • hotfix/*
  • ^\d{2}-\d{2}-*

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch observability

Tip

🧪 Early access (models): enabled

We are currently testing Sonnet 4.5 code review models, which should lead to better review quality. However, this model may result in higher noise levels in the review comments. Please disable the early access features if the noise level causes any inconvenience.

Note:

  • Public repositories are always opted into early access features.
  • You can enable or disable early access features from the CodeRabbit UI or by updating the CodeRabbit configuration file.

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

Copy link
Contributor Author

akshaydeo commented Sep 20, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@akshaydeo akshaydeo mentioned this pull request Sep 20, 2025
18 tasks
@akshaydeo akshaydeo marked this pull request as ready for review September 20, 2025 16:21
@akshaydeo akshaydeo force-pushed the observability branch 3 times, most recently from d8e1c6e to 64e4837 Compare September 23, 2025 11:41
@akshaydeo akshaydeo mentioned this pull request Sep 23, 2025
18 tasks
@akshaydeo akshaydeo force-pushed the sso branch 2 times, most recently from d575a49 to 769797c Compare September 24, 2025 21:58
@akshaydeo akshaydeo mentioned this pull request Sep 24, 2025
18 tasks
@akshaydeo akshaydeo force-pushed the observability branch 2 times, most recently from e41f49d to 1df0ef5 Compare September 29, 2025 13:18
@akshaydeo akshaydeo closed this Sep 29, 2025
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.

1 participant