Skip to content

Commit 37fd8c7

Browse files
Merge branch 'main' into main
2 parents aa2a736 + 83386a4 commit 37fd8c7

File tree

15 files changed

+29
-314
lines changed

15 files changed

+29
-314
lines changed

.github/workflows/general.yml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -322,17 +322,6 @@ jobs:
322322
run: |
323323
uv run pyright
324324
325-
- name: Install wasm-pack
326-
run: npm install -g wasm-pack
327-
328-
- name: Build minijinja WASM bindings
329-
working-directory: ui/app/utils/minijinja
330-
run: wasm-pack build --dev --features console_error_panic_hook
331-
332-
- name: Run minijinja WASM tests
333-
working-directory: ui/app/utils/minijinja
334-
run: wasm-pack test --node --features console_error_panic_hook
335-
336325
- name: Build npm workspace
337326
run: pnpm -r build
338327

@@ -461,7 +450,6 @@ jobs:
461450
R2_ACCESS_KEY_ID: ${{ secrets.R2_ACCESS_KEY_ID }}
462451
R2_SECRET_ACCESS_KEY: ${{ secrets.R2_SECRET_ACCESS_KEY }}
463452

464-
465453
steps:
466454
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
467455
- uses: dtolnay/rust-toolchain@stable

.github/workflows/ui-tests.yml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -36,13 +36,6 @@ jobs:
3636
path: |
3737
./ui/fixtures/s3-fixtures
3838
39-
- name: Setup `wasm-pack`
40-
run: npm install -g wasm-pack
41-
42-
- name: Build minijinja WASM bindings
43-
working-directory: ui/app/utils/minijinja
44-
run: wasm-pack build --features console_error_panic_hook
45-
4639
- name: Build `tensorzero-node`
4740
working-directory: internal/tensorzero-node
4841
run: pnpm build

CONTRIBUTING.md

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -194,10 +194,9 @@ It also requires a one-time build of a WebAssembly module from Rust source code
194194
Here are the steps in order to run or test the UI assuming you have the prerequisites installed and this repository checked out:
195195

196196
1. Install dependencies: `pnpm install`
197-
2. Build the WebAssembly module following instructions in `ui/app/utils/minijinja/README.md`.
198-
3. Build the internal N-API client for TensorZero using `pnpm -r build`. If you have changed your Rust code, you may also have to run `pnpm build-bindings` from `internal/tensorzero-node`.
199-
4. Create a `ui/fixtures/.env` following the `ui/fixtures/.env.example`.
200-
5. Create a `ui/.env` file and set the following environment variables for the server:
197+
2. Build the internal N-API client for TensorZero using `pnpm -r build`. If you have changed your Rust code, you may also have to run `pnpm build-bindings` from `internal/tensorzero-node`.
198+
3. Create a `ui/fixtures/.env` following the `ui/fixtures/.env.example`.
199+
4. Create a `ui/.env` file and set the following environment variables for the server:
201200

202201
```bash
203202
OPENAI_API_KEY=<your-key>
@@ -207,7 +206,7 @@ TENSORZERO_CLICKHOUSE_URL=<your-clickhouse-url> # For testing, set to http://chu
207206
TENSORZERO_UI_CONFIG_PATH=<path-to-config-file> # For testing, set to ./fixtures/config/tensorzero.toml
208207
```
209208

210-
6. Run the dependencies: `docker compose -f ui/fixtures/docker-compose.yml up --build --force-recreate`
209+
5. Run the dependencies: `docker compose -f ui/fixtures/docker-compose.yml up --build --force-recreate`
211210
(you can omit these last 2 flags to skip the build step, but they ensure you're using the latest gateway)
212211

213212
With the dependencies running, you can run the tests with `pnpm ui:test` and the Playwright tests with `pnpm ui:test:e2e`.

Cargo.lock

Lines changed: 20 additions & 88 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ members = [
33
"tensorzero-core",
44
"tensorzero-core/tests/mock-inference-provider",
55
"gateway",
6-
"ui/app/utils/minijinja",
76
"internal/tensorzero-node",
87
"clients/rust",
98
"clients/python",

ui/.dockerignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
**/build
33
**/node_modules
44
README.md
5-
app/utils/minijinja/pkg/
65
**/.env
76
**/.envrc
87
**/target

ui/.prettierignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,4 @@
33
**/playwright-report/
44
**/.react-router/
55
**/test-results/
6-
app/utils/minijinja/pkg/
76
build/

ui/Dockerfile

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -49,17 +49,6 @@ RUN tar -vxf /tmp/packages.tar -C .
4949

5050
RUN pnpm install --frozen-lockfile --prod
5151

52-
# ========== minijinja-build-env ==========
53-
54-
FROM rust:latest AS minijinja-build-env
55-
56-
COPY . /build
57-
58-
WORKDIR /build/ui/app/utils/minijinja
59-
RUN apt-get update && apt-get install -y nodejs npm
60-
RUN npm install -g wasm-pack
61-
RUN wasm-pack build --features console_error_panic_hook
62-
6352
# ========== tensorzero-node-build-env ==========
6453

6554
FROM rust:latest AS tensorzero-node-build-env
@@ -103,8 +92,6 @@ COPY --from=tensorzero-node-build-env /build/internal/tensorzero-node/*linux*.no
10392

10493
COPY . .
10594

106-
COPY --from=minijinja-build-env /build/ui/app/utils/minijinja/pkg ./ui/app/utils/minijinja/pkg
107-
10895
RUN pnpm --filter=tensorzero-ui run build
10996

11097
# ========== ui ==========

ui/README.md

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,11 @@ We provide fixture data for development purposes, but you can also use the UI wi
1818
The instructions below assume you're using the provided setup with fixture data.
1919

2020
1. Build the `evaluations` binary. Run: `cargo build -p evaluations`
21-
2. Build the MiniJinja WASM module. See `./app/utils/minijinja/README.md` for reference.
22-
3. Set the environment variables for the gateway. Create a `.env` file in `fixtures/` with credentials. See `fixtures/.env.example` for reference.
23-
4. Launch the TensorZero Gateway and ClickHouse with `docker compose -f fixtures/docker-compose.yml up`.
24-
5. Set the UI environment variables in the shell (not `.env`). See `./.env.example` for reference.
25-
6. Build the internal N-API client for TensorZero using `pnpm -r build`. If you have changed your Rust code, you may also have to run `pnpm build-bindings` from `internal/tensorzero-node`.
26-
7. Run `pnpm` scripts from the root of the repository to start the app:
21+
2. Set the environment variables for the gateway. Create a `.env` file in `fixtures/` with credentials. See `fixtures/.env.example` for reference.
22+
3. Launch the TensorZero Gateway and ClickHouse with `docker compose -f fixtures/docker-compose.yml up`.
23+
4. Set the UI environment variables in the shell (not `.env`). See `./.env.example` for reference.
24+
5. Build the internal N-API client for TensorZero using `pnpm -r build`. If you have changed your Rust code, you may also have to run `pnpm build-bindings` from `internal/tensorzero-node`.
25+
6. Run `pnpm` scripts from the root of the repository to start the app:
2726
- Run `pnpm install` to install the dependencies.
2827
- Run `pnpm ui:dev` to start the development server.
2928
- You can also run tests with `pnpm ui:test` and Storybook with `pnpm ui:storybook`.

ui/app/utils/config/variant.ts

Lines changed: 0 additions & 34 deletions
This file was deleted.

0 commit comments

Comments
 (0)