Fix list spacing in display and HTML generation (v1.0.2)

htmlToText: strip <p><br></p> spacers around lists, process <ul> as
block to prevent source whitespace creating extra blank lines.
markdownToHtml: skip spacer paragraphs before/after list blocks.
Also fix two pre-existing typecheck errors in client.ts.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: waynemsmith

package.json

@@ -1,6 +1,6 @@
 {
   "name": "fizzy-cli",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "CLI for Fizzy kanban board management",
   "type": "module",
   "main": "src/index.ts",

src/client.ts

@@ -85,7 +85,7 @@ export class FizzyClient {
     let nextUrl: string | null = path.startsWith('http') ? path : `${this.config.baseUrl}${path}`;
 
     while (nextUrl) {
-      const response = await this.getPaginated<T>(nextUrl);
+      const response: PaginatedResponse<T> = await this.getPaginated<T>(nextUrl);
       results.push(...response.data);
       nextUrl = response.nextUrl;
     }
@@ -197,7 +197,7 @@ export class FizzyClient {
     const response = await fetch(url, {
       method: 'PUT',
       headers,
-      body: file,
+      body: file as unknown as BodyInit,
     });
 
     if (!response.ok) {

src/utils/html.ts

@@ -10,6 +10,10 @@ export function htmlToText(html: string | null | undefined): string {
 
   let text = html;
 
+  // Strip <p><br></p> spacers immediately before and after lists
+  text = text.replace(/<p><br><\/p>\s*(?=<[uo]l)/gi, '');
+  text = text.replace(/(<\/[uo]l>)\s*<p><br><\/p>/gi, '$1');
+
   // Add newlines before/after block elements
   text = text.replace(/<h[1-6][^>]*>/gi, '\n');
   text = text.replace(/<\/h[1-6]>/gi, '\n');
@@ -21,17 +25,20 @@ export function htmlToText(html: string | null | undefined): string {
   // Handle ordered lists with numbers
   text = text.replace(/<ol[^>]*>([\s\S]*?)<\/ol>/gi, (match, content) => {
     const items = content.match(/<li[^>]*>([\s\S]*?)<\/li>/gi) || [];
-    return '\n' + items.map((item: string, i: number) => {
+    return items.map((item: string, i: number) => {
       const inner = item.replace(/<li[^>]*>/i, '').replace(/<\/li>/i, '');
       return `  ${i + 1}. ${inner}`;
     }).join('\n') + '\n';
   });
 
-  // Unordered lists
-  text = text.replace(/<ul[^>]*>/gi, '\n');
-  text = text.replace(/<\/ul>/gi, '');
-  text = text.replace(/<li[^>]*>/gi, '  * ');
-  text = text.replace(/<\/li>/gi, '\n');
+  // Unordered lists — process as a block to avoid extra blank lines
+  text = text.replace(/<ul[^>]*>([\s\S]*?)<\/ul>/gi, (match, content) => {
+    const items = content.match(/<li[^>]*>([\s\S]*?)<\/li>/gi) || [];
+    return items.map((item: string) => {
+      const inner = item.replace(/<li[^>]*>/i, '').replace(/<\/li>/i, '');
+      return `  * ${inner}`;
+    }).join('\n') + '\n';
+  });
 
   // Inline formatting - keep content
   text = text.replace(/<strong[^>]*>([\s\S]*?)<\/strong>/gi, '$1');

src/utils/markdown.ts

@@ -278,8 +278,10 @@ function convertParagraphs(text: string): string {
       const current = htmlParts[i];
       const next = htmlParts[i + 1];
 
-      // Skip spacing around <hr> elements
-      if (!current.startsWith('<hr') && !next.startsWith('<hr')) {
+      // Skip spacing around <hr> elements and before/after lists
+      if (!current.startsWith('<hr') && !next.startsWith('<hr') &&
+          !next.startsWith('<ul') && !next.startsWith('<ol') &&
+          !current.startsWith('<ul') && !current.startsWith('<ol')) {
         result.push('<p><br></p>');
       }
     }

tests/html.test.ts

@@ -30,6 +30,24 @@ describe('htmlToText', () => {
     expect(result).toContain('* Item 2');
   });
 
+  test('no blank line between heading and list', () => {
+    const html = '<p><b><strong>Known gaps</strong></b></p><p><br></p><ul><li>Item 1</li><li>Item 2</li></ul>';
+    const result = htmlToText(html);
+    expect(result).toBe('Known gaps\n * Item 1\n * Item 2');
+  });
+
+  test('no blank lines between ul items with source newlines', () => {
+    const html = '<ul>\n<li>A</li>\n<li>B</li>\n</ul>';
+    const result = htmlToText(html);
+    expect(result).toBe('* A\n * B');
+  });
+
+  test('strips spacer after closing list tag', () => {
+    const html = '<ul><li>X</li></ul><p><br></p><p>Next</p>';
+    const result = htmlToText(html);
+    expect(result).toBe('* X\n\nNext');  // one blank line between list and next paragraph
+  });
+
   test('handles ordered lists with numbers', () => {
     const html = '<ol><li>First</li><li>Second</li></ol>';
     const result = htmlToText(html);

tests/markdown.test.ts

@@ -172,6 +172,14 @@ describe('markdownToHtml', () => {
     expect(result).toContain('<p>Line two</p>');
   });
 
+  test('no spacer paragraph before or after lists', () => {
+    const md = 'Heading:\n\n- Item 1\n- Item 2\n\nNext paragraph';
+    const result = markdownToHtml(md);
+    // No <p><br></p> between heading and list or between list and next para
+    expect(result).not.toMatch(/<p><br><\/p><ul>/);
+    expect(result).not.toMatch(/<\/ul><p><br><\/p>/);
+  });
+
   test('blank lines between list items produce one list', () => {
     const md = '- Item 1\n\n- Item 2\n\n- Item 3';
     const result = markdownToHtml(md);