fix: bundle @agent-relay/cloud + add Sign-in button + ignore .workflow-artifacts#6
Conversation
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ 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: Organization UI Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (3)
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
…w-artifacts Three small, atomic fixes that surfaced while wiring the cloud-agent UI: - electron.vite.config.ts: add @agent-relay/cloud to externalizeDepsPlugin exclude so it gets bundled into the CJS main process. The package's exports map only defines `import`/`types` conditions (no `require`/`default`), so leaving it externalized caused `ERR_PACKAGE_PATH_NOT_EXPORTED` on app load. - src/renderer/src/components/sidebar/Sidebar.tsx: UserMenu rendered null when logged out and contained no login affordance, so signed-out users had no way into Account Settings. Add a Sign-in button (icon-only when collapsed) that runs pear.auth.login() and refreshes the avatar via pear.auth.status(). - .gitignore: ignore .workflow-artifacts/. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
khaliqgant
force-pushed
the
fix/bundle-and-signin
branch
from
ddaf314 to
e2f52f3
Compare
…lush Fix #1/#13: initIfReady schedules a requestAnimationFrame retry when the container has no layout (hidden tab, split page mounted at display:none). mount() now always invokes initIfReady regardless of prior parentage so the retry actually fires once the container becomes visible. Fix #6: dispose() cancels the init rAF and calls clearPtyBuffer(key) before flipping the disposed flag, so a queued rAF flush can't write into a torn-down xterm. writeFromBuffer also early-returns when disposed as belt-and-braces.
…lush Fix #1/#13: initIfReady schedules a requestAnimationFrame retry when the container has no layout (hidden tab, split page mounted at display:none). mount() now always invokes initIfReady regardless of prior parentage so the retry actually fires once the container becomes visible. Fix #6: dispose() cancels the init rAF and calls clearPtyBuffer(key) before flipping the disposed flag, so a queued rAF flush can't write into a torn-down xterm. writeFromBuffer also early-returns when disposed as belt-and-braces.
1 participant
Three small, atomic fixes that surfaced while wiring the cloud-agent UI.
1. Bundle
@agent-relay/cloud(electron.vite.config.ts)externalizeDepsPluginleft the package as a runtimerequire(). Itsexportsmap only definesimport/typesconditions — norequire/default— so Node's CJS resolution threwERR_PACKAGE_PATH_NOT_EXPORTEDon app load. Adding it toexcludebundles it (same treatment@agent-relay/sdkalready gets).2. Sign-in button in
Sidebar.tsxUserMenureturnednullwhen logged out and had no login affordance, so signed-out users couldn't reach Account Settings at all. Adds a button (icon-only when collapsed) that runspear.auth.login()and re-fetchesauth.status()on success.3.
.gitignore.workflow-artifacts/Ricky and friends leave runtime artifacts here; ignore the directory.
🤖 Generated with Claude Code