Fix lineWidth issue

Author: nic-olo

src/index.ts

@@ -7,6 +7,7 @@ import {
   ViewType,
 } from '@augmentos/sdk';
 import { TranscriptProcessor } from './utils/src/text-wrapping/TranscriptProcessor';
+import { convertLineWidth } from './utils/src/text-wrapping/convertLineWidth';
 
 // Configuration constants
 const PORT = process.env.PORT ? parseInt(process.env.PORT) : 80;
@@ -380,10 +381,12 @@ class TeleprompterApp extends TpaServer {
   ): Promise<void> {
     try {
       // Extract settings from the session
-      const lineWidth = session.settings.get<number>('line_width', 38);
+      const lineWidthString = session.settings.get<string>('line_width', "Medium");
       const scrollSpeed = session.settings.get<number>('scroll_speed', 120);
       const numberOfLines = session.settings.get<number>('number_of_lines', 4);
       const customText = session.settings.get<string>('custom_text', '');
+
+      const lineWidth = convertLineWidth(lineWidthString, false);
 
       console.log(`Applied settings for user ${userId}: lineWidth=${lineWidth}, scrollSpeed=${scrollSpeed}, numberOfLines=${numberOfLines}`);
 

src/utils/src/text-wrapping/TranscriptProcessor.ts

@@ -164,6 +164,9 @@ export class TranscriptProcessor {
 
   // Wrap text into lines based on max line length
   public wrapText(text: string, maxLineLength: number): string[] {
+    if (typeof maxLineLength !== "number" || isNaN(maxLineLength)) {
+      throw new Error(`wrapText: maxLineLength must be a number, got ${typeof maxLineLength}: ${maxLineLength}`);
+    }
     const result: string[] = [];
     // Split the text by newlines first
     const lines = text.split(/\r?\n/);

src/utils/src/text-wrapping/convertLineWidth.ts

@@ -0,0 +1,29 @@
+/**
+ * Converts a line width setting to a numeric character count
+ * @param width The width setting as a string or number
+ * @param isHanzi Whether the text uses Hanzi characters (Chinese, Japanese)
+ * @returns The number of characters per line
+ */
+export function convertLineWidth(width: string | number, isHanzi: boolean): number {
+  if (typeof width === 'number') return width;
+
+  if (!isHanzi) {
+    switch (width.toLowerCase()) {
+      case 'very narrow': return 21;
+      case 'narrow': return 30;
+      case 'medium': return 38;
+      case 'wide': return 44;
+      case 'very wide': return 52;
+      default: return 45;
+    }
+  } else {
+    switch (width.toLowerCase()) {
+      case 'very narrow': return 7;
+      case 'narrow': return 10;
+      case 'medium': return 14;
+      case 'wide': return 18;
+      case 'very wide': return 21;
+      default: return 14;
+    }
+  }
+}