Update README and detection modules for clarity and validation improvements

Author: fsvreddit

README.md

@@ -46,4 +46,4 @@ If an image is detected as AI, a report like this will be made:
 
 This app is open source under the BSD 3-Clause licence. The source code can be found [here](https://github.com/fsvreddit/image-moderator).
 
-Interested in detections for things other than AI? Get in touch by messaging /u/fsv! The list of possibilities can be seen in Sightengine's documentation [here](https://sightengine.com/docs/models).
+Interested in detections for things not yet supported? Get in touch by messaging /u/fsv! The list of possibilities can be seen in Sightengine's documentation [here](https://sightengine.com/docs/models). It would be really useful if you could outline the kind of content you're looking to avoid on your subreddit.

src/detections/DetectDeepfake.ts

@@ -21,14 +21,7 @@ export class DetectDeepfake extends DetectionBase {
             name: ModuleSetting.Threshold,
             helpText: "App will report the post if the deepfake likelihood is greater than this percentage.",
             defaultValue: 80,
-            onValidate: ({ value }) => {
-                if (!value) {
-                    return;
-                }
-                if (value < 0 || value > 99) {
-                    return "Value must be between 0 and 99.";
-                }
-            },
+            onValidate: event => this.validatePercentage(event),
         },
     ];
 

src/detections/DetectGenAI.ts

@@ -40,7 +40,7 @@ export class DetectGenAI extends DetectionBase {
         }
 
         if (aiLikelihood > this.getSetting<number>(ModuleSetting.Threshold, 80)) {
-            return `AI Likelihood: ${aiLikelihood}%.`;
+            return `AI Likelihood: ${aiLikelihood}%`;
         }
     }
 

src/detections/DetectImageQuality.ts

@@ -36,16 +36,16 @@ export class DetectImageQuality extends DetectionBase {
     public detectProactive (sightEngineResponse: SightengineResponse): string | undefined {
         const quality = this.getQualityScore(sightEngineResponse);
         if (quality && quality < this.getSetting(ModuleSetting.QualityThreshold, 50)) {
-            return `Image quality is low (${quality}).`;
+            return `Image quality is low (${quality})`;
         }
         return undefined;
     }
 
     public detectByMenu (sightEngineResponse: SightengineResponse): string | undefined {
         const quality = this.getQualityScore(sightEngineResponse);
         if (quality && quality < this.getSetting(ModuleSetting.QualityThreshold, 50)) {
-            return `Image quality is low (${quality}).`;
+            return `Image quality is low (${quality})`;
         }
-        return "Image quality is acceptable.";
+        return "Image quality is acceptable";
     }
 }

src/detections/DetectQRCodes.ts

@@ -70,7 +70,7 @@ export class DetectQRCodeSpam extends DetectionBase {
         if (results.length > 0) {
             return results.join(", ");
         } else {
-            return "No QR Code spam or social media links detected.";
+            return "No QR Code spam or social media links detected";
         }
     }
 }

src/detections/DetectTextSpam.ts

@@ -5,7 +5,7 @@ import { SightengineResponse } from "../checkSightEngineAPI.js";
 export class DetectTextSpam extends DetectionBase {
     public name = "TextSpam";
     public friendlyName = "Text Spam Detection";
-    public helpText = "Looks for text that may attempt to drive users externally, such as Telegram or Snapchat handles";
+    public helpText = "Looks for text in images that may attempt to drive users externally, such as Telegram or Snapchat handles";
     public sightengineType = "text-content";
 
     public moduleSettings: SettingsFormField[] = [];

src/settings.ts

@@ -28,7 +28,7 @@ const appSettings: SettingsFormField[] = [
     {
         type: "group",
         label: "Detect content in newly created posts",
-        helpText: "Warning: you should be mindful of API usage, and set account age and karma thresholds to keep usage low.",
+        helpText: "Warning: you should be mindful of API usage, and set account age and karma thresholds to keep usage low. Detection will only occur if at least one content type is chosen for proactive detection.",
         fields: [
             {
                 type: "boolean",