-
Notifications
You must be signed in to change notification settings - Fork 22
Add ability to iterate on a TUI (like v0.dev) #7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I'm working on something to address #3, do you have any preferences in how to do it? I was thinking instead of saving the script to
When termite is called, it could create a dto a la: @dataclass
class Project:
_metadata_path: Path
_tui_path: Path
@property
def script(self) -> str:
...
@property
def metadata(self) -> Metadata:
...
@staticmethod
def from_dir() -> "Project":
...
@staticmethod
def create(script, design) -> "Project":
... Then metadata can contain converstations and context from past runs. If the user calls To run a tool, perhaps Let me know what you think, I'm happy to implement this. Cheers! |
This is a good idea. Only thing I'm wondering is how the actual UI should look for iterating on a TUI. In v0.dev, the chat is side-by-side with the output. That might not be so doable in the terminal. Any ideas? |
It's a good point. In this application, I suppose you don't really care to see what the LLM has to say, only what the TUI looks like. So maybe the workflow is:
User presses
Then it jumps back to loading bars / showing the TUI. Thoughts? On the next repitition it might show the same prompt:
etc etc |
That makes sense. How do we guarantee that pressing |
That I do not know 😁 It might be easier to just have Ctrl+C exit the TUI, we catch the KeyboardException in the main chat loop, and then go back to prompting. If they then Ctrl+C while in the chat loop, it exits the program. That's nice from a UX standpoint, it keeps the "exit" mechanics the same. If you like the general proposal, I can play around with implementations and see if I can experiment with possible ways to exit the TUI reliably. |
That seems clean and simple. We should make sure that Ctrl+C and |
Also, how do you think issue #6 should be supported in this new workflow? Maybe just a simple Y/n input prompt before opening the TUI? |
Hello there :) First, congrats and thanks for sharing this pretty exciting tool 🙏 After giving it a few tries, this is indeed the first feature I thought I'd like because it is not that simple to drop the perfect prompt the first time and of course, each new "answer" seeds new ideas... and bugs 😅 The second, which is related to things discussed here is that I wished Termite would:
TY |
I'd merge any PR that allows iteration on the TUI. And as for your other ideas, those also sound good to me, although much lower priority. |
I dunno about you guys, but I'm thinking that the newest "thinking" models might replace an agentic approach.... I may just have to try a stab at PR myself |
I wouldn't call Termite an agent, but yes I think using a reasoning model would improve results. @chrisdlees If you give this a go, I'd recommend using LiteLLM. |
No description provided.
The text was updated successfully, but these errors were encountered: