chore: manually forward-merge release/0.1 into main - #156
Conversation
#### Overview Slim the Python adapter dependency boundary for the `release/0.1` line without changing adapter behavior. - `nemo-fabric` remains a metadata-only package that unconditionally installs the exact matching `nemo-fabric-runtime`. - Root harness extras delegate to the matching adapter package's `harness` extra. - Bare adapter distributions contain only adapter-owned runtime dependencies. - Every adapter provides `harness` and `full`; only Deep Agents and Hermes provide the Python `relay` extra. Claude and Codex continue to use the separately installed `nemo-relay` CLI. - Wrapped harness packages required by repository tests remain in the non-published `adapter-tests` dependency group. #### Details The supported install modes are now explicit: - `nemo-fabric[<harness>]`: Runtime, adapter, and supported harness dependencies in one environment. - `nemo-fabric` plus a bare adapter package: Runtime and adapter with an existing host-managed harness. - `nemo-fabric-adapters-<adapter>[harness]`: Adapter and supported harness without Runtime, for a separate adapter interpreter. - Bare `nemo-fabric-adapters-<adapter>`: Adapter-owned dependencies only. - Adapter `full`: All package-installable dependencies; for Claude and Codex this equals `harness` because Relay is an external CLI. The README, package description, install guide, adapter guides, Harbor examples, CI, consumer skill, and maintainer packaging skills now describe the same matrix. Metadata tests guard the root-to-adapter delegation and the exact adapter dependency boundaries. #### Validation - Focused metadata, README, and Harbor checks: `55 passed` - Full Python suite: `529 passed, 15 skipped` - Latest upstream Hermes streaming coverage plus metadata checks: `21 passed` - Root and all adapter lockfiles: `uv lock --check` - `just build-python` - `cargo check -p fabric-python --locked` - `just docs` (all checks passed; unauthenticated redirect check skipped with the expected warning) - `just wheels`, followed by root and adapter wheel `METADATA` inspection - License diff against `upstream/release/0.1`: no Rust or Python additions, removals, or changes - Full `pre-commit run --all-files` - `just --fmt --check` - `git diff --check` #### Where should the reviewer start? Start with `pyproject.toml`, the four adapter `pyproject.toml` files, and `tests/adapters/test_adapter_package_metadata.py`. Then review `docs/getting-started/install.mdx`, `.github/workflows/ci_python.yml`, and `.agents/skills/contribute-adapter/SKILL.md`. The branch is rebased onto the latest `release/0.1`. It includes and preserves the adapter configuration changes from NVIDIA#117. #### Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to) - Closes [FABRIC-125](https://linear.app/nvidia/issue/FABRIC-125/slim-adapter-runtime-dependencies) - Relates to NVIDIA#117 - [x] I confirm this contribution is my own work, or I have the right to submit it under this project's license. - [x] I searched existing issues and open pull requests, and this does not duplicate existing work. ## Summary by CodeRabbit * **Documentation** * Clarified installation options for runtime, adapters, harnesses, and Relay integrations. * Added supported Python version guidance and separate-environment setup instructions. * Updated Hermes, Claude, Codex, and Deep Agents quick starts and deployment scenarios. * Improved Harbor setup and command examples, including version-alignment guidance. * **Bug Fixes** * Improved Hermes configuration writing when YAML support is unavailable. * Clarified Relay CLI requirements and adapter-specific integration behavior. * **Tests** * Added validation for package dependency boundaries, installation metadata, and adapter configuration behavior. Authors: - Zhongxuan (Daniel) Wang (https://github.com/zhongxuanwang-nv) Approvers: - Anuradha Karuppiah (https://github.com/AnuradhaKaruppiah) - David Gardner (https://github.com/dagardner-nv) URL: NVIDIA#105 Signed-off-by: Zhongxuan (Daniel) Wang <daniewang@nvidia.com>
Signed-off-by: Anuradha Karuppiah <26330987+AnuradhaKaruppiah@users.noreply.github.com>
Signed-off-by: Zhongxuan Wang <daniewang@nvidia.com>
This reverts commit 557b629. Signed-off-by: Ajay Thorve <athorve@nvidia.com>
Signed-off-by: Ajay Thorve <athorve@nvidia.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Enterprise Run ID: 📒 Files selected for processing (6)
📜 Recent review details⏰ Context from checks skipped due to timeout. (2)
🧰 Additional context used📓 Path-based instructions (29)**/*.{md,rst}📄 CodeRabbit inference engine (.agents/skills/contribute-api/SKILL.md)
Files:
{README.md,docs/**/*.{md,mdx,yml},examples/**/*.{md,mdx,yml}}📄 CodeRabbit inference engine (.agents/skills/contribute-docs/SKILL.md)
Files:
{docs/**/*.{md,mdx,yml},examples/**/*.{md,mdx,yml}}📄 CodeRabbit inference engine (.agents/skills/contribute-docs/SKILL.md)
Files:
**/*📄 CodeRabbit inference engine (.agents/skills/karpathy-guidelines/SKILL.md)
Files:
**/*.{md,mdx,rst}📄 CodeRabbit inference engine (.agents/skills/review-doc-style/assets/nvidia-style-guide.md)
Files:
**/*.{md,rst,txt,adoc}📄 CodeRabbit inference engine (.agents/skills/review-doc-style/assets/nvidia-style-language-mechanics.md)
Files:
**/README.md📄 CodeRabbit inference engine (.agents/skills/review-doc-style/SKILL.md)
Files:
**/*.{md,mdx}📄 CodeRabbit inference engine (.agents/skills/review-doc-style/SKILL.md)
Files:
**/*.md📄 CodeRabbit inference engine (.agents/skills/README.md)
Files:
**/{*.md,*.rst,*.mdx}📄 CodeRabbit inference engine (.agents/skills/maintain-packaging/SKILL.md)
Files:
**/*.{md,mdx,html}📄 CodeRabbit inference engine (CONTRIBUTING.md)
Files:
{adapters/**,examples/**}⚙️ CodeRabbit configuration file
Files:
{*.md,**/*.md,**/*.mdx,**/*.ipynb}⚙️ CodeRabbit configuration file
Files:
**/*.{rs,py,pyi,json,yaml,yml}📄 CodeRabbit inference engine (.agents/skills/contribute-api/SKILL.md)
Files:
**/*.{rs,py}📄 CodeRabbit inference engine (.agents/skills/prepare-pr/SKILL.md)
Files:
**/*.{py,pyi}📄 CodeRabbit inference engine (.agents/skills/validate-change/SKILL.md)
Files:
**/*.{rs,py,pyi}📄 CodeRabbit inference engine (.agents/skills/validate-change/SKILL.md)
Files:
**/*.{py,pyi,rs}📄 CodeRabbit inference engine (.agents/skills/validate-change/SKILL.md)
Files:
**/*.{rs,py,toml}📄 CodeRabbit inference engine (.agents/skills/update-project-version/SKILL.md)
Files:
**/*.{toml,rs,py}📄 CodeRabbit inference engine (.agents/skills/update-project-version/SKILL.md)
Files:
tests/**/*.py📄 CodeRabbit inference engine (.agents/skills/python-tests/SKILL.md)
Files:
**/tests/**/*.{rs,py,pyi}📄 CodeRabbit inference engine (CONTRIBUTING.md)
Files:
{tests/**,python/tests/**}⚙️ CodeRabbit configuration file
Files:
**/*.mdx📄 CodeRabbit inference engine (.agents/skills/contribute-docs/SKILL.md)
Files:
docs/**/*.{md,mdx,yml}📄 CodeRabbit inference engine (.agents/skills/contribute-docs/SKILL.md)
Files:
docs/**/*📄 CodeRabbit inference engine (AGENTS.md)
Files:
{docs/**,README.md,AGENTS.md}⚙️ CodeRabbit configuration file
Files:
{README.md,docs/index.yml}📄 CodeRabbit inference engine (.agents/skills/contribute-docs/SKILL.md)
Files:
README.md📄 CodeRabbit inference engine (AGENTS.md)
Files:
🧠 Learnings (1)📓 Common learnings🪛 Ruff (0.16.0)tests/python/test_harbor_integration.py[warning] 357-357: Async functions should not use pathlib.Path methods, use trio.Path or anyio.path (ASYNC240) 🔇 Additional comments (7)
WalkthroughThe PR restructures NeMo Fabric adapter packaging around slim leaf dependencies, exact runtime pinning, harness/full/Relay extras, adapter-test synchronization, updated installation documentation, and revised Harbor examples and metadata validation. ChangesAdapter packaging boundary
Estimated code review effort: 4 (Complex) | ~60 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Fern docs preview: https://nvidia-preview-pull-request-156.docs.buildwithfern.com/nemo/fabric |
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 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 `@docs/integrations/harness/deepagents.mdx`:
- Around line 30-32: Add the supported root-level Relay installation path to the
Deep Agents documentation, using either the command pip install
"nemo-fabric[deepagents,relay]" or an explicit nemo-fabric installation with the
adapter’s full extra. Place it alongside the existing explanation of the relay
and full extras.
In `@examples/harbor/README.md`:
- Around line 68-77: Synchronize all Harbor package references with the
repository’s 0.2.0 metadata: update the dependency versions in
examples/harbor/README.md lines 68-77 and FABRIC_PACKAGE in
examples/harbor/swebench/README.md lines 30-32; in
tests/integrations/test_harbor_runner.py lines 321-332 and
tests/python/test_harbor_integration.py lines 355-373, derive expected versions
and pip specifications from pyproject.toml instead of hardcoded 0.1.x values.
In `@README.md`:
- Around line 151-153: Update all stale package version pins in README.md:
change every task-environment example at lines 151-153, the runtime example at
line 170, and the Hermes adapter example at line 179 from 0.1.0 to 0.2.0,
preserving the existing extras and command structure.
🪄 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: ASSERTIVE
Plan: Enterprise
Run ID: 71cd84c0-d1cf-4767-80ce-52f956b4aedb
⛔ Files ignored due to path filters (6)
Cargo.lockis excluded by!**/*.lockadapters/claude/uv.lockis excluded by!**/*.lockadapters/codex/uv.lockis excluded by!**/*.lockadapters/deepagents/uv.lockis excluded by!**/*.lockadapters/hermes/uv.lockis excluded by!**/*.lockuv.lockis excluded by!**/*.lock
📒 Files selected for processing (35)
.agents/skills/contribute-adapter/SKILL.md.agents/skills/maintain-packaging/SKILL.md.agents/skills/python-tests/SKILL.md.github/workflows/ci_python.ymlCONTRIBUTING.mdREADME.mdadapters/claude/README.mdadapters/claude/pyproject.tomladapters/codex/README.mdadapters/codex/pyproject.tomladapters/deepagents/README.mdadapters/deepagents/pyproject.tomladapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pyadapters/hermes/README.mdadapters/hermes/pyproject.tomldocs/getting-started/install.mdxdocs/getting-started/quickstart.mdxdocs/integrations/harness/claude.mdxdocs/integrations/harness/codex.mdxdocs/integrations/harness/deepagents.mdxdocs/integrations/harness/hermes.mdxdocs/sdk/python.mdxexamples/harbor/README.mdexamples/harbor/calculator/README.mdexamples/harbor/calculator/task/environment/Dockerfileexamples/harbor/swebench/README.mdjustfilepypi.mdpyproject.tomlskills/integrations/consumer/nemo-fabric-integrate/SKILL.mdtests/adapters/test_adapter_package_metadata.pytests/adapters/test_hermes_config_builder.pytests/integrations/test_harbor_runner.pytests/python/test_harbor_integration.pytests/python/test_readme_examples.py
📜 Review details
⏰ Context from checks skipped due to timeout. (7)
- GitHub Check: Test (Python 3.13, macos-arm64)
- GitHub Check: Pre-commit
- GitHub Check: Test (Python 3.14, windows-amd64)
- GitHub Check: Test (Python 3.13, windows-amd64)
- GitHub Check: Test (Python 3.11, windows-amd64)
- GitHub Check: Test (Python 3.12, windows-amd64)
- GitHub Check: Test (Python 3.11, macos-arm64)
🧰 Additional context used
📓 Path-based instructions (57)
**/*.{md,rst}
📄 CodeRabbit inference engine (.agents/skills/contribute-api/SKILL.md)
Update documentation and examples in the same branch as the public API change.
Files:
adapters/hermes/README.mdCONTRIBUTING.mdskills/integrations/consumer/nemo-fabric-integrate/SKILL.mdadapters/deepagents/README.mdpypi.mdexamples/harbor/calculator/README.mdadapters/codex/README.mdexamples/harbor/swebench/README.mdexamples/harbor/README.mdREADME.mdadapters/claude/README.md
**/*
📄 CodeRabbit inference engine (.agents/skills/karpathy-guidelines/SKILL.md)
**/*: Before implementing, explicitly state assumptions, surface ambiguity and tradeoffs, present multiple interpretations when relevant, and ask for clarification rather than silently deciding or proceeding when requirements are unclear.
Prefer the minimum code needed to solve the requested problem: avoid speculative features, unnecessary abstractions, unrequested flexibility, and handling of impossible scenarios; simplify overcomplicated solutions.
When editing existing code, make surgical changes only: do not modify unrelated code, comments, formatting, or pre-existing dead code; match the existing style, and remove only unused imports, variables, or functions introduced by your changes.
Define verifiable success criteria for each task, such as writing regression tests for bugs and invalid-input tests for validation, then verify the implementation against those criteria. For multi-step work, state a brief plan with a verification check for each step.
**/*: Always spellNVIDIAin all caps; do not useNvidia,nvidia,nVidia,nVIDIA, orNV.
Usean NVIDIAbefore a noun, because the name begins with an “en” sound.
Do not add a registered trademark symbol afterNVIDIAwhen referring to the company; use trademark symbols with product names only when required by the document type or legal guidance.
Verify official capitalization, spacing, hyphenation, and spelling for NVIDIA and third-party product names; do not rewrite official product names for grammar or title-case rules.
Precede NVIDIA product names withNVIDIAon first mention when natural and accurate, and link the first mention when the destination helps the reader.
On first use, include the company name and full model qualifier when it helps identify the model; preserve official capitalization and punctuation, and use shorter family names only after establishing the full name.
For learning-oriented and developer content, do not force trademark symbols unless explicitly required; for press, ...
Files:
adapters/hermes/README.mddocs/sdk/python.mdxdocs/getting-started/quickstart.mdxexamples/harbor/calculator/task/environment/Dockerfiletests/python/test_readme_examples.pyCONTRIBUTING.mdtests/adapters/test_hermes_config_builder.pyskills/integrations/consumer/nemo-fabric-integrate/SKILL.mdadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pyadapters/deepagents/README.mddocs/integrations/harness/hermes.mdxdocs/integrations/harness/deepagents.mdxdocs/integrations/harness/codex.mdxpypi.mdadapters/claude/pyproject.tomlexamples/harbor/calculator/README.mdjustfiledocs/integrations/harness/claude.mdxadapters/codex/README.mdadapters/codex/pyproject.tomlexamples/harbor/swebench/README.mdtests/python/test_harbor_integration.pyadapters/deepagents/pyproject.tomlexamples/harbor/README.mdtests/adapters/test_adapter_package_metadata.pydocs/getting-started/install.mdxadapters/hermes/pyproject.tomltests/integrations/test_harbor_runner.pypyproject.tomlREADME.mdadapters/claude/README.md
**/*.{md,mdx,rst}
📄 CodeRabbit inference engine (.agents/skills/review-doc-style/assets/nvidia-style-guide.md)
**/*.{md,mdx,rst}: For NeMo Fabric documentation, verify technical claims against the current repository, public API, or documented command before reviewing style.
Always spellNVIDIAin all caps; do not useNvidia,nvidia, orNV.
Format commands, code elements, expressions, package names, file names, and paths as inline code.
Use descriptive link text; avoid raw URLs and weak anchors such ashereorread more.
Use title case consistently for technical documentation headings.
Introduce code blocks, lists, tables, and images with complete sentences.
Write procedures as imperative, parallel steps; split long procedures into smaller tasks.
Prefer active voice, present tense, short sentences, contractions, and plain English while preserving necessary technical precision.
Usecanfor possibility and reservemayfor permission.
Useafterfor temporal relationships instead ofonce, and preferrefer tooverseewhen directing readers to another resource.
Avoid culture-specific idioms, unnecessary Latinisms, jokes, and marketing exaggeration in technical documentation.
Spell out months in body text, avoid ordinal dates, and use clear time zones.
Spell out whole numbers from zero through nine unless they are technical values, parameters, versions, or UI values; use numerals for 10 or greater and commas in thousands.
Do not add trademark symbols to learning-oriented documentation unless the source, platform, or legal guidance explicitly requires them.
Do not replace precise technical terms with simpler words when doing so would lose precision.
Do not flag passive voice when the actor is unknown or the action is the important part.
Do not rewrite API names, package names, command flags, or code literals for style.
**/*.{md,mdx,rst}: Use consistent title case for technical-document headings and table headers; avoid quotation marks, ampersands, and exclamation marks in headings, while preserving official product, event, research, and whitepaper title ...
Files:
adapters/hermes/README.mddocs/sdk/python.mdxdocs/getting-started/quickstart.mdxCONTRIBUTING.mdskills/integrations/consumer/nemo-fabric-integrate/SKILL.mdadapters/deepagents/README.mddocs/integrations/harness/hermes.mdxdocs/integrations/harness/deepagents.mdxdocs/integrations/harness/codex.mdxpypi.mdexamples/harbor/calculator/README.mddocs/integrations/harness/claude.mdxadapters/codex/README.mdexamples/harbor/swebench/README.mdexamples/harbor/README.mddocs/getting-started/install.mdxREADME.mdadapters/claude/README.md
**/*.{md,rst,txt,adoc}
📄 CodeRabbit inference engine (.agents/skills/review-doc-style/assets/nvidia-style-language-mechanics.md)
**/*.{md,rst,txt,adoc}: For technical documentation, use professional, active, conversational, engaging, precise, and plain-English prose. Prefer active voice, present tense, short sentences, and scannable paragraphs. Avoid casual or imprecise language, swearing, threats, insults, jokes, puns, culture-specific idioms, marketing exaggeration, and unsupported third-party comparisons.
Usecanfor possibility and reservemayfor permission; useafterfor temporal order; userefer tofor cross-references; prefer short direct sentences and specific verbs; avoid unnecessarypleasein technical documentation.
Prefer active voice when the actor matters. Passive voice is acceptable when the actor is unknown or irrelevant, when the action or result is the focus, or in programmer documentation.
Use natural contractions in conversational technical prose, but do not force them in formal legal copy, API references, or generated text.
Prefer simpler English over Latinisms: usefor exampleorsuch asinstead ofe.g.,and so oninstead ofetc.,that isinstead ofi.e.,compared toinstead ofvs., andby,through, orusinginstead ofvia. Use industry-standard terms such as in silico, in vitro, and in vivo when appropriate, and italicize them in running text.
Usethatwithout commas for essential clauses, andwhichwith commas for nonessential clauses.
Format dates and times clearly: spell out months in body text; use forms such asJune 12, 2025; avoid numeric or ordinal dates; capitalize days; use 12-hour time when appropriate; include a space beforea.m.orp.m.; useETandPTfor needed time zones; avoid24/7; and preferfrom 12:30 to 1:00 p.m.for prose ranges.
Format numbers consistently: spell out zero through nine in body text, use numerals for 10 or greater and for technical values, use commas in thousands, do not begin a sentence with a numeral, spell out ordinals, and use numerals consistently within a category wh...
Files:
adapters/hermes/README.mdCONTRIBUTING.mdskills/integrations/consumer/nemo-fabric-integrate/SKILL.mdadapters/deepagents/README.mdpypi.mdexamples/harbor/calculator/README.mdadapters/codex/README.mdexamples/harbor/swebench/README.mdexamples/harbor/README.mdREADME.mdadapters/claude/README.md
adapters/*/README.md
📄 CodeRabbit inference engine (AGENTS.md)
Update adapter README files when public behavior, examples, or supported bindings change.
Files:
adapters/hermes/README.mdadapters/deepagents/README.mdadapters/codex/README.mdadapters/claude/README.md
**/README.md
📄 CodeRabbit inference engine (.agents/skills/review-doc-style/SKILL.md)
Update relevant package, crate, adapter, and integration README files when public behavior or entry-point documentation changes.
Files:
adapters/hermes/README.mdadapters/deepagents/README.mdexamples/harbor/calculator/README.mdadapters/codex/README.mdexamples/harbor/swebench/README.mdexamples/harbor/README.mdREADME.mdadapters/claude/README.md
**/*.{md,mdx}
📄 CodeRabbit inference engine (.agents/skills/review-doc-style/SKILL.md)
**/*.{md,mdx}: Use the full product nameNVIDIA NeMo Fabricon its first usage, typically in the title or H1; useNeMo Fabricthereafter.
Usefabricby itself only when referring to the CLI tool, and surround those references with backticks.
CapitalizeNVIDIAcorrectly in public documentation.
Format commands, code elements, expressions, file names, paths, and filenames as inline code where needed.
Use title case consistently for headings in technical documentation.
Introduce code blocks, tables, and lists with complete lead-in sentences.
Use descriptive anchor text instead of raw URLs or generic link text such ashere.
Prefer active voice, present tense, short sentences, and plain English.
Use consistent terminology for the same concept throughout a document.
Write procedures as imperative, parallel, easy-to-scan steps, and split long sequences into smaller tasks.
Useafterinstead ofoncewhen expressing temporal sequence.
Usecaninstead ofmaywhen the intended meaning is possibility rather than permission.
Avoid ambiguous numeric dates and ordinal dates in body text.
For learning-oriented documentation, do not force trademark symbols unless the source document explicitly requires them.
Introduce examples' code blocks with full sentences and ensure examples match current APIs and build commands.
Files:
adapters/hermes/README.mddocs/sdk/python.mdxdocs/getting-started/quickstart.mdxCONTRIBUTING.mdskills/integrations/consumer/nemo-fabric-integrate/SKILL.mdadapters/deepagents/README.mddocs/integrations/harness/hermes.mdxdocs/integrations/harness/deepagents.mdxdocs/integrations/harness/codex.mdxpypi.mdexamples/harbor/calculator/README.mddocs/integrations/harness/claude.mdxadapters/codex/README.mdexamples/harbor/swebench/README.mdexamples/harbor/README.mddocs/getting-started/install.mdxREADME.mdadapters/claude/README.md
**/*.md
📄 CodeRabbit inference engine (.agents/skills/README.md)
Documentation and examples must be updated consistently with changes to public behavior and reviewed for NVIDIA technical-writing style.
Files:
adapters/hermes/README.mdCONTRIBUTING.mdskills/integrations/consumer/nemo-fabric-integrate/SKILL.mdadapters/deepagents/README.mdpypi.mdexamples/harbor/calculator/README.mdadapters/codex/README.mdexamples/harbor/swebench/README.mdexamples/harbor/README.mdREADME.mdadapters/claude/README.md
adapters/**/*
📄 CodeRabbit inference engine (.agents/skills/contribute-adapter/SKILL.md)
Place each repository adapter under
adapters/<name>/, keep it independent and small, and includeLICENSE -> ../../LICENSE,README.md,fabric-adapter.json, language-native package and lock files, a source entry point, and focused tests.
Files:
adapters/hermes/README.mdadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pyadapters/deepagents/README.mdadapters/claude/pyproject.tomladapters/codex/README.mdadapters/codex/pyproject.tomladapters/deepagents/pyproject.tomladapters/hermes/pyproject.tomladapters/claude/README.md
adapters/**/*.{py,md,json}
📄 CodeRabbit inference engine (.agents/skills/contribute-adapter/SKILL.md)
Add tests covering descriptor shape and exact capabilities, positive normalized-surface mappings, unsupported values and unenforceable policy rejection, result normalization without secret leakage, one-shot execution, continuation, runtime isolation, and a packaged-entry-point subprocess.
Files:
adapters/hermes/README.mdadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pyadapters/deepagents/README.mdadapters/codex/README.mdadapters/claude/README.md
adapters/**/README.md
📄 CodeRabbit inference engine (.agents/skills/contribute-adapter/SKILL.md)
Document installation, supported configuration, harness-only settings, credentials, lifecycle, telemetry, artifacts, limitations, focused test commands, a canonical typed SDK example, and a canonical harness-native YAML fixture when required.
Files:
adapters/hermes/README.mdadapters/deepagents/README.mdadapters/codex/README.mdadapters/claude/README.md
**/*.{md,rst,py,yml,yaml,sh}
📄 CodeRabbit inference engine (.agents/skills/maintain-packaging/SKILL.md)
Documentation, examples, CI workflows, and install/build commands must use current package names, import paths, and build commands consistently.
Files:
adapters/hermes/README.mdtests/python/test_readme_examples.pyCONTRIBUTING.mdtests/adapters/test_hermes_config_builder.pyskills/integrations/consumer/nemo-fabric-integrate/SKILL.mdadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pyadapters/deepagents/README.mdpypi.mdexamples/harbor/calculator/README.mdadapters/codex/README.mdexamples/harbor/swebench/README.mdtests/python/test_harbor_integration.pyexamples/harbor/README.mdtests/adapters/test_adapter_package_metadata.pytests/integrations/test_harbor_runner.pyREADME.mdadapters/claude/README.md
{pyproject.toml,python/pyproject.toml,Cargo.toml,**/*.{md,rst}}
📄 CodeRabbit inference engine (.agents/skills/maintain-packaging/SKILL.md)
Public packaging changes must be reflected in release-facing documentation, and editable maturin builds must continue producing
nemo_fabric._native.
Files:
adapters/hermes/README.mdCONTRIBUTING.mdskills/integrations/consumer/nemo-fabric-integrate/SKILL.mdadapters/deepagents/README.mdpypi.mdexamples/harbor/calculator/README.mdadapters/codex/README.mdexamples/harbor/swebench/README.mdexamples/harbor/README.mdpyproject.tomlREADME.mdadapters/claude/README.md
**/{README.md,*.md,*.mdx}
📄 CodeRabbit inference engine (CONTRIBUTING.md)
For documentation changes, update affected README, SDK/API reference, adapter/example, integration, and support documentation; run
just docsfor docs site changes.
Files:
adapters/hermes/README.mddocs/sdk/python.mdxdocs/getting-started/quickstart.mdxCONTRIBUTING.mdskills/integrations/consumer/nemo-fabric-integrate/SKILL.mdadapters/deepagents/README.mddocs/integrations/harness/hermes.mdxdocs/integrations/harness/deepagents.mdxdocs/integrations/harness/codex.mdxpypi.mdexamples/harbor/calculator/README.mddocs/integrations/harness/claude.mdxadapters/codex/README.mdexamples/harbor/swebench/README.mdexamples/harbor/README.mddocs/getting-started/install.mdxREADME.mdadapters/claude/README.md
**/*.{html,md}
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Every HTML or Markdown file must include the specified SPDX copyright and Apache-2.0 license header using HTML comment syntax.
Files:
adapters/hermes/README.mdCONTRIBUTING.mdskills/integrations/consumer/nemo-fabric-integrate/SKILL.mdadapters/deepagents/README.mdpypi.mdexamples/harbor/calculator/README.mdadapters/codex/README.mdexamples/harbor/swebench/README.mdexamples/harbor/README.mdREADME.mdadapters/claude/README.md
{adapters/**,examples/**}
⚙️ CodeRabbit configuration file
{adapters/**,examples/**}: Review adapter and example changes for command correctness, config/schema consistency, artifact handling, and compatibility with the public NeMo Fabric contracts.
Files:
adapters/hermes/README.mdexamples/harbor/calculator/task/environment/Dockerfileadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pyadapters/deepagents/README.mdadapters/claude/pyproject.tomlexamples/harbor/calculator/README.mdadapters/codex/README.mdadapters/codex/pyproject.tomlexamples/harbor/swebench/README.mdadapters/deepagents/pyproject.tomlexamples/harbor/README.mdadapters/hermes/pyproject.tomladapters/claude/README.md
{*.md,**/*.md,**/*.mdx,**/*.ipynb}
⚙️ CodeRabbit configuration file
{*.md,**/*.md,**/*.mdx,**/*.ipynb}: Enforce the product name in user-facing prose: use "NVIDIA NeMo Fabric" on first use and "NeMo Fabric" thereafter. Flag standalone capitalized "Fabric" when it refers to the product. Do not flag the lowercasefabricCLI command, package/import/crate names, code identifiers, API symbols, configuration keys, file paths, or unrelated generic uses of the word.
Files:
adapters/hermes/README.mddocs/sdk/python.mdxdocs/getting-started/quickstart.mdxCONTRIBUTING.mdskills/integrations/consumer/nemo-fabric-integrate/SKILL.mdadapters/deepagents/README.mddocs/integrations/harness/hermes.mdxdocs/integrations/harness/deepagents.mdxdocs/integrations/harness/codex.mdxpypi.mdexamples/harbor/calculator/README.mddocs/integrations/harness/claude.mdxadapters/codex/README.mdexamples/harbor/swebench/README.mdexamples/harbor/README.mddocs/getting-started/install.mdxREADME.mdadapters/claude/README.md
{README.md,docs/**/*.{md,mdx,yml},examples/**/*.{md,mdx,yml}}
📄 CodeRabbit inference engine (.agents/skills/contribute-docs/SKILL.md)
Keep package names, repository references, and build commands current in documentation and examples.
Files:
docs/sdk/python.mdxdocs/getting-started/quickstart.mdxdocs/integrations/harness/hermes.mdxdocs/integrations/harness/deepagents.mdxdocs/integrations/harness/codex.mdxexamples/harbor/calculator/README.mddocs/integrations/harness/claude.mdxexamples/harbor/swebench/README.mdexamples/harbor/README.mddocs/getting-started/install.mdxREADME.md
{docs/**/*.{md,mdx,yml},examples/**/*.{md,mdx,yml}}
📄 CodeRabbit inference engine (.agents/skills/contribute-docs/SKILL.md)
Update relevant getting-started, reference, adapter, and example documentation when the corresponding examples or adapters change.
Files:
docs/sdk/python.mdxdocs/getting-started/quickstart.mdxdocs/integrations/harness/hermes.mdxdocs/integrations/harness/deepagents.mdxdocs/integrations/harness/codex.mdxexamples/harbor/calculator/README.mddocs/integrations/harness/claude.mdxexamples/harbor/swebench/README.mdexamples/harbor/README.mddocs/getting-started/install.mdx
**/*.mdx
📄 CodeRabbit inference engine (.agents/skills/contribute-docs/SKILL.md)
In MDX files, use JSX comment delimiters (
{/*and*/}) for top-of-file comments, including SPDX headers; do not use HTML comments.Use
{/* ... */}delimiters for top-of-file MDX SPDX comments, not HTML comment delimiters.Every MDX file must include the specified SPDX copyright and Apache-2.0 license header using a JSX comment.
Files:
docs/sdk/python.mdxdocs/getting-started/quickstart.mdxdocs/integrations/harness/hermes.mdxdocs/integrations/harness/deepagents.mdxdocs/integrations/harness/codex.mdxdocs/integrations/harness/claude.mdxdocs/getting-started/install.mdx
docs/**/*.{md,mdx,yml}
📄 CodeRabbit inference engine (.agents/skills/contribute-docs/SKILL.md)
Run
just docswhen the documentation site changes.
Files:
docs/sdk/python.mdxdocs/getting-started/quickstart.mdxdocs/integrations/harness/hermes.mdxdocs/integrations/harness/deepagents.mdxdocs/integrations/harness/codex.mdxdocs/integrations/harness/claude.mdxdocs/getting-started/install.mdx
docs/**/*
📄 CodeRabbit inference engine (AGENTS.md)
Update Fern documentation under
docs/when public behavior, thenemo-fabricpackage, examples, or supported bindings change.
Files:
docs/sdk/python.mdxdocs/getting-started/quickstart.mdxdocs/integrations/harness/hermes.mdxdocs/integrations/harness/deepagents.mdxdocs/integrations/harness/codex.mdxdocs/integrations/harness/claude.mdxdocs/getting-started/install.mdx
docs/sdk/python.mdx
📄 CodeRabbit inference engine (AGENTS.md)
Keep
docs/sdk/python.mdxcurrent when the public Python API changes.
Files:
docs/sdk/python.mdx
{docs/**,README.md,AGENTS.md}
⚙️ CodeRabbit configuration file
{docs/**,README.md,AGENTS.md}: Review documentation for technical accuracy against the current API, command correctness, and consistency with generated schemas.
For links between files under docs/, require paths relative to the source file with the target file's .mdx extension so they work in both Fern builds and repository browsers. Flag Fern site-root links such as NeMo Fabric overview; use the repository-relative equivalent, such as NeMo Fabric overview.
Files:
docs/sdk/python.mdxdocs/getting-started/quickstart.mdxdocs/integrations/harness/hermes.mdxdocs/integrations/harness/deepagents.mdxdocs/integrations/harness/codex.mdxdocs/integrations/harness/claude.mdxdocs/getting-started/install.mdxREADME.md
.agents/skills/**
📄 CodeRabbit inference engine (AGENTS.md)
Keep maintainer skills under
.agents/skills/focused on repository contribution work, including API, documentation, testing, CI, packaging, release, and PR preparation; do not add consumer integration guidance.Repository maintainer skills belong under
.agents/skills/and should use descriptive, task-based names such ascontribute-apiandvalidate-change.
Files:
.agents/skills/python-tests/SKILL.md.agents/skills/contribute-adapter/SKILL.md.agents/skills/maintain-packaging/SKILL.md
**/*.{rs,py,pyi,json,yaml,yml}
📄 CodeRabbit inference engine (.agents/skills/contribute-api/SKILL.md)
Determine and update every affected public surface, including the CLI, PyO3 bindings, Python SDK, type stubs, schemas, and adapter contract, so they remain in parity.
Files:
tests/python/test_readme_examples.pytests/adapters/test_hermes_config_builder.pyadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pytests/python/test_harbor_integration.pytests/adapters/test_adapter_package_metadata.pytests/integrations/test_harbor_runner.py
**/*.{rs,py}
📄 CodeRabbit inference engine (.agents/skills/prepare-pr/SKILL.md)
For native binding changes, run
cargo check -p fabric-python --locked.Keep package names, import paths, and module names internally consistent, and ensure native extension artifacts retain the expected
nemo_fabric._nativename and placement underpython/src/nemo_fabric.
Files:
tests/python/test_readme_examples.pytests/adapters/test_hermes_config_builder.pyadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pytests/python/test_harbor_integration.pytests/adapters/test_adapter_package_metadata.pytests/integrations/test_harbor_runner.py
**/*.{py,pyi}
📄 CodeRabbit inference engine (.agents/skills/validate-change/SKILL.md)
If Python code or a Python-facing adapter changes, run
just test-python.
**/*.{py,pyi}: Use type annotations for public Python APIs and keep native Python binding declarations synchronized with their Rust implementations.
Every Python source or stub file must include the specified SPDX copyright and Apache-2.0 license header using Python comment syntax.
Files:
tests/python/test_readme_examples.pytests/adapters/test_hermes_config_builder.pyadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pytests/python/test_harbor_integration.pytests/adapters/test_adapter_package_metadata.pytests/integrations/test_harbor_runner.py
**/*.{rs,py,pyi}
📄 CodeRabbit inference engine (.agents/skills/validate-change/SKILL.md)
**/*.{rs,py,pyi}: If public configuration types change, confirm schema snapshot tests injust test-rustpass and review generated schema diffs.
For schema or public contract changes, run both language suites and review changes underschemas/and generated API references.Use language-appropriate naming:
snake_casefor Rust and Python functions and variables;PascalCasefor Rust types and Python classes.
Files:
tests/python/test_readme_examples.pytests/adapters/test_hermes_config_builder.pyadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pytests/python/test_harbor_integration.pytests/adapters/test_adapter_package_metadata.pytests/integrations/test_harbor_runner.py
**/*.{py,pyi,rs}
📄 CodeRabbit inference engine (.agents/skills/validate-change/SKILL.md)
For Python SDK or PyO3 binding changes, use
python-tests, run focused pytest tests first, thenjust test-python; rebuild withjust build-pythonwhen native code or packaging changes.
Files:
tests/python/test_readme_examples.pytests/adapters/test_hermes_config_builder.pyadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pytests/python/test_harbor_integration.pytests/adapters/test_adapter_package_metadata.pytests/integrations/test_harbor_runner.py
**/*.{rs,py,toml}
📄 CodeRabbit inference engine (.agents/skills/update-project-version/SKILL.md)
When editing version helpers, verify every
nemo-fabric-*workspace package through Cargo metadata and reject a static version inpython/pyproject.toml.
Files:
tests/python/test_readme_examples.pytests/adapters/test_hermes_config_builder.pyadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pyadapters/claude/pyproject.tomladapters/codex/pyproject.tomltests/python/test_harbor_integration.pyadapters/deepagents/pyproject.tomltests/adapters/test_adapter_package_metadata.pyadapters/hermes/pyproject.tomltests/integrations/test_harbor_runner.pypyproject.toml
**/*.{toml,rs,py}
📄 CodeRabbit inference engine (.agents/skills/update-project-version/SKILL.md)
Avoid blind repository-wide replacement of version-like strings; distinguish package-version references from examples and unrelated dependency versions.
Files:
tests/python/test_readme_examples.pytests/adapters/test_hermes_config_builder.pyadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pyadapters/claude/pyproject.tomladapters/codex/pyproject.tomltests/python/test_harbor_integration.pyadapters/deepagents/pyproject.tomltests/adapters/test_adapter_package_metadata.pyadapters/hermes/pyproject.tomltests/integrations/test_harbor_runner.pypyproject.toml
tests/**/*.py
📄 CodeRabbit inference engine (.agents/skills/python-tests/SKILL.md)
tests/**/*.py: Use pytest to run Python tests.
Do not add@pytest.mark.asyncioto tests; async tests are automatically detected by the async runner.
Do not add-> Nonereturn annotations to test functions.
When mocking classes, useunittest.mock.MagicMockorAsyncMock, usingspecwhen necessary; do not define replacement classes.
Prefix mocked class names withmock, notfake.
Prefer pytest fixtures over helper methods.
Define fixtures shared across multiple test files inconftest.pyrather than repeating them.
Define fixtures using@pytest.fixture(name="<fixture_name>")and a<fixture_name>_fixturefunction; specifyscopeonly when it is notfunction.
Preferpytest.mark.parametrizeover separate tests for different input types.
Use@pytest.mark.usefixtureswhen a fixture is needed but its value is unused or it returns no value.
Avoid defensive programming in tests; access required values directly so missing data fails loudly and clearly.
When adapter installation metadata changes, assert the published package composition directly, including root runtime dependency versions, harness extras, leaf ownership, adapter-tests installation, andfull/relaybehavior.
Use the documented pytest commands:uv run pytest -k "<pattern>"for focused tests anduv run pytestfor the full suite.
Files:
tests/python/test_readme_examples.pytests/adapters/test_hermes_config_builder.pytests/python/test_harbor_integration.pytests/adapters/test_adapter_package_metadata.pytests/integrations/test_harbor_runner.py
**/*.{rs,py,pyi,json}
📄 CodeRabbit inference engine (CONTRIBUTING.md)
When adding functionality, include tests in the corresponding Rust crate or relevant
tests/area; public contract changes must keep checked-in JSON Schema snapshots and native Python binding declarations synchronized.
Files:
tests/python/test_readme_examples.pytests/adapters/test_hermes_config_builder.pyadapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.pytests/python/test_harbor_integration.pytests/adapters/test_adapter_package_metadata.pytests/integrations/test_harbor_runner.py
{tests/**,python/tests/**}
⚙️ CodeRabbit configuration file
{tests/**,python/tests/**}: Tests should cover the behavior promised by the changed API surface, including error paths, lifecycle cleanup, and SDK/native parity where relevant.
Files:
tests/python/test_readme_examples.pytests/adapters/test_hermes_config_builder.pytests/python/test_harbor_integration.pytests/adapters/test_adapter_package_metadata.pytests/integrations/test_harbor_runner.py
tests/adapters/**/*.py
📄 CodeRabbit inference engine (.agents/skills/validate-change/SKILL.md)
tests/adapters/**/*.py: If an adapter or integration changes, run its focused tests.
For adapter behavior changes, run focused adapter tests undertests/adapters, then runjust test-python.
Files:
tests/adapters/test_hermes_config_builder.pytests/adapters/test_adapter_package_metadata.py
skills/**
📄 CodeRabbit inference engine (AGENTS.md)
skills/**: Keep consumer skills underskills/self-contained and dependent only on supported public Python SDK contracts and published documentation; do not add repository-internal contribution guidance.
Keep consumer skills in parity with the public SDK guide, model, and type details when the Python/Rust binding contract changes.
Files:
skills/integrations/consumer/nemo-fabric-integrate/SKILL.md
**/SKILL.md
📄 CodeRabbit inference engine (.agents/skills/README.md)
**/SKILL.md: Maintainer skills must begin with YAML frontmatter containing at leastnameanddescription.
Maintainer skills may reference repository internals, build and test commands, and contribution workflows, but external usage guidance must remain in the top-levelskills/directory.
Files:
skills/integrations/consumer/nemo-fabric-integrate/SKILL.md
⚙️ CodeRabbit configuration file
**/SKILL.md: Do not flag SKILL.md files for missing SPDX headers. Skill entrypoints intentionally start with YAML frontmatter instead.
Verify that every SKILL.md keeps valid YAML frontmatter with at least name and description fields before the Markdown body.
Files:
skills/integrations/consumer/nemo-fabric-integrate/SKILL.md
**/.github/workflows/*.{yml,yaml}
📄 CodeRabbit inference engine (.agents/skills/maintain-ci/SKILL.md)
**/.github/workflows/*.{yml,yaml}: Definepermissions:per job that needs token access, using the minimum required scopes; avoid workflow-level permissions unless centralized inheritance is intentionally documented.
Pin every third-party GitHub Action to a full commit SHA and retain a readable version comment after the SHA.
Use action-native or ecosystem-native caching instead of genericactions/cachewhen available.
Tie cache invalidation to lockfiles, dependency manifests, or explicit tool versions; avoid caching generated outputs unless deliberate repository behavior requires it.
Useastral-sh/setup-uvcache support withcache-dependency-globanchored touv.lock.
UseSwatinem/rust-cachewith explicitshared-keyandworkspacesinstead of ad hoc target-directory caching.
Keep deploy and publish permissions isolated to only the jobs that require them; restrictpages: writeandid-token: writeto Pages deployment jobs and their reusable-workflow callers.
Usecontents: readas the minimum permission for checkout-based build, test, documentation, and packaging jobs.
Grantpull-requests: readonly to jobs that perform pull-request metadata lookups.
When usingworkflow_call, inspect both caller and callee, and ensure the caller grants every permission required by the called jobs because the callee cannot elevate permissions.
Keep local workflow commands aligned with equivalentjustfilerecipes, and keep Python, Rust, and documentation jobs aligned with their lockfiles.
Preserve intended concurrency settings, branch filters, and documentation publish guards when modifying workflows.
Before editing workflows, inspect workflow relationships and settings with searches coveringuses:,permissions:,secrets:,concurrency:,cache, andjust.
Files:
.github/workflows/ci_python.yml
.github/workflows/**/*.{yml,yaml}
📄 CodeRabbit inference engine (.agents/skills/README.md)
GitHub Actions workflows must use pinned actions and be validated locally when maintained.
Files:
.github/workflows/ci_python.yml
.github/workflows/*.{yml,yaml}
📄 CodeRabbit inference engine (.agents/skills/maintain-packaging/SKILL.md)
CI workflows must reference the same package names and installation commands used by local workflows.
Files:
.github/workflows/ci_python.yml
adapters/**/*.py
📄 CodeRabbit inference engine (.agents/skills/contribute-adapter/SKILL.md)
adapters/**/*.py: For Python adapters, usepyproject.toml,uv.lock,src/nemo_fabric_adapters/<name>/adapter.py, andtests/adapters/test_<name>*.py.
Use the existing Fabricpythonorprocessrunner and normalized request/result contracts; do not add a runner or abstraction for one adapter.
Use the complete Fabric invocation for adapters consuming normalized configuration or runtime context. Treatconfig,capability_plan,telemetry_plan, andruntime_contextas authoritative, reservingharness.settingsfor harness-specific behavior.
Apply configuration precedence in this order: normalizedconfig, Fabric-resolved plans and runtime context, harness-specific settings, then descriptor and adapter defaults. Reject conflicting duplicate declarations or unsupported behavior with an actionable field-specific error; never silently drop configuration.
Run dependency and authentication preflight before invoking the harness; declare fixed dependencies in descriptor requirements and never expose credential values in output, errors, events, logs, or fixtures.
Forward only required system variables, selected credential variables, telemetry variables, and documented harness-specific environment; never forward or log unrelated environment values.
Start one local adapter host per Fabric runtime and keep it alive for orderedstart→invoke*→stop. Emit one JSON lifecycle response per request on stdout and diagnostics on stderr.
Return harness-level invoke failures in a successful lifecycle response as adapter output withresponse: null,failed: true, and structurederrorcontainingcode,message,retryable, and optionalmetadata.
Do not emit NeMo Relay stream records on adapter stdout; return exactly one terminal lifecycle response while Relay sends raw records through the SDK-owned out-of-band NDJSON ATOF endpoint.
Scope workspace, generated configuration, state, sessions, and artifacts to resolved runtime context, and isolate stateful adap...
Files:
adapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.py
adapters/**/*.{py,json}
📄 CodeRabbit inference engine (.agents/skills/contribute-adapter/SKILL.md)
Claim only capabilities that the adapter implements and tests end to end. Reject unsupported providers, capability values, transports, fields, or unenforceable policies, and assert exact descriptor capability values in focused tests.
Files:
adapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.py
**/*.{rs,toml}
📄 CodeRabbit inference engine (.agents/skills/prepare-pr/SKILL.md)
For any Rust change, run
just test-rustandcargo fmt --all -- --check.For Rust core, CLI, or shared runtime semantic changes, run Rust formatting and tests, and add Python tests when behavior is exposed through the SDK.
For Rust changes, run
cargo fmt --all, verify formatting withcargo fmt --all -- --check, and compile withcargo check --workspace --locked.
Files:
adapters/claude/pyproject.tomladapters/codex/pyproject.tomladapters/deepagents/pyproject.tomladapters/hermes/pyproject.tomlpyproject.toml
**/{Cargo.toml,Cargo.lock,pyproject.toml,package.json}
📄 CodeRabbit inference engine (.agents/skills/prepare-pr/SKILL.md)
For new or updated dependencies, document the functional need, alternatives considered, and why the selected dependency is the narrowest fit.
Files:
adapters/claude/pyproject.tomladapters/codex/pyproject.tomladapters/deepagents/pyproject.tomladapters/hermes/pyproject.tomlpyproject.toml
**/*.{toml,lock}
📄 CodeRabbit inference engine (.agents/skills/validate-change/SKILL.md)
If a manifest or lockfile changes, run the license-diff script against
origin/main, review transitive license changes, and run theattributions-rustandattributions-pythonpre-commit hooks.
Files:
adapters/claude/pyproject.tomladapters/codex/pyproject.tomladapters/deepagents/pyproject.tomladapters/hermes/pyproject.tomlpyproject.toml
**/*.{yml,yaml,toml,lock}
📄 CodeRabbit inference engine (.agents/skills/validate-change/SKILL.md)
For CI or packaging changes, use
maintain-ciormaintain-packaging, then run recipes and checks whose behavior changed.
Files:
adapters/claude/pyproject.tomladapters/codex/pyproject.tomladapters/deepagents/pyproject.tomladapters/hermes/pyproject.tomlpyproject.toml
{pyproject.toml,adapters/**/pyproject.toml}
📄 CodeRabbit inference engine (.agents/skills/update-project-version/SKILL.md)
{pyproject.toml,adapters/**/pyproject.toml}: Update the literalproject.versionin the root setuptools project and every adapterpyproject.toml.
Keep internal exact-version requirements aligned: rootnemo-fabric-* == <version>optional dependencies and each adapter'snemo-fabric-adapters-common == <version>dependency.
Files:
adapters/claude/pyproject.tomladapters/codex/pyproject.tomladapters/deepagents/pyproject.tomladapters/hermes/pyproject.tomlpyproject.toml
adapters/**/pyproject.toml
📄 CodeRabbit inference engine (.agents/skills/contribute-adapter/SKILL.md)
adapters/**/pyproject.toml: Keep leaf adapter runtime dependencies razor-thin and adapter-owned; do not declare the wrapped harness/SDK or dependencies already declared by its supported package. Provide dependency-free fallbacks for optional libraries.
Give every installable Python leaf adapter aharnessextra, afullextra for package-installable optional integrations, and arelayextra when it imports NeMo Relay Python APIs; if Relay is external, omitrelayand makefullequivalent toharness.
Files:
adapters/claude/pyproject.tomladapters/codex/pyproject.tomladapters/deepagents/pyproject.tomladapters/hermes/pyproject.toml
**/{Cargo.toml,pyproject.toml}
📄 CodeRabbit inference engine (.agents/skills/maintain-packaging/SKILL.md)
**/{Cargo.toml,pyproject.toml}: Treat every direct dependency as a long-lived API, supply-chain, and licensing commitment; prefer the standard library, existing dependencies, or a small local implementation when appropriate.
When selecting dependencies, prefer maintained OSS options with clear SPDX metadata, smaller transitive graphs, and permissive licenses such as Apache-2.0, MIT, BSD, or ISC. Inspect resolved transitive dependencies, not only direct licenses.
TreatUNKNOWN, non-SPDX/custom, proprietary, source-available, copyleft, and network-copyleft licenses as explicit review points; route them to dependency approvers with distribution and linkage context rather than silently accepting or rejecting them.
For new dependencies, record the functional need, viable alternatives, why the selected dependency is the narrowest fit, and any unresolved licensing question.
Files:
adapters/claude/pyproject.tomladapters/codex/pyproject.tomladapters/deepagents/pyproject.tomladapters/hermes/pyproject.tomlpyproject.toml
**/*.{toml,yml,yaml,sh}
📄 CodeRabbit inference engine (CONTRIBUTING.md)
Every TOML, YAML, or shell file must include the specified SPDX copyright and Apache-2.0 license header using hash-comment syntax.
Files:
adapters/claude/pyproject.tomladapters/codex/pyproject.tomladapters/deepagents/pyproject.tomladapters/hermes/pyproject.tomlpyproject.toml
justfile
📄 CodeRabbit inference engine (.agents/skills/update-project-version/SKILL.md)
justfile: Theset-versionrecipe must runjust lock-pythonafter source metadata is updated.
set_project_versionmust invoke both Cargo and Python project version helpers.Keep build, test, clean, and documentation recipes consistent with the current packaging and release workflow.
Files:
justfile
{pyproject.toml,justfile,uv.lock,**/ci/**/*}
📄 CodeRabbit inference engine (.agents/skills/contribute-adapter/SKILL.md)
Wire public adapter packages into language-native installation, descriptor, build, catalog, dependency, and CI surfaces; for Python update root extras, dependency groups,
[tool.uv.sources],python_projects, catalogs, CI enumerations, and packaged descriptor data.
Files:
justfilepyproject.toml
pyproject.toml
📄 CodeRabbit inference engine (.agents/skills/contribute-adapter/SKILL.md)
Keep
nemo-fabric-runtimeas an exact-version unconditional dependency of the rootnemo-fabricmetapackage, and give every installable Python adapter one canonical root extra depending on the matching adapter package and its harness extra.Keep the root
nemo-fabricpackage as a metapackage that unconditionally installs the exact-versionnemo-fabric-runtimedistribution; root harness extras must delegate to version-matched leaf adapterharnessextras, without root adapter-only aliases.
pyproject.toml: Packaging metadata tests must verify that the root project unconditionally depends on the exact-versionnemo-fabric-runtimedistribution.
Packaging metadata tests must verify root harness extras delegate to matching leaf adapterharnessextras, bare leaf dependencies remain adapter-owned, and the rootadapter-testsgroup installs each leaf through itsharnessextra.
Packaging metadata tests must verify every leaf providesfull; only adapters importing NeMo Relay Python APIs providerelay, while adapters using an external Relay executable havefullequal toharness.
Files:
pyproject.toml
{pyproject.toml,python/pyproject.toml,Cargo.toml,Cargo.lock,uv.lock}
📄 CodeRabbit inference engine (.agents/skills/maintain-packaging/SKILL.md)
Keep workspace, Python, and lockfile versions aligned wherever required by the packaging configuration.
Files:
pyproject.toml
{README.md,docs/index.yml}
📄 CodeRabbit inference engine (.agents/skills/contribute-docs/SKILL.md)
Update
README.mdordocs/index.ymlwhen documentation entry points or example reading paths change.
Files:
README.md
README.md
📄 CodeRabbit inference engine (AGENTS.md)
Update the root
README.mdwhen public behavior, thenemo-fabricpackage, examples, or supported bindings change.Update the top-level
README.mdwhen changed behavior affects an entry point users would naturally consult.
Files:
README.md
🧠 Learnings (2)
📓 Common learnings
Learnt from: CR
Repo: NVIDIA/NeMo-Fabric
Timestamp: 2026-07-29T19:02:03.400Z
Learning: Read the authoritative reference surfaces before editing, use the closest existing adapter only for harness-specific patterns, and keep the change scoped to the adapter contract.
Learnt from: CR
Repo: NVIDIA/NeMo-Fabric
Timestamp: 2026-07-29T19:02:03.400Z
Learning: Run the applicable validation matrix, including adapter tests, Python/Rust tests as relevant, lockfile and wheel generation for package changes, schema generation for schema changes, documentation generation for docs changes, pre-commit, and `git diff --check`.
Learnt from: CR
Repo: NVIDIA/NeMo-Fabric
Timestamp: 2026-07-29T19:02:26.252Z
Learning: Run tests for every language surface affected by a change; changes touching the Rust core or public schemas require both Rust and Python test suites.
Learnt from: CR
Repo: NVIDIA/NeMo-Fabric
Timestamp: 2026-07-29T19:02:26.252Z
Learning: If a change affects public behavior, adapters, examples, or workspace structure, update the corresponding documentation in the same branch.
Learnt from: CR
Repo: NVIDIA/NeMo-Fabric
Timestamp: 2026-07-29T19:02:26.252Z
Learning: Versioned release tags must use raw Rust-compatible SemVer without a leading `v`, such as `0.1.0` or `0.1.0-rc.1`.
Learnt from: CR
Repo: NVIDIA/NeMo-Fabric
Timestamp: 2026-07-29T19:02:26.252Z
Learning: All contributors must sign off every commit using `git commit -s`; unsigned commits are not accepted.
Learnt from: CR
Repo: NVIDIA/NeMo-Fabric
Timestamp: 2026-07-29T19:02:26.252Z
Learning: Before submitting a pull request, identify an issue, run relevant tests, verify affected packages build, update documentation, and rebase onto the latest `main`.
Learnt from: CR
Repo: NVIDIA/NeMo-Fabric
Timestamp: 2026-07-29T19:02:26.252Z
Learning: Pull requests require at least one approving review, must have passing CI, and contributors should address feedback with additional commits without force-pushing during review.
Learnt from: CR
Repo: NVIDIA/NeMo-Fabric
Timestamp: 2026-07-29T19:02:26.252Z
Learning: Use commit messages formatted as `type: short description`, with one of the documented types, and keep the first line under 72 characters.
📚 Learning: 2026-07-09T22:28:51.689Z
Learnt from: AjayThorve
Repo: NVIDIA/NeMo-Fabric PR: 43
File: adapters/claude-sdk/src/nemo_fabric_adapters/claude_sdk/adapter.py:164-168
Timestamp: 2026-07-09T22:28:51.689Z
Learning: In the NeMo-Fabric adapters, treat path values used in Fabric adapter configuration (including logic like `_resolve_path` in adapter.py) as config-root-relative. Do not apply `Path.expanduser()` (or otherwise apply `~`/home or shell-style expansion), because it will make the resolved paths normalize inconsistently across adapters. Also, do not rely on or add any resolution behavior that uses `harness.settings.cwd` as an override point for these adapter paths—`harness.settings.cwd` is explicitly unsupported in this adapter context.
Applied to files:
adapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.py
🪛 Ruff (0.16.0)
tests/adapters/test_hermes_config_builder.py
[warning] 52-52: Avoid specifying long messages outside the exception class
(TRY003)
tests/integrations/test_harbor_runner.py
[warning] 325-325: Consider f-string instead of string join
Replace with f-string
(FLY002)
🪛 zizmor (1.28.0)
.github/workflows/ci_python.yml
[warning] 91-91: code injection via template expansion (template-injection): may expand into attacker-controllable code
(template-injection)
[warning] 92-92: code injection via template expansion (template-injection): may expand into attacker-controllable code
(template-injection)
[warning] 92-92: code injection via template expansion (template-injection): may expand into attacker-controllable code
(template-injection)
🔇 Additional comments (34)
CONTRIBUTING.md (1)
54-67: LGTM!README.md (1)
38-52: LGTM!Also applies to: 73-80
docs/getting-started/install.mdx (1)
10-48: LGTM!Also applies to: 49-124, 126-160, 182-190
docs/getting-started/quickstart.mdx (1)
12-25: LGTM!pypi.md (1)
34-113: LGTM!docs/sdk/python.mdx (1)
509-509: LGTM!skills/integrations/consumer/nemo-fabric-integrate/SKILL.md (1)
47-71: LGTM!docs/integrations/harness/hermes.mdx (1)
15-55: LGTM!adapters/claude/README.md (1)
12-31: LGTM!Also applies to: 114-117
adapters/codex/README.md (1)
15-30: LGTM!Also applies to: 66-66, 138-144
adapters/deepagents/README.md (1)
12-29: LGTM!Also applies to: 49-49, 143-145
adapters/hermes/README.md (1)
12-31: LGTM!Also applies to: 78-79
docs/integrations/harness/claude.mdx (1)
13-60: LGTM!docs/integrations/harness/codex.mdx (1)
14-56: LGTM!docs/integrations/harness/deepagents.mdx (1)
16-28: LGTM!Also applies to: 34-56
.agents/skills/contribute-adapter/SKILL.md (1)
52-83: LGTM!Also applies to: 179-179
.agents/skills/maintain-packaging/SKILL.md (1)
37-43: LGTM!.agents/skills/python-tests/SKILL.md (1)
39-52: LGTM!adapters/claude/pyproject.toml (1)
32-38: LGTM!adapters/codex/pyproject.toml (1)
32-38: LGTM!adapters/deepagents/pyproject.toml (1)
37-48: LGTM!adapters/hermes/pyproject.toml (1)
32-41: LGTM!pyproject.toml (1)
40-58: LGTM!Also applies to: 76-81
tests/adapters/test_adapter_package_metadata.py (1)
1-146: LGTM!tests/python/test_readme_examples.py (1)
21-21: LGTM!adapters/deepagents/src/nemo_fabric_adapters/deepagents/adapter.py (1)
121-133: LGTM!tests/adapters/test_hermes_config_builder.py (1)
6-9: LGTM!Also applies to: 42-65
.github/workflows/ci_python.yml (1)
86-92: LGTM!justfile (1)
260-264: LGTM!Also applies to: 350-350, 421-421
examples/harbor/README.md (1)
118-120: LGTM!Also applies to: 138-138
examples/harbor/calculator/README.md (1)
61-61: LGTM!Also applies to: 81-81, 106-106, 148-148, 177-177
examples/harbor/calculator/task/environment/Dockerfile (1)
22-22: LGTM!examples/harbor/swebench/README.md (1)
54-54: LGTM!Also applies to: 75-75, 103-103, 141-141, 179-179, 197-197, 207-210, 231-231, 252-252, 263-263
tests/integrations/test_harbor_runner.py (1)
6-7: LGTM!Also applies to: 32-38, 267-267, 296-307
|
Thank you so much Ajay for making this!! |
Signed-off-by: Ajay Thorve <athorve@nvidia.com>
|
/merge nosquash |
|
Could not determine original ForwardMerger PR from branch name. The branch name should follow the pattern |
1 similar comment
|
Could not determine original ForwardMerger PR from branch name. The branch name should follow the pattern |
Overview
Manually forward-merge
release/0.1intomainafter the automated forward-merge in #150 could not complete.This preserves the release branch commits while retaining
main's0.2.0package identity. The resolution synchronizes Cargo, Python package metadata, internal dependency pins, and generated lockfiles to0.2.0. It also updates the version helper to recognize exact internal dependencies containing extras, such asnemo-fabric-adapters-claude[harness] == 0.2.0.Operational note: Do not close #150. Merge this PR with
/merge nosquashso branch history is preserved and the original forward-merger PR can complete automatically.Details
release/0.1.main's0.2.0versions across the Rust workspace and Python distributions.just set-versionto update internal package pins containing optional extras.nemo-fabric-runtimepackage contract unchanged.Validation
uv lock --checkfor the root, runtime, and all adapter projectsuv run --no-sync pytest -q tests/adapters/test_adapter_package_metadata.py16 passedjust test-python529 passed, 15 skippedjust build-pythoncargo check -p fabric-python --lockedcargo check --workspace --lockedjust wheels0.2.0wheelsuv run pre-commit run --all-files attributions-rustuv run pre-commit run --all-files attributions-pythonjust --fmt --checkgit diff --checkA license comparison directly against
upstream/maincannot generate its Rust baseline because that ref currently hasCargo.tomlat0.2.0while its local workspace entries inCargo.lockremain at0.1.0. This forward-merge resolution corrects that inconsistency.Where should the reviewer start?
Start with the final resolution commit,
2743690, particularlyjustfile,Cargo.toml, and the rootpyproject.toml. Then verify the corresponding local-package entries inCargo.lock,uv.lock, and the adapter lockfiles.The remaining forwarded changes were previously reviewed on the
release/0.1line.Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)
Relates to Forward-merge release/0.1 into main #150
Relates to refactor: slim adapter runtime dependencies #105
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.
Summary by CodeRabbit
New Features
hermes-agent/harness/full/relaysemantics).Documentation
Tests