rari404
rari404
Hey @paralin, the format CI is set up to auto-push formatting fixes, but this doesn't work for PRs from forks (the GitHub token can't push to external repos). You'll need...
@paralin Don't worry too much about the format CI - it fails on fork PRs because it can't push back to external repos. Once a maintainer approves and merges to...
If you really wanted all green lights, then: ```bash # 1. Generate SDK bun ./packages/sdk/js/script/build.ts # 2. Format everything (including generated SDK) bun run prettier --ignore-unknown --write . # 3....
Would you like me to leave PR open or close it
## Bug Found: State Synchronization Issue While reviewing this PR, I found a **state synchronization bug** that would cause the click-to-copy feature to stop working in certain scenarios. ### The...
I definitely think thats a better long term solution and would be much cleaner. Should prioritize getting the ctrl+s behavior documented.
Created the opentui implementation here: https://github.com/sst/opentui/pull/411 Once that's merged, I'll update this PR to just wire up the `onCopySelection` callback. Closing this for now.
## Status Update The opentui PR #411 has been merged (commit `bd159ef85e5111a41928d16807ae2a51f1d0bce7`), but a new npm version hasn't been published yet. **Current state:** - Latest published: `@opentui/[email protected]` (Dec 11) -...
Updated implementation to use opentui's `onCopySelection` callback from PR #411. **Changes:** - Updated `@opentui/core` and `@opentui/solid` to `0.1.61` - Replaced manual click-to-copy with `renderer.console.onCopySelection` callback - Removed `MouseParser` and console...