koszewscy/setup-github-release
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor: Extract filename matching logic into a separate function

Browse Source
This commit is contained in:
Sławek Koszewski
2026-04-06 12:18:19 +02:00
parent dfa641afd4
commit 3a75adedc6
1 changed files with 21 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
src/core/matcher.ts
View File

@@ -59,6 +59,24 @@ function getExtPattern(fileType: string | undefined, system: string): string {
return normalizeCustomExtensionPattern(fileType || ''); return normalizeCustomExtensionPattern(fileType || '');
} }
function matchFilenameString(re: string, pi: PlatformInfo, extRe: string): string {
const hasSystem = re.includes('{{SYSTEM}}');
const hasArch = re.includes('{{ARCH}}');
const hasExt = re.includes('{{EXT_PATTERN}}');
const hasEnd = re.endsWith('$');
const finalRe = (!hasSystem && !hasArch && !hasExt && !hasEnd)
? `${re}.*{{SYSTEM}}[_-]{{ARCH}}.*{{EXT_PATTERN}}$`
: (hasSystem && hasArch && !hasExt && !hasEnd)
? `${re}.*{{EXT_PATTERN}}$`
: re;
return finalRe
.replace(/{{SYSTEM}}/g, pi.systemPattern)
.replace(/{{ARCH}}/g, pi.archPattern)
.replace(/{{EXT_PATTERN}}/g, extRe);
}
export function getMatchingAsset(assets: any[], platform: PlatformInfo, options: MatchOptions): any { export function getMatchingAsset(assets: any[], platform: PlatformInfo, options: MatchOptions): any {
const { fileName, fileType } = options; const { fileName, fileType } = options;
const extPattern = getExtPattern(fileType, platform.system); const extPattern = getExtPattern(fileType, platform.system);
@@ -77,27 +95,11 @@ export function getMatchingAsset(assets: any[], platform: PlatformInfo, options:
return matchingAssets[0]; return matchingAssets[0];
} else if (fileName.startsWith('~')) { } else if (fileName.startsWith('~')) {
// Rule 3: Regex matching rule // Rule 3: Regex matching rule
let pattern = fileName.substring(1); const pattern = matchFilenameString(fileName.substring(1), platform, extPattern);
const hasSystem = pattern.includes('{{SYSTEM}}'); const regex = new RegExp(pattern, 'i');
const hasArch = pattern.includes('{{ARCH}}');
const hasExt = pattern.includes('{{EXT_PATTERN}}');
const hasEnd = pattern.endsWith('$');
if (!hasSystem && !hasArch && !hasExt && !hasEnd) {
pattern += `.*{{SYSTEM}}[_-]{{ARCH}}.*{{EXT_PATTERN}}$`;
} else if (hasSystem && hasArch && !hasExt && !hasEnd) {
pattern += `.*{{EXT_PATTERN}}$`;
}
const finalPattern = pattern
.replace(/{{SYSTEM}}/g, platform.systemPattern)
.replace(/{{ARCH}}/g, platform.archPattern)
.replace(/{{EXT_PATTERN}}/g, extPattern);
const regex = new RegExp(finalPattern, 'i');
const matchingAssets = assets.filter((a: any) => regex.test(a.name)); const matchingAssets = assets.filter((a: any) => regex.test(a.name));
if (matchingAssets.length === 0) { if (matchingAssets.length === 0) {
throw new Error(`No assets matched the regex: ${finalPattern}`); throw new Error(`No assets matched the regex: ${pattern}`);
} }
if (matchingAssets.length > 1) { if (matchingAssets.length > 1) {
throw new Error(`Multiple assets matched the criteria: ${matchingAssets.map((a: any) => a.name).join(', ')}`); throw new Error(`Multiple assets matched the criteria: ${matchingAssets.map((a: any) => a.name).join(', ')}`);