Skip to content

feat: add coding-agent sidecar integrations#63

Draft
willkill07 wants to merge 3 commits intoNVIDIA:mainfrom
willkill07:wkk_coding-agent-sidecar-integrations
Draft

feat: add coding-agent sidecar integrations#63
willkill07 wants to merge 3 commits intoNVIDIA:mainfrom
willkill07:wkk_coding-agent-sidecar-integrations

Conversation

@willkill07
Copy link
Copy Markdown
Member

@willkill07 willkill07 commented May 5, 2026

Overview

Adds a Rust nemo-flow-sidecar binary for coding-agent observability plus installable hook integrations and setup documentation for Claude Code, Codex, and Cursor. The sidecar accepts canonical agent hook payloads, normalizes them internally, and provides passthrough OpenAI/Anthropic gateway routes for LLM lifecycle observability.

  • I confirm this contribution is my own work, or I have the right to submit it under this project's license.
  • I searched existing issues and open pull requests, and this does not duplicate existing work.

Details

  • Adds the nemo-flow-sidecar crate with /hooks/claude-code, /hooks/codex, and /hooks/cursor endpoints.
  • Adds passthrough gateway routes for OpenAI Responses, OpenAI Chat Completions, Anthropic Messages, Anthropic token counting, and model listing.
  • Adds sidecar session management for agent, subagent, tool, LLM, ATIF, and OpenInference lifecycle handling.
  • Adds nemo-flow-sidecar install claude-code|codex|cursor and hook-forward support with safe merge, backup, --dry-run, --print, metadata, ATIF, OpenInference, plugin config, and gateway mode options.
  • Adds installable Claude Code and Codex hook/plugin bundles plus a Cursor hook bundle under integrations/coding-agents/.
  • Adds end-to-end sidecar docs and self-contained per-agent README setup guides.
  • Updates Rust attribution data for the new sidecar dependencies.

Validation run locally:

  • cargo test -p nemo-flow-sidecar
  • just test-rust
  • just docs
  • cargo clippy -p nemo-flow-sidecar --all-targets -- -D warnings
  • Commit pre-checks passed with lychee skipped because it failed on unrelated generated Rust API doc links in docs/reference/api/rust/_generated/....

Where should the reviewer start?

Start with crates/sidecar/src/server.rs for the external HTTP surface, then crates/sidecar/src/installer.rs for the install/merge behavior, and integrations/coding-agents/README.md for the packaged integration contract.

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • Closes NMF-57
  • Closes NMF-58
  • Closes NMF-59
  • Closes NMF-60

Signed-off-by: Will Killian <wkillian@nvidia.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 5, 2026

Important

Review skipped

Draft detected.

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: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: 70655959-e8d4-4602-9ade-138abee6b6b7

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
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added size:XXL PR is very large feat PR introduces new feature or functionality lang:rust PR changes/introduces Rust code labels May 5, 2026
@willkill07 willkill07 self-assigned this May 5, 2026
@willkill07 willkill07 added this to the 0.2.0 milestone May 5, 2026
willkill07 added 2 commits May 5, 2026 16:06
Signed-off-by: Will Killian <wkillian@nvidia.com>
Signed-off-by: Will Killian <wkillian@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat PR introduces new feature or functionality lang:rust PR changes/introduces Rust code size:XXL PR is very large

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant