Skip to content

opentui: add toast & port flags (-c, -s) - #3425

Merged
rekram1-node merged 9 commits into
anomalyco:opentuifrom
veracioux:feat/port-features-to-opentui
Oct 24, 2025
Merged

rekram1-node merged 9 commits into
anomalyco:opentuifrom
veracioux:feat/port-features-to-opentui

Conversation

@veracioux

@veracioux veracioux commented Oct 24, 2025 •

Copy link
Copy Markdown
Contributor
image image

@rekram1-node

Copy link
Copy Markdown
Collaborator

generally if possible I'd recommend splitting into 2 prs if we can help it

},
})
onSelect: async (dialog) => {
try {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

can we use .catch instead of try catch

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure thing

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

@veracioux
veracioux marked this pull request as draft October 24, 2025 19:45
Comment thread packages/opencode/src/cli/cmd/tui.ts Outdated
}
return undefined
})()
const sessionID = "make-typechecker-happy"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think you should just leave this file as it was we aren't using it anywhere right? so no reason to make edits unless u wanted to delete it?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I wanted to mark stuff that was done. Since there is no such tracking by the other devs rn, I'll remove it before marking this PR as ready. But I'll keep it for now for my own organization.

@rekram1-node

Copy link
Copy Markdown
Collaborator

Yeah I think this is good, if you drop that one file edit I think this is good to merge

@veracioux

Copy link
Copy Markdown
Contributor Author

@rekram1-node Please check the newest commits. Now it's ready to merge. Made these changes:

  • Changed the toast duration to optional with 5s default to match the dev branch
  • Added error handling for invalid sessionID specified in --session, matching dev branch
image

@veracioux
veracioux marked this pull request as ready for review October 24, 2025 21:46
@rekram1-node rekram1-node changed the title Port some features to opentui version opentui: add toast & port flags (-c, -s) Oct 24, 2025
@rekram1-node

Copy link
Copy Markdown
Collaborator

I think this is awesome, great work

@rekram1-node
rekram1-node merged commit a256056 into anomalyco:opentui Oct 24, 2025
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