setup-deploy
“Configure deployment settings for /land-and-deploy.”
aipkg.jsonLICENSE.txtREADME.mdSKILL.mdSKILL.md.tmpl- 01SQP-270%MEDIUM
SKILL.md:884
detailhide
The skill performs HTTP health checks against production URLs without explicit user confirmation. If the user supplies a malformed URL or a URL pointing to an internal/sensitive endpoint, the skill will blindly request it. This could trigger unintended side effects (logs, monitoring alerts, or accidental data exposure if the URL includes credentials).
882After writing, verify the configuration works:8838841. If a health check URL was configured, try it:885```bash886curl -sf "{health-check-url}" -o /dev/null -w "%{http_code}" 2>/dev/null || echo "UNREACHABLE"fix Before running health checks, explicitly confirm the URL with the user. Wrap the curl commands in an AskUserQuestion asking the user to approve the health check URL. Example: 'Health check will request [URL] — continue?' This prevents blind requests to user-supplied or inferred URLs.
- 02SQP-280%MEDIUM
SKILL.md:860
detailhide
The skill writes to CLAUDE.md without explicit user confirmation after the configuration is inferred. If detection is inaccurate (e.g., wrong platform guessed), the skill will overwrite CLAUDE.md with incorrect data, potentially breaking future deploys. The instructions say 'always show the detected config and ask for confirmation' but the actual Step 4 implementation does not enforce this gate.
858### Step 4: Write configuration859860Read CLAUDE.md (or create it). Find and replace the `## Deploy Configuration` section861if it exists, or append it at the end.862fix Add an explicit AskUserQuestion before writing to CLAUDE.md. Show the full proposed configuration block and ask: 'Write this deploy configuration to CLAUDE.md?' with A) Yes, B) Edit config, C) Cancel. Only proceed with Write tool if user selects A.
- 03SQP-275%MEDIUM
SKILL.md.tmpl:163
detailhide
The skill writes deployment configuration to CLAUDE.md without explicit inline user confirmation immediately before the write operation. While Step 4 mentions reading CLAUDE.md, the actual write logic lacks a clear confirmation prompt. This could allow unintended configuration overwrites if the user is not fully aware of what is being written, especially since this configuration controls deployment behavior across future operations.
161### Step 4: Write configuration162163Read CLAUDE.md (or create it). Find and replace the `## Deploy Configuration` section164if it exists, or append it at the end.165fix Add an explicit AskUserQuestion step immediately before writing to CLAUDE.md that displays the complete configuration section to be written and requires the user to confirm (e.g., 'Review the configuration above. Confirm to write to CLAUDE.md? (yes/no)'). Make this confirmation mandatory and non-bypassable.
- 04SQP-270%MEDIUM
SKILL.md.tmpl:89
detailhide
The skill executes multiple bash commands (platform detection, health checks, deploy status verification) without consistent inline logging or confirmation checkpoints. Users may not see what commands are being executed or their results in real-time, reducing transparency and making it difficult to audit what the skill is doing. This is especially concerning for commands that interact with external services or infrastructure.
87881. Extract app name: `grep -m1 "^app" fly.toml | sed 's/app = "\(.*\)"/\1/'`892. Check if `fly` CLI is installed: `which fly 2>/dev/null`903. If installed, verify: `fly status --app {app} 2>/dev/null`914. Infer URL: `https://{app}.fly.dev`fix Wrap each major bash execution block with inline logging that shows: (1) what command is about to run, (2) the command itself (sanitized of secrets), and (3) the result. Use AskUserQuestion to confirm before executing commands that access external systems or modify infrastructure state.