Skip to content

fix(ci): grant PR write permission to dogfood comment writer - #1586

Open
afscrome wants to merge 3 commits into
mainfrom
codex/fix-dogfood-comment-permissions
Open

fix(ci): grant PR write permission to dogfood comment writer#1586
afscrome wants to merge 3 commits into
mainfrom
codex/fix-dogfood-comment-permissions

Conversation

@afscrome

Copy link
Copy Markdown
Contributor

Summary

  • Grant the dogfood comment writer pull-requests: write.
  • Allow it to create or update the marker comment on pull requests.

Problem

The failed job https://github.com/CommunityToolkit/Aspire/actions/runs/34688286328/job/103539028007 downloaded its artifact successfully, then failed on POST /repos//issues/1585/comments. GitHub returned 403 Resource not accessible by integration.

The job token had Issues: write but PullRequests: read. This grants the missing pull-request write scope required for PR comments.

Validation

  • git diff --check

Copilot AI lite review requested due to automatic review settings September 12, 2026 10:34

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

No unresolved issues were identified.

Pull request overview

Updates the dogfood comment writer workflow to allow creating or updating pull-request comments.

Changes:

  • Changes pull-requests permission from read to write.
File summaries
File Description
.github/workflows/dogfood-comment-writer.yml Grants required pull-request write access.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

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

@afscrome
afscrome enabled auto-merge September 12, 2026 10:39
@afscrome

Copy link
Copy Markdown
Contributor Author

CI failures are pre-existing issues which should be fixed by #1585 .

@github-actions

Copy link
Copy Markdown
Contributor

Code Coverage

Package Line Rate Branch Rate Complexity Health
CommunityToolkit.Aspire.Bitwarden.SecretManager 54% 38% 52
CommunityToolkit.Aspire.DuckDB.Api 99% 100% 13
CommunityToolkit.Aspire.DuckDB.NET.Data 67% 77% 32
CommunityToolkit.Aspire.GoFeatureFlag 100% 97% 44
CommunityToolkit.Aspire.Hosting.ActiveMQ 88% 48% 95
CommunityToolkit.Aspire.Hosting.ActiveMQ.MassTransit 100% 100% 15
CommunityToolkit.Aspire.Hosting.Adminer 89% 70% 20
CommunityToolkit.Aspire.Hosting.Azure.Dapr 28% 5% 38
CommunityToolkit.Aspire.Hosting.Azure.Dapr.Redis 56% 46% 26
CommunityToolkit.Aspire.Hosting.Azure.DataApiBuilder 85% 83% 18
CommunityToolkit.Aspire.Hosting.Azure.Extensions 64% 30% 27
CommunityToolkit.Aspire.Hosting.Bitwarden.SecretManager 59% 51% 777
CommunityToolkit.Aspire.Hosting.Bun 88% 69% 20
CommunityToolkit.Aspire.Hosting.Dapr 43% 25% 648
CommunityToolkit.Aspire.Hosting.DbGate 96% 62% 12
CommunityToolkit.Aspire.Hosting.Dbx 94% 59% 68
CommunityToolkit.Aspire.Hosting.Deno 95% 85% 24
CommunityToolkit.Aspire.Hosting.DuckDB 98% 90% 16
CommunityToolkit.Aspire.Hosting.Elasticsearch.Extensions 100% 94% 30
CommunityToolkit.Aspire.Hosting.Flagd 80% 100% 16
CommunityToolkit.Aspire.Hosting.Floci 94% 87% 274
CommunityToolkit.Aspire.Hosting.Floci.ApiService 65% 60% 67
CommunityToolkit.Aspire.Hosting.Flyway 88% 100% 6
CommunityToolkit.Aspire.Hosting.GoFeatureFlag 80% 50% 24
CommunityToolkit.Aspire.Hosting.GoFeatureFlag.ApiService 100% 100% 3
CommunityToolkit.Aspire.Hosting.Java 86% 75% 207
CommunityToolkit.Aspire.Hosting.Java.ApiApp 65% 50% 11
CommunityToolkit.Aspire.Hosting.Java.WebApp 25% 18% 59
CommunityToolkit.Aspire.Hosting.JavaScript.Extensions 92% 83% 190
CommunityToolkit.Aspire.Hosting.K3s 76% 58% 317
CommunityToolkit.Aspire.Hosting.k6 70% 10% 8
CommunityToolkit.Aspire.Hosting.k6.ApiService 64% 67% 13
CommunityToolkit.Aspire.Hosting.Keycloak.Extensions 100% 100% 11
CommunityToolkit.Aspire.Hosting.Kind 89% 75% 659
CommunityToolkit.Aspire.Hosting.KurrentDB 86% 88% 19
CommunityToolkit.Aspire.Hosting.LavinMQ 90% 83% 18
CommunityToolkit.Aspire.Hosting.LavinMQ.MassTransit 100% 100% 15
CommunityToolkit.Aspire.Hosting.Listmonk 95% 100% 24
CommunityToolkit.Aspire.Hosting.Logto 73% 67% 24
CommunityToolkit.Aspire.Hosting.MailPit 91% 100% 13
CommunityToolkit.Aspire.Hosting.McpInspector 85% 60% 100
CommunityToolkit.Aspire.Hosting.McpInspector.McpServer 89% 100% 2
CommunityToolkit.Aspire.Hosting.Meilisearch 95% 80% 30
CommunityToolkit.Aspire.Hosting.Meilisearch.ApiService 82% 100% 15
CommunityToolkit.Aspire.Hosting.MongoDB.Extensions 88% 81% 18
CommunityToolkit.Aspire.Hosting.Mosquitto 90% 67% 17
CommunityToolkit.Aspire.Hosting.MySql.Extensions 94% 88% 44
CommunityToolkit.Aspire.Hosting.Ngrok 55% 43% 106
CommunityToolkit.Aspire.Hosting.Ollama 77% 67% 267
CommunityToolkit.Aspire.Hosting.OpenTelemetryCollector 79% 70% 42
CommunityToolkit.Aspire.Hosting.PapercutSmtp 100% 100% 9
CommunityToolkit.Aspire.Hosting.Perl 88% 79% 745
CommunityToolkit.Aspire.Hosting.Posta 97% 80% 304
CommunityToolkit.Aspire.Hosting.PostgreSQL.Extensions 91% 87% 54
CommunityToolkit.Aspire.Hosting.PowerShell 76% 59% 107
CommunityToolkit.Aspire.Hosting.Python.Extensions 46% 31% 44
CommunityToolkit.Aspire.Hosting.RavenDB 69% 49% 131
CommunityToolkit.Aspire.Hosting.RavenDB.ApiService 21% 33% 16
CommunityToolkit.Aspire.Hosting.Redis.Extensions 92% 68% 22
CommunityToolkit.Aspire.Hosting.RedPanda 99% 75% 50
CommunityToolkit.Aspire.Hosting.RedPanda.Consumer 69% 83% 13
CommunityToolkit.Aspire.Hosting.Rust 96% 90% 13
CommunityToolkit.Aspire.Hosting.RustFs 83% 66% 108
CommunityToolkit.Aspire.Hosting.SeaweedFS 88% 80% 49
CommunityToolkit.Aspire.Hosting.Sftp 90% 62% 15
CommunityToolkit.Aspire.Hosting.Sftp.ApiService 94% 100% 10
CommunityToolkit.Aspire.Hosting.Solr 88% 100% 19
CommunityToolkit.Aspire.Hosting.SqlDatabaseProjects 68% 59% 146
CommunityToolkit.Aspire.Hosting.Sqlite 88% 93% 25
CommunityToolkit.Aspire.Hosting.SqlServer.Extensions 86% 79% 48
CommunityToolkit.Aspire.Hosting.Squad 94% 80% 95
CommunityToolkit.Aspire.Hosting.Squad.ApiApp 63% 16% 41
CommunityToolkit.Aspire.Hosting.StableDiffusionCpp 55% 47% 93
CommunityToolkit.Aspire.Hosting.Stripe 50% 17% 119
CommunityToolkit.Aspire.Hosting.SurrealDb 89% 74% 200
CommunityToolkit.Aspire.Hosting.SurrealDb.ApiService 54% 38% 508
CommunityToolkit.Aspire.Hosting.Umami 98% 75% 9
CommunityToolkit.Aspire.Hosting.Zitadel 97% 86% 31
CommunityToolkit.Aspire.KurrentDB 97% 95% 33
CommunityToolkit.Aspire.Logto.Client 53% 62% 38
CommunityToolkit.Aspire.MassTransit.RabbitMQ 100% 100% 24
CommunityToolkit.Aspire.Meilisearch 97% 96% 38
CommunityToolkit.Aspire.Microsoft.Data.Sqlite 94% 85% 26
CommunityToolkit.Aspire.Microsoft.EntityFrameworkCore.Sqlite 71% 73% 117
CommunityToolkit.Aspire.OllamaSharp 78% 74% 76
CommunityToolkit.Aspire.Posta 28% 61% 1083
CommunityToolkit.Aspire.RavenDB.Client 96% 73% 87
CommunityToolkit.Aspire.SeaweedFS.Client 95% 86% 93
CommunityToolkit.Aspire.Sftp 83% 76% 54
CommunityToolkit.Aspire.Sqlite.Api 93% 93% 68
CommunityToolkit.Aspire.SurrealDb 86% 75% 38
SeaweedFS.ApiService 100% 100% 15
Summary 75% (15798 / 20962) 61% (4117 / 6718) 9408

@afscrome
afscrome deployed to azure-artifacts September 12, 2026 12:40 — with GitHub Actions Active
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.

2 participants