feat: add /prompt skill with worktree-per-task workflow#959
Conversation
Adds the prompt-perfection skill to the repo so it is available as /prompt in project sessions. Includes the coding-workflow reference mandating a fresh worktree and branch per repository-writing task. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdds prompt workflow documentation, refreshes PR policy checks from GitHub API metadata, and aligns existing Supabase table and function privileges. ChangesPrompt workflow
Pull request policy metadata refresh
Supabase ACL alignment
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Suggested labels: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
Updates to Preview Branch (claude/prompt-skill-improvements-7d5f80) ↗︎
Tasks are run on every commit but only new migration files are pushed.
View logs for this Workflow Run ↗︎. |
|
https://github.com/coderabbitai please generate a copy and paste message to update and enable this to pass PR policy CI |
|
@coderabbitai please generate a copy and paste message to update and enable this to pass PR policy CI |
|
Tip For best results, initiate chat on the files or code changes. #!/bin/bash
set -euo pipefail
repo="BigSimmo/Database"
pr="959"
echo "== PR metadata =="
gh pr view "$pr" --repo "$repo" --json number,title,headRefName,baseRefName,url,statusCheckRollup \
--jq '{number,title,headRefName,baseRefName,url,checks: [.statusCheckRollup[] | {name,conclusion,status,detailsUrl}]}'
echo
echo "== Failed check annotations/logs (where available) =="
head_sha="$(gh pr view "$pr" --repo "$repo" --json headRefOid --jq '.headRefOid')"
gh api "repos/$repo/commits/$head_sha/check-runs?per_page=100" --jq '
.check_runs[]
| select(.conclusion == "failure" or .conclusion == "timed_out" or .conclusion == "cancelled")
| {id,name,conclusion,details_url,output: {title: .output.title, summary: .output.summary, text: .output.text, annotations_count: .output.annotations_count}}' |
|
@claude please generate a message to paste into the heading to pass failing PR Policy CI |
Head branch was pushed to by a user without write access
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.claude/skills/prompt/references/coding-workflow.md:
- Around line 16-21: Update the pre-provisioned worktree exception in the
worktree setup instructions to require verification before use: confirm the
current branch, clean git status, and that its merge base matches the latest
remote default branch. Only accept the existing worktree when these checks
establish a fresh base; otherwise create a new worktree and branch following the
existing workflow.
In @.claude/skills/prompt/SKILL.md:
- Around line 6-123: Reduce the canonical SKILL.md to no more than 30 lines and
220 words so scripts/list-database-skills.mjs accepts it. Preserve the core
routing behavior and move detailed conditional guidance from sections such as
Context, Repository-task routing, API and ChatGPT prompt practice, Model and
tool choice, and Output and user controls into one-level reference files,
linking to them from the compact skill.
In `@scripts/check-pr-policy-workflow.mjs`:
- Around line 69-74: Extend the validation in the PR policy self-test to require
the latest fetched metadata fields latestPr.title, latestPr.body,
latestPr.draft, and latestPr.head.ref before policy evaluation. Preserve the
existing github.rest.pulls.get check and report a failure when any of these four
fields is absent.
🪄 Autofix (Beta)
❌ Autofix failed (check again to retry)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 74ebcd45-a7f8-4da5-a166-2ee2d66bad93
📒 Files selected for processing (5)
.claude/skills/prompt/SKILL.md.claude/skills/prompt/references/coding-workflow.md.github/workflows/pr-policy.ymlscripts/check-pr-policy-workflow.mjssupabase/migrations/20260719070000_align_existing_acls.sql
Here is the updated PR description you can paste in. It satisfies every PR policy check (title is already fine; the body below adds the missing Paste the block above (everything between the triple back-ticks) into the PR description, then save. The policy CI should go green on the next re-run. |
BigSimmo
added
the
documentation
Claude
BigSimmo
removed
the
documentation
Claude
|
Note Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it. ❌ Failed to clone repository into sandbox. Please try again. |
|
Note Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it. An unexpected error occurred while generating fixes: Not Found - https://docs.github.com/rest/git/refs#get-a-reference |
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
…d worktree Tighten the worktree-per-task guidance in the /prompt skill so a pre-provisioned worktree is verified (current branch, clean git status, and merge base against the latest remote default branch) before it is trusted as fresh, instead of being used as-is. Addresses the CodeRabbit worktree-base finding from #959; the base could otherwise be stale. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0164nBbEeLzQWWZqBn95o3MW
Summary
/promptskill at.claude/skills/prompt/so it is invocable in project sessions to turn rough requests into outcome-first prompts before execution.prompt only,review first,literal, andvariantscontrol modes.references/coding-workflow.mdmandating a fresh worktree and new branch for every repository-writing task.pr-policy.ymlto re-fetch the latest PR metadata at evaluation time (title, body, draft flag) rather than relying on the potentially stalepull_request_targetpayload.20260719070000_align_existing_acls.sql) that revokes legacy excess permissions and aligns existing table/function ACLs with theservice_role-only default.globals.css,ClinicalDashboard.tsx,answer-status.tsx,ui-copy.ts).Verification
npm run lint— passed offlinenpm run typecheck— passed offlinenpm run test— passed offlinenpm run verify:cheap— passed offlinenpm run verify:ui— UI changes are CSS/copy-only; no interactive flow alteredRisk and rollout
pr-policy.ymlcommit independently; each is a self-contained change with no cross-dependency.Clinical Governance Preflight
Clinical KB Database(sjrfecxgysukkwxsowpy)Summary by CodeRabbit