confluence-cli 1.10.1 → 1.11.1

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [1.11.1](https://github.com/pchuri/confluence-cli/compare/v1.11.0...v1.11.1) (2025-12-17)
+
+
+### Bug Fixes
+
+* support children macro, improve macro handling, and filter attachments ([#23](https://github.com/pchuri/confluence-cli/issues/23)) ([15b721a](https://github.com/pchuri/confluence-cli/commit/15b721acdc296e72470ee438c9fe3470e09ae52e))
+
+# [1.11.0](https://github.com/pchuri/confluence-cli/compare/v1.10.1...v1.11.0) (2025-12-12)
+
+
+### Features
+
+* Support for Confluence display URLs ([#20](https://github.com/pchuri/confluence-cli/issues/20)) ([3bda7c2](https://github.com/pchuri/confluence-cli/commit/3bda7c2aad8ec02dac60f3b7c34c31b549a31cce))
+
 ## [1.10.1](https://github.com/pchuri/confluence-cli/compare/v1.10.0...v1.10.1) (2025-12-08)
 
 

package/bin/confluence.js

@@ -445,7 +445,9 @@ program
       const contentExt = formatExt[format] || 'txt';
 
       const pageInfo = await client.getPageInfo(pageId);
-      const content = await client.readPage(pageId, format);
+      // Read page with attachment extraction enabled
+      const content = await client.readPage(pageId, format, { extractReferencedAttachments: true });
+      const referencedAttachments = client._referencedAttachments || new Set();
 
       const baseDir = path.resolve(options.dest || '.');
       const folderName = sanitizeTitle(pageInfo.title || 'page');
@@ -462,8 +464,16 @@ program
 
       if (!options.skipAttachments) {
         const pattern = options.pattern ? options.pattern.trim() : null;
-        const attachments = await client.getAllAttachments(pageId);
-        const filtered = pattern ? attachments.filter(att => client.matchesPattern(att.title, pattern)) : attachments;
+        const allAttachments = await client.getAllAttachments(pageId);
+        
+        // Filter: only referenced attachments (unless pattern is specified, then use pattern)
+        let filtered;
+        if (pattern) {
+          filtered = allAttachments.filter(att => client.matchesPattern(att.title, pattern));
+        } else {
+          // Only download attachments that are referenced in the page content
+          filtered = allAttachments.filter(att => referencedAttachments.has(att.title));
+        }
 
         if (filtered.length === 0) {
           console.log(chalk.yellow('No attachments to download.'));

package/lib/confluence-client.js

@@ -49,7 +49,7 @@ class ConfluenceClient {
   /**
    * Extract page ID from URL or return the ID if it's already a number
    */
-  extractPageId(pageIdOrUrl) {
+  async extractPageId(pageIdOrUrl) {
     if (typeof pageIdOrUrl === 'number' || /^\d+$/.test(pageIdOrUrl)) {
       return pageIdOrUrl;
     }
@@ -62,25 +62,83 @@ class ConfluenceClient {
         return pageIdMatch[1];
       }
       
-      // Handle display URLs - would need to search by space and title
+      // Handle display URLs - search by space and title
       const displayMatch = pageIdOrUrl.match(/\/display\/([^/]+)\/(.+)/);
       if (displayMatch) {
-        throw new Error('Display URLs not yet supported. Please use page ID or viewpage URL with pageId parameter.');
+        const spaceKey = displayMatch[1];
+        // Confluence friendly URLs for child pages might look like /display/SPACE/Parent/Child
+        // We only want the last part as the title
+        const urlPath = displayMatch[2];
+        const lastSegment = urlPath.split('/').pop();
+
+        // Confluence uses + for spaces in URL titles, but decodeURIComponent doesn't convert + to space
+        const rawTitle = lastSegment.replace(/\+/g, '%20');
+        const title = decodeURIComponent(rawTitle);
+
+        try {
+          const response = await this.client.get('/content', {
+            params: {
+              spaceKey: spaceKey,
+              title: title,
+              limit: 1
+            }
+          });
+
+          if (response.data.results && response.data.results.length > 0) {
+            return response.data.results[0].id;
+          }
+        } catch (error) {
+          // Ignore error and fall through
+          console.error('Error resolving page ID from display URL:', error);
+        }
+
+        throw new Error(`Could not resolve page ID from display URL: ${pageIdOrUrl}`);
       }
     }
 
     return pageIdOrUrl;
   }
 
+  /**
+   * Extract referenced attachment filenames from HTML content
+   * @param {string} htmlContent - HTML content in storage format
+   * @returns {Set<string>} Set of referenced attachment filenames
+   */
+  extractReferencedAttachments(htmlContent) {
+    const referenced = new Set();
+    
+    // Extract from ac:image with ri:attachment
+    const imageRegex = /<ac:image[^>]*>[\s\S]*?<ri:attachment\s+ri:filename="([^"]+)"[^>]*\/?>[\s\S]*?<\/ac:image>/g;
+    let match;
+    while ((match = imageRegex.exec(htmlContent)) !== null) {
+      referenced.add(match[1]);
+    }
+    
+    // Extract from view-file macro
+    const viewFileRegex = /<ac:structured-macro ac:name="view-file"[^>]*>[\s\S]*?<ri:attachment\s+ri:filename="([^"]+)"[^>]*\/?>[\s\S]*?<\/ac:structured-macro>/g;
+    while ((match = viewFileRegex.exec(htmlContent)) !== null) {
+      referenced.add(match[1]);
+    }
+    
+    // Extract from any ri:attachment references
+    const attachmentRegex = /<ri:attachment\s+ri:filename="([^"]+)"[^>]*\/?>/g;
+    while ((match = attachmentRegex.exec(htmlContent)) !== null) {
+      referenced.add(match[1]);
+    }
+    
+    return referenced;
+  }
+
   /**
    * Read a Confluence page content
    * @param {string} pageIdOrUrl - Page ID or URL
    * @param {string} format - Output format: 'text', 'html', or 'markdown'
    * @param {object} options - Additional options
    * @param {boolean} options.resolveUsers - Whether to resolve userkeys to display names (default: true for markdown)
+   * @param {boolean} options.extractReferencedAttachments - Whether to extract referenced attachments (default: false)
    */
   async readPage(pageIdOrUrl, format = 'text', options = {}) {
-    const pageId = this.extractPageId(pageIdOrUrl);
+    const pageId = await this.extractPageId(pageIdOrUrl);
     
     const response = await this.client.get(`/content/${pageId}`, {
       params: {
@@ -90,6 +148,11 @@ class ConfluenceClient {
 
     let htmlContent = response.data.body.storage.value;
     
+    // Extract referenced attachments if requested
+    if (options.extractReferencedAttachments) {
+      this._referencedAttachments = this.extractReferencedAttachments(htmlContent);
+    }
+    
     if (format === 'html') {
       return htmlContent;
     }
@@ -108,6 +171,9 @@ class ConfluenceClient {
         htmlContent = await this.resolvePageLinksInHtml(htmlContent);
       }
       
+      // Resolve children macro to child pages list
+      htmlContent = await this.resolveChildrenMacro(htmlContent, pageId);
+      
       return this.storageToMarkdown(htmlContent);
     }
     
@@ -127,7 +193,7 @@ class ConfluenceClient {
    * Get page information
    */
   async getPageInfo(pageIdOrUrl) {
-    const pageId = this.extractPageId(pageIdOrUrl);
+    const pageId = await this.extractPageId(pageIdOrUrl);
     
     const response = await this.client.get(`/content/${pageId}`, {
       params: {
@@ -331,11 +397,63 @@ class ConfluenceClient {
     return resolvedHtml;
   }
 
+  /**
+   * Resolve children macro to child pages list
+   * @param {string} html - HTML content with children macro
+   * @param {string} pageId - Page ID to get children from
+   * @returns {Promise<string>} - HTML with children macro replaced by markdown list
+   */
+  async resolveChildrenMacro(html, pageId) {
+    // Check if there's a children macro (self-closing or with closing tag)
+    const childrenMacroRegex = /<ac:structured-macro\s+ac:name="children"[^>]*(?:\/>|>[\s\S]*?<\/ac:structured-macro>)/g;
+    const hasChildrenMacro = childrenMacroRegex.test(html);
+    
+    if (!hasChildrenMacro) {
+      return html;
+    }
+
+    try {
+      // Get child pages with full info including _links
+      const response = await this.client.get(`/content/${pageId}/child/page`, {
+        params: {
+          limit: 500,
+          expand: 'space,version'
+        }
+      });
+      
+      const childPages = response.data.results || [];
+      
+      if (childPages.length === 0) {
+        // No children, remove the macro
+        return html.replace(childrenMacroRegex, '');
+      }
+
+      // Convert child pages to markdown list
+      // Format: - [Page Title](URL)
+      const childPagesList = childPages.map(page => {
+        const webui = page._links?.webui || '';
+        const url = webui ? `https://${this.domain}/wiki${webui}` : '';
+        if (url) {
+          return `- [${page.title}](${url})`;
+        } else {
+          return `- ${page.title}`;
+        }
+      }).join('\n');
+
+      // Replace children macro with markdown list
+      return html.replace(childrenMacroRegex, `\n${childPagesList}\n`);
+    } catch (error) {
+      // If error getting children, just remove the macro
+      console.error(`Error resolving children macro: ${error.message}`);
+      return html.replace(childrenMacroRegex, '');
+    }
+  }
+
   /**
    * List attachments for a page with pagination support
    */
   async listAttachments(pageIdOrUrl, options = {}) {
-    const pageId = this.extractPageId(pageIdOrUrl);
+    const pageId = await this.extractPageId(pageIdOrUrl);
     const limit = this.parsePositiveInt(options.limit, 50);
     const start = this.parsePositiveInt(options.start, 0);
     const params = {
@@ -402,7 +520,7 @@ class ConfluenceClient {
       downloadUrl = attachmentIdOrAttachment.downloadLink;
     } else {
       // Otherwise, fetch attachment info to get the download link
-      const pageId = this.extractPageId(pageIdOrUrl);
+      const pageId = await this.extractPageId(pageIdOrUrl);
       const attachmentId = attachmentIdOrAttachment;
       const response = await this.client.get(`/content/${pageId}/child/attachment`, {
         params: { limit: 500 }
@@ -529,33 +647,8 @@ class ConfluenceClient {
     // Convert markdown to HTML first
     const html = this.markdown.render(markdown);
     
-    // Simple HTML to Storage format conversion
-    // This is a basic implementation - for full support, we'd need a more sophisticated converter
-    let storage = html
-      .replace(/<h1>/g, '<h1>')
-      .replace(/<\/h1>/g, '</h1>')
-      .replace(/<h2>/g, '<h2>')
-      .replace(/<\/h2>/g, '</h2>')
-      .replace(/<h3>/g, '<h3>')
-      .replace(/<\/h3>/g, '</h3>')
-      .replace(/<p>/g, '<p>')
-      .replace(/<\/p>/g, '</p>')
-      .replace(/<strong>/g, '<strong>')
-      .replace(/<\/strong>/g, '</strong>')
-      .replace(/<em>/g, '<em>')
-      .replace(/<\/em>/g, '</em>')
-      .replace(/<ul>/g, '<ul>')
-      .replace(/<\/ul>/g, '</ul>')
-      .replace(/<ol>/g, '<ol>')
-      .replace(/<\/ol>/g, '</ol>')
-      .replace(/<li>/g, '<li>')
-      .replace(/<\/li>/g, '</li>')
-      .replace(/<code>/g, '<code>')
-      .replace(/<\/code>/g, '</code>')
-      .replace(/<pre><code>/g, '<ac:structured-macro ac:name="code"><ac:plain-text-body><![CDATA[')
-      .replace(/<\/code><\/pre>/g, ']]></ac:plain-text-body></ac:structured-macro>');
-    
-    return storage;
+    // Delegate to htmlToConfluenceStorage for proper conversion including code blocks
+    return this.htmlToConfluenceStorage(html);
   }
 
   /**
@@ -591,6 +684,76 @@ class ConfluenceClient {
     });
   }
 
+  /**
+   * Detect language from text content and return appropriate labels
+   * @param {string} text - Text content to analyze
+   * @returns {object} Object with language-specific labels
+   */
+  detectLanguageLabels(text) {
+    const labels = {
+      includePage: 'Include Page',
+      sharedBlock: 'Shared Block',
+      includeSharedBlock: 'Include Shared Block',
+      fromPage: 'from page',
+      expandDetails: 'Expand Details'
+    };
+    
+    if (/[\u4e00-\u9fa5]/.test(text)) {
+      // Chinese
+      labels.includePage = '包含页面';
+      labels.sharedBlock = '共享块';
+      labels.includeSharedBlock = '包含共享块';
+      labels.fromPage = '来自页面';
+      labels.expandDetails = '展开详情';
+    } else if (/[\u3040-\u309f\u30a0-\u30ff]/.test(text)) {
+      // Japanese
+      labels.includePage = 'ページを含む';
+      labels.sharedBlock = '共有ブロック';
+      labels.includeSharedBlock = '共有ブロックを含む';
+      labels.fromPage = 'ページから';
+      labels.expandDetails = '詳細を表示';
+    } else if (/[\uac00-\ud7af]/.test(text)) {
+      // Korean
+      labels.includePage = '페이지 포함';
+      labels.sharedBlock = '공유 블록';
+      labels.includeSharedBlock = '공유 블록 포함';
+      labels.fromPage = '페이지에서';
+      labels.expandDetails = '상세 보기';
+    } else if (/[\u0400-\u04ff]/.test(text)) {
+      // Russian/Cyrillic
+      labels.includePage = 'Включить страницу';
+      labels.sharedBlock = 'Общий блок';
+      labels.includeSharedBlock = 'Включить общий блок';
+      labels.fromPage = 'со страницы';
+      labels.expandDetails = 'Подробнее';
+    } else if ((text.match(/[àâäéèêëïîôùûüÿœæç]/gi) || []).length >= 2) {
+      // French (requires at least 2 French-specific characters to avoid false positives)
+      labels.includePage = 'Inclure la page';
+      labels.sharedBlock = 'Bloc partagé';
+      labels.includeSharedBlock = 'Inclure le bloc partagé';
+      labels.fromPage = 'de la page';
+      labels.expandDetails = 'Détails';
+    } else if ((text.match(/[äöüß]/gi) || []).length >= 2) {
+      // German (requires at least 2 German-specific characters)
+      // Note: French is checked before German because French regex includes more characters
+      // that overlap with German (ä, ü). The threshold helps distinguish between them.
+      labels.includePage = 'Seite einbinden';
+      labels.sharedBlock = 'Gemeinsamer Block';
+      labels.includeSharedBlock = 'Gemeinsamen Block einbinden';
+      labels.fromPage = 'von Seite';
+      labels.expandDetails = 'Details';
+    } else if ((text.match(/[áéíóúñ¿¡]/gi) || []).length >= 2) {
+      // Spanish (requires at least 2 Spanish-specific characters)
+      labels.includePage = 'Incluir página';
+      labels.sharedBlock = 'Bloque compartido';
+      labels.includeSharedBlock = 'Incluir bloque compartido';
+      labels.fromPage = 'de la página';
+      labels.expandDetails = 'Detalles';
+    }
+    
+    return labels;
+  }
+
   /**
    * Convert Confluence storage format to markdown
    * @param {string} storage - Confluence storage format HTML
@@ -601,6 +764,9 @@ class ConfluenceClient {
     const attachmentsDir = options.attachmentsDir || 'attachments';
     let markdown = storage;
     
+    // Detect language from content
+    const labels = this.detectLanguageLabels(markdown);
+    
     // Remove table of contents macro
     markdown = markdown.replace(/<ac:structured-macro ac:name="toc"[^>]*\s*\/>/g, '');
     markdown = markdown.replace(/<ac:structured-macro ac:name="toc"[^>]*>[\s\S]*?<\/ac:structured-macro>/g, '');
@@ -625,20 +791,8 @@ class ConfluenceClient {
     });
     
     // Convert expand macro - extract content from rich-text-body
-    // Detect language based on content for the expand summary text
-    const detectExpandSummary = (text) => {
-      if (/[\u4e00-\u9fa5]/.test(text)) return '展开详情';      // Chinese
-      if (/[\u3040-\u309f\u30a0-\u30ff]/.test(text)) return '詳細を表示'; // Japanese
-      if (/[\uac00-\ud7af]/.test(text)) return '상세 보기';    // Korean
-      if (/[\u0400-\u04ff]/.test(text)) return 'Подробнее';    // Russian/Cyrillic
-      if (/[àâäéèêëïîôùûüÿœæç]/i.test(text)) return 'Détails'; // French
-      if (/[äöüß]/i.test(text)) return 'Details';              // German
-      if (/[áéíóúñ¿¡]/i.test(text)) return 'Detalles';         // Spanish
-      return 'Expand Details';  // Default: English
-    };
-    const expandSummary = detectExpandSummary(markdown);
     markdown = markdown.replace(/<ac:structured-macro ac:name="expand"[^>]*>[\s\S]*?<ac:rich-text-body>([\s\S]*?)<\/ac:rich-text-body>[\s\S]*?<\/ac:structured-macro>/g, (_, content) => {
-      return `\n<details>\n<summary>${expandSummary}</summary>\n\n${content}\n\n</details>\n`;
+      return `\n<details>\n<summary>${labels.expandDetails}</summary>\n\n${content}\n\n</details>\n`;
     });
     
     // Convert Confluence code macros to markdown
@@ -689,6 +843,58 @@ class ConfluenceClient {
       return tasks.length > 0 ? '\n' + tasks.join('\n') + '\n' : '';
     });
     
+    // Convert panel macro to markdown blockquote with title
+    markdown = markdown.replace(/<ac:structured-macro ac:name="panel"[^>]*>[\s\S]*?<ac:parameter ac:name="title">([^<]*)<\/ac:parameter>[\s\S]*?<ac:rich-text-body>([\s\S]*?)<\/ac:rich-text-body>[\s\S]*?<\/ac:structured-macro>/g, (_, title, content) => {
+      const cleanContent = this.htmlToMarkdown(content);
+      return `\n> **${title}**\n>\n${cleanContent.split('\n').map(line => line ? `> ${line}` : '>').join('\n')}\n`;
+    });
+    
+    // Convert include macro - extract page link and convert to markdown link
+    // Handle both with and without parameter name
+    markdown = markdown.replace(/<ac:structured-macro ac:name="include"[^>]*>[\s\S]*?<ac:parameter ac:name="">[\s\S]*?<ac:link>[\s\S]*?<ri:page\s+ri:space-key="([^"]+)"\s+ri:content-title="([^"]+)"[^>]*\/>[\s\S]*?<\/ac:link>[\s\S]*?<\/ac:parameter>[\s\S]*?<\/ac:structured-macro>/g, (_, spaceKey, title) => {
+      // Try to build a proper URL - if spaceKey starts with ~, it's a user space
+      if (spaceKey.startsWith('~')) {
+        const spacePath = `display/${spaceKey}/${encodeURIComponent(title)}`;
+        return `\n> 📄 **${labels.includePage}**: [${title}](https://${this.domain}/wiki/${spacePath})\n`;
+      } else {
+        // For non-user spaces, we cannot construct a valid link without the page ID.
+        // Document that manual correction is required.
+        return `\n> 📄 **${labels.includePage}**: [${title}](https://${this.domain}/wiki/spaces/${spaceKey}/pages/[PAGE_ID_HERE]) _(manual link correction required)_\n`;
+      }
+    });
+    
+    // Convert shared-block and include-shared-block macros - extract content
+    markdown = markdown.replace(/<ac:structured-macro ac:name="(shared-block|include-shared-block)"[^>]*>[\s\S]*?<ac:parameter ac:name="shared-block-key">([^<]*)<\/ac:parameter>[\s\S]*?<ac:rich-text-body>([\s\S]*?)<\/ac:rich-text-body>[\s\S]*?<\/ac:structured-macro>/g, (_, macroType, blockKey, content) => {
+      const cleanContent = this.htmlToMarkdown(content);
+      return `\n> **${labels.sharedBlock}: ${blockKey}**\n>\n${cleanContent.split('\n').map(line => line ? `> ${line}` : '>').join('\n')}\n`;
+    });
+    
+    // Convert include-shared-block with page parameter
+    markdown = markdown.replace(/<ac:structured-macro ac:name="include-shared-block"[^>]*>[\s\S]*?<ac:parameter ac:name="shared-block-key">([^<]*)<\/ac:parameter>[\s\S]*?<ac:parameter ac:name="page">[\s\S]*?<ac:link>[\s\S]*?<ri:page\s+ri:space-key="([^"]+)"\s+ri:content-title="([^"]+)"[^>]*\/>[\s\S]*?<\/ac:link>[\s\S]*?<\/ac:parameter>[\s\S]*?<\/ac:structured-macro>/g, (_, blockKey, spaceKey, pageTitle) => {
+      // The page ID is not available, so we cannot generate a valid link.
+      // Instead, document that the link needs manual correction.
+      return `\n> 📄 **${labels.includeSharedBlock}**: ${blockKey} (${labels.fromPage}: ${pageTitle} [link needs manual correction])\n`;
+    });
+    
+    // Convert view-file macro to file link
+    // Handle both orders: name first or height first
+    markdown = markdown.replace(/<ac:structured-macro ac:name="view-file"[^>]*>[\s\S]*?<ac:parameter ac:name="name">[\s\S]*?<ri:attachment\s+ri:filename="([^"]+)"[^>]*\/>[\s\S]*?<\/ac:parameter>[\s\S]*?<\/ac:structured-macro>/g, (_, filename) => {
+      return `\n📎 [${filename}](${attachmentsDir}/${filename})\n`;
+    });
+    
+    // Also handle view-file with height parameter (which might appear after name)
+    markdown = markdown.replace(/<ac:structured-macro ac:name="view-file"[^>]*>[\s\S]*?<ac:parameter ac:name="name">[\s\S]*?<ri:attachment\s+ri:filename="([^"]+)"[^>]*\/>[\s\S]*?<\/ac:parameter>[\s\S]*?<ac:parameter ac:name="height">([^<]*)<\/ac:parameter>[\s\S]*?<\/ac:structured-macro>/g, (_, filename, _height) => {
+      return `\n📎 [${filename}](${attachmentsDir}/${filename})\n`;
+    });
+    
+    // Remove layout macros but preserve content
+    markdown = markdown.replace(/<ac:layout>/g, '');
+    markdown = markdown.replace(/<\/ac:layout>/g, '');
+    markdown = markdown.replace(/<ac:layout-section[^>]*>/g, '');
+    markdown = markdown.replace(/<\/ac:layout-section>/g, '');
+    markdown = markdown.replace(/<ac:layout-cell[^>]*>/g, '');
+    markdown = markdown.replace(/<\/ac:layout-cell>/g, '');
+    
     // Remove other unhandled macros (replace with empty string for now)
     markdown = markdown.replace(/<ac:structured-macro[^>]*>[\s\S]*?<\/ac:structured-macro>/g, '');
     
@@ -985,7 +1191,7 @@ class ConfluenceClient {
    * Get page content for editing
    */
   async getPageForEdit(pageIdOrUrl) {
-    const pageId = this.extractPageId(pageIdOrUrl);
+    const pageId = await this.extractPageId(pageIdOrUrl);
     
     const response = await this.client.get(`/content/${pageId}`, {
       params: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "confluence-cli",
-  "version": "1.10.1",
+  "version": "1.11.1",
   "description": "A command-line interface for Atlassian Confluence with page creation and editing capabilities",
   "main": "index.js",
   "bin": {
@@ -32,6 +32,7 @@
   },
   "devDependencies": {
     "@types/node": "^20.10.0",
+    "axios-mock-adapter": "^2.1.0",
     "eslint": "^8.55.0",
     "jest": "^29.7.0"
   },

package/tests/confluence-client.test.js

@@ -1,4 +1,5 @@
 const ConfluenceClient = require('../lib/confluence-client');
+const MockAdapter = require('axios-mock-adapter');
 
 describe('ConfluenceClient', () => {
   let client;
@@ -63,19 +64,64 @@ describe('ConfluenceClient', () => {
   });
 
   describe('extractPageId', () => {
-    test('should return numeric page ID as is', () => {
-      expect(client.extractPageId('123456789')).toBe('123456789');
-      expect(client.extractPageId(123456789)).toBe(123456789);
+    test('should return numeric page ID as is', async () => {
+      expect(await client.extractPageId('123456789')).toBe('123456789');
+      expect(await client.extractPageId(123456789)).toBe(123456789);
     });
 
-    test('should extract page ID from URL with pageId parameter', () => {
+    test('should extract page ID from URL with pageId parameter', async () => {
       const url = 'https://test.atlassian.net/wiki/spaces/TEST/pages/123456789/Page+Title';
-      expect(client.extractPageId(url + '?pageId=987654321')).toBe('987654321');
+      expect(await client.extractPageId(url + '?pageId=987654321')).toBe('987654321');
     });
 
-    test('should throw error for display URLs', () => {
+    test('should resolve display URLs', async () => {
+      // Mock the API response for display URL resolution
+      const mock = new MockAdapter(client.client);
+
+      mock.onGet('/content').reply(200, {
+        results: [{
+          id: '12345',
+          title: 'Page Title',
+          _links: { webui: '/display/TEST/Page+Title' }
+        }]
+      });
+
       const displayUrl = 'https://test.atlassian.net/display/TEST/Page+Title';
-      expect(() => client.extractPageId(displayUrl)).toThrow('Display URLs not yet supported');
+      expect(await client.extractPageId(displayUrl)).toBe('12345');
+
+      mock.restore();
+    });
+
+    test('should resolve nested display URLs', async () => {
+      // Mock the API response for display URL resolution
+      const mock = new MockAdapter(client.client);
+
+      mock.onGet('/content').reply(200, {
+        results: [{
+          id: '67890',
+          title: 'Child Page',
+          _links: { webui: '/display/TEST/Parent/Child+Page' }
+        }]
+      });
+
+      const displayUrl = 'https://test.atlassian.net/display/TEST/Parent/Child+Page';
+      expect(await client.extractPageId(displayUrl)).toBe('67890');
+
+      mock.restore();
+    });
+
+    test('should throw error when display URL cannot be resolved', async () => {
+      const mock = new MockAdapter(client.client);
+
+      // Mock empty result
+      mock.onGet('/content').reply(200, {
+        results: []
+      });
+
+      const displayUrl = 'https://test.atlassian.net/display/TEST/NonExistentPage';
+      await expect(client.extractPageId(displayUrl)).rejects.toThrow(/Could not resolve page ID/);
+
+      mock.restore();
     });
   });
 
@@ -135,6 +181,23 @@ describe('ConfluenceClient', () => {
     });
   });
 
+  describe('markdownToNativeStorage', () => {
+    test('should act as an alias to htmlToConfluenceStorage via markdown render', () => {
+      const markdown = '# Native Storage Test';
+      const result = client.markdownToNativeStorage(markdown);
+
+      expect(result).toContain('<h1>Native Storage Test</h1>');
+    });
+
+    test('should handle code blocks correctly', () => {
+      const markdown = '```javascript\nconst a = 1;\n```';
+      const result = client.markdownToNativeStorage(markdown);
+
+      expect(result).toContain('<ac:structured-macro ac:name="code">');
+      expect(result).toContain('const a = 1;');
+    });
+  });
+
   describe('storageToMarkdown', () => {
     test('should convert Confluence storage format to markdown', () => {
       const storage = '<h1>Hello World</h1><p>This is a <strong>test</strong> page.</p>';