Skip to content

fix: display stored procedure result sets - #287

Merged
Maxteabag merged 1 commit into
mainfrom
agent/stored-procedure-results
Aug 6, 2026
Merged

fix: display stored procedure result sets#287
Maxteabag merged 1 commit into
mainfrom
agent/stored-procedure-results

Conversation

@Maxteabag

Copy link
Copy Markdown
Owner

Summary

  • route MySQL/MariaDB CALL and SQL Server EXEC/EXECUTE procedure calls through row-returning execution
  • return the first row-bearing result set and drain all remaining status/result sets
  • keep procedure classification provider-specific so Oracle and Firebird non-query commit behavior is unchanged
  • distinguish SQL Server procedures from EXECUTE AS and dynamic SQL

Supersedes #265 while preserving its intended behavior and resolving the review blockers.

Verification

  • uv run pytest -q tests/unit (1112 passed, 2 skipped)
  • focused stored-procedure and SQL Server suite (43 passed)
  • uv run ruff check ...
  • autoreview clean: no accepted/actionable findings

Thanks @probeldev for the original report and implementation in #265.

@Maxteabag
Maxteabag merged commit 0dd06fa into main Aug 6, 2026
1 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant