ется ли плохим явно проверять логическое значение true? Было бы лучше сделать простойif(success) ?

Я видел разные анекдоты о том, какif (someBoolean === true) ужасный код в строго типизированном языке, но считается ли он плохим в слабо типизированных языках?

Это применимо к любому слабо типизированному языку, который выполняет приведение типов в операторе if.

Конкретный пример будет:

var onSuccess = function (JSONfromServer) {
    // explicitly check for the boolean value `true`
    if (JSONfromServer === true) {
         // do some things
    }
}

// pass it to an ajax as a callback
doSomeAjax(onSuccess);

[Редактировать]

В этом конкретном случае переменная успеха - это любой допустимый JSON, возвращенный с сервера. Так что это может быть что угодно. если это логическое значение true, то успех случился. Если это какой-то объект обработки ошибок, то он будет обработан. Если это что-то еще, то, вероятно, будет обработано спокойно.

Вопрос был в том, чтобы заставить сервер вернутьсяtrue как JSON и проверяя хороший способ обработки случая, когда действие завершилось успешно.

Я хотел избежать специфичности JavaScript и AJAX.