Skip to content

improvement(ship): skip confirmation prompts - #6977

Merged
TheodoreSpeaks merged 1 commit into
stagingfrom
improvement/ship-skill
Aug 22, 2026
Merged

improvement(ship): skip confirmation prompts#6977
TheodoreSpeaks merged 1 commit into
stagingfrom
improvement/ship-skill

Conversation

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator

Summary

  • Update the ship skill to skip commit message and PR description confirmation prompts

Type of Change

  • Improvement

Testing

  • bun run lint
  • bun run apps/sim/scripts/check-block-registry.ts origin/staging
  • bun run check:audits

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Aug 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Aug 22, 2026 5:57pm

Request Review

@cursor

cursor Bot commented Aug 22, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Documentation-only change to an agent skill; no application, auth, or data-handling code is modified.

Overview
Updates the /ship skill so agents no longer pause for user confirmation of the commit message or PR description before committing and opening the PR.

Reviewed by Cursor Bugbot for commit 0a54cf7. Bugbot is set up for automated code reviews on this repo. Configure here.

@TheodoreSpeaks
TheodoreSpeaks merged commit 6a5e250 into staging Aug 22, 2026
6 checks passed
@TheodoreSpeaks
TheodoreSpeaks deleted the improvement/ship-skill branch August 22, 2026 17:57
@greptile-apps

greptile-apps Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR streamlines the ship workflow by removing explicit confirmation prompts for generated commit messages and pull request descriptions.

  • Updates the ship skill’s execution guidance.
  • Leaves its existing content-scrubbing and PR validation steps unchanged.

Confidence Score: 5/5

The PR appears safe to merge because the documentation change implements the explicitly stated workflow improvement without violating repository guidance.

The ship workflow intentionally removes its user confirmation prompt while retaining its existing automated content checks, and no accepted blocking or non-blocking issue remains.

Important Files Changed

Filename Overview
.agents/skills/ship/SKILL.md Replaces the commit-message and PR-description confirmation requirement with an explicit instruction to proceed without prompting; no actionable defect was found.

Reviews (1): Last reviewed commit: "improvement(ship): skip confirmation pro..." | Re-trigger Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant