docs(autopilot): auto-open review artifacts at human review gates#187
docs(autopilot): auto-open review artifacts at human review gates#187jankneumann wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5c0afa733e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| If argument was a description (no existing change-id): | ||
| - Invoke `/plan-feature <description>` (tier auto-detected based on coordinator availability) | ||
| - **Before showing/answering the proposal approval gate prompt**, invoke `/review-artifacts <change-id>` to open proposal/design/spec/tasks artifacts in a new VS Code review session |
There was a problem hiding this comment.
Make review-artifacts invocation best-effort
This new step makes /autopilot unconditionally call /review-artifacts at approval gates, but that helper is documented as local-only and depends on the code CLI (.agents/skills/review-artifacts/SKILL.md notes cloud harnesses lack it and exit 127). In cloud/CI-style runs, this can fail right before the human gate and block or derail the workflow instead of cleanly prompting for approval. Please gate this behind local capability checks (or explicitly treat failures as non-fatal) in both approval-gate insertions.
Useful? React with 👍 / 👎.
Motivation
Description
skills/autopilot/SKILL.mdto invoke/review-artifacts <change-id>automatically before the PLAN-phase proposal-approval prompt and before the DONE/merge-approval questions; change is documentation-only (docs(autopilot): auto-open review artifacts at human review gates).Testing
skills/autopilot/SKILL.md).Codex Task