Fix TODOs, #364 phetsims/chipper#1489

js/grunt/check.ts

@@ -92,7 +92,7 @@ const check = async ( providedOptions?: Partial<CheckOptions> ): Promise<boolean
 
   const startTime = Date.now();
   if ( options.clean ) {
-    // TODO: Keep these as 'node'? https://github.com/phetsims/chipper/issues/1481
+    // Don't use tsx here, just node since it is for a third-part runnable.
     const cleanResults = await runCommand( 'node', [ tscRunnable, '-b', '--clean' ], cwd, false );
     if ( !cleanResults.success ) {
       throw new Error( 'Checking failed to clean' );

tsconfig-chipper.json

@@ -6,8 +6,9 @@
     "js/common/**/*",
     "js/npm-dependencies/**/*",
     "js/grunt/tasks/util/**/*",
-    "js/grunt/lint.ts",
     "js/grunt/check.ts",
+    "js/grunt/lint.ts",
+    "js/grunt/tasks/lint.ts",
     // TODO: should we split buildjson into a subsection that can be used by this one too (just phet-core?)   // TODO: split buildjson into a subsection that can be used by this one too (just phet-core?)
     "../phet-core/js/types/**/*"
   ]

tsconfig.json

@@ -1,7 +1,5 @@
 {
-  // TODO: Remove devDependencies from chipper and use many of them from perennial-alias, https://github.com/phetsims/perennial/issues/364
   // TODO: If you want to use optionize() in perennial, we need to move it from phet-core to perennial-alias. Or JO HELP, https://github.com/phetsims/perennial/issues/371
-  // TODO: Doesn't shared/tsconfig-node and core need to be in perennial-alias? https://github.com/phetsims/perennial/issues/364
   "extends": "./tsconfig/shared/tsconfig-node.json",
   "include": [
     "js/**/*",

tsconfig/buildtools/tsconfig.json

@@ -1,11 +1,11 @@
 {
+  // TODO: move this file back to chipper, since perennial shouldn't know about it. https://github.com/phetsims/perennial/issues/364
   // Chipper and its node dependencies is what we call "buildtools"
   "compilerOptions": {
     "composite": true
   },
   "references": [
     {
-    // TODO:fix this so it doesn't hard code importing from chipper, https://github.com/phetsims/perennial/issues/364
       "path": "../../../chipper/tsconfig-chipper-node.json"
     }
   ]