Skip to content

docs: add optional Backer Concierge AI Agent exercise (CLI harness) - #177

Open
juliamuiruri4 wants to merge 8 commits into
mainfrom
copilot-cli-foundry-agent
Open

juliamuiruri4 wants to merge 8 commits into
mainfrom
copilot-cli-foundry-agent

Conversation

@juliamuiruri4

@juliamuiruri4 juliamuiruri4 commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds a new optional exercise to the copilot CLI harness where learners build a hosted Microsoft Foundry backer concierge AI agent. This step is clearly positioned as optional — the required workshop path (through the review exercise) does not depend on it in any way, and learners can skip it entirely.

Type of change

  • Workshop content (lesson Markdown, images)

Verification

  • Type checks and a clean site build pass.
  • Affected pages and translations match their source and intended routes.
  • Offline internal link checks pass.
  • Changed external links have been checked separately.

Screenshots

Backer Concierge chat widget recommending a game

22738d79-161a-40e4-9440-b4d28c4a98ab-tailspin-toys-backer-concierge-agent.mp4

Add lesson 8 (optional) walking learners through building a
catalog-grounded Backer Concierge hosted agent with GitHub Copilot
CLI and the Microsoft Foundry Skill, and renumber the review lesson
from 8 to 9 to make room for it.

- Wire the new lesson into the sidebar, harness README, lesson 7's
  closing navigation, and the review lesson, following the
  '<n>. <title> (Optional)' label convention and offering an explicit
  skip-ahead link for learners who aren't interested.
- Fix translated (es-es, ja-jp, ko-kr, pt-br, zh-cn) README and lesson
  7 next-lesson links that pointed at the now-renamed 8-review.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 15, 2026 22:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Several unresolved moderate issues remain in the lesson content, integration workflow, and localized navigation.

Get a fresh assessment by requesting another Copilot review.

Review tier: Lite
Findings: 10 Medium severity · 1 Low severity

Open (11)
What changed in this PR

Adds an optional Microsoft Foundry Backer Concierge AI Agent exercise to the CLI workshop and updates review navigation.

Changes:

  • Adds the Foundry exercise and integration workflow.
  • Updates English navigation and review numbering.
  • Updates localized CLI navigation references.
File Summary
website/​astro.config.mjs Adds the optional lesson to the CLI sidebar.
docs/​zh-cn/​cli/​README.md Updates localized exercise and review navigation.
docs/​zh-cn/​cli/​7-slash-commands.md Updates the localized next-lesson link.
docs/​pt-br/​cli/​README.md Updates localized exercise and review navigation.
docs/​pt-br/​cli/​7-slash-commands.md Updates the localized next-lesson link.
docs/​ko-kr/​cli/​README.md Updates localized exercise and review navigation.
docs/​ko-kr/​cli/​7-slash-commands.md Updates the localized next-lesson link.
docs/​ja-jp/​cli/​README.md Updates localized exercise and review navigation.
docs/​ja-jp/​cli/​7-slash-commands.md Updates the localized next-lesson link.
docs/​es-es/​cli/​README.md Updates localized exercise and review navigation.
docs/​es-es/​cli/​7-slash-commands.md Updates the localized next-lesson link.
docs/​cli/​README.md Adds Exercises 8 and 9 to the CLI overview.
docs/​cli/​9-review.md Renumbers the review exercise and links to the optional lesson.
docs/​cli/​8-foundry-agent.md Adds the Microsoft Foundry agent exercise.
docs/​cli/​7-slash-commands.md Links to the optional exercise and updated review.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/cli/8-foundry-agent.md
Comment thread docs/cli/8-foundry-agent.md Outdated
Comment thread docs/cli/8-foundry-agent.md Outdated
Comment thread docs/cli/8-foundry-agent.md Outdated
Comment thread docs/cli/8-foundry-agent.md Outdated
Comment thread docs/ja-jp/cli/README.md Outdated
Comment thread docs/ko-kr/cli/README.md Outdated
Comment thread docs/pt-br/cli/README.md Outdated
Comment thread docs/zh-cn/cli/README.md Outdated
Comment thread docs/cli/8-foundry-agent.md Outdated
juliamuiruri4 and others added 3 commits September 16, 2026 09:09
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
- Add a local-only Vite dev-server proxy instruction to the Foundry
  Function scaffolding prompt: the Astro site is output: 'static'
  with no proxy configured, so a relative POST /api/concierge from
  the browser was hitting the Astro origin instead of the separately
  running Azure Function and failing during local testing.
- Rename docs/<locale>/cli/8-review.md to 9-review.md for es-es,
  ja-jp, ko-kr, pt-br, and zh-cn so the translated review page's path
  mirrors the English docs/cli/9-review.md again. Previously only the
  README/nav link targets were repointed to 9-review/, which left the
  translated review content orphaned and unreachable.
- Add the missing optional Backer Concierge row (8) to each locale's
  README lesson table, falling back to the English lesson per this
  repo's untranslated-content convention, and renumber the review row
  to 9 so visible labels and link targets agree again.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
…er (cli)

tailspin-toys#101 adds .github/bootstrap-issues/09-backer-concierge.md,
so a fresh template instance auto-creates the Backer Concierge issue
as #9 — but the exact number still isn't guaranteed across repos or
learners since other issues could already exist. Drop the hardcoded
#14 placeholder and its 'replace this number' note, and have the
model-recommendation prompt reference the issue by its title instead
so Copilot can find it regardless of its actual number.

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread docs/cli/8-foundry-agent.md Outdated
Comment thread docs/cli/8-foundry-agent.md Outdated
Comment thread docs/cli/8-foundry-agent.md
Comment thread docs/cli/8-foundry-agent.md
Comment thread docs/cli/8-foundry-agent.md
Comment thread docs/es-es/cli/7-slash-commands.md
Comment thread docs/ja-jp/cli/7-slash-commands.md
Comment thread docs/ko-kr/cli/7-slash-commands.md
Comment thread docs/pt-br/cli/7-slash-commands.md
Comment thread docs/zh-cn/cli/7-slash-commands.md
juliamuiruri4 and others added 2 commits September 16, 2026 12:52
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: juliamuiruri4 <40116776+juliamuiruri4@users.noreply.github.com>
…graphs (cli)

The English 7-slash-commands.md closing paragraph offers both the
optional Foundry lesson and a direct skip to the review, but the 5
translated copies (es-es, ja-jp, ko-kr, pt-br, zh-cn) only ever
mentioned the review — they were never updated when the optional
lesson and its non-blocking dual-link pattern were added to English.
Add the same dual-link wording, translated per locale, with a new
foundry-lesson ref pointing to ../8-foundry-agent/.

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🔵 Needs a closer look

Localized review titles remain numbered 8, and the terminal step uses a shortcut that cannot create the required second terminal.

Review tier: Balanced
Findings: None

Resolved since last review (10)
Previously missed (1)

In code that hasn't changed since last review

Medium severity Use the new-terminal shortcut

docs/​cli/​8-foundry-agent.md:285

Ctrl+backtick only toggles the terminal panel; it does not create the second terminal this step requires. Include Shift, use “press” for the keyboard action, and spell out the Mac modifier so learners keep the Copilot CLI process running in the first terminal.

Comment thread docs/cli/README.md Outdated
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.

4 participants