[VSCodeSweeper MVP — test only] Completion-inserted () no longer tracked as auto-closed pair

[egamma]

⚠️ VSCodeSweeper MVP — test only. This draft PR was generated by an AI experiment
(vscodesweeper) inside a personal fork. It is
not intended for microsoft/vscode and is not a contribution proposal.

Upstream issue: microsoft#315131

Change summary

Accepting a completion that inserts foo() via a snippet no longer registers the parens as an auto-closed pair, breaking backspace-deletes-both and overtype behavior.

Validation

Add a test in src/vs/editor/common/cursor/test (or the existing autoClosingPairs test suite) that calls executeEdits with source='suggest' and verifies _findAutoClosingPairs detects the inserted parens; it should fail before the fix and pass after.

---

Branch vscodesweeper/fix-315131 · generated fix · review loop runs separately (codex review).

[egamma]

🤖 codex review — round 1

VSCodeSweeper MVP codex cross-review of a copilot-generated fix (fork-only experiment).

The change extends existing auto-closed pair tracking to suggestion-sourced edits and includes a targeted regression test. I did not find a discrete introduced issue that would break existing behavior.

[egamma]

✅ Round 1: no actionable findings — the review loop converged.