From 85a3f6a206d5f94b1ded7dc755bdfd387399b267 Mon Sep 17 00:00:00 2001 From: LEI WANG Date: Wed, 7 Jan 2026 14:52:17 +0800 Subject: [PATCH 1/2] fix(prerequisite): harden WrapErr behavior and error format Adds a nil-check in `BasePrerequisite.WrapErr` so that a nil `err` returns nil instead of wrapping. Panics with `"BasePrerequisite.WrapErr called with nil receiver"` when the receiver `p` is nil to catch incorrect usage early. Also updates the wrapped error message from `"failed to check [%s], caused by %w"` to `"failed to check [%s]: %w"` for a more conventional error format. --- pkg/prerequisite/util.go | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/pkg/prerequisite/util.go b/pkg/prerequisite/util.go index 7ab0857..cdeda02 100644 --- a/pkg/prerequisite/util.go +++ b/pkg/prerequisite/util.go @@ -3,5 +3,13 @@ package prerequisite import "fmt" func (p *BasePrerequisite) WrapErr(err error) error { - return fmt.Errorf("failed to check [%s], caused by %w", p.GetName(), err) + if err == nil { + return nil + } + + if p == nil { + panic("BasePrerequisite.WrapErr called with nil receiver") + } + + return fmt.Errorf("failed to check [%s]: %w", p.GetName(), err) } From b24561aa546ebf3fcc72d2a29aa5b29c1aa109c8 Mon Sep 17 00:00:00 2001 From: LEI WANG Date: Wed, 7 Jan 2026 14:53:27 +0800 Subject: [PATCH 2/2] chore(git): ignore .cursor directory --- .gitignore | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.gitignore b/.gitignore index 723ef36..000bd78 100644 --- a/.gitignore +++ b/.gitignore @@ -1 +1,2 @@ -.idea \ No newline at end of file +.idea +.cursor \ No newline at end of file