nes: log tree: fix: XtabProvider debug log tree nesting

[ulugbekna]

Problem

XtabProvider entries sometimes appeared unnested at the top level of the Chat Debug log tree instead of under their NES parent entry.

Root Cause

`flattenSingleChild=true` on the NES `CapturingToken` promoted the XtabProvider child to the top level when the sibling NES `MarkdownContentRequest` entry was hidden (via `setIsSkipped()` or cancellation). The hidden entry was skipped entirely in the tree's grouping loop, making XtabProvider the sole visible child, which then got flattened.

Fix

Remove `flattenSingleChild` and `promoteMainEntry` from `CapturingToken` entirely, making it a pure correlation token with no rendering hints.

Instead of adding the NES log context document as a child entry that gets "promoted" into the parent, `ChatPromptItem` now directly owns the document via `setMainEntry()`. A `MarkdownContentRequest` whose `debugName` matches the token's `label` is wired to the parent's icon and click command — never added as a tree child. This means hidden entries cannot displace XtabProvider from its parent.

Groups with no visible children render as non-expandable leaf items (no arrow). Groups with children render expanded.

Changes

• `CapturingToken`: Removed `flattenSingleChild` and `promoteMainEntry` constructor params
• `requestLogTree.ts`: Rewrote `getChildren()` grouping — main entries are associated directly with the parent via `setMainEntry()`, never added as children. Removed post-processing loop. `withFilteredChildren()` sets `collapsibleState` based on children count.
• 12 caller files + 2 test files: Updated `CapturingToken` constructor args

[Copilot]

Pull request overview

Fixes incorrect nesting of XtabProvider items in the Chat Debug request log tree by removing “rendering hints” from CapturingToken and changing how the request-log tree associates a group’s “main” markdown document with its parent node.

Changes:

• Remove flattenSingleChild / promoteMainEntry from CapturingToken so it’s purely a correlation token.
• Rewrite request-log tree grouping so “main” MarkdownContentRequest entries are wired to the parent via setMainEntry() instead of being added as children and post-processed.
• Update CapturingToken construction across call sites and adjust affected tests.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/platform/requestLogger/common/capturingToken.ts	Removes tree-rendering hint fields from the token API.
src/extension/log/vscode-node/requestLogTree.ts	Reworks grouping to attach main markdown entry directly to parent; removes flatten/promote post-processing; updates collapsible state behavior.
src/extension/inlineEdits/vscode-node/inlineCompletionProvider.ts	Updates NES capturing token construction after token API change.
src/extension/inlineEdits/node/nextEditProvider.ts	Updates NES/spec capturing token construction after token API change.
src/extension/completions-core/vscode-node/extension/src/vscodeInlineCompletionItemProvider.ts	Updates Ghost capturing token construction after token API change.
src/extension/tools/node/searchSubagentTool.ts	Updates capturing token construction after token API change.
src/extension/tools/node/executionSubagentTool.ts	Updates capturing token construction after token API change.
src/extension/prompt/node/defaultIntentRequestHandler.ts	Updates capturing token construction after token API change (incl. session/parent grouping args).
src/extension/prompt/node/title.ts	Updates capturing token construction after token API change (child-session grouping).
src/extension/prompt/node/summarizer.ts	Updates capturing token construction after token API change (child-session grouping).
src/extension/prompt/node/promptCategorizer.ts	Updates capturing token construction after token API change (child-session grouping).
src/extension/chatSessions/copilotcli/node/copilotcliSession.ts	Updates capturing token construction after token API change; minor import ordering tweak.
src/extension/chatSessions/claude/node/claudeCodeAgent.ts	Updates capturing token construction after token API change.
src/extension/chatSessions/claude/vscode-node/slashCommands/terminalCommand.ts	Updates capturing token construction after token API change.
src/extension/chatSessions/claude/vscode-node/slashCommands/test/terminalCommand.spec.ts	Updates assertions after removal of flattenSingleChild.
src/extension/replay/vscode-node/test/chatReplayNotebook.spec.ts	Updates capturing token construction in tests after token API change.

Comments suppressed due to low confidence (1)

src/extension/log/vscode-node/requestLogTree.ts:634

• prompt.children is incrementally appended to but never pruned. When a MarkdownContentRequest child was previously added while visible and later flips to hidden (isVisible() becomes false), the continue here prevents adding it again but does not remove the existing child, so it may remain visible in the tree. If isVisible is intended to dynamically hide entries, consider removing any existing child with the same id when skipping, or rebuilding children from scratch each refresh.

				// Skip non-main hidden entries
				if (currReq.kind === LoggedInfoKind.Request &&
					currReq.entry.type === LoggedRequestKind.MarkdownContentRequest &&
					currReq.entry.isVisible && !currReq.entry.isVisible()) {
					continue;
				}