fix: handle nil message return in internalGo function and add tests f…

[kazeburo]

User description

…or error message type

---

PR Type

Bug fix, Tests

---

Description

• Safely handle nil runner messages.

• Preserve exit codes for typed nil returns.

• Add reusable tests for any and error.

---

Diagram Walkthrough

flowchart LR
  Runner["Runner returns message and code"]
  NilCheck["Validate nil message value"]
  Result["Return empty message with code"]
  Format["Format non-nil message"]
  Runner -- "provides result" --> NilCheck
  NilCheck -- "nil" --> Result
  NilCheck -- "non-nil" --> Format

Loading

File Walkthrough

	Relevant files
Bug fix	flagrun.go Handle nil messages from generic runners                                  flagrun.go Detect invalid and typed nil runner messages using reflection. Return an empty message while preserving the runner exit code. Avoid formatting nil values as . +12/-0
Tests	flagrun_go_test.go Test nil error runner message handling                                      flagrun_go_test.go Add an error-typed runner that returns nil on success. Extract shared message-type test cases into a generic helper. Verify nil error messages return an empty string. +24/-4

---

[github-actions]

PR Reviewer Guide 🔍

(Review updated until commit 391aa80)

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ No major issues detected

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

Persistent review updated to latest commit 391aa80

[github-actions]

PR Code Suggestions ✨

No code suggestions found for the PR.