kyush/misskey
1
0
Fork 0
forked from mirrors/misskey
Code Pull requests Projects Releases Packages 1 Activity Actions

fix(dev): tweak frontend-bundle-report-comment

Browse source
This commit is contained in:
syuilo 2026-06-22 22:11:46 +09:00
commit 5d8c31b6e5
1 changed files with 29 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

43
.github/workflows/frontend-bundle-report-comment.yml vendored
View file

@ -190,7 +190,7 @@ jobs:
const headShaPath = path.join(reportDir, 'head-sha.txt'); const headShaPath = path.join(reportDir, 'head-sha.txt');
const workflowRun = context.payload.workflow_run; const workflowRun = context.payload.workflow_run;
const pullRequest = context.payload.pull_request; const pullRequest = context.payload.pull_request;
const headSha = workflowRun?.head_sha ?? pullRequest?.head?.sha ?? null; const eventHeadSha = workflowRun?.head_sha ?? pullRequest?.head?.sha ?? null;
const { owner, repo } = context.repo; const { owner, repo } = context.repo;
if (!fs.existsSync(jsSizeReportPath)) { if (!fs.existsSync(jsSizeReportPath)) {
@ -205,10 +205,10 @@ jobs:
const artifactHeadSha = fs.existsSync(headShaPath) const artifactHeadSha = fs.existsSync(headShaPath)
? fs.readFileSync(headShaPath, 'utf8').trim() ? fs.readFileSync(headShaPath, 'utf8').trim()
: null; : null;
if (headSha != null && artifactHeadSha != null && artifactHeadSha !== headSha) { if (eventHeadSha != null && artifactHeadSha != null && artifactHeadSha !== eventHeadSha) {
core.setFailed(`The artifact head SHA (${artifactHeadSha}) does not match the workflow head SHA (${headSha}).`); core.info(`The artifact head SHA (${artifactHeadSha}) differs from the event head SHA (${eventHeadSha}). Using artifact metadata for PR validation.`);
return;
} }
const reportHeadSha = artifactHeadSha ?? eventHeadSha;
const artifactPrNumber = fs.existsSync(prNumberPath) const artifactPrNumber = fs.existsSync(prNumberPath)
? Number(fs.readFileSync(prNumberPath, 'utf8').trim()) ? Number(fs.readFileSync(prNumberPath, 'utf8').trim())
@ -228,25 +228,29 @@ jobs:
} }
} }
const pullRequestsForCommit = await github.paginate(github.rest.repos.listPullRequestsAssociatedWithCommit, { if (reportHeadSha != null) {
owner, const pullRequestsForCommit = await github.paginate(github.rest.repos.listPullRequestsAssociatedWithCommit, {
repo, owner,
commit_sha: headSha, repo,
per_page: 100, commit_sha: reportHeadSha,
}); per_page: 100,
for (const pullRequest of pullRequestsForCommit) { });
associatedPullRequests.set(pullRequest.number, pullRequest); for (const pullRequest of pullRequestsForCommit) {
associatedPullRequests.set(pullRequest.number, pullRequest);
}
} }
if (Number.isInteger(artifactPrNumber) && associatedPullRequests.has(artifactPrNumber)) { if (Number.isInteger(artifactPrNumber) && associatedPullRequests.has(artifactPrNumber)) {
issue_number = artifactPrNumber; issue_number = artifactPrNumber;
} else if (Number.isInteger(artifactPrNumber) && associatedPullRequests.size === 0) {
issue_number = artifactPrNumber;
} else if (!Number.isInteger(artifactPrNumber) && associatedPullRequests.size === 1) { } else if (!Number.isInteger(artifactPrNumber) && associatedPullRequests.size === 1) {
issue_number = [...associatedPullRequests.keys()][0]; issue_number = [...associatedPullRequests.keys()][0];
} else if (Number.isInteger(artifactPrNumber)) { } else if (Number.isInteger(artifactPrNumber)) {
core.setFailed(`The artifact pull request number (${artifactPrNumber}) is not associated with ${headSha}.`); core.setFailed(`The artifact pull request number (${artifactPrNumber}) is not associated with ${reportHeadSha}.`);
return; return;
} else { } else {
core.setFailed(`Could not determine the pull request associated with ${headSha}.`); core.setFailed(`Could not determine the pull request associated with ${reportHeadSha}.`);
return; return;
} }
} else { } else {
@ -254,6 +258,17 @@ jobs:
return; return;
} }
const currentPullRequest = await github.rest.pulls.get({
owner,
repo,
pull_number: issue_number,
});
const currentHeadSha = currentPullRequest.data.head?.sha;
if (reportHeadSha != null && currentHeadSha != null && reportHeadSha !== currentHeadSha) {
core.info(`The report head SHA (${reportHeadSha}) is not the current pull request head SHA (${currentHeadSha}). Skipping stale frontend bundle report.`);
return;
}
const jsSizeReport = fs.readFileSync(jsSizeReportPath, 'utf8').trim(); const jsSizeReport = fs.readFileSync(jsSizeReportPath, 'utf8').trim();
if (!jsSizeReport.includes(jsSizeMarker)) { if (!jsSizeReport.includes(jsSizeMarker)) {
core.setFailed('The frontend JS size report is missing the expected marker.'); core.setFailed('The frontend JS size report is missing the expected marker.');