removed unnecessary console logs (#159)

* removed unnecessary console logs

* format fix

* updated version in package.json

biome.json

@@ -25,7 +25,11 @@
 			"recommended": true,
 			"suspicious": {
 				"recommended": true,
-				"noExplicitAny": "off"
+				"noExplicitAny": "off",
+				"noConsole": {
+					"level": "error",
+					"options": { "allow": ["assert", "error", "info", "warn"] }
+				}
 			},
 			"style": {
 				"recommended": true

package.json

@@ -2,7 +2,7 @@
 	"name": "remix-development-tools",
 	"description": "Remix development tools - a set of tools for developing/debugging Remix.run apps",
 	"author": "Alem Tuzlak",
-	"version": "4.7.3",
+	"version": "4.7.4",
 	"license": "MIT",
 	"keywords": [
 		"remix",
@@ -99,7 +99,7 @@
 		"vite": ">=5.0.0"
 	},
 	"devDependencies": {
-		"@biomejs/biome": "1.8.3",
+		"@biomejs/biome": "^1.9.4",
 		"@remix-run/dev": "^2.9.1",
 		"@remix-run/node": "^2.9.1",
 		"@remix-run/react": "^2.9.1",

scripts/icons.ts

@@ -12,10 +12,12 @@ const files = glob.sync("**/*.svg", {
 	cwd: inputDir,
 })
 if (files.length === 0) {
+	// biome-ignore lint/suspicious/noConsole: disable noConsole rule for this line
 	console.log(`No SVG files found in ${inputDirRelative}`)
 	process.exit(0)
 }
 // The relative paths are just for cleaner logs
+// biome-ignore lint/suspicious/noConsole: disable noConsole rule for this line
 console.log(`Generating sprite for ${inputDirRelative}`)
 
 generateSvgSprite({

scripts/setup.ts

@@ -3,13 +3,15 @@ import chalk from "chalk"
 import prompt from "prompt"
 // Helper method used to verify the run
 const verifyRun = async () => {
+	// biome-ignore lint/suspicious/noConsole: disable noConsole rule for this line
 	console.log("About to execute the command")
 
 	const { sure } = await prompt.get([
 		{ name: "sure", description: "Are you sure? (y/n)", type: "string", required: true },
 	])
 
 	if (sure !== "y") {
+		// biome-ignore lint/suspicious/noConsole: disable noConsole rule for this line
 		console.log(chalk.bold.red("Command aborted!\n"))
 		process.exit(1)
 	}

src/client/components/jsonRenderer.tsx

@@ -30,7 +30,6 @@ const JsonRenderer = ({ data }: JsonRendererProps) => {
 								value
 									.then((res) => {
 										if (!ref.current) return
-										console.log(res)
 										setJson((json: any) => ({
 											...json,
 											[key]: res,

src/client/hooks/useDevServerConnection.ts

@@ -14,7 +14,7 @@ const updateRouteInfo = (
 	const { data, type } = event
 	const { id, ...rest } = data
 	// Get existing route
-	const existingRouteInfo = !includeServerInfo ? routes?.[id] : routes?.[id] ?? server?.routes?.[id]
+	const existingRouteInfo = !includeServerInfo ? routes?.[id] : (routes?.[id] ?? server?.routes?.[id])
 	let newRouteData = [...(existingRouteInfo?.[type === "loader" ? "loaders" : "actions"] || []), rest]
 	// Makes sure there are no more than 20 entries per loader/action
 	newRouteData = cutArrayToLastN(newRouteData, 20)
@@ -40,10 +40,10 @@ const updateRouteInfo = (
 		highestExecutionTime: max,
 		averageExecutionTime: Number(Number(total / newRouteData.length).toFixed(2)),
 		loaderTriggerCount: type === "loader" ? loaderTriggerCount + 1 : loaderTriggerCount,
-		loaders: type === "loader" ? newRouteData : existingRouteInfo?.loaders ?? [],
-		actions: type === "action" ? newRouteData : existingRouteInfo?.actions ?? [],
-		lastLoader: type === "loader" ? rest : existingRouteInfo?.lastLoader ?? {},
-		lastAction: type === "action" ? rest : existingRouteInfo?.lastAction ?? {},
+		loaders: type === "loader" ? newRouteData : (existingRouteInfo?.loaders ?? []),
+		actions: type === "action" ? newRouteData : (existingRouteInfo?.actions ?? []),
+		lastLoader: type === "loader" ? rest : (existingRouteInfo?.lastLoader ?? {}),
+		lastAction: type === "action" ? rest : (existingRouteInfo?.lastAction ?? {}),
 		actionTriggerCount: type === "action" ? actionTriggerCount + 1 : actionTriggerCount,
 	}
 }

src/server/logger.ts

@@ -6,7 +6,8 @@ const log = (message: string) => {
 	if (config.silent) {
 		return
 	}
-	// eslint-disable-next-line no-console
+
+	// biome-ignore lint/suspicious/noConsole: disable noConsole rule for this line
 	console.log(message)
 }
 

src/vite/plugin.tsx

@@ -175,6 +175,7 @@ export const remixDevTools: (args?: RemixViteConfig) => Plugin[] = (args) => {
 			async configResolved(resolvedViteConfig) {
 				if (!args?.suppressDeprecationWarning && resolvedViteConfig.appType === "custom") {
 					// Log a warning message
+					// biome-ignore lint/suspicious/noConsole: disable noConsole rule for this line
 					console.log(
 						`\n\n⚠️  ${chalk.yellowBright("remix-development-tools")} are going to be deprecated and will be renamed to ${chalk.greenBright("react-router-devtools ")} when React Router v7 is released ⚠️`,
 						`\n⚠️  Set suppressDeprecationWarning to true in your ${chalk.greenBright("vite.config.ts")} file to silence this warning ⚠️`