Skip to content

test fixes#2569

Merged
akshaydeo merged 1 commit intov1.5.0from
04-08-test_fixes
Apr 8, 2026
Merged

test fixes#2569
akshaydeo merged 1 commit intov1.5.0from
04-08-test_fixes

Conversation

@akshaydeo
Copy link
Copy Markdown
Contributor

@akshaydeo akshaydeo commented Apr 8, 2026

Summary

Add missing go.mod entries for golang.org/x/oauth2 v0.36.0 and github.com/maximhq/maxim-go v0.2.1 dependencies across the project modules.

Changes

  • Added missing go.mod entries for golang.org/x/oauth2 v0.36.0 in go.sum files across framework and plugin modules
  • Added missing go.mod entry for github.com/maximhq/maxim-go v0.2.1 in the maxim plugin
  • Added github.com/maximhq/bifrost/plugins/prompts v1.0.1 dependency to the transports module
  • Added various missing AWS SDK and other dependency entries in the prompts plugin go.sum

Type of change

  • Chore/CI

Affected areas

  • Core (Go)
  • Transports (HTTP)
  • Plugins

How to test

Verify that all modules can resolve their dependencies correctly:

# Verify dependency resolution
go mod tidy
go mod verify

# Run tests to ensure dependencies work
go test ./...

# Check specific modules
cd framework && go mod verify
cd plugins/governance && go mod verify
cd plugins/maxim && go mod verify
cd transports && go mod verify

Screenshots/Recordings

N/A

Breaking changes

  • No

Related issues

N/A

Security considerations

No security implications - this only adds missing dependency metadata entries.

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

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 8, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1905a760-17f7-4687-8ef9-4ef2fdac03d8

📥 Commits

Reviewing files that changed from the base of the PR and between dc7bd24 and 2964ead.

⛔ Files ignored due to path filters (12)
  • framework/go.sum is excluded by !**/*.sum
  • plugins/governance/go.sum is excluded by !**/*.sum
  • plugins/jsonparser/go.sum is excluded by !**/*.sum
  • plugins/litellmcompat/go.sum is excluded by !**/*.sum
  • plugins/logging/go.sum is excluded by !**/*.sum
  • plugins/maxim/go.sum is excluded by !**/*.sum
  • plugins/mocker/go.sum is excluded by !**/*.sum
  • plugins/otel/go.sum is excluded by !**/*.sum
  • plugins/prompts/go.sum is excluded by !**/*.sum
  • plugins/semanticcache/go.sum is excluded by !**/*.sum
  • plugins/telemetry/go.sum is excluded by !**/*.sum
  • transports/go.sum is excluded by !**/*.sum
📒 Files selected for processing (1)
  • transports/go.mod

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated project dependencies to include the latest prompts plugin version.

Walkthrough

A new direct dependency on github.com/maximhq/bifrost/plugins/prompts v1.0.1 was added to the Go module file. This is a metadata-only change that introduces a single new package requirement to the transports module.

Changes

Cohort / File(s) Summary
Go Module Dependency
transports/go.mod
Added direct dependency on github.com/maximhq/bifrost/plugins/prompts v1.0.1 in the require block.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A whisker twitch, a dep so neat,
Prompts plugin makes the toolset complete,
One line added, the module does grow,
To bifrost's bounty, hooray we go! 🎉

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 04-08-test_fixes

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

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@akshaydeo akshaydeo marked this pull request as ready for review April 8, 2026 15:27
Copy link
Copy Markdown
Contributor Author

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

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 8, 2026

Confidence Score: 5/5

Safe to merge — only dependency lockfiles were updated, no source code changed.

All changes are to generated/managed files (go.sum, go.mod). No logic, tests, or configuration were altered, and no P0/P1 findings were identified.

No files require special attention.

Vulnerabilities

No security concerns identified. The changes are limited to go.sum and go.mod files; all checksum entries follow standard Go module format and no suspicious or unknown packages were introduced.

Important Files Changed

Filename Overview
transports/go.mod Dependency specification for the transports module — no removals or suspicious additions detected; direct dependencies look consistent with the module's purpose.
transports/go.sum Checksum lockfile regenerated to match updated dependency set; no concerns with the entries present.
framework/go.sum Checksum lockfile updated for the framework module; entries appear consistent and well-formed.
plugins/governance/go.sum Lockfile update for the governance plugin; no anomalies observed.
plugins/semanticcache/go.sum Lockfile update for the semanticcache plugin; no anomalies observed.
plugins/otel/go.sum Lockfile update for the otel plugin; no anomalies observed.

Reviews (1): Last reviewed commit: "test fixes" | Re-trigger Greptile

Copy link
Copy Markdown
Contributor Author

Merge activity

  • Apr 8, 3:30 PM UTC: A user started a stack merge that includes this pull request via Graphite.

@akshaydeo akshaydeo merged commit 6c5f942 into v1.5.0 Apr 8, 2026
11 of 16 checks passed
@akshaydeo akshaydeo deleted the 04-08-test_fixes branch April 8, 2026 15:30
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