meno-core 1.0.53 → 1.1.0

package/lib/server/services/cmsService.test.ts

@@ -65,15 +65,65 @@ const mockAuthorItems: CMSItem[] = [
 ];
 
 const mockBlogItems: CMSItem[] = [
-  { _id: '1', _filename: 'hello-world', _slug: 'hello-world', slug: 'hello-world', title: 'Hello World', content: 'First post', author: 'john-doe' },
-  { _id: '2', _filename: 'getting-started', _slug: 'getting-started', slug: 'getting-started', title: 'Getting Started', content: 'Guide', author: 'john-doe', relatedPosts: ['hello-world'] },
-  { _id: '3', _filename: 'advanced-tips', _slug: 'advanced-tips', slug: 'advanced-tips', title: 'Advanced Tips', content: 'Pro tips', author: 'jane-smith', relatedPosts: ['hello-world', 'getting-started'] },
+  {
+    _id: '1',
+    _filename: 'hello-world',
+    _slug: 'hello-world',
+    slug: 'hello-world',
+    title: 'Hello World',
+    content: 'First post',
+    author: 'john-doe',
+  },
+  {
+    _id: '2',
+    _filename: 'getting-started',
+    _slug: 'getting-started',
+    slug: 'getting-started',
+    title: 'Getting Started',
+    content: 'Guide',
+    author: 'john-doe',
+    relatedPosts: ['hello-world'],
+  },
+  {
+    _id: '3',
+    _filename: 'advanced-tips',
+    _slug: 'advanced-tips',
+    slug: 'advanced-tips',
+    title: 'Advanced Tips',
+    content: 'Pro tips',
+    author: 'jane-smith',
+    relatedPosts: ['hello-world', 'getting-started'],
+  },
 ];
 
 const mockProductItems: CMSItem[] = [
-  { _id: '1', _filename: 'laptop', slug: 'laptop', name: 'Laptop Pro', price: 999, featured: true, category: 'Electronics' },
-  { _id: '2', _filename: 'tshirt', slug: 'tshirt', name: 'Cool T-Shirt', price: 29, featured: false, category: 'Clothing' },
-  { _id: '3', _filename: 'headphones', slug: 'headphones', name: 'Wireless Headphones', price: 199, featured: true, category: 'Electronics' },
+  {
+    _id: '1',
+    _filename: 'laptop',
+    slug: 'laptop',
+    name: 'Laptop Pro',
+    price: 999,
+    featured: true,
+    category: 'Electronics',
+  },
+  {
+    _id: '2',
+    _filename: 'tshirt',
+    slug: 'tshirt',
+    name: 'Cool T-Shirt',
+    price: 29,
+    featured: false,
+    category: 'Clothing',
+  },
+  {
+    _id: '3',
+    _filename: 'headphones',
+    slug: 'headphones',
+    name: 'Wireless Headphones',
+    price: 199,
+    featured: true,
+    category: 'Electronics',
+  },
 ];
 
 describe('CMSService', () => {
@@ -82,11 +132,12 @@ describe('CMSService', () => {
 
   beforeEach(async () => {
     mockProvider = {
-      getAllSchemas: async () => new Map([
-        ['authors', mockAuthorSchemaInfo],
-        ['blog-posts', mockSchemaInfo],
-        ['products', mockProductSchemaInfo],
-      ]),
+      getAllSchemas: async () =>
+        new Map([
+          ['authors', mockAuthorSchemaInfo],
+          ['blog-posts', mockSchemaInfo],
+          ['products', mockProductSchemaInfo],
+        ]),
       getItems: async (collection: string) => {
         if (collection === 'authors') return mockAuthorItems;
         if (collection === 'blog-posts') return mockBlogItems;
@@ -95,28 +146,28 @@ describe('CMSService', () => {
       },
       getItemBySlug: async (collection: string, slug: string) => {
         if (collection === 'blog-posts') {
-          return mockBlogItems.find(i => i.slug === slug) || null;
+          return mockBlogItems.find((i) => i.slug === slug) || null;
         }
         if (collection === 'products') {
-          return mockProductItems.find(i => i.slug === slug) || null;
+          return mockProductItems.find((i) => i.slug === slug) || null;
         }
         return null;
       },
       getItemByFilename: async (collection: string, filename: string) => {
         if (collection === 'blog-posts') {
-          return mockBlogItems.find(i => i._filename === filename || i.slug === filename) || null;
+          return mockBlogItems.find((i) => i._filename === filename || i.slug === filename) || null;
         }
         if (collection === 'products') {
-          return mockProductItems.find(i => i._filename === filename || i.slug === filename) || null;
+          return mockProductItems.find((i) => i._filename === filename || i.slug === filename) || null;
         }
         return null;
       },
       getItemById: async (collection: string, id: string) => {
         if (collection === 'blog-posts') {
-          return mockBlogItems.find(i => i._id === id) || null;
+          return mockBlogItems.find((i) => i._id === id) || null;
         }
         if (collection === 'products') {
-          return mockProductItems.find(i => i._id === id) || null;
+          return mockProductItems.find((i) => i._id === id) || null;
         }
         return null;
       },
@@ -128,7 +179,9 @@ describe('CMSService', () => {
       hasDraft: async () => false,
       saveDraft: async () => {},
       discardDraft: async () => {},
-      publishDraft: async () => { throw new Error('publishDraft not implemented in default mock'); },
+      publishDraft: async () => {
+        throw new Error('publishDraft not implemented in default mock');
+      },
     };
     service = new CMSService(mockProvider);
     await service.initialize();
@@ -175,7 +228,7 @@ describe('CMSService', () => {
       expect(match).toEqual({
         collection: 'blog-posts',
         slug: 'hello-world',
-        item: mockBlogItems[0],
+        item: mockBlogItems[0]!,
         pagePath: '/templates/blog-post.json',
       });
     });
@@ -185,7 +238,7 @@ describe('CMSService', () => {
       expect(match).toEqual({
         collection: 'products',
         slug: 'laptop',
-        item: mockProductItems[0],
+        item: mockProductItems[0]!,
         pagePath: '/templates/product.json',
       });
     });
@@ -214,20 +267,12 @@ describe('CMSService', () => {
   describe('getCollectionURLs', () => {
     it('should return all URLs for blog-posts collection', async () => {
       const urls = await service.getCollectionURLs('blog-posts');
-      expect(urls).toEqual([
-        '/blog/hello-world',
-        '/blog/getting-started',
-        '/blog/advanced-tips',
-      ]);
+      expect(urls).toEqual(['/blog/hello-world', '/blog/getting-started', '/blog/advanced-tips']);
     });
 
     it('should return all URLs for products collection', async () => {
       const urls = await service.getCollectionURLs('products');
-      expect(urls).toEqual([
-        '/shop/laptop',
-        '/shop/tshirt',
-        '/shop/headphones',
-      ]);
+      expect(urls).toEqual(['/shop/laptop', '/shop/tshirt', '/shop/headphones']);
     });
 
     it('should return empty array for unknown collection', async () => {
@@ -262,7 +307,7 @@ describe('CMSService', () => {
         filter: { featured: true },
       });
       expect(items).toHaveLength(2);
-      expect(items.every(i => i.featured === true)).toBe(true);
+      expect(items.every((i) => i.featured === true)).toBe(true);
     });
 
     it('should apply filter with eq operator', async () => {
@@ -271,7 +316,7 @@ describe('CMSService', () => {
         filter: { field: 'category', operator: 'eq', value: 'Electronics' },
       });
       expect(items).toHaveLength(2);
-      expect(items.every(i => i.category === 'Electronics')).toBe(true);
+      expect(items.every((i) => i.category === 'Electronics')).toBe(true);
     });
 
     it('should apply filter with neq operator', async () => {
@@ -280,7 +325,7 @@ describe('CMSService', () => {
         filter: { field: 'category', operator: 'neq', value: 'Electronics' },
       });
       expect(items).toHaveLength(1);
-      expect(items[0].category).toBe('Clothing');
+      expect(items[0]!.category).toBe('Clothing');
     });
 
     it('should apply filter with gt operator', async () => {
@@ -289,7 +334,7 @@ describe('CMSService', () => {
         filter: { field: 'price', operator: 'gt', value: 100 },
       });
       expect(items).toHaveLength(2);
-      expect(items.every(i => (i.price as number) > 100)).toBe(true);
+      expect(items.every((i) => (i.price as number) > 100)).toBe(true);
     });
 
     it('should apply filter with gte operator', async () => {
@@ -306,7 +351,7 @@ describe('CMSService', () => {
         filter: { field: 'price', operator: 'lt', value: 100 },
       });
       expect(items).toHaveLength(1);
-      expect(items[0].name).toBe('Cool T-Shirt');
+      expect(items[0]!.name).toBe('Cool T-Shirt');
     });
 
     it('should apply filter with lte operator', async () => {
@@ -323,7 +368,7 @@ describe('CMSService', () => {
         filter: { field: 'name', operator: 'contains', value: 'Wireless' },
       });
       expect(items).toHaveLength(1);
-      expect(items[0].slug).toBe('headphones');
+      expect(items[0]!.slug).toBe('headphones');
     });
 
     it('should apply filter with in operator', async () => {
@@ -350,9 +395,9 @@ describe('CMSService', () => {
         collection: 'products',
         sort: { field: 'price', order: 'asc' },
       });
-      expect(items[0].price).toBe(29);
-      expect(items[1].price).toBe(199);
-      expect(items[2].price).toBe(999);
+      expect(items[0]!.price).toBe(29);
+      expect(items[1]!.price).toBe(199);
+      expect(items[2]!.price).toBe(999);
     });
 
     it('should apply descending sort', async () => {
@@ -360,9 +405,9 @@ describe('CMSService', () => {
         collection: 'products',
         sort: { field: 'price', order: 'desc' },
       });
-      expect(items[0].price).toBe(999);
-      expect(items[1].price).toBe(199);
-      expect(items[2].price).toBe(29);
+      expect(items[0]!.price).toBe(999);
+      expect(items[1]!.price).toBe(199);
+      expect(items[2]!.price).toBe(29);
     });
 
     it('should apply multi-field sort', async () => {
@@ -374,9 +419,9 @@ describe('CMSService', () => {
         ],
       });
       // Featured items first (true > false), then by price ascending
-      expect(items[0].slug).toBe('headphones'); // featured=true, price=199
-      expect(items[1].slug).toBe('laptop');     // featured=true, price=999
-      expect(items[2].slug).toBe('tshirt');     // featured=false, price=29
+      expect(items[0]!.slug).toBe('headphones'); // featured=true, price=199
+      expect(items[1]!.slug).toBe('laptop'); // featured=true, price=999
+      expect(items[2]!.slug).toBe('tshirt'); // featured=false, price=29
     });
 
     it('should apply limit', async () => {
@@ -393,7 +438,7 @@ describe('CMSService', () => {
         offset: 1,
       });
       expect(items).toHaveLength(2);
-      expect(items[0].slug).toBe('tshirt');
+      expect(items[0]!.slug).toBe('tshirt');
     });
 
     it('should apply limit and offset together', async () => {
@@ -404,7 +449,7 @@ describe('CMSService', () => {
         limit: 1,
       });
       expect(items).toHaveLength(1);
-      expect(items[0].price).toBe(199);
+      expect(items[0]!.price).toBe(199);
     });
 
     it('should return empty array for empty collection', async () => {
@@ -432,23 +477,23 @@ describe('CMSService', () => {
     it('should return items matching IDs in order', async () => {
       const items = await service.getItemsByIds('blog-posts', ['2', '1', '3']);
       expect(items).toHaveLength(3);
-      expect(items[0]._id).toBe('2');
-      expect(items[1]._id).toBe('1');
-      expect(items[2]._id).toBe('3');
+      expect(items[0]!._id).toBe('2');
+      expect(items[1]!._id).toBe('1');
+      expect(items[2]!._id).toBe('3');
     });
 
     it('should support _filename lookup', async () => {
       const items = await service.getItemsByIds('blog-posts', ['hello-world', 'advanced-tips']);
       expect(items).toHaveLength(2);
-      expect(items[0]._filename).toBe('hello-world');
-      expect(items[1]._filename).toBe('advanced-tips');
+      expect(items[0]!._filename).toBe('hello-world');
+      expect(items[1]!._filename).toBe('advanced-tips');
     });
 
     it('should filter out non-existent IDs', async () => {
       const items = await service.getItemsByIds('blog-posts', ['1', 'non-existent', '2']);
       expect(items).toHaveLength(2);
-      expect(items[0]._id).toBe('1');
-      expect(items[1]._id).toBe('2');
+      expect(items[0]!._id).toBe('1');
+      expect(items[1]!._id).toBe('2');
     });
 
     it('should return empty array for empty input', async () => {
@@ -459,9 +504,9 @@ describe('CMSService', () => {
     it('should handle mixed _id and _filename lookups', async () => {
       const items = await service.getItemsByIds('blog-posts', ['1', 'getting-started', '3']);
       expect(items).toHaveLength(3);
-      expect(items[0]._id).toBe('1');
-      expect(items[1]._filename).toBe('getting-started');
-      expect(items[2]._id).toBe('3');
+      expect(items[0]!._id).toBe('1');
+      expect(items[1]!._filename).toBe('getting-started');
+      expect(items[2]!._id).toBe('3');
     });
 
     it('should return empty array for unknown collection', async () => {
@@ -480,24 +525,24 @@ describe('CMSService', () => {
       // john-doe is referenced by 2 blog posts via 'author' field
       const refs = await service.findReferencesTo('authors', 'john-doe');
       expect(refs).toHaveLength(2);
-      expect(refs[0].collection).toBe('blog-posts');
-      expect(refs[0].fieldName).toBe('author');
-      expect(refs[0].itemFilename).toBe('hello-world');
-      expect(refs[1].itemFilename).toBe('getting-started');
+      expect(refs[0]!.collection).toBe('blog-posts');
+      expect(refs[0]!.fieldName).toBe('author');
+      expect(refs[0]!.itemFilename).toBe('hello-world');
+      expect(refs[1]!.itemFilename).toBe('getting-started');
     });
 
     it('should find references in multi-reference fields', async () => {
       // hello-world is referenced by 2 posts via 'relatedPosts' field
       const refs = await service.findReferencesTo('blog-posts', 'hello-world');
       expect(refs).toHaveLength(2);
-      expect(refs.every(r => r.fieldName === 'relatedPosts')).toBe(true);
+      expect(refs.every((r) => r.fieldName === 'relatedPosts')).toBe(true);
     });
 
     it('should return empty array when no references exist', async () => {
       // jane-smith is only referenced by advanced-tips
       const refs = await service.findReferencesTo('authors', 'jane-smith');
       expect(refs).toHaveLength(1);
-      expect(refs[0].itemFilename).toBe('advanced-tips');
+      expect(refs[0]!.itemFilename).toBe('advanced-tips');
     });
 
     it('should return empty array for non-existent item', async () => {
@@ -510,12 +555,12 @@ describe('CMSService', () => {
       // but not products (which has no reference fields to authors)
       const refs = await service.findReferencesTo('authors', 'john-doe');
       // All references should be from blog-posts, not products
-      expect(refs.every(r => r.collection === 'blog-posts')).toBe(true);
+      expect(refs.every((r) => r.collection === 'blog-posts')).toBe(true);
     });
 
     it('should include item metadata in results', async () => {
       const refs = await service.findReferencesTo('authors', 'john-doe');
-      const firstRef = refs[0];
+      const firstRef = refs[0]!;
       expect(firstRef.itemId).toBe('1');
       expect(firstRef.itemFilename).toBe('hello-world');
       expect(firstRef.itemSlug).toBe('hello-world');
@@ -543,8 +588,8 @@ describe('CMSService', () => {
       expect(savedItems).toHaveLength(2);
 
       // Both items should have author set to null
-      expect(savedItems[0].item.author).toBeNull();
-      expect(savedItems[1].item.author).toBeNull();
+      expect(savedItems[0]!.item.author).toBeNull();
+      expect(savedItems[1]!.item.author).toBeNull();
     });
 
     it('should filter multi-reference arrays', async () => {
@@ -554,11 +599,11 @@ describe('CMSService', () => {
       expect(removedCount).toBe(2);
 
       // getting-started had ['hello-world'] -> should become null (empty array)
-      const gettingStarted = savedItems.find(s => s.item._filename === 'getting-started');
+      const gettingStarted = savedItems.find((s) => s.item._filename === 'getting-started');
       expect(gettingStarted?.item.relatedPosts).toBeNull();
 
       // advanced-tips had ['hello-world', 'getting-started'] -> should become ['getting-started']
-      const advancedTips = savedItems.find(s => s.item._filename === 'advanced-tips');
+      const advancedTips = savedItems.find((s) => s.item._filename === 'advanced-tips');
       expect(advancedTips?.item.relatedPosts).toEqual(['getting-started']);
     });
 
@@ -566,7 +611,7 @@ describe('CMSService', () => {
       // getting-started has relatedPosts: ['hello-world'] - when hello-world is removed, should be null
       await service.removeReferencesTo('blog-posts', 'hello-world');
 
-      const gettingStarted = savedItems.find(s => s.item._filename === 'getting-started');
+      const gettingStarted = savedItems.find((s) => s.item._filename === 'getting-started');
       // Empty array should become null, not []
       expect(gettingStarted?.item.relatedPosts).toBeNull();
     });
@@ -593,7 +638,7 @@ describe('CMSService', () => {
     it('should preserve other item fields', async () => {
       await service.removeReferencesTo('authors', 'john-doe');
 
-      const helloWorld = savedItems.find(s => s.item._filename === 'hello-world');
+      const helloWorld = savedItems.find((s) => s.item._filename === 'hello-world');
       expect(helloWorld?.item._id).toBe('1');
       expect(helloWorld?.item._filename).toBe('hello-world');
       expect(helloWorld?.item.title).toBe('Hello World');
@@ -653,7 +698,9 @@ describe('CMSService', () => {
         hasDraft: async () => false,
         saveDraft: async () => {},
         discardDraft: async () => {},
-        publishDraft: async () => { throw new Error('not implemented'); },
+        publishDraft: async () => {
+          throw new Error('not implemented');
+        },
       };
 
       const richService = new CMSService(richTextProvider);
@@ -662,7 +709,7 @@ describe('CMSService', () => {
       const items = await richService.queryItems({ collection: 'articles' });
 
       expect(items).toHaveLength(1);
-      const item = items[0];
+      const item = items[0]!;
 
       // Content should be converted to __richtext__ marker
       expect(item.content).toBeDefined();
@@ -708,14 +755,16 @@ describe('CMSService', () => {
         hasDraft: async () => false,
         saveDraft: async () => {},
         discardDraft: async () => {},
-        publishDraft: async () => { throw new Error('not implemented'); },
+        publishDraft: async () => {
+          throw new Error('not implemented');
+        },
       };
 
       const svc = new CMSService(provider);
       await svc.initialize();
 
       const items = await svc.queryItems({ collection: 'articles' });
-      const item = items[0];
+      const item = items[0]!;
 
       expect((item.content as Record<string, unknown>).__richtext__).toBe(true);
       expect((item.content as Record<string, unknown>).html).toBe('<p>Already HTML</p>');
@@ -736,9 +785,7 @@ describe('CMSService', () => {
         pagePath: '/templates/article.json',
       };
 
-      const mockItems: CMSItem[] = [
-        { _id: '1', _filename: 'test', slug: 'test', summary: 'Plain text summary' },
-      ];
+      const mockItems: CMSItem[] = [{ _id: '1', _filename: 'test', slug: 'test', summary: 'Plain text summary' }];
 
       const provider: CMSProvider = {
         getAllSchemas: async () => new Map([['articles', schemaInfo]]),
@@ -753,7 +800,9 @@ describe('CMSService', () => {
         hasDraft: async () => false,
         saveDraft: async () => {},
         discardDraft: async () => {},
-        publishDraft: async () => { throw new Error('not implemented'); },
+        publishDraft: async () => {
+          throw new Error('not implemented');
+        },
       };
 
       const svc = new CMSService(provider);
@@ -761,7 +810,7 @@ describe('CMSService', () => {
 
       const items = await svc.queryItems({ collection: 'articles' });
       // Regular text field should remain unchanged
-      expect(items[0].summary).toBe('Plain text summary');
+      expect(items[0]!.summary).toBe('Plain text summary');
     });
 
     it('should skip null/undefined richtext fields', async () => {
@@ -797,39 +846,37 @@ describe('CMSService', () => {
         hasDraft: async () => false,
         saveDraft: async () => {},
         discardDraft: async () => {},
-        publishDraft: async () => { throw new Error('not implemented'); },
+        publishDraft: async () => {
+          throw new Error('not implemented');
+        },
       };
 
       const svc = new CMSService(provider);
       await svc.initialize();
 
       const items = await svc.queryItems({ collection: 'articles' });
-      expect(items[0].content).toBeNull();
-      expect(items[1].content).toBeUndefined();
+      expect(items[0]!.content).toBeNull();
+      expect(items[1]!.content).toBeUndefined();
     });
   });
 
   describe('drafts', () => {
-    function makeService(opts: {
-      published?: CMSItem[];
-      drafts?: CMSItem[];
-    }): CMSService {
+    function makeService(opts: { published?: CMSItem[]; drafts?: CMSItem[] }): CMSService {
       const published = opts.published ?? [];
       const drafts = opts.drafts ?? [];
-      const findByFilename = (list: CMSItem[], f: string) =>
-        list.find(i => i._filename === f) || null;
+      const findByFilename = (list: CMSItem[], f: string) => list.find((i) => i._filename === f) || null;
 
       const provider: CMSProvider = {
         getAllSchemas: async () => new Map([['blog-posts', mockSchemaInfo]]),
         getItems: async () => published,
         getItemBySlug: async (_c, s) => findByFilename(published, s),
         getItemByFilename: async (_c, f) => findByFilename(published, f),
-        getItemById: async (_c, id) => published.find(i => i._id === id) ?? null,
+        getItemById: async (_c, id) => published.find((i) => i._id === id) ?? null,
         saveItem: async () => {},
         deleteItem: async () => {},
         getDraft: async (_c, f) => findByFilename(drafts, f),
         getAllDrafts: async () => drafts,
-        hasDraft: async (_c, f) => drafts.some(d => d._filename === f),
+        hasDraft: async (_c, f) => drafts.some((d) => d._filename === f),
         saveDraft: async () => {},
         discardDraft: async () => {},
         publishDraft: async (_c, f) => {
@@ -870,15 +917,13 @@ describe('CMSService', () => {
           { _id: '1', _filename: 'hello', slug: 'hello', title: 'Hello' },
           { _id: '2', _filename: 'second', slug: 'second', title: 'Second' },
         ],
-        drafts: [
-          { _id: '1', _filename: 'hello', slug: 'hello', title: 'Hello (WIP)', _isDraft: true },
-        ],
+        drafts: [{ _id: '1', _filename: 'hello', slug: 'hello', title: 'Hello (WIP)', _isDraft: true }],
       });
       await svc.initialize();
       const list = await svc.listItemsWithDraftFlag('blog-posts');
       expect(list).toHaveLength(2);
-      const hello = list.find(i => i._filename === 'hello');
-      const second = list.find(i => i._filename === 'second');
+      const hello = list.find((i) => i._filename === 'hello');
+      const second = list.find((i) => i._filename === 'second');
       expect(hello?._hasDraft).toBe(true);
       expect(second?._hasDraft).toBeUndefined();
     });
@@ -886,28 +931,24 @@ describe('CMSService', () => {
     it('listItemsWithDraftFlag includes draft-only items at end', async () => {
       const svc = makeService({
         published: [{ _id: '1', _filename: 'hello', slug: 'hello', title: 'Hello' }],
-        drafts: [
-          { _id: 'new', _filename: 'brand-new', slug: 'brand-new', title: 'New', _isDraft: true },
-        ],
+        drafts: [{ _id: 'new', _filename: 'brand-new', slug: 'brand-new', title: 'New', _isDraft: true }],
       });
       await svc.initialize();
       const list = await svc.listItemsWithDraftFlag('blog-posts');
       expect(list).toHaveLength(2);
-      const draftOnly = list.find(i => i._filename === 'brand-new');
+      const draftOnly = list.find((i) => i._filename === 'brand-new');
       expect(draftOnly?._isDraft).toBe(true);
     });
 
     it('queryItems is unaffected by drafts (still published-only)', async () => {
       const svc = makeService({
         published: [{ _id: '1', _filename: 'hello', slug: 'hello', title: 'Hello' }],
-        drafts: [
-          { _id: 'new', _filename: 'brand-new', slug: 'brand-new', title: 'New', _isDraft: true },
-        ],
+        drafts: [{ _id: 'new', _filename: 'brand-new', slug: 'brand-new', title: 'New', _isDraft: true }],
       });
       await svc.initialize();
       const items = await svc.queryItems({ collection: 'blog-posts' });
       expect(items).toHaveLength(1);
-      expect(items[0]._filename).toBe('hello');
+      expect(items[0]!._filename).toBe('hello');
     });
 
     it('publishDraft delegates to provider and invalidates cache', async () => {
@@ -922,30 +963,27 @@ describe('CMSService', () => {
   });
 
   describe('preview mode', () => {
-    function makeService(opts: {
-      previewMode?: boolean;
-      published?: CMSItem[];
-      drafts?: CMSItem[];
-    }): CMSService {
+    function makeService(opts: { previewMode?: boolean; published?: CMSItem[]; drafts?: CMSItem[] }): CMSService {
       const published = opts.published ?? [];
       const drafts = opts.drafts ?? [];
-      const findByFilename = (list: CMSItem[], f: string) =>
-        list.find(i => i._filename === f) || null;
+      const findByFilename = (list: CMSItem[], f: string) => list.find((i) => i._filename === f) || null;
 
       const provider: CMSProvider = {
         getAllSchemas: async () => new Map([['blog-posts', mockSchemaInfo]]),
         getItems: async () => published,
         getItemBySlug: async (_c, s) => findByFilename(published, s),
         getItemByFilename: async (_c, f) => findByFilename(published, f),
-        getItemById: async (_c, id) => published.find(i => i._id === id) ?? null,
+        getItemById: async (_c, id) => published.find((i) => i._id === id) ?? null,
         saveItem: async () => {},
         deleteItem: async () => {},
         getDraft: async (_c, f) => findByFilename(drafts, f),
         getAllDrafts: async () => drafts,
-        hasDraft: async (_c, f) => drafts.some(d => d._filename === f),
+        hasDraft: async (_c, f) => drafts.some((d) => d._filename === f),
         saveDraft: async () => {},
         discardDraft: async () => {},
-        publishDraft: async () => { throw new Error('not used'); },
+        publishDraft: async () => {
+          throw new Error('not used');
+        },
       };
       return new CMSService(provider, { previewMode: opts.previewMode });
     }
@@ -959,8 +997,8 @@ describe('CMSService', () => {
       await svc.initialize();
       const items = await svc.queryItems({ collection: 'blog-posts' });
       expect(items).toHaveLength(1);
-      expect(items[0].title).toBe('Hello (WIP)');
-      expect(items[0]._isDraft).toBe(true);
+      expect(items[0]!.title).toBe('Hello (WIP)');
+      expect(items[0]!._isDraft).toBe(true);
     });
 
     it('queryItems includes draft-only items in preview mode', async () => {
@@ -972,7 +1010,7 @@ describe('CMSService', () => {
       await svc.initialize();
       const items = await svc.queryItems({ collection: 'blog-posts' });
       expect(items).toHaveLength(2);
-      expect(items.find(i => i._filename === 'brand-new')?._isDraft).toBe(true);
+      expect(items.find((i) => i._filename === 'brand-new')?._isDraft).toBe(true);
     });
 
     it('getItemsByIds merges drafts in preview mode', async () => {
@@ -1003,7 +1041,7 @@ describe('CMSService', () => {
       expect(versions.draft?.title).toBe('Hello (WIP)');
 
       const list = await svc.listItemsWithDraftFlag('blog-posts');
-      const item = list.find(i => i._filename === 'hello');
+      const item = list.find((i) => i._filename === 'hello');
       expect(item?.title).toBe('Hello');
       expect(item?._hasDraft).toBe(true);
     });
@@ -1019,8 +1057,8 @@ describe('CMSService', () => {
       await svc.initialize();
       const items = await svc.queryItems({ collection: 'blog-posts' });
       expect(items).toHaveLength(1);
-      expect(items[0].title).toBe('Hello');
-      expect(items[0]._isDraft).toBeUndefined();
+      expect(items[0]!.title).toBe('Hello');
+      expect(items[0]!._isDraft).toBeUndefined();
     });
   });
 });