Skip to content

fix(): Add package write permissions#112

Merged
kevintylerstark merged 2 commits into
mainfrom
fix/kevintylerstark/package-write
Jul 6, 2026
Merged

fix(): Add package write permissions#112
kevintylerstark merged 2 commits into
mainfrom
fix/kevintylerstark/package-write

Conversation

@kevintylerstark

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
src
   exceptions.py40100% 
   main.py871616 82%
src/clients/github
   graphql.py4022 95%
   queries.py20100% 
src/clients/slack
   client.py2333 87%
tests
   __init__.py00100% 
   conftest.py00100% 
   test_main.py570100% 
tests/clients/github
   test_github_graphql.py410100% 
tests/clients/slack
   test_slack.py370100% 
TOTAL2912193% 

Tests Skipped Failures Errors Time
24 0 💤 0 ❌ 0 🔥 0.351s ⏱️

@diff-vader-bot diff-vader-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Risk: LOW — no substantive findings

✅ Diff Vader · Approved

Council reviewed 1 file · 11 reviewers · 0 findings · $0.25

  • ✅ blast‑radius
  • ✅ correctness
  • ✅ github‑actions
  • ❓ iacskipped: filter: not applicable to changed files
  • ❓ migration‑safetyskipped: filter: not applicable to changed files
  • ✅ performance
  • ✅ revertibility
  • ✅ security‑tenant‑isolation
  • ❓ supply‑chainskipped: filter: not applicable to changed files
  • ✅ supportability
  • ✅ test‑adequacy

Why: All approval gates passed.

💡 /diff-vader-review <name> adds a one-shot reviewer. Beyond this PR's council, you can request: backup-correctness, loop-design-system.

@@ -13,6 +13,9 @@ env:
REGISTRY: ghcr.io
IMAGE_NAME: ${{ github.repository }}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pr-reviewer-github-actions — MEDIUM severity · confidence 85% (advisory)

New packages: write permission added to the top-level permissions: block. This grants the workflow the ability to publish packages to GHCR (or other GitHub Packages registries). Verify this is intentional and scoped to only the jobs that require it — if only the docker-action job needs to push images, consider moving this permission to the job-level permissions: block to limit blast radius.

React with 👍 / 👎 / 😕 to help us calibrate — why.

@kevintylerstark
kevintylerstark merged commit 49991d4 into main Jul 6, 2026
4 checks passed
@kevintylerstark
kevintylerstark deleted the fix/kevintylerstark/package-write branch July 6, 2026 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants