fix(issue-sync): view on crowdin url (#11)

Author: ReenigneArcher

src/sync-crowdin-issues.js

@@ -392,11 +392,14 @@ function buildIssueBody(crowdinIssue, projectId, projectSlug) {
 
   // Build the deep-link URL. If we have the slug, link straight to the string
   // in the Crowdin editor for that language; otherwise fall back to the project page.
-  // The numeric project ID is never used in URLs as it does not resolve on crowdin.com.
+  // URL format: /editor/{slug}/{fileId}/en-{normalizedLang}?view=comfortable#{stringId}
+  // The language segment strips hyphens and lowercases (e.g. "zh-CN" → "en-zhcn").
   const stringId = crowdinIssue.string?.id;
+  const fileId = crowdinIssue.string?.file?.id;
   let crowdinUrl;
-  if (projectSlug && crowdinIssue.languageId && stringId) {
-    crowdinUrl = `https://crowdin.com/editor/${projectSlug}/${crowdinIssue.languageId}#${stringId}`;
+  if (projectSlug && crowdinIssue.languageId && fileId && stringId) {
+    const normalizedLang = crowdinIssue.languageId.replaceAll(/[-_]/g, '').toLowerCase();
+    crowdinUrl = `https://crowdin.com/editor/${projectSlug}/${fileId}/en-${normalizedLang}?view=comfortable#${stringId}`;
   } else if (projectSlug) {
     crowdinUrl = `https://crowdin.com/project/${projectSlug}`;
   } else {

tests/sync-crowdin-issues.test.js

@@ -557,10 +557,23 @@ describe('buildIssueBody', () => {
     expect(body).toContain('https://crowdin.com/project/my-project');
   });
 
-  it('links directly to the string in the editor when slug and string.id are present', () => {
+  it('links directly to the string in the editor when slug, file.id and string.id are present', () => {
+    const issue = { ...baseIssue, languageId: 'fr', string: { text: 'Source', id: 999, file: { id: 5842 } } };
+    const body = buildIssueBody(issue, projectId, 'my-project');
+    expect(body).toContain('https://crowdin.com/editor/my-project/5842/en-fr?view=comfortable#999');
+  });
+
+  it('links directly to the editor with a compound language code normalized (zh-CN → en-zhcn)', () => {
+    const issue = { ...baseIssue, languageId: 'zh-CN', string: { text: 'Source', id: 91860, file: { id: 5842 } } };
+    const body = buildIssueBody(issue, projectId, 'my-project');
+    expect(body).toContain('https://crowdin.com/editor/my-project/5842/en-zhcn?view=comfortable#91860');
+  });
+
+  it('falls back to project page when file.id is absent', () => {
     const issue = { ...baseIssue, languageId: 'fr', string: { text: 'Source', id: 999 } };
     const body = buildIssueBody(issue, projectId, 'my-project');
-    expect(body).toContain('https://crowdin.com/editor/my-project/fr#999');
+    expect(body).toContain('https://crowdin.com/project/my-project');
+    expect(body).not.toContain('/editor/');
   });
 
   it('includes the File row when string.file.path is present', () => {