Better error handeling of unchecked GM api's

Author: MrBlankCoding

README.md

@@ -51,7 +51,6 @@ node build.js
 
   * [ ] `GM_download`
   * [ ] `GM_confirm`
-  * [ ] `GM_unregisterMenuCommand`
   * [ ] Additional APIs as needed
 
 ### Security & Permissions
@@ -62,15 +61,8 @@ node build.js
 * [ ] Add better trusted type managment
 
 ### Core Improvements
-
-* [ ] Fix version checking logic
-* [ ] Add script version checking
-* [ ] Add script update functionality
-* [ ] Improve `ExternalScriptLoader`
-* [ ] Refine `getScriptDescription` function
 * [ ] Deduplicate logic between `background.js`, `inject.js`, and `GM_core.js`
 * [ ] Unify or reuse declarations between `GM_core` and `inject`
-* [ ] Make menu commands reactive 
 
 ### Editor
 
@@ -79,29 +71,18 @@ node build.js
 * [ ] Enhance extension management for CodeMirror
 * [ ] Link CodeMirror minimap toggle
 * [ ] Move helper functions (`exportScript`, `generateTamperMonkeyHeader`) to separate files
-* [ ] Modularize script import logic
 * [ ] Add option to toggle execution in main world or isolated world
 * [ ] Display execution world and run-at time in editor/dashboard
-* [ ] Log script errors and display in editor
-* [ ] Add a cap to how long text can be in the editor
 
 ### UI
 * [ ] Add iframe preview with live refresh
 * [ ] Fix editor optimizations
-* [ ] Add a cap to how long text can be in the dashboard
-
-
-
-### Debugging & Notifications
-
-* [ ] Display script errors directly on page (without console)
-* [ ] Add "Debug Mode" to enable in-page error display
-* [ ] Separate script notification logic into its own file
 
 ### Communication & Helpers
 
 * [ ] Simplify or centralize communication between content scripts and background scripts
 * [ ] Optional Trusted Types helpers
+* [ ] Fix script loading other scripts against CORS
 
 ---
 

src/editor/editor.html

@@ -115,7 +115,6 @@ <h3 class="panel-title">Script Information</h3>
                               <div class="form-group">
                                  <label for="scriptIcon" class="form-label">Icon URL</label>
                                  <input type="url" id="scriptIcon" class="form-input" placeholder="https://... or data:image/png;base64,...">
-                                 <div class="form-hint">Supports http://, https://, or data:image/ URLs</div>
                               </div>
                            </div>
                         </div>

src/editor/editor.js

@@ -865,6 +865,7 @@ class ScriptEditor {
     scriptData.gmAddStyle = this.elements.gmAddStyle?.checked || false;
     scriptData.gmAddElement = this.elements.gmAddElement?.checked || false;
     scriptData.gmRegisterMenuCommand = this.elements.gmRegisterMenuCommand?.checked || false;
+    scriptData.gmUnregisterMenuCommand = this.elements.gmUnregisterMenuCommand?.checked || false;
     scriptData.gmXmlhttpRequest = this.elements.gmXmlhttpRequest?.checked || false;
     scriptData.unsafeWindow = this.elements.unsafeWindow?.checked || false;
 

src/editor/editor_managers.js

@@ -1076,7 +1076,6 @@ export class FormValidator {
     const validations = [
       // Name is optional (auto-generated on save if empty)
       this.validateTargetUrls(),
-      this.validateVersion(),
       this.validateRunAt(),
       this.validateIconUrl(),
       this.validateRequireUrls(),
@@ -1108,20 +1107,6 @@ export class FormValidator {
     return { isValid: true };
   }
 
-  validateVersion() {
-    const versionEl = this.elements.scriptVersion;
-    const version = (versionEl?.value || "").trim();
-    if (!version) return { isValid: true };
-
-    // Accept X.Y or X.Y.Z (with numeric parts). Allow additional patch labels like -beta.1? Keep it simple: X.Y.Z
-    const semverLike = /^\d+\.\d+\.\d+$/;
-    if (!semverLike.test(version)) {
-      this.showValidationError("Version must be in the format X.Y.Z (e.g., 1.0.0).");
-      return { isValid: false };
-    }
-    return { isValid: true };
-  }
-
   validateRunAt() {
     const allowed = new Set(["document_start", "document_end", "document_idle"]);
     const runAt = this.elements.runAt?.value;

src/popup/popup.js

@@ -233,7 +233,7 @@ document.addEventListener("DOMContentLoaded", async () => {
                 try {
                   command.onClick();
                 } catch (error) {
-                  console.error("Menu command error:", error);
+                  console.error("CodeTweak: Menu command failed. Please ensure the menu command API is enabled in the editor settings.", error);
                 }
               }
             },