feat(skills): add one-hour quick delivery path - #2805
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e798014170
ℹ️ 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".
| ## Execute — minute 5–35 | ||
|
|
||
| 1. Start exactly one executor through the active runtime's native delegation surface, inheriting the active runtime model and configuration. | ||
| 2. Use one isolated worktree from the current target base. No fan-out, board/group ceremony, independent reviewer or model-selection machinery. |
There was a problem hiding this comment.
When Quick reaches integration, this explicitly excludes an independent reviewer and the following phase substitutes a self-review before merging to dev. That removes the repository's mandatory separation between engineer and reviewer, so defects can ship without independent evidence; the one-hour path must retain a distinct reviewer or stop as quick-missed when review cannot fit.
AGENTS.md reference: AGENTS.md:L34-L36
Useful? React with 👍 / 👎.
| 1. Run the focused test and every affected repository check. | ||
| 2. Perform one self-review of the exact diff for correctness, scope, secrets and target identity. | ||
| 3. Apply at most one bounded correction while time remains. | ||
| 4. Open the PR to `dev` and wait for every required CI check. Never bypass or weaken checks. |
There was a problem hiding this comment.
When the existing authorization covers only merging, this step still publishes a PR even though the deleted skills/pm/SKILL.md authority table required an explicit PR-publication grant separately from the merge grant. Quick admission checks only merge authority, so this can perform an external write outside the user's granted scope; require proof covering PR creation as well as the eventual merge.
Useful? React with 👍 / 👎.
| 'perf', | ||
| 'pm', | ||
| 'qa', | ||
| 'quick', |
There was a problem hiding this comment.
For Kimi plugin installations, replacing pm with quick in the shipped inventory leaves .kimi-plugin/commands/pm.md invoking the now-missing pm skill, while no quick.md wrapper exposes the new slash command; plugins/genie/README.md also continues advertising /genie:pm. Consequently the advertised PM command fails and the new Quick command is absent from Kimi's command surface.
Useful? React with 👍 / 👎.
e798014 to
ef7addc
Compare
1 participant
Summary
/quickfor tiny, low-risk request → deployed-dev delivery within 60 minutespmwithquick; PM now lives in Sofia's profile rather than the Genie product payloadValidation
skills/quick/SKILL.mdabsentbun test scripts/release-docs.test.ts: 43/43 passedskills:lint, typecheck, dead-code, wish lint, complexity budget, council, hook and plugin executable gates passedbun run check: 3,368 tests passed; 17 unrelated release timing, installer and frozen-fixture tests remain red in this environmentScope
Atomic skill-distribution change only. No application deploy, production mutation or merge is included.