Skip to content

Allow MauiBot to trigger docs-from-code workflow#3341

Open
jfversluis wants to merge 2 commits into
mainfrom
allow-mauibot-workflow-trigger
Open

Allow MauiBot to trigger docs-from-code workflow#3341
jfversluis wants to merge 2 commits into
mainfrom
allow-mauibot-workflow-trigger

Conversation

@jfversluis
Copy link
Copy Markdown
Member

Problem

The 'Docs from Create Draft PR' workflow fails when triggered by MauiBot opening an issue because MauiBot only has read permission on this repo. The pre_activation job's permission check requires admin, maintainer, or write, so all subsequent jobs get skipped.Code

See: https://github.com/dotnet/docs-maui/actions/runs/25920414126

Fix

Add github.actor == 'MauiBot' as an alternative condition in the activated output. This allows MauiBot-triggered issues to proceed to the agent job without requiring write access, while still enforcing the permission check for all other users.

Alternative considered

Granting MauiBot write access at the org level would also fix this but requires org admin involvement.

MauiBot only has read permission on this repo, which causes the
pre_activation permission check to fail. Add MauiBot as an allowed
actor so issues it creates with '[maui-labs docs]' in the title
proceed to the agent job without requiring write access.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jfversluis jfversluis requested a review from a team as a code owner May 15, 2026 14:12
The if conditions check github.event.issue.title which doesn't exist
when triggered via workflow_dispatch, causing all jobs to be skipped.
Add github.event_name == 'workflow_dispatch' as an alternative condition.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 2b877f3:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/docs-from-code.lock.yml ✅Succeeded

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit fa54836:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/docs-from-code.lock.yml ✅Succeeded

For more details, please refer to the build report.

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.

1 participant