Merge pull request #554 from callumalpass/callumalpass/issue520

Author: callumalpass

src/services/FieldMapper.ts

@@ -109,7 +109,9 @@ export class FieldMapper {
         }
 
         if (frontmatter[this.mapping.timeEntries] !== undefined) {
-            mapped.timeEntries = frontmatter[this.mapping.timeEntries];
+            // Ensure timeEntries is always an array
+            const timeEntriesValue = frontmatter[this.mapping.timeEntries];
+            mapped.timeEntries = Array.isArray(timeEntriesValue) ? timeEntriesValue : [];
         }
 
         if (frontmatter[this.mapping.completeInstances] !== undefined) {
@@ -137,7 +139,8 @@ export class FieldMapper {
 
         // Handle time entries
         if (frontmatter.timeEntries !== undefined) {
-            mapped.timeEntries = frontmatter.timeEntries;
+            // Ensure timeEntries is always an array
+            mapped.timeEntries = Array.isArray(frontmatter.timeEntries) ? frontmatter.timeEntries : [];
         }
 
         return mapped;

src/services/NaturalLanguageParser.ts

@@ -226,6 +226,10 @@ export class NaturalLanguageParser {
                 const candidates = [config.label, config.value];
 
                 for (const candidate of candidates) {
+                    // Skip empty candidates
+                    if (!candidate || candidate.trim() === '') {
+                        continue;
+                    }
                     const match = this.findStatusMatch(text, candidate);
                     if (match) {
                         result.status = config.value;
@@ -251,6 +255,11 @@ export class NaturalLanguageParser {
      * Returns the match details or null if no valid match found.
      */
     private findStatusMatch(text: string, statusText: string): { fullMatch: string; startIndex: number } | null {
+        // Guard against empty status text to prevent infinite loop
+        if (!statusText || statusText.trim() === '') {
+            return null;
+        }
+        
         const lowerText = text.toLowerCase();
         const lowerStatus = statusText.toLowerCase();
 

src/services/StatusSuggestionService.ts

@@ -38,6 +38,7 @@ export class StatusSuggestionService {
         const q = query.toLowerCase();
         return statusConfigs
             .filter(s => s && typeof s.value === 'string' && typeof s.label === 'string')
+            .filter(s => s.value.trim() !== '' && s.label.trim() !== '') // Filter out empty values
             .filter(s => s.value.toLowerCase().includes(q) || s.label.toLowerCase().includes(q))
             .slice(0, limit)
             .map(s => ({