From 7fe3ca84842c292b612e7902fe7c6ab0ea06a6d1 Mon Sep 17 00:00:00 2001 From: Gauvain <68083474+Gauvino@users.noreply.github.com> Date: Mon, 29 Sep 2025 22:31:41 +0200 Subject: [PATCH] feat: add automated PR comments for build artifacts (#1100) Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- .github/workflows/artifact-comment.yml | 103 +++++++++++++++---------- 1 file changed, 61 insertions(+), 42 deletions(-) diff --git a/.github/workflows/artifact-comment.yml b/.github/workflows/artifact-comment.yml index 112f7113..67e31268 100644 --- a/.github/workflows/artifact-comment.yml +++ b/.github/workflows/artifact-comment.yml @@ -1,12 +1,17 @@ name: 📝 Artifact Comment on PR concurrency: - group: artifact-comment-${{ github.event.workflow_run.head_branch }} - cancel-in-progress: true + group: artifact-comment-${{ github.event.workflow_run.id }} + cancel-in-progress: false on: + workflow_dispatch: workflow_run: - workflows: ["🤖 iOS IPA Build (Phone + TV)", "🤖 Android APK Build (Phone + TV)"] + workflows: + - "🏗️ Build iOS IPA (phone)" + - "🏗️ Build iOS IPA (tv)" + - "🏗️ Build Android APK (phone)" + - "🏗️ Build Android APK (tv)" types: - completed @@ -24,33 +29,39 @@ jobs: uses: actions/github-script@v8 with: script: | + console.log('Workflow run event:', JSON.stringify(github.event.workflow_run, null, 2)); + // Find PR associated with this commit const { data: pullRequests } = await github.rest.repos.listPullRequestsAssociatedWithCommit({ owner: context.repo.owner, repo: context.repo.repo, commit_sha: "${{ github.event.workflow_run.head_sha }}" }); - + if (pullRequests.length === 0) { - core.setFailed('No pull request found for this commit'); + console.log('No pull request found for commit: ${{ github.event.workflow_run.head_sha }}'); return; } - + const pr = pullRequests[0]; const runId = "${{ github.event.workflow_run.id }}"; - + + console.log(`Found PR #${pr.number} for commit ${pr.head.sha.substring(0, 7)}`); + // Get artifacts from the workflow run const { data: artifacts } = await github.rest.actions.listWorkflowRunArtifacts({ owner: context.repo.owner, repo: context.repo.repo, run_id: runId }); - - if (artifacts.artifacts.length === 0) { + + console.log(`Found ${artifacts?.artifacts?.length || 0} artifacts`); + + if (!artifacts || artifacts.artifacts.length === 0) { console.log('No artifacts found for this run'); return; } - + // Sort and categorize artifacts const androidArtifacts = artifacts.artifacts .filter(a => a.name.includes('android')) @@ -58,51 +69,59 @@ jobs: const iosArtifacts = artifacts.artifacts .filter(a => a.name.includes('ios')) .sort((a, b) => a.name.localeCompare(b.name)); - + + console.log(`Android artifacts: ${androidArtifacts.length}, iOS artifacts: ${iosArtifacts.length}`); + // Build comment body with table format let commentBody = `## 📱 Build Artifacts Ready!\n\n`; commentBody += `✅ **Workflow completed successfully** for PR #${pr.number}\n`; commentBody += `📦 **${artifacts.artifacts.length} artifacts** generated from commit [\`${pr.head.sha.substring(0, 7)}\`](https://github.com/${context.repo.owner}/${context.repo.repo}/commit/${pr.head.sha})\n\n`; - - // Create table for better organization - commentBody += `| Platform | Device Type | Download Link |\n`; - commentBody += `|----------|-------------|---------------|\n`; - - // Add Android artifacts - androidArtifacts.forEach(artifact => { - const isTV = artifact.name.includes('tv'); - const deviceType = isTV ? '📺 Android TV' : '📱 Android Phone'; - const nightlyLink = `https://nightly.link/${context.repo.owner}/${context.repo.repo}/actions/runs/${runId}/${artifact.name}.zip`; - commentBody += `| 🤖 Android | ${deviceType} | [📥 Download APK](${nightlyLink}) |\n`; - }); - - // Add iOS artifacts - iosArtifacts.forEach(artifact => { - const isTV = artifact.name.includes('tv'); - const deviceType = isTV ? '📺 Apple TV' : '📱 iPhone/iPad'; - const nightlyLink = `https://nightly.link/${context.repo.owner}/${context.repo.repo}/actions/runs/${runId}/${artifact.name}.zip`; - commentBody += `| 🍎 iOS | ${deviceType} | [📥 Download IPA](${nightlyLink}) |\n`; - }); - - commentBody += `\n`; - commentBody += `### 🔧 Installation Instructions\n\n`; - commentBody += `- **Android APK**: Download and install directly on your device (enable "Install from unknown sources")\n`; - commentBody += `- **iOS IPA**: Install using [AltStore](https://altstore.io/), [Sideloadly](https://sideloadly.io/), or Xcode\n\n`; - commentBody += `> ⚠️ **Note**: Artifacts expire in 7 days from build date\n\n`; - commentBody += `*Auto-generated by [GitHub Actions](${context.payload.workflow_run.html_url})*`; - + + if (androidArtifacts.length === 0 && iosArtifacts.length === 0) { + commentBody += `⚠️ No mobile app artifacts found in this build.\n\n`; + } else { + // Create table for better organization + commentBody += `| Platform | Device Type | Download Link |\n`; + commentBody += `|----------|-------------|---------------|\n`; + + // Add Android artifacts + androidArtifacts.forEach(artifact => { + const isTV = artifact.name.includes('tv'); + const deviceType = isTV ? '📺 Android TV' : '📱 Android Phone'; + const nightlyLink = `https://nightly.link/${context.repo.owner}/${context.repo.repo}/actions/runs/${runId}/${artifact.name}.zip`; + commentBody += `| 🤖 Android | ${deviceType} | [📥 Download APK](${nightlyLink}) |\n`; + }); + + // Add iOS artifacts + iosArtifacts.forEach(artifact => { + const isTV = artifact.name.includes('tv'); + const deviceType = isTV ? '📺 Apple TV' : '📱 iPhone/iPad'; + const nightlyLink = `https://nightly.link/${context.repo.owner}/${context.repo.repo}/actions/runs/${runId}/${artifact.name}.zip`; + commentBody += `| 🍎 iOS | ${deviceType} | [📥 Download IPA](${nightlyLink}) |\n`; + }); + + commentBody += `\n`; + commentBody += `### 🔧 Installation Instructions\n\n`; + commentBody += `- **Android APK**: Download and install directly on your device (enable "Install from unknown sources")\n`; + commentBody += `- **iOS IPA**: Install using [AltStore](https://altstore.io/), [Sideloadly](https://sideloadly.io/), or Xcode\n\n`; + commentBody += `> ⚠️ **Note**: Artifacts expire in 7 days from build date\n\n`; + } + + commentBody += `*Auto-generated by [GitHub Actions](https://github.com/${context.repo.owner}/${context.repo.repo}/actions/runs/${runId})*`; + commentBody += `\n`; + // Find existing bot comment to update const { data: comments } = await github.rest.issues.listComments({ owner: context.repo.owner, repo: context.repo.repo, issue_number: pr.number }); - + const botComment = comments.find(comment => comment.user.type === 'Bot' && - comment.body.includes('Build Artifacts Ready!') + comment.body.includes('') ); - + if (botComment) { // Update existing comment await github.rest.issues.updateComment({