Make ./scripts a package, make scripts typecheck, remove old stuff, workflow updates (DefinitelyTyped#67128)

Author: jakebailey

.github/workflows/CI.yml

@@ -76,3 +76,22 @@ jobs:
           TOKEN='ghp_i5wtj1l2AbpFv3OU96w6R'
           TOKEN+='On3bHOkcV2AmVY6'
           DANGER_GITHUB_API_TOKEN=$TOKEN pnpm danger ci || $( exit 0 )
+
+  scripts:
+    runs-on: ubuntu-latest
+    if: github.repository == 'DefinitelyTyped/DefinitelyTyped'
+
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v3
+        with:
+          node-version: '16'
+
+      - uses: pnpm/action-setup@v2
+        name: Install pnpm
+        with:
+          version: latest
+          run_install: |
+            - args: [--filter, ., --filter, '{./scripts}...']
+
+      - run: pnpm tsc -p ./scripts

.github/workflows/UpdateCodeowners.yml

@@ -31,11 +31,16 @@ jobs:
         with:
           version: latest
           run_install: |
-            - args: [--filter, .]
+            - args: [--filter, ., --filter, '{./scripts}...']
 
       - run: git config --global user.email "[email protected]"
       - run: git config --global user.name "TypeScript Bot"
 
       - run: pnpm run update-codeowners
-        env:
-          GH_TOKEN: ${{ secrets.GH_TOKEN }}
+
+      - uses: stefanzweifel/[email protected]
+        with:
+          commit_author: 'TypeScript Bot <[email protected]>'
+          commit_message: '🤖 Update CODEOWNERS'
+          commit_user_email: '[email protected]'
+          commit_user_name: 'TypeScript Bot'

.github/workflows/format-and-commit.yml

@@ -25,7 +25,7 @@ jobs:
         with:
           version: latest
           run_install: |
-            - args: [--filter, .]
+            - args: [--filter, ., --filter, '{./scripts}...']
 
       - name: Get date
         id: date

.github/workflows/ghostbuster.yml

@@ -24,8 +24,6 @@ jobs:
 
     steps:
       - uses: actions/checkout@v3
-        with:
-          fetch-depth: 0
 
       - uses: actions/setup-node@v4
 
@@ -34,7 +32,7 @@ jobs:
         with:
           version: latest
           run_install: |
-            - args: [--filter, .]
+            - args: [--filter, ., --filter, '{./scripts}...']
 
       - run: node ./scripts/ghostbuster.js
         env:

.github/workflows/lint-md.yml

@@ -17,5 +17,5 @@ jobs:
         with:
           version: latest
           run_install: |
-            - args: [--filter, .]
+            - args: [--filter, ., --filter, '{./scripts}...']
       - run: pnpm remark --frail . .github

.github/workflows/support-window.yml

@@ -31,7 +31,7 @@ jobs:
         with:
           version: latest
           run_install: |
-            - args: [--filter, .]
+            - args: [--filter, ., --filter, '{./scripts}...']
       - name: Fetch TypeScript versions and release dates from npm
         run: |
           npm view --json typescript time |

package.json

@@ -20,42 +20,30 @@
         "preinstall": "npx only-allow pnpm",
         "not-needed": "node scripts/not-needed.js",
         "update-codeowners": "node scripts/update-codeowners.js",
-        "test-all": "node --require source-map-support/register node_modules/@definitelytyped/dtslint-runner/ --path .",
+        "test-all": "node --enable-source-maps node_modules/@definitelytyped/dtslint-runner/ --path .",
         "clean": "node scripts/remove-empty.js",
         "clean-node-modules": "node scripts/clean-node-modules.js",
-        "test": "node --require source-map-support/register node_modules/@definitelytyped/dtslint/ types",
-        "lint": "node --require source-map-support/register node_modules/@definitelytyped/dtslint/ types",
+        "test": "node --enable-source-maps node_modules/@definitelytyped/dtslint/ types",
+        "lint": "node --enable-source-maps node_modules/@definitelytyped/dtslint/ types",
         "prettier": "prettier"
     },
     "devDependencies": {
         "@definitelytyped/definitions-parser": "latest",
+        "@definitelytyped/dts-critic": "latest",
         "@definitelytyped/dtslint": "latest",
         "@definitelytyped/dtslint-runner": "latest",
         "@definitelytyped/eslint-plugin": "latest",
         "@definitelytyped/header-parser": "latest",
         "@definitelytyped/typescript-versions": "latest",
         "@definitelytyped/utils": "latest",
-        "@octokit/core": "^3.5.1",
-        "@octokit/rest": "^16.0.0",
-        "comment-json": "^4.2.3",
-        "d3-array": "^3.0.2",
-        "d3-axis": "^3.0.0",
-        "d3-scale": "^4.0.0",
-        "d3-selection": "^3.0.0",
-        "d3-time": "^3.0.0",
-        "d3-time-format": "^4.0.0",
         "danger": "^11.2.3",
         "dprint": "^0.42.3",
         "eslint-plugin-jsdoc": "^44.2.7",
-        "jsdom": "^17.0.0",
         "remark-cli": "^11.0.0",
         "remark-gfm": "^3.0.0",
         "remark-validate-links": "^12.0.0",
-        "shelljs": "^0.8.5",
-        "source-map-support": "^0.5.21",
-        "typescript": "next",
-        "w3c-xmlserializer": "^2.0.0",
-        "yargs": "^17.1.1"
+        "tslint": "latest",
+        "typescript": "next"
     },
     "type": "module",
     "husky": {

pnpm-workspace.yaml

@@ -1,2 +1,3 @@
 packages:
+  - 'scripts'
   - 'types/**'

scripts/clean-node-modules.js

@@ -6,6 +6,10 @@ const __filename = url.fileURLToPath(new URL(import.meta.url));
 const __dirname = path.dirname(__filename);
 const repoRoot = path.resolve(__dirname, "..");
 
+/**
+ * @param {string} p
+ * @returns {Iterable<string>}
+ */
 function* iterateNodeModules(p) {
     const dirents = fs.readdirSync(p, { withFileTypes: true });
     for (const dirent of dirents) {
@@ -20,6 +24,9 @@ function* iterateNodeModules(p) {
     }
 }
 
+/**
+ * @param {string} p
+ */
 function rimraf(p) {
     // The rimraf package uses maxRetries=10 on Windows, but Node's fs.rm does not have that special case.
     return fs.rmSync(p, { recursive: true, force: true, maxRetries: process.platform === "win32" ? 10 : 0 });