feat: add workshop step 28 — Orchestrate Multiple Agentic Workflows - #2414
Conversation
- Create workshop/28-orchestrate-workflows.md covering dispatch-workflow safe-output orchestration pattern - Update workshop/README.md curriculum table with step 28 - Update workshop/27-evaluate-workflow-quality.md to add Next: link - Update simulator workshop-student-journey.js to add step 27 and 28 mappings (step 27 was missing, causing simulator test failure) Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Warning
Firewall blocked 1 domain
The following domain was blocked by the firewall during workflow execution:
awmgmcpg
To allow these domains, add them to the
network.allowedlist in your workflow frontmatter:
network:
allowed:
- defaults
- "awmgmcpg"See Network Configuration for more information.
Generated by ✅ AW Accuracy PR Reviewer for #2414 · 36.8 AIC · ⌖ 4.82 AIC · ⊞ 4.3K
| ``` | ||
|
|
||
| > [!TIP] | ||
| > Use `gh aw compile --watch` while you iterate on the brief so the lock file stays in sync without manual recompiles. |
There was a problem hiding this comment.
Inaccurate flag: gh aw compile --watch does not exist.
The gh aw compile command supports --strict, --validate, --fail-fast, --purge, and --approve, but has no --watch flag. Running this command as written will fail with an unknown flag error.
Suggested fix: Remove the tip, or replace with a plain recompile instruction:
gh aw compile repo-orchestrator
| 2. Check whether a GitHub issue with the title prefix "Daily Repository Status" was created today. | ||
|
|
||
| Based on what you find: | ||
| - If stale open PRs exist, dispatch the `daily-status` workflow. |
There was a problem hiding this comment.
Routing logic is reversed compared to the decision table above.
The decision table (lines 51-52) correctly maps:
- Stale open PRs → dispatch the PR reviewer
- No status issue today → dispatch the daily-status reporter
But this template brief reverses the mapping:
- Line 95:
If stale open PRs exist, dispatch thedaily-statusworkflow← should bepr-reviewer - Line 96:
If no status issue exists today, dispatch thepr-reviewerworkflow← should bedaily-status
The same reversal appears in the AI prompt block above (lines 62-63).
Suggested fix: Swap the target workflow names to match the decision table:
- If stale open PRs exist, dispatch the `pr-reviewer` workflow.
- If no status issue exists today, dispatch the `daily-status` workflow.
|
@copilot apply reviews from github action actor |
- Swap dispatch targets to match decision table: stale PRs → pr-reviewer, no status issue → daily-status (was reversed in both prompt block and terminal template) - Remove tip referencing gh aw compile --watch which does not exist Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Applied both reviews from the AW Accuracy PR Reviewer in commit
|
The workshop taught building individual workflows (sub-agents, memory, MCP, evals, cost controls) but had no step on composing them into a pipeline. The
dispatch-workflowsafe-output makes cross-workflow orchestration possible but appeared nowhere in the curriculum.Changes
workshop/28-orchestrate-workflows.md— New advanced step covering the orchestrator pattern: read repository signals, route to a specialist viadispatch-workflow, enforcemax: 1to keep dispatch predictable. Includes a decision-table design exercise, both AI-agent prompt and terminal paths, and compile/run/verify loop.workshop/README.md— Step 28 added to Part 2 curriculum table.workshop/27-evaluate-workflow-quality.md— Footer changed from "return to hub" toNext: Orchestrate Multiple Agentic Workflowsto preserve linear curriculum flow..github/skills/micro-environment-simulator/workshop-student-journey.js— Added step 27 and step 28 toSTEP_FILE_ALIASES,STEP_IDS, andbuildTransitions(). Step 27 was already missing from the simulator, causing the mapping assertion test to fail; this also fixes that pre-existing failure.Key frontmatter pattern introduced