feat: add OpenExplorerUi restricted action (ExplorerUi, OpenResult)#446
Draft
popuz wants to merge 11 commits into
Draft
feat: add OpenExplorerUi restricted action (ExplorerUi, OpenResult)#446popuz wants to merge 11 commits into
popuz wants to merge 11 commits into
Conversation
Adds an OpenExplorerUi RPC to the existing RestrictedActionsService so SDK7 scenes can open native explorer UI panels. Introduces the ExplorerUi enum (7 fullscreen sections) and a dedicated OpenExplorerUiResponse carrying an OpenResult verdict (opened / rejected reasons). Additive, non-breaking. Part of the Scene-triggered Explorer UI feature (iteration 1). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Test this pull request on NPM or Yarn
|
…iggered-explorer-ui
… prefix) Naming-only change for the openExplorerUi restricted action. Renames the verdict enum OpenResult -> OpenExplorerUiResult, prefixes its values OR_ -> OEUR_, and updates the OpenExplorerUiResponse field to open_result. ExplorerUi/EU_* unchanged. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Enum values are now bare names (UNSPECIFIED/OPENED/REJECTED_*) instead of the OEUR_ prefix. Enum type OpenExplorerUiResult and ExplorerUi/EU_* unchanged. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Comment-only. Records the additive-evolution intent for ExplorerUi and OpenExplorerUiResult, the oneof extension point for future per-panel params on OpenExplorerUiRequest, and that OpenExplorerUiResponse carries only the open verdict (post-open events belong to a separate future event channel). No wire/semantic change. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
pravusjif
reviewed
Jul 20, 2026
pravusjif
left a comment
pravusjif
left a comment
Member
There was a problem hiding this comment.
maybe the UNSPECIFIED values could be removed? I can't think of their use case, maybe only the response one makes sense
Additive enum value (=5) for the gesture-gate on openExplorerUi. Existing values unchanged. Rejected when the call did not originate from a user gesture. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
kuruk-mm
reviewed
Jul 22, 2026
Co-authored-by: Mateo Miccino <mateomiccino@gmail.com> Signed-off-by: Vitaly Popuzin <35366872+popuz@users.noreply.github.com>
Contributor
Author
ah, you are regarding |
…FIED Remove EU_UNSPECIFIED and renumber ExplorerUi contiguously so EU_SETTINGS holds 0, making the least-intrusive panel the default for an unset ui field. Trim standalone doc-comment lines on ExplorerUi, OpenExplorerUiResult and the OpenExplorerUi rpc. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…' into feat/scene-triggered-explorer-ui
This was referenced Jul 24, 2026
popuz
marked this pull request as draft
July 24, 2026 15:02
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds an
OpenExplorerUiRPC to the existingRestrictedActionsServiceso SDK7 scenes can open native explorer UI panels (Map, Settings, Backpack, Camera Reel, Communities, Places, Events) through~system/RestrictedActions.ExplorerUi— 7 fullscreen sections (+EU_UNSPECIFIEDzero value).OpenResult— open verdict (OR_OPENED+ rejection reasons).OR_REJECTED_ALREADY_OPENis reserved for a later iteration.OpenExplorerUiRequest { ExplorerUi ui }/OpenExplorerUiResponse { OpenResult open_result }(dedicated response, not the sharedSuccessResponse).rpc OpenExplorerUi(OpenExplorerUiRequest) returns (OpenExplorerUiResponse)onRestrictedActionsService.Additive and non-breaking. Only
proto/decentraland/kernel/apis/restricted_actions.protochanged.Context
Iteration 1 of the Scene-triggered Explorer UI feature (opens a panel, returns an open verdict; always
OR_OPENEDin the explorer for now, no event channel yet).Testing
Consumed downstream by js-sdk-toolchain (
~system/RestrictedActionscodegen) and unity-explorer (C# RestrictedActions implementation) on matchingfeat/scene-triggered-explorer-uibranches.🤖 Generated with Claude Code