Skip to content

Reorganize TS sdk#3915

Merged
coolreader18 merged 6 commits intomasterfrom
noa/ts-refactor
Feb 3, 2026
Merged

Reorganize TS sdk#3915
coolreader18 merged 6 commits intomasterfrom
noa/ts-refactor

Conversation

@coolreader18
Copy link
Collaborator

Description of Changes

This moves a bunch of stuff from lib back into server and sdk, and removes all but one global variable from the server sdk in preparation for export-based reducer definition.

Expected complexity level and risk

2 - a pretty big refactor, but it's mostly just code movement.

Testing

  • Refactor, so automated tests are sufficient.

@coolreader18 coolreader18 force-pushed the noa/ts-refactor branch 2 times, most recently from 3bdb37e to ba7efc4 Compare December 19, 2025 18:27
@bfops bfops added the release-any To be landed in any release window label Jan 12, 2026
@coolreader18 coolreader18 force-pushed the noa/ts-refactor branch 3 times, most recently from 3f4603c to c6485e0 Compare February 2, 2026 18:43
@coolreader18 coolreader18 changed the base branch from master to noa/optimize-ts-serde February 2, 2026 18:44
Copy link
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.

I went over this with Noa and this aligns with our goals. We should do additional testing of the quickstart once this merges.

@coolreader18 coolreader18 changed the base branch from noa/optimize-ts-serde to master February 3, 2026 03:58
@coolreader18 coolreader18 added this pull request to the merge queue Feb 3, 2026
Merged via the queue into master with commit aa2f70c Feb 3, 2026
28 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-any To be landed in any release window

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants