search

intorust / intorust.vscode

A VSCode extension for learning Rust
Other
9 stars 2 forks source link

Carry context in response to user messages #2

Open nikomatsakis opened 3 months ago

nikomatsakis commented 3 months ago

The followUp function is providing zero context:

https://github.com/nikomatsakis/intorust.vscode/blob/ffff983688652133fe4a5fc80221fba683a6e6f5/src/advisor.ts#L25-L36

This results in amusingly bad responses. e.g., I asked how to convert a number into a string and it talked about Python.

kwdev commented 3 months ago

I updated this a little bit by adding a user prompt in the follow-up to keep it Rust-focused. https://github.com/intorust/intorust.vscode/pull/17

kwdev commented 3 months ago

Implemented very basic context in https://github.com/intorust/intorust.vscode/pull/18 It does not yet summarize or pay attention to the size of the context growing yet.

kwdev commented 3 months ago

Ensure token size stays within bounds for the model - https://github.com/intorust/intorust.vscode/pull/19