Skip to content

Remove redundant custom type hierarchy view - #4565

Merged
datho7561 merged 2 commits into
redhat-developer:mainfrom
MeherSru:fix-4560-type-hierarchy
Oct 30, 2026
Merged

datho7561 merged 2 commits into
redhat-developer:mainfrom
MeherSru:fix-4560-type-hierarchy

Conversation

@MeherSru

Copy link
Copy Markdown
Contributor

Fixes #4560

Removes the custom type hierarchy view, since VS Code already provides one.

Signed-off-by: Meher Srujana Matcha <mehersrujana.matcha@ibm.com>
@datho7561

Copy link
Copy Markdown
Contributor

Found a few more references to type hierarchy that can probably be removed:

diff --git a/src/commands.ts b/src/commands.ts
index 40aba70..9e388b2 100644
--- a/src/commands.ts
+++ b/src/commands.ts
@@ -228,14 +228,6 @@ export namespace Commands {
      * Navigate To Super Method Command.
      */
     export const NAVIGATE_TO_SUPER_IMPLEMENTATION_COMMAND = 'java.action.navigateToSuperImplementation';
-    /**
-     * Open the given TypeHierarchy Item.
-     */
-    export const OPEN_TYPE_HIERARCHY = 'java.navigate.openTypeHierarchy';
-    /**
-     * Resolve the given TypeHierarchy Item.
-     */
-    export const RESOLVE_TYPE_HIERARCHY = 'java.navigate.resolveTypeHierarchy';
     /**
      * Show server task status
      */
diff --git a/test/standard-mode-suite/extension.test.ts b/test/standard-mode-suite/extension.test.ts
index 154c64a..e7a08e0 100644
--- a/test/standard-mode-suite/extension.test.ts
+++ b/test/standard-mode-suite/extension.test.ts
@@ -91,7 +91,6 @@ suite('Java Language Extension - Standard', () => {
                                Commands.OPEN_SERVER_LOG,
                                Commands.OPEN_SERVER_STDOUT_LOG,
                                Commands.OPEN_SERVER_STDERR_LOG,
-                               Commands.OPEN_TYPE_HIERARCHY,
                                Commands.ORGANIZE_IMPORTS,
                                Commands.ORGANIZE_IMPORTS_SILENTLY,
                                Commands.OVERRIDE_METHODS_PROMPT,
@@ -102,7 +101,6 @@ suite('Java Language Extension - Standard', () => {
                                Commands.REMOVE_FROM_SOURCEPATH_CMD,
                                Commands.RENAME_COMMAND,
                                "java.project.resolveStackTraceLocation",
-                               Commands.RESOLVE_TYPE_HIERARCHY,
                                Commands.RESOLVE_WORKSPACE_SYMBOL,
                                Commands.RUNTIME_VALIDATION_OPEN,
                                Commands.RESTART_LANGUAGE_SERVER,

Signed-off-by: Meher Srujana Matcha <mehersrujana.matcha@ibm.com>

@datho7561 datho7561 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, Meher!

@datho7561
datho7561 merged commit aea353a into redhat-developer:main Oct 30, 2026
2 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.

Remove Redundant Type Hierarchy

2 participants