@atikk-co-jp/notion-mcp-server 0.6.0 → 0.8.0

package/dist/src/converters/__tests__/markdown-to-blocks.test.js

@@ -1,5 +1,10 @@
 import { describe, expect, it } from 'vitest';
 import { markdownToBlocks, parseInlineMarkdown } from '../markdown-to-blocks.js';
+/**
+ * BlockObjectRequestから特定のプロパティにアクセスするヘルパー
+ * SDK型は discriminated union なので、直接プロパティアクセスできない
+ */
+const getBlockProp = (block, prop) => block[prop];
 describe('parseInlineMarkdown', () => {
     describe('plain text', () => {
         it('should return plain text as single rich text item', () => {
@@ -75,7 +80,7 @@ describe('markdownToBlocks', () => {
             const result = markdownToBlocks('Hello world');
             expect(result).toHaveLength(1);
             expect(result[0].type).toBe('paragraph');
-            expect(result[0].paragraph).toEqual({
+            expect(getBlockProp(result[0], 'paragraph')).toEqual({
                 rich_text: [{ type: 'text', text: { content: 'Hello world' } }],
             });
         });
@@ -91,7 +96,7 @@ describe('markdownToBlocks', () => {
             const result = markdownToBlocks('# Heading 1');
             expect(result).toHaveLength(1);
             expect(result[0].type).toBe('heading_1');
-            expect(result[0].heading_1).toEqual({
+            expect(getBlockProp(result[0], 'heading_1')).toEqual({
                 rich_text: [{ type: 'text', text: { content: 'Heading 1' } }],
             });
         });
@@ -130,7 +135,7 @@ describe('markdownToBlocks', () => {
             const result = markdownToBlocks('- [ ] Task');
             expect(result).toHaveLength(1);
             expect(result[0].type).toBe('to_do');
-            expect(result[0].to_do).toEqual({
+            expect(getBlockProp(result[0], 'to_do')).toEqual({
                 rich_text: [{ type: 'text', text: { content: 'Task' } }],
                 checked: false,
             });
@@ -139,11 +144,11 @@ describe('markdownToBlocks', () => {
             const result = markdownToBlocks('- [x] Done');
             expect(result).toHaveLength(1);
             expect(result[0].type).toBe('to_do');
-            expect(result[0].to_do.checked).toBe(true);
+            expect(getBlockProp(result[0], 'to_do').checked).toBe(true);
         });
         it('should handle - [X] (uppercase)', () => {
             const result = markdownToBlocks('- [X] Done');
-            expect(result[0].to_do.checked).toBe(true);
+            expect(getBlockProp(result[0], 'to_do').checked).toBe(true);
         });
     });
     describe('code blocks', () => {
@@ -151,18 +156,18 @@ describe('markdownToBlocks', () => {
             const result = markdownToBlocks('```\nconst x = 1\n```');
             expect(result).toHaveLength(1);
             expect(result[0].type).toBe('code');
-            expect(result[0].code).toEqual({
+            expect(getBlockProp(result[0], 'code')).toEqual({
                 rich_text: [{ type: 'text', text: { content: 'const x = 1' } }],
                 language: 'plain text',
             });
         });
         it('should parse language from code fence', () => {
             const result = markdownToBlocks('```typescript\nconst x: number = 1\n```');
-            expect(result[0].code.language).toBe('typescript');
+            expect(getBlockProp(result[0], 'code').language).toBe('typescript');
         });
         it('should handle multi-line code', () => {
             const result = markdownToBlocks('```js\nline1\nline2\nline3\n```');
-            const code = result[0].code;
+            const code = getBlockProp(result[0], 'code');
             expect(code.rich_text[0].text.content).toBe('line1\nline2\nline3');
         });
     });
@@ -171,14 +176,14 @@ describe('markdownToBlocks', () => {
             const result = markdownToBlocks('> This is a quote');
             expect(result).toHaveLength(1);
             expect(result[0].type).toBe('quote');
-            expect(result[0].quote).toEqual({
+            expect(getBlockProp(result[0], 'quote')).toEqual({
                 rich_text: [{ type: 'text', text: { content: 'This is a quote' } }],
             });
         });
         it('should combine consecutive quote lines', () => {
             const result = markdownToBlocks('> Line 1\n> Line 2');
             expect(result).toHaveLength(1);
-            const quote = result[0].quote;
+            const quote = getBlockProp(result[0], 'quote');
             expect(quote.rich_text[0].text.content).toBe('Line 1\nLine 2');
         });
     });
@@ -187,7 +192,7 @@ describe('markdownToBlocks', () => {
             const result = markdownToBlocks('---');
             expect(result).toHaveLength(1);
             expect(result[0].type).toBe('divider');
-            expect(result[0].divider).toEqual({});
+            expect(getBlockProp(result[0], 'divider')).toEqual({});
         });
         it('should handle ---- (multiple dashes)', () => {
             const result = markdownToBlocks('----');
@@ -199,7 +204,7 @@ describe('markdownToBlocks', () => {
             const result = markdownToBlocks('![Alt text](https://example.com/image.png)');
             expect(result).toHaveLength(1);
             expect(result[0].type).toBe('image');
-            expect(result[0].image).toEqual({
+            expect(getBlockProp(result[0], 'image')).toEqual({
                 type: 'external',
                 external: { url: 'https://example.com/image.png' },
                 caption: [{ type: 'text', text: { content: 'Alt text' } }],
@@ -207,7 +212,88 @@ describe('markdownToBlocks', () => {
         });
         it('should handle image without alt text', () => {
             const result = markdownToBlocks('![](https://example.com/image.png)');
-            expect(result[0].image.caption).toEqual([]);
+            expect(getBlockProp(result[0], 'image').caption).toEqual([]);
+        });
+    });
+    describe('tables', () => {
+        it('should convert simple table to table block', () => {
+            const markdown = `| Name | Age |
+|------|-----|
+| Alice | 30 |
+| Bob | 25 |`;
+            const result = markdownToBlocks(markdown);
+            expect(result).toHaveLength(1);
+            expect(result[0].type).toBe('table');
+            const table = result[0].table;
+            expect(table.table_width).toBe(2);
+            expect(table.has_column_header).toBe(true);
+            expect(table.has_row_header).toBe(false);
+            expect(table.children).toHaveLength(3); // header + 2 data rows
+        });
+        it('should handle table without header separator', () => {
+            const markdown = `| Name | Age |
+| Alice | 30 |
+| Bob | 25 |`;
+            const result = markdownToBlocks(markdown);
+            expect(result).toHaveLength(1);
+            expect(result[0].type).toBe('table');
+            const table = result[0].table;
+            expect(table.has_column_header).toBe(false);
+            expect(table.children).toHaveLength(3);
+        });
+        it('should parse inline formatting in table cells', () => {
+            const markdown = `| Feature | Status |
+|---------|--------|
+| **Bold** | *italic* |
+| \`code\` | [link](https://example.com) |`;
+            const result = markdownToBlocks(markdown);
+            expect(result).toHaveLength(1);
+            expect(result[0].type).toBe('table');
+            const table = result[0].table;
+            // Check first data row (index 1, after header)
+            const firstDataRow = table.children[1].table_row.cells;
+            expect(firstDataRow[0][0].text.content).toBe('Bold');
+            expect(firstDataRow[0][0].annotations?.bold).toBe(true);
+            expect(firstDataRow[1][0].text.content).toBe('italic');
+            expect(firstDataRow[1][0].annotations?.italic).toBe(true);
+            // Check second data row
+            const secondDataRow = table.children[2].table_row.cells;
+            expect(secondDataRow[0][0].text.content).toBe('code');
+            expect(secondDataRow[0][0].annotations?.code).toBe(true);
+        });
+        it('should handle table with 3+ columns', () => {
+            const markdown = `| エージェント名 | 目的 | プロンプトの特徴 |
+|--------------|------|----------------|
+| ResearchAgent | トピックのリサーチ | 5つの視点 |
+| ContentAgent | コンテンツ生成 | 品質ガイドライン |`;
+            const result = markdownToBlocks(markdown);
+            expect(result).toHaveLength(1);
+            expect(result[0].type).toBe('table');
+            const table = result[0].table;
+            expect(table.table_width).toBe(3);
+            expect(table.children).toHaveLength(3); // header + 2 data rows
+        });
+        it('should handle table with alignment indicators', () => {
+            const markdown = `| Left | Center | Right |
+|:-----|:------:|------:|
+| L | C | R |`;
+            const result = markdownToBlocks(markdown);
+            expect(result).toHaveLength(1);
+            expect(result[0].type).toBe('table');
+            const table = result[0].table;
+            expect(table.has_column_header).toBe(true);
+        });
+        it('should handle empty cells', () => {
+            const markdown = `| A | B |
+|---|---|
+|   | value |
+| value |  |`;
+            const result = markdownToBlocks(markdown);
+            expect(result).toHaveLength(1);
+            const table = result[0].table;
+            // Empty cells should have empty content
+            expect(table.children[1].table_row.cells[0]).toEqual([]);
+            expect(table.children[2].table_row.cells[1]).toEqual([]);
         });
     });
     describe('mixed content', () => {
@@ -241,5 +327,22 @@ const x = 1
             expect(result[7].type).toBe('divider');
             expect(result[8].type).toBe('image');
         });
+        it('should handle document with tables among other content', () => {
+            const markdown = `## エージェント一覧
+
+以下のエージェントがあります：
+
+| Name | Purpose |
+|------|---------|
+| Agent1 | Research |
+| Agent2 | Generate |
+
+詳細は下記を参照。`;
+            const result = markdownToBlocks(markdown);
+            expect(result[0].type).toBe('heading_2');
+            expect(result[1].type).toBe('paragraph');
+            expect(result[2].type).toBe('table');
+            expect(result[3].type).toBe('paragraph');
+        });
     });
 });