Merge pull request #148 from IBM/feature/imd_log_lines

worksofliam · web-flow · commit f35419954515 · 2025-07-22T11:15:01.000-04:00
Enhance log message formatting with line numbers
diff --git a/cli/src/builders/imd.ts b/cli/src/builders/imd.ts
@@ -74,7 +74,7 @@ export class ImpactMarkdown {
           lines.push(
             `##### \`${relativePath}\``,
             ``,
-            ...fileLogs.map(log => `* ${LogEmoji[log.type] || `❔`} ${log.message}`),
+            ...fileLogs.map(log => `* ${LogEmoji[log.type] || `❔`}${log.line ? `line ${log.line}:` : ''} ${log.message}`),
             ``
           )
         }
@@ -110,7 +110,7 @@ export class ImpactMarkdown {
         warningMarkdown.push(
           `**\`${relativePath}\`**`,
           ``,
-          ...logs.filter(l => l.type === 'warning').map(log => `* [ ] ${LogEmoji[log.type] || `❔`} ${log.message}`),
+          ...logs.filter(l => l.type === 'warning').map(log => `* [ ] ${LogEmoji[log.type] || `❔`}${log.line ? `line ${log.line}:` : ''} ${log.message}`),
           ``
         )
       }

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ export class ImpactMarkdown {`
`74`	`74`	`lines.push(`
`75`	`75`	`##### \`${relativePath}\``,
`76`	`76`	``,
`77`		- ...fileLogs.map(log => `* ${LogEmoji[log.type] \|\| `❔`} ${log.message}`),
	`77`	+ ...fileLogs.map(log => `* ${LogEmoji[log.type] \|\| `❔`}${log.line ? `line ${log.line}:` : ''} ${log.message}`),
`78`	`78`	``
`79`	`79`	`)`
`80`	`80`	`}`
`@@ -110,7 +110,7 @@ export class ImpactMarkdown {`
`110`	`110`	`warningMarkdown.push(`
`111`	`111`	`\`${relativePath}\``,
`112`	`112`	``,
`113`		- ...logs.filter(l => l.type === 'warning').map(log => `* [ ] ${LogEmoji[log.type] \|\| `❔`} ${log.message}`),
	`113`	+ ...logs.filter(l => l.type === 'warning').map(log => `* [ ] ${LogEmoji[log.type] \|\| `❔`}${log.line ? `line ${log.line}:` : ''} ${log.message}`),
`114`	`114`	``
`115`	`115`	`)`
`116`	`116`	`}`