Какой выбрать анализатор кода JavaScript?

Question

[Дядька Серёжа]

Какой выбрать анализатор кода JavaScript?
Поиск уязвимостей и ошибок

Comments

[Дядька Серёжа]

SmInc, нужно на сервер встроить чужой код

[Дядька Серёжа]

SmInc, у специалистов мало времени, нужно дополнительно проверять

[xmoonlight]

Дядька Серёжа, Купите у них время на эту задачу. (пока - только ревью кода).

[Дядька Серёжа]

xmoonlight, да они и так наши, просто код периодически придется обновлять и снова проверять

[xmoonlight]

Дядька Серёжа, GIT и/или юнит-тестирование.

Answer 1

[Александр Таратин]

Используйте typescript для транспиляции js кода.
Благодаря тому, что ts является подмножеством js, перевод проекта на ts можно проводить постепенно.
Новый функционал пишем сразу на ts, а в старые js файлы добавляем типизацию по необходимости.

Линтеры - херня (сейчас в меня тапки полетят), только отнимают время разработчика, достаточно просто положить в проект конфигурацию ide (рекомендую vscode), чтобы автоформат у всех разработчиков одинаково работал.

Comments

[Roman K]

Ну как бэ TS и линтеры — это тёплое и мягкое.

Линтеры — это не только о форматировании.
Есть, например, любители делать return await <some_promise>, линтер за это умеет бить. Порядок импортов, логический контекст (для любителей `if (!!something)`). Да кучи правил и плагинов.

[sim3x]

/me кинул кирпич

[Александр Таратин]

Roman Kitaev,
return await <some_promise>
Даст более информативный коллстек
Сравните

Причем, если заменить содержимое на то, что в комментарии - стеки станут одинаковыми. Что тоже имхо не есть хорошо.
if (!!something)!! вырежет компрессор.
Короче сколько я ни пытался понять, где мне линтеры могут быть полезны, так и не нашел таких кейсов. Лишь лишние палки в колеса и доп время на настройку.

[Александр Таратин]

Roman Kitaev, Порядок импортов настраивается теми же автоформаторами кода.

[Александр Таратин]

Roman Kitaev, Кстати

async function main1(){
    try{
	return somePromise();//если тут не поставить await 
//somePromiseFinalizer отработает раньше ресолва somePromise
    } finally {
        await somePromiseFinalizer();
    }
}

[ⓒⓢⓢ]

Александр Таратин,

достаточно просто положить в проект конфигурацию ide (рекомендую vscode), чтобы автоформат у всех разработчиков одинаково работал.

Эм, а кто будет этим заниматься? ... конфигурировать все ide\notepad`ы разработчиков, чтобы форматирование у всех было одинаковое ...
Имхо сомнительное решение отказываться от линтеров, это же каждый должен будет свою ide настроить, если не найдёт для себя конфигов в репо

[Александр Таратин]

JRK_DV, Имхо в команде все должны работать в одинаковом окружении.
Никого же в 2018 году не смущает, что в каждый проект тянется в зависимости webpack/babel/typescript и т. д.
С ide аналогично.

Понятно что в реальности далеко не так, но линтеры имхо все-таки костыль.

[ⓒⓢⓢ]

Александр Таратин, так линтер и даёт единое форматирование между разными ide с единым конфигом - если по вашему это костыль, то что можете предложить взамен, чтобы не настраивать каждую среду разработки? очень интересно

Имхо в команде все должны работать в одинаковом окружении.

ни одной вакансии не видел, чтобы хотя бы рекомендация была использовать конкретную ide в работе.

[Александр Таратин]

JRK_DV, Линтер же не форматирует ничего самостоятельно, он просто постоянно пинает тебя иногда по делу, но большую часть времени просто потому что (например не нравиться использование цикла for).

ни одной вакансии не видел, чтобы хотя бы рекомендация была использовать конкретную ide в работе.

А зачем это писать в вакансии?

[Александр Таратин]

JRK_DV,

то что можете предложить взамен,

Ну можно еще хук в гите поставить для автоформата (как и для линта впринципе), благо утилит в npm куча на любой вкус.

[Александр Таратин]

Я тут в общем к чему все веду. Большая часть правил в линтерах слишком строгая и непонятно зачем вообще придумана, но если их все повырубать, то в итоге останется не больше возможностей чем в автоформатерах + typescript.
Но зачем тогда нужна эта старая бабка, которая только и умеет кричать по поводу и без, если автоформатер может тоже самое + правит самостоятельно.

[ⓒⓢⓢ]

Александр Таратин,
да, что хуки, что линтеры - требуют время, как не крути, чтобы пнуть за единый формат.
А требовать работать в конкретной ide, да ещё если будет под лицензией - честно говоря это бред.
Для разных ide конфигов не наделаешь, сюда ещё добавить линтеры для стилей\документации.

Но вообщем я вас понял, хотел услышать про альтернативы, мне они не подходят, спасибо за ответы

[Сергей delphinpro]

Александр Таратин,

достаточно просто положить в проект конфигурацию ide (рекомендую vscode)

Лови тапок =)

Я не люблю vscode, я люблю phpStorm, коллега нетбинс предпочитает, а третий вообще под линуксом в виме фигачит.

И все трое категорически не согласны менять привычный инструмент!

Что делать?

Пихать все три конфигурации не предлагать, потому что завтра возьмем четвертого со своими тараканами и т.д.

Что касается конфигов сборщиков, тот тут как раз проблемы нет. На любой тачке npm install и погнали.

[Александр Таратин]

Сергей delphinpro, https://editorconfig.org/ но решение спорное и далеко не полное.

[Сергей delphinpro]

Александр Таратин, да, это уже лучше и использовать стоит.

Answer 2

[Константин Величко]

Eslint

Answer 3

[Anubis]

js - eslint
ts - tslint