From 493ddd465c6effcdfb377ecf74f0de89cfa2ceb3 Mon Sep 17 00:00:00 2001 From: Bissbert <43237892+Bissbert@users.noreply.github.com> Date: Sun, 15 Feb 2026 00:16:43 +0700 Subject: [PATCH] fix: resolve mypy return-value type errors in completion - Rename reused `inside` variable to `brace_content` and `bracket_content` to eliminate shadowed variable causing mypy return-value type mismatch Refs: mypy strict mode compliance --- src/cdl_lsp/features/completion.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/cdl_lsp/features/completion.py b/src/cdl_lsp/features/completion.py index f4f988f..29b9bc7 100644 --- a/src/cdl_lsp/features/completion.py +++ b/src/cdl_lsp/features/completion.py @@ -121,8 +121,8 @@ def _detect_context(line: str, col: int) -> tuple[CompletionContext, str]: brace_match = re.search(r"\{[^}]*$", text_before) if brace_match: # Get content inside braces - inside = brace_match.group(0)[1:] # Remove leading { - return (CompletionContext.MILLER_INDEX, inside) + brace_content = brace_match.group(0)[1:] # Remove leading { + return (CompletionContext.MILLER_INDEX, brace_content) # After @ if text_before_stripped.endswith("@"): @@ -131,8 +131,8 @@ def _detect_context(line: str, col: int) -> tuple[CompletionContext, str]: # Inside point group brackets bracket_match = re.search(r"\[[^\]]*$", text_before) if bracket_match: - inside = bracket_match.group(0)[1:] # Remove leading [ - return (CompletionContext.POINT_GROUP, inside) + bracket_content = bracket_match.group(0)[1:] # Remove leading [ + return (CompletionContext.POINT_GROUP, bracket_content) # After | if text_before_stripped.endswith("|") or re.search(r"\|\s*\w*$", text_before):