Skip to content

fix(bitbucket): Adjust workspace parsing for updated API response#877

Merged
drazisil-codecov merged 2 commits intomainfrom
seer/fix/bitbucket-workspace-api-parsing
Apr 20, 2026
Merged

fix(bitbucket): Adjust workspace parsing for updated API response#877
drazisil-codecov merged 2 commits intomainfrom
seer/fix/bitbucket-workspace-api-parsing

Conversation

@sentry
Copy link
Copy Markdown
Contributor

@sentry sentry Bot commented Apr 20, 2026

Fixes API-E43. The issue was that: Bitbucket API /user/workspaces response changed, nesting workspace data and omitting 'name' key, causing KeyError when code expects direct access.

  • Updated the parsing logic for Bitbucket workspace data in get_teams.
  • Adapted to a change in the Bitbucket API response structure for /user/workspaces.
  • name, id, and username fields are now extracted from a nested workspace object within the API response.
  • The name and username fields now use the slug from the nested workspace object.

This fix was generated by Seer in Sentry, triggered automatically. 👁️ Run ID: 13591205

Not quite right? Click here to continue debugging with Seer.

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.


Note

Medium Risk
Medium risk because it changes how workspace/team identities are derived, which affects Bitbucket repo discovery for users and depends on an external API response format.

Overview
Fixes Bitbucket team/workspace listing by updating Bitbucket.list_teams() to read name, id, and username from the nested workspace object returned by /user/workspaces (using the workspace slug and uuid).

Updates integration VCR cassettes and adjusts integration/unit test expectations to reflect the new API payload shape and the fact that the returned name now matches the slug rather than the previous display name.

Reviewed by Cursor Bugbot for commit 9912498. Bugbot is set up for automated code reviews on this repo. Configure here.

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Apr 20, 2026

Merging this PR will not alter performance

✅ 9 untouched benchmarks


Comparing seer/fix/bitbucket-workspace-api-parsing (9912498) with main (548e0c1)

Open in CodSpeed

@sentry
Copy link
Copy Markdown
Contributor Author

sentry Bot commented Apr 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.26%. Comparing base (548e0c1) to head (9912498).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #877   +/-   ##
=======================================
  Coverage   92.26%   92.26%           
=======================================
  Files        1307     1307           
  Lines       48011    48011           
  Branches     1632     1632           
=======================================
  Hits        44295    44295           
  Misses       3407     3407           
  Partials      309      309           
Flag Coverage Δ
apiunit 96.35% <ø> (ø)
sharedintegration 36.90% <ø> (ø)
sharedunit 84.91% <ø> (ø)
workerintegration 58.54% <ø> (ø)
workerunit 90.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

…cess response shape

The /user/workspaces endpoint returns workspace_access objects with nested
workspace_base (slug/uuid only, no name). Update VCR cassettes, integration
test assertions, and unit test mock to match actual API structure.

Fixes API-E43

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov-notifications
Copy link
Copy Markdown

codecov-notifications Bot commented Apr 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@drazisil-codecov drazisil-codecov marked this pull request as ready for review April 20, 2026 18:35
@drazisil-codecov drazisil-codecov added this pull request to the merge queue Apr 20, 2026
Merged via the queue into main with commit b3cda5a Apr 20, 2026
53 checks passed
@drazisil-codecov drazisil-codecov deleted the seer/fix/bitbucket-workspace-api-parsing branch April 20, 2026 18:48
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