📑 Description

What’s Changed

• Refactored Query.Cursor() to simplify the return signature by removing the (chan, error) tuple.
• Now, errors during row iteration are sent through the channel by wrapping them in a Row struct with the err field set.
• Deferred cleanup of resources (channel close and Rows.Close()) is handled reliably even in case of early errors.
• Prevents potential goroutine leaks and ensures consumers always receive a closed channel.

Why

Previously, errors in goroutine setup (e.g., query.instance.Rows()) were only returned via the outer error return, which could be misleading since the channel would never yield results. This refactor ensures a more Go-idiomatic pattern: the channel always closes, and all consumers can handle errors uniformly through the same channel.

Potential Impact

Downstream consumers of Cursor() should now check Row.err to determine if an error occurred, instead of relying on the function's return error.

Example Usage (Updated)

for row := range query.Cursor() {
	if row.Err() != nil {
		log.Println("cursor error:", row.Err())
		break
	}
	// Process row
}

✅ Checks

• Added test cases for my code