Skip to content

Fix Archie’s nested Mermaid fences - #59311

Merged
pelikhan merged 3 commits into
mainfrom
copilot/fix-archer-comment-template
Sep 8, 2026
Merged

pelikhan merged 3 commits into
mainfrom
copilot/fix-archer-comment-template

Conversation

Copilot AI commented Sep 7, 2026 •

Copy link
Copy Markdown
Contributor

Archie’s comment template escaped nested Mermaid fences, causing agents to emit literal backslashes and preventing GitHub from rendering diagrams.

  • Template
    • Wrap the Markdown example in a four-backtick fence.
    • Use standard, unescaped Mermaid fences inside it.
```mermaid
graph TD
    Start --> Finish
```
  • Generated workflow
    • Refresh the compiled workflow metadata for the updated prompt body.

pr-sous-chef

Branch refresh requested by pr-sous-chef: https://github.com/github/gh-aw/actions/runs/34170823680

Warning

Firewall blocked 1 domain

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

  • github.com

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

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

See Network Configuration for more information.

Generated by 👨‍🍳 PR Sous Chef · pi · gpt54 · 30.9 AIC · ⌖ 8.68 AIC · ⊞ 9.2K · ◷
Comment /souschef to run again

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix comment template for Mermaid blocks in Archie's workflow Fix Archie’s nested Mermaid fences Sep 7, 2026
Copilot AI requested a review from pelikhan September 7, 2026 23:22
@pelikhan
pelikhan marked this pull request as ready for review September 7, 2026 23:36
Copilot AI balanced review requested due to automatic review settings September 7, 2026 23:36

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.

🟢 Approval recommended

The focused formatting and metadata updates have no unresolved issues.

Pull request overview

Fixes Archie’s nested Mermaid example so generated diagrams render correctly.

Changes:

  • Uses a four-backtick outer fence with standard Mermaid fences.
  • Refreshes compiled workflow metadata.
File summaries
File Description
.github/workflows/archie.md Corrects nested Mermaid fence formatting.
.github/workflows/archie.lock.yml Updates the generated body hash.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@gh-aw-bot

Copy link
Copy Markdown
Collaborator
pr-sous-chef

@copilot PR #59311 is eligible for a maintainer-ready pass. Please run the pr-finisher skill, re-check branch freshness against main, and report any remaining merge/test/review blockers in one concise update.

Warning

Firewall blocked 1 domain

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

  • github.com

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

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

See Network Configuration for more information.

Generated by 👨‍🍳 PR Sous Chef · pi · gpt54 · 30.9 AIC · ⌖ 8.68 AIC · ⊞ 9.2K · ◷
Comment /souschef to run again

Copilot AI commented Sep 7, 2026

Copy link
Copy Markdown
Contributor Author
pr-sous-chef
```...
  • ✅ Reviews — 1 approval from copilot-pull-request-reviewer[bot], 0 unresolved review threads.
  • ✅ Checks (local) — make fmt, make lint, make test-unit, and make recompile all passed locally.
  • ❓ Checks (CI) — stale after agent push; needs human re-trigger.
  • ✅ Mergeable — PR Fix Archie’s nested Mermaid fences #59311 is OPEN, non-draft, clean of merge conflicts, and up to date with main (8c9a5eb).

Actions taken: Performed pr-finisher pass, re-checked branch freshness against main, and validated code formatting, linting, unit tests, and workflow compilation locally.
Hand-off: CI must be re-triggered by a maintainer (close/reopen PR, workflow_dispatch, or push) before merge.
Still needed: Human maintainer CI re-trigger and final merge approval.

Copilot AI requested a review from gh-aw-bot September 7, 2026 23:59
@pelikhan
pelikhan merged commit 0e835d3 into main Sep 8, 2026
1 check passed
@pelikhan
pelikhan deleted the copilot/fix-archer-comment-template branch September 8, 2026 01:35
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.88.7

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.

fix(archie): use 4-backtick fence in comment template to avoid literal backslashes in Mermaid blocks

4 participants