fix(plugin): tag registration banner lines with gateway source tag

[laitingsheng]

Summary

Tags every line of the NemoClaw plugin's registration banner with the [gateway] source prefix on stderr, matching the tag already carried by other gateway log output. Previously the banner printed as raw, untagged lines, so nemoclaw <name> logs --follow showed banner lines with no indication of which source produced them.

Related Issue

Fixes #7314

Changes

• nemoclaw/src/index.ts: prefix each registration banner line written to stderr with [gateway] , matching the tag already present on other gateway log output. The banner still bypasses api.logger entirely (unchanged) — routing it through api.logger was reverted by fix(plugin): route registration banner to stderr #6480 (fixing [Ubuntu 26.04][CLI&UX] nemoclaw <name> agent writes "[plugins] NemoClaw registered" init box to stdout in non-json mode, polluting piped output #5654) because it leaked banner text onto machine-readable stdout during non-JSON agent passthrough; this change keeps that stdout-isolation fix intact and only adds the tag at the existing stderr write site. The bounding blank-spacer lines around the box stay untagged.
• nemoclaw/src/register.test.ts: add a test asserting every non-blank banner line written to stderr carries the [gateway] prefix.

Type of Change

• Code change (feature, bug fix, or refactor)
• Code change with doc updates
• Doc only (prose changes, no code sample modifications)
• Doc only (includes code sample changes)

Quality Gates

• Tests added or updated for changed behavior
• Existing tests cover changed behavior — justification:
• Tests not applicable — justification:
• Docs updated for user-facing behavior changes
• Docs not applicable — justification: no doc page shows the banner's raw log-output format or a [gateway]-tagged example line; the logs --follow command reference itself is unchanged.
• Sensitive paths changed (security, policy, credentials, preflight, onboarding, inference, runner, sandbox, or messaging)
• Sensitive-path review completed or maintainer-approved waiver recorded — reviewer/approval link/justification:
• Non-success, skipped, or missing CI check accepted by maintainer — check name, approval link, and follow-up issue:

Documentation Writer Review

• Documentation writer subagent reviewed the completed changes
• Result: docs-not-needed
• Evidence: The registration banner remains on stderr; the diff only adds a [gateway] source tag to diagnostic banner lines and changes no documented command, configuration, or supported behavior.
• Agent: Codex Desktop

DGX Station Hardware Evidence

• Tested on DGX Station
• Tested commit:
• Station profile/scenario:
• Result:
• Supporting evidence:

Verification

• PR description includes a Signed-off-by: line and every commit appears as Verified in GitHub
• Normal pre-commit, commit-msg, and pre-push hooks passed, or npm run check:diff passed when hooks were skipped or unavailable
• Targeted behavior tests pass for the current change set, or tests are marked not applicable above — command/result or justification: npm test -- src/register.test.ts (nemoclaw) 25/25; npx vitest run --project cli src/lib/actions/sandbox/agent/passthrough.test.ts 38/38; npm test -- src/banner.test.ts (nemoclaw) 3/3; npm --prefix nemoclaw run typecheck clean.
• Applicable broad gate passed — npm test for broad runtime/test-harness changes; npm run check for repo-wide validation/coverage changes — command/result:
• Quality Gates section completed with required justifications or waivers
• No secrets, API keys, or credentials committed
• npm run docs builds without warnings (doc changes only)
• Doc pages follow the style guide (doc changes only)
• New doc pages include SPDX header and frontmatter (new pages only)

---

Signed-off-by: Tinson Lai tinsonl@nvidia.com

Summary by CodeRabbit

• Bug Fixes
  • Updated the startup/registration banner output to prefix every non-empty stderr line with [gateway] for clearer source identification.
• Tests
  • Added an assertion to confirm the [gateway] prefix is present on every non-empty registration banner line.

[coderabbitai]

📝 Walkthrough

Walkthrough

The startup registration banner now prefixes each stderr line with [gateway] . A registration test verifies that all non-empty banner lines include the prefix.

Changes

Gateway banner source tagging

Layer / File(s)	Summary
Prefix registration banner output nemoclaw/src/index.ts, nemoclaw/src/register.test.ts	Startup banner lines are written with the [gateway] prefix, and the registration test verifies every non-empty stderr line begins with that prefix.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The code and test changes satisfy #7314 by prefixing non-blank banner lines with [gateway].
Out of Scope Changes check	✅ Passed	The diff is limited to the banner prefix fix and its test, with no unrelated changes evident.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change: tagging the plugin registration banner with the gateway source tag.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/tag-gateway-banner-log-lines

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with 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.

Inline comments:
In `@nemoclaw/src/register.test.ts`:
- Around line 170-181: Update the test “tags every registration banner line with
the gateway source tag (`#7314`)” to assert the raw stderrOutput() begins with one
newline and ends with two newlines before filtering blank lines. Retain the
existing non-empty banner line count and [gateway] prefix assertions.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 1c41c7eb-ae77-46a5-b958-b12b5155a5b3

📥 Commits

Reviewing files that changed from the base of the PR and between d5fa194 and 34dddc3.

📒 Files selected for processing (2)

• nemoclaw/src/index.ts
• nemoclaw/src/register.test.ts

[github-code-quality]

Code Coverage Overview

Languages: TypeScript

TypeScript / code-coverage/plugin

The overall coverage in commit bc9d12f in the fix/tag-gateway-bann... branch remains at 96%, unchanged from commit e84f5ee in the main branch.

TypeScript / code-coverage/cli

The overall coverage in commit bc9d12f in the fix/tag-gateway-bann... branch remains at 80%, unchanged from commit e84f5ee in the main branch.

Show a code coverage summary of the most impacted files.

File	main e84f5ee	fix/tag-gateway-bann... bc9d12f	+/-
src/lib/domain/.../connect-env.ts	97%	89%	-8%
src/lib/state/m...-acquisition.ts	87%	84%	-3%
src/lib/state/g...way-registry.ts	95%	94%	-1%
src/lib/sandbox...rce-identity.ts	87%	87%	0%
src/lib/state/m...ock-identity.ts	95%	95%	0%
src/lib/state/m...lock-storage.ts	97%	97%	0%

---

_{Updated July 23, 2026 23:54 UTC}

[github-actions]

PR Review Advisor — Informational

Advisor assessment: Informational / high confidence
Next action: No advisor follow-up needed.
Findings: 0 blockers · 0 warnings · 0 suggestions
Status: No actionable findings remain in the canonical review ledger.

Model lanes

• GPT-5.6 Terra (primary): Completed · high confidence · 0 blockers · 0 warnings · 0 suggestions
• Nemotron 3 Ultra (second opinion): Completed · high confidence · 0 blockers · 0 warnings · 0 suggestions
• Model comparison: normalized findings match; normalized E2E selections match; severity counts match.

Nemotron output stays in workflow artifacts and does not change the assessment above.

E2E guidance

Advisory only. E2E / PR Gate selects and runs jobs independently.

Recommended E2E: None

Workflow run details

This automated review informs maintainers. Warnings and suggestions do not require a response. A maintainer decides whether to merge.

[prekshivyas]

Reviewed exact PR SHA 9140178ffb038224100c4064c4be77403c34b1ef. The focused plugin change tags every nonblank banner line while preserving the boundary newlines. The regression test covers that rendering contract. Current CI, DCO and signatures, CodeRabbit, and repository merge gates pass.