Skip to content

test: add needs_session_stickiness to MCP client e2e collection - #6049

Merged
akshaydeo merged 1 commit into
devfrom
08-11-fix_mcp_reconnect_test_case_in_e2e-api
Aug 11, 2026
Merged

test: add needs_session_stickiness to MCP client e2e collection#6049
akshaydeo merged 1 commit into
devfrom
08-11-fix_mcp_reconnect_test_case_in_e2e-api

Conversation

@impoiler

Copy link
Copy Markdown
Contributor

Summary

Adds needs_session_stickiness: true to the MCP client creation request in the end-to-end API test collection, ensuring the test accurately reflects the expected payload for MCP clients that require session stickiness.

Changes

  • Added needs_session_stickiness: true to the raw request body in the bifrost-api-management Postman collection for the MCP client creation endpoint.

Type of change

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

Affected areas

  • Core (Go)
  • Transports (HTTP)
  • Providers/Integrations
  • Plugins
  • UI (React)
  • Docs

How to test

Run the Postman collection against a running Bifrost instance and verify the MCP client creation request succeeds with the needs_session_stickiness field included.

go test ./...

Breaking changes

  • Yes
  • No

Related issues

Security considerations

No security implications. This change only adds a session stickiness flag to a test request payload.

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

@impoiler
impoiler marked this pull request as ready for review August 11, 2026 07:55
@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Summary by CodeRabbit

  • Bug Fixes
    • Updated MCP client creation requests to enable session stickiness, improving session consistency.

Walkthrough

The Add MCP Client Postman request now sends needs_session_stickiness: true with its existing connection and authentication fields.

Changes

MCP client configuration

Layer / File(s) Summary
Add session stickiness to MCP client creation
tests/e2e/api/collections/bifrost-api-management.postman_collection.json
The request payload now includes needs_session_stickiness: true.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: akshaydeo, sammaji

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The description explains the payload change, test method, impact, and security considerations, but some checklist items and affected-area selections remain incomplete.
Title check ✅ Passed The title clearly identifies the test collection change and the added needs_session_stickiness field.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 08-11-fix_mcp_reconnect_test_case_in_e2e-api

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

@impoiler impoiler self-assigned this Aug 11, 2026

impoiler commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

coderabbitai[bot]
coderabbitai Bot previously approved these changes Aug 11, 2026
@impoiler impoiler mentioned this pull request Aug 11, 2026
18 tasks
@akshaydeo
akshaydeo changed the base branch from 08-11-fix_fix_governance.test.ts_file to graphite-base/6049 August 11, 2026 08:40
@akshaydeo
akshaydeo force-pushed the 08-11-fix_mcp_reconnect_test_case_in_e2e-api branch from d178766 to e0d2175 Compare August 11, 2026 08:40
@graphite-app
graphite-app Bot changed the base branch from graphite-base/6049 to dev August 11, 2026 08:41
@graphite-app
graphite-app Bot dismissed coderabbitai[bot]’s stale review August 11, 2026 08:41

The base branch was changed.

@akshaydeo
akshaydeo force-pushed the 08-11-fix_mcp_reconnect_test_case_in_e2e-api branch from e0d2175 to 9ff69a5 Compare August 11, 2026 08:41

akshaydeo commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Merge activity

  • Aug 11, 8:42 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Aug 11, 8:43 AM UTC: @akshaydeo merged this pull request with Graphite.

@akshaydeo
akshaydeo merged commit 372ec8a into dev Aug 11, 2026
15 checks passed
@akshaydeo
akshaydeo deleted the 08-11-fix_mcp_reconnect_test_case_in_e2e-api branch August 11, 2026 08:43
akshaydeo pushed a commit that referenced this pull request Aug 13, 2026
)

## Summary

Adds `needs_session_stickiness: true` to the MCP client creation request in the end-to-end API test collection, ensuring the test accurately reflects the expected payload for MCP clients that require session stickiness.

## Changes

- Added `needs_session_stickiness: true` to the raw request body in the `bifrost-api-management` Postman collection for the MCP client creation endpoint.

## Type of change

- [ ] Bug fix
- [x] Feature
- [ ] Refactor
- [ ] Documentation
- [ ] Chore/CI

## Affected areas

- [ ] Core (Go)
- [ ] Transports (HTTP)
- [ ] Providers/Integrations
- [ ] Plugins
- [ ] UI (React)
- [ ] Docs

## How to test

Run the Postman collection against a running Bifrost instance and verify the MCP client creation request succeeds with the `needs_session_stickiness` field included.

```sh
go test ./...
```

## Breaking changes

- [ ] Yes
- [x] No

## Related issues

## Security considerations

No security implications. This change only adds a session stickiness flag to a test request payload.

## Checklist

- [ ] I read `docs/contributing/README.md` and followed the guidelines
- [x] 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
akshaydeo pushed a commit that referenced this pull request Aug 13, 2026
)

## Summary

Adds `needs_session_stickiness: true` to the MCP client creation request in the end-to-end API test collection, ensuring the test accurately reflects the expected payload for MCP clients that require session stickiness.

## Changes

- Added `needs_session_stickiness: true` to the raw request body in the `bifrost-api-management` Postman collection for the MCP client creation endpoint.

## Type of change

- [ ] Bug fix
- [x] Feature
- [ ] Refactor
- [ ] Documentation
- [ ] Chore/CI

## Affected areas

- [ ] Core (Go)
- [ ] Transports (HTTP)
- [ ] Providers/Integrations
- [ ] Plugins
- [ ] UI (React)
- [ ] Docs

## How to test

Run the Postman collection against a running Bifrost instance and verify the MCP client creation request succeeds with the `needs_session_stickiness` field included.

```sh
go test ./...
```

## Breaking changes

- [ ] Yes
- [x] No

## Related issues

## Security considerations

No security implications. This change only adds a session stickiness flag to a test request payload.

## Checklist

- [ ] I read `docs/contributing/README.md` and followed the guidelines
- [x] 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
akshaydeo pushed a commit that referenced this pull request Aug 14, 2026
)

## Summary

Adds `needs_session_stickiness: true` to the MCP client creation request in the end-to-end API test collection, ensuring the test accurately reflects the expected payload for MCP clients that require session stickiness.

## Changes

- Added `needs_session_stickiness: true` to the raw request body in the `bifrost-api-management` Postman collection for the MCP client creation endpoint.

## Type of change

- [ ] Bug fix
- [x] Feature
- [ ] Refactor
- [ ] Documentation
- [ ] Chore/CI

## Affected areas

- [ ] Core (Go)
- [ ] Transports (HTTP)
- [ ] Providers/Integrations
- [ ] Plugins
- [ ] UI (React)
- [ ] Docs

## How to test

Run the Postman collection against a running Bifrost instance and verify the MCP client creation request succeeds with the `needs_session_stickiness` field included.

```sh
go test ./...
```

## Breaking changes

- [ ] Yes
- [x] No

## Related issues

## Security considerations

No security implications. This change only adds a session stickiness flag to a test request payload.

## Checklist

- [ ] I read `docs/contributing/README.md` and followed the guidelines
- [x] 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
akshaydeo pushed a commit that referenced this pull request Aug 19, 2026
)

## Summary

Adds `needs_session_stickiness: true` to the MCP client creation request in the end-to-end API test collection, ensuring the test accurately reflects the expected payload for MCP clients that require session stickiness.

## Changes

- Added `needs_session_stickiness: true` to the raw request body in the `bifrost-api-management` Postman collection for the MCP client creation endpoint.

## Type of change

- [ ] Bug fix
- [x] Feature
- [ ] Refactor
- [ ] Documentation
- [ ] Chore/CI

## Affected areas

- [ ] Core (Go)
- [ ] Transports (HTTP)
- [ ] Providers/Integrations
- [ ] Plugins
- [ ] UI (React)
- [ ] Docs

## How to test

Run the Postman collection against a running Bifrost instance and verify the MCP client creation request succeeds with the `needs_session_stickiness` field included.

```sh
go test ./...
```

## Breaking changes

- [ ] Yes
- [x] No

## Related issues

## Security considerations

No security implications. This change only adds a session stickiness flag to a test request payload.

## Checklist

- [ ] I read `docs/contributing/README.md` and followed the guidelines
- [x] 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
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