Fix regex to handle attributes without Version history section

Co-authored-by: abraham <[email protected]>

Author: Copilot

dist/schema.json

@@ -34723,12 +34723,23 @@
         "type": "object",
         "description": "Represents the last read position within a user's timelines.",
         "properties": {
+          "last_read_id": {
+            "description": "The ID of the most recently viewed entity.",
+            "type": "string"
+          },
+          "updated_at": {
+            "description": "The timestamp of when the marker was set.",
+            "type": "string",
+            "format": "date-time"
+          },
           "version": {
             "description": "An incrementing counter, used for locking to prevent write conflicts.",
             "type": "integer"
           }
         },
         "required": [
+          "last_read_id",
+          "updated_at",
           "version"
         ],
         "externalDocs": {
@@ -34791,6 +34802,20 @@
                 "type": [
                   "object",
                   "null"
+                ],
+                "properties": {
+                  "x": {
+                    "description": "Horizontal focal point",
+                    "type": "number"
+                  },
+                  "y": {
+                    "description": "Vertical focal point",
+                    "type": "number"
+                  }
+                },
+                "required": [
+                  "x",
+                  "y"
                 ]
               }
             }

src/__tests__/integration/application-redirect-uri-description.test.ts

@@ -5,19 +5,25 @@ describe('Integration test - Application redirect_uri description', () => {
     const entityParser = new EntityParser();
     const entities = entityParser.parseAllEntities();
 
-    const applicationEntity = entities.find(entity => entity.name === 'Application');
+    const applicationEntity = entities.find(
+      (entity) => entity.name === 'Application'
+    );
     expect(applicationEntity).toBeDefined();
 
-    const redirectUriAttr = applicationEntity?.attributes.find(attr => attr.name === 'redirect_uri');
+    const redirectUriAttr = applicationEntity?.attributes.find(
+      (attr) => attr.name === 'redirect_uri'
+    );
     expect(redirectUriAttr).toBeDefined();
 
     // Verify the full description is captured including the newline reference
     expect(redirectUriAttr?.description).toContain('May contain');
     expect(redirectUriAttr?.description).toContain('\\n');
-    expect(redirectUriAttr?.description).toContain('characters when multiple redirect URIs are registered');
-    
+    expect(redirectUriAttr?.description).toContain(
+      'characters when multiple redirect URIs are registered'
+    );
+
     // Check that it doesn't contain any version history content
     expect(redirectUriAttr?.description).not.toContain('4.3.0');
     expect(redirectUriAttr?.description).not.toContain('deprecated in favour');
   });
-});
+});

src/__tests__/parsers/AttributeParser.newline-description.test.ts

@@ -19,7 +19,9 @@ describe('AttributeParser - Newline in description', () => {
       expect(attributes[0].name).toBe('redirect_uri');
       expect(attributes[0].description).toContain('May contain');
       expect(attributes[0].description).toContain('\\n');
-      expect(attributes[0].description).toContain('characters when multiple redirect URIs are registered');
+      expect(attributes[0].description).toContain(
+        'characters when multiple redirect URIs are registered'
+      );
       expect(attributes[0].deprecated).toBe(true);
     });
 
@@ -39,9 +41,15 @@ completely until the Type section.\\
 
       expect(attributes).toHaveLength(1);
       expect(attributes[0].name).toBe('multiline_field');
-      expect(attributes[0].description).toContain('This is a description that spans');
-      expect(attributes[0].description).toContain('multiple lines and should be captured');
-      expect(attributes[0].description).toContain('completely until the Type section');
+      expect(attributes[0].description).toContain(
+        'This is a description that spans'
+      );
+      expect(attributes[0].description).toContain(
+        'multiple lines and should be captured'
+      );
+      expect(attributes[0].description).toContain(
+        'completely until the Type section'
+      );
     });
   });
 
@@ -63,7 +71,9 @@ completely until the Type section.\\
       expect(attributes[0].name).toBe('redirect_uri');
       expect(attributes[0].description).toContain('May contain');
       expect(attributes[0].description).toContain('\\n');
-      expect(attributes[0].description).toContain('characters when multiple redirect URIs are registered');
+      expect(attributes[0].description).toContain(
+        'characters when multiple redirect URIs are registered'
+      );
       expect(attributes[0].deprecated).toBe(true);
     });
 
@@ -83,9 +93,15 @@ completely until the Type section.\\
 
       expect(attributes).toHaveLength(1);
       expect(attributes[0].name).toBe('multiline_field');
-      expect(attributes[0].description).toContain('This is a description that spans');
-      expect(attributes[0].description).toContain('multiple lines and should be captured');
-      expect(attributes[0].description).toContain('completely until the Type section');
+      expect(attributes[0].description).toContain(
+        'This is a description that spans'
+      );
+      expect(attributes[0].description).toContain(
+        'multiple lines and should be captured'
+      );
+      expect(attributes[0].description).toContain(
+        'completely until the Type section'
+      );
     });
   });
-});
+});

src/parsers/AttributeParser.ts

@@ -60,7 +60,7 @@ export class AttributeParser {
         /\*\*Description:\*\*\s*(.*?)\\?\s*\n\*\*Type:\*\*/s
       );
       const typeMatch = sectionContent.match(
-        /\*\*Type:\*\*\s*(.*?)(?=\n.*?(?:`[^`]+`\s*=|\*\*Version history:\*\*))/s
+        /\*\*Type:\*\*\s*(.*?)(?=\n(?:\*\*Version history:\*\*|###|####|`[^`]+`\s*=|\d+\.\d+\.\d+\s*-|$))/s
       );
 
       // Look for Version history in this specific section