Skip to content

Fix agentic-wiki-writer: fix backslash pipe escaping in wiki links#282

Merged
dsyme merged 7 commits intomainfrom
copilot/fix-push-wiki-call-issues
Mar 19, 2026
Merged

Fix agentic-wiki-writer: fix backslash pipe escaping in wiki links#282
dsyme merged 7 commits intomainfrom
copilot/fix-push-wiki-call-issues

Conversation

Copy link
Contributor

Copilot AI commented Mar 17, 2026

Wiki links were being generated with backslash-escaped pipes ([[Control Plane\|Control-Plane]]), which GitHub wiki doesn't support.

Changes

  • Bare pipe in wiki links — Added explicit rule in the Link Rules section and Self-Review Checklist: the | separator in [[Display Text|Page-Slug]] must be a bare pipe, never backslash-escaped.

Before (broken):

[[Control Plane\|Control-Plane]]   ← renders incorrectly

After (correct):

[[Control Plane|Control-Plane]]   ← bare pipe, GitHub wiki standard

Note: An earlier iteration attempted to add max to the custom push-wiki job config, but custom jobs under safe-outputs/jobs/ don't support the max property (it's only valid for standard safe-output actions). That invalid property was removed to fix CI compilation.


💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

…links

Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix multiple push_wiki calls during wiki generation Fix agentic-wiki-writer: single push-wiki call and bare pipe in wiki links Mar 17, 2026
Copilot AI requested a review from mrjf March 17, 2026 22:07
Copilot finished work on behalf of mrjf March 17, 2026 22:07
@mrjf
Copy link
Contributor

mrjf commented Mar 17, 2026

@copilot The thing is, we just did this: #279

So do we need the batching or not? Please do a deep analysis.

…pipe

Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 17, 2026

@copilot The thing is, we just did this: #279

So do we need the batching or not? Please do a deep analysis.

Deep analysis done — we need the batching from PR #279.

Root cause: The push-wiki job config had no max property, so safe-outputs defaulted to max: 1 — silently rejecting batches 2–5. Other workflows in this repo set explicit limits (e.g. repo-assist.md uses create-issue: max: 4, issue-arborist.md uses link-sub-issue: max: 50).

Tradeoff:

Fix in ed2f6c7: Reverted the single-call changes and restored PR #279's batching instructions. Added max: 5 to the push-wiki job config so safe-outputs accepts up to 5 batched calls per run. Still includes the backslash-pipe fix.

Copilot AI changed the title Fix agentic-wiki-writer: single push-wiki call and bare pipe in wiki links Fix agentic-wiki-writer: allow batched push-wiki calls and fix backslash pipe escaping Mar 17, 2026
Copilot finished work on behalf of mrjf March 17, 2026 22:32
@mrjf
Copy link
Contributor

mrjf commented Mar 17, 2026

@copilot make it max 10 to be safer

@mrjf
Copy link
Contributor

mrjf commented Mar 17, 2026

@copilot fix CI checks

Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 17, 2026

@copilot make it max 10 to be safer

Done — bumped to max: 10 in 14fe29a.

@mrjf mrjf marked this pull request as ready for review March 17, 2026 23:07
Copilot finished work on behalf of mrjf March 17, 2026 23:08
Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 17, 2026

@copilot fix CI checks

Fixed in 55c0787. The max property isn't valid for custom jobs under safe-outputs/jobs/ — it only applies to standard safe-output actions like add-comment, create-issue, etc. Removed it so CI passes.

Copilot AI changed the title Fix agentic-wiki-writer: allow batched push-wiki calls and fix backslash pipe escaping Fix agentic-wiki-writer: fix backslash pipe escaping in wiki links Mar 17, 2026
Copilot finished work on behalf of mrjf March 17, 2026 23:16
@dsyme dsyme merged commit c7e1542 into main Mar 19, 2026
1 check passed
@dsyme dsyme deleted the copilot/fix-push-wiki-call-issues branch March 19, 2026 00:44
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.

agentic-wiki-writer tries to call push_wiki multiple times

3 participants