jfmengels / node-elm-review

CLI for elm-review
https://package.elm-lang.org/packages/jfmengels/elm-review/latest/
BSD 3-Clause "New" or "Revised" License
47 stars 25 forks source link

When running NoUnused.Exports we get a stack overflow #142

Closed wolfadex closed 9 months ago

wolfadex commented 9 months ago

This is only the NoUnused.Exports rule, without any flags/options on the full Vendr codebase (4576 files & 601,140 loc). I know there's quite a bit of unused code in our generated code (which is what prompted this). Running with --watch --fix doesn't hit this error.

-- UNEXPECTED ERROR ------------------------------------------------------------

I ran into an unexpected error. Please open an issue at the following link:
  https://github.com/jfmengels/node-elm-review/issues/new

Please include this error message and as much detail as you can provide. If you
can, please provide a setup that makes it easy to reproduce the error. That will
make it much easier to fix the issue.

Below is the error that was encountered.
--------------------------------------------------------------------------------
RangeError: Maximum call stack size exceeded
    at A2 (/Users/wolfgangschuster/Development/ironzion/apps/frontend/elm-stuff/generated-code/jfmengels/elm-review/cli/2.10.0/review-applications/fa687fbfdffbaad504aab1722e993d74.js:56:28)
    at Function.$author$project$Elm$Review$Reporter$formatReports$fn [as f] (/Users/wolfgangschuster/Development/ironzion/apps/frontend/elm-stuff/generated-code/jfmengels/elm-review/cli/2.10.0/review-applications/fa687fbfdffbaad504aab1722e993d74.js:8382:8)
    at A3 (/Users/wolfgangschuster/Development/ironzion/apps/frontend/elm-stuff/generated-code/jfmengels/elm-review/cli/2.10.0/review-applications/fa687fbfdffbaad504aab1722e993d74.js:59:28)
    at Function.$author$project$Elm$Review$Reporter$formatReports$fn [as f] (/Users/wolfgangschuster/Development/ironzion/apps/frontend/elm-stuff/generated-code/jfmengels/elm-review/cli/2.10.0/review-applications/fa687fbfdffbaad504aab1722e993d74.js:8383:8)
    at A3 (/Users/wolfgangschuster/Development/ironzion/apps/frontend/elm-stuff/generated-code/jfmengels/elm-review/cli/2.10.0/review-applications/fa687fbfdffbaad504aab1722e993d74.js:59:28)
    at Function.$author$project$Elm$Review$Reporter$formatReports$fn [as f] (/Users/wolfgangschuster/Development/ironzion/apps/frontend/elm-stuff/generated-code/jfmengels/elm-review/cli/2.10.0/review-applications/fa687fbfdffbaad504aab1722e993d74.js:8383:8)
    at A3 (/Users/wolfgangschuster/Development/ironzion/apps/frontend/elm-stuff/generated-code/jfmengels/elm-review/cli/2.10.0/review-applications/fa687fbfdffbaad504aab1722e993d74.js:59:28)
    at Function.$author$project$Elm$Review$Reporter$formatReports$fn [as f] (/Users/wolfgangschuster/Development/ironzion/apps/frontend/elm-stuff/generated-code/jfmengels/elm-review/cli/2.10.0/review-applications/fa687fbfdffbaad504aab1722e993d74.js:8383:8)
    at A3 (/Users/wolfgangschuster/Development/ironzion/apps/frontend/elm-stuff/generated-code/jfmengels/elm-review/cli/2.10.0/review-applications/fa687fbfdffbaad504aab1722e993d74.js:59:28)
    at Function.$author$project$Elm$Review$Reporter$formatReports$fn [as f] (/Users/wolfgangschuster/Development/ironzion/apps/frontend/elm-stuff/generated-code/jfmengels/elm-review/cli/2.10.0/review-applications/fa687fbfdffbaad504aab1722e993d74.js:8383:8)

undefined
/Users/wolfgangschuster/Development/ironzion/apps/frontend:
 ERR_PNPM_RECURSIVE_RUN_FIRST_FAIL  @ironzion/frontend@0.1.0 review:ci:no-fix: `elm-review --config review/review-preflight`
Exit status 1

Node elm-review v2.10.0

elm-review elm.json

{
    "type": "application",
    "source-directories": [
        "src"
    ],
    "elm-version": "0.19.1",
    "dependencies": {
        "direct": {
            "Arkham/elm-review-no-missing-type-constructor": "1.0.2",
            "dasch/levenshtein": "1.0.3",
            "elm/core": "1.0.5",
            "elm/json": "1.1.3",
            "elm/project-metadata-utils": "1.0.2",
            "elm/regex": "1.0.0",
            "elm-community/list-extra": "8.7.0",
            "elm-community/string-extra": "4.0.1",
            "henriquecbuss/elm-review-no-function-outside-of-modules": "1.0.1",
            "jfmengels/elm-review": "2.13.1",
            "jfmengels/elm-review-code-style": "1.1.4",
            "jfmengels/elm-review-common": "1.3.3",
            "jfmengels/elm-review-simplify": "2.1.0",
            "jfmengels/elm-review-the-elm-architecture": "1.0.3",
            "jfmengels/elm-review-unused": "1.2.0",
            "lue-bird/elm-no-record-type-alias-constructor-function": "1.0.8",
            "stil4m/elm-syntax": "7.2.9"
        },
        "indirect": {
            "elm/bytes": "1.0.8",
            "elm/html": "1.0.0",
            "elm/parser": "1.1.0",
            "elm/random": "1.0.0",
            "elm/time": "1.0.0",
            "elm/virtual-dom": "1.0.3",
            "elm-explorations/test": "2.1.1",
            "miniBill/elm-unicode": "1.0.3",
            "pzp1997/assoc-list": "1.0.0",
            "rtfeldman/elm-hex": "1.0.0",
            "stil4m/structured-writer": "1.0.3"
        }
    },
    "test-dependencies": {
        "direct": {
            "elm-explorations/test": "2.1.1"
        },
        "indirect": {}
    }
}

Node v16.16.0