test/onboard-selection.test.ts (1)

505-505: ⚡ Quick win

Prefix unused catch binding with underscore.

The error variable e is bound but never used in the catch block. As per coding guidelines, unused variables should be prefixed with underscore. Either use catch (_e) or omit the binding entirely with catch { }.

♻️ Proposed fix

-  } catch (e) {
+  } catch {
     // Downstream paths (model probe, gateway, etc.) are not mocked here; we
     // only care about the menu-build log emitted before any failure.
   }

As per coding guidelines: Unused variables should be prefixed with underscore (_) following Biome conventions.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@test/onboard-selection.test.ts` at line 505, In the catch block in
test/onboard-selection.test.ts where the code currently uses "catch (e)" (near
the end of the test), rename the unused error binding to "_e" or remove the
binding entirely (use "catch { }") to comply with Biome conventions for unused
variables; update the catch clause around the failing try/catch in that test so
the unused exception variable is either prefixed with an underscore or omitted.

docs/get-started/quickstart.md (1)

258-260: ⚡ Quick win

Multiple sentences on the same line.

The documentation style guide requires one sentence per line to make diffs readable. Lines 258, 259, and 260 each contain multiple sentences.

📝 Split into one sentence per line

-- **Local NVIDIA NIM** appears when `NEMOCLAW_EXPERIMENTAL=1` is set and the host has a NIM-capable GPU. NemoClaw pulls and manages a NIM container.
-- **Local vLLM (already running)** appears whenever NemoClaw detects a vLLM server on `localhost:8000`. No flag is required for the menu entry. NemoClaw auto-detects the loaded model.
-- **Local vLLM (managed install/start)** requires `NEMOCLAW_EXPERIMENTAL=1` or `NEMOCLAW_PROVIDER=install-vllm`. NemoClaw pulls and starts a vLLM container on supported DGX Spark, DGX Station, and Linux NVIDIA GPU hosts.
+- **Local NVIDIA NIM** appears when `NEMOCLAW_EXPERIMENTAL=1` is set and the host has a NIM-capable GPU.
+  NemoClaw pulls and manages a NIM container.
+- **Local vLLM (already running)** appears whenever NemoClaw detects a vLLM server on `localhost:8000`.
+  No flag is required for the menu entry.
+  NemoClaw auto-detects the loaded model.
+- **Local vLLM (managed install/start)** requires `NEMOCLAW_EXPERIMENTAL=1` or `NEMOCLAW_PROVIDER=install-vllm`.
+  NemoClaw pulls and starts a vLLM container on supported DGX Spark, DGX Station, and Linux NVIDIA GPU hosts.

As per coding guidelines, "One sentence per line in source (makes diffs readable). Flag paragraphs where multiple sentences appear on the same line."

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/get-started/quickstart.md` around lines 258 - 260, The three bullet
entries for "**Local NVIDIA NIM**", "**Local vLLM (already running)**", and
"**Local vLLM (managed install/start)**" contain multiple sentences on the same
line; edit the quickstart.md bullets so each sentence is on its own line (e.g.,
break the NIM bullet into one line for the condition and a second line for the
behavior, and do the same for both vLLM bullets), keeping the exact wording but
ensuring one sentence per source line to satisfy the "one sentence per line" doc
style.

docs/security/best-practices.md (1)

501-501: ⚡ Quick win

Multiple sentences on the same line.

The documentation style guide requires one sentence per line to make diffs readable. Line 501 contains two sentences.

📝 Split into one sentence per line

-The `NEMOCLAW_EXPERIMENTAL=1` environment variable gates local NVIDIA NIM and the managed vLLM install/start path. An already-running vLLM server on `localhost:8000` is offered in the menu without a flag, because selecting it is an explicit user action.
+The `NEMOCLAW_EXPERIMENTAL=1` environment variable gates local NVIDIA NIM and the managed vLLM install/start path.
+An already-running vLLM server on `localhost:8000` is offered in the menu without a flag, because selecting it is an explicit user action.

As per coding guidelines, "One sentence per line in source (makes diffs readable). Flag paragraphs where multiple sentences appear on the same line."

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/security/best-practices.md` at line 501, The sentence pair describing
NEMOCLAW_EXPERIMENTAL and the vLLM server on localhost:8000 must be split so
each sentence is on its own line; edit the paragraph containing "The
`NEMOCLAW_EXPERIMENTAL=1` environment variable..." and place the first sentence
on one line and the second sentence ("An already-running vLLM server on
`localhost:8000` is offered...") on the following line so the file follows the
one-sentence-per-line docstyle.

docs/security/best-practices.mdx (2)

464-464: ⚡ Quick win

Multiple sentences on the same line.

The documentation style guide requires one sentence per line to make diffs readable. Line 464 contains two sentences.

📝 Split into one sentence per line

-The `NEMOCLAW_EXPERIMENTAL=1` environment variable gates local NVIDIA NIM and the managed vLLM install/start path. An already-running vLLM server on `localhost:8000` is offered in the menu without a flag, because selecting it is an explicit user action.
+The `NEMOCLAW_EXPERIMENTAL=1` environment variable gates local NVIDIA NIM and the managed vLLM install/start path.
+An already-running vLLM server on `localhost:8000` is offered in the menu without a flag, because selecting it is an explicit user action.

As per coding guidelines, "One sentence per line in source (makes diffs readable). Flag paragraphs where multiple sentences appear on the same line."

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/security/best-practices.mdx` at line 464, The line containing the text
about NEMOCLAW_EXPERIMENTAL and the vLLM server (mentions
NEMOCLAW_EXPERIMENTAL=1 and localhost:8000) has multiple sentences on one line;
split that line into separate lines so each sentence occupies its own line
(e.g., one line for "The `NEMOCLAW_EXPERIMENTAL=1` environment variable gates
local NVIDIA NIM and the managed vLLM install/start path." and a separate line
for "An already-running vLLM server on `localhost:8000` is offered in the menu
without a flag, because selecting it is an explicit user action."). Ensure the
surrounding paragraph formatting remains unchanged and follow the
one-sentence-per-line rule used across the docs.

---

4-4: ⚡ Quick win

Colon in title.

The documentation style guide prohibits colons in section titles. The title uses a colon to separate the main heading from the subtitle.

📝 Remove colon from title

Consider one of these alternatives:

-title: "NemoClaw Security Best Practices: Controls, Risks, and Posture Profiles"
+title: "NemoClaw Security Best Practices for Controls, Risks, and Posture Profiles"

Or:

-title: "NemoClaw Security Best Practices: Controls, Risks, and Posture Profiles"
+title: "NemoClaw Security Best Practices – Controls, Risks, and Posture Profiles"

As per coding guidelines, "No colons in titles. Flag 'Inference: Cloud and Local' — should be 'Cloud and Local Inference.'"

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/security/best-practices.mdx` at line 4, Change the page title string
"NemoClaw Security Best Practices: Controls, Risks, and Posture Profiles" to
remove the colon per the style guide—e.g., "NemoClaw Security Best Practices —
Controls, Risks and Posture Profiles" or split into a main title and subtitle
such as "NemoClaw Security Best Practices" with a subtitle "Controls, Risks and
Posture Profiles"; update the title value in the document header (the title
string shown in the diff) and ensure any references or TOC entries use the new
punctuation-free form (also fix similar instances like "Inference: Cloud and
Local" to "Cloud and Local Inference").

docs/get-started/quickstart.mdx (1)

236-238: ⚡ Quick win

Multiple sentences on the same line.

The documentation style guide requires one sentence per line to make diffs readable. Lines 236, 237, and 238 each contain multiple sentences.

📝 Split into one sentence per line

-- **Local NVIDIA NIM** appears when `NEMOCLAW_EXPERIMENTAL=1` is set and the host has a NIM-capable GPU. NemoClaw pulls and manages a NIM container.
-- **Local vLLM (already running)** appears whenever NemoClaw detects a vLLM server on `localhost:8000`. No flag is required for the menu entry. NemoClaw auto-detects the loaded model.
-- **Local vLLM (managed install/start)** requires `NEMOCLAW_EXPERIMENTAL=1` or `NEMOCLAW_PROVIDER=install-vllm`. NemoClaw pulls and starts a vLLM container on supported DGX Spark, DGX Station, and Linux NVIDIA GPU hosts.
+- **Local NVIDIA NIM** appears when `NEMOCLAW_EXPERIMENTAL=1` is set and the host has a NIM-capable GPU.
+  NemoClaw pulls and manages a NIM container.
+- **Local vLLM (already running)** appears whenever NemoClaw detects a vLLM server on `localhost:8000`.
+  No flag is required for the menu entry.
+  NemoClaw auto-detects the loaded model.
+- **Local vLLM (managed install/start)** requires `NEMOCLAW_EXPERIMENTAL=1` or `NEMOCLAW_PROVIDER=install-vllm`.
+  NemoClaw pulls and starts a vLLM container on supported DGX Spark, DGX Station, and Linux NVIDIA GPU hosts.

As per coding guidelines, "One sentence per line in source (makes diffs readable). Flag paragraphs where multiple sentences appear on the same line."

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/get-started/quickstart.mdx` around lines 236 - 238, The three bullet
lines for "**Local NVIDIA NIM**", "**Local vLLM (already running)**", and
"**Local vLLM (managed install/start)**" contain multiple sentences on the same
line; split each sentence onto its own line in docs/get-started/quickstart.mdx
so each sentence becomes a separate line (e.g., break the bullet sentences after
the period), preserving the exact wording and bullet structure and ensuring no
other text is changed.