Skip to content

Remove initialize - #88

Draft
cafalchio wants to merge 9 commits into
mainfrom
remove_initialize
Draft

Remove initialize#88
cafalchio wants to merge 9 commits into
mainfrom
remove_initialize

Conversation

@cafalchio

Copy link
Copy Markdown
Contributor

Initial phase of remove initialize.

  • removed initialize
  • added a stripped version of initialized per call connect_backend_for_request
  • removed session_id from validate
  • removed merge and build capabilities
  • Added tool call for protocol
  • tested tool call using new protocol

Ignore tests for now.

image

@cafalchio
cafalchio requested a review from dawid-nowak August 11, 2026 18:33
@cafalchio
cafalchio marked this pull request as draft August 11, 2026 21:14
@dawid-nowak

Copy link
Copy Markdown
Contributor

This is fine but:

  1. I would expect the initialize to be gone and return Error when the initialize is somehow needed.
  2. We should remove the all traces of connection management etc. we will not need it.

Signed-off-by: cafalchio <mcafalchio@gmail.com>
Signed-off-by: cafalchio <mcafalchio@gmail.com>
Signed-off-by: cafalchio <mcafalchio@gmail.com>
Signed-off-by: cafalchio <mcafalchio@gmail.com>
Signed-off-by: cafalchio <mcafalchio@gmail.com>
Signed-off-by: cafalchio <mcafalchio@gmail.com>
Signed-off-by: cafalchio <mcafalchio@gmail.com>
Signed-off-by: cafalchio <mcafalchio@gmail.com>
Signed-off-by: cafalchio <mcafalchio@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.

2 participants