fix: resolve lint and typecheck issues

- Remove unused lineBreak import from heading.test.ts
- Remove unused measureDepth from skip-flags-regression.spec.ts
- Fix prettier formatting in fuzz.test.ts
- Update compact.ts Heading serialization to use inline children
- Update compact tests and CBlock type for new heading format

Author: ggazzo

packages/message-parser/src/compact.ts

@@ -68,7 +68,7 @@ export type CInlineNode =
 
 export type CBlock =
 	| { t: 'p'; c: CInline[] }
-	| { t: 'h'; l: 1 | 2 | 3 | 4; r: Span }
+	| { t: 'h'; l: 1 | 2 | 3 | 4; c: CInline[]; r?: Span }
 	| { t: '```'; l?: string; r: Span }
 	| { t: '>'; c: CBlock[] }
 	| { t: 'q'; c: CBlock[] }
@@ -182,7 +182,7 @@ function expandBlock(
 		case 'p':
 			return { type: 'PARAGRAPH', value: block.c.map((c) => expandInline(msg, c)) } as Paragraph;
 		case 'h':
-			return { type: 'HEADING', level: block.l, value: [makePlain(msg, block.r)] } as Heading;
+			return { type: 'HEADING', level: block.l, value: block.c.map((c) => expandInline(msg, c)) } as Heading;
 		case '```': {
 			const content = msg.slice(block.r[0], block.r[1]);
 			const lines: CodeLine[] = content.split('\n').map((line) => ({
@@ -446,9 +446,8 @@ function compactBlock(ctx: Ctx, node: Paragraph | Root[number]): CBlock {
 			const heading = node as Heading;
 			advance(ctx, heading.level); // #, ##, ###, ####
 			advance(ctx, 1); // space
-			const text = heading.value.map(textOf).join('');
-			const r = spanFor(ctx, text);
-			return { t: 'h', l: heading.level, r };
+			const c = heading.value.map((ch) => compactInline(ctx, ch));
+			return { t: 'h', l: heading.level, c };
 		}
 
 		case 'CODE': {

packages/message-parser/tests/compact.test.ts

@@ -129,7 +129,7 @@ describe('expand (compact → verbose)', () => {
 	});
 
 	it('heading', () => {
-		expectExpand('## Title', [{ t: 'h', l: 2, r: [3, 8] }], [
+		expectExpand('## Title', [{ t: 'h', l: 2, c: [[3, 8]] }], [
 			{ type: 'HEADING', level: 2, value: [{ type: 'PLAIN_TEXT', value: 'Title' }] },
 		] as unknown as Root);
 	});
@@ -202,7 +202,7 @@ describe('compactify (verbose → compact)', () => {
 	it('heading', () => {
 		const msg = '## Title';
 		const compact = compactify(parse(msg), msg);
-		expect(compact).toEqual([{ t: 'h', l: 2, r: [3, 8] }]);
+		expect(compact).toEqual([{ t: 'h', l: 2, c: [[3, 8]] }]);
 	});
 });
 

packages/message-parser/tests/fuzz.test.ts

@@ -2,21 +2,13 @@ import fc from 'fast-check';
 
 import { parse } from '../src';
 
-const plainTextArbitrary = fc
-	.stringMatching(/[a-zA-Z0-9 ]{1,20}/)
-	.filter((value) => value.length > 0);
+const plainTextArbitrary = fc.stringMatching(/[a-zA-Z0-9 ]{1,20}/).filter((value) => value.length > 0);
 
-const timestampArbitrary = fc
-	.integer({ min: 1_000_000_000, max: 2_000_000_000 })
-	.map((value) => `<t:${value}>`);
+const timestampArbitrary = fc.integer({ min: 1_000_000_000, max: 2_000_000_000 }).map((value) => `<t:${value}>`);
 
-const relativeTimestampArbitrary = fc
-	.integer({ min: 1_000_000_000, max: 2_000_000_000 })
-	.map((value) => `<t:${value}:R>`);
+const relativeTimestampArbitrary = fc.integer({ min: 1_000_000_000, max: 2_000_000_000 }).map((value) => `<t:${value}:R>`);
 
-const mentionArbitrary = fc
-	.stringMatching(/[a-z0-9._-]{1,12}/)
-	.map((value) => `@${value}`);
+const mentionArbitrary = fc.stringMatching(/[a-z0-9._-]{1,12}/).map((value) => `@${value}`);
 
 const boldArbitrary = plainTextArbitrary.map((value) => `*${value}*`);
 

packages/message-parser/tests/heading.test.ts

@@ -1,5 +1,5 @@
 import { parse } from '../src';
-import { emoji, heading, lineBreak, link, mentionChannel, paragraph, plain } from './helpers';
+import { emoji, heading, link, mentionChannel, paragraph, plain } from './helpers';
 
 test.each([
 	['# h1', [heading([plain('h1')], 1)]],
@@ -47,7 +47,6 @@ test.each([
 	['# :newspaper: Headline', [heading([emoji('newspaper'), plain(' Headline')], 1)]],
 	['## Hello :smile:', [heading([plain('Hello '), emoji('smile')], 2)]],
 	['### :smile: text :newspaper:', [heading([emoji('smile'), plain(' text '), emoji('newspaper')], 3)]],
-
 ])('parses %p', (input, output) => {
 	expect(parse(input)).toEqual(output);
 });

packages/message-parser/tests/joinEmoji.test.ts

@@ -1,32 +1,25 @@
-import { plain, emoji, reducePlainTexts } from '../src/utils';
-
-describe('joinEmoji behavior through reducePlainTexts', () => {
-	it('keeps emoji when alone', () => {
-		const result = reducePlainTexts([emoji('smile')]);
-		expect(result[0].type).toBe('EMOJI');
-	});
-
-	it('merges consecutive plain texts', () => {
-		const result = reducePlainTexts([
-			plain('hello '),
-			plain('world'),
-		]);
-
-		expect(result).toHaveLength(1);
-		expect(result[0]).toMatchObject({ type: 'PLAIN_TEXT', value: 'hello world' });
-	});
-
-	it('converts emoji with plain text neighbors to shortCode and merges', () => {
-		const result = reducePlainTexts([
-			plain('hello'),
-			emoji('smile'),
-			plain('world'),
-		]);
-        
-		expect(result).toHaveLength(1);
-		expect(result[0]).toMatchObject({
-			type: 'PLAIN_TEXT',
-			value: 'hello:smile:world',
-		});
-	});
-});
+import { plain, emoji, reducePlainTexts } from '../src/utils';
+
+describe('joinEmoji behavior through reducePlainTexts', () => {
+	it('keeps emoji when alone', () => {
+		const result = reducePlainTexts([emoji('smile')]);
+		expect(result[0].type).toBe('EMOJI');
+	});
+
+	it('merges consecutive plain texts', () => {
+		const result = reducePlainTexts([plain('hello '), plain('world')]);
+
+		expect(result).toHaveLength(1);
+		expect(result[0]).toMatchObject({ type: 'PLAIN_TEXT', value: 'hello world' });
+	});
+
+	it('converts emoji with plain text neighbors to shortCode and merges', () => {
+		const result = reducePlainTexts([plain('hello'), emoji('smile'), plain('world')]);
+
+		expect(result).toHaveLength(1);
+		expect(result[0]).toMatchObject({
+			type: 'PLAIN_TEXT',
+			value: 'hello:smile:world',
+		});
+	});
+});

packages/message-parser/tests/skip-flags-regression.spec.ts

@@ -1,13 +1,6 @@
 import { parse } from '../src';
 
 describe('Skip Flags Regression (Complexity Audit)', () => {
-	const measureDepth = (depth: number) => {
-		const input = `${'*'.repeat(depth)}text${'*'.repeat(depth)}`;
-		const start = performance.now();
-		parse(input);
-		return performance.now() - start;
-	};
-
 	it('should parse nested formatting across multiple depths without throwing', () => {
 		for (let d = 1; d <= 50; d++) {
 			const input = `${'*'.repeat(d)}text${'*'.repeat(d)}`;