From ecc27e9b0f2b2dbc198df0f751a4fb974167331e Mon Sep 17 00:00:00 2001 From: wizardchen Date: Mon, 22 Jun 2026 15:56:06 +0800 Subject: [PATCH] fix(docparser): rename close variable to avoid shadowing builtin The angle-bracket image target parser declared a local variable named `close`, which shadows Go's built-in `close` function and is flagged by revive's redefines-builtin-id rule, breaking `make lint`. Rename it to `closeIdx`. --- .../docparser/markdown_image_scanner.go | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/internal/infrastructure/docparser/markdown_image_scanner.go b/internal/infrastructure/docparser/markdown_image_scanner.go index d36d6fcb..87dc2768 100644 --- a/internal/infrastructure/docparser/markdown_image_scanner.go +++ b/internal/infrastructure/docparser/markdown_image_scanner.go @@ -144,25 +144,25 @@ func splitAngleMarkdownImageTarget( end int, refMap map[string]types.ImageRef, ) (path string, pathStart int, pathEnd int, ok bool) { - close := -1 + closeIdx := -1 for i := start + 1; i < end; i++ { if raw[i] == '>' && !isEscaped(raw, i) { - close = i + closeIdx = i break } } - if close == -1 { + if closeIdx == -1 { return "", 0, 0, false } - path = raw[start+1 : close] + path = raw[start+1 : closeIdx] if _, found := refMap[path]; !found { return "", 0, 0, false } - if !isEmptyOrMarkdownImageTitleSuffix(raw[close+1 : end]) { + if !isEmptyOrMarkdownImageTitleSuffix(raw[closeIdx+1 : end]) { return "", 0, 0, false } - return path, start + 1, close, true + return path, start + 1, closeIdx, true } func parseMarkdownImageTitleSuffix(raw string) (titleStart int, ok bool) {