Update snapshots

src/_tests/fixtures/38979/derived.json

@@ -299,12 +299,16 @@
         {
           "path": "types/es-abstract/tsconfig.json",
           "kind": "package-meta",
-          "suspect": "couldn't fetch contents"
+          "suspect": {
+            "body": "couldn't fetch contents"
+          }
         },
         {
           "path": "types/es-abstract/tslint.json",
           "kind": "package-meta",
-          "suspect": "couldn't fetch contents"
+          "suspect": {
+            "body": "couldn't fetch contents"
+          }
         }
       ],
       "owners": [

src/_tests/fixtures/38979/mutations.json

@@ -36,7 +36,7 @@
     "variables": {
       "input": {
         "id": "MDEyOklzc3VlQ29tbWVudDYyNzA5NzAxMg==",
-        "body": "@ExE-Boss Thank you for submitting this PR!\n\n***This is a live comment which I will keep updated.***\n\n## 1 package in this PR\n\n* `es-abstract` [on npm](https://www.npmjs.com/package/es-abstract), [on unpkg](https://unpkg.com/browse/es-abstract@latest/)\n\n## Code Reviews\n\nBecause this is a widely-used package, a DT maintainer will need to review it before it can be merged.\n\n## Status\n\n * ✅ No merge conflicts\n * ✅ Continuous integration tests have passed\n * ❌ A DT maintainer needs to approve changes which affect module config files\n   - [`es-abstract/tsconfig.json`](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/38979/files/222334139e52fc16369464cfb5dc95c82f71192f#diff-abcd3ac52c6c4c77c7fa2a0e5bc09313ca1cbfd335f929838b0a4e3a607774cc): couldn't fetch contents\n   - [`es-abstract/tslint.json`](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/38979/files/222334139e52fc16369464cfb5dc95c82f71192f#diff-1eda518cd7bfbcd5fa96a7f844b631954cbc9db9ff168fc3731abb874369a4f6): couldn't fetch contents\n\nOnce every item on this list is checked, I'll ask you for permission to merge and publish the changes.\n\n## Inactive\n\nThis PR has been inactive for 129 days — it is *still* unreviewed!\n\n----------------------\n... diagnostics scrubbed ...\n<!--typescript_bot_welcome-->"
+        "body": "@ExE-Boss Thank you for submitting this PR!\n\n***This is a live comment which I will keep updated.***\n\n## 1 package in this PR\n\n* `es-abstract` [on npm](https://www.npmjs.com/package/es-abstract), [on unpkg](https://unpkg.com/browse/es-abstract@latest/)\n\n## Code Reviews\n\nBecause this is a widely-used package, a DT maintainer will need to review it before it can be merged.\n\n## Status\n\n * ✅ No merge conflicts\n * ✅ Continuous integration tests have passed\n * ❌ A DT maintainer needs to approve changes which affect module config files ([`es-abstract/tsconfig.json`](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/38979/files/222334139e52fc16369464cfb5dc95c82f71192f#diff-abcd3ac52c6c4c77c7fa2a0e5bc09313ca1cbfd335f929838b0a4e3a607774cc), [`es-abstract/tslint.json`](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/38979/files/222334139e52fc16369464cfb5dc95c82f71192f#diff-1eda518cd7bfbcd5fa96a7f844b631954cbc9db9ff168fc3731abb874369a4f6))\n\nOnce every item on this list is checked, I'll ask you for permission to merge and publish the changes.\n\n## Inactive\n\nThis PR has been inactive for 129 days — it is *still* unreviewed!\n\n----------------------\n... diagnostics scrubbed ...\n<!--typescript_bot_welcome-->"
       }
     }
   },
@@ -66,5 +66,35 @@
         "body": "It has been more than two weeks and this PR still has no reviews.\n\nI'll bump it to the DT maintainer queue. Thank you for your patience, @ExE-Boss.\n\n(Ping @RReverser.)\n<!--typescript_bot_Unreviewed:done-->"
       }
     }
+  },
+  {
+    "query": "mutation($input: AddPullRequestReviewThreadInput!) { addPullRequestReviewThread(input: $input) { clientMutationId } }",
+    "variables": {
+      "input": {
+        "pullRequestId": "MDExOlB1bGxSZXF1ZXN0MzI1ODk5Njc0",
+        "path": "types/es-abstract/tsconfig.json",
+        "body": "couldn't fetch contents"
+      }
+    }
+  },
+  {
+    "query": "mutation($input: AddPullRequestReviewThreadInput!) { addPullRequestReviewThread(input: $input) { clientMutationId } }",
+    "variables": {
+      "input": {
+        "pullRequestId": "MDExOlB1bGxSZXF1ZXN0MzI1ODk5Njc0",
+        "path": "types/es-abstract/tslint.json",
+        "body": "couldn't fetch contents"
+      }
+    }
+  },
+  {
+    "query": "mutation($input: SubmitPullRequestReviewInput!) { submitPullRequestReview(input: $input) { clientMutationId } }",
+    "variables": {
+      "input": {
+        "pullRequestId": "MDExOlB1bGxSZXF1ZXN0MzI1ODk5Njc0",
+        "body": "@ExE-Boss I noticed these differences from the required form. If you can revise your changes to avoid them, so much the better! Otherwise please reply with explanations why they're needed and a maintainer will take a look. Thanks!",
+        "event": "COMMENT"
+      }
+    }
   }
 ]

src/_tests/fixtures/38979/result.json

@@ -10,7 +10,7 @@
   "responseComments": [
     {
       "tag": "welcome",
-      "status": "@ExE-Boss Thank you for submitting this PR!\n\n***This is a live comment which I will keep updated.***\n\n## 1 package in this PR\n\n* `es-abstract` [on npm](https://www.npmjs.com/package/es-abstract), [on unpkg](https://unpkg.com/browse/es-abstract@latest/)\n\n## Code Reviews\n\nBecause this is a widely-used package, a DT maintainer will need to review it before it can be merged.\n\n## Status\n\n * ✅ No merge conflicts\n * ✅ Continuous integration tests have passed\n * ❌ A DT maintainer needs to approve changes which affect module config files\n   - [`es-abstract/tsconfig.json`](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/38979/files/222334139e52fc16369464cfb5dc95c82f71192f#diff-abcd3ac52c6c4c77c7fa2a0e5bc09313ca1cbfd335f929838b0a4e3a607774cc): couldn't fetch contents\n   - [`es-abstract/tslint.json`](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/38979/files/222334139e52fc16369464cfb5dc95c82f71192f#diff-1eda518cd7bfbcd5fa96a7f844b631954cbc9db9ff168fc3731abb874369a4f6): couldn't fetch contents\n\nOnce every item on this list is checked, I'll ask you for permission to merge and publish the changes.\n\n## Inactive\n\nThis PR has been inactive for 129 days — it is *still* unreviewed!\n\n----------------------\n... diagnostics scrubbed ..."
+      "status": "@ExE-Boss Thank you for submitting this PR!\n\n***This is a live comment which I will keep updated.***\n\n## 1 package in this PR\n\n* `es-abstract` [on npm](https://www.npmjs.com/package/es-abstract), [on unpkg](https://unpkg.com/browse/es-abstract@latest/)\n\n## Code Reviews\n\nBecause this is a widely-used package, a DT maintainer will need to review it before it can be merged.\n\n## Status\n\n * ✅ No merge conflicts\n * ✅ Continuous integration tests have passed\n * ❌ A DT maintainer needs to approve changes which affect module config files ([`es-abstract/tsconfig.json`](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/38979/files/222334139e52fc16369464cfb5dc95c82f71192f#diff-abcd3ac52c6c4c77c7fa2a0e5bc09313ca1cbfd335f929838b0a4e3a607774cc), [`es-abstract/tslint.json`](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/38979/files/222334139e52fc16369464cfb5dc95c82f71192f#diff-1eda518cd7bfbcd5fa96a7f844b631954cbc9db9ff168fc3731abb874369a4f6))\n\nOnce every item on this list is checked, I'll ask you for permission to merge and publish the changes.\n\n## Inactive\n\nThis PR has been inactive for 129 days — it is *still* unreviewed!\n\n----------------------\n... diagnostics scrubbed ..."
     },
     {
       "tag": "pinging-reviewers",
@@ -25,7 +25,16 @@
       "status": "It has been more than two weeks and this PR still has no reviews.\n\nI'll bump it to the DT maintainer queue. Thank you for your patience, @ExE-Boss.\n\n(Ping @RReverser.)"
     }
   ],
-  "suggestions": [],
+  "explanations": [
+    {
+      "path": "types/es-abstract/tsconfig.json",
+      "body": "couldn't fetch contents"
+    },
+    {
+      "path": "types/es-abstract/tslint.json",
+      "body": "couldn't fetch contents"
+    }
+  ],
   "shouldClose": false,
   "shouldMerge": false,
   "shouldUpdateLabels": true,

src/_tests/fixtures/43136/result.json

@@ -18,7 +18,7 @@
       "status": "@RReverser Thank you for reviewing this PR! The author has pushed new commits since your last review. Could you take another look and submit a fresh review?"
     }
   ],
-  "suggestions": [],
+  "explanations": [],
   "shouldClose": false,
   "shouldMerge": false,
   "shouldUpdateLabels": true,

src/_tests/fixtures/43144/result.json

@@ -15,7 +15,7 @@
       "status": "@jeffreymeng Everything looks good here. Great job! I am ready to merge this PR (at f1f5c4b) on your behalf.\n\nIf you'd like that to happen, please post a comment saying:\n\n> Ready to merge\n\nand I'll merge this PR almost instantly. Thanks for helping out! :heart:\n\n(@pocesar: you can do this too.)"
     }
   ],
-  "suggestions": [],
+  "explanations": [],
   "shouldClose": false,
   "shouldMerge": false,
   "shouldUpdateLabels": true,

src/_tests/fixtures/43151/derived.json

@@ -27,12 +27,16 @@
         {
           "path": "types/gaze/tsconfig.json",
           "kind": "package-meta",
-          "suspect": "couldn't fetch contents"
+          "suspect": {
+            "body": "couldn't fetch contents"
+          }
         },
         {
           "path": "types/gaze/tslint.json",
           "kind": "package-meta",
-          "suspect": "couldn't fetch contents"
+          "suspect": {
+            "body": "couldn't fetch contents"
+          }
         }
       ],
       "owners": [],

src/_tests/fixtures/43151/mutations.json

@@ -36,5 +36,35 @@
         "body": "🔔 @adamzerella — you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/43151/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)\n<!--typescript_bot_pinging-reviewers-others-->"
       }
     }
+  },
+  {
+    "query": "mutation($input: AddPullRequestReviewThreadInput!) { addPullRequestReviewThread(input: $input) { clientMutationId } }",
+    "variables": {
+      "input": {
+        "pullRequestId": "MDExOlB1bGxSZXF1ZXN0Mzg4Njk0NDU5",
+        "path": "types/gaze/tsconfig.json",
+        "body": "couldn't fetch contents"
+      }
+    }
+  },
+  {
+    "query": "mutation($input: AddPullRequestReviewThreadInput!) { addPullRequestReviewThread(input: $input) { clientMutationId } }",
+    "variables": {
+      "input": {
+        "pullRequestId": "MDExOlB1bGxSZXF1ZXN0Mzg4Njk0NDU5",
+        "path": "types/gaze/tslint.json",
+        "body": "couldn't fetch contents"
+      }
+    }
+  },
+  {
+    "query": "mutation($input: SubmitPullRequestReviewInput!) { submitPullRequestReview(input: $input) { clientMutationId } }",
+    "variables": {
+      "input": {
+        "pullRequestId": "MDExOlB1bGxSZXF1ZXN0Mzg4Njk0NDU5",
+        "body": "@adamzerella I noticed these differences from the required form. If you can revise your changes to avoid them, so much the better! Otherwise please reply with explanations why they're needed and a maintainer will take a look. Thanks!",
+        "event": "COMMENT"
+      }
+    }
   }
 ]

src/_tests/fixtures/43151/result.json

@@ -14,7 +14,16 @@
       "status": "🔔 @adamzerella — you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/43151/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)"
     }
   ],
-  "suggestions": [],
+  "explanations": [
+    {
+      "path": "types/gaze/tsconfig.json",
+      "body": "couldn't fetch contents"
+    },
+    {
+      "path": "types/gaze/tslint.json",
+      "body": "couldn't fetch contents"
+    }
+  ],
   "shouldClose": false,
   "shouldMerge": false,
   "shouldUpdateLabels": true,

src/_tests/fixtures/43160/result.json

@@ -16,7 +16,7 @@
       "status": "@rikkertkoppes The CI build failed! Please [review the logs for more information](https://github.com/DefinitelyTyped/DefinitelyTyped/commit/22c73c88cc9c09efd4c2998ec360607dd4c36c2e/checks?check_suite_id=731664306).\r\n\r\nOnce you've pushed the fixes, the build will automatically re-run. Thanks!"
     }
   ],
-  "suggestions": [],
+  "explanations": [],
   "shouldClose": false,
   "shouldMerge": false,
   "shouldUpdateLabels": true,

src/_tests/fixtures/43175/result.json

@@ -19,7 +19,7 @@
       "status": "@couven92 Thank you for reviewing this PR! The author has pushed new commits since your last review. Could you take another look and submit a fresh review?"
     }
   ],
-  "suggestions": [],
+  "explanations": [],
   "shouldClose": false,
   "shouldMerge": false,
   "shouldUpdateLabels": true,

src/_tests/fixtures/43235/result.json

@@ -12,7 +12,7 @@
       "status": "🔔 @Esri @bsvensson — please [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/43235/files) in the next few days. Be sure to explicitly select **`Approve`** or **`Request Changes`** in the GitHub UI so I know what's going on."
     }
   ],
-  "suggestions": [],
+  "explanations": [],
   "shouldClose": false,
   "shouldMerge": false,
   "shouldUpdateLabels": true,

src/_tests/fixtures/43314/derived.json

@@ -27,12 +27,16 @@
         {
           "path": "types/carbon__icon-helpers/tsconfig.json",
           "kind": "package-meta",
-          "suspect": "couldn't fetch contents"
+          "suspect": {
+            "body": "couldn't fetch contents"
+          }
         },
         {
           "path": "types/carbon__icon-helpers/tslint.json",
           "kind": "package-meta",
-          "suspect": "couldn't fetch contents"
+          "suspect": {
+            "body": "couldn't fetch contents"
+          }
         }
       ],
       "owners": [],

src/_tests/fixtures/43314/mutations.json

@@ -25,5 +25,35 @@
         "body": "🔔 @metonym — you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/43314/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)\n<!--typescript_bot_pinging-reviewers-others-->"
       }
     }
+  },
+  {
+    "query": "mutation($input: AddPullRequestReviewThreadInput!) { addPullRequestReviewThread(input: $input) { clientMutationId } }",
+    "variables": {
+      "input": {
+        "pullRequestId": "MDExOlB1bGxSZXF1ZXN0MzkyMDM2NjA4",
+        "path": "types/carbon__icon-helpers/tsconfig.json",
+        "body": "couldn't fetch contents"
+      }
+    }
+  },
+  {
+    "query": "mutation($input: AddPullRequestReviewThreadInput!) { addPullRequestReviewThread(input: $input) { clientMutationId } }",
+    "variables": {
+      "input": {
+        "pullRequestId": "MDExOlB1bGxSZXF1ZXN0MzkyMDM2NjA4",
+        "path": "types/carbon__icon-helpers/tslint.json",
+        "body": "couldn't fetch contents"
+      }
+    }
+  },
+  {
+    "query": "mutation($input: SubmitPullRequestReviewInput!) { submitPullRequestReview(input: $input) { clientMutationId } }",
+    "variables": {
+      "input": {
+        "pullRequestId": "MDExOlB1bGxSZXF1ZXN0MzkyMDM2NjA4",
+        "body": "@metonym I noticed these differences from the required form. If you can revise your changes to avoid them, so much the better! Otherwise please reply with explanations why they're needed and a maintainer will take a look. Thanks!",
+        "event": "COMMENT"
+      }
+    }
   }
 ]

src/_tests/fixtures/43314/result.json

@@ -14,7 +14,16 @@
       "status": "🔔 @metonym — you're the only owner, but it would still be good if you find someone to [review this PR](https://github.com/DefinitelyTyped/DefinitelyTyped/pull/43314/files) in the next few days, otherwise a maintainer will look at it. (And if you do find someone, maybe even recruit them to be a second owner to make future changes easier...)"
     }
   ],
-  "suggestions": [],
+  "explanations": [
+    {
+      "path": "types/carbon__icon-helpers/tsconfig.json",
+      "body": "couldn't fetch contents"
+    },
+    {
+      "path": "types/carbon__icon-helpers/tslint.json",
+      "body": "couldn't fetch contents"
+    }
+  ],
   "shouldClose": false,
   "shouldMerge": false,
   "shouldUpdateLabels": true,

src/_tests/fixtures/43695-duplicate-comment/derived.json

@@ -27,12 +27,16 @@
         {
           "path": "types/accedo__accedo-one/tsconfig.json",
           "kind": "package-meta",
-          "suspect": "couldn't fetch contents"
+          "suspect": {
+            "body": "couldn't fetch contents"
+          }
         },
         {
           "path": "types/accedo__accedo-one/tslint.json",
           "kind": "package-meta",
-          "suspect": "couldn't fetch contents"
+          "suspect": {
+            "body": "couldn't fetch contents"
+          }
         }
       ],
       "owners": [],

src/_tests/fixtures/43695-duplicate-comment/mutations.json

@@ -66,5 +66,35 @@
         "body": "It has been more than two weeks and this PR still has no reviews.\n\nI'll bump it to the DT maintainer queue. Thank you for your patience, @alexandercerutti.\n\n(Ping «anyone?».)\n<!--typescript_bot_Unreviewed:done-->"
       }
     }
+  },
+  {
+    "query": "mutation($input: AddPullRequestReviewThreadInput!) { addPullRequestReviewThread(input: $input) { clientMutationId } }",
+    "variables": {
+      "input": {
+        "pullRequestId": "MDExOlB1bGxSZXF1ZXN0NDAwMTAwMTk2",
+        "path": "types/accedo__accedo-one/tsconfig.json",
+        "body": "couldn't fetch contents"
+      }
+    }
+  },
+  {
+    "query": "mutation($input: AddPullRequestReviewThreadInput!) { addPullRequestReviewThread(input: $input) { clientMutationId } }",
+    "variables": {
+      "input": {
+        "pullRequestId": "MDExOlB1bGxSZXF1ZXN0NDAwMTAwMTk2",
+        "path": "types/accedo__accedo-one/tslint.json",
+        "body": "couldn't fetch contents"
+      }
+    }
+  },
+  {
+    "query": "mutation($input: SubmitPullRequestReviewInput!) { submitPullRequestReview(input: $input) { clientMutationId } }",
+    "variables": {
+      "input": {
+        "pullRequestId": "MDExOlB1bGxSZXF1ZXN0NDAwMTAwMTk2",
+        "body": "@alexandercerutti I noticed these differences from the required form. If you can revise your changes to avoid them, so much the better! Otherwise please reply with explanations why they're needed and a maintainer will take a look. Thanks!",
+        "event": "COMMENT"
+      }
+    }
   }
 ]

src/_tests/fixtures/43695-duplicate-comment/result.json

@@ -23,7 +23,16 @@
       "status": "It has been more than two weeks and this PR still has no reviews.\n\nI'll bump it to the DT maintainer queue. Thank you for your patience, @alexandercerutti.\n\n(Ping «anyone?».)"
     }
   ],
-  "suggestions": [],
+  "explanations": [
+    {
+      "path": "types/accedo__accedo-one/tsconfig.json",
+      "body": "couldn't fetch contents"
+    },
+    {
+      "path": "types/accedo__accedo-one/tslint.json",
+      "body": "couldn't fetch contents"
+    }
+  ],
   "shouldClose": false,
   "shouldMerge": false,
   "shouldUpdateLabels": true,

src/_tests/fixtures/43695-post-review/derived.json

@@ -27,12 +27,16 @@
         {
           "path": "types/accedo__accedo-one/tsconfig.json",
           "kind": "package-meta",
-          "suspect": "couldn't fetch contents"
+          "suspect": {
+            "body": "couldn't fetch contents"
+          }
         },
         {
           "path": "types/accedo__accedo-one/tslint.json",
           "kind": "package-meta",
-          "suspect": "couldn't fetch contents"
+          "suspect": {
+            "body": "couldn't fetch contents"
+          }
         }
       ],
       "owners": [],