fix: wrap rel_fname in quote

[princejoogie]

resolves: #3182

Summary

Wraps rel_fname in double quotes (") for linter.py so that filepaths with ()[] characters e.g. src/(main)/product/[id]/page.tsx is recognized as one argument instead of shell trying to parse it as glob paths.

[miradnanali]

I posted a slightly more general solution to this exact problem in PR #3192 . The same solution would apply here, I believe.

[princejoogie]

im fine with either to be merged @miradnanali. currently still using a forked version since the /lint command doest work

[miradnanali]

@princejoogie Could I ask, what do you use for linting?

Currently I set lint-cmd to "./lint-nextjs.sh #" to ignore the filename passed and use a script as follows, which verifies the entire codebase. Lines too slow to run every time are commented out, and the timestamp stops the script from running per file changed. Some kind of global lint mode, running once over the entire code base, might be helpful for this kind of linting.

lint-nextjs.sh

#!/bin/bash
set -e

TS_FILE=".lint-nextjs-timestamp"

should_run() {
    [[ ! -f "$TS_FILE" ]] && return 0
    find . \( -path "./.git" -o -path "./.next" -o -path "./node_modules" \) -prune -o \
           -type f \( -name "*.json" -o -name "*.ts" -o -name "*.tsx" -o -name "*.js" \) \
           -newer "$TS_FILE" -print | grep -q .
}

run_lint() {
    echo "lint-nextjs : Running lint checks..."
    touch "$TS_FILE"
    tsc --noEmit
    #npm run lint
    find . -name "*.json" -print0 | xargs -0 npx jsonlint -q
    node --check "next.config.mjs"
    set -o allexport && source .env.local && set +o allexport # one for each .env file
    #npx madge --extensions ts,tsx,js --circular --orphans src/
    #npm run build
    echo "lint-nextjs : Completed without error."
}

if should_run; then
    run_lint
else
    echo "lint-nextjs : Lint checks skipped."
fi

[princejoogie]

@miradnanali i simply just run eslint --fix --quiet and aider passes the filepath.

lint-cmd:
  - "typescript:npx eslint --fix --quiet"
  - "typescriptreact:npx eslint --fix --quiet"
  - "javascript:npx eslint --fix --quiet"
  - "javascriptreact:npx eslint --fix --quiet"

then aider runs it like: eslint --fix --quiet src/app/(main)/page.tsx.

i'm having problems since i need the path to be wrapped in quotes for the shell to recognize it as 1 argument: eslint --fix --quiet "src/app/(main)/page.tsx"