bump version to 8.41.0-3

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch to using `Command.opts()` to access options

this fixes an issue  where the options where not properly passed to
eslint, which rendered them useless. uses the `opts()` function to
access them. see [1] for more on info on option parsing with
commander.

[1]: https://www.npmjs.com/package/commander#user-content-options

Signed-off-by: Stefan Sterz <s.sterz@proxmox.com>

change binary name from 'eslint' to 'pve-eslint'

so that we don't conflict with 'eslint' package in debian, which ships
the same binary

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 8.41.0-2

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

d/control: define compat level via build-depends and raise to 13

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

changelog: drop author from entries to avoid overly lengthy lines

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 8.41.0-1

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

d/source: override some false-positive lintian warning/errors

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: let dinstall target directly depend on $(DEB)

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: add sbuild target for convenience

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: fix DSC target and set source format to 3.0 (quilt)

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: fix build-dir generation target

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: rework downloading and vendoring upstream

try to ensure that a clean dependency/prerequisites chain exists

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: derive upload dist automatically

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: prepare builddir atomically

so an aborted build has no artifacts left over

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

buildsys: switch to regular parenthesis

for consistency and ease of use with QWERTZ layouts

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

commit build of 8.41.0

and update make 'buildupstream' make target to work again

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

import 8.41.0 source

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

build: add missing dh-nodejs to build-dependencies

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

d/scripts/prepare-changes: make argument check more robust

set -u aborts if we access any undefined variable, which $1 can be if
no argument got passed..

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 8.23.1-1

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

commit build of 8.23.1

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

fix eslint warnings/errors for own sources

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

adapt webkpack patch for 8.23.1

we have to remove the (unnecessary) polyfill plugin, because some
transitive dependency (nodelib/fs.scandir) tries to access node specific
information that is stripped with these (browser targeting) polyfills

it's not necessary as we're targeting node anyway

also update the patch message to reflect the actual changes

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

import 8.23.1 source

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 8.4.0-3

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch to es2021

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 8.4.0-2

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

repair '--fix' option

when changing to ESLint from CLIEgnine, i forgot to change
the outputFixes call. It's now async and takes a list of results.

fixes 7a730cb60189b4b2b4c0e167797d9aa1ccf15ea6

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

buildsys: change upload product to devel

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 8.4.0-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

d/scripts: trim build and jenkins too

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

d/scripts: make changes trimmer case insensitive

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

commit build of 8.4.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

import 8.4.0 source

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

change from CLIEngine to ESLint

8.0 officialy removed CLIEngine (though it's still in git) so use
the current and supported API.

Necessary changes:
* use 'ESLint' instead of 'CLIEngine'
* use 'lintFiles' instead of 'executeOnFiles' (thats async now)
* adapt to the change of return value of 'lintFiles'

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 8.3.0-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

fixup! import 8.3.0 source

commit build of 8.3.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

import 8.3.0 source

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

build upstream to umd

as we want to avoid requiring npm et al for every build, this is also
effectively vendoring all in, so easier to avoid supply-chain attacks

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

use worker_threads for linting

instead linting all files in the main thread, use worker threads
for that (4 by default) and add the '-t' switch to able to control that

a basic benchmark of eslint of pve-manager showed some performance
gains:

Benchmark #1: Current
  Time (mean ± σ):      6.468 s ±  0.116 s    [User: 9.803 s, System: 0.333 s]
  Range (min … max):    6.264 s …  6.647 s    10 runs

Benchmark #2: 2Threads
  Time (mean ± σ):      4.509 s ±  0.106 s    [User: 12.706 s, System: 0.530 s]
  Range (min … max):    4.335 s …  4.674 s    10 runs

Benchmark #3: 4Threads
  Time (mean ± σ):      3.471 s ±  0.033 s    [User: 16.390 s, System: 0.630 s]
  Range (min … max):    3.431 s …  3.542 s    10 runs

Benchmark #4: 8Threads
  Time (mean ± σ):      2.880 s ±  0.044 s    [User: 22.454 s, System: 0.938 s]
  Range (min … max):    2.813 s …  2.964 s    10 runs

Summary
  '8Threads' ran
    1.21 ± 0.02 times faster than '4Threads'
    1.57 ± 0.04 times faster than '2Threads'
    2.25 ± 0.05 times faster than 'Current'

after 8 threads, there were no real performance benefits since the
overhead to load the module seems to be the biggest factor.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

remove unnecessary eslint.js

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

ship proper nodejs module 'pve-eslint'

instead of concatenating the eslint module into our app.js, ship
a 'pve-eslint' module that exports the built eslint module

to do this, we have to leave the module type on 'umd' instead of
changing to 'var' so that nodejs can properly import it.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 7.28.0-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

set cwd of CLIEngine to process.cwd()

it seems it now defaults to '/' as the current working dir, but we
assume the cwd of the process

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

switch to es2020 for bullseyse

at least in theory this should be enough

https://eslint.org/docs/user-guide/configuring/language-options#specifying-environments

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

actually build upstream release 7.28.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

import eslint 7.28.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

buildsys: change upload dist to bullseye

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add small script to help extracting more relevant changes

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 7.18.0-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

switch to ECMA version 2018

Mainly to enable the rest/spread operator (...variable).

Other new features include asynchronous iteration,
Promise.prototype.finally and additions to RegExp (look behind, named
capture groups, s flag)

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

import eslint 7.18.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 7.12.1-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

import 7.12.1 upstream release

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

exit with error also on warnings

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

app: eslint fixes

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 7.2.0-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

import and build new upstream release 7.2.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

ship an example config and extend-rules sample files

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add HelpInfo variables to globals

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

no-mixed-operators: do not check basic math precedence

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

improve no-extra-parens rule

We use both, 'no-extra-parens' and 'no-mixed-operators' rules and
those can conflict. Thus follow eslints recommendation[0] and allow
parenthesis in nestedBinaryExpressions to avoid the linter
complaining either way. Do this also fo enforceForArrowConditionals
as there I saw the issue already in practice for me.

[0]: https://eslint.org/docs/rules/no-mixed-operators#rule-details

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

try to improve object-curly-newline

for me consistency on open/close is most important..

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

disable 'no-else-return' rule

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

disable "no-underscore-dangle" as used in explicit unused variables

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

disable semi-style for now

In generally we follow it but we want also to allow statements like:

let foo = bar
    ? 'baz'
    : 'foz'
    ;

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

adapt some rules

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add --output-config CLI option

can help to get the default config for adaption

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add possibiliity to extend base config over CLI

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

factor out path expansion

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

no-unused-vars: allow starting with _underscore

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

checkin build of 7.1.0

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

update to 7.1.0 sources

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 7.0.0-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

upgrade to v7.0.0

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 7.0.0~alpha3-2

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

ignore unused 'me'

sometimes we want to have it just in case

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

print relative file path

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

re do output formatting, save some lines

Moving the message and the info if something is auto-fixable to a
single line, saving ~ 2/3rd of output lines.

The auto-fixable got replaced with a (*) marker, it's not ideal but
IMO OK; explain what it means in the summaries "X issues marked with
(*) could be auto-fixed"

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

eslint: autofixes

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

allow '!!' for boolean conversion

we use this many times, and it really should not be confusing,
also it is much shorter than: 'Boolean(foo)',

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 7.0.0~alpha3-1

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

drop "lints cwd on no params" hint

This is a bit strange behaviour, especially if there's nothing to
lint and one gets just a spewed out nodejs backtracke.. It's not hard
to just to `eslint *.js` or the like..

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

rules: dot-location: set location parameter to property

who the heck makes the following syntax the default:
> foo.
>    bar().
>    baz();

So let's enforce the right thing™ and enforce the dot sticking to the
property, not the object, i.e.:

> foo
>    .bar().
>    .baz();

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

exit sanely if no args are given

I mean some seem to like python style throwing up in ones face on
every occasion.. I don't though..

nodes drops the original argv0 in favor of process.execPath and the
path to the JavaScript file being executed, so we need more than two
args to be OK.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

globals: sort and add PBS

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 7.0.0~alpha3

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

add dsc target, fix various packaging related stuff

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

first commit

includes a (minimal) working wrapper

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>