From bebeb8053791a6efcb4a36b97f0597ddf0c7f06f Mon Sep 17 00:00:00 2001 From: Quratulain-bilal Date: Wed, 29 Jul 2026 13:48:22 +0500 Subject: [PATCH] fix: use missing_ok=True in integration JSON removal Replace check-then-act pattern (exists()+unlink()) with unlink(missing_ok=True) to eliminate TOCTOU race condition. --- src/specify_cli/integrations/_helpers.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/specify_cli/integrations/_helpers.py b/src/specify_cli/integrations/_helpers.py index 5c16935f1f..8ffc1b17a2 100644 --- a/src/specify_cli/integrations/_helpers.py +++ b/src/specify_cli/integrations/_helpers.py @@ -121,8 +121,7 @@ def _clear_init_options_for_integration(project_root: Path, integration_key: str def _remove_integration_json(project_root: Path) -> None: """Remove ``.specify/integration.json`` if it exists.""" path = project_root / INTEGRATION_JSON - if path.exists(): - path.unlink() + path.unlink(missing_ok=True) # ---------------------------------------------------------------------------