fix: matching all nested files with a directory name (#1197)

Co-authored-by: GitHub Action <[email protected]>

Author: jackton1

.github/workflows/test.yml

@@ -508,6 +508,16 @@ jobs:
           echo '${{ toJSON(steps.changed-files-since-last-remote-commit.outputs) }}'
         shell:
           bash
+      - name: Run changed-files with dir name
+        id: changed-files-dir-name
+        uses: ./
+        with:
+          files: .github/workflows
+      - name: Show output
+        run: |
+          echo '${{ toJSON(steps.changed-files-dir-name.outputs) }}'
+        shell:
+          bash
       - name: Run changed-files with write_output_files
         id: changed-files-write-output-files
         uses: ./

dist/index.js

@@ -265,10 +265,9 @@ export async function run(): Promise<void> {
   core.debug(`All other changed files: ${allOtherChangedFiles}`)
 
   const otherChangedFiles = allOtherChangedFiles
-    .split(inputs.filesSeparator)
+    .split(inputs.separator)
     .filter(
-      filePath =>
-        !allChangedFiles.split(inputs.filesSeparator).includes(filePath)
+      filePath => !allChangedFiles.split(inputs.separator).includes(filePath)
     )
 
   const onlyChanged =
@@ -282,7 +281,7 @@ export async function run(): Promise<void> {
 
   await setOutput({
     key: 'other_changed_files',
-    value: otherChangedFiles.join(inputs.filesSeparator),
+    value: otherChangedFiles.join(inputs.separator),
     inputs
   })
 
@@ -318,10 +317,9 @@ export async function run(): Promise<void> {
   })
 
   const otherModifiedFiles = allOtherModifiedFiles
-    .split(inputs.filesSeparator)
+    .split(inputs.separator)
     .filter(
-      filePath =>
-        !allModifiedFiles.split(inputs.filesSeparator).includes(filePath)
+      filePath => !allModifiedFiles.split(inputs.separator).includes(filePath)
     )
 
   const onlyModified =
@@ -335,7 +333,7 @@ export async function run(): Promise<void> {
 
   await setOutput({
     key: 'other_modified_files',
-    value: otherModifiedFiles.join(inputs.filesSeparator),
+    value: otherModifiedFiles.join(inputs.separator),
     inputs
   })
 
@@ -371,9 +369,9 @@ export async function run(): Promise<void> {
   })
 
   const otherDeletedFiles = allOtherDeletedFiles
-    .split(inputs.filesSeparator)
+    .split(inputs.separator)
     .filter(
-      filePath => !deletedFiles.split(inputs.filesSeparator).includes(filePath)
+      filePath => !deletedFiles.split(inputs.separator).includes(filePath)
     )
 
   const onlyDeleted = otherDeletedFiles.length === 0 && deletedFiles.length > 0
@@ -386,7 +384,7 @@ export async function run(): Promise<void> {
 
   await setOutput({
     key: 'other_deleted_files',
-    value: otherDeletedFiles.join(inputs.filesSeparator),
+    value: otherDeletedFiles.join(inputs.separator),
     inputs
   })
 

dist/index.js.map

@@ -757,7 +757,23 @@ export const getFilePatterns = async ({
 
   core.debug(`file patterns: ${filePatterns}`)
 
-  return filePatterns.trim().split('\n').filter(Boolean)
+  return filePatterns
+    .trim()
+    .split('\n')
+    .filter(Boolean)
+    .map(pattern => {
+      if (pattern.endsWith('/')) {
+        return `${pattern}**`
+      } else {
+        const pathParts = pattern.split('/')
+        const lastPart = pathParts[pathParts.length - 1]
+        if (!lastPart.includes('.')) {
+          return `${pattern}/**`
+        } else {
+          return pattern
+        }
+      }
+    })
 }
 
 export const setOutput = async ({