-
Notifications
You must be signed in to change notification settings - Fork 6.4k
"Did you know?" start screen tips #5982
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
Conversation
show a tip on the start screen from a list of 101 opencode tips
|
/review |
|
/oc address the comment: Suggestion: The hideTips and enableTips functions are each called once and could be inlined. Additionally, the toggle logic with else could be simplified to a single line, per the style guide preference to avoid else statements and keep things in one function: useKeyboard((evt) => { —- Additionally clean up this code a bit so it follows our style guide |
Co-authored-by: rekram1-node <rekram1-node@users.noreply.github.com>
|
The changes have been applied. The code now:
|
Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: rekram1-node <rekram1-node@users.noreply.github.com> Co-authored-by: Aiden Cline <aidenpcline@gmail.com>
Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: rekram1-node <rekram1-node@users.noreply.github.com> Co-authored-by: Aiden Cline <aidenpcline@gmail.com>
Merged upstream dev changes into my_dev job system branch. Conflicts resolved: - sidebar.tsx: Combined job system UI with upstream TodoItem and MCP count helpers - compaction.ts: Kept InvalidTool + added upstream prompt replacement feature - retry.test.ts: Kept both test suites (socket error + ECONNRESET tests) Upstream features merged: - Prompt stashing (anomalyco#6021) - Did you know tips (anomalyco#5982) - MCP OAuth debug command (anomalyco#5980) - Cloudflare AI Gateway (anomalyco#5174) - Install CLI flags (anomalyco#5885) - Windows CRLF grep fix (anomalyco#5948) - Dialog keybind blocking fix (anomalyco#6017) - Model filtering fix (anomalyco#6053)

Summary
Details
New files:
Features: