bugfix-314-error-merging-release-branch-after-successful-deployment: Fix commit URL generation to properly encode owner and repository names. Added tests to verify URL encoding for special characters in commit links.

Author: efraespada

src/utils/__tests__/comment_watermark.test.ts

@@ -22,4 +22,19 @@ describe('comment_watermark', () => {
     expect(w).toContain('github.com/o/r/commit/abc123');
     expect(w).toContain('This will update automatically on new commits');
   });
+
+  it('URL-encodes owner and repo in commit link when they contain special characters', () => {
+    const w = getCommentWatermark({
+      commitSha: 'abc123',
+      owner: 'my.org',
+      repo: 'my-repo',
+    });
+    expect(w).toContain('github.com/my.org/my-repo/commit/abc123');
+    const w2 = getCommentWatermark({
+      commitSha: 'def456',
+      owner: 'org with spaces',
+      repo: 'repo',
+    });
+    expect(w2).toContain('github.com/org%20with%20spaces/repo/commit/def456');
+  });
 });

src/utils/comment_watermark.ts

@@ -15,7 +15,7 @@ export interface BugbotWatermarkOptions {
 }
 
 function commitUrl(owner: string, repo: string, sha: string): string {
-    return `https://github.com/${owner}/${repo}/commit/${sha}`;
+    return `https://github.com/${encodeURIComponent(owner)}/${encodeURIComponent(repo)}/commit/${sha}`;
 }
 
 export function getCommentWatermark(options?: BugbotWatermarkOptions): string {