Skip to content

Remove installation restrictions from AGENTS.md#1200

Merged
rapids-bot[bot] merged 1 commit into
mainfrom
mlubin-patch-2
May 12, 2026
Merged

Remove installation restrictions from AGENTS.md#1200
rapids-bot[bot] merged 1 commit into
mainfrom
mlubin-patch-2

Conversation

@mlubin
Copy link
Copy Markdown
Contributor

@mlubin mlubin commented May 12, 2026

I just saw a case where the agent refused to run the standard build process because build.sh installs cuopt packages into the python environment. Plus, there are autorun environments where we do want the agent to install packages autonomously. This type of instruction doesn't belong at the individual package level.

I just saw a case where the agent refused to run the standard build process because build.sh installs cuopt packages into the python environment. Plus, there are autorun environments where we do want the agent to install packages autonomously. This type of instruction doesn't belong at the individual package level.
@mlubin mlubin requested a review from rgsl888prabhu May 12, 2026 00:27
@mlubin mlubin requested a review from a team as a code owner May 12, 2026 00:27
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 12, 2026

Review Change Stack
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: a7fdd85b-312a-4662-8e1b-deb714286404

📥 Commits

Reviewing files that changed from the base of the PR and between e7df73d and 724a474.

📒 Files selected for processing (1)
  • AGENTS.md
💤 Files with no reviewable changes (1)
  • AGENTS.md

📝 Walkthrough

Walkthrough

A single mandatory security directive line was removed from AGENTS.md. The line forbade package installation, upgrades, and modification, and required providing an exact command for users to run. No code or exported entities were changed.

Changes

Documentation Update

Layer / File(s) Summary
Security Directive Removal
AGENTS.md
Mandatory security instruction forbidding package operations and requiring exact user-run commands removed.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title directly and accurately summarizes the main change: removing installation restrictions from AGENTS.md documentation.
Description check ✅ Passed The description is clearly related to the changeset, explaining the rationale for removing installation restrictions and providing concrete context about agent behavior.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch mlubin-patch-2

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

@mlubin mlubin added non-breaking Introduces a non-breaking change improvement Improves an existing functionality Agentic This label is used to track agentic and skill related issues labels May 12, 2026
@rgsl888prabhu
Copy link
Copy Markdown
Collaborator

/merge

@rapids-bot rapids-bot Bot merged commit c3f8003 into main May 12, 2026
26 checks passed
@mlubin mlubin deleted the mlubin-patch-2 branch May 12, 2026 10:36
chris-maes pushed a commit to chris-maes/cuopt that referenced this pull request May 18, 2026
I just saw a case where the agent refused to run the standard build process because build.sh installs cuopt packages into the python environment. Plus, there are autorun environments where we do want the agent to install packages autonomously. This type of instruction doesn't belong at the individual package level.

Authors:
  - Miles Lubin (https://github.com/mlubin)

Approvers:
  - Ramakrishnap (https://github.com/rgsl888prabhu)

URL: NVIDIA#1200
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Agentic This label is used to track agentic and skill related issues improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants