Fix pnpm install script for large PRs

[jakebailey]

This should unblock PRs like #67455 which touch many packages.

pnpm builds a massive string at some point (pnpm/pnpm#5056 or pnpm/pnpm#7079). The easiest workaround is to detect when the hack this script does fails, and just give up and resort to a full install.

[typescript-bot]

@jakebailey Thank you for submitting this PR!

This is a live comment which I will keep updated.

This PR touches some part of DefinitelyTyped infrastructure, so a DT maintainer will need to review it. This is rare — did you mean to do this?

0 packages in this PR (and infra files)

• Infra files
  • .github/workflows/CI.yml
  • azure-pipelines.yml
  • scripts/pnpm-install.sh

Code Reviews

There aren't any other owners of this package, so a DT maintainer will review it.

You can test the changes of this PR in the Playground.

Status

• ✅ No merge conflicts
• ✅ Continuous integration tests have passed
• ✅ A DT maintainer needs to approve changes which affect DT infrastructure (.github/workflows/CI.yml, azure-pipelines.yml, scripts/pnpm-install.sh)

All of the items on the list are green. To merge, you need to post a comment including the string "Ready to merge" to bring in your changes.

---

Diagnostic Information: What the bot saw about this PR

{
  "type": "info",
  "now": "-",
  "pr_number": 67460,
  "author": "jakebailey",
  "headCommitOid": "18c38773e683d2e729102e64c6662490742e2711",
  "mergeBaseOid": "952921af31f19e0a78161d3b0a2b856bdf6a400d",
  "lastPushDate": "2023-11-20T22:22:08.000Z",
  "lastActivityDate": "2023-11-20T22:31:13.000Z",
  "mergeOfferDate": "2023-11-20T22:29:59.000Z",
  "mergeRequestDate": "2023-11-20T22:31:13.000Z",
  "mergeRequestUser": "jakebailey",
  "hasMergeConflict": false,
  "isFirstContribution": false,
  "tooManyFiles": false,
  "hugeChange": false,
  "popularityLevel": "Well-liked by everyone",
  "pkgInfo": [
    {
      "name": null,
      "kind": "edit",
      "files": [
        {
          "path": ".github/workflows/CI.yml",
          "kind": "infrastructure"
        },
        {
          "path": "azure-pipelines.yml",
          "kind": "infrastructure"
        },
        {
          "path": "scripts/pnpm-install.sh",
          "kind": "infrastructure"
        }
      ],
      "owners": [],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Critical"
    }
  ],
  "reviews": [
    {
      "type": "approved",
      "reviewer": "sandersn",
      "date": "2023-11-20T22:28:03.000Z",
      "isMaintainer": true
    }
  ],
  "mainBotCommentID": 1819900457,
  "ciResult": "pass"
}

[typescript-bot]

🔔 @jakebailey — there are no owners, but it would still be good if you find someone to review this PR 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...)

[jakebailey]

while I'm here...

[typescript-bot]

@jakebailey: Everything looks good here. I am ready to merge this PR (at 18c3877) on your behalf whenever you think it's ready.

If you'd like that to happen, please post a comment saying:

Ready to merge

and I'll merge this PR almost instantly. Thanks for helping out! ❤️

[jakebailey]

Ready to merge