chore(turbo): Fix Knip, Simplify turbo.json, Associate turbo.json wit…

…h JSONC
skyblock-finance · Apr 20, 2024 · ea04ed7 · ea04ed7
1 parent 97f901c
commit ea04ed7
Show file tree

Hide file tree

Showing 7 changed files with 22 additions and 18 deletions.
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -10,5 +10,8 @@
 		"jsonc",
 		"typescript",
 		"typescriptreact"
-	]
+	],
+	"files.associations": {
+		"turbo.json": "jsonc"
+	}
 }
diff --git a/internals/eslint-config/source/index.ts b/internals/eslint-config/source/index.ts
@@ -38,7 +38,7 @@ export default {
 		json: tseslint.config(
 			{
 				files: ['**/*.json'],
-				ignores: ['**/tsconfig*.json'],
+				ignores: ['**/tsconfig*.json', '**/turbo.json'],
 				languageOptions: {
 					parser: jsoncEslintParser,
 				},
@@ -62,7 +62,7 @@ export default {
 				},
 			},
 			{
-				files: ['**/tsconfig*.json'],
+				files: ['**/tsconfig*.json', '**/turbo.json'],
 				languageOptions: {
 					parser: jsoncEslintParser,
 				},

diff --git a/internals/eslint-config/turbo.json b/internals/eslint-config/turbo.json
@@ -4,7 +4,6 @@
 	"pipeline": {
 		"build": {
 			"inputs": ["./scripts/**", "./source/**", "tsconfig.*"],
-			"outputMode": "new-only",
 			"outputs": ["./dist/**"]
 		}
 	}

diff --git a/internals/fake-root/turbo.json b/internals/fake-root/turbo.json
@@ -5,26 +5,27 @@
 		"check:eslint": {
 			"inputs": [
 				"../../*.{cjs,js,mjs,json,mjs,mts,ts}",
+				"../../.vscode/**/*",
 				"../../eslint.config.mjs"
 			]
 		},
+		"check:knip": {
+			"inputs": [
+				"../../internals/*/scripts/**/*",
+				"../../internals/*/source/**/*",
+				"../../knip.json",
+				"../../packages/*/scripts/**/*",
+				"../../packages/*/source/**/*"
+			]
+		},
 		"check:prettier": {
-			"inputs": ["../../*.{cjs,js,mjs,json,mjs,mts,ts}"],
-			"outputMode": "new-only"
+			"inputs": ["../../*.{cjs,js,mjs,json,mjs,mts,ts}", "../../.vscode/**/*"]
 		},
-		"download": {},
 		"fix:eslint": {
-			"inputs": ["../../*.{cjs,js,mjs,json,mjs,mts,ts}"],
-			"outputMode": "new-only"
+			"inputs": ["../../*.{cjs,js,mjs,json,mjs,mts,ts}", "../../.vscode/**/*"]
 		},
 		"fix:prettier": {
-			"inputs": ["../../*.{cjs,js,mjs,json,mjs,mts,ts}"],
-			"outputMode": "new-only"
-		},
-		"watch": {
-			"cache": false,
-			"dependsOn": ["^build", "download"],
-			"persistent": true
+			"inputs": ["../../*.{cjs,js,mjs,json,mjs,mts,ts}", "../../.vscode/**/*"]
 		}
 	}
 }
diff --git a/knip.json b/knip.json
@@ -2,6 +2,7 @@
 	"$schema": "https://unpkg.com/knip@5/schema.json",
 	"ignore": ["dist/**"],
 	"ignoreBinaries": ["prettier", "publint"],
+	"ignoreDependencies": ["@arethetypeswrong/cli"],
 	"workspaces": {
 		"internals/*": {
 			"entry": ["scripts/*.ts", "source/index.ts"],

diff --git a/packages/schemas/turbo.json b/packages/schemas/turbo.json
@@ -4,7 +4,6 @@
 	"pipeline": {
 		"build": {
 			"inputs": ["./source/**", "tsconfig.json"],
-			"outputMode": "new-only",
 			"outputs": ["./dist/**"]
 		},
 		"download": {

diff --git a/turbo.json b/turbo.json
@@ -3,7 +3,8 @@
 	"globalDependencies": ["bun.lockb"],
 	"pipeline": {
 		"build": {
-			"dependsOn": ["^build"]
+			"dependsOn": ["^build"],
+			"outputMode": "new-only"
 		},
 		"check": {
 			"dependsOn": [