[AI-BUILDER-21] refactor: combine chat and readiness check#1444
Merged
kevinkim-ogp merged 5 commits intotrunk/aifrom Mar 10, 2026
Merged
[AI-BUILDER-21] refactor: combine chat and readiness check#1444kevinkim-ogp merged 5 commits intotrunk/aifrom
kevinkim-ogp merged 5 commits intotrunk/aifrom
Conversation
This was referenced Mar 4, 2026
kevinkim-ogp
changed the title
This was referenced Mar 4, 2026
Contributor
Author
This was referenced Mar 4, 2026
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
This comment has been minimized.
This comment has been minimized.
kevinkim-ogp
force-pushed
the
ai-builder/refactor-feature-flag
branch
from
e933200 to
d77b9f3
Compare
kevinkim-ogp
force-pushed
the
ai-builder/combine-chat-and-readiness
branch
from
dac773f to
c355b2b
Compare
6 tasks
kevinkim-ogp
force-pushed
the
ai-builder/combine-chat-and-readiness
branch
from
c355b2b to
f44f443
Compare
kevinkim-ogp
force-pushed
the
ai-builder/refactor-feature-flag
branch
from
d77b9f3 to
db89e40
Compare
This was referenced Mar 6, 2026
This was referenced Mar 10, 2026
Contributor
Author
Merge activity
|
kevinkim-ogp
changed the base branch from
ai-builder/refactor-feature-flag
to
graphite-base/1444
kevinkim-ogp
force-pushed
the
ai-builder/combine-chat-and-readiness
branch
from
f44f443 to
d0a45d5
Compare
kevinkim-ogp
added a commit
that referenced
this pull request
Mar 10, 2026
### TL;DR Combined chat readiness check with the chat streaming API. ### What changed? - Removed `getChatReadiness` GraphQL query and resolver - Added `chatReadinessModel` configuration to feature flags - Implemented chat readiness checking within the streaming chat API using `createUIMessageStream` - Added `getChatReadiness` function - Created stream utilities for piping web responses to Express - Updated frontend to extract readiness status from streaming data annotations instead of separate GraphQL calls - Added comprehensive test coverage for error handling scenarios ### How to test? 1. Start a chat conversation in the AI Builder 2. Send a message and verify the response streams correctly - [x] There should no longer be additional call to get the chat readiness, it now comes at the end of the chat stream. - [ ] Verify that 'Preview steps' button is shown when the chat is ready to generate steps. - [ ] Verify that readiness status resets to false when sending new messages. ### Why make this change? This prepares for the next PR where we automatically start previewing the steps without user clicking on the button.
This was referenced Mar 11, 2026
This was referenced Mar 18, 2026
kevinkim-ogp
added a commit
that referenced
this pull request
Mar 26, 2026
### TL;DR Combined chat readiness check with the chat streaming API. ### What changed? - Removed `getChatReadiness` GraphQL query and resolver - Added `chatReadinessModel` configuration to feature flags - Implemented chat readiness checking within the streaming chat API using `createUIMessageStream` - Added `getChatReadiness` function - Created stream utilities for piping web responses to Express - Updated frontend to extract readiness status from streaming data annotations instead of separate GraphQL calls - Added comprehensive test coverage for error handling scenarios ### How to test? 1. Start a chat conversation in the AI Builder 2. Send a message and verify the response streams correctly - [x] There should no longer be additional call to get the chat readiness, it now comes at the end of the chat stream. - [ ] Verify that 'Preview steps' button is shown when the chat is ready to generate steps. - [ ] Verify that readiness status resets to false when sending new messages. ### Why make this change? This prepares for the next PR where we automatically start previewing the steps without user clicking on the button.
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
2 participants
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.
TL;DR
Combined chat readiness check with the chat streaming API.
What changed?
getChatReadinessGraphQL query and resolverchatReadinessModelconfiguration to feature flagscreateUIMessageStreamgetChatReadinessfunctionHow to test?
Why make this change?
This prepares for the next PR where we automatically start previewing the steps without user clicking on the button.