/azure-app-onboard

Approval Gates — Steps 6 & 8

Gate summary: AppOnboard has 2 approval gates: (1) Scaffold Gate (orchestrator Step 6) — approve architecture plan before generating IaC, (2) Deploy Gate (orchestrator Step 8 / deploy/SKILL.md Step 4) — approve cost + resource summary before az deployment. Both are mandatory and SEPARATE — scaffold approval does NOT imply deploy approval.

Scaffold Approval Gate (Step 6)

Display the architecture plan for user approval BEFORE generating any files:

⛔ Resource group edit is MANDATORY in the gate display. Show this exact block:

🏢 **Subscription:** {subscriptionName} (`{subscriptionId}`)
📦 **Resource Group:** {rg-name} ({region})
   Want a different name or region? Say "Edit plan".

⛔ Gate MUST show Subscription (name + ID), Resource Group, and Region as standalone lines above the service table — see pipeline-rules.md § Approval gates. Do NOT omit or bury in a table.

Also display: services + SKUs + region + resource names + monthly cost estimate + files to generate. Check context.json.overrides[] for iacFormat — if Terraform, display "Terraform templates (infra/*.tf)"; if Bicep (default), display "Bicep templates (infra/main.bicep)". Show resource names so the user sees what will be created.

⛔ Surface plan assumptions. If prepare-plan.json.assumptions[] is present (e.g., free-tier degradation to a paid SKU), display each note prefixed with ⚠️ ABOVE the approval prompt — the user MUST see WHY the cost or SKU differs from the fast-track default. Do not bury or omit them.

Verify file list against target SKU — F1/D1: no Dockerfile (built-in runtime). ⛔ Exclude azure.yaml from file list (see pipeline-rules.md).

⛔ Container Apps code deploy preview (when plan includes Container Apps). After the service table, preview the deploy path: build via ACR, replace placeholder images, redeploy. If buildRequirements.hasBuildKitSyntax == true, note ACR-compatible versions will be created.

⛔ Data-loss warnings at the gate. If prereq-output.json.warnings[] contains any data-loss findings (SQLite on App Service, in-memory sessions, local file storage), surface them prominently with ⛔ formatting ABOVE the approval prompt. The user must see these before approving — do not bury them in a table or omit them.

⛔ Database network access (when plan includes PostgreSQL/MySQL). Show a Database access line above the approval prompt: the default AllowAllAzureServicesAndResourcesWithinAzureIps (0.0.0.0) lets the app connect but opens the server to all Azure services. ⛔ Render the exposure as its own bold sentence so a fast "Yes" is still an informed "Yes": "Proceeding opens the database to all Azure services — not just this app." Offer the alternative: "For private networking (VNet integration + private endpoint), say 'Private access' — AppOnboard does not build private networking itself, so it will hand off to azure-enterprise-infra-planner, which then owns the secure networking design and its deployment. AppOnboard stops here — it does not resume afterward." If the user proceeds (Yes), that counts as consent to the 0.0.0.0 rule.

⛔ Private networking redirect. If the user chooses Private access (here or during Edit plan): set context.json.routeToSkill: "azure-enterprise-infra-planner" and routeReason: "private-networking-requested", then HALT — do NOT generate IaC. Tell the user: "AppOnboard can't generate private networking (VNet + private endpoint). Handing off to azure-enterprise-infra-planner, which will design the secure topology and deploy it from here — it takes over the rest of the onboarding. Your AppOnboard session is saved for reference." Then invoke {"skill": "azure-enterprise-infra-planner"}. This mirrors the prereq routeToSkill halt (SKILL.md Step 3) but fires at the Scaffold Gate.

⛔ Pick the prompt variant FIRST (based on whether the plan has a database), then use it verbatim — do NOT paraphrase or reword:

• Plan includes PostgreSQL/MySQL → "✅ Ready to proceed with scaffolding? (Yes / Edit plan / Private access / Cancel)" — the ask_user choices MUST be exactly: Yes, Edit plan, Private access, Cancel.
• No database in plan → "✅ Ready to proceed with scaffolding? (Yes / Edit plan / Cancel)" — the ask_user choices MUST be exactly: Yes, Edit plan, Cancel.

The ask_user choices MUST match the prompt text exactly — never name an option in one and omit it from the other (e.g. Private access must be a selectable choice, not something the user has to type).

⛔ RESPONSE BOUNDARY — MANDATORY. The approval gate MUST be the LAST content in your response. Do NOT generate any files, write any IaC, create any Dockerfiles, or execute any commands in the same response as the gate. Your next action MUST be reading the user's reply. If the user has not yet responded, WAIT — do not proceed.

Three options:

• Yes → proceed to Step 7 (scaffold only — NOT deploy)
• Edit plan → ask what to change → write to context.json.overrides[] → re-run Step 5 → show updated gate
• Cancel → preserve session artifacts for later resumption, stop

Deploy Approval Gate (Step 8)

This is a SEPARATE gate from Step 6.

⛔ Context refresh: Before presenting this gate, re-read this SKILL.md Steps 8-9 if you have not read them in the last 5 turns. Scaffold reference loading (Steps 6-7) consumes significant context — deploy rules may have been evicted.

Display:

• Validation status from scaffold-manifest.json.validationResult
• Self-review summary (count of VERIFIED/PLAUSIBLE/FLAGGED findings)
• Resource group name + region
• Services + SKUs + estimated cost
• End with "🚀 Ready to deploy? (Yes / Run manually / Edit plan / Cancel)"

⛔ After deploy approval: Your NEXT action MUST be: read deploy/SKILL.md, then read .copilot-azure/sessions/{id}/deploy-checklist.md. Do NOT call the azure-deploy skill — AppOnboard uses its own embedded deploy sub-skill.

If "Run manually" is selected → point to deploy-checklist-template.md § Deployment Summary for manual execution steps.

If validation failed or any FLAGGED findings exist at L1 (Security) or L3 (Hallucination), block Yes until resolved.

Only after user approves: proceed to deploy sub-skill (Step 9). context.json already has currentPhase: "deploy" from the post-scaffold checkpoint (main SKILL.md Step 7).

⛔ Before entering deploy: ⛔ Read deploy/SKILL.md before any deployment action. After mid-session compaction, re-read deploy/SKILL.md Steps 4-8. You MUST write deploy-result.json.

⛔ Deploy via az deployment sub create (see pipeline-rules.md). AppOnboard-generated azure.yaml found → never delete/overwrite; move to .copilot-azure/sessions/<id>/replaced-files/ (mirror path).

⛔ Container Apps code deploy is NOT optional. After IaC placeholder deploys, complete: az acr build → update image params → redeploy → health check. Do NOT present manual CLI "Next Steps" for core deploy tasks. If hasBuildKitSyntax, create Dockerfile.azure first.

⛔ Phase exit: deploy-result.json MUST be written before proceeding to Step 9. See deploy/SKILL.md for the full exit gate checklist.