Repository navigation
Bind the Workspace-Edit Rule Only to an Agent Running Inside VS Code - #2815
Conversation
A workspace reload drops only a session that VS Code hosts, so an agent running in a standalone console now edits a .code-workspace directly, even while a window has that file loaded. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…Rule Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 27 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai review |
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #2815 +/- ##
==========================================
Coverage ? 60.30%
==========================================
Files ? 16
Lines ? 8479
Branches ? 0
==========================================
Hits ? 5113
Misses ? 3366
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Post-merge review, recorded as coverage evidence for promotion #2816, since this pull request's only Copilot round was a quota refusal. A frontier-tier, read-only review read all 5 changed files at develop 4035555, plus every other surface that states the rule. It raised one defect and one nit, and the maintainer accepted both as tracked rather than blocking. Both are filed as #2834. The defect is that the console exemption permits an edit that reloads a window hosting another agent session. The nit is that the "Editor and Tasks" closing sentence is now over-general. The generated |
…ilure Reporting Fixes to Main (#2816) ## Summary Promotes these pull requests from develop to main: - #2832 Require Every CLI Script to Answer Help and Refuse Unknown Arguments - #2833 Exempt nopep8 as a Leading Directive in the Prose Gate - #2815 Bind the Workspace-Edit Rule Only to an Agent Running Inside VS Code - #2813 Record HomeAssistant-Config's Incidental Python as a driftNote - #2827 Read nopep8 in Any Letter Case in the Prose Gate - #2825 Spend a Head Refusal a Later Genuine Round on That Head Answers - #2819 Read noqa and CSpell Directives in Any Letter Case - #2814 Judge the Prose in Front of a Trailing noqa or pylint Directive - #2811 Name Why a Windows Tool Install Failed and Print Docker's Status in UTF-8 - #2808 Read a Tool Name as a Prose Gate Directive Only Where It Opens the Comment - #2802 Name a File the Prose Gate Cannot Decode in Its Scope Line - #2800 Read a First-Line Comment After a Byte Order Mark as Leading in Every Language - #2797 Name a Python File the Prose Gate Cannot Parse in Its Scope Line - #2794 Let the Unattended Orchestrator Retry a Transient Dispatch Failure - #2792 Close the Remaining Blocked-Promotion Gaps in the Unattended Worker - #2790 Read a Carried Change Set Past the Compare Endpoint's 300-File Ceiling - #2784 Let a Round on the Head Outrank pr_review.py wait's Repo-Wide Quota Reading - #2782 Say What pr_review.py wait --min-rounds Counts and Name It at the Timeout - #2778 Share the Bounded Step Reader Across the Test Harnesses - #2776 Pin the Prose Gate's Override Wiring by Structure - #2773 Exempt a List Item's Label Separator from the Dash Rule for Every Marker and Bold Spelling - #2771 Strip a List Item's Opening Label for Every Marker and Bold Spelling in the Prose Gate - #2768 Page pr_review.py's Changed-File List Past 100 So a Large Promotion Can State Coverage - #2765 Let an Attested Local Pass Cover a Head Beside a Table-Less Copilot Round Closes #2285 Closes #2820 Closes #2818 Closes #2809 Closes #2807 Closes #1659 Closes #2796 Closes #2798 Closes #2163 Closes #1984 Closes #2614 Closes #2767 Closes #2781 Closes #2741 Closes #2775 Closes #1665 Closes #2770 Closes #2738 Closes #2758 Closes #2739 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Summary
The rule against editing a loaded
.code-workspacefile applied wherever the editing agent ran, so agents in a standalone console kept handing trivial workspace edits back to the maintainer. A VS Code reload drops only a session that VS Code hosts. The rule now binds an agent running inside VS Code, in its extension or its integrated terminal, and an agent in a standalone console edits the file directly, even while a window has it loaded.GOVERNANCE.md"Verification Discipline": scopes the rule to an agent hosted by VS Code. It keeps the exemption for a file no window has loaded, and asks the maintainer only when the agent cannot tell whether the rule binds it.GOVERNANCE.md"Editor and Tasks": the pointer bullet now states the same split.agent-conductSkill: regenerated, since it carries "Verification Discipline" as an include.Verification
scripts/build_dist.py --checkreports the generated distributions are current.prose_lint.py --diffagainst the merge base reports no violations.spec/validate.pypasses.local-strict-reviewpasses ran. The first two raised findings: the dropped not-loaded exemption, an unreliable environment-variable sign, and an over-broad fallback. Each was fixed. The final pass raised none.🤖 Generated with Claude Code