[JonatanAntoni]

Fixes

• [Bug]: Trace Generation dialog - usability issues #1292 Edit field behavior [..]

Changes

• Intercept Enter key press to commit pending edit but stop propagation to keep focus/cursor in place for further editing

Screenshots

Checklist

• 🤖 This change is covered by unit tests (if applicable).
• 🤹 Manual testing has been performed (if necessary).
• 🛡️ Security impacts have been considered (if relevant).
• 📖 Documentation updates are complete (if required).
• 🧠 Third-party dependencies and TPIP updated (if required).

[Copilot]

🟡 Changes recommended

A skipped edit after an on-disk reload can leave the input displaying an uncommitted value that subsequent Enter or blur events cannot retry.

1 open issue

• Skipped stale edits remain visible but cannot be committed · New

What changed in this PR

This PR addresses the Enter-key editing behavior in #1292, keeping focus and the caret in Trace Generation text fields while the host processes edits.

Changes:

• Commit on Enter without blurring, while leaving composition events untouched.
• Track text-edit completion across host updates to preserve active drafts and selection.
• Add renderer and provider tests for focus retention, overlapping edits, and completion handling.

File	Description
src/​webviews/​trace-configuration/​src/​trace-configuration.ts	Preserves active text edits and focus across host renders.
src/​webviews/​trace-configuration/​src/​trace-configuration.test.ts	Tests Enter behavior, draft retention, and focus boundaries.
src/​views/​trace-configuration/​trace-configuration-webview-provider.ts	Sends completion IDs after handling text edits.
src/​views/​trace-configuration/​trace-configuration-webview-provider.test.ts	Tests completion timing, failures, and disposal.
src/​views/​trace-configuration/​trace-configuration-protocol.ts	Adds optional edit and completion IDs to messages.

_{Review effort: Balanced}

[qltysh]

Coverage Impact

This PR will not change total coverage.

Modified Files with Diff Coverage (1)

Rating	File	% Diff	Uncovered Line #s
	...ws/trace-configuration/trace-configuration-webview-provider.ts	100.0%
	Total	100.0%

🚦 See full report on Qlty Cloud »

🛟 Help

• Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

• Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

• File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

  • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

[arneschmid]

LGTM