Add selected collaborators to session visibility

[pierceboggan]

Summary

Extend the experimental session visibility contract with explicit collaborator access.

• Add optional collaboratorLogins to VisibilitySetRequest, VisibilityGetResult, and VisibilitySetResult.
• Preserve status-only requests when the field is absent, clear explicit collaborators for an empty list, and carry canonical GitHub logins for non-empty lists.
• Keep SessionVisibilityStatus unchanged. repo represents repository collaborators with push access, while unshared plus a non-empty collaborator list represents selected-person sharing.
• Add an idempotent schema bridge for TypeScript and Rust generation until the pinned CLI schema includes the field.
• Cover absent, empty, and non-empty Rust serialization and deserialization.
• Make both Rust CLI snapshot scripts compatible with stock macOS Bash 3 while preserving ordered snapshot output and failure behavior.

Validation

• cargo +nightly-2026-04-14 fmt --check
• COPILOT_SKIP_CLI_DOWNLOAD=1 cargo clippy --all-features --all-targets -- -D warnings
• Rust library tests: 241 passed
• Generated API tests: 12 passed, including all collaborator wire shapes
• Shared codegen utility tests: 6 passed
• Generated Node type parsing and focused formatting checks passed
• Both snapshot scripts passed syntax and functional checks under macOS /bin/bash 3.2
• The checksum script rejects a missing platform entry without writing a partial snapshot

The pinned @github/copilot@1.0.83-1 tarball was unavailable from the local registry, so a full codegen rerun could not complete locally. The same missing runtime stopped the bundled-runtime-dependent cli_resolution_test::stale_env_override_falls_through after the preceding Rust suites passed.

[Copilot]

Copilot review overview

🟡 Changes recommended

Generated RPC documentation still describes the obsolete repository-reader visibility contract.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review tier: Balanced
Findings: 1

New issues introduced by this change (1)

Severity	Finding
	scripts/​codegen/​utils.ts — This bridge updates only the definition-level status text, so regenerated public APIs still publish…

What changed in this PR

Adds selected-collaborator session visibility support to generated TypeScript and Rust RPC APIs.

Changes:

• Adds an interim schema bridge and generated collaborator fields.
• Tests absent, empty, and populated Rust wire shapes.
• Makes Rust snapshot scripts Bash 3 compatible.

File	Description
scripts/​codegen/​utils.ts	Adds the visibility schema bridge.
scripts/​codegen/​typescript.ts	Applies the bridge to TypeScript generation.
scripts/​codegen/​rust.ts	Applies the bridge to Rust generation.
nodejs/​src/​generated/​rpc.ts	Exposes collaborator fields in TypeScript.
nodejs/​test/​shared-codegen.test.ts	Tests schema augmentation.
rust/​src/​generated/​api_types.rs	Exposes collaborator fields in Rust.
rust/​tests/​api_types_test.rs	Tests collaborator wire shapes.
rust/​tests/​e2e/​rpc_session_state_extras.rs	Updates an existing request literal.
rust/​scripts/​snapshot-bundled-cli-version.sh	Replaces associative hash storage.
rust/​scripts/​snapshot-bundled-in-process-version.sh	Replaces associative integrity storage.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[SteveSandersonMS]

I moved this PR to draft for tracking. Thanks for pushing on richer session-sharing controls — that is a reasonable product direction. The pinned runtime/schema does not currently expose collaboratorLogins, and the SDK should not hand-patch a capability that has not shipped. Once the runtime field lands, please mark this ready for review after replacing the bridge with normal codegen across all six SDKs, plus the corresponding E2E coverage and concise docs.

[pierceboggan]

Closing at the author's request; continuing local branch-only validation.