Skip to content

Commit 0d6a009

Browse files
committed
chore: return if underscript actually updated
1 parent 69684a1 commit 0d6a009

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

src/underscript.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ function checkForUpdates(localVersion) {
2828
return needle('https://unpkg.com/underscript@latest/package.json', needleOptions).then((res) => {
2929
const version = res.body.version;
3030
if (version !== localVersion) return downloadScript(version);
31-
return file.readFile(path.resolve(app.getPath('userData'), 'scripts', 'underscript.bundle.js')).then((buffer) => new String(buffer))
31+
return file.readFile(path.resolve(app.getPath('userData'), 'scripts', 'underscript.bundle.js')).then(() => false)
3232
});
3333
}
3434

@@ -43,7 +43,7 @@ function downloadScript(version) {
4343
return Promise.all([
4444
downloadFile('dependencies.js', version),
4545
downloadFile('undercards.user.js', version),
46-
]).then(([...args]) => bundleScript(...args));
46+
]).then(([...args]) => bundleScript(...args)).then(() => true);
4747
}
4848

4949
function bundleScript(depends, script) {
@@ -66,7 +66,9 @@ function bundleScript(depends, script) {
6666
script,
6767
'})();',
6868
'}',
69-
'window.addEventListener(\'DOMContentLoaded\', UnderScriptWrapper);',
69+
`document.addEventListener('readystatechange', () => {`,
70+
' UnderScriptWrapper();',
71+
'}, { once: true });',
7072
].join('\n');
7173
return file.mkdir(path.resolve(app.getPath('userData'), 'scripts'), { recursive: true }) // Doesn't need recursive, but I'm gonna put it anyway
7274
.then(() => file.writeFile(path.resolve(app.getPath('userData'), 'scripts', 'underscript.bundle.js'), bundle).then(() => bundle));

0 commit comments

Comments
 (0)