search

jesus2099 / konami-command

power‐ups for various web sites
123 stars 27 forks source link

Libraries (require) not loaded depending on run-at #708

Closed jesus2099 closed 2 years ago

jesus2099 commented 2 years ago

I had my laptop reset. And since then I have some problems: #708, #709, #710

My userscripts with // @require libraries and without exactly // @run-at document-start (either none or other value), have the libraries not loaded by Violentmonkey.

// ==UserScript==
// @name         New OK script
// @require      https://github.com/jesus2099/konami-command/raw/de88f870c0e6c633e02f32695e32c4f50329fc3e/lib/SUPER.js?version=2022.3.24.224
// @run-at       document-start
// ==/UserScript==
"use strict";
console.log("NEW OK SCRIPT");
createTag("span", {}, "test");

image

This works with // @run-at document-start !?

// ==UserScript==
// @name         New NG script
// @require      https://github.com/jesus2099/konami-command/raw/de88f870c0e6c633e02f32695e32c4f50329fc3e/lib/SUPER.js?version=2022.3.24.224
// ==/UserScript==
"use strict";
console.log("NEW NG SCRIPT");
createTag("span", {}, "test");

image

This does not work, the SUPER.js code is not append. I also tried // @run-at document-ready and // @run-at document-idle both no good (NG), as well.

Other values than document-start are blocking the @require loading. OK script must be first in the list and NG script must be second in the list, to reproduce the bug. So maybe it's not only related to run-at.

I don't know what has changed except that I had to install programs as non-admin, in %LocalAppData%\Programs\

(I don't remember previous versions)

jesus2099 commented 2 years ago

Reproduced on Android 9 Kiwi Browser 105.0.5195.33 Violentmonkey 2.13.1

IMG_20220923_173302

jesus2099 commented 2 years ago

I see several changes around run-at, document-start and document-end in Violentmonkey V2.13.1 release notes… 🤔

jesus2099 commented 2 years ago

Now also confirmed in Chrome 105.0.5195.127 (Build officiel) (64 bits).

I discovered that OK script must be put first in the list and NG script must be put second, to reproduce the bug!

jesus2099 commented 2 years ago

The bug has been fixed by @tophf but we now have to wait for next Violentmonkey release.

jesus2099 commented 2 years ago

The bugfix has been deployed.