Ну, это зависит, я думаю. Вы не можете использовать теги struts2 в файлах JS, если не сохраните их как .jsp. В случае, если вы делаете это или у вас есть JS-код на вашей странице, лично у меня нет проблем с первым вариантом, поскольку теги будут обрабатываться обратно на сервере. Я использую это часто, когда есть простое задание.
Мне не нравится второй вариант, потому что он делает код немного уродливым и может вызвать некоторые проблемы с обслуживанием.
Подводя итог, лучше разделить коды JS и серверной части. Если вы обнаружите, что все перепутаете, возможно, вы захотите взглянуть на свой дизайн, который может измениться.
Опять же, это зависит от того, как вы предпочитаете это делать.