ci: bump actions/checkout, test against Node 24, fix other issues

Author: double-beep

.github/workflows/nodejs.yml

@@ -10,10 +10,10 @@ jobs:
       fail-fast: false
       max-parallel: 2
       matrix:
-        node: [ 20, 22 ]
+        node: [ 20, 22, 24 ]
 
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v5
     - name: Using Node ${{ matrix.node }}
       uses: actions/setup-node@v4
       with:

src/UserscriptTools/MetaSmokeAPI.ts

@@ -343,7 +343,7 @@ export class MetaSmokeAPI extends Reporter {
 
                     if (!token) return;
 
-                    resolve(token.toString());
+                    resolve(token);
                 });
         });
     }

src/UserscriptTools/Post.ts

@@ -312,7 +312,7 @@ export default class Post {
             const flagPopup = document.querySelector('#popup-flag-post');
             const submit = flagPopup?.querySelector('.js-popup-submit');
 
-            if (!submit || !flagPopup || submit.textContent?.trim().startsWith('Retract')) return;
+            if (!submit || !flagPopup || submit.textContent.trim().startsWith('Retract')) return;
 
             // add "Send feedback to: ..." buttons
             appendLabelAndBoxes(submit, this);
@@ -559,7 +559,7 @@ export default class Post {
     private getScore(): number {
         const voteElement = this.element.querySelector('.js-vote-count');
 
-        return Number(voteElement?.textContent?.trim()) || 0;
+        return Number(voteElement?.textContent.trim()) || 0;
     }
 
     private getOpReputation(): number {
@@ -588,7 +588,7 @@ export default class Post {
         // in Flags page, authorName will be empty, but we aren't interested in it there anyways...
         const lastNameEl = [...this.element.querySelectorAll('.user-info .user-details a')].pop();
 
-        return lastNameEl?.textContent?.trim() ?? '';
+        return lastNameEl?.textContent.trim() ?? '';
     }
 
     private getCreationDate(): Date {

src/popover.ts

@@ -524,7 +524,9 @@ function increaseTooltipWidth(menu: HTMLUListElement): void {
         .filter(li => li.firstElementChild?.classList.contains('s-block-link'))
         .map(reportLink => reportLink.nextElementSibling)
         .forEach(tooltip => {
-            const textLength = tooltip?.textContent?.length;
+            if (!tooltip) return;
+
+            const textLength = tooltip.textContent.length;
             if (!textLength) return;
 
             tooltip.classList.add(