chore: 🪟🫂Windows friendly tooling (#2587)

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

Author: doctrino

.pre-commit-config.yaml

@@ -40,15 +40,15 @@ repos:
 
       - id: custom-checks
         name: custom repo checks
-        entry: poetry run python scripts/run_checks.py
+        entry: python scripts/run_checks.py
         files: ^cognite/.*.(py|pyi)$
         language: system
         pass_filenames: true
         require_serial: true  # avoid possible race conditions
 
       - id: sync-client-codegen
         name: run sync codegen on changed files
-        entry: poetry run python scripts/sync_client_codegen/main.py run --files
+        entry: python scripts/sync_client_codegen/main.py run --files
         files: ^cognite/.*.(py|pyi)$
         language: system
         pass_filenames: true

scripts/sync_client_codegen/codegen_utils.py

@@ -64,7 +64,7 @@ def is_md5_hash(s: str) -> bool:
 
 
 def read_hash_from_file(path: Path) -> tuple[bool, str]:
-    with path.open("r") as f:
+    with path.open("r", encoding="utf-8") as f:
         f.readline()
         f.readline()
         maybe_hash = f.readline().strip()
@@ -260,7 +260,7 @@ def ensure_parent_dir(file: Path) -> None:
 
 @cache
 def get_source_code(file: Path) -> str:
-    return file.read_text()
+    return file.read_text(encoding="utf-8")
 
 
 def filter_base_apis_and_sort_alphabetically(dct: dict[str, str]) -> list[tuple[str, str]]:

scripts/sync_client_codegen/create_sync_api.py

@@ -41,7 +41,7 @@
     verify_cognite_client_is_up_to_date,
 )
 
-SYNC_API_TEMPLATE = Path("scripts/sync_client_codegen/sync_api_template.txt").read_text()
+SYNC_API_TEMPLATE = Path("scripts/sync_client_codegen/sync_api_template.txt").read_text(encoding="utf-8")
 
 
 def _generate_code_for_single_sync_api(
@@ -258,7 +258,7 @@ def _compare_normalized_files_and_maybe_update_hash_only(files_to_update: set[Si
         for f in files_to_update:
             f.temp_filepath = tmp_dir / f.filepath
             ensure_parent_dir(f.temp_filepath)
-            f.temp_filepath.write_text(f.get_new_source)
+            f.temp_filepath.write_text(f.get_new_source, encoding="utf-8")
 
         # Before compare, we need to run ruff (large overhead, so we do it once for all files):
         run_ruff([f.temp_filepath for f in files_to_update], verbose=False)
@@ -295,7 +295,7 @@ def _compare_normalized_files_and_maybe_update_hash_only(files_to_update: set[Si
             # Note: DO NOT use normalized_src here, the reason we do all of this is precisely to preserve
             # e.g. type-ignore comments that would otherwise be lost on regeneration.
             current_source = get_source_code(write_file)
-            write_file.write_text(current_source.replace(existing_hash, f.get_new_hash))
+            write_file.write_text(current_source.replace(existing_hash, f.get_new_hash), encoding="utf-8")
 
 
 def _report_after_verification(
@@ -335,7 +335,7 @@ def _report_after_verification(
 def _write_updated_sync_api_files(files_to_update: set[SingleAPIFile]) -> None:
     for f in files_to_update:
         ensure_parent_dir(f.filepath)
-        f.filepath.write_text(f.get_new_source)
+        f.filepath.write_text(f.get_new_source, encoding="utf-8")
         if f.class_name:
             print(f"- Generated/updated sync API code for {f.class_name} from '{f.read_filepath}' ✅")
         else:
@@ -391,7 +391,7 @@ def _run_files(
 
     if client_has_changed:
         to_be_linted.append(SYNC_CLIENT_PATH)
-        SYNC_CLIENT_PATH.write_text(new_client_source_code)
+        SYNC_CLIENT_PATH.write_text(new_client_source_code, encoding="utf-8")
         print(f"- Regenerated (sync) CogniteClient: '{SYNC_CLIENT_PATH}' ✅")
     elif args.verbose:
         print(f"- Skipped (sync) CogniteClient, is up to date: '{SYNC_CLIENT_PATH}' ⏭️")

scripts/sync_client_codegen/create_sync_client.py

@@ -1,5 +1,5 @@
 from pathlib import Path
-from tempfile import NamedTemporaryFile
+from tempfile import TemporaryDirectory
 
 from scripts.sync_client_codegen.codegen_utils import (
     filter_base_apis_and_sort_alphabetically,
@@ -14,7 +14,7 @@
     SYNC_CLIENT_PATH,
 )
 
-COGNITE_CLIENT_TEMPLATE = Path("scripts/sync_client_codegen/sync_client_template.txt").read_text()
+COGNITE_CLIENT_TEMPLATE = Path("scripts/sync_client_codegen/sync_client_template.txt").read_text(encoding="utf-8")
 
 
 def create_sync_cognite_client(
@@ -39,10 +39,9 @@ def create_sync_cognite_client(
 
 
 def verify_cognite_client_is_up_to_date(new_source: str) -> bool:
-    with NamedTemporaryFile(mode="w+", suffix=".py") as f:
-        f.write(new_source)
-        f.flush()  # Ensure content is written before ruff reads it
-        path = Path(f.name)
+    with TemporaryDirectory() as tmp_dir:
+        path = Path(tmp_dir) / "sync_client.py"
+        path.write_text(new_source, encoding="utf-8")
         run_ruff_direct(path)
         new_file_ast = get_canonical_source(path)