Agent host: fix realsdk integration tests

[roblourens]

The realsdk integration tests in src/vs/platform/agentHost/test/node/protocol/ had two issues that prevented the suite from running cleanly:

1. Provisional sessions broke the notify/sessionAdded wait pattern

After commit 8309b22 made createSession return provisional sessions, notify/sessionAdded is deferred until the agent materializes on first sendMessage. The realsdk tests waited for sessionAdded immediately after createSession and timed out (15s) for every test.

Fix: subscribe directly using the URI we passed to createSession (the provisional state is registered in the state manager and addressable by that URI). Synthesize the SessionAddedNotification from the snapshot when needed. The worktree test reorders to wait for sessionAdded after dispatchTurn, since the .worktrees/... path only exists post-materialization.

2. Busy-loop in strips redundant cd test

The teardown loop at the end of the test called waitForNotification with a predicate matching ANY session/toolCallReady, including the one we'd already approved. waitForNotification returns synchronously when an existing notification matches, so the loop kept re-finding the same already-handled notification on every iteration without yielding the event loop — spinning at 100% CPU until the 90s timeout fired.

Fix: track processed serverSeqs in the predicate so the loop only matches new toolCallReady notifications, mirroring the pattern used by driveTurnToCompletion and startBackgroundApprovalLoop.

Validation

• ✅ npm run compile-check-ts-native
• ✅ npm run gulp compile-client
• ✅ Most realsdk tests pass: simple message, tool call permission, abort, working directory, worktree resolution, subagent routing, listModels, strips-redundant-cd, sessionDiffs assertCleanState
• ⚠️ Two known failures, likely model-behavior / SDK behavior rather than agent-host bugs:
  • planning-mode session-state writes are auto-approved in default mode — model emits a tool call that triggers a confirmation
  • terminal-driven file edit shows up in summary.diffs — git-driven diff not produced for the expected URI
• ⚠️ write_bash never triggers a permission request not yet observed end-to-end

No production code changes — test scaffolding only.

[github-actions]

Base: c7b2d000 Current: 4a5ae955

No screenshot changes.

[Copilot]

Pull request overview

This PR updates the @github/copilot npm dependency from 1.0.39 to 1.0.43 in the main VS Code repo and the remote/ package, including corresponding package-lock.json updates so CI/builds consume the new version.

Changes:

• Bump @github/copilot to 1.0.43 in root package.json and remote/package.json.
• Regenerate root and remote/ package-lock.json entries for @github/copilot and its platform-specific optional dependencies.

Show a summary per file

File	Description
package.json	Updates root dependency pin for @github/copilot to 1.0.43.
package-lock.json	Updates resolved tarball/integrity and optional platform packages for @github/copilot@1.0.43.
remote/package.json	Updates remote/ dependency pin for @github/copilot to 1.0.43.
remote/package-lock.json	Updates resolved tarball/integrity and optional platform packages for @github/copilot@1.0.43.

Copilot's findings

Files not reviewed (1)

• remote/package-lock.json: Language not supported

• Files reviewed: 2/4 changed files
• Comments generated: 1