fix: raise CodeBundleError for a missing include on the copy_style="none" path

`ls_relative_files` is the third site in `_code_bundle/_utils.py` that rejects an
`Environment.include` entry, and the only one still raising a bare `ValueError`.
It is reached from `build_code_bundle(copy_style="none", additional_files=...)`
via `build_code_bundle_from_relative_paths` -> `list_relative_files_to_bundle`,
so the path it complains about came straight from the user's `include=`.

Same reasoning as the two sites already converted in this PR: an include that
matches no file, directory or glob is the user's configuration, not an SDK bug,
so it should be a typed user error that crash reporting filters out instead of a
raw `ValueError` reported as a crash. The message is also reworded from
"File ... is not a valid file, directory, or glob pattern" to name the concept
the user actually typed ("include path ..."), matching its sibling in `ls_files`.

fixes FLYTE-SDK-5M

Signed-off-by: Haytham Abuelfutuh <haytham@afutuh.com>

Author: EngHabu

src/flyte/_code_bundle/_utils.py

@@ -231,7 +231,13 @@ def ls_relative_files(relative_paths: list[str], source_path: pathlib.Path) -> t
                 # Filter out directories from glob results
                 all_files.extend([str(f) for f in glob_files if pathlib.Path(f).is_file()])
             else:
-                raise ValueError(f"File {path} is not a valid file, directory, or glob pattern")
+                from flyte.errors import CodeBundleError
+
+                # Same user error as the `additional_files` branch of `ls_files` above,
+                # reached by the `copy_style="none"` bundle path: the entry came from the
+                # user's `Environment.include`, so a path that matches nothing is their
+                # configuration, not an SDK bug (FLYTE-SDK-5M).
+                raise CodeBundleError(f"include path {str(path)!r} is not a file, directory, or glob pattern.")
 
     all_files.sort()
     abs_source = os.path.abspath(str(source_path))

tests/flyte/code_bundle/test_build_code_bundle.py

@@ -441,6 +441,28 @@ async def test_build_code_bundle_include_outside_source_raises():
             )
 
 
+@pytest.mark.asyncio
+async def test_build_code_bundle_copy_style_none_missing_include_raises():
+    """
+    A ``copy_style='none'`` include that matches nothing must fail with the same
+    typed user error as the ``copy_style='all'`` path. This route reaches
+    ``ls_relative_files`` instead of ``ls_files``, and used to raise a bare
+    ``ValueError`` that was crash-reported as an SDK bug.
+    """
+    with tempfile.TemporaryDirectory() as tmp:
+        tmp_dir = Path(tmp)
+        layout = _copy_layout("single_file", tmp_dir)
+
+        with pytest.raises(CodeBundleError, match="is not a file, directory, or glob pattern"):
+            await build_code_bundle(
+                from_dir=layout,
+                dryrun=True,
+                copy_style="none",
+                additional_files=(str(layout / "does_not_exist.yaml"),),
+                copy_bundle_to=_bundle_out(tmp_dir),
+            )
+
+
 @pytest.mark.asyncio
 async def test_build_code_bundle_empty_source_raises(tmp_path):
     """An empty source directory with copy_style='all' surfaces a clear error."""

tests/flyte/code_bundle/test_code_bundle.py

@@ -234,15 +234,17 @@ def test_ls_relative_files_with_mixed_inputs():
 
 
 def test_ls_relative_files_invalid_path():
-    """Test ls_relative_files raises ValueError for invalid paths."""
+    """Test ls_relative_files raises CodeBundleError for invalid paths."""
+    from flyte.errors import CodeBundleError
+
     with tempfile.TemporaryDirectory() as tmpdir:
         test_dir = pathlib.Path(tmpdir)
 
         # Create a valid file
         (test_dir / "main.py").write_text("print('hello')")
 
         # Test with non-existent path that doesn't match any glob
-        with pytest.raises(ValueError, match="is not a valid file, directory, or glob pattern"):
+        with pytest.raises(CodeBundleError, match="is not a file, directory, or glob pattern"):
             ls_relative_files(["nonexistent.py"], test_dir)