github2gerrit 0.1.5__py3-none-any.whl → 0.1.6__py3-none-any.whl

github2gerrit/cli.py

@@ -42,9 +42,7 @@ def _is_verbose_mode() -> bool:
     return os.getenv("G2G_VERBOSE", "").lower() in ("true", "1", "yes")
 
 
-def _log_exception_conditionally(
-    logger: logging.Logger, message: str, *args: Any
-) -> None:
+def _log_exception_conditionally(logger: logging.Logger, message: str, *args: Any) -> None:
     """Log exception with traceback only if verbose mode is enabled."""
     if _is_verbose_mode():
         logger.exception(message, *args)
@@ -120,13 +118,9 @@ class _ContextProto(Protocol):
 
 
 class _SingleUsageGroup(BaseGroup):
-    def format_usage(
-        self, ctx: _ContextProto, formatter: _FormatterProto
-    ) -> None:
+    def format_usage(self, ctx: _ContextProto, formatter: _FormatterProto) -> None:
         # Force a simplified usage line without COMMAND [ARGS]...
-        formatter.write_usage(
-            ctx.command_path, "[OPTIONS] TARGET_URL", prefix="Usage: "
-        )
+        formatter.write_usage(ctx.command_path, "[OPTIONS] TARGET_URL", prefix="Usage: ")
 
 
 # Error message constants to comply with TRY003
@@ -137,7 +131,7 @@ _MSG_ISSUE_ID_MULTILINE = "Issue ID must be single line"
 app: typer.Typer = typer.Typer(
     add_completion=False,
     no_args_is_help=False,
-    cls=_SingleUsageGroup,
+    cls=cast(Any, _SingleUsageGroup),
 )
 
 
@@ -154,9 +148,7 @@ def _resolve_org(default_org: str | None) -> str:
 if TYPE_CHECKING:
     F = TypeVar("F", bound=Callable[..., object])
 
-    def typed_app_command(
-        *args: object, **kwargs: object
-    ) -> Callable[[F], F]: ...
+    def typed_app_command(*args: object, **kwargs: object) -> Callable[[F], F]: ...
 else:
     typed_app_command = app.command
 
@@ -269,6 +261,14 @@ def main(
         envvar="ALLOW_DUPLICATES",
         help="Allow submitting duplicate changes without error.",
     ),
+    duplicates: str = typer.Option(
+        "open",
+        "--duplicates",
+        envvar="DUPLICATES",
+        help=(
+            'Gerrit statuses for duplicate detection (comma-separated). E.g. "open,merged,abandoned". Default: "open".'
+        ),
+    ),
     verbose: bool = typer.Option(
         False,
         "--verbose",
@@ -328,6 +328,8 @@ def main(
         os.environ["ISSUE_ID"] = issue_id
     if allow_duplicates:
         os.environ["ALLOW_DUPLICATES"] = "true"
+    if duplicates:
+        os.environ["DUPLICATES"] = duplicates
     # URL mode handling
     if target_url:
         org, repo, pr = _parse_github_target(target_url)
@@ -355,10 +357,7 @@ def main(
 def _setup_logging() -> logging.Logger:
     level_name = os.getenv("G2G_LOG_LEVEL", "INFO").upper()
     level = getattr(logging, level_name, logging.INFO)
-    fmt = (
-        "%(asctime)s %(levelname)-8s %(name)s "
-        "%(filename)s:%(lineno)d | %(message)s"
-    )
+    fmt = "%(asctime)s %(levelname)-8s %(name)s %(filename)s:%(lineno)d | %(message)s"
     logging.basicConfig(level=level, format=fmt)
     return logging.getLogger(APP_NAME)
 
@@ -397,9 +396,7 @@ def _build_inputs_from_env() -> Inputs:
         gerrit_ssh_privkey_g2g=_env_str("GERRIT_SSH_PRIVKEY_G2G"),
         gerrit_ssh_user_g2g=_env_str("GERRIT_SSH_USER_G2G"),
         gerrit_ssh_user_g2g_email=_env_str("GERRIT_SSH_USER_G2G_EMAIL"),
-        organization=_env_str(
-            "ORGANIZATION", _env_str("GITHUB_REPOSITORY_OWNER")
-        ),
+        organization=_env_str("ORGANIZATION", _env_str("GITHUB_REPOSITORY_OWNER")),
         reviewers_email=_env_str("REVIEWERS_EMAIL", ""),
         preserve_github_prs=_env_bool("PRESERVE_GITHUB_PRS", False),
         dry_run=_env_bool("DRY_RUN", False),
@@ -408,6 +405,7 @@ def _build_inputs_from_env() -> Inputs:
         gerrit_project=_env_str("GERRIT_PROJECT"),
         issue_id=_env_str("ISSUE_ID"),
         allow_duplicates=_env_bool("ALLOW_DUPLICATES", False),
+        duplicates_filter=_env_str("DUPLICATES", "open"),
     )
 
 
@@ -441,20 +439,23 @@ def _process_bulk(data: Inputs, gh: GitHubContext) -> None:
 
         log.info("Starting processing of PR #%d", pr_number)
         log.debug(
-            "Processing PR #%d in multi-PR mode with event_name=%s, "
-            "event_action=%s",
+            "Processing PR #%d in multi-PR mode with event_name=%s, event_action=%s",
             pr_number,
             gh.event_name,
             gh.event_action,
         )
 
         try:
-            check_for_duplicates(
-                per_ctx, allow_duplicates=data.allow_duplicates
-            )
+            if data.duplicates_filter:
+                os.environ["DUPLICATES"] = data.duplicates_filter
+            check_for_duplicates(per_ctx, allow_duplicates=data.allow_duplicates)
         except DuplicateChangeError as exc:
             _log_exception_conditionally(log, "Skipping PR #%d", pr_number)
-            typer.echo(f"Skipping PR #{pr_number}: {exc}")
+            log.warning(
+                "Skipping PR #%d due to duplicate detection: %s. Use --allow-duplicates to override this check.",
+                pr_number,
+                exc,
+            )
             continue
 
         try:
@@ -479,9 +480,7 @@ def _process_bulk(data: Inputs, gh: GitHubContext) -> None:
                         result_multi.change_numbers,
                     )
         except Exception as exc:
-            _log_exception_conditionally(
-                log, "Failed to process PR #%d", pr_number
-            )
+            _log_exception_conditionally(log, "Failed to process PR #%d", pr_number)
             typer.echo(f"Failed to process PR #{pr_number}: {exc}")
             log.info("Continuing to next PR despite failure")
             continue
@@ -493,12 +492,8 @@ def _process_bulk(data: Inputs, gh: GitHubContext) -> None:
 
     _append_github_output(
         {
-            "gerrit_change_request_url": os.getenv(
-                "GERRIT_CHANGE_REQUEST_URL", ""
-            ),
-            "gerrit_change_request_num": os.getenv(
-                "GERRIT_CHANGE_REQUEST_NUM", ""
-            ),
+            "gerrit_change_request_url": os.getenv("GERRIT_CHANGE_REQUEST_URL", ""),
+            "gerrit_change_request_num": os.getenv("GERRIT_CHANGE_REQUEST_NUM", ""),
         }
     )
 
@@ -524,30 +519,20 @@ def _process_single(data: Inputs, gh: GitHubContext) -> None:
         except Exception as exc:
             log.debug("Execution failed; continuing to write outputs: %s", exc)
 
-            result = SubmissionResult(
-                change_urls=[], change_numbers=[], commit_shas=[]
-            )
+            result = SubmissionResult(change_urls=[], change_numbers=[], commit_shas=[])
         if result.change_urls:
-            os.environ["GERRIT_CHANGE_REQUEST_URL"] = "\n".join(
-                result.change_urls
-            )
+            os.environ["GERRIT_CHANGE_REQUEST_URL"] = "\n".join(result.change_urls)
             # Output Gerrit change URL(s) to console
             for url in result.change_urls:
                 log.info("Gerrit change URL: %s", url)
         if result.change_numbers:
-            os.environ["GERRIT_CHANGE_REQUEST_NUM"] = "\n".join(
-                result.change_numbers
-            )
+            os.environ["GERRIT_CHANGE_REQUEST_NUM"] = "\n".join(result.change_numbers)
 
         # Also write outputs to GITHUB_OUTPUT if available
         _append_github_output(
             {
-                "gerrit_change_request_url": os.getenv(
-                    "GERRIT_CHANGE_REQUEST_URL", ""
-                ),
-                "gerrit_change_request_num": os.getenv(
-                    "GERRIT_CHANGE_REQUEST_NUM", ""
-                ),
+                "gerrit_change_request_url": os.getenv("GERRIT_CHANGE_REQUEST_URL", ""),
+                "gerrit_change_request_num": os.getenv("GERRIT_CHANGE_REQUEST_NUM", ""),
                 "gerrit_commit_sha": os.getenv("GERRIT_COMMIT_SHA", ""),
             }
         )
@@ -559,13 +544,9 @@ def _process_single(data: Inputs, gh: GitHubContext) -> None:
         return
 
 
-def _prepare_local_checkout(
-    workspace: Path, gh: GitHubContext, data: Inputs
-) -> None:
+def _prepare_local_checkout(workspace: Path, gh: GitHubContext, data: Inputs) -> None:
     repo_full = gh.repository.strip() if gh.repository else ""
-    server_url = gh.server_url or os.getenv(
-        "GITHUB_SERVER_URL", "https://github.com"
-    )
+    server_url = gh.server_url or os.getenv("GITHUB_SERVER_URL", "https://github.com")
     server_url = (server_url or "https://github.com").rstrip("/")
     base_ref = gh.base_ref or ""
     pr_num_str: str = str(gh.pr_number) if gh.pr_number else "0"
@@ -577,6 +558,26 @@ def _prepare_local_checkout(
     run_cmd(["git", "init"], cwd=workspace)
     run_cmd(["git", "remote", "add", "origin", repo_url], cwd=workspace)
 
+    # Non-interactive SSH/Git environment for any network operations
+    env = {
+        "GIT_SSH_COMMAND": (
+            "ssh -F /dev/null "
+            "-o IdentitiesOnly=yes "
+            "-o IdentityAgent=none "
+            "-o BatchMode=yes "
+            "-o PreferredAuthentications=publickey "
+            "-o StrictHostKeyChecking=yes "
+            "-o PasswordAuthentication=no "
+            "-o PubkeyAcceptedKeyTypes=+ssh-rsa "
+            "-o ConnectTimeout=10"
+        ),
+        "SSH_AUTH_SOCK": "",
+        "SSH_AGENT_PID": "",
+        "SSH_ASKPASS": "/usr/bin/false",
+        "DISPLAY": "",
+        "SSH_ASKPASS_REQUIRE": "never",
+    }
+
     # Fetch base branch and PR head
     if base_ref:
         try:
@@ -590,15 +591,13 @@ def _prepare_local_checkout(
                     branch_ref,
                 ],
                 cwd=workspace,
+                env=env,
             )
         except Exception as exc:
             log.debug("Base branch fetch failed for %s: %s", base_ref, exc)
 
     if pr_num_str:
-        pr_ref = (
-            f"refs/pull/{pr_num_str}/head:"
-            f"refs/remotes/origin/pr/{pr_num_str}/head"
-        )
+        pr_ref = f"refs/pull/{pr_num_str}/head:refs/remotes/origin/pr/{pr_num_str}/head"
         run_cmd(
             [
                 "git",
@@ -608,6 +607,7 @@ def _prepare_local_checkout(
                 pr_ref,
             ],
             cwd=workspace,
+            env=env,
         )
         run_cmd(
             [
@@ -618,6 +618,7 @@ def _prepare_local_checkout(
                 f"refs/remotes/origin/pr/{pr_num_str}/head",
             ],
             cwd=workspace,
+            env=env,
         )
 
 
@@ -626,11 +627,7 @@ def _load_effective_inputs() -> Inputs:
     data = _build_inputs_from_env()
 
     # Load per-org configuration and apply to environment before validation
-    org_for_cfg = (
-        data.organization
-        or os.getenv("ORGANIZATION")
-        or os.getenv("GITHUB_REPOSITORY_OWNER")
-    )
+    org_for_cfg = data.organization or os.getenv("ORGANIZATION") or os.getenv("GITHUB_REPOSITORY_OWNER")
     cfg = load_org_config(org_for_cfg)
 
     # Apply dynamic parameter derivation for missing Gerrit parameters
@@ -642,9 +639,7 @@ def _load_effective_inputs() -> Inputs:
     data = _build_inputs_from_env()
 
     # Derive reviewers from local git config if running locally and unset
-    if not os.getenv("REVIEWERS_EMAIL") and (
-        os.getenv("G2G_TARGET_URL") or not os.getenv("GITHUB_EVENT_NAME")
-    ):
+    if not os.getenv("REVIEWERS_EMAIL") and (os.getenv("G2G_TARGET_URL") or not os.getenv("GITHUB_EVENT_NAME")):
         try:
             from .gitutils import enumerate_reviewer_emails
 
@@ -668,6 +663,7 @@ def _load_effective_inputs() -> Inputs:
                     gerrit_project=data.gerrit_project,
                     issue_id=data.issue_id,
                     allow_duplicates=data.allow_duplicates,
+                    duplicates_filter=data.duplicates_filter,
                 )
                 log.info("Derived reviewers: %s", data.reviewers_email)
         except Exception as exc:
@@ -696,24 +692,14 @@ def _append_github_output(outputs: dict[str, str]) -> None:
 
 
 def _augment_pr_refs_if_needed(gh: GitHubContext) -> GitHubContext:
-    if (
-        os.getenv("G2G_TARGET_URL")
-        and gh.pr_number
-        and (not gh.head_ref or not gh.base_ref)
-    ):
+    if os.getenv("G2G_TARGET_URL") and gh.pr_number and (not gh.head_ref or not gh.base_ref):
         try:
             client = build_client()
             repo = get_repo_from_env(client)
             pr_obj = get_pull(repo, int(gh.pr_number))
-            base_ref = str(
-                getattr(getattr(pr_obj, "base", object()), "ref", "") or ""
-            )
-            head_ref = str(
-                getattr(getattr(pr_obj, "head", object()), "ref", "") or ""
-            )
-            head_sha = str(
-                getattr(getattr(pr_obj, "head", object()), "sha", "") or ""
-            )
+            base_ref = str(getattr(getattr(pr_obj, "base", object()), "ref", "") or "")
+            head_ref = str(getattr(getattr(pr_obj, "head", object()), "ref", "") or "")
+            head_sha = str(getattr(getattr(pr_obj, "head", object()), "sha", "") or "")
             if base_ref:
                 os.environ["GITHUB_BASE_REF"] = base_ref
                 log.info("Resolved base_ref via GitHub API: %s", base_ref)
@@ -751,21 +737,17 @@ def _process() -> None:
 
     # Bulk mode for URL/workflow_dispatch
     sync_all = _env_bool("SYNC_ALL_OPEN_PRS", False)
-    if sync_all and (
-        gh.event_name == "workflow_dispatch" or os.getenv("G2G_TARGET_URL")
-    ):
+    if sync_all and (gh.event_name == "workflow_dispatch" or os.getenv("G2G_TARGET_URL")):
         _process_bulk(data, gh)
         return
 
     if not gh.pr_number:
         log.error(
-            "PR_NUMBER is empty. This tool requires a valid pull request "
-            "context. Current event: %s",
+            "PR_NUMBER is empty. This tool requires a valid pull request context. Current event: %s",
             gh.event_name,
         )
         typer.echo(
-            "PR_NUMBER is empty. This tool requires a valid pull request "
-            f"context. Current event: {gh.event_name}",
+            f"PR_NUMBER is empty. This tool requires a valid pull request context. Current event: {gh.event_name}",
             err=True,
         )
         raise typer.Exit(code=2)
@@ -779,13 +761,16 @@ def _process() -> None:
     # Check for duplicates in single-PR mode (before workspace setup)
     if gh.pr_number and not _env_bool("SYNC_ALL_OPEN_PRS", False):
         try:
+            if data.duplicates_filter:
+                os.environ["DUPLICATES"] = data.duplicates_filter
             check_for_duplicates(gh, allow_duplicates=data.allow_duplicates)
         except DuplicateChangeError as exc:
-            _log_exception_conditionally(log, "Duplicate change detected")
-            typer.echo(f"Error: {exc}", err=True)
-            typer.echo(
-                "Use --allow-duplicates to override this check.", err=True
+            _log_exception_conditionally(
+                log,
+                "Duplicate detection blocked submission for PR #%d",
+                gh.pr_number,
             )
+            log.info("Use --allow-duplicates to override this check.")
             raise typer.Exit(code=3) from exc
 
     _process_single(data, gh)
@@ -804,9 +789,7 @@ def _load_event(path: Path | None) -> dict[str, Any]:
     if not path or not path.exists():
         return {}
     try:
-        return cast(
-            dict[str, Any], json.loads(path.read_text(encoding="utf-8"))
-        )
+        return cast(dict[str, Any], json.loads(path.read_text(encoding="utf-8")))
     except Exception as exc:
         log.warning("Failed to parse GITHUB_EVENT_PATH: %s", exc)
         return {}
@@ -873,10 +856,7 @@ def _read_github_context() -> GitHubContext:
 
 def _validate_inputs(data: Inputs) -> None:
     if data.use_pr_as_commit and data.submit_single_commits:
-        msg = (
-            "USE_PR_AS_COMMIT and SUBMIT_SINGLE_COMMITS cannot be enabled at "
-            "the same time"
-        )
+        msg = "USE_PR_AS_COMMIT and SUBMIT_SINGLE_COMMITS cannot be enabled at the same time"
         raise ConfigurationError(msg)
 
     # Context-aware validation: different requirements for GH Actions vs CLI
@@ -890,7 +870,6 @@ def _validate_inputs(data: Inputs) -> None:
     if is_github_actions:
         # In GitHub Actions: allow derivation if organization is available
         if not data.organization:
-            # No organization means no derivation possible
             required_fields.extend(
                 [
                     "gerrit_ssh_user_g2g",
@@ -901,9 +880,7 @@ def _validate_inputs(data: Inputs) -> None:
         # In local CLI: require explicit values or organization + derivation
         # This prevents unexpected behavior when running locally
         missing_gerrit_params = [
-            field
-            for field in ["gerrit_ssh_user_g2g", "gerrit_ssh_user_g2g_email"]
-            if not getattr(data, field)
+            field for field in ["gerrit_ssh_user_g2g", "gerrit_ssh_user_g2g_email"] if not getattr(data, field)
         ]
         if missing_gerrit_params:
             if data.organization:
@@ -931,24 +908,18 @@ def _validate_inputs(data: Inputs) -> None:
                     if is_github_actions:
                         log.error(
                             "These fields can be derived automatically from "
-                            "organization '%s'",
+                            "organization '%s' if G2G_ENABLE_DERIVATION=true",
                             data.organization,
                         )
                     else:
                         log.error(
-                            "These fields can be derived from organization "
-                            "'%s'",
+                            "These fields can be derived from organization '%s'",
                             data.organization,
                         )
                         log.error("Set G2G_ENABLE_DERIVATION=true to enable")
                 else:
-                    log.error(
-                        "These fields require either explicit values or an "
-                        "ORGANIZATION for derivation"
-                    )
-            raise ConfigurationError(
-                _MSG_MISSING_REQUIRED_INPUT.format(field_name=field_name)
-            )
+                    log.error("These fields require either explicit values or an ORGANIZATION for derivation")
+            raise ConfigurationError(_MSG_MISSING_REQUIRED_INPUT.format(field_name=field_name))
 
     # Validate fetch depth is a positive integer
     if data.fetch_depth <= 0:
@@ -967,9 +938,7 @@ def _log_effective_config(data: Inputs, gh: GitHubContext) -> None:
     log.info("  SUBMIT_SINGLE_COMMITS: %s", data.submit_single_commits)
     log.info("  USE_PR_AS_COMMIT: %s", data.use_pr_as_commit)
     log.info("  FETCH_DEPTH: %s", data.fetch_depth)
-    known_hosts_status = (
-        "<provided>" if data.gerrit_known_hosts else "<will auto-discover>"
-    )
+    known_hosts_status = "<provided>" if data.gerrit_known_hosts else "<will auto-discover>"
     log.info("  GERRIT_KNOWN_HOSTS: %s", known_hosts_status)
     log.info("  GERRIT_SSH_PRIVKEY_G2G: %s", safe_privkey)
     log.info("  GERRIT_SSH_USER_G2G: %s", data.gerrit_ssh_user_g2g)

github2gerrit/config.py

@@ -130,11 +130,7 @@ def _coerce_value(raw: str) -> str:
     # Normalize escaped newline sequences into real newlines so that values
     # like SSH keys or known_hosts entries can be specified inline using
     # '\n' or '\r\n' in configuration files.
-    normalized_newlines = (
-        unquoted.replace("\\r\\n", "\n")
-        .replace("\\n", "\n")
-        .replace("\r\n", "\n")
-    )
+    normalized_newlines = unquoted.replace("\\r\\n", "\n").replace("\\n", "\n").replace("\r\n", "\n")
     b = _normalize_bool_like(normalized_newlines)
     return b if b is not None else normalized_newlines
 
@@ -317,10 +313,7 @@ def filter_known(
 
 def _is_github_actions_context() -> bool:
     """Detect if running in GitHub Actions environment."""
-    return (
-        os.getenv("GITHUB_ACTIONS") == "true"
-        or os.getenv("GITHUB_EVENT_NAME", "").strip() != ""
-    )
+    return os.getenv("GITHUB_ACTIONS") == "true" or os.getenv("GITHUB_EVENT_NAME", "").strip() != ""
 
 
 def _is_local_cli_context() -> bool:
@@ -346,9 +339,7 @@ def derive_gerrit_parameters(organization: str | None) -> dict[str, str]:
     org = organization.strip().lower()
     return {
         "GERRIT_SSH_USER_G2G": f"{org}.gh2gerrit",
-        "GERRIT_SSH_USER_G2G_EMAIL": (
-            f"releng+{org}-gh2gerrit@linuxfoundation.org"
-        ),
+        "GERRIT_SSH_USER_G2G_EMAIL": (f"releng+{org}-gh2gerrit@linuxfoundation.org"),
         "GERRIT_SERVER": f"gerrit.{org}.org",
     }
 
@@ -369,7 +360,8 @@ def apply_parameter_derivation(
 
     Derivation behavior depends on execution context:
     - GitHub Actions: Automatic derivation when organization is available
-    - Local CLI: Requires G2G_ENABLE_DERIVATION=true for automatic derivation
+    - Local CLI: Requires G2G_ENABLE_DERIVATION=true for automatic
+      derivation
 
     Args:
         cfg: Configuration dictionary to augment
@@ -384,9 +376,12 @@ def apply_parameter_derivation(
 
     # Check execution context to determine derivation strategy
     is_github_actions = _is_github_actions_context()
-    enable_derivation = is_github_actions or os.getenv(
-        "G2G_ENABLE_DERIVATION", ""
-    ).strip().lower() in ("1", "true", "yes", "on")
+    enable_derivation = is_github_actions or os.getenv("G2G_ENABLE_DERIVATION", "").strip().lower() in (
+        "1",
+        "true",
+        "yes",
+        "on",
+    )
 
     if not enable_derivation:
         log.debug(
@@ -412,12 +407,22 @@ def apply_parameter_derivation(
             result[key] = value
             newly_derived[key] = value
 
+    if newly_derived:
+        log.info(
+            "Derived parameters applied for organization '%s' (%s): %s",
+            organization,
+            "GitHub Actions" if is_github_actions else "Local CLI",
+            ", ".join(f"{k}={v}" for k, v in newly_derived.items()),
+        )
     # Save newly derived parameters to configuration file for future use
     # Default to true for local CLI, false for GitHub Actions
     default_auto_save = "false" if _is_github_actions_context() else "true"
-    auto_save_enabled = os.getenv(
-        "G2G_AUTO_SAVE_CONFIG", default_auto_save
-    ).strip().lower() in ("1", "true", "yes", "on")
+    auto_save_enabled = os.getenv("G2G_AUTO_SAVE_CONFIG", default_auto_save).strip().lower() in (
+        "1",
+        "true",
+        "yes",
+        "on",
+    )
     if save_to_config and newly_derived and auto_save_enabled:
         # Save to config in local CLI mode to create persistent configuration
         try:
@@ -459,15 +464,18 @@ def save_derived_parameters_to_config(
         return
 
     if config_path is None:
-        config_path = (
-            os.getenv("G2G_CONFIG_PATH", "").strip() or DEFAULT_CONFIG_PATH
-        )
+        config_path = os.getenv("G2G_CONFIG_PATH", "").strip() or DEFAULT_CONFIG_PATH
 
     config_file = Path(config_path).expanduser()
 
     try:
-        # Ensure the directory exists
-        config_file.parent.mkdir(parents=True, exist_ok=True)
+        # Only update when a configuration file already exists
+        if not config_file.exists():
+            log.debug(
+                "Configuration file does not exist; skipping auto-save of derived parameters: %s",
+                config_file,
+            )
+            return
 
         # Parse existing content using configparser
         cp = _load_ini(config_file)