"use strict"; // Global variables, categorized by place of definition. var globals = // Third-party libraries ' $ _ jQuery Spinner Handlebars XDate zxcvbn Intl Notification' + ' LazyLoad Dropbox SockJS marked i18n' // Node-based unit tests + ' module require' // Cocoa<-> Javascript bridge + ' bridge' // index.html + ' page_params' // common.js + ' status_classes password_quality' // setup.js + ' csrf_token' // Modules, defined in their respective files. + ' compose compose_fade rows hotkeys narrow reload search subs pointer' + ' composebox_typeahead server_events typeahead_helper notifications hashchange' + ' invite ui viewport util activity timerender message_list MessageListView blueslip unread stream_list' + ' message_edit tab_bar emoji popovers navigate people settings alert_words_ui message_store' + ' avatar feature_flags search_suggestion referral stream_color Dict' + ' Filter summary admin stream_data muting WinChan muting_ui Socket channel gear_menu' + ' message_flags bot_data loading favicon resize scroll_bar condense floating_recipient_bar' + ' copy_and_paste click_handlers topic_list pm_list unread_ui components' // colorspace.js + ' colorspace' // tutorial.js + ' tutorial' // templates.js + ' templates' // alert_words.js + ' alert_words' // fenced_code.js + ' fenced_code' // echo.js + ' echo' // localstorage.js + ' localstorage' // zulip.js + ' home_msg_list current_msg_list' ; var options = { vars: true, // Allow multiple 'var' per function sloppy: true, // Don't require "use strict" white: true, // Lenient whitespace rules plusplus: true, // Allow increment/decrement operators regexp: true, // Allow . and [^...] in regular expressions todo: true, // Allow "TODO" comments. newcap: true, // Don't assume that capitalized functions are // constructors (and the converse) nomen: true, // Tolerate underscore at the beginning of a name stupid: true // Allow synchronous methods }; // For each error.raw message, we can return 'true' to ignore // the error. var exceptions = { "Unexpected 'else' after 'return'." : function () { return true; }, "Don't make functions within a loop." : function () { return true; }, // We use typeof to test if a variable exists at all. "Unexpected 'typeof'. Use '===' to compare directly with {a}.": function (error) { return error.a === 'undefined'; } }; var fs = require('fs'); var path = require('path'); var JSLINT = require(path.join(__dirname, 'jslint')).JSLINT; var cwd = process.cwd(); var exit_code = 0; var i; // Drop 'node' and the script name from args. for (i=0; i<2; i++) { process.argv.shift(); } process.argv.forEach(function (filepath) { var contents = fs.readFileSync(filepath, 'utf8'); var messages = []; // We mutate 'options' so be sure to clear everything. if (filepath.indexOf('static/js/') !== -1) { // Frontend browser code options.browser = true; options.node = false; options.predef = globals.split(/\s+/); } else { // Backend code for Node.js options.browser = false; options.node = true; if (filepath.indexOf('frontend_tests/') !== -1) { // Include '$' and browser globals because we use them inside // casper.evaluate options.predef = ['casper', '$', 'document', 'window', 'set_global', 'add_dependencies', 'patch_builtin', 'assert', 'current_msg_list']; } else { options.predef = []; } } if (!JSLINT(contents, options)) { JSLINT.errors.forEach(function (error) { if (error === null) { // JSLint stopping error messages.push(' (JSLint giving up)'); return; } var exn = exceptions[error.raw]; if (exn && exn(error)) { // Ignore this error. return; } // NB: this will break on a 10,000 line file var line = (' ' + error.line).slice(-4); messages.push(' ' + line + ' ' + error.reason); }); if (messages.length > 0) { exit_code = 1; console.log(path.relative(cwd, filepath)); // Something very wacky happens if we do // .forEach(console.log) directly. messages.forEach(function (msg) { console.log(msg); }); console.log(''); } } }); process.exit(exit_code);