Update release asset naming

Author: satyakigh

.github/workflows/release.yml

@@ -95,11 +95,11 @@ jobs:
 
           TAG=${{ needs.version-and-tag.outputs.tag }}
           if [[ "$TAG" =~ -alpha$ ]]; then
-            FILE_NAME="${APP_NAME}-${VERSION}-alpha-glib2.28-linux-${{ matrix.arch }}-node${NODE_MAJOR}"
+            FILE_NAME="${APP_NAME}-${VERSION}-alpha-linuxglib2.28-${{ matrix.arch }}-node${NODE_MAJOR}"
           elif [[ "$TAG" =~ -beta$ ]]; then
-            FILE_NAME="${APP_NAME}-${VERSION}-beta-glib2.28-linux-${{ matrix.arch }}-node${NODE_MAJOR}"
+            FILE_NAME="${APP_NAME}-${VERSION}-beta-linuxglib2.28-${{ matrix.arch }}-node${NODE_MAJOR}"
           else
-            FILE_NAME="${APP_NAME}-${VERSION}-glib2.28-linux-${{ matrix.arch }}-node${NODE_MAJOR}"
+            FILE_NAME="${APP_NAME}-${VERSION}-linuxglib2.28-${{ matrix.arch }}-node${NODE_MAJOR}"
           fi
 
           ASSET_NAME=$(echo "$FILE_NAME" | tr '[:upper:]' '[:lower:]')

tools/generate-release-manifest.ts

@@ -71,10 +71,10 @@ function fetchReleases(): GHRelease[] {
     });
 }
 
-function parseTarget(filename: string): { platform: string; arch: string; nodejs?: string } | null {
+export function parseTarget(filename: string): { platform: string; arch: string; nodejs?: string } | null {
     // eslint-disable-next-line security/detect-unsafe-regex
-    const match = filename.match(/(darwin|linux|win32)-(x64|arm64)(?:-node(\d+))?/);
-    return match ? { platform: match[1], arch: match[2], nodejs: match[3] } : null;
+    const match = filename.match(/^cloudformation-languageserver-(.*)-(.*)-(x64|arm64)(?:-node(\d+))?\.zip$/);
+    return match ? { platform: match[2], arch: match[3], nodejs: match[4] } : null;
 }
 
 function generateManifest() {

tst/unit/tools/GenerateReleaseManifest.test.ts

@@ -1,5 +1,6 @@
 import { rcompare } from 'semver';
 import { describe, it, expect } from 'vitest';
+import { parseTarget } from '../../../tools/generate-release-manifest';
 
 describe('Generate Release Manifest', () => {
     describe('semver version sorting within environments', () => {
@@ -29,4 +30,54 @@ describe('Generate Release Manifest', () => {
             expect(sorted).toEqual(['v1.2.0', 'v1.1.0', 'v1.0.0']);
         });
     });
+
+    describe('parseTarget', () => {
+        it('should parse linux with glib postfix', () => {
+            expect(parseTarget('cloudformation-languageserver-1.2.0-alpha-linuxglib2.28-x64-node18.zip')).toEqual({
+                platform: 'linuxglib2.28',
+                arch: 'x64',
+                nodejs: '18',
+            });
+        });
+
+        it('should parse linux without postfix', () => {
+            expect(parseTarget('cloudformation-languageserver-1.2.0-alpha-linux-arm64-node22.zip')).toEqual({
+                platform: 'linux',
+                arch: 'arm64',
+                nodejs: '22',
+            });
+        });
+
+        it('should parse darwin', () => {
+            expect(parseTarget('cloudformation-languageserver-1.2.0-alpha-darwin-arm64-node22.zip')).toEqual({
+                platform: 'darwin',
+                arch: 'arm64',
+                nodejs: '22',
+            });
+        });
+
+        it('should parse win32', () => {
+            expect(parseTarget('cloudformation-languageserver-1.2.0-alpha-win32-x64-node22.zip')).toEqual({
+                platform: 'win32',
+                arch: 'x64',
+                nodejs: '22',
+            });
+        });
+
+        it('should parse without nodejs version', () => {
+            expect(parseTarget('cloudformation-languageserver-1.2.0-linux-x64.zip')).toEqual({
+                platform: 'linux',
+                arch: 'x64',
+                nodejs: undefined,
+            });
+        });
+
+        it('should return null for non-zip files', () => {
+            expect(parseTarget('cloudformation-languageserver-1.2.0-linux-x64.tar.gz')).toBeNull();
+        });
+
+        it('should return null for invalid architecture', () => {
+            expect(parseTarget('cloudformation-languageserver-1.2.0-linux-i386.zip')).toBeNull();
+        });
+    });
 });