Address reviews

Author: JyotinderSingh

keras_remote/infra/container_builder.py

@@ -83,7 +83,7 @@ def _filter_jax_requirements(requirements_content: str) -> str:
   return "".join(filtered_lines)
 
 
-def _parse_pyproject_dependencies(pyproject_path: str) -> str | None:
+def _parse_pyproject_dependencies(pyproject_path: str) -> str:
   """Extract ``[project.dependencies]`` from a pyproject.toml file.
 
   Reads only the core dependency list defined under the ``[project]`` table.
@@ -95,14 +95,15 @@ def _parse_pyproject_dependencies(pyproject_path: str) -> str | None:
 
   Returns:
       Newline-separated dependency strings in PEP 508 format suitable for
-      ``pip install``, or ``None`` if the file declares no dependencies.
+      ``pip install``, or an empty string if the file declares no
+      dependencies.
   """
   with open(pyproject_path, "rb") as f:
     data = tomllib.load(f)
 
   deps = data.get("project", {}).get("dependencies", [])
   if not deps:
-    return None
+    return ""
   return "\n".join(deps) + "\n"
 
 

keras_remote/infra/container_builder_test.py

@@ -93,17 +93,17 @@ def test_extracts_dependencies(self):
     result = _parse_pyproject_dependencies(path)
     self.assertEqual(result, "numpy>=1.20\npandas\n")
 
-  def test_returns_none_when_no_dependencies(self):
+  def test_returns_empty_when_no_dependencies(self):
     path = self._write_toml("[project]\nname = 'foo'\n")
-    self.assertIsNone(_parse_pyproject_dependencies(path))
+    self.assertEqual(_parse_pyproject_dependencies(path), "")
 
-  def test_returns_none_when_no_project_table(self):
+  def test_returns_empty_when_no_project_table(self):
     path = self._write_toml("[tool.ruff]\nline-length = 88\n")
-    self.assertIsNone(_parse_pyproject_dependencies(path))
+    self.assertEqual(_parse_pyproject_dependencies(path), "")
 
-  def test_returns_none_for_empty_dependencies(self):
+  def test_returns_empty_for_empty_dependencies(self):
     path = self._write_toml("[project]\ndependencies = []\n")
-    self.assertIsNone(_parse_pyproject_dependencies(path))
+    self.assertEqual(_parse_pyproject_dependencies(path), "")
 
   def test_ignores_optional_dependencies(self):
     path = self._write_toml(