Back to all reviewers

Explain complex logic

snyk/cli
Based on 2 comments
Go

Add explanatory comments for non-obvious code sections, complex business logic, and public methods to clarify their purpose and reasoning. Comments should explain the "why" behind decisions and the "what" for functions that aren't immediately clear from their implementation.

Documentation Go

Reviewer Prompt

Add explanatory comments for non-obvious code sections, complex business logic, and public methods to clarify their purpose and reasoning. Comments should explain the “why” behind decisions and the “what” for functions that aren’t immediately clear from their implementation.

For conditional logic with business rules, explain the rationale:

// Cobra has its own help mechanism, however since we have help documentation 
// only in legacy CLI, we should fallback to calling it for flag errors
if commandError {
    resultError = handleErrorFallbackToLegacyCLI
} else if flagError {
    resultError = handleErrorShowHelp
}

For public methods, provide high-level documentation describing what the method does:

// ConnectToProxy establishes a connection to the specified proxy server
// and handles the authentication handshake for the target address
func (p *ProxyAuthenticator) ConnectToProxy(ctx context.Context, proxyURL *url.URL, target string, connection net.Conn) error {

Focus on areas where the code’s intent or business logic isn’t immediately apparent to future maintainers.

2
Comments Analyzed
Go
Primary Language
Documentation
Category

Source Discussions