4.16

.claude/brand-context.md

@@ -0,0 +1,53 @@
+# Brand context for Text Grab
+
+---
+
+## Product
+
+What it is, in one sentence a stranger would understand: Text Grab is a Windows app to copy any text you can see or hear, then: edit, clean, save, recalled, and automate all of your text heavy tasks with clever functions and local AI!
+
+What it actually does (the mechanism, not the promise): Uses 100% local models to do Optical Character Recognition (OCR) and transcription. Many build in functions to automate tedious tasks, as well as using local LLMs for other text actions like translation, summarization, and more!
+
+What it does NOT do (this prevents overclaiming): Text Grab does not ship OCR models they are all built into Windows, or 3rd party.
+
+## Audience
+
+Who buys it: Power users, information workers, developers, designers, engineers, technical folks
+
+What they believe before they arrive: working with text is tedious and annoying, usually avoided as much as possible, manually transcribing text from an image or audio is annoying and there must be a better way.
+
+What they worry about at 2am: They are wasting their mind doing tedious data manipulation or looking up snippets, minor formatting, etc. and changing their workflows or relying on others more than they need to.
+
+What they'd use instead if you didn't exist: Doing the work manually, not being as confident, using tools like Excel, or Google Sheets and VS Code, maybe AI agents with some light scripting.
+
+## Positioning
+
+The one thing true about us that a competitor could not also say: Laser focused on making you more productive with text, from getting the text in to transformations. Native first; local first, always offline: built with WPF and only using local models.
+
+Category we compete in:
+
+Named competitors: Windows Snipping Tool, PowerToys Text Extractor, LightPDF, OCRSpace, FreeOCR, OnlineOCR, Simple OCR, Adobe Acrobat Pro DC, PDFelement, Easy Screen OCR, Boxoft Free OCR, ABBYY FineReader, Nanonets, Free OCR to Word, Nanonets, Filestack
+
+## Proof
+
+Numbers we can cite (with source and date): as of September 1st: over 160k downloads on GitHub, and nearly 5k stars, over 12k downloads from the Microsoft Store
+
+Named customers we're allowed to name: Windows Snipping Tool, PowerToys Text Extractor, ABBYY FineReader, Adobe Acrobat Pro DC
+
+Claims that need legal sign-off: none
+
+## Voice
+
+How we sound: Excited to save users time, to save them from tedium, to make them more productive. skeptical of cloud services that you never know what is going on with your data or when the app will change. Serious about the end to end text value pipeline.
+
+How we never sound: Like we are the best or first doing OCR. 
+
+Words we always use: Productive, speed, faster, integrated, end-to-end
+
+Words we never use: Proprietary,  —
+
+## Constraints
+
+Regulatory or legal limits: The OCR is not perfect and always needs review
+
+Anything off-limits: Guarantee support

.github/workflows/Release.yml

@@ -24,13 +24,19 @@ env:
   PROJECT: 'Text-Grab'
   PROJECT_PATH: 'Text-Grab/Text-Grab.csproj'
   TEST_PATH: 'Tests/Tests.csproj'
+  TEST_CORE_PATH: 'Tests.Core/Tests.Core.csproj'
+  TEST_CORE_WINDOWS_PATH: 'Tests.Core.Windows/Tests.Core.Windows.csproj'
   BUILD_X64: 'bld/x64'
   BUILD_X64_SC: 'bld/x64/Text-Grab-Self-Contained'
   BUILD_ARM64: 'bld/arm64'
   BUILD_ARM64_SC: 'bld/arm64/Text-Grab-Self-Contained'
   ARTIFACT_SIGNING_ENDPOINT: 'https://eus.codesigning.azure.net/'
   ARTIFACT_SIGNING_ACCOUNT_NAME: 'JoeFinAppsSigningCerts'
   ARTIFACT_SIGNING_CERTIFICATE_PROFILE_NAME: 'JoeFinApps'
+  # Unlock token for the Windows AI language model (a Limited Access Feature). Absent secrets
+  # leave the properties empty, which just builds without on-device text AI rather than failing.
+  LAF_TOKEN: ${{ secrets.LAF_TOKEN }}
+  LAF_PUBLISHER_ID: ${{ secrets.LAF_PUBLISHER_ID }}
 
 jobs:
   build:
@@ -46,8 +52,16 @@ jobs:
       - name: Install dependencies
         run: dotnet restore ${{ env.PROJECT_PATH }}
 
+      # The pure tier runs first: it needs no display and finishes in about a second, so a
+      # logic regression fails the job before the slow WPF/STA suite starts.
+      - name: Run Core tests
+        run: dotnet test --project ${{ env.TEST_CORE_PATH }}
+
+      - name: Run Core.Windows tests
+        run: dotnet test --project ${{ env.TEST_CORE_WINDOWS_PATH }} -r win-x64
+
       - name: Run tests
-        run: dotnet test ${{ env.TEST_PATH }} -r win-x64
+        run: dotnet test --project ${{ env.TEST_PATH }} -r win-x64
 
       - name: Compute build version, archive paths, and release metadata
         id: compute
@@ -89,6 +103,8 @@ jobs:
           -p:PublishReadyToRun=false
           -p:PublishSingleFile=true
           -p:CopyOutputSymbolsToPublishDirectory=false
+          -p:LafToken=$env:LAF_TOKEN
+          -p:LafPublisherId=$env:LAF_PUBLISHER_ID
           --nologo
 
       - name: Build x64 self-contained
@@ -103,6 +119,8 @@ jobs:
           -p:PublishReadyToRun=true
           -p:PublishSingleFile=true
           -p:CopyOutputSymbolsToPublishDirectory=false
+          -p:LafToken=$env:LAF_TOKEN
+          -p:LafPublisherId=$env:LAF_PUBLISHER_ID
           --nologo
 
       - name: Build ARM64 framework-dependent
@@ -116,6 +134,8 @@ jobs:
           -p:PublishSingleFile=true
           -p:EnableMsixTooling=true
           -p:CopyOutputSymbolsToPublishDirectory=false
+          -p:LafToken=$env:LAF_TOKEN
+          -p:LafPublisherId=$env:LAF_PUBLISHER_ID
           --nologo
 
       - name: Build ARM64 self-contained
@@ -129,6 +149,8 @@ jobs:
           -p:PublishSingleFile=true
           -p:EnableMsixTooling=true
           -p:CopyOutputSymbolsToPublishDirectory=false
+          -p:LafToken=$env:LAF_TOKEN
+          -p:LafPublisherId=$env:LAF_PUBLISHER_ID
           --nologo
 
       - name: Rename ARM64 executables

.github/workflows/buildDev.yml

@@ -12,6 +12,12 @@ concurrency:
 env:
   PROJECT_PATH: "Text-Grab/Text-Grab.csproj"
   TEST_PATH: "Tests/Tests.csproj"
+  TEST_CORE_PATH: "Tests.Core/Tests.Core.csproj"
+  TEST_CORE_WINDOWS_PATH: "Tests.Core.Windows/Tests.Core.Windows.csproj"
+  # Unlock token for the Windows AI language model (a Limited Access Feature). Absent secrets
+  # leave the properties empty, which just builds without on-device text AI rather than failing.
+  LAF_TOKEN: ${{ secrets.LAF_TOKEN }}
+  LAF_PUBLISHER_ID: ${{ secrets.LAF_PUBLISHER_ID }}
 
 jobs:
   build:
@@ -26,11 +32,17 @@ jobs:
         run: dotnet restore ${{ env.PROJECT_PATH }}
       - name: Build
         run: dotnet build ${{ env.PROJECT_PATH }} -p:EnableMsixTooling=true
+      # The pure tier runs first: it needs no display and finishes in about a second, so a
+      # logic regression fails the job before the slow WPF/STA suite starts.
+      - name: Test Core
+        run: dotnet test --project ${{ env.TEST_CORE_PATH }}
+      - name: Test Core.Windows
+        run: dotnet test --project ${{ env.TEST_CORE_WINDOWS_PATH }} -r win-x64
       - name: Test
-        run: dotnet test ${{ env.TEST_PATH }} -r win-x64
+        run: dotnet test --project ${{ env.TEST_PATH }} -r win-x64
 
       - name: Build for release and Publish
-        run: dotnet publish ${{ env.PROJECT_PATH }} -c Release --self-contained -r win-x64 -p:PublishSingleFile=true -p:EnableMsixTooling=true -o publish
+        run: dotnet publish ${{ env.PROJECT_PATH }} -c Release --self-contained -r win-x64 -p:PublishSingleFile=true -p:EnableMsixTooling=true -p:LafToken=$env:LAF_TOKEN -p:LafPublisherId=$env:LAF_PUBLISHER_ID -o publish
 
       - name: Upload artifact
         uses: actions/upload-artifact@v7

Tests/BarcodeUtilitiesTests.cs → Tests.Core.Windows/BarcodeUtilitiesTests.cs

@@ -5,12 +5,14 @@
 using Text_Grab;
 using Text_Grab.Models;
 using Text_Grab.Utilities;
-using UnitsNet;
 using Windows.Storage.Streams;
-using static System.Net.Mime.MediaTypeNames;
 
-namespace Tests;
+namespace Text_Grab.Tests.Core.Windows;
 
+// Headless half of the original Tests/BarcodeUtilitiesTests.cs (batch 7a). ReadTestSingleQRCode
+// stayed behind as Tests/BarcodeUtilitiesImageTests.cs: it is [WpfFact]-tagged, and Xunit.StaFact
+// cannot be referenced here (it pulls in WindowsBase, which TierBoundaryTests bans). This half has
+// 3 methods against that one's 1, so it kept the original name.
 public class BarcodeUtilitiesTests
 {
     [Fact]
@@ -47,20 +49,6 @@ public void TryToReadBarcodes_WithTwoQrCodes_ReturnsTwoResults()
         Assert.Contains(results, r => r.RawOutput == "https://example.org");
     }
 
-    [WpfFact]
-    public void ReadTestSingleQRCode()
-    {
-        string expectedOutput = "This is a test of the QR Code system";
-        string testFilePath = FileUtilities.GetPathToLocalFile(@".\Images\QrCodeTestImage.png");
-
-        Bitmap testBmp = new(testFilePath);
-
-        List<OcrOutput> result = BarcodeUtilities.TryToReadBarcodes(testBmp);
-
-        Assert.Single(result);
-        Assert.Equal(expectedOutput, result[0].RawOutput);
-    }
-
     [Fact]
     public async Task GetBitmapFromIRandomAccessStream_ReturnsBitmapIndependentOfSourceStream()
     {
@@ -73,7 +61,7 @@ public async Task GetBitmapFromIRandomAccessStream_ReturnsBitmapIndependentOfSou
         using InMemoryRandomAccessStream randomAccessStream = new();
         _ = await randomAccessStream.WriteAsync(memoryStream.ToArray().AsBuffer());
 
-        Bitmap clonedBitmap = ImageMethods.GetBitmapFromIRandomAccessStream(randomAccessStream);
+        Bitmap clonedBitmap = BitmapUtilities.GetBitmapFromIRandomAccessStream(randomAccessStream);
 
         Assert.Equal(8, clonedBitmap.Width);
         Assert.Equal(8, clonedBitmap.Height);

Tests.Core.Windows/FakeTextGrabSettings.cs

@@ -0,0 +1,50 @@
+using System.Runtime.CompilerServices;
+using Text_Grab.Interfaces;
+using Text_Grab.Services;
+
+namespace Text_Grab.Tests.Core.Windows;
+
+/// <summary>
+/// Registers a <see cref="SettingsAccess"/> resolver for this test host. Tests.Core.Windows has
+/// no app assembly to supply one via a <c>[ModuleInitializer]</c> the way Tests does (see
+/// SettingsAccess.Current's remarks), so any moved test whose production code path reads settings
+/// - here, a handful of OcrTests methods that call into OcrUtilities.BuildTextFromOcrLines, which
+/// reads ParagraphDetection/RemoveFurigana/CorrectErrors/CorrectToLatin internally - would throw
+/// InvalidOperationException without one installed.
+///
+/// Every default below is copied from Text-Grab/Properties/Settings.settings's "(Default)"
+/// profile, so a moved test that depends on a default value (e.g. RemoveFurigana=true dropping
+/// furigana words) sees exactly what it saw running inside the app-hosted Tests project.
+/// </summary>
+internal static class TestSettingsInitializer
+{
+    [ModuleInitializer]
+    internal static void Register() => SettingsAccess.SetResolver(() => new FakeTextGrabSettings());
+}
+
+/// <summary>Minimal ITextGrabSettings double seeded with Settings.settings's shipped defaults.</summary>
+internal sealed class FakeTextGrabSettings : ITextGrabSettings
+{
+    public bool CorrectErrors { get; set; } = true;
+    public bool CorrectToLatin { get; set; } = true;
+    public bool OverrideAiArchCheck { get; set; }
+    public bool ParagraphDetection { get; set; } = true;
+    public bool RemoveFurigana { get; set; } = true;
+    public bool TryToReadBarcodes { get; set; } = true;
+    public bool HdrCaptureCorrection { get; set; }
+    public bool HdrBorderlessGranted { get; set; }
+    public bool UiAutomationEnabled { get; set; }
+    public bool WindowsAiDescriptionEnabled { get; set; }
+    public bool UiAutomationFallbackToOcr { get; set; } = true;
+    public bool UseTesseract { get; set; }
+    public string TesseractPath { get; set; } = string.Empty;
+    public string LastUsedLang { get; set; } = string.Empty;
+    public int TtsSpeakWordLimit { get; set; } = 100;
+    public string TtsVoiceName { get; set; } = string.Empty;
+    public double TtsSpeakingRate { get; set; } = 1;
+    public string AudioTranscriptionModel { get; set; } = "BaseMultilingual";
+    public string LiveTranscriptionModel { get; set; } = "BaseMultilingual";
+    public bool EnableFileBackedManagedSettings { get; set; }
+
+    public void Save() { }
+}

Tests.Core.Windows/FileUtilitiesTests.cs

@@ -0,0 +1,36 @@
+using Text_Grab.Utilities;
+
+namespace Text_Grab.Tests.Core.Windows;
+
+// Pure half of the original Tests/FilesIoTests.cs (batch 7a): FileUtilities.GetVisualDocumentFilter
+// is Core.Windows-only and needs no app type. The rest of that file needed WPF or app-side members
+// and kept the FilesIoTests name in Tests; the IoUtilities-only tests moved separately to
+// Tests.Core/IoUtilitiesTests.cs.
+public class FileUtilitiesTests
+{
+    [Fact]
+    public void GetVisualDocumentFilter_IncludesPdfSupport()
+    {
+        string filter = FileUtilities.GetVisualDocumentFilter();
+
+        Assert.Contains("Image and PDF files|", filter);
+        Assert.Contains("PDF files|*.pdf", filter);
+        Assert.Contains("Image files|", filter);
+    }
+
+    // Joined FileUtilities in 7b once GrabFrameFileUtilities followed HistoryInfo to
+    // Core.Windows and GetOpenDocumentFilter() no longer needed the app-side
+    // OpenDocumentFilterUtilities split.
+    [Fact]
+    public void GetOpenDocumentFilter_IncludesVisualAndTextOptions()
+    {
+        string filter = FileUtilities.GetOpenDocumentFilter();
+
+        Assert.Contains("Supported documents|", filter);
+        Assert.Contains("Image and PDF files|", filter);
+        Assert.Contains("Spreadsheet documents|*.csv;*.tsv;*.tab", filter);
+        Assert.Contains("Markdown documents|*.md;*.markdown", filter);
+        Assert.Contains("Text documents (*.txt)|*.txt", filter);
+        Assert.Contains("All files (*.*)|*.*", filter);
+    }
+}

Tests/GrabFrameFileTests.cs → Tests.Core.Windows/GrabFrameFileTests.cs

@@ -2,13 +2,16 @@
 using System.IO;
 using System.IO.Compression;
 using System.Text.Json;
-using System.Windows;
 using Text_Grab;
 using Text_Grab.Models;
 using Text_Grab.Utilities;
 
-namespace Tests;
+namespace Text_Grab.Tests.Core.Windows;
 
+// Moved wholesale in 7b: GrabFrameFileUtilities followed HistoryInfo to Core.Windows once the
+// HistoryInfo blocker cleared (a8591aa), and every assertion here is against that headless pair
+// (GrabFrameFileUtilities, HistoryInfo/WordBorderInfo) with no WPF type in sight - the file had
+// an unused `using System.Windows;` from before that move, since dropped.
 public class GrabFrameFileTests
 {
     [Fact]
@@ -21,13 +24,13 @@ public async Task SaveAndLoad_RoundTripsMetadataWordBordersAndImage()
             new()
             {
                 Word = "Hello",
-                BorderRect = new Rect(1, 2, 30, 12),
+                BorderRect = new RectangleF(1, 2, 30, 12),
                 LineNumber = 0,
             },
             new()
             {
                 Word = "World",
-                BorderRect = new Rect(35, 2, 32, 12),
+                BorderRect = new RectangleF(35, 2, 32, 12),
                 LineNumber = 0,
             },
         ];
@@ -40,7 +43,7 @@ public async Task SaveAndLoad_RoundTripsMetadataWordBordersAndImage()
             IsTable = true,
             LanguageTag = "en-US",
             LanguageKind = LanguageKind.Global,
-            PositionRect = new Rect(100, 120, 400, 300),
+            PositionRect = new RectangleF(100, 120, 400, 300),
             WordBorderInfoJson = JsonSerializer.Serialize(wordBorders),
             ImageContent = new Bitmap(64, 48),
         };
@@ -60,7 +63,7 @@ public async Task SaveAndLoad_RoundTripsMetadataWordBordersAndImage()
             Assert.True(loaded.IsTable);
             Assert.Equal("en-US", loaded.LanguageTag);
             Assert.Equal(LanguageKind.Global, loaded.LanguageKind);
-            Assert.Equal(new Rect(100, 120, 400, 300), loaded.PositionRect);
+            Assert.Equal(new RectangleF(100, 120, 400, 300), loaded.PositionRect);
 
             Assert.NotNull(loaded.ImageContent);
             Assert.Equal(64, loaded.ImageContent!.Width);
@@ -91,7 +94,7 @@ public async Task SaveGrabFrameFileAsync_DoesNotMutateSuppliedInfo()
 
         string originalWordBordersJson = JsonSerializer.Serialize(new List<WordBorderInfo>
         {
-            new() { Word = "Hello", BorderRect = new Rect(1, 2, 30, 12), LineNumber = 0 },
+            new() { Word = "Hello", BorderRect = new RectangleF(1, 2, 30, 12), LineNumber = 0 },
         });
         Bitmap originalImage = new(64, 48);
 

Tests/HdrScreenCaptureTests.cs → Tests.Core.Windows/HdrScreenCaptureTests.cs

@@ -1,7 +1,7 @@
 using System.Drawing;
 using Text_Grab.Utilities.Hdr;
 
-namespace Tests;
+namespace Text_Grab.Tests.Core.Windows;
 
 public class HdrScreenCaptureTests
 {

Tests/ImageChangeDetectorTests.cs → Tests.Core.Windows/ImageChangeDetectorTests.cs

@@ -1,7 +1,7 @@
 using System.Drawing;
 using Text_Grab.Utilities;
 
-namespace Tests;
+namespace Text_Grab.Tests.Core.Windows;
 
 public class ImageChangeDetectorTests
 {

Tests/LanguageTests.cs → Tests.Core.Windows/LanguageTests.cs

@@ -2,7 +2,7 @@
 using Text_Grab;
 using Text_Grab.Models;
 
-namespace Tests;
+namespace Text_Grab.Tests.Core.Windows;
 
 public class LanguageTests
 {