Disconnect when the oob program quits.

[lotusl-code]

Split from #1201. When the oob host program is terminated, exit connection on the headset.

Summary by CodeRabbit

• Bug Fixes
  • Stale teleoperation tabs are now closed on the selected headset during shutdown. If closing a tab fails, shutdown continues.

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

• Configuration used: Repository: NVIDIA/IsaacCapture/.coderabbit.yaml
• Review profile: CHILL
• Plan: Enterprise
• Run ID: 81280617-beb9-4920-a63d-d379877160fa

---

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: NVIDIA/IsaacCapture/.coderabbit.yaml
• Review profile: CHILL
• Plan: Enterprise
• Run ID: 387a638e-97e9-4e51-9e5c-d5f25571bbd7

---

📥 Commits

Reviewing files that changed from the base of the PR and between 3641911 and 2b3ec0e.

---

📒 Files selected for processing (2)

• src/python/isaaccapture/cloudxr/oob_teleop_lifecycle.py
• tests/python/core/cloudxr/test_oob_teleop_lifecycle.py

---

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.

---

---

📝 Walkthrough

Walkthrough

OobLifecycle now closes stale teleop tabs during shutdown. It performs this step after stopping the monitor and before cleaning up owned ADB mappings. If tab closure fails, the lifecycle logs the failure and continues shutdown. The cable-flap recovery test now expects one stale-tab close during shutdown.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

---

Merge Risk: ⚪ Minimal · up to 2b3ec

On the supported stop path, stale teleop tabs are closed before owned ADB mappings are cleaned up, and the inspected cancellation flow does not interrupt that sequence. No actionable merge risk remains.

Pre-merge checks | 4 | 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 60.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 5 functions across 2 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly describes the main change: disconnect the headset when the OOB program exits.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

---

---

✨ Finishing Touches 💡 2

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

---

🛠️ Fix failing CI checks 💡

• Commit to this branch
• Create a new PR

---

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

---

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

_{Comment @coderabbitai help to get the list of available commands.}