Skip to content

Removing session management from ContextForge dataplane and making it stateless - #113

Merged
dawid-nowak merged 12 commits into
mainfrom
dawid.nowak/removal_of_session_management
Aug 26, 2026
Merged

Removing session management from ContextForge dataplane and making it stateless#113
dawid-nowak merged 12 commits into
mainfrom
dawid.nowak/removal_of_session_management

Conversation

@dawid-nowak

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Dawid Nowak <nowakd@gmail.com>
Signed-off-by: Dawid Nowak <nowakd@gmail.com>
…sources

Signed-off-by: Dawid Nowak <nowakd@gmail.com>
Signed-off-by: Dawid Nowak <nowakd@gmail.com>
…er necessary and will change

Signed-off-by: Dawid Nowak <nowakd@gmail.com>
…er necessary and will change.2

Signed-off-by: Dawid Nowak <nowakd@gmail.com>
Signed-off-by: Dawid Nowak <nowakd@gmail.com>
Signed-off-by: Dawid Nowak <nowakd@gmail.com>
@gandhipratik203

Copy link
Copy Markdown
Contributor

This PR changes the gateway’s request/session model. Should we update the relevant wiki/book page here so that they are in sync, or are you planning to cover the stateless flow docs in a follow-up PR?

Comment thread crates/contextforge-data-plane-apis/src/user_store.rs

@gandhipratik203 gandhipratik203 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for removing the session management. The PR looks good! This should enable moving to the new version of the protocol. A few inline comments.

lucarlig
lucarlig previously approved these changes Aug 25, 2026

@lucarlig lucarlig left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM
only thing would be good if you get bob to update the wiki, otherwise i will follow up PR

Comment thread crates/contextforge-data-plane-lib/src/common.rs
@dawid-nowak

Copy link
Copy Markdown
Contributor Author

If the wiki updates are generated by Bob, then I would wait until we have more code in and do one large update at the end of the epic and have a separate PRs. Not sure how many/if people are following those wikis at the moment and I don't want "code" PR being blocked by wiki updates.

Comment thread crates/contextforge-data-plane-lib/src/gateway/backend_client.rs
…p validation

Signed-off-by: Dawid Nowak <nowakd@gmail.com>
Signed-off-by: Dawid Nowak <nowakd@gmail.com>
Signed-off-by: Dawid Nowak <nowakd@gmail.com>
Signed-off-by: Dawid Nowak <nowakd@gmail.com>

@gandhipratik203 gandhipratik203 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank for the changes! LGTM!

@dawid-nowak
dawid-nowak merged commit dd65751 into main Aug 26, 2026
10 checks passed
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.

4 participants