mediawiki/extensions/DiscussionTools (main)

sourcepatches
$ date
--- stdout ---
Tue Jul  7 00:36:50 UTC 2026

--- end ---
$ git clone file:///srv/git/mediawiki-extensions-DiscussionTools.git /src/repo --depth=1 -b master
--- stderr ---
Cloning into '/src/repo'...
--- stdout ---

--- end ---
$ git config user.name libraryupgrader
--- stdout ---

--- end ---
$ git config user.email tools.libraryupgrader@tools.wmflabs.org
--- stdout ---

--- end ---
$ git submodule update --init
--- stdout ---

--- end ---
$ grr init
--- stdout ---
Installed commit-msg hook.

--- end ---
$ git show-ref refs/heads/master
--- stdout ---
cb478d6dfe0920a9f7bd385bc46496b0661805a1 refs/heads/master

--- end ---
$ /usr/bin/npm audit --json
--- stdout ---
{
  "auditReportVersion": 2,
  "vulnerabilities": {
    "grunt": {
      "name": "grunt",
      "severity": "moderate",
      "isDirect": true,
      "via": [
        "js-yaml"
      ],
      "effects": [
        "grunt-eslint"
      ],
      "range": ">=0.4.0-a",
      "nodes": [
        "node_modules/grunt"
      ],
      "fixAvailable": {
        "name": "grunt",
        "version": "0.3.17",
        "isSemVerMajor": true
      }
    },
    "grunt-eslint": {
      "name": "grunt-eslint",
      "severity": "moderate",
      "isDirect": true,
      "via": [
        "grunt"
      ],
      "effects": [],
      "range": "<=1.0.0 || >=18.1.0",
      "nodes": [
        "node_modules/grunt-eslint"
      ],
      "fixAvailable": {
        "name": "grunt-eslint",
        "version": "18.0.0",
        "isSemVerMajor": true
      }
    },
    "js-yaml": {
      "name": "js-yaml",
      "severity": "moderate",
      "isDirect": false,
      "via": [
        {
          "source": 1121859,
          "name": "js-yaml",
          "dependency": "js-yaml",
          "title": "JS-YAML: Quadratic-complexity DoS in merge key handling via repeated aliases",
          "url": "https://github.com/advisories/GHSA-h67p-54hq-rp68",
          "severity": "moderate",
          "cwe": [
            "CWE-407"
          ],
          "cvss": {
            "score": 5.3,
            "vectorString": "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:L"
          },
          "range": "<3.15.0"
        },
        {
          "source": 1121860,
          "name": "js-yaml",
          "dependency": "js-yaml",
          "title": "JS-YAML: Quadratic-complexity DoS in merge key handling via repeated aliases",
          "url": "https://github.com/advisories/GHSA-h67p-54hq-rp68",
          "severity": "moderate",
          "cwe": [
            "CWE-407"
          ],
          "cvss": {
            "score": 5.3,
            "vectorString": "CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:L"
          },
          "range": ">=4.0.0 <=4.1.1"
        }
      ],
      "effects": [
        "grunt"
      ],
      "range": "<=3.14.2 || 4.0.0 - 4.1.1",
      "nodes": [
        "node_modules/grunt/node_modules/js-yaml",
        "node_modules/js-yaml"
      ],
      "fixAvailable": {
        "name": "grunt",
        "version": "0.3.17",
        "isSemVerMajor": true
      }
    }
  },
  "metadata": {
    "vulnerabilities": {
      "info": 0,
      "low": 0,
      "moderate": 3,
      "high": 0,
      "critical": 0,
      "total": 3
    },
    "dependencies": {
      "prod": 1,
      "dev": 441,
      "optional": 0,
      "peer": 1,
      "peerOptional": 0,
      "total": 441
    }
  }
}

--- end ---
$ /usr/bin/composer install
--- stderr ---
No composer.lock file present. Updating dependencies to latest instead of installing from lock file. See https://getcomposer.org/install for more information.
Loading composer repositories with package information
Updating dependencies
Lock file operations: 38 installs, 0 updates, 0 removals
  - Locking composer/pcre (3.4.0)
  - Locking composer/semver (3.4.4)
  - Locking composer/spdx-licenses (1.6.0)
  - Locking composer/xdebug-handler (3.0.5)
  - Locking danog/advanced-json-rpc (v3.2.3)
  - Locking dealerdirect/phpcodesniffer-composer-installer (v1.2.1)
  - Locking doctrine/deprecations (1.1.6)
  - Locking mediawiki/mediawiki-codesniffer (v51.0.0)
  - Locking mediawiki/mediawiki-phan-config (0.20.0)
  - Locking mediawiki/minus-x (2.0.1)
  - Locking mediawiki/phan-taint-check-plugin (9.1.0)
  - Locking netresearch/jsonmapper (v5.0.1)
  - Locking phan/phan (6.0.2)
  - Locking phan/tolerant-php-parser (v0.2.0)
  - Locking phan/var_representation_polyfill (0.1.4)
  - Locking php-parallel-lint/php-console-color (v1.0.1)
  - Locking php-parallel-lint/php-console-highlighter (v1.0.0)
  - Locking php-parallel-lint/php-parallel-lint (v1.4.0)
  - Locking phpcsstandards/phpcsextra (1.5.0)
  - Locking phpcsstandards/phpcsutils (1.2.2)
  - Locking phpdocumentor/reflection-common (2.2.0)
  - Locking phpdocumentor/reflection-docblock (6.0.3)
  - Locking phpdocumentor/type-resolver (2.0.0)
  - Locking phpstan/phpdoc-parser (2.3.2)
  - Locking psr/container (2.0.2)
  - Locking psr/log (3.0.2)
  - Locking sabre/event (6.1.0)
  - Locking squizlabs/php_codesniffer (3.13.5)
  - Locking symfony/console (v8.1.1)
  - Locking symfony/deprecation-contracts (v3.7.1)
  - Locking symfony/polyfill-ctype (v1.37.0)
  - Locking symfony/polyfill-intl-grapheme (v1.38.1)
  - Locking symfony/polyfill-intl-normalizer (v1.38.0)
  - Locking symfony/polyfill-mbstring (v1.38.2)
  - Locking symfony/polyfill-php85 (v1.38.1)
  - Locking symfony/service-contracts (v3.7.1)
  - Locking symfony/string (v8.1.0)
  - Locking webmozart/assert (2.4.1)
Writing lock file
Installing dependencies from lock file (including require-dev)
Package operations: 38 installs, 0 updates, 0 removals
    0 [>---------------------------]    0 [->--------------------------]
  - Installing squizlabs/php_codesniffer (3.13.5): Extracting archive
  - Installing dealerdirect/phpcodesniffer-composer-installer (v1.2.1): Extracting archive
  - Installing composer/pcre (3.4.0): Extracting archive
  - Installing phpcsstandards/phpcsutils (1.2.2): Extracting archive
  - Installing phpcsstandards/phpcsextra (1.5.0): Extracting archive
  - Installing symfony/polyfill-mbstring (v1.38.2): Extracting archive
  - Installing composer/spdx-licenses (1.6.0): Extracting archive
  - Installing composer/semver (3.4.4): Extracting archive
  - Installing mediawiki/mediawiki-codesniffer (v51.0.0): Extracting archive
  - Installing symfony/polyfill-intl-normalizer (v1.38.0): Extracting archive
  - Installing symfony/polyfill-intl-grapheme (v1.38.1): Extracting archive
  - Installing symfony/polyfill-ctype (v1.37.0): Extracting archive
  - Installing symfony/string (v8.1.0): Extracting archive
  - Installing symfony/deprecation-contracts (v3.7.1): Extracting archive
  - Installing psr/container (2.0.2): Extracting archive
  - Installing symfony/service-contracts (v3.7.1): Extracting archive
  - Installing symfony/polyfill-php85 (v1.38.1): Extracting archive
  - Installing symfony/console (v8.1.1): Extracting archive
  - Installing sabre/event (6.1.0): Extracting archive
  - Installing phan/var_representation_polyfill (0.1.4): Extracting archive
  - Installing phan/tolerant-php-parser (v0.2.0): Extracting archive
  - Installing netresearch/jsonmapper (v5.0.1): Extracting archive
  - Installing webmozart/assert (2.4.1): Extracting archive
  - Installing phpstan/phpdoc-parser (2.3.2): Extracting archive
  - Installing phpdocumentor/reflection-common (2.2.0): Extracting archive
  - Installing doctrine/deprecations (1.1.6): Extracting archive
  - Installing phpdocumentor/type-resolver (2.0.0): Extracting archive
  - Installing phpdocumentor/reflection-docblock (6.0.3): Extracting archive
  - Installing danog/advanced-json-rpc (v3.2.3): Extracting archive
  - Installing psr/log (3.0.2): Extracting archive
  - Installing composer/xdebug-handler (3.0.5): Extracting archive
  - Installing phan/phan (6.0.2): Extracting archive
  - Installing mediawiki/phan-taint-check-plugin (9.1.0): Extracting archive
  - Installing mediawiki/mediawiki-phan-config (0.20.0): Extracting archive
  - Installing mediawiki/minus-x (2.0.1): Extracting archive
  - Installing php-parallel-lint/php-console-color (v1.0.1): Extracting archive
  - Installing php-parallel-lint/php-console-highlighter (v1.0.0): Extracting archive
  - Installing php-parallel-lint/php-parallel-lint (v1.4.0): Extracting archive
  0/36 [>---------------------------]   0%
 29/36 [======================>-----]  80%
 36/36 [============================] 100%
1 package suggestions were added by new dependencies, use `composer suggest` to see details.
Generating autoload files
17 packages you are using are looking for funding.
Use the `composer fund` command to find out more!
--- stdout ---
PHP CodeSniffer Config installed_paths set to ../../mediawiki/mediawiki-codesniffer,../../phpcsstandards/phpcsextra,../../phpcsstandards/phpcsutils

--- end ---
Upgrading n:grunt-stylelint from 0.20.1 -> 0.21.0
Upgrading n:stylelint-config-wikimedia from 0.18.0 -> 0.19.2
$ /usr/bin/npm install
--- stderr ---
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package: 'grunt-stylelint@0.21.0',
npm WARN EBADENGINE   required: { node: '>=20.19.5' },
npm WARN EBADENGINE   current: { node: 'v20.19.2', npm: '9.2.0' }
npm WARN EBADENGINE }
npm WARN deprecated @humanwhocodes/config-array@0.13.0: Use @eslint/config-array instead
npm WARN deprecated @humanwhocodes/object-schema@2.0.3: Use @eslint/object-schema instead
npm WARN deprecated eslint@8.57.1: This version is no longer supported. Please see https://eslint.org/version-support for other options.
--- stdout ---

added 473 packages, and audited 474 packages in 8s

125 packages are looking for funding
  run `npm fund` for details

3 moderate severity vulnerabilities

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

--- end ---
$ package-lock-lint /src/repo/package-lock.json
--- stdout ---
Checking /src/repo/package-lock.json

--- end ---
$ ./node_modules/.bin/grunt stylelint
--- stdout ---
Running "stylelint:all" (stylelint) task

modules/dt.highlighter.less
  12:2  ⚠  .darkmode-override should only be used as a last resort, prefer design tokens instead. See https://w.wiki/LApq for best practices.  wikimedia/no-darkmode-override

modules/dt.ui.ReplyWidget.less
  106:2  ✖  Needless disable for "no-descending-specificity"                                                                                                                                                                   --report-needless-disables
  351:3  ✖  Expected selector "&-open" ("-newComments.ext-discussiontools-ui-replyWidget:is(-open)") to come before selector ".skin-minerva &" (".skin-minerva -newComments.ext-discussiontools-ui-replyWidget"), at line 347  no-descending-specificity

modules/init.styles/dt.init.newtopic.less
  95:2  ✖  Needless disable for "declaration-block-no-redundant-longhand-properties"  --report-needless-disables

modules/init.styles/dt.init.topicsubscriptions.less
  160:3  ⚠  .darkmode-override should only be used as a last resort, prefer design tokens instead. See https://w.wiki/LApq for best practices.  wikimedia/no-darkmode-override

✖ 5 problems (3 errors, 2 warnings)


⚠ 5 warnings

Warning: Task "stylelint:all" failed. Use --force to continue.

Aborted due to warnings.

--- end ---
$ ./node_modules/.bin/stylelint modules/dt.debug.less modules/init.styles/dt.init.newtopic.less modules/debughighlighter.less modules/init.styles/dt.init.mobilelede.less modules/init.styles/dt.init.topiccontainers.less modules/dt.ui.ReplyWidget.less modules/init.styles/dt.init.topicsubscriptions.less modules/dt-ve/CommentTargetWidget.less modules/dt.ui.NewTopicController.less modules/dt.highlighter.less modules/init.styles/dt.init.less modules/init.styles/dt.init.pageframe.less modules/init.styles/dt.init.replybuttons.less --fix
--- stderr ---
modules/dt.ui.ReplyWidget.less
  106:2  ✖  Needless disable for "no-descending-specificity"                                                                                                                                                                   --report-needless-disables
  351:3  ✖  Expected selector "&-open" ("-newComments.ext-discussiontools-ui-replyWidget:is(-open)") to come before selector ".skin-minerva &" (".skin-minerva -newComments.ext-discussiontools-ui-replyWidget"), at line 347  no-descending-specificity

modules/dt.highlighter.less
  12:2  ⚠  .darkmode-override should only be used as a last resort, prefer design tokens instead. See https://w.wiki/LApq for best practices.  wikimedia/no-darkmode-override

modules/init.styles/dt.init.newtopic.less
  95:2  ✖  Needless disable for "declaration-block-no-redundant-longhand-properties"  --report-needless-disables

modules/init.styles/dt.init.topicsubscriptions.less
  160:3  ⚠  .darkmode-override should only be used as a last resort, prefer design tokens instead. See https://w.wiki/LApq for best practices.  wikimedia/no-darkmode-override

✖ 5 problems (3 errors, 2 warnings)
--- stdout ---

--- end ---
$ ./node_modules/.bin/stylelint modules/dt.debug.less modules/init.styles/dt.init.newtopic.less modules/debughighlighter.less modules/init.styles/dt.init.mobilelede.less modules/init.styles/dt.init.topiccontainers.less modules/dt.ui.ReplyWidget.less modules/init.styles/dt.init.topicsubscriptions.less modules/dt-ve/CommentTargetWidget.less modules/dt.ui.NewTopicController.less modules/dt.highlighter.less modules/init.styles/dt.init.less modules/init.styles/dt.init.pageframe.less modules/init.styles/dt.init.replybuttons.less -f json
--- stdout ---
[{"source":"/src/repo/modules/dt.debug.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":false,"warnings":[]},{"source":"/src/repo/modules/debughighlighter.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":false,"warnings":[]},{"source":"/src/repo/modules/dt.ui.ReplyWidget.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":true,"warnings":[{"line":351,"column":3,"endLine":351,"endColumn":9,"rule":"no-descending-specificity","severity":"error","text":"Expected selector \"&-open\" (\"-newComments.ext-discussiontools-ui-replyWidget:is(-open)\") to come before selector \".skin-minerva &\" (\".skin-minerva -newComments.ext-discussiontools-ui-replyWidget\"), at line 347 (no-descending-specificity)"},{"line":106,"column":2,"endLine":106,"endColumn":51,"rule":"--report-needless-disables","severity":"error","text":"Needless disable for \"no-descending-specificity\""}]},{"source":"/src/repo/modules/dt.ui.NewTopicController.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":false,"warnings":[]},{"source":"/src/repo/modules/dt.highlighter.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":false,"warnings":[{"line":12,"column":2,"endLine":12,"endColumn":20,"rule":"wikimedia/no-darkmode-override","severity":"warning","text":".darkmode-override should only be used as a last resort, prefer design tokens instead. See https://w.wiki/LApq for best practices. (wikimedia/no-darkmode-override)"}]},{"source":"/src/repo/modules/init.styles/dt.init.newtopic.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":true,"warnings":[{"line":95,"column":2,"endLine":95,"endColumn":33,"rule":"--report-needless-disables","severity":"error","text":"Needless disable for \"declaration-block-no-redundant-longhand-properties\""}]},{"source":"/src/repo/modules/init.styles/dt.init.mobilelede.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":false,"warnings":[]},{"source":"/src/repo/modules/init.styles/dt.init.topiccontainers.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":false,"warnings":[]},{"source":"/src/repo/modules/init.styles/dt.init.topicsubscriptions.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":false,"warnings":[{"line":160,"column":3,"endLine":160,"endColumn":21,"rule":"wikimedia/no-darkmode-override","severity":"warning","text":".darkmode-override should only be used as a last resort, prefer design tokens instead. See https://w.wiki/LApq for best practices. (wikimedia/no-darkmode-override)"}]},{"source":"/src/repo/modules/init.styles/dt.init.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":false,"warnings":[]},{"source":"/src/repo/modules/init.styles/dt.init.pageframe.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":false,"warnings":[]},{"source":"/src/repo/modules/init.styles/dt.init.replybuttons.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":false,"warnings":[]},{"source":"/src/repo/modules/dt-ve/CommentTargetWidget.less","deprecations":[],"invalidOptionWarnings":[],"parseErrors":[],"errored":false,"warnings":[]}]
--- end ---
Removing stylelint-disable (x1) from /src/repo/modules/dt.ui.ReplyWidget.less
Removing stylelint-disable (x1) from /src/repo/modules/init.styles/dt.init.newtopic.less
$ /usr/bin/npm ci
--- stderr ---
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package: 'grunt-stylelint@0.21.0',
npm WARN EBADENGINE   required: { node: '>=20.19.5' },
npm WARN EBADENGINE   current: { node: 'v20.19.2', npm: '9.2.0' }
npm WARN EBADENGINE }
npm WARN deprecated @humanwhocodes/config-array@0.13.0: Use @eslint/config-array instead
npm WARN deprecated @humanwhocodes/object-schema@2.0.3: Use @eslint/object-schema instead
npm WARN deprecated eslint@8.57.1: This version is no longer supported. Please see https://eslint.org/version-support for other options.
--- stdout ---

added 473 packages, and audited 474 packages in 9s

125 packages are looking for funding
  run `npm fund` for details

3 moderate severity vulnerabilities

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

--- end ---
$ /usr/bin/npm test
--- stdout ---

> DiscussionTools@0.0.0 test
> grunt test

Running "tyops:src" (tyops) task
>> No typos found; 419 files checked for 23 typos.

Running "eslint:all" (eslint) task

/src/repo/.svgo.config.mjs
  7:1  warning  Syntax error in type: import('svgo').Config  jsdoc/valid-types

✖ 1 problem (0 errors, 1 warning)


Running "stylelint:all" (stylelint) task

modules/dt.highlighter.less
  12:2  ⚠  .darkmode-override should only be used as a last resort, prefer design tokens instead. See https://w.wiki/LApq for best practices.  wikimedia/no-darkmode-override

modules/dt.ui.ReplyWidget.less
  190:24  ✖  Needless disable for "no-descending-specificity"  --report-needless-disables

modules/init.styles/dt.init.newtopic.less
  4:1  ✖  Needless disable for "no-descending-specificity"  --report-needless-disables

modules/init.styles/dt.init.replybuttons.less
  4:1  ✖  Needless disable for "no-descending-specificity"  --report-needless-disables

modules/init.styles/dt.init.topiccontainers.less
  4:1  ✖  Needless disable for "no-descending-specificity"  --report-needless-disables

modules/init.styles/dt.init.topicsubscriptions.less
    4:1  ✖  Needless disable for "no-descending-specificity"                                                                                    --report-needless-disables
  160:3  ⚠  .darkmode-override should only be used as a last resort, prefer design tokens instead. See https://w.wiki/LApq for best practices.  wikimedia/no-darkmode-override

✖ 7 problems (5 errors, 2 warnings)


⚠ 7 warnings

Warning: Task "stylelint:all" failed. Use --force to continue.

Aborted due to warnings.

--- end ---
Traceback (most recent call last):
  File "/venv/lib/python3.13/site-packages/runner/__init__.py", line 1268, in main
    libup.run()
    ~~~~~~~~~^^
  File "/venv/lib/python3.13/site-packages/runner/__init__.py", line 1202, in run
    self.npm_upgrade(plan)
    ~~~~~~~~~~~~~~~~^^^^^^
  File "/venv/lib/python3.13/site-packages/runner/__init__.py", line 1064, in npm_upgrade
    self.npm_test()
    ~~~~~~~~~~~~~^^
  File "/venv/lib/python3.13/site-packages/runner/__init__.py", line 289, in npm_test
    self.check_call(["npm", "test"])
    ~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^
  File "/venv/lib/python3.13/site-packages/runner/shell2.py", line 66, in check_call
    res.check_returncode()
    ~~~~~~~~~~~~~~~~~~~~^^
  File "/usr/lib/python3.13/subprocess.py", line 508, in check_returncode
    raise CalledProcessError(self.returncode, self.args, self.stdout,
                             self.stderr)
subprocess.CalledProcessError: Command '['/usr/bin/npm', 'test']' returned non-zero exit status 3.

composer dependencies

Dependencies
Development dependencies

npm dependencies

Development dependencies

Logs

Source code is licensed under the AGPL.