Skip to content

Commit 83289a0

Browse files
committed
chore: remove non-engine drivers
1 parent ee1bb10 commit 83289a0

File tree

348 files changed

+775
-35651
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

348 files changed

+775
-35651
lines changed

CLAUDE.md

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -108,11 +108,9 @@ git commit -m "chore(my-pkg): foo bar"
108108
- If `rivetkit` type or DTS builds fail with missing `@rivetkit/*` declarations, run `pnpm build -F rivetkit` from repo root (Turbo build path) before changing TypeScript `paths`.
109109
- Do not add temporary `@rivetkit/*` path aliases in `rivetkit-typescript/packages/rivetkit/tsconfig.json` to work around stale or missing built declarations.
110110

111-
### RivetKit Driver Registry Variants
112-
- Keep `rivetkit-typescript/packages/rivetkit/fixtures/driver-test-suite/registry.ts` as the canonical type anchor for fixtures and test typing.
113-
- Run driver runtime suites through `registry-static.ts` and `registry-dynamic.ts` instead of executing `registry.ts` directly.
114-
- Load static fixture actors with dynamic ESM `import()` from the `fixtures/driver-test-suite/actors/` directory.
115-
- Skip dynamic registry parity only for the explicit nested dynamic harness gate or missing secure-exec dist, and still treat full static and dynamic compatibility as the target for all normal driver suites.
111+
### RivetKit Test Fixtures
112+
- Keep RivetKit test fixtures scoped to the engine-only runtime.
113+
- Prefer targeted integration tests under `rivetkit-typescript/packages/rivetkit/tests/` over shared multi-driver matrices.
116114

117115
### SQLite Package
118116
- Use `@rivetkit/sqlite` for SQLite WebAssembly support.
@@ -287,7 +285,7 @@ let error_with_meta = ApiRateLimited { limit: 100, reset_at: 1234567890 }.build(
287285

288286
**Inspector HTTP API**
289287
- When updating the WebSocket inspector (`rivetkit-typescript/packages/rivetkit/src/inspector/`), also update the HTTP inspector endpoints in `rivetkit-typescript/packages/rivetkit/src/actor/router.ts`. The HTTP API mirrors the WebSocket inspector for agent-based debugging.
290-
- When adding or modifying inspector endpoints, also update the driver test at `rivetkit-typescript/packages/rivetkit/src/driver-test-suite/tests/actor-inspector.ts` to cover all inspector HTTP endpoints.
288+
- When adding or modifying inspector endpoints, also update the relevant RivetKit tests in `rivetkit-typescript/packages/rivetkit/tests/` to cover all inspector HTTP endpoints.
291289
- When adding or modifying inspector endpoints, also update the documentation in `website/src/metadata/skill-base-rivetkit.md` and `website/src/content/docs/actors/debugging.mdx` to keep them in sync.
292290

293291
**Database Usage**

README.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,6 @@ Works with Claude Code, Cursor, Windsurf, and other AI coding tools.
221221
- [Node.js & Bun](https://www.rivet.dev/docs/actors/quickstart/backend)
222222
- [React](https://www.rivet.dev/docs/actors/quickstart/react)
223223
- [Next.js](https://www.rivet.dev/docs/actors/quickstart/next-js)
224-
- [Cloudflare Workers](https://www.rivet.dev/docs/actors/quickstart/cloudflare-workers)
225224

226225
[View documentation →](https://www.rivet.dev/docs)
227226

@@ -237,7 +236,7 @@ Serverless, containers, or your own servers — Rivet Actors work with your exis
237236

238237
**Frameworks**: [React](https://www.rivet.dev/docs/clients/react)[Next.js](https://www.rivet.dev/docs/clients/next-js)[Hono](https://github.com/rivet-dev/rivet/tree/main/examples/hono)[Express](https://github.com/rivet-dev/rivet/tree/main/examples/express)[Elysia](https://github.com/rivet-dev/rivet/tree/main/examples/elysia)[tRPC](https://github.com/rivet-dev/rivet/tree/main/examples/trpc)
239238

240-
**Runtimes**: [Node.js](https://www.rivet.dev/docs/actors/quickstart/backend)[Bun](https://www.rivet.dev/docs/actors/quickstart/backend)[Deno](https://github.com/rivet-dev/rivet/tree/main/examples/deno)[Cloudflare Workers](https://www.rivet.dev/docs/actors/quickstart/cloudflare-workers)
239+
**Runtimes**: [Node.js](https://www.rivet.dev/docs/actors/quickstart/backend)[Bun](https://www.rivet.dev/docs/actors/quickstart/backend)[Deno](https://github.com/rivet-dev/rivet/tree/main/examples/deno)
241240

242241
**Tools**: [Vitest](https://www.rivet.dev/docs/actors/testing)[Pino](https://www.rivet.dev/docs/general/logging)[AI SDK](https://github.com/rivet-dev/rivet/tree/main/examples/ai-agent)[OpenAPI](https://github.com/rivet-dev/rivet/tree/main/rivetkit-openapi)[AsyncAPI](https://github.com/rivet-dev/rivet/tree/main/rivetkit-asyncapi)
243242

@@ -270,4 +269,4 @@ Serverless, containers, or your own servers — Rivet Actors work with your exis
270269

271270
## License
272271

273-
[Apache 2.0](LICENSE)
272+
[Apache 2.0](LICENSE)

docs-internal/rivetkit-typescript/DYNAMIC_ACTORS_ARCHITECTURE.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,14 +7,14 @@ Dynamic actors let a registry entry resolve actor source code at actor start tim
77
Dynamic actors are represented by `dynamicActor({ load, auth?, options? })`
88
and still participate in normal registry routing and actor lifecycle.
99

10-
Driver parity is verified by running the same driver test suites against two
11-
fixture registries:
10+
Dynamic actor parity is verified by running the same engine-focused integration
11+
tests against two fixture registries:
1212

13-
- `fixtures/driver-test-suite/registry-static.ts`
14-
- `fixtures/driver-test-suite/registry-dynamic.ts`
13+
- `examples/sandbox/src/index.ts` for shared actor behavior
14+
- dedicated static and dynamic registry fixtures in test coverage
1515

16-
Both registries are built from `fixtures/driver-test-suite/actors/` to keep
17-
actor behavior consistent between static and dynamic execution.
16+
The shared actor fixtures keep behavior consistent between static and dynamic
17+
execution.
1818

1919
## Main Components
2020

docs-internal/rivetkit-typescript/DYNAMIC_ACTOR_FAILED_START_RELOAD_SPEC.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -424,7 +424,7 @@ export interface DynamicStartupOptions {
424424
### Backoff Implementation
425425

426426
Reuse the `p-retry` exponential backoff algorithm that is already used in
427-
`remote-manager-driver/metadata.ts` and `client/actor-conn.ts`. The
427+
`engine-client/metadata.ts` and `client/actor-conn.ts`. The
428428
implementation does not need to use `p-retry` directly (since retries are
429429
passive, not loop-driven), but must compute backoff delays using the same
430430
formula: `min(maxDelay, initialDelay * multiplier^attempt)` with optional
@@ -658,9 +658,9 @@ Comments should explain intent and invariants, not implementation history.
658658

659659
## Test Requirements
660660

661-
All failed-start tests must be added to the shared driver-test-suite
662-
(`src/driver-test-suite/`) so both file-system and engine drivers run the same
663-
test cases. This enforces the parity requirement.
661+
All failed-start tests must be added to the shared engine-focused integration
662+
suite so the runtime path uses one common set of cases. This enforces the
663+
parity requirement.
664664

665665
Add or update tests for:
666666

docs-internal/rivetkit-typescript/DYNAMIC_ACTOR_SQLITE_PROXY_SPEC.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -244,15 +244,15 @@ All data crosses the bridge as JSON strings:
244244

245245
## Testing
246246

247-
Add a driver test in `src/driver-test-suite/tests/` that:
247+
Add an engine-focused integration test that:
248248

249249
1. Creates a dynamic actor that uses `db()` (raw) with a simple schema
250250
2. Runs migrations, inserts rows, queries them back
251251
3. Verifies data persists across actor sleep/wake cycles
252252
4. Creates a dynamic actor that uses `db()` from `rivetkit/db/drizzle` with schema + migrations
253253
5. Verifies drizzle queries work through the proxy
254254

255-
Add corresponding fixture actors in `fixtures/driver-test-suite/`.
255+
Add corresponding fixture actors in the shared sandbox-style test fixtures.
256256

257257
## Files to modify
258258

@@ -263,8 +263,8 @@ Add corresponding fixture actors in `fixtures/driver-test-suite/`.
263263
| `src/dynamic/isolate-runtime.ts` | Wire `sqliteExec`/`sqliteBatch` refs in `#setIsolateBridge()` |
264264
| `src/dynamic/host-runtime.ts` | Wire bridge refs + add `overrideRawDatabaseClient` to isolate-side `actorDriver` |
265265
| `src/db/drizzle/mod.ts` | Add override check at top of `createClient` |
266-
| `src/driver-test-suite/tests/` | New test file for dynamic SQLite proxy |
267-
| `fixtures/driver-test-suite/` | New fixture actors using `db()` in dynamic actors |
266+
| `tests/` | New engine-focused integration test for dynamic SQLite proxy |
267+
| shared test fixtures | New fixture actors using `db()` in dynamic actors |
268268
| `docs-internal/rivetkit-typescript/DYNAMIC_ACTORS_ARCHITECTURE.md` | Document SQLite proxy bridge |
269269

270270
## Non-goals

examples/actor-actions/vite.config.ts

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

examples/cloudflare-workers-hono/README.md

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

examples/cloudflare-workers-hono/package.json

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

examples/cloudflare-workers-hono/scripts/client.ts

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

examples/cloudflare-workers-hono/src/index.ts

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

0 commit comments

Comments
 (0)