__tests__/util.test.ts

@@ -39,18 +39,6 @@ describe('util', () => {
         'loom',
       ]);
     });
-    it('handles globs with brace groups containing commas', () => {
-      assert.deepStrictEqual(parseInputFiles('./**/*.{exe,deb,tar.gz}\nfoo,bar'), [
-        './**/*.{exe,deb,tar.gz}',
-        'foo',
-        'bar',
-      ]);
-    });
-    it('handles single-line brace pattern correctly', () => {
-      assert.deepStrictEqual(parseInputFiles('./**/*.{exe,deb,tar.gz}'), [
-        './**/*.{exe,deb,tar.gz}',
-      ]);
-    });
   });
   describe('releaseBody', () => {
     it('uses input body', () => {
@@ -122,52 +110,6 @@ describe('util', () => {
         }),
       );
     });
-    it('falls back to body when body_path is missing', () => {
-      assert.equal(
-        releaseBody({
-          github_ref: '',
-          github_repository: '',
-          github_token: '',
-          input_body: 'fallback-body',
-          input_body_path: '__tests__/does-not-exist.txt',
-          input_draft: false,
-          input_prerelease: false,
-          input_files: [],
-          input_overwrite_files: undefined,
-          input_preserve_order: undefined,
-          input_name: undefined,
-          input_tag_name: undefined,
-          input_target_commitish: undefined,
-          input_discussion_category_name: undefined,
-          input_generate_release_notes: false,
-          input_make_latest: undefined,
-        }),
-        'fallback-body',
-      );
-    });
-    it('returns undefined when body_path is missing and body is not provided', () => {
-      assert.equal(
-        releaseBody({
-          github_ref: '',
-          github_repository: '',
-          github_token: '',
-          input_body: undefined,
-          input_body_path: '__tests__/does-not-exist.txt',
-          input_draft: false,
-          input_prerelease: false,
-          input_files: [],
-          input_overwrite_files: undefined,
-          input_preserve_order: undefined,
-          input_name: undefined,
-          input_tag_name: undefined,
-          input_target_commitish: undefined,
-          input_discussion_category_name: undefined,
-          input_generate_release_notes: false,
-          input_make_latest: undefined,
-        }),
-        undefined,
-      );
-    });
   });
   describe('parseConfig', () => {
     it('parses basic config', () => {
@@ -490,36 +432,3 @@ describe('util', () => {
     });
   });
 });
-
-describe('parseInputFiles edge cases', () => {
-  it('handles multiple brace groups on same line', () => {
-    assert.deepStrictEqual(parseInputFiles('./**/*.{exe,deb},./dist/**/*.{zip,tar.gz}'), [
-      './**/*.{exe,deb}',
-      './dist/**/*.{zip,tar.gz}',
-    ]);
-  });
-
-  it('handles nested braces', () => {
-    assert.deepStrictEqual(parseInputFiles('path/{a,{b,c}}/file.txt'), ['path/{a,{b,c}}/file.txt']);
-  });
-
-  it('handles empty comma-separated values', () => {
-    assert.deepStrictEqual(parseInputFiles('foo,,bar'), ['foo', 'bar']);
-  });
-
-  it('handles commas with spaces around braces', () => {
-    assert.deepStrictEqual(parseInputFiles(' ./**/*.{exe,deb} , file.txt '), [
-      './**/*.{exe,deb}',
-      'file.txt',
-    ]);
-  });
-
-  it('handles mixed newlines and commas with braces', () => {
-    assert.deepStrictEqual(parseInputFiles('file1.txt\n./**/*.{exe,deb},file2.txt\nfile3.txt'), [
-      'file1.txt',
-      './**/*.{exe,deb}',
-      'file2.txt',
-      'file3.txt',
-    ]);
-  });
-});

src/util.ts

@@ -35,53 +35,23 @@ export const uploadUrl = (url: string): string => {
 };
 
 export const releaseBody = (config: Config): string | undefined => {
-  if (config.input_body_path) {
+  return (
-    try {
+    (config.input_body_path && readFileSync(config.input_body_path).toString('utf8')) ||
-      const contents = readFileSync(config.input_body_path, 'utf8');
+    config.input_body
-      return contents;
+  );
-    } catch (err: any) {
-      console.warn(
-        `⚠️ Failed to read body_path "${config.input_body_path}" (${err?.code ?? 'ERR'}). Falling back to 'body' input.`,
-      );
-    }
-  }
-  return config.input_body;
 };
 
 type Env = { [key: string]: string | undefined };
 
-const smartSplit = (input: string): string[] => {
-  const result: string[] = [];
-  let current = '';
-  let braceDepth = 0;
-
-  for (const ch of input) {
-    if (ch === '{') {
-      braceDepth++;
-    }
-    if (ch === '}') {
-      braceDepth--;
-    }
-    if (ch === ',' && braceDepth === 0) {
-      if (current.trim()) {
-        result.push(current.trim());
-      }
-      current = '';
-    } else {
-      current += ch;
-    }
-  }
-  if (current.trim()) {
-    result.push(current.trim());
-  }
-  return result;
-};
-
 export const parseInputFiles = (files: string): string[] => {
-  return files
+  return files.split(/\r?\n/).reduce<string[]>(
-    .split(/\r?\n/)
+    (acc, line) =>
-    .flatMap((line) => smartSplit(line))
+      acc
-    .filter((pat) => pat.trim() !== '');
+        .concat(line.split(','))
+        .filter((pat) => pat)
+        .map((pat) => pat.trim()),
+    [],
+  );
 };
 
 export const parseConfig = (env: Env): Config => {