Skip to content

feat(scripts): add 46 workshop screenshots and fix capture script#3

Merged
emmanuelknafo merged 5 commits intomainfrom
feature/2132-screenshot-automation
Mar 29, 2026
Merged

feat(scripts): add 46 workshop screenshots and fix capture script#3
emmanuelknafo merged 5 commits intomainfrom
feature/2132-screenshot-automation

Conversation

@emmanuelknafo
Copy link
Copy Markdown
Contributor

Summary

Captured all 46 workshop screenshots for labs 00-07 and fixed the capture-screenshots.ps1 script with Playwright auth, Phase parameter, Windows-compatible commands, and working directory management.

Changes

Script Fixes (Phase 1)

  • Added Playwright auth state support with --load-storage for GitHub and Azure Portal screenshots
  • Added Phase parameter (-Phase 1|2|3) with PhaseMap and Test-ShouldCapture helper for phased execution
  • Fixed PSRule version command from Invoke-PSRule --version to Get-Module PSRule -ListAvailable
  • Fixed Windows-incompatible cat | python -m json.tool | head -30 to PowerShell-native Get-Content | ConvertFrom-Json | ConvertTo-Json
  • Added Push-Location/Pop-Location for scan commands requiring demo-app working directory
  • Fixed freeze --execute on Windows by wrapping commands in temp .ps1 files
  • Fixed New-LabDirectory to return absolute paths via Resolve-Path
  • Fixed freeze --lines format from hyphenated to comma-separated
  • Fixed PSRule scan screenshots to use Invoke-CapturedFreezeScreenshot with inline options
  • Fixed Infracost file references for diff and SARIF screenshots

Screenshots Captured

  • Phase 1 (29 PNGs): Labs 00-07 no-deploy screenshots (tool versions, file captures, Bicep, scans)
  • Phase 2 (8 PNGs): Post-deploy screenshots (Azure Portal resource groups/tags, Cloud Custodian live scans)
  • Phase 3 (9 PNGs): Post-scan screenshots (GitHub Security tab, code scanning alerts, workflow runs)

Infrastructure Lifecycle

  • Deployed 5 Azure resource groups via deploy-all workflow
  • Ran finops-scan workflow uploading SARIF to Security tabs
  • Torn down all resources via teardown-all workflow

Related Issues

Fixes AB#2132

Notes

  • lab-06-gh-api-upload.png shows a 422 error because commit_sha=HEAD is a literal string; SARIF was already uploaded by the scan workflow
  • c7n-azure installation deferred due to Windows long path limit; core c7n installed

- Added lab-07-oidc-setup.png (3948 KB) to illustrate the OIDC setup process.
- Added lab-07-scan-workflow.png (1389 KB) to demonstrate the scan workflow.
…hot capture logic in capture-screenshots.ps1
- add 4 lab-06 screenshots: gh-api-upload, security-tab, alert-detail, alert-triage
- add 5 lab-07 screenshots: workflow-run, matrix-jobs, sarif-artifacts, cost-gate-pr, deploy-teardown

📸 - Generated by Copilot
@emmanuelknafo emmanuelknafo merged commit c681ea3 into main Mar 29, 2026
@emmanuelknafo emmanuelknafo deleted the feature/2132-screenshot-automation branch March 29, 2026 03:16
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