Skip to content

feat(navigation): add direct navigator submenu commands - #114

Open
Zoriot wants to merge 2 commits into
mainfrom
issue-62-nav-submenu-commands
Open

Zoriot wants to merge 2 commits into
mainfrom
issue-62-nav-submenu-commands

Conversation

@Zoriot

@Zoriot Zoriot commented Oct 10, 2026

Copy link
Copy Markdown
Collaborator

Register navigator destinations through the Cloud command tree so users can open explore, continent, country, state, city, warp, build, plot system, and tutorial menus directly. Add greedy, case-insensitive destination arguments with tab suggestions, and apply navigator and warp permissions at command registration.

Reuse the main menu's configured actions for direct Build, Plot System, and Tutorials commands. Share state and city destination discovery with the existing state menu logic. Remove the standalone /explore command and its alias configuration so exploring is available through /navigator explore.

Closes #62


GitHub Copilot was used for most of the implementation.

@Zoriot
Zoriot requested a review from a team October 10, 2026 23:31
@Zoriot Zoriot self-assigned this Oct 10, 2026
@Zoriot
Zoriot added this pull request to stack #113 October 10, 2026 23:31
@Zoriot Zoriot added ⏰ medium priority This currently has medium priority. 🐻 medium size This is a medium issue. labels Oct 10, 2026
@Zoriot Zoriot linked an issue Oct 10, 2026 that may be closed by this pull request
@github-actions

github-actions Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Qodana for JVM

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@Zoriot
Zoriot force-pushed the issue-62-nav-submenu-commands branch from f263ba6 to f9f3d02 Compare October 11, 2026 20:17
@Zoriot
Zoriot requested a review from Jasupa October 11, 2026 20:20
@Zoriot
Zoriot force-pushed the issue-62-nav-submenu-commands branch from f9f3d02 to 1694f58 Compare October 11, 2026 20:24
Base automatically changed from command-system-rework to main October 11, 2026 20:27
@Zoriot
Zoriot force-pushed the issue-62-nav-submenu-commands branch from 1694f58 to 34145df Compare October 11, 2026 20:27
Register navigator destinations through the Cloud command tree so users can open explore, continent, country, state, city, warp, build, plot system, and tutorial menus directly. Add greedy, case-insensitive destination arguments with tab suggestions, and apply navigator and warp permissions at command registration.

Reuse the main menu's configured actions for direct Build, Plot System, and Tutorials commands. Share state and city destination discovery with the existing state menu logic. Remove the standalone /explore command and its alias configuration so exploring is available through /navigator explore.

Closes #62
Remove /btt nav warps since /btt warp already opens the warp menu. Only register the nav build, plotsystem and tutorials subcommands when their menu item is enabled in the navigation config. Adds BuildTeamTools#getNavigationConfig so commands can read the config at registration time, before the module's own config field is set.
@Zoriot
Zoriot force-pushed the issue-62-nav-submenu-commands branch from 5704120 to 89802d7 Compare October 11, 2026 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⏰ medium priority This currently has medium priority. 🐻 medium size This is a medium issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create Commands for Nav Submenues

1 participant