Skip to content

Commit 1965a09

Browse files
committed
console logger fix
1 parent d96f8fc commit 1965a09

3 files changed

Lines changed: 15 additions & 9 deletions

File tree

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
"name": "remix-development-tools",
33
"description": "Remix development tools - a set of tools for developing/debugging Remix.run apps",
44
"author": "Alem Tuzlak",
5-
"version": "4.1.1",
5+
"version": "4.1.2",
66
"license": "MIT",
77
"keywords": [
88
"remix",

src/test-apps/remix-vite/app/routes/_index.tsx

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,13 @@ redis.on("connect", () => console.debug("Redis connected"));
3030
redis.on("close", () => console.debug("Redis connection closed"));
3131
redis.on("reconnecting", () => console.log("Redis reconnecting"));
3232
redis.removeAllListeners("error");
33-
redis.on("error", (error: Error) => console.error("Redis error", error));
33+
redis.on("error", console.error);
34+
// console.log(redis);
35+
/**
36+
*
37+
* @param param0
38+
* @returns
39+
*/
3440
export const loader = async ({ request }: LoaderFunctionArgs) => {
3541

3642
const test = new Promise((resolve) => {

src/vite/plugin.tsx

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -108,25 +108,25 @@ export const remixDevTools: (args?:RemixViteConfig) => Plugin[] = (args) => {
108108
if(code.includes("console.")) {
109109
const lines = code.split("\n");
110110
return lines.map((line, lineNumber) => {
111-
// Do not add for arrow functions
112-
if(line.replaceAll(" ", "").includes("=>console.")) {
111+
if(line.trim().startsWith("//") || line.trim().startsWith("/**") || line.trim().startsWith("*")){
113112
return line;
114113
}
115-
// Do not add if it is a return statement
116-
if(line.includes("return console.")) {
114+
// Do not add for arrow functions or return statements
115+
if(line.replaceAll(" ", "").includes("=>console.") || line.includes("return console.")) {
117116
return line;
118117
}
118+
119119
const column = line.indexOf("console.");
120120
const logMessage = `"${chalk.magenta("LOG")} Logged in ${chalk.blueBright(`${id.replace(normalizePath(process.cwd()),"")}:${lineNumber+1}:${column+1}`)}"`;
121-
if (line.includes("console.log")) {
121+
if (line.includes("console.log(")) {
122122
const newLine = `console.log(${logMessage});\nconsole.log`;
123123
return line.replace("console.log", newLine);
124124
}
125-
else if (line.includes("console.error")) {
125+
else if (line.includes("console.error(")) {
126126
const newLine = `console.error(${logMessage});\nconsole.error`;
127127
return line.replace("console.error", newLine);
128128
}
129-
else if (line.includes("console.table")) {
129+
else if (line.includes("console.table(")) {
130130
const newLine = `console.table(${logMessage});\nconsole.table`;
131131
return line.replace("console.table",newLine);
132132
}

0 commit comments

Comments
 (0)