oven-sh/bun

"create table foo;" from example isn't working

Open

#12119 opened on Jun 24, 2024

View on GitHub
 (2 comments) (0 reactions) (1 assignee)Rust (90,348 stars) (4,486 forks)batch import
bugbun:sqliteconfirmed bugdocsgood first issue

Description

What version of Bun is running?

1.1.16+bf7b327f6

What platform is your computer?

Darwin 23.5.0 arm64 arm

What steps can reproduce the bug?

const db = new Database(":memory:");
const query = db.query(`create table foo;`);

What is the expected behavior?

A created table.

What do you see instead?

22 | const query = db.query(create table foo); ^ SQLiteError: incomplete input errno: 1

  at prepare (bun:sqlite:277:10)
  at query (bun:sqlite:284:8)
  at /Users/steida/dev/bun/SkiplistSql.ts:22:18

Additional information

No response

Contributor guide

Tech stack
typescriptsqlite
Domain
backenddatabase
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
1
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
blocked
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
basic SQL syntaxfamiliarity with Bun's SQLite API
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70
Research direction
The issue is that the SQL statement 'create table foo;' is invalid because at least one column is required. The stack trace points to the 'prepare' function in bun:sqlite (line 277). Look at the Bun source code, likely in src/sqlite/sqlite.zig or similar, to see how SQL statements are passed to sqlite3 prepare v2. Consider adding validation to either auto add a dummy column or throw a clearer error. Check if any existing PRs or comments address this; none were linked.