Fix: Match files_sub_directory as a prefix

Properly match `files_sub_directory` as a prefix instead of partial string matching.

Before

- If directory named "files_sub_directory" exists then all files
  matching "files_sub_directory" are selected. For example if
  files_sub_directory is set to "dir" and dir exists under the workspace
  directory then "mode/direct/file.yml" and "new_directory/f.yml" will
  be selected

After

- If directory named "files_sub_directory" exists then all files
  which names start with "files_sub_directory" are selected. For example if
  files_sub_directory is set to "dir" (or "./dir" or "dir/" or "./dir/")
  and dir exists under the workspace directory then "dir/file.yml" will be
  selected but not "new_directory/f.yml" and not "some/dir/f.yml"

CHANGELOG.md

@@ -12,6 +12,7 @@ Note: Can be used with `oxsecurity/megalinter@beta` in your GitHub Action mega-l
 - Fix version in URL in logs produced by reporters
 - Improve documentation for TAP_REPORTER
 - Fix flavors suggestions to ignore linters not relevant for such flavor ([#1746](https://github.com/oxsecurity/megalinter/issues/1746))
+- Fix: Properly match `files_sub_directory` as a prefix instead of partial string matching
 
 - Linter versions upgrades
   - [eslint-plugin-jsonc](https://ota-meshi.github.io/eslint-plugin-jsonc/) from 2.3.1 to **2.4.0** on 2022-08-16

megalinter/Linter.py

@@ -781,6 +781,7 @@ def collect_files(self, all_files):
             file_contains_regex=self.file_contains_regex,
             files_sub_directory=self.files_sub_directory,
             lint_all_other_linters_files=self.lint_all_other_linters_files,
+            prefix=self.workspace,
         )
         self.files_number = len(self.files)
         logging.debug(

megalinter/utils.py

@@ -72,6 +72,7 @@ def filter_files(
     file_contains_regex: Optional[Sequence[str]] = None,
     files_sub_directory: Optional[str] = None,
     lint_all_other_linters_files: bool = False,
+    prefix: Optional[str] = "",
 ) -> Sequence[str]:
     file_extensions = set(file_extensions)
     filter_regex_include_object = (
@@ -114,7 +115,7 @@ def filter_files(
         if filter_regex_exclude_object and filter_regex_exclude_object.search(file):
             continue
         # Skip if file is not in defined files_sub_directory
-        if files_sub_directory and files_sub_directory not in file:
+        if files_sub_directory and not os.path.relpath(file, prefix).startswith(os.path.normpath(files_sub_directory) + os.path.sep):
             continue
 
         # Skip according to file extension (only if lint_all_other_linter_files is false)