|
30 | 30 | REPO: ${{ github.repository }} |
31 | 31 | run: | |
32 | 32 | MARKER="<!-- api-change-detector -->" |
| 33 | + BREAKING_LABEL="breaking-api-change" |
33 | 34 |
|
34 | 35 | api_files=$(gh api "repos/${REPO}/pulls/${PR_NUMBER}/files" --paginate \ |
35 | 36 | --jq '.[] | select(.filename | startswith("docs/apidiffs/current_vs_latest/")) | .filename') |
|
40 | 41 | if [[ -z "$api_files" ]]; then |
41 | 42 | echo "No API diff files changed." |
42 | 43 | gh pr edit "$PR_NUMBER" --repo "$REPO" --remove-label "api-change" 2>/dev/null || true |
| 44 | + gh pr edit "$PR_NUMBER" --repo "$REPO" --remove-label "$BREAKING_LABEL" 2>/dev/null || true |
43 | 45 | if [[ -n "$comment_id" ]]; then |
44 | 46 | gh api --method DELETE "repos/${REPO}/issues/comments/${comment_id}" |
45 | 47 | fi |
|
57 | 59 | | sort \ |
58 | 60 | | sed 's/^/- /') |
59 | 61 |
|
60 | | - body=$(cat <<EOF |
| 62 | + HEAD_SHA=$(gh api "repos/${REPO}/pulls/${PR_NUMBER}" --jq .head.sha) |
| 63 | + breaking_files=() |
| 64 | + while IFS= read -r file; do |
| 65 | + [[ -z "$file" ]] && continue |
| 66 | + content=$(gh api \ |
| 67 | + -H "Accept: application/vnd.github.raw" \ |
| 68 | + "repos/${REPO}/contents/${file}?ref=${HEAD_SHA}") |
| 69 | + if grep -Eq '^[[:space:]]*(\*\*\*!|---!|\+\+\+!)' <<<"$content"; then |
| 70 | + breaking_files+=("$file") |
| 71 | + fi |
| 72 | + done <<<"$api_files" |
| 73 | +
|
| 74 | + if [[ ${#breaking_files[@]} -gt 0 ]]; then |
| 75 | + gh pr edit "$PR_NUMBER" --repo "$REPO" --add-label "$BREAKING_LABEL" |
| 76 | + breaking_modules=$(printf '%s\n' "${breaking_files[@]}" \ |
| 77 | + | sed 's|docs/apidiffs/current_vs_latest/||' \ |
| 78 | + | sed 's|\.txt$||' \ |
| 79 | + | sort \ |
| 80 | + | sed 's/^/- /') |
| 81 | + body=$(cat <<EOF |
| 82 | + ${MARKER} |
| 83 | + ## :warning: Breaking API changes detected — maintainer review required |
| 84 | +
|
| 85 | + This PR modifies the published API diff for the following module(s): |
| 86 | +
|
| 87 | + ${modules} |
| 88 | +
|
| 89 | + The committed API diff contains breaking-change markers for: |
| 90 | +
|
| 91 | + ${breaking_modules} |
| 92 | +
|
| 93 | + Please review the changes in \`docs/apidiffs/current_vs_latest/\` carefully before approving. |
| 94 | + EOF |
| 95 | + ) |
| 96 | + else |
| 97 | + gh pr edit "$PR_NUMBER" --repo "$REPO" --remove-label "$BREAKING_LABEL" 2>/dev/null || true |
| 98 | +
|
| 99 | + body=$(cat <<EOF |
61 | 100 | ${MARKER} |
62 | 101 | ## :warning: API changes detected — maintainer review required |
63 | 102 |
|
|
67 | 106 |
|
68 | 107 | Please review the changes in \`docs/apidiffs/current_vs_latest/\` carefully before approving. |
69 | 108 | EOF |
70 | | - ) |
| 109 | + ) |
| 110 | + fi |
71 | 111 |
|
72 | 112 | if [[ -n "$comment_id" ]]; then |
73 | 113 | gh api --method PATCH "repos/${REPO}/issues/comments/${comment_id}" \ |
|
0 commit comments