Skip to content

[workshop-sim] Add prerequisite pre-flight guardrails to Steps 1 and 6#67

Merged
pelikhan merged 10 commits into
mainfrom
copilot/workshop-sim-repair-add-prerequisite-checker
Jul 11, 2026
Merged

[workshop-sim] Add prerequisite pre-flight guardrails to Steps 1 and 6#67
pelikhan merged 10 commits into
mainfrom
copilot/workshop-sim-repair-add-prerequisite-checker

Conversation

Copilot AI commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Steps 1 and 6 were losing learners for the same reason: gh wasn’t reliably installed before gh-aw install time. This PR makes prerequisite checks explicitly blocking in Step 1 and adds a second early guardrail in Step 6.

  • Step 1: make prerequisites operational, not informational

    • Added a copy-paste pre-flight block that checks gh, git, and optional node in one run.
    • Reframed prerequisites as explicit Required vs Optional with visual markers (✅ / ℹ️).
    • Kept platform install commands as direct one-liners for macOS, Ubuntu/Debian, and Windows.
  • Step 6: add an early failure gate before install

    • Added an IMPORTANT pre-flight callout at the top of the step (same multi-check script pattern).
    • Added a one-line gh --version guard before extension install guidance.
    • Replaced generic “go back to Step 1” guidance with direct links to exact platform-specific install commands.
  • Consistency/clarity refinements

    • Aligned wording so fallback/error text points to the correct install-command locations.
    • Kept shell snippets copy-paste safe for interactive terminals.
# Pre-flight check -- run this before proceeding
gh --version && echo "✅ gh CLI OK" || echo "❌ gh CLI missing — see install commands below"
git --version && echo "✅ git OK" || echo "❌ git missing"
node --version 2>/dev/null && echo "✅ node OK" || echo "ℹ️  node not required but useful"
echo "Pre-flight complete."

Copilot AI and others added 9 commits July 11, 2026 13:42
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add pre-flight prerequisite checker to Step 1 [workshop-sim] Add prerequisite pre-flight guardrails to Steps 1 and 6 Jul 11, 2026
Copilot AI requested a review from pelikhan July 11, 2026 13:48
@pelikhan pelikhan marked this pull request as ready for review July 11, 2026 14:12
@pelikhan pelikhan merged commit 103920d into main Jul 11, 2026
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.

[workshop-sim] Repair: Add a pre-flight prerequisite checker to Step 1 to prevent Step 6 install failures

2 participants