Skip to content

[workflow-editor] workflow-skills-editor: remove duplicated Octicon list in workshop-explanatory-diagrams #3477

Description

@github-actions

Caution

Protected files were modified in this change.
This pull request is in request_review mode and requires explicit human scrutiny before merge.

Protected files: .github/workflows/workshop-explanatory-diagrams.lock.yml, .github/workflows/workshop-explanatory-diagrams.md

What changed

.github/workflows/workshop-explanatory-diagrams.md — trimmed the "GitHub icon usage in diagrams" section.

Bloat removed

The workflow prompt restated a "Key Octicons" bullet list (icon names, colors, and entities) that duplicates the "Licensed Octicons" and "Primer semantic colors" tables already defined as the single source of truth in .github/workflows/guidelines.md. Keeping two copies of the same mapping risked drift if one was updated without the other.

How it's more Skills-like

The section now points directly to the canonical tables in guidelines.md instead of repeating them, matching the short, direct instructional style used elsewhere in this workflow (for example the SVG style rules already reference the guidelines file for semantic state colors). No behavior, safe outputs, permissions, or triggers changed.

Validation

Ran gh aw compile workshop-explanatory-diagrams --validate — compiles successfully (pre-existing, unrelated concurrency warning only). Regenerated .lock.yml is included in this PR.

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • api.github.com

[!TIP]
api.github.com is blocked because GitHub API access uses the built-in GitHub tools by default. Instead of adding api.github.com to network.allowed, use tools.github.mode: gh-proxy for direct pre-authenticated GitHub CLI access without requiring network access to api.github.com:

tools:
  github:
    mode: gh-proxy

See GitHub Tools for more information on gh-proxy mode.

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

network:
  allowed:
    - defaults
    - "api.github.com"

See Network Configuration for more information.

Generated by ✂️ Workflow Skills Editor · copilot · auto · 84.4 AIC · ⌖ 9.64 AIC · ⊞ 7.9K ·

  • expires on Sep 18, 2026, 7:39 AM UTC

Note

This was originally intended as a pull request, but the git push operation failed.

Original error: The process '/usr/bin/git' failed with exit code 1

Workflow Run: View run details and download bundle artifact

The bundle file is available in the agent artifact in the workflow run linked above.

Create the pull request manually
# Download the artifact from the workflow run
gh run download '35195034513' -n agent -D '/tmp/agent-35195034513'

# Resolve the bundle source ref, fetch it into a temporary ref, then create the local branch
bundle_path='/tmp/agent-35195034513/aw-workflow-editor-dedup-octicon-list.bundle'
temp_ref='refs/bundles/create-pr-workflow-editor-dedup-octicon-list-f7750eb736509945-5a990d3f'
target_ref='refs/heads/workflow-editor/dedup-octicon-list-f7750eb736509945'
bundle_source_ref=$(git bundle list-heads "$bundle_path" | awk '$2 ~ /^refs\/heads\// { print $2 }')
if [ -z "$bundle_source_ref" ]; then
  bundle_source_ref=$(git bundle list-heads "$bundle_path" | awk '$2 == "HEAD" { print $2 }')
fi
if [ "$(printf '%s\n' "$bundle_source_ref" | sed '/^$/d' | wc -l | tr -d ' ')" != "1" ]; then
  echo "Expected exactly one bundle source ref, found: $bundle_source_ref" >&2
  exit 1
fi
git fetch "$bundle_path" "${bundle_source_ref}:${temp_ref}"
git update-ref "$target_ref" "$temp_ref"
git checkout 'workflow-editor/dedup-octicon-list-f7750eb736509945'
# Ensure the working tree matches the updated branch
git reset --hard
# Remove the temporary bundle ref
git update-ref -d "$temp_ref"

# Push the branch to the target remote
git push 'origin' 'workflow-editor/dedup-octicon-list-f7750eb736509945'

# Create the pull request
gh pr create --title '[workflow-editor] workflow-skills-editor: remove duplicated Octicon list in workshop-explanatory-diagrams' --base 'main' --head 'workflow-editor/dedup-octicon-list-f7750eb736509945' --repo 'githubnext/gh-aw-workshop'

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions