Skip to content

Commit fb26784

Browse files
committed
Merge remote-tracking branch 'origin/main' into HEAD
# Conflicts: # tests/unit/ReportActionsUtilsTest.ts
2 parents 7ae30ad + 761a3f5 commit fb26784

655 files changed

Lines changed: 16416 additions & 7037 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.claude/agents/deploy-blocker-investigator.md

Lines changed: 14 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
---
22
name: deploy-blocker-investigator
33
description: Investigates deploy blockers to find the causing PR and recommend resolution.
4-
tools: Glob, Grep, Read, Bash, BashOutput
4+
tools: Glob, Grep, Read, Write, Bash, BashOutput
55
model: inherit
66
---
77

@@ -114,11 +114,9 @@ See Decision Tree below for label actions.
114114

115115
### Step 7: Post comment and update labels
116116

117-
Post your findings (use single quotes for the body to handle special characters):
117+
Write the comment body to a temp file using the Write tool, then post it:
118118
```bash
119-
gh issue comment "$ISSUE_URL" --body '## 🔍 Investigation Summary
120-
...your comment here...
121-
'
119+
gh issue comment "$ISSUE_URL" --body-file /tmp/investigation-summary.md
122120
```
123121

124122
Remove label only if the decision tree warrants it:
@@ -131,7 +129,12 @@ Call scripts by name only (e.g., `removeDeployBlockerLabel.sh`), not with full p
131129

132130
### Step 8: Assign contributors
133131

134-
When a causing PR is identified with medium or high confidence, assign the PR author and all approving reviewers to the deploy blocker issue using `gh issue edit "$ISSUE_URL" --add-assignee`. Extract approving reviewers from `gh pr view <PR_NUMBER> --json reviews`.
132+
If the primary causing PR has at least medium confidence, assign its author and approving reviewers to the deploy blocker issue. Only assign contributors from this single PR.
133+
134+
```bash
135+
gh pr view <PR_NUMBER> --json reviews
136+
gh issue edit "$ISSUE_URL" --add-assignee <logins>
137+
```
135138

136139
---
137140

@@ -172,7 +175,7 @@ Choose ONE:
172175

173176
## Comment Format
174177

175-
Post ONE comment using this exact format:
178+
Use the Write tool to create `/tmp/investigation-summary.md` with the following markdown structure. Every field and heading must be present.
176179

177180
```markdown
178181
## 🔍 Investigation Summary
@@ -186,7 +189,7 @@ Post ONE comment using this exact format:
186189
Brief explanation of why this recommendation (1-2 sentences).
187190

188191

189-
**Assigned**: @author (PR author), @reviewer (approving reviewer) — or omit if no causing PR identified
192+
**Assigned**: @author (PR author), @reviewer (approving reviewer) from highest-confidence causing PR only — omit if no causing PR identified
190193
**Labels**: [Describe any label changes made]
191194

192195
<details>
@@ -206,6 +209,8 @@ Technical explanation of what went wrong in the code.
206209

207210
</details>
208211
```
212+
213+
Then post with: `gh issue comment "$ISSUE_URL" --body-file /tmp/investigation-summary.md`
209214
---
210215

211216
## Constraints
@@ -218,7 +223,7 @@ Technical explanation of what went wrong in the code.
218223
- Make assumptions about code you haven't read
219224
- Recommend DEMOTE for bugs affecting core functionality (auth, payments, data loss)
220225
- Close the issue—only update labels and comment
221-
- Use heredocs, temp files, or shell redirects for comments
226+
- Use heredocs or shell redirects for comments
222227

223228
**DO:**
224229
- Always verify the causing PR touches the affected code before concluding

.claude/commands/investigate-deploy-blocker.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
allowed-tools: Bash(gh issue view:*),Bash(gh issue comment:*),Bash(gh issue edit:*),Bash(gh issue list:*),Bash(gh pr view:*),Bash(gh pr list:*),Bash(gh pr diff:*),Bash(gh api:*),Bash(git log:*),Bash(git show:*),Bash(git blame:*),Bash(removeDeployBlockerLabel.sh:*),Glob,Grep,Read
2+
allowed-tools: Bash(gh issue view:*),Bash(gh issue comment:*),Bash(gh issue edit:*),Bash(gh issue list:*),Bash(gh pr view:*),Bash(gh pr list:*),Bash(gh pr diff:*),Bash(gh api:*),Bash(git log:*),Bash(git show:*),Bash(git blame:*),Bash(removeDeployBlockerLabel.sh:*),Glob,Grep,Read,Write
33
description: Investigate a deploy blocker issue to find the causing PR and recommend resolution
44
---
55

.github/actions/javascript/authorChecklist/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15767,11 +15767,11 @@ class GithubUtils {
1576715767
/**
1576815768
* Fetch all pull requests given a list of PR numbers.
1576915769
*/
15770-
static fetchAllPullRequests(pullRequestNumbers) {
15770+
static fetchAllPullRequests(pullRequestNumbers, repo = CONST_1.default.APP_REPO) {
1577115771
const oldestPR = pullRequestNumbers.sort((a, b) => a - b).at(0);
1577215772
return this.paginate(this.octokit.pulls.list, {
1577315773
owner: CONST_1.default.GITHUB_OWNER,
15774-
repo: CONST_1.default.APP_REPO,
15774+
repo,
1577515775
state: 'all',
1577615776
sort: 'created',
1577715777
direction: 'desc',

.github/actions/javascript/awaitStagingDeploys/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12545,11 +12545,11 @@ class GithubUtils {
1254512545
/**
1254612546
* Fetch all pull requests given a list of PR numbers.
1254712547
*/
12548-
static fetchAllPullRequests(pullRequestNumbers) {
12548+
static fetchAllPullRequests(pullRequestNumbers, repo = CONST_1.default.APP_REPO) {
1254912549
const oldestPR = pullRequestNumbers.sort((a, b) => a - b).at(0);
1255012550
return this.paginate(this.octokit.pulls.list, {
1255112551
owner: CONST_1.default.GITHUB_OWNER,
12552-
repo: CONST_1.default.APP_REPO,
12552+
repo,
1255312553
state: 'all',
1255412554
sort: 'created',
1255512555
direction: 'desc',

.github/actions/javascript/checkAndroidStatus/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -737288,11 +737288,11 @@ class GithubUtils {
737288737288
/**
737289737289
* Fetch all pull requests given a list of PR numbers.
737290737290
*/
737291-
static fetchAllPullRequests(pullRequestNumbers) {
737291+
static fetchAllPullRequests(pullRequestNumbers, repo = CONST_1.default.APP_REPO) {
737292737292
const oldestPR = pullRequestNumbers.sort((a, b) => a - b).at(0);
737293737293
return this.paginate(this.octokit.pulls.list, {
737294737294
owner: CONST_1.default.GITHUB_OWNER,
737295-
repo: CONST_1.default.APP_REPO,
737295+
repo,
737296737296
state: 'all',
737297737297
sort: 'created',
737298737298
direction: 'desc',

.github/actions/javascript/checkDeployBlockers/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11812,11 +11812,11 @@ class GithubUtils {
1181211812
/**
1181311813
* Fetch all pull requests given a list of PR numbers.
1181411814
*/
11815-
static fetchAllPullRequests(pullRequestNumbers) {
11815+
static fetchAllPullRequests(pullRequestNumbers, repo = CONST_1.default.APP_REPO) {
1181611816
const oldestPR = pullRequestNumbers.sort((a, b) => a - b).at(0);
1181711817
return this.paginate(this.octokit.pulls.list, {
1181811818
owner: CONST_1.default.GITHUB_OWNER,
11819-
repo: CONST_1.default.APP_REPO,
11819+
repo,
1182011820
state: 'all',
1182111821
sort: 'created',
1182211822
direction: 'desc',

.github/actions/javascript/checkSVGCompression/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20337,11 +20337,11 @@ class GithubUtils {
2033720337
/**
2033820338
* Fetch all pull requests given a list of PR numbers.
2033920339
*/
20340-
static fetchAllPullRequests(pullRequestNumbers) {
20340+
static fetchAllPullRequests(pullRequestNumbers, repo = CONST_1.default.APP_REPO) {
2034120341
const oldestPR = pullRequestNumbers.sort((a, b) => a - b).at(0);
2034220342
return this.paginate(this.octokit.pulls.list, {
2034320343
owner: CONST_1.default.GITHUB_OWNER,
20344-
repo: CONST_1.default.APP_REPO,
20344+
repo,
2034520345
state: 'all',
2034620346
sort: 'created',
2034720347
direction: 'desc',

.github/actions/javascript/formatCodeCovComment/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11999,11 +11999,11 @@ class GithubUtils {
1199911999
/**
1200012000
* Fetch all pull requests given a list of PR numbers.
1200112001
*/
12002-
static fetchAllPullRequests(pullRequestNumbers) {
12002+
static fetchAllPullRequests(pullRequestNumbers, repo = CONST_1.default.APP_REPO) {
1200312003
const oldestPR = pullRequestNumbers.sort((a, b) => a - b).at(0);
1200412004
return this.paginate(this.octokit.pulls.list, {
1200512005
owner: CONST_1.default.GITHUB_OWNER,
12006-
repo: CONST_1.default.APP_REPO,
12006+
repo,
1200712007
state: 'all',
1200812008
sort: 'created',
1200912009
direction: 'desc',

.github/actions/javascript/getArtifactInfo/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11773,11 +11773,11 @@ class GithubUtils {
1177311773
/**
1177411774
* Fetch all pull requests given a list of PR numbers.
1177511775
*/
11776-
static fetchAllPullRequests(pullRequestNumbers) {
11776+
static fetchAllPullRequests(pullRequestNumbers, repo = CONST_1.default.APP_REPO) {
1177711777
const oldestPR = pullRequestNumbers.sort((a, b) => a - b).at(0);
1177811778
return this.paginate(this.octokit.pulls.list, {
1177911779
owner: CONST_1.default.GITHUB_OWNER,
11780-
repo: CONST_1.default.APP_REPO,
11780+
repo,
1178111781
state: 'all',
1178211782
sort: 'created',
1178311783
direction: 'desc',

.github/actions/javascript/getDeployPullRequestList/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12154,11 +12154,11 @@ class GithubUtils {
1215412154
/**
1215512155
* Fetch all pull requests given a list of PR numbers.
1215612156
*/
12157-
static fetchAllPullRequests(pullRequestNumbers) {
12157+
static fetchAllPullRequests(pullRequestNumbers, repo = CONST_1.default.APP_REPO) {
1215812158
const oldestPR = pullRequestNumbers.sort((a, b) => a - b).at(0);
1215912159
return this.paginate(this.octokit.pulls.list, {
1216012160
owner: CONST_1.default.GITHUB_OWNER,
12161-
repo: CONST_1.default.APP_REPO,
12161+
repo,
1216212162
state: 'all',
1216312163
sort: 'created',
1216412164
direction: 'desc',

0 commit comments

Comments
 (0)