chore(test): drop tox orchestration

[Pouyanpi]

Description

drop tox orchestration

Related Issue(s)

Verification

AI Assistance

• No AI tools were used.
• AI tools were used; a human reviewed and can explain every change (tool: ___).

Checklist

• I've read the CONTRIBUTING guidelines.
• This PR links to a triaged issue assigned to me.
• My PR title follows the project commit convention.
• I've updated the documentation if applicable.
• I've added tests if applicable.
• I've noted any verification beyond CI and any checks I couldn't run.
• I did not update generated changelog files manually.
• I addressed all CodeRabbit, Greptile, and other review comments, or replied with why no change is needed.
• @mentions of the person or team responsible for reviewing proposed changes.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[greptile-apps]

Greptile Summary

This PR removes the tox test orchestration layer, which was used to run the test suite against Python 3.10–3.13. The functionality is already covered by the GitHub Actions matrix in pr-tests.yml, making tox redundant.

• Deletes tox.ini and removes tox + tox-uv from pyproject.toml's test_integration extra and the corresponding uv.lock entries.
• Updates CONTRIBUTING.md and AGENTS.md to drop references to uv run tox, keeping the documentation consistent with the remaining tooling.

Confidence Score: 5/5

Safe to merge — the removed tox layer was redundant with the existing GitHub Actions matrix that already tests Python 3.10–3.13 on every PR.

The deletion of tox.ini and its dependencies is fully backed by the pre-existing multi-Python CI matrix in pr-tests.yml, which covers the same four Python versions. No test coverage gap is introduced, no CI workflows reference tox, and the documentation updates are consistent with the code changes.

No files require special attention.

Important Files Changed

Filename	Overview
tox.ini	Deleted entirely; the four-Python-version matrix it defined (py310–py313) is already covered by the GitHub Actions matrix in pr-tests.yml.
pyproject.toml	Removes tox and tox-uv from the test_integration optional extra; no other dependency changes.
uv.lock	Drops tox, tox-uv, tox-uv-bare, pyproject-api, tomli-w, and the uv binary package entries that were pulled in transitively by tox-uv.
CONTRIBUTING.md	Removes the 'Supported Python versions / uv run tox' row from the Validation table.
AGENTS.md	Updates the cross-reference sentence to drop the mention of tox.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Developer runs tests] --> B{Local or CI?}
    B -->|Local| C[make test]
    B -->|CI - PR| D[GitHub Actions pr-tests.yml]
    C --> E[pytest via uv]
    D --> F[Matrix: Python 3.10 / 3.11 / 3.12 / 3.13]
    F --> G[_test.yml reusable workflow]
    G --> E

    style A fill:#f0f0f0
    classDef kept fill:#ccffcc,stroke:#009900
    class E,F,G,C kept

Loading

%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A[Developer runs tests] --> B{Local or CI?}
    B -->|Local| C[make test]
    B -->|CI - PR| D[GitHub Actions pr-tests.yml]
    C --> E[pytest via uv]
    D --> F[Matrix: Python 3.10 / 3.11 / 3.12 / 3.13]
    F --> G[_test.yml reusable workflow]
    G --> E

    style A fill:#f0f0f0
    classDef kept fill:#ccffcc,stroke:#009900
    class E,F,G,C kept

Loading

_{Reviews (1): Last reviewed commit: "chore(test): drop tox orchestration" | Re-trigger Greptile}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: dbf7ce4d-078c-4127-8ad1-0b48b1862cf7

📥 Commits

Reviewing files that changed from the base of the PR and between 8645364 and 88ef926.

⛔ Files ignored due to path filters (1)

• uv.lock is excluded by !**/*.lock

📒 Files selected for processing (4)

• AGENTS.md
• CONTRIBUTING.md
• pyproject.toml
• tox.ini

💤 Files with no reviewable changes (3)

• tox.ini
• CONTRIBUTING.md
• pyproject.toml

---

📝 Walkthrough

Walkthrough

References to tox are removed from AGENTS.md and CONTRIBUTING.md validation instructions, the tox and tox-uv dependencies are removed from pyproject.toml's test_integration group, and the tox.ini configuration file is deleted entirely.

Changes

Tox removal

Layer / File(s)	Summary
Documentation updates removing tox references AGENTS.md, CONTRIBUTING.md	Validation guidance text and table no longer mention tox or the "Supported Python versions" tox check.
Dependency group cleanup pyproject.toml, tox.ini	test_integration dependency group drops tox/tox-uv entries and the tox.ini file is deleted.

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

Possibly related PRs

• NVIDIA-NeMo/Guardrails#2106: Also modifies tox integration by updating tooling and validation instructions around uv/tox usage.

Suggested reviewers: tgasser-nv, cparisien

🚥 Pre-merge checks | ✅ 6

✅ Passed checks (6 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly summarizes the main change: removing tox-based test orchestration.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Test Results For Major Changes	✅ Passed	Diff is limited to docs/config/lockfile changes and tox.ini removal; no major feature, breaking, or performance change, so test-results notes aren’t required.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/drop-tox

---

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