Skip to content

fix(apps): use app-bound sharing grant preflight - #33

Merged
torstenrudolf merged 1 commit into
mainfrom
nthillairajah/bot-2418-sharing-preflight
Oct 27, 2026
Merged

torstenrudolf merged 1 commit into
mainfrom
nthillairajah/bot-2418-sharing-preflight

Conversation

@nathan-thillairajah

@nathan-thillairajah nathan-thillairajah commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Sharing an app from a Buzz remote agent should depend on owning the app and being allowed to share it with the target workspace. Today, bl apps share grant can fail before it attempts the grant because its preliminary access check uses ordinary app-management credentials, whose workspace can change with the owner's latest session.

For example, an app created in Personal can become unshareable from the agent after the owner uses BuilderLab in another workspace. The preliminary check returns workspace_mismatch, even though the owner still owns the app and is allowed to share it with the chosen team.

Use the existing owner/app/destination-bound sharing endpoint to read the lifecycle and revision needed to create the grant, matching how revoke already works. This removes the dependency on the ordinary credential's selected workspace. The server still checks ownership, target eligibility, restricted visibility and the current revision when applying the grant.

This fixes sharing an already restricted app. It does not change visibility automatically or fix workspace selection for ordinary app management; that broader work remains in BOT-2418.

@nathan-thillairajah
nathan-thillairajah marked this pull request as ready for review October 23, 2026 22:33
@nathan-thillairajah
nathan-thillairajah requested a review from a team as a code owner October 23, 2026 22:33
@torstenrudolf
torstenrudolf merged commit afba4be into main Oct 27, 2026
4 checks passed
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