[skill-activity] workshop: fix content issues in security and runner infrastructure side quests - #2682
Closed
github-actions[bot] wants to merge 1 commit into
Closed
Conversation
…de quests - side-quest-17-07: replace broken nested triple-backtick fence in the attack scenario with clear prose description; avoids markdown rendering failure that broke the attack scenario illustration - side-quest-24-01: correct network field name from 'network.allowed' to 'network.allowed-domains' to match the actual gh-aw frontmatter spec; also update the corresponding checkpoint item Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
|
This pull request was automatically closed because it expired on 2026-08-18T05:03:40.196Z.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What was improved
Two low-scoring security/infrastructure side quests had content correctness issues:
side-quest-17-07-repo-poisoning.md(Part 2 lowest-scoring file, 5.45/10):The attack scenario example contained nested triple-backtick fences inside an outer code block. This is invalid Markdown that would break rendering for learners — the attacker payload example was invisible or malformed. Replaced with clear prose that conveys the same information without the rendering bug.
side-quest-24-01-runner-infrastructure.md(Part 2, 6.13/10):The network isolation section used
network.allowedas the frontmatter field name, but the correct gh-aw field isnetwork.allowed-domains. A learner copying this example would get a silent misconfiguration with no network restriction applied. Fixed field name and updated the checkpoint item.Learner pain point addressed
Why it helps workshop flow
Both fixes prevent learners from hitting silent failures. The security page now clearly describes the attack, and the runner infrastructure page provides copy-pasteable frontmatter that actually works.
Warning
Firewall blocked 1 domain
The following domain was blocked by the firewall during workflow execution:
awmgmcpgSee Network Configuration for more information.