fix: improve hidden index tree item visibility and menu actions

[lte-z]

Fixes #656

Changes

This PR improves hidden index visibility and context menu behavior in the tree view.

Tree item updates

• Added hidden/default state segments to IndexItem context values
• Added a hidden description for hidden indexes in the tree view

Context menu updates

Updated menu when clauses so only valid actions are shown:

• Hidden indexes show only "Unhide Index…"
• Non-hidden indexes show only "Hide Index…"
• _id_ indexes show neither action

Behavior

Hidden indexes

• Display hidden next to the index name in the tree
• Show only "Unhide Index…" in the context menu

Regular indexes

• Show only "Hide Index…" in the context menu

_id_ index

• Does not show hide/unhide actions

Screenshots

id index

Regular and hidden indexes

Notes

• Tooltip behavior remains unchanged
• Uses createContextValue() to compose index state context segments

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds richer context and UI cues for DocumentDB index tree items so the Hide/Unhide Index commands only appear when appropriate (e.g., not for default _id_ indexes, and toggling based on hidden state).

Changes:

• Add index_hidden / index_default context values and show a “hidden” description for hidden indexes in the tree.
• Update command when clauses to hide/show Hide/Unhide actions based on those context values.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
src/tree/documentdb/IndexItem.ts	Adds contextValue tokens for default/hidden indexes and displays “hidden” in the tree item description.
package.json	Refines command visibility logic for Hide/Unhide Index using the new contextValue tokens.

[tnaum-ms]

Thanks for the contribution! I pushed two small follow-ups directly to your branch:

1. 74a9a80 rename the new context values to align with the existing tree naming convention (single-axis prefixes like treeItem_* and experience_*). The values are now state_hidden and state_default, which scopes them as state modifiers and avoids collisions if other element types gain similar flags later.
   74a9a805

2. e9148ad wrap the new 'hidden' description in vscode.l10n.t() so it gets picked up by our localization pipeline, and regenerate l10n/bundle.l10n.json via npm run l10n.
   e9148ad1

Reviewing the rest now.

[tnaum-ms]

One more follow-up:

3. e1b8325 replace showCreatingChild() with runWithTemporaryDescription() in the hide and unhide index commands. The previous approach inserted a virtual phantom sub-tree item under the index while the operation was in flight, which was confusing. Now the index node itself shows a brief 'Hiding…' or 'Unhiding…' description, matching the pattern used by other long-running operations in the extension (export, import, copy connection string, etc.).
   e1b8325c

[tnaum-ms]

Thanks @lte-z for another solid contribution!

This is your second PR to the project I'm merging, and the quality is consistently good. The fix is well scoped and the when clause logic is correct. I pushed a few small follow-ups directly to your branch, all noted in the comments above. Once CI passes I'll approve and merge. Keep them coming!

[lte-z]

Thank you so much for merging this and for the detailed follow-up commits!

The changes around state_hidden / state_default, localization, and using the temporary description for Hiding… / Unhiding… all make sense to me. I learned a lot from seeing how you aligned the fix with the existing tree naming and long-running operation patterns.

Thanks again for the review and guidance!