Skip to content

fix: surface mid-stream SQLite step errors (Issue #229) - #244

Merged
vmvarela merged 1 commit into
masterfrom
fix/step-errors-229
Sep 12, 2026
Merged

vmvarela merged 1 commit into
masterfrom
fix/step-errors-229

Conversation

@vmvarela

Copy link
Copy Markdown
Owner

Closes #229.

What: writeStreaming chequea rc tras el loop (espejo table/markdown) y StepFailed sale con exit 3 y mensaje SQLite real en vez de truncar en silencio con exit 0.

  • src/main.zig: rc check + StepFailed al brazo fatalSqlWithContext.
  • src/modes/repl.zig: StepFailed → printSqlError sin salir del REPL.

Test:

  • SELECT abs(-9223372036854775808) → error: integer overflow exit 3 (antes vacío exit 0).
  • Mid-stream tras filas + matriz json/csv/table/markdown OK; query sana OK.
  • zig build unit-test y zig build test exit 0.

@github-actions github-actions Bot added the type:bug Something isn't working label Sep 12, 2026
@vmvarela
vmvarela merged commit 120ba6b into master Sep 12, 2026
5 checks passed
@vmvarela
vmvarela deleted the fix/step-errors-229 branch September 12, 2026 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Mid-stream SQLite step errors silently ignored in streaming output

1 participant