fix(zero): Change strategy for version consts (#2837)

We used to rely on the build replacing these consts. This does not work
when we use `tsx` to run typescript directly. Instead, we will use
`process.env` to set the version consts.

We still replace `process.env.REPLICACHE_VERSION` and
`process.env.ZERO_VERSION` in the build step.

Author: arv

package-lock.json

@@ -1,7 +1,12 @@
-// Filled in by esbuild.
-declare const REPLICACHE_VERSION: string;
+// The env value should be filled in by esbuild.
+
+declare const process: {
+  env: {
+    ['REPLICACHE_VERSION']?: string;
+  };
+};
 
 /**
  * The current version of Replicache.
  */
-export const version: string = REPLICACHE_VERSION;
+export const version: string = process.env.REPLICACHE_VERSION ?? '0.0.0';

packages/replicache/src/version.ts

@@ -55,8 +55,10 @@ function getVersion(name) {
 export function makeDefine(mode = 'unknown') {
   /** @type {Record<string, string>} */
   const define = {
-    ['REPLICACHE_VERSION']: JSON.stringify(getVersion('replicache')),
-    ['ZERO_VERSION']: JSON.stringify(getVersion('zero-client')),
+    ['process.env.REPLICACHE_VERSION']: JSON.stringify(
+      getVersion('replicache'),
+    ),
+    ['process.env.ZERO_VERSION']: JSON.stringify(getVersion('zero')),
     ['TESTING']: 'false',
   };
   if (mode === 'unknown') {

packages/shared/src/build.js

@@ -2,5 +2,5 @@ import {expect, test} from 'vitest';
 import {version} from './version.js';
 
 test('version basics', () => {
-  expect(version).to.match(/^\d+\.\d+\.\d+$/);
+  expect(version).to.match(/^\d+\.\d+\.\d+(\+[a-f0-9]+)?$/);
 });

packages/zero-client/src/client/version.test.ts

@@ -1,6 +1,12 @@
-declare const ZERO_VERSION: string;
+// The env value should be filled in by esbuild.
+
+declare const process: {
+  env: {
+    ['ZERO_VERSION']?: string;
+  };
+};
 
 /**
  * The current version of Zero.
  */
-export const version = ZERO_VERSION;
+export const version = process.env.ZERO_VERSION ?? '0.0.0';