Skip to content

Re-architect Step 7–9 agent capabilities - #9

Merged
Pierre Malarme (pmalarme) merged 1 commit into
mainfrom
pmalarme-scaling-tribble
Jul 7, 2026
Merged

Re-architect Step 7–9 agent capabilities#9
Pierre Malarme (pmalarme) merged 1 commit into
mainfrom
pmalarme-scaling-tribble

Conversation

@pmalarme

Copy link
Copy Markdown
Collaborator

Summary

Re-architects the Step 7 multi-agent Coordinator/specialists and propagates the same capability split consistently through Steps 8 (workflow) and 9 (memory).

Capability changes

  • Coordinator (Step 7) / finalize_itinerary step (Steps 8–9) now own both skills — the local travel-guide skill (renders the PDF trip guide) and the Foundry response-guardrails skill — and always use them.
  • Hotels gains web search via the Foundry toolbox.
  • Flights now always reports concrete fares/prices.
  • Activities uses RAG + web only — the skills were removed.
  • Guardrails are Coordinator/finalize-only (removed from the Activities specialist).

Consistency & housekeeping

  • Kept the three surfaces in sync: docs → solutions → step_files/07 scaffold.
  • Rewrote the 3 specialist prompt constants + their agent.yaml slices, and updated the agent.manifest.yaml files (specialist + root) to match the new ownership.
  • Added agent-framework-orchestrations to step_files/00/requirements.txt.
  • Added a Tip in the Step 7 doc to prefer full-size gpt-5.4 over -mini (context grows large with multi-agent handoffs).
  • Reconciled mermaid diagrams, capability tables, and prose in the 07/08/09 docs and READMEs; carried the "skipped the Foundry skill?" fallback note into the Step 8 finalize section and added a sentence explaining why finalize is excluded from memory.

Validation

  • python .workshop/scripts/lint_steps.py0 failures (2 pre-existing unrelated warnings).
  • Custom parity check: every specialist prompt constant matches its slice verbatim; step_files/07 matches the Step 7 solution; all edited Python parses and all YAML loads.
  • A full rubber-duck review found no blocking issues; its non-blocking findings (stale manifest/README wording) are addressed in this branch.

Co-authored-by: Copilot App 223556219+Copilot@users.noreply.github.com

Move the travel-guide and response-guardrails skills onto the Coordinator (Step 7) and the finalize_itinerary step (Steps 8-9); give Hotels web search via the toolbox; have Flights always report fares; make Activities RAG + web only (no skills); keep guardrails Coordinator/finalize-only. Sync constants, slices, manifests, docs, and READMEs across steps 7-9 and add a full-size gpt-5.4 model hint.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@pmalarme
Pierre Malarme (pmalarme) merged commit 3f6e745 into main Jul 7, 2026
9 checks passed
@pmalarme
Pierre Malarme (pmalarme) deleted the pmalarme-scaling-tribble branch July 7, 2026 21:35
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