Refactor SqlStatement to use SqlResultCursor

[Copilot]

• Explore repository structure and understand existing code
• Modify SqlStatement.hpp:
  • Make all Execute*() variants return [[nodiscard]] SqlResultCursor
  • Move fetch methods (FetchRow, TryFetchRow, GetColumn*, CloseCursor, GetResultCursor, GetVariantRowCursor) to private
  • Add friend class SqlResultCursor to SqlStatement
  • Add BindOutputColumnsToRecord to SqlResultCursor
  • Update SqlRowIterator to use SqlResultCursor
  • Update ExecuteDirectScalar and MigrateDirect to use cursors internally
• Modify SqlStatement.cpp: Update ExecuteDirect, ExecuteWithVariants, ExecuteBatch to return SqlResultCursor
• Update SqlSchema.cpp to use SqlResultCursor
• Update SqlMigrationLock.cpp to use SqlResultCursor
• Update DataMapper/DataMapper.hpp to use SqlResultCursor
• Update src/tools/LupMigrationsPlugin/TransitionGlue.cpp
• Update test files (CoreTests.cpp, DataBinderTests.cpp, Utils.hpp, DataMapper/ReadTests.cpp, SqlBackup/*Tests.cpp)
• Build and verify changes compile correctly
• Run tests to validate changes

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Refactor SqlStatement to force the use of SqlResultCursor</issue_title>
<issue_description>- All SqlStatement.Execute() variants need t oreturn the SqlResultCursor as nodiscard.

• All of the fetch methods in SqlStatement need to be removed, as they're replaced by SqlResultCursor's members.
• Adapt all the existing code to the API changes.</issue_description>

<agent_instructions>Stick to modern C++23 API and languages. Embrace ranges, std::format, etc.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

• Fixes Refactor SqlStatement to force the use of SqlResultCursor #455

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.