fix: drop legacy enforcement.strategy token from single-agent-integrate.md (unblocks #696)#697
Conversation
…rom ci-pr prose test-legacy-cleanup-grep.sh forbids legacy config-key references in plugins/dso/skills/. The ci-pr review-gating note referenced the literal `enforcement.strategy=ci` (legacy alias) in explanatory prose. Replace with a generic "CI-enforcement skip path" pointer to REVIEW-WORKFLOW.md + rule:no-bypass-review. The canonical dso.workflow gate logic is unchanged.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
DSO-Review-Cycle: 1 pr_number=697 commit_sha=9ae5cd36306467170ce0c96cc1870fea77ff7166 findings_hash=e3b0c44298fc1c14 tuples=[] |
4db7c79
into
staged-46b5387451f3-1780771548
Remediation for PR #696 (staging→main) failing check Script Tests →
test_no_legacy_key_refs_in_scope_files.The ci-pr review-gating note added to
single-agent-integrate.mdreferenced the literal legacy keyenforcement.strategy=ciin explanatory prose, whichtest-legacy-cleanup-grep.shforbids inplugins/dso/skills/. Replaced with a generic 'CI-enforcement skip path' pointer to REVIEW-WORKFLOW.md +rule:no-bypass-review. Canonicaldso.workflowgate logic unchanged. Verifiedtest-legacy-cleanup-grep.shPASS locally.🤖 Generated with Claude Code