chore: update documentation generation workflow and improve links in documentation

- Added pull request trigger to the documentation generation workflow for better automation.
- Updated links in PAGE_TEMPLATES.md to use absolute paths for consistency and clarity.
- Enhanced the generate-schema-docs test suite with additional cases for handling schemas with array items and single items.

Author: mokhld

.github/workflows/documentation-generation.yml

@@ -1,9 +1,8 @@
 name: Documentation Generation
 
 on:
+  pull_request:
   push:
-    branches:
-      - main
 
 permissions:
   contents: read

docs/features/configuration-based/PAGE_TEMPLATES.md

@@ -155,8 +155,8 @@ Full example of the minified and escaped component, which can be appended to [th
 
 ## Providing your own filters
 
-Whilst DXT offers some out of the box filters, teams using the plugin have the capability to provide their own. See [PLUGIN_OPTIONS.md](../../PLUGIN_OPTIONS.md#custom-filters) for more information.
+Whilst DXT offers some out of the box filters, teams using the plugin have the capability to provide their own. See [PLUGIN_OPTIONS.md](/PLUGIN_OPTIONS) for more information.
 
 ## Using page templates with data from your own API
 
-Page templates have access to``{{ context.data  }}` , which is an attribute made available when a page event is triggered. It represents the entire response body from your API. To learn more about this, [see our guidance on page events](./PAGE_EVENTS.md).
+Page templates have access to``{{ context.data  }}` , which is an attribute made available when a page event is triggered. It represents the entire response body from your API. To learn more about this, [see our guidance on page events](/features/configuration-based/PAGE_EVENTS).

scripts/generate-schema-docs.test.js

@@ -73,19 +73,27 @@ jest.mock('path', () => {
 
       if (typeof args[0] === 'string') {
         if (args[0] === '/mock/schemas/dir' && args[1]) {
-          return `/mock/schemas/dir/${args[1]}`
+          const arg1String =
+            typeof args[1] === 'string' ? args[1] : JSON.stringify(args[1])
+          return `/mock/schemas/dir/${arg1String}`
         }
 
         if (args[0] === '/mock/docs/dir' && args[1]) {
-          return `/mock/docs/dir/${args[1]}`
+          const arg1String =
+            typeof args[1] === 'string' ? args[1] : JSON.stringify(args[1])
+          return `/mock/docs/dir/${arg1String}`
         }
 
         if (args[0] === '/mock/temp/dir' && args[1]) {
-          return `/mock/temp/dir/${args[1]}`
+          const arg1String =
+            typeof args[1] === 'string' ? args[1] : JSON.stringify(args[1])
+          return `/mock/temp/dir/${arg1String}`
         }
 
         if (args[0].includes('/docs/dir') && args[1]) {
-          return `/mock/docs/dir/${args[1]}`
+          const arg1String =
+            typeof args[1] === 'string' ? args[1] : JSON.stringify(args[1])
+          return `/mock/docs/dir/${arg1String}`
         }
       }
 
@@ -365,6 +373,42 @@ describe('Schema Documentation Generator', () => {
       expect(capturedContent).toContain('* [schema1](schema1.md)')
       expect(capturedContent).toContain('* [schema2](schema2.md)')
     })
+
+    it('categorizes schemas correctly into core and advanced', () => {
+      path.basename.mockImplementation((filename) =>
+        filename.replace('.json', '')
+      )
+      let capturedContent = ''
+      fs.writeFileSync.mockImplementation((path, content) => {
+        capturedContent = content
+      })
+
+      const schemaFiles = [
+        'component-schema-v2.json', // core
+        'form-metadata-author-schema.json', // advanced
+        'uncategorized-schema.json' // neither
+      ]
+
+      const consoleSpy = jest.spyOn(console, 'log').mockImplementation()
+
+      createIndexFile(schemaFiles)
+
+      expect(capturedContent).toContain(
+        '* [component-schema-v2](component-schema-v2.md)'
+      )
+
+      expect(capturedContent).toContain(
+        '* [form-metadata-author-schema](form-metadata-author-schema.md)'
+      )
+
+      expect(consoleSpy).toHaveBeenCalledWith(
+        expect.stringContaining(
+          "Schema 'uncategorized-schema' is not categorised"
+        )
+      )
+
+      consoleSpy.mockRestore()
+    })
   })
 
   describe('cleanupFiles', () => {
@@ -566,6 +610,67 @@ describe('Schema Documentation Generator', () => {
         'Display Component Name'
       )
     })
+
+    it('handles array items in schema', () => {
+      const schemaWithArrayItems = {
+        title: 'Array Items Schema',
+        items: [
+          {
+            title: 'First Item',
+            properties: { prop: { title: 'Property' } }
+          },
+          {
+            type: 'string'
+          }
+        ]
+      }
+
+      const titleMap = {}
+      buildTitleMap(schemaWithArrayItems, 'array-items', titleMap)
+
+      expect(titleMap['array-items']).toBe('Array Items Schema')
+      expect(titleMap['array-items/items/0']).toBe('First Item')
+      expect(titleMap['array-items/items/1']).toBe('Item 2')
+      expect(titleMap['array-items/items/0/properties/prop']).toBe('Property')
+    })
+
+    it('handles single item schema', () => {
+      const schemaWithSingleItem = {
+        title: 'Single Item Schema',
+        items: {
+          title: 'The Item',
+          properties: {
+            name: { title: 'Name Property' },
+            age: { type: 'number' }
+          }
+        }
+      }
+
+      const titleMap = {}
+      buildTitleMap(schemaWithSingleItem, 'single-item', titleMap)
+
+      expect(titleMap['single-item']).toBe('Single Item Schema')
+      expect(titleMap['single-item/items']).toBe('The Item')
+      expect(titleMap['single-item/items/properties/name']).toBe(
+        'Name Property'
+      )
+      expect(titleMap['single-item/items/properties/age']).toBe('Age')
+    })
+
+    it('handles item without a title', () => {
+      const schemaWithoutItemTitle = {
+        title: 'No Item Title Schema',
+        items: {
+          type: 'string',
+          properties: { prop: { type: 'string' } }
+        }
+      }
+
+      const titleMap = {}
+      buildTitleMap(schemaWithoutItemTitle, 'no-item-title', titleMap)
+
+      expect(titleMap['no-item-title/items']).toBe('Item')
+    })
   })
 
   describe('formatPropertyName', () => {