dont default file version to 1.json

Author: caila-marashaj

api/src/opentrons/protocols/labware.py

@@ -240,7 +240,7 @@ def _get_standard_labware_definition(
         )
 
     namespace = namespace.lower()
-    def_path = _get_path_to_labware(load_name, namespace, checked_version)
+    def_path = _get_path_to_labware(load_name, namespace, version)
 
     try:
         with open(def_path, "rb") as f:
@@ -254,26 +254,31 @@ def _get_standard_labware_definition(
 
 
 def _get_path_to_labware(
-    load_name: str, namespace: str, version: int, base_path: Optional[Path] = None
+    load_name: str,
+    namespace: str,
+    version: Optional[int] = None,
+    base_path: Optional[Path] = None,
 ) -> Path:
     if namespace == OPENTRONS_NAMESPACE:
         # all labware in OPENTRONS_NAMESPACE is stored in shared data
-        schema_3_path = (
-            get_shared_data_root()
-            / STANDARD_DEFS_PATH
-            / "3"
-            / load_name
-            / f"{version}.json"
-        )
-        schema_2_path = (
-            get_shared_data_root()
-            / STANDARD_DEFS_PATH
-            / "2"
-            / load_name
-            / f"{version}.json"
-        )
-        return schema_3_path if schema_3_path.exists() else schema_2_path
+        schema_3_dir = get_shared_data_root() / STANDARD_DEFS_PATH / "3" / load_name
+        if schema_3_dir.exists():
+            schema_3_files = os.listdir(schema_3_dir)
+            version_filename = f"{version}.json" if version else max(schema_3_files)
+            schema_3_path = schema_3_dir / version_filename
+            return schema_3_path
+        else:
+            version_filename = f"{version}.json" if version else "1.json"
+            schema_2_path = (
+                get_shared_data_root()
+                / STANDARD_DEFS_PATH
+                / "2"
+                / load_name
+                / f"{version_filename}.json"
+            )
+        return schema_2_path
     if not base_path:
         base_path = USER_DEFS_PATH
-    def_path = base_path / namespace / load_name / f"{version}.json"
+    version_filename = f"{version}.json" if version else "1.json"
+    def_path = base_path / namespace / load_name / f"{version_filename}.json"
     return def_path