[test-improver] Improve tests for launcher package

[github-actions]

Summary

Improved test coverage for internal/launcher package from 96.8% → 97.3%.

File Analyzed

internal/launcher/getorlaunchforsession_test.go and internal/launcher/launcher_runtime_test.go

Improvements Made

New tests in getorlaunchforsession_test.go

• TestGetOrLaunchForSession_ServerNotFound — covers the ErrServerNotFound error path when an unknown server ID is requested
• TestGetOrLaunchForSession_HTTPBackendRecordsStart — covers the HTTP backend path through GetOrLaunchForSession → getOrLaunchWithConfig → recordStart, and verifies server state is marked as "running"
• TestGetOrLaunchForSession_DoubleCheckLockPoolHit — exercises the session pool cache hit path where a pre-populated pool entry is returned without launching a new backend

Updated tests in launcher_runtime_test.go

• nil config case added to TestIsDirectStdioCommand table — covers the if serverCfg == nil { return true } guard in isDirectStdioCommand
• TestNew_RunningInContainer — covers the log.Println("[LAUNCHER] Detected running inside a container") branch in New() by setting RUNNING_IN_CONTAINER=true

Coverage Before/After

Function	Before	After
launcher.go: New	95.2%	100.0%
launcher.go: isDirectStdioCommand	92.9%	100.0%
Total	96.8%	97.3%

Test Output

ok  	github.com/github/gh-aw-mcpg/internal/launcher	16.620s	coverage: 97.3% of statements

All tests pass with -count=3 for stability verification.

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

• index.crates.io

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "index.crates.io"

See Network Configuration for more information.

Generated by Test Improver · sonnet46 · 118.3 AIC · ⊞ 5.8K · ◷