Почему UglifyJS не работает?

Question

[Максим Тарабрин]

Почему не обьъединяются файлы? что не так? Прикрепляю ниже код:

var uglifyjs = require('uglify-js');
var fs = require('fs');

var appClientFiles = [
  "app_client/app.js",
  "app_client/home/home.controller.js",
  "app_client/common/services/geolocation.service.js",
  "app_client/common/services/loc8rData.service.js",
  "app_client/common/filters/formatDistance.filter.js",
  "app_client/common/directives/ratingStars/ratingStars.directive.js"
];

var uglified = uglifyjs.minify(appClientFiles,{compress: false});
console.log(uglified.code);
fs.writeFile('public/angular/loc8r.min.js',uglified.code, function(err){
  if(err){
    console.log(err);
  }else{
    console.log('Script generated and saved: loc8r.min.js');
  }
});

В консоле вывожу чтобы посмотреть что там конкатенировалось, а там тупо пути конкатенировались.

app_client/app.js;app_client/home/home.controller.js;app_client/common/services/geolocation.service.js;app_client/common/services/loc8rData.service.js;app_client/common/filters/
formatDistance.filter.js;app_client/common/directives/ratingStars/ratingStars.directive.js;

Answer 1

[Никита Кит]

Естественно. Ты ему массивом код передаёшь. gulp думает что это код, потому что uglify это плагин минификации и ренейма переменных. Сборщик это browserify, requirejs, webpack. Вот, лови нормальный конфиг
УПД
Но если тебе нужно тупо конкатенировать браузерный js, не затрагивающий package json и его зависимости, то можешь обойтись этим

Comments

[Максим Тарабрин]

Да у меня не gulpы и не gruntы. Мне просто в приложение стека MEAN надо некоторый список файлов минифицировать и положить одним файлом. Чтобы не подключать весь список и юзер не делал кучу запросов.

[Никита Кит]

Максим Тарабрин, а почняк. require fs. Недоглядел. Ну тогда просто concat-js в npm ищи. Но если в этих браузерных скриптах есть require, то ставь browserify, пример конфига скинул выше. Да и на офсайте их хватает

[Максим Тарабрин]

Никита Кит, мне бы что то простое. Я проект запускаю nodemonом. Gulp не варик с его задачами, надо всё тогда на gulp переводить. Для browserify та же картина как я понимаю. Там задача просто собрать все файлы и минифицировть при рестарте nodemon. Uglify как то был изменён что ли? Я читаю по книге Саймона Холмса "Стек MEAN", там был этот вариант прописан с UglifyJS.

[Никита Кит]

Максим Тарабрин, https://www.npmjs.com/package/concat
Ставь глобально и локально - в нём есть cli.
You can also use it from node:

const concat = require('concat');
 
concat([file1, file2, file3]).then(result => console.log(result))
 
// or 
concat([file1, file2, file3], outputFile)

[Никита Кит]

Максим Тарабрин, и, не, забудь про or. Пиши промисом на всякпожарн

[Никита Кит]

Максим Тарабрин, да, и раз уж ареквайрил fs, хватай скрипты из определённого каталога, через метод join. Так тебе в будущем будет удобнее, если понадобится ещё несколько файлов законкатить

[Максим Тарабрин]

Никита Кит, чё то не понял про промисы. Я юзаю вобщем:

var appClientFiles = [
  "app_client/app.js",
  "app_client/home/home.controller.js",
  "app_client/common/services/geolocation.service.js",
  "app_client/common/services/loc8rData.service.js",
  "app_client/common/filters/formatDistance.filter.js",
  "app_client/common/directives/ratingStars/ratingStars.directive.js"
];

concat(appClientFiles, 'public/angular/loc8r.min.js');

Но там потом при конкатенации ломается всё.

Uncaught TypeError: (intermediate value)(intermediate value)(...) is not a function
    at loc8r.min.js:66

[Никита Кит]

Максим Тарабрин, наверно потому что ему нужен абсолютный путь. попробуй пересобрать массив, приведя все пути к абсолютным через path.resolve(eachPath); ну и require path не забудь. Доки в помощь:
https://nodejs.org/api/path.html#path_path_resolve...

[Никита Кит]

Максим Тарабрин, ан, нет. on line 66. Значит он начал процесс, но содержимое скрипта его не порадовало. Вальни в каждый из них use strict и проверь что не так. В каком-то из скриптов что-то не по понятиям, йоу.

[Максим Тарабрин]

Ну на первом 2торой же строчке ошибку выбросило. Ругается на (function(){

"use strict"
(function(){
    angular.module('loc8rApp',['ngRoute'])
    
    function config($ro...

[Никита Кит]

Максим Тарабрин, ага, самовызывающаяся функция теперь зло... Я и не знал, хотя и не использую их. Попробуй задекларировать в переменную и вызвать

[Максим Тарабрин]

Никита Кит, всё спасибо, ошибка была из за ";" в конце самовызывающейся функции.

[Максим Тарабрин]

Никита Кит, а минифицировал с помощью minify
https://www.npmjs.com/package/minify

[Никита Кит]

Максим Тарабрин, маладЭц, вообще по красоте сделал))) и minify, и uglify заюзай. Лишним не будет. uglify тоже экономит пространство переименовкой переменных. Ну и ещё пасхалочек накидать можно чтоб никто не нашёл)))))