Skip to content

[workshop-diagrams] Add theme-aware explanatory diagram for safe-output limit flow - #4776

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
workshop-diagrams/safe-output-limit-flow-71d0b113788c5928
Closed

github-actions[bot] wants to merge 1 commit into
mainfrom
workshop-diagrams/safe-output-limit-flow-71d0b113788c5928

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

File changed: workshop/side-quest-09-01c-pattern-safe-output-blocked.md (one <picture> block added; no existing diagram needed migration, so this creates a new concept).

SVGs added:

  • workshop/images/side-quest-09-01c-safe-output-limit-flow-light.svg
  • workshop/images/side-quest-09-01c-safe-output-limit-flow-dark.svg

Why: The page explains E002 limit reached only in prose. The diagram shows the agent preparing two add_comment requests, the safe-outputs job allowing the first (max 1), and blocking the second. It also shows the next step: raise max or tighten the brief.

QA: Playwright confirmed currentSrc selects -light.svg and -dark.svg in the matching color schemes. Screenshots were nonblank with no text overflow. check-svg-visual-language.js passes. markdownlint could not run because npm returned a 403 in the sandbox.

Warning

Firewall blocked 7 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
  • update.googleapis.com
  • 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"
    - "update.googleapis.com"
    - "www.google.com"
    - "www.gstatic.com"

See Network Configuration for more information.

Generated by 🧠 Workshop Explanatory Diagram Generator · copilot · auto · 40.7 AIC · ⌖ 0.957 AIC · ⊞ 10.8K · ◷

  • expires on Oct 21, 2026, 2:38 AM UTC

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

Copy link
Copy Markdown
Contributor Author

This pull request was automatically closed because it expired on 2026-10-21T02:38:05.049Z.

Closed by Workflow

@github-actions github-actions Bot closed this Oct 21, 2026
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.

0 participants