Skip to content

Import TypeScript SDK repo#2941

Merged
bfops merged 166 commits intomasterfrom
bfops/import-typescript-sdk
Jul 16, 2025
Merged

Import TypeScript SDK repo#2941
bfops merged 166 commits intomasterfrom
bfops/import-typescript-sdk

Conversation

@bfops
Copy link
Copy Markdown
Collaborator

@bfops bfops commented Jul 16, 2025

Description of Changes

I just ran the most mechanical version possible:

git clone git@github.com:clockworklabs/spacetimedb-typescript-sdk temp-spacetimedb-typescript-sdk
cd temp-spacetimedb-typescript-sdk
git-filter-repo --to-subdirectory-filter sdks/typescript
cd ../SpacetimeDB
git remote add temp-typescript-sdk ../temp-spacetimedb-typescript-sdk
git fetch temp-typescript-sdk
git checkout -b bfops/import-typescript-sdk origin/master
git merge temp-typescript-sdk/main --allow-unrelated-histories

This should be merged with a merge commit, for two reasons:

  1. To maintain the commit history of the TypeScript repo (a squash commit would make all of the history prunable since it wouldn't be relevant anymore)
  2. So that migrated branches + PRs from the TypeScript repo can merge in master without getting horrible merge conflicts (a squash commit would essentially newly create all of those files, causing merge conflicts)

Because of that, I think we should skip the merge queue to merge this PR (so that we can use a merge commit). It's entirely new files, so it should not cause any issues for other PRs.

API and ABI breaking changes

No breaking changes

Expected complexity level and risk

1

Testing

None

Followup work

jdetter and others added 24 commits March 4, 2025 13:40
Co-authored-by: John Detter <no-reply@boppygames.gg>
* Fix some subscription lifecycle issues.

* remove the subscription after an error.
Co-authored-by: John Detter <no-reply@boppygames.gg>
Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
* Make ws connection fail when token is not verified

* add changeset

---------

Co-authored-by: Jeffrey Dallatezza <jeffreydallatezza@gmail.com>
* Allow spacetimedb sdk to work when spacetimedb url is more than a host name.

* run pnpm format

---------

Co-authored-by: DeveloperChaseLewis <developer.chase.lewis@gmail.com>
* Run lint for PRs.

* Fix dir

* Also hide changes to generated test app files.

* Only run on pushes to main.
Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
* Version Packages

* revert major version bump

* changelog

* fix changelog

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
* Add semver parsing.

* Add a minimum version check.

* Use qmark syntax.

* Regen quickstart-chat bindings with 1.2.0 cli

* Bump minimum codegen version.

* Regen test app with 1.2
* Test quickstart-chat in CI

* fix?

* ignore diffs containing SpacetimeDB commit

* prettier config

* fix?

* [REVERT THIS TEST COMMIT] add diff to module bindings

* revert previous test commit

* [REVERT THIS TEST COMMIT] introduce a typechecking error

* revert previous test commit

* tiny review

---------

Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
* Get it working with base64 encoded row ids, and start adding the better way for indexing.

* Use concat to avoid creating too many arguments for big updates.

* Use primitives as rowIds

* Remove OperationsMap

* Add some table cache tests and regen stuff.

* Cleanup

* revert accidental cargo change.

* Add semver parsing.

* Add a minimum version check.

* Use qmark syntax.

* Regen quickstart-chat bindings with 1.2.0 cli

* Bump minimum codegen version.

* Regen test app with 1.2
Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
…escript-sdk/main' into bfops/import-typescript-sdk
@bfops bfops marked this pull request as ready for review July 16, 2025 14:43
@bfops bfops requested a review from cloutiertyler as a code owner July 16, 2025 14:43
Copy link
Copy Markdown
Contributor

@cloutiertyler cloutiertyler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Exciting stuff!

@bfops bfops merged commit 88dc369 into master Jul 16, 2025
21 checks passed
@bfops bfops deleted the bfops/import-typescript-sdk branch July 16, 2025 15:16
mamcx added a commit that referenced this pull request Aug 26, 2025
Co-authored-by: Piotr Sarnacki <drogus@gmail.com>
Co-authored-by: John <no-reply@boppygames.gg>
Co-authored-by: NateTheDev1 <nthnlrichards@gmail.com>
Co-authored-by: Nathaniel Richards <46858886+NateTheDev1@users.noreply.github.com>
Co-authored-by: dbrinkmanncw <109690865+dbrinkmanncw@users.noreply.github.com>
Co-authored-by: Derek Brinkmann <dbrinkmann@citadelstudios.net>
Co-authored-by: John Detter <4099508+jdetter@users.noreply.github.com>
Co-authored-by: Mario Montoya <mamcx@elmalabarista.com>
Co-authored-by: Kamil Jakubus <kamil.jakubus@usagi.coffee>
Co-authored-by: Derek Brinkmann <dbrinkmann@clockworklabs.io>
Co-authored-by: Phoebe Goldman <phoebe@goldman-tribe.org>
Co-authored-by: HSReina <hontoshadow@shadosoft-tm.com>
Co-authored-by: Gérald Divoux <gerald.divoux@ninsight.io>
Co-authored-by: Zeke Foppa <github.com/bfops>
Co-authored-by: Mazdak Farrokhzad <twingoow@gmail.com>
Co-authored-by: Halu <35203441+tcardlab@users.noreply.github.com>
Co-authored-by: Puru Vijay <47742487+PuruVJ@users.noreply.github.com>
Co-authored-by: Tyler Cloutier <cloutiertyler@users.noreply.github.com>
Co-authored-by: Puru Vijay <awesomepuruvj@gmail.com>
Co-authored-by: Tyler Cloutier <cloutiertyler@aol.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: james gilles <jameshgilles@gmail.com>
Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
Co-authored-by: Noa <coolreader18@gmail.com>
Co-authored-by: Jeffrey Dallatezza <jeffreydallatezza@gmail.com>
Co-authored-by: Bastian Ganze <bastianganze@gmail.com>
Co-authored-by: ChaseLewis <ChaseRLewis73003@gmail.com>
Co-authored-by: DeveloperChaseLewis <developer.chase.lewis@gmail.com>
Co-authored-by: Chip <36650721+Lethalchip@users.noreply.github.com>
Co-authored-by: Julien Lavocat <julien.lavocat@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.