-
Notifications
You must be signed in to change notification settings - Fork 2.8k
docs: fill manage sandbox command gaps (Fixes #3680) #4338
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -41,7 +41,9 @@ Telegram uses a bot token from [BotFather](https://t.me/BotFather). | |||||||||
| Open Telegram, send `/newbot` to [@BotFather](https://t.me/BotFather), follow the prompts, and copy the token. | ||||||||||
| For Telegram group chats, disable privacy mode before testing group replies: in @BotFather, run `/setprivacy`, choose the bot, then choose **Disable**. | ||||||||||
| After changing privacy mode, remove the bot from each Telegram group and add it back so Telegram applies the new delivery setting to that group. | ||||||||||
| `TELEGRAM_ALLOWED_IDS` is a comma-separated list of Telegram user IDs for DM access. | ||||||||||
| `TELEGRAM_ALLOWED_IDS` is a comma-separated list of Telegram user or private-chat IDs for DM access. | ||||||||||
| For compatibility with older QA scripts, NemoClaw also treats `TELEGRAM_AUTHORIZED_CHAT_IDS` and `TELEGRAM_CHAT_ID` as aliases, but new automation should use `TELEGRAM_ALLOWED_IDS`. | ||||||||||
| Keep these aliases until QA automation and public repro templates have stopped exporting them for at least one full release. | ||||||||||
| Group chats stay open by default so rebuilt sandboxes do not silently drop Telegram group messages because of an empty group allowlist. | ||||||||||
| Set `TELEGRAM_REQUIRE_MENTION=1` to make the bot reply in Telegram groups only when users mention it. | ||||||||||
| Pairing and `TELEGRAM_ALLOWED_IDS` still govern direct messages. | ||||||||||
|
|
@@ -158,6 +160,8 @@ If applying the preset fails, NemoClaw warns and tells you to re-apply manually | |||||||||
| Choose the rebuild so the running sandbox image picks up the new channel. | ||||||||||
| For Telegram, Discord, and Slack, `channels add` also checks the rebuilt runtime for the selected bridge and reports startup, credential, or missing-plugin warnings before returning. | ||||||||||
| If you need optional channel settings such as `TELEGRAM_ALLOWED_IDS`, `TELEGRAM_REQUIRE_MENTION`, `DISCORD_SERVER_ID`, `DISCORD_USER_ID`, `DISCORD_REQUIRE_MENTION`, `SLACK_ALLOWED_USERS`, or `SLACK_ALLOWED_CHANNELS`, export them before the rebuild starts. | ||||||||||
| Telegram Bot API `sendMessage` calls prove outbound delivery from the bot; to test inbound agent replies, send a message from the Telegram client as an allowed user. | ||||||||||
| For a repeatable live Telegram reply check, run `test/e2e/test-messaging-providers.sh` with `TELEGRAM_BOT_TOKEN_REAL`, `TELEGRAM_AUTHORIZED_CHAT_IDS` or `TELEGRAM_CHAT_ID`, and `NEMOCLAW_TELEGRAM_INBOUND_REPLY_E2E=1`. | ||||||||||
| If you defer the rebuild, apply the change later: | ||||||||||
|
Comment on lines
+164
to
165
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Prefer This example currently spotlights legacy variables, which conflicts with the “new automation should use Suggested doc tweak-For a repeatable live Telegram reply check, run `test/e2e/test-messaging-providers.sh` with `TELEGRAM_BOT_TOKEN_REAL`, `TELEGRAM_AUTHORIZED_CHAT_IDS` or `TELEGRAM_CHAT_ID`, and `NEMOCLAW_TELEGRAM_INBOUND_REPLY_E2E=1`.
+For a repeatable live Telegram reply check, run `test/e2e/test-messaging-providers.sh` with `TELEGRAM_BOT_TOKEN_REAL`, `TELEGRAM_ALLOWED_IDS` (preferred; legacy aliases: `TELEGRAM_AUTHORIZED_CHAT_IDS` or `TELEGRAM_CHAT_ID`), and `NEMOCLAW_TELEGRAM_INBOUND_REPLY_E2E=1`.📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||
|
|
||||||||||
| ```console | ||||||||||
|
|
||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🌐 Web query:
For NVIDIA-hosted NemoClaw/OpenAI-compatible inference, what is the correct base URL to configure in router/provider examples: https://integrate.api.nvidia.com/v1 or https://inference-api.nvidia.com ?💡 Result:
Correct base URL for NVIDIA-hosted NemoClaw/OpenAI-compatible inference: https://integrate.api.nvidia.com/v1. NVIDIA’s NemoClaw inference documentation states that for hosted models on integrate.api.nvidia.com, NemoClaw sets the base URL to https://integrate.api.nvidia.com/v1 (and the APIs are OpenAI-compatible under the /v1/chat/completions route). [1][2] The alternative base URL you mentioned (https://inference-api.nvidia.com) is not what NemoClaw’s official docs specify for hosted NVIDIA inference. [1][2]
Citations:
🏁 Script executed:
Repository: NVIDIA/NemoClaw
Length of output: 148
🏁 Script executed:
Repository: NVIDIA/NemoClaw
Length of output: 3001
Fix
api_basemismatch in Model Router example (NVIDIA-hosted endpoints)The doc states NVIDIA-hosted endpoints use
https://integrate.api.nvidia.com/v1(Line 62), but the router example useshttps://inference-api.nvidia.comfor bothnanoandsuper(Lines 91-96), which will break copy/paste setup.Proposed doc fix
models: - name: nano litellm_model: "openai/nvidia/nvidia/Nemotron-3-Nano-30B-A3B" cost_per_m_input_tokens: 0.05 - api_base: "https://inference-api.nvidia.com" + api_base: "https://integrate.api.nvidia.com/v1" - name: super litellm_model: "openai/nvidia/nvidia/nemotron-3-super-v3" cost_per_m_input_tokens: 0.10 - api_base: "https://inference-api.nvidia.com" + api_base: "https://integrate.api.nvidia.com/v1"🤖 Prompt for AI Agents