feat: use unique comment IDs for each command/schema combination

Each job now creates a separate PR comment identified by command and schema file.
Comments use HTML comment IDs to prevent collision between different sqldef commands.

Author: gfx

dist/index.js

@@ -249,7 +249,7 @@ async function runSqldef(binaryPath, config) {
     // Return combined output for successful runs
     return output + stderr;
 }
-async function createComment(body) {
+async function createComment(body, command, schemaFile) {
     const context = github.context;
     if (context.eventName !== "pull_request") {
         core.warning("Not a pull request event, skipping comment");
@@ -266,11 +266,19 @@ async function createComment(body) {
         repo: context.repo.repo,
         issue_number: context.payload.pull_request.number,
     });
-    const title = "SQLDef Migration Preview";
-    const previousComment = comments.find((comment) => comment.user?.type === "Bot" && comment.body?.includes(title));
+    // Create a unique ID for this command/schema combination
+    const commentId = `${command}-${schemaFile}`;
+    const htmlCommentId = `<!-- sqldef-preview-action-id: ${commentId} -->`;
+    // Find previous comment by searching for the HTML comment ID
+    const previousComment = comments.find((comment) => comment.user?.type === "Bot" && comment.body?.includes(htmlCommentId));
+    const title = `SQLDef Migration Preview (${command})`;
+    const subtitle = `Schema file: \`${schemaFile}\``;
     const commentBody = `
+${htmlCommentId}
 ## ${title}
 
+${subtitle}
+
 ~~~sql
 ${body}
 ~~~
@@ -367,14 +375,14 @@ async function run() {
                 core.info(output);
                 // Only create comment for actual PR events
                 if (context.eventName === "pull_request" && !baselineSchemaFile) {
-                    await createComment(output);
+                    await createComment(output, command, schemaFile);
                 }
             }
             else {
                 core.info("No schema changes detected");
                 // Only create comment for actual PR events
                 if (context.eventName === "pull_request" && !baselineSchemaFile) {
-                    await createComment("No schema changes detected.");
+                    await createComment("No schema changes detected.", command, schemaFile);
                 }
             }
             if (!baselineSchemaFile && actualBaselineFile && actualBaselineFile !== "" && fs.existsSync(actualBaselineFile)) {

dist/index.js.map

@@ -234,7 +234,7 @@ async function runSqldef(binaryPath: string, config: CommandConfig): Promise<str
     return output + stderr;
 }
 
-async function createComment(body: string): Promise<void> {
+async function createComment(body: string, command: string, schemaFile: string): Promise<void> {
     const context = github.context;
 
     if (context.eventName !== "pull_request") {
@@ -256,13 +256,22 @@ async function createComment(body: string): Promise<void> {
         issue_number: context.payload.pull_request!.number,
     });
 
-    const title = "SQLDef Migration Preview";
+    // Create a unique ID for this command/schema combination
+    const commentId = `${command}-${schemaFile}`;
+    const htmlCommentId = `<!-- sqldef-preview-action-id: ${commentId} -->`;
 
-    const previousComment = comments.find((comment) => comment.user?.type === "Bot" && comment.body?.includes(title));
+    // Find previous comment by searching for the HTML comment ID
+    const previousComment = comments.find((comment) => comment.user?.type === "Bot" && comment.body?.includes(htmlCommentId));
+
+    const title = `SQLDef Migration Preview (${command})`;
+    const subtitle = `Schema file: \`${schemaFile}\``;
 
     const commentBody = `
+${htmlCommentId}
 ## ${title}
 
+${subtitle}
+
 ~~~sql
 ${body}
 ~~~
@@ -369,13 +378,13 @@ async function run(): Promise<void> {
                 core.info(output);
                 // Only create comment for actual PR events
                 if (context.eventName === "pull_request" && !baselineSchemaFile) {
-                    await createComment(output);
+                    await createComment(output, command, schemaFile);
                 }
             } else {
                 core.info("No schema changes detected");
                 // Only create comment for actual PR events
                 if (context.eventName === "pull_request" && !baselineSchemaFile) {
-                    await createComment("No schema changes detected.");
+                    await createComment("No schema changes detected.", command, schemaFile);
                 }
             }