Merge pull request #3032 from evidence-dev/fix/query-rows-undefined-o…

…n-page-load

.changeset/metal-walls-listen.md

@@ -0,0 +1,5 @@
+---
+'@evidence-dev/sdk': patch
+---
+
+Fix: Ensure that length is loaded before we returned undefined for out of range indeces

e2e/basic/pages/index.md

@@ -47,6 +47,8 @@ This page can be found in your project at `/pages/index.md`. Make a change to th
     series=category
 />
 
+The top category is {orders_by_category[0].category}
+
 ## What's Next?
 
 - [Connect your data sources](settings)

e2e/basic/sources/needful_things/connection.yaml

@@ -0,0 +1,5 @@
+# This file was automatically generated
+name: needful_things
+type: duckdb
+options:
+  filename: needful_things.duckdb

e2e/basic/sources/needful_things/orders.sql

@@ -0,0 +1 @@
+select * from orders

e2e/basic/tests/tests.spec.js

@@ -28,3 +28,10 @@ test('shows different logo in light and dark mode', async ({ page }) => {
 	logo = await getLogo(page);
 	await expect(logo).toHaveAttribute('src', '/darkLogo.png');
 });
+
+test('bare query reference', async ({ page }) => {
+	await page.goto('/');
+	await waitForPageToLoad(page);
+
+	await expect(page.getByText('The top category is Odd Equipment')).toBeVisible();
+});

packages/lib/sdk/src/usql/query/Query.js

@@ -634,7 +634,7 @@ DESCRIBE ${this.text.trim()}
 					if (target === null)
 						if (typeof prop !== 'number') return undefined;
 						else {
-							if (prop > this.#length) return undefined;
+							if (this.lengthLoaded && prop > this.#length) return undefined;
 							return this.#mockRow ?? {};
 						}