Skip to content

docs(teleop): correct broker key and identity guidance - #4087

Merged
paul-nechifor merged 3 commits into
mainfrom
ruthwik/docs/hosted-broker-guidance
Sep 16, 2026
Merged

paul-nechifor merged 3 commits into
mainfrom
ruthwik/docs/hosted-broker-guidance

Conversation

@ruthwikdasyam

Copy link
Copy Markdown
Contributor

Contribution path

Small follow-up to #4063 addressing Greptile feedback.

Problem

Key-creation instructions are stale; the guide incorrectly claims a blueprint-default robot ID.

Solution

Point to Console API keys, document key-derived identity, and refresh E2E credential guidance.

How to Test

TRANSPORTS__BROKER__API_KEY=dimos_sk_... dimos run teleop-hosted-go2-transport

Pre-commit passed; lfs_check skipped with approval for existing local datasets.

AI assistance

Checklist

  • I have read and approved the CLA.

@codecov

codecov Bot commented Sep 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main    #4087      +/-   ##
==========================================
- Coverage   78.89%   78.89%   -0.01%     
==========================================
  Files        1468     1468              
  Lines      138785   138785              
  Branches    11934    11934              
==========================================
- Hits       109500   109492       -8     
- Misses      25889    25896       +7     
- Partials     3396     3397       +1     
Components Coverage Δ
Tests 94.81% <ø> (-0.01%) ⬇️
Flag Coverage Δ
OS-ubuntu-24.04-arm 74.84% <ø> (ø)
OS-ubuntu-latest 75.54% <ø> (-0.01%) ⬇️
Py-3.10 75.31% <ø> (-0.01%) ⬇️
Py-3.11 75.52% <ø> (-0.01%) ⬇️
Py-3.12 75.52% <ø> (ø)
SelfHosted-Large 30.75% <ø> (+<0.01%) ⬆️
SelfHosted-Linux 36.29% <ø> (-0.01%) ⬇️
SelfHosted-macOS 35.62% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...os/protocol/pubsub/impl/webrtc/providers/broker.py 34.03% <ø> (ø)
...rotocol/pubsub/impl/webrtc/providers/cloudflare.py 32.62% <ø> (ø)

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@spomichter
spomichter force-pushed the ruthwik/docs/hosted-broker-guidance branch from 83334c6 to 48be81f Compare September 13, 2026 02:43
@ruthwikdasyam
ruthwikdasyam marked this pull request as ready for review September 13, 2026 04:50
@github-actions github-actions Bot added the ready-to-merge Required CI checks have passed on this PR label Sep 13, 2026
@greptile-apps

greptile-apps Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

Safe to merge; no actionable issues were established in the changed broker, E2E, or documentation guidance.

What we checked:

  • Authored the hosted broker default identity validation script. T-Rex
  • Archived the before-output state from the validation run for comparison. T-Rex
  • Archived the after-output state from the validation run for comparison. T-Rex
  • Documented the live end-to-end test output used to validate the contract during the run. T-Rex

Summary

  • Updated hosted teleoperation guidance aligns with the broker's default identity behavior and the live E2E credential gate.

T-Rex validation blocked

  • Live broker verification requires TELEOP_API_KEY and TELEOP_OPERATOR_TOKEN, which were not available. The local virtual-environment Python launcher also points to a missing interpreter.

Reviews (1) · Last reviewed commit: "docs(teleop): correct broker key and ide..."

@github-actions github-actions Bot removed the ready-to-merge Required CI checks have passed on this PR label Sep 13, 2026
@ruthwikdasyam
ruthwikdasyam force-pushed the ruthwik/docs/hosted-broker-guidance branch from 5f15da8 to 3c1f035 Compare September 13, 2026 17:54
@github-actions github-actions Bot added the ready-to-merge Required CI checks have passed on this PR label Sep 13, 2026
@paul-nechifor
paul-nechifor added this pull request to the merge queue Sep 16, 2026
Merged via the queue into main with commit 3852bc4 Sep 16, 2026
33 of 35 checks passed
@paul-nechifor
paul-nechifor deleted the ruthwik/docs/hosted-broker-guidance branch September 16, 2026 06:38
@github-actions

Copy link
Copy Markdown
Contributor

Successfully created backport PR for release/0.0.14:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport release/0.0.14 ready-to-merge Required CI checks have passed on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants