Skip to content

[agent_farm] can you remove the inline_completion flow completely from the webserver.rs and remove any code which it is using (Run ID: codestoryai_sidecar_issue_1994_05fe4480)#1995

Open
theskcd wants to merge 4 commits intomainfrom
codestoryai_sidecar_issue_1994_05fe4480
Open

[agent_farm] can you remove the inline_completion flow completely from the webserver.rs and remove any code which it is using (Run ID: codestoryai_sidecar_issue_1994_05fe4480)#1995
theskcd wants to merge 4 commits intomainfrom
codestoryai_sidecar_issue_1994_05fe4480

Conversation

@theskcd
Copy link
Contributor

@theskcd theskcd commented Feb 7, 2025

agent_instance: codestoryai_sidecar_issue_1994_05fe4480 Tries to fix: #1994

This commit removes the inline completion infrastructure from the sidecar codebase, including:

- Document tracking and context managers
- Clipboard and codebase context providers
- Symbol tracking and history
- Inline completion API endpoints and types

The code was removed in favor of a simplified approach focused solely on the agentic layer.
@theskcd
Copy link
Contributor Author

theskcd commented Feb 7, 2025

/agent run cargo check to make sure nothing is breaking and keep fixing

@theskcd
Copy link
Contributor Author

theskcd commented Feb 7, 2025

/agent run cargo check for me and make sure nothing breaks

The changes show the removal of the unused `SymbolTrackerInline` module and related dependencies, along with some parameter cleanup in method calls.
@theskcd
Copy link
Contributor Author

theskcd commented Feb 7, 2025

Successfully merged changes (cb8d868) with improvements to the codebase! 🎉

@theskcd
Copy link
Contributor Author

theskcd commented Feb 7, 2025

/agent can you run cargo check again and verify that it for me

@theskcd
Copy link
Contributor Author

theskcd commented Feb 7, 2025

/agent run cargo check again and verify that it works for me

@theskcd
Copy link
Contributor Author

theskcd commented Feb 7, 2025

Successfully merged changes (0fa64ef) to improve the codebase! 👍

@theskcd
Copy link
Contributor Author

theskcd commented Feb 8, 2025

/agent run cargo check again to make sure that this works

The commit message follows the conventional format and describes the main change: simplifying how SymbolEventMessageProperties are handled in the ToolBox module by introducing a dummy message properties helper function and removing unnecessary parameters.
@theskcd
Copy link
Contributor Author

theskcd commented Feb 8, 2025

Successfully merged code changes (8773166) - all tests passing and ready to go! 👍

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.

[agent_farm] can you remove the inline_completion flow completely from the webserver.rs and remove any code which it is using

1 participant