Improve and consolidate package.json scripts

.github/workflows/ci.yml

@@ -42,7 +42,7 @@ jobs:
           pnpm-args: '--ignore-scripts'
           node-version: 22.13.0
           repo-token: ${{ secrets.GITHUB_TOKEN }}
-      - run: pnpm repo:lint:all
+      - run: pnpm lint:all
 
 
   verify:
@@ -126,7 +126,7 @@ jobs:
         with:
           node-version: 22.13.0
           repo-token: ${{ secrets.GITHUB_TOKEN }}
-      - run: pnpm repo:lint:types
+      - run: pnpm lint:types
 
   floating-dependencies:
     name: Floating Dependencies

CLAUDE.md

@@ -19,10 +19,12 @@ To run a single test or test module, use the browser test interface with `pnpm s
 
 ### Linting & Type Checking
 
+- `pnpm lint` - Run ESLint quietly
 - `pnpm lint:fix` - Auto-fix linting issues and format with Prettier (required before commits)
-- `pnpm test:lint` - Run ESLint
-- `pnpm lint:format` - Check Prettier formatting
-- `pnpm repo:lint:types` - Type check all packages via Turbo
+- `pnpm lint:check` - Check Prettier formatting
+- `pnpm lint:types` - Type check all packages via Turbo
+- `pnpm lint:all` - Run all linting checks
+- `pnpm lint:published` - Lint published packages
 
 ### CI Preparation
 

bin/link-all.mts

@@ -1,3 +1,20 @@
+/**
+ * This script prepares all @glimmer packages for local testing against ember.js.
+ *
+ * It performs the following steps:
+ * 1. Packs all @glimmer packages into tarballs (similar to npm pack)
+ * 2. Extracts them into a dist/ directory with proper structure
+ * 3. Creates a temporary pnpm workspace with all the packages
+ * 4. Globally links each package so they can be used by ember.js
+ *
+ * This allows testing Glimmer VM changes in ember.js without publishing to npm.
+ * The ember.js build can then use `pnpm link --global <package>` to use these
+ * local versions instead of the published npm packages.
+ *
+ * Usage: pnpm link:all
+ * To undo: pnpm unlink:all
+ */
+
 import { writeFileSync } from 'node:fs';
 import { readFile, unlink, writeFile } from 'node:fs/promises';
 import { join } from 'node:path';

bin/ts

@@ -0,0 +1,2 @@
+#!/usr/bin/env bash
+exec node --disable-warning=ExperimentalWarning --experimental-strip-types "$@"

bin/unlink-all.mts

@@ -1,3 +1,12 @@
+/**
+ * This script removes the global pnpm links created by link-all.mts.
+ *
+ * It unlinks all @glimmer packages from the global pnpm store and
+ * removes the dist/ directory that was created for testing.
+ *
+ * Usage: pnpm unlink:all
+ */
+
 import chalk from 'chalk';
 import { execa } from 'execa';
 import { rimraf } from 'rimraf';

eslint.config.js

@@ -17,7 +17,7 @@ export default config(
   gitignore(),
   {
     name: '@glimmer-workspace/ignores',
-    ignores: ['ts-dist/**/*'],
+    ignores: ['ts-dist/**/*', '.reference/**/*'],
   },
   override('no-console packages', {
     filter: 'env!=console',

package.json

@@ -12,29 +12,26 @@
   "type": "module",
   "exports": null,
   "scripts": {
-    "benchmark:run": "node --disable-warning=ExperimentalWarning --experimental-strip-types ./bin/setup-bench.mts",
-    "benchmark:setup": "node --disable-warning=ExperimentalWarning --experimental-strip-types ./bin/bench-packages.mts",
+    "benchmark:run": "./bin/ts ./bin/setup-bench.mts",
+    "benchmark:setup": "./bin/ts ./bin/bench-packages.mts",
     "clean": "node ./bin/clean.mjs",
-    "link:all": "esyes ./bin/link-all.mts",
-    "lint:fix": "turbo test:lint -- --fix && prettier -w .",
-    "lint:format": "prettier -c .",
+    "link:all": "./bin/ts ./bin/link-all.mts",
+    "lint": "eslint . --quiet",
+    "lint:all": "turbo run lint:all",
+    "lint:check": "prettier -c .",
+    "lint:fix": "turbo lint -- --fix && prettier -w .",
+    "lint:published": "turbo run test:publint",
+    "lint:types": "turbo run //#test:types",
     "postinstall": "./bin/post-install.sh",
-    "repo:lint:all": "turbo run lint:all",
-    "repo:lint:files": "turbo run //#test:lint",
-    "repo:lint:fix": "turbo run test:lint -- --fix && prettier -w .",
-    "repo:lint:pub": "turbo run test:publint",
-    "repo:lint:types": "turbo run //#test:types",
     "repo:prepack": "turbo run prepack",
-    "repo:update:conventions": "node --experimental-strip-types --no-warnings ./repo-metadata/lib/package-updater.ts",
-    "repo:update:metadata": "node --experimental-strip-types --no-warnings ./repo-metadata/lib/update.ts",
-    "smoke:setup": "node --disable-warning=ExperimentalWarning --experimental-strip-types ./smoke-tests/node/setup.ts",
+    "repo:update:conventions": "./bin/ts ./repo-metadata/lib/package-updater.ts",
+    "repo:update:metadata": "./bin/ts ./repo-metadata/lib/update.ts",
+    "smoke:setup": "./bin/ts ./smoke-tests/node/setup.ts",
     "start": "vite",
     "test": "node bin/run-tests.mjs",
-    "test:babel-plugins": "yarn workspace @glimmer/vm-babel-plugins test",
-    "test:lint": "eslint . --quiet",
+    "test:babel-plugins": "pnpm --filter @glimmer/vm-babel-plugins test",
     "test:node": "pnpm turbo test:node",
-    "ts": "node --disable-warning=ExperimentalWarning --experimental-strip-types",
-    "unlink:all": "esyes ./bin/unlink-all.mts"
+    "unlink:all": "./bin/ts ./bin/unlink-all.mts"
   },
   "devDependencies": {
     "@babel/plugin-syntax-dynamic-import": "^7.8.3",
@@ -90,7 +87,6 @@
     "eslint-plugin-regexp": "^2.7.0",
     "eslint-plugin-simple-import-sort": "12.1.1",
     "eslint-plugin-unused-imports": "4.1.4",
-    "esyes": "^1.0.3",
     "execa": "^7.2.0",
     "fast-glob": "^3.3.3",
     "glob": "^10.4.5",

turbo.json

@@ -24,10 +24,10 @@
   "tasks": {
     "lint:all": {
       "cache": false,
-      "dependsOn": ["test:lint", "test:publint"]
+      "dependsOn": ["lint", "test:publint"]
     },
     "test:node": {},
-    "test:lint": {},
+    "lint": {},
     "test:publint": {
       "dependsOn": ["prepack", "prepare"]
     },