Merge branch 'main' into feat/newsletter-hello-world

Author: mbifulco

.github/workflows/sync-newsletter.yml

@@ -10,6 +10,7 @@ jobs:
   sync:
     permissions:
       contents: read
+      pull-requests: write  # Allow commenting on PRs
     runs-on: ubuntu-latest
     timeout-minutes: 10
 
@@ -26,8 +27,6 @@ jobs:
 
       - name: Setup pnpm
         uses: pnpm/action-setup@v4
-        with:
-          version: 10
 
       - name: Install dependencies
         run: pnpm install --frozen-lockfile
@@ -43,9 +42,12 @@ jobs:
         uses: actions/github-script@v7
         with:
           script: |
-            github.rest.issues.createComment({
-              issue_number: context.issue.number,
+            github.rest.pulls.createReviewComment({
               owner: context.repo.owner,
               repo: context.repo.repo,
-              body: '❌ Newsletter sync to Resend failed. Check the [workflow logs](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}) for details.'
+              pull_number: context.issue.number,
+              body: '❌ Newsletter sync to Resend failed. Check the [workflow logs](https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}) for details.',
+              commit_id: context.payload.pull_request.head.sha,
+              path: 'src/data/newsletters',
+              line: 1
             })

package.json

@@ -4,7 +4,7 @@
   "version": "0.1.0",
   "author": "Mike Bifulco <hello@mikebifulco.com>",
   "license": "MIT",
-  "packageManager": "pnpm@10.19.0",
+  "packageManager": "pnpm@10.21.0",
   "scripts": {
     "analyze": "ANALYZE=true next build",
     "build": "next build",