Use forked bubbletea with IME fix#13
Conversation
Points Gemfile to twokidsCarl/bubbletea-ruby fork (branch fix/ime-multi-byte-input) which adds internal byte buffering in poll_event to correctly handle CJK/IME multi-byte sequences. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 18 minutes and 48 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
Walkthrough在 Gemfile 中条件性地添加了 Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@Gemfile`:
- Line 7: The Gemfile currently points to the GitHub branch
fix/ime-multi-byte-input (commit f1b64c5d15b38...) for gem "bubbletea", but that
branch declares a native extension (ext/bubbletea/extconf.rb) which causes CI to
fail with Gem::Ext::BuildError; replace the branch reference with a pinned,
known-good commit SHA or a released tag that has been verified to build in CI
(update the gem "bubbletea" entry to use ref: "<verified-commit-sha>"), and add
a brief root-cause note to the PR describing the native-extension build failure
(compilation/environment/dependencies) so we can later investigate the
ext/bubbletea/extconf.rb changes without blocking the merge.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: c8ccd0f0-6e5b-4eb5-b171-394fc22b495e
⛔ Files ignored due to path filters (1)
Gemfile.lockis excluded by!**/*.lock
📒 Files selected for processing (2)
Gemfilelib/claw/version.rb
CI uses precompiled gem from RubyGems (no Go build needed). Local dev uses path to ../bubbletea-ruby fork with IME fix. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Summary
twokidsCarl/bubbletea-rubyfork (branchfix/ime-multi-byte-input) for CI../bubbletea-rubyfor developmentpoll_event) until upstream merges marcoroth/bubbletea-ruby#7Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit
发布说明