Skip to content

[workshop-diagrams] Add theme-aware explanatory diagram for skill injection strategies - #3145

Merged
pelikhan merged 1 commit into
mainfrom
add-skill-strategy-diagram-48032592ee17f38c
Sep 10, 2026
Merged

[workshop-diagrams] Add theme-aware explanatory diagram for skill injection strategies#3145
pelikhan merged 1 commit into
mainfrom
add-skill-strategy-diagram-48032592ee17f38c

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Summary

Step 29 (Teach Your Agent Domain Knowledge with Skills) explained the hint/fusion/inline skill injection strategies only in prose bullet points. This adds a conceptual diagram to make the concept easier to learn.

  • File changed: workshop/29-skills-and-domain-knowledge.md (added a theme-aware <picture> block right after the strategy bullets; no other prose was touched)
  • New concept, not a migration: the file had zero existing images (image_count: 0 in the diagram state), so this creates a brand-new diagram rather than migrating one
  • SVGs added:
    • workshop/images/29-skill-injection-strategies-light.svg
    • workshop/images/29-skill-injection-strategies-dark.svg

Why this helps learners

The step asks learners to choose between three skill injection strategies (hint, fusion, inline) based on scope and context budget, but that relationship was only described in bullet text. The diagram shows the shared SKILL.md source flowing through all three strategies into the workflow prompt the agent reads, with a numbered left-to-right reading order (1 → 2 → 3) so the flow is clear even without color.

QA performed

  • Ran node scripts/check-svg-visual-language.js against both new files — no violations
  • Served the workshop directory locally and used Playwright to render the exact <picture> block:
    • colorScheme: "light" → confirmed currentSrc ends with -light.svg
    • colorScheme: "dark" → confirmed currentSrc ends with -dark.svg
  • Visually inspected both renders: all labels stay inside their boxes/chips, connectors terminate at nodes, no clipping or overflow

Warning

Firewall blocked 6 domains

The following domains were blocked by the firewall during workflow execution:

  • accounts.google.com
  • android.clients.google.com
  • clients2.google.com
  • registry.npmjs.org
  • www.google.com
  • www.gstatic.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "accounts.google.com"
    - "android.clients.google.com"
    - "clients2.google.com"
    - "registry.npmjs.org"
    - "www.google.com"
    - "www.gstatic.com"

See Network Configuration for more information.

Generated by 🧠 Workshop Explanatory Diagram Generator · copilot · auto · 108.7 AIC · ⌖ 8.53 AIC · ⊞ 10.8K ·

  • expires on Sep 11, 2026, 2:41 AM UTC

Adds a light/dark SVG pair illustrating how a SKILL.md convention flows
through the hint, fusion, and inline strategies into the workflow prompt,
and references it from step 29 with a theme-aware <picture> block.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@pelikhan
pelikhan marked this pull request as ready for review September 10, 2026 13:15
@pelikhan
pelikhan merged commit 378d042 into main Sep 10, 2026
1 check passed
@pelikhan
pelikhan deleted the add-skill-strategy-diagram-48032592ee17f38c branch September 10, 2026 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant