Merge pull request #1245 from ponder-sh/kjs/fix-default

fix: default columns

.changeset/warm-snails-mix.md

@@ -0,0 +1,5 @@
+---
+"@ponder/core": patch
+---
+
+Fixed a bug with the `.default()` column modifier causing the value to be undefined.

packages/core/src/indexing-store/historical.test.ts

@@ -556,7 +556,7 @@ test("default", async (context) => {
   const schema = {
     account: onchainTable("account", (p) => ({
       address: p.hex().primaryKey(),
-      balance: p.bigint().default(10n),
+      balance: p.integer().default(0),
     })),
   };
 
@@ -573,7 +573,7 @@ test("default", async (context) => {
     address: zeroAddress,
   });
 
-  expect(result).toStrictEqual({ address: zeroAddress, balance: 10n });
+  expect(result).toStrictEqual({ address: zeroAddress, balance: 0 });
 
   await cleanup();
 });

packages/core/src/indexing-store/historical.ts

@@ -216,9 +216,9 @@ export const createHistoricalIndexingStore = ({
     if (type === EntryType.UPDATE && column.onUpdateFn) {
       return column.onUpdateFn();
     }
-    if (column.default) return column.default;
-    if (column.defaultFn) return column.defaultFn();
-    if (column.onUpdateFn) return column.onUpdateFn();
+    if (column.default !== undefined) return column.default;
+    if (column.defaultFn !== undefined) return column.defaultFn();
+    if (column.onUpdateFn !== undefined) return column.onUpdateFn();
 
     // TODO(kyle) is it an invariant that it doesn't get here