Draft
Conversation
|
A preview of e96ad0a is uploaded and can be seen here: ✨ https://burr.dagworks.io/pull/610 ✨ Changes may take a few minutes to propagate. Since this is a preview of production, content with |
added 3 commits
December 23, 2025 15:44
This uses some type-safety to make it more user-friendly. Otherwise we implement a subset of features.
1. Adds type-checking for state updates 2. Cascades schema changes 3. Tracks read/write limitations to enable compile-time type-checks
These ensure appropriate error handling of typescript types i.e. state updates, etc...
9a5b5d8 to
bced2c1
Compare
added 2 commits
December 28, 2025 12:51
This has type-checking based on actions, accrues the types/unions. Has some compatibility in state with graph, etc...
This allows us to create it with zod defaults -- this is less safe, power user mode
bced2c1 to
fa0366e
Compare
added 6 commits
December 28, 2025 21:24
A lot of generics math -- this basically allows the action definitions to work properly.
This is simpler. We will also have streamingAction later. But that's it.
This also does some refactors to make typing cleaner/more centralized -- a lot of complex typing logic.
… any> This ensures we don't bypass typing for now.
See README: 1. We have each one in it's own file 2. We have a simple metadata definition 3. It's easy enough for an LLM to generate/debug
2fbf074 to
d6ab0b4
Compare
State schema is required -- things can be optional
d6ab0b4 to
a701a54
Compare
827738d to
ccf42a7
Compare
added 3 commits
January 3, 2026 13:17
- Add runStep() with 4-phase execution (fork/launch/gather/commit) - Make State.subset() strict (throws on missing keys) - Add defense-in-depth validation at phase boundaries - Remove underscore prefix from private methods (idiomatic TS) - Remove 4 redundant E2E tests, add 5 subset validation unit tests - Application validates inputs (reads), Action validates outputs (result/writes)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[Short description explaining the high-level reason for the pull request]
Changes
How I tested this
Notes
Checklist