Upgrade local dev to use electric beta.15 #1699
Conversation
|
WalkthroughThe changes update the Redis configuration in test cases to use an explicit object with host, port, and a Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant Server
participant TaskHandler
Client->>Server: Trigger example task
Server->>TaskHandler: Execute tasks.trigger()
TaskHandler-->>Server: Return handle with publicAccessToken
Server-->>Client: Set cookie using publicAccessToken
Possibly related PRs
Poem
Warning There were issues while running some tools. Please review the errors and either fix the tool’s configuration or disable the tool if it’s a critical failure. 🔧 ESLint
internal-packages/testcontainers/src/utils.tsOops! Something went wrong! :( ESLint: 8.45.0 ESLint couldn't find the config "custom" to extend from. Please check that the name of the config is correct. The config "custom" was referenced from the config file in "/.eslintrc.js". If you still have problems, please stop by https://eslint.org/chat/help to chat with the team. references/nextjs-realtime/src/app/actions.tsOops! Something went wrong! :( ESLint: 8.45.0 ESLint couldn't find the config "custom" to extend from. Please check that the name of the config is correct. The config "custom" was referenced from the config file in "/.eslintrc.js". If you still have problems, please stop by https://eslint.org/chat/help to chat with the team. apps/webapp/test/realtimeClient.test.tsOops! Something went wrong! :( ESLint: 8.45.0 ESLint couldn't find the config "custom" to extend from. Please check that the name of the config is correct. The config "custom" was referenced from the config file in "/.eslintrc.js". If you still have problems, please stop by https://eslint.org/chat/help to chat with the team. ✨ Finishing Touches
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
apps/webapp/test/realtimeClient.test.ts (1)
13-17: LGTM! Redis configuration is now more explicit.Breaking down the Redis configuration into explicit properties improves clarity and maintainability.
Consider adding a comment about TLS being disabled.
While disabling TLS is appropriate for local development/testing, it would be good to add a comment explaining this to prevent confusion.
redis: { host: redis.options.host, port: redis.options.port, + // TLS disabled for local development/testing only tlsDisabled: true, },Also applies to: 153-157, 236-240
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
apps/webapp/test/realtimeClient.test.ts(3 hunks)docker/docker-compose.yml(1 hunks)internal-packages/testcontainers/src/utils.ts(1 hunks)references/nextjs-realtime/src/app/actions.ts(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- internal-packages/testcontainers/src/utils.ts
🔇 Additional comments (2)
docker/docker-compose.yml (1)
143-143: Verify the stability of Electric SQL beta.15.The upgrade from beta.1 to beta.15 is a significant version jump that might introduce breaking changes.
❌ Verification inconclusive
🌐 Web query:
What are the breaking changes in ElectricSQL 1.0.0-beta.15 compared to beta.1?Error during web search
Action required: Manually verify ElectricSQL beta.15 stability.
The web query to determine breaking changes in ElectricSQL 1.0.0-beta.15 returned an error. Given the significant version jump from beta.1 to beta.15, please review the official release notes or documentation for ElectricSQL and confirm with the vendor about potential breaking changes that could affect your deployment.
- Check the ElectricSQL documentation/release notes for beta.15-specific changes.
- Confirm stability and any breaking changes with the vendor or trusted community channels.
references/nextjs-realtime/src/app/actions.ts (1)
15-15: LGTM! Token management simplified.The direct use of
handle.publicAccessTokenreduces complexity by eliminating the intermediate token creation step.
* Uses image cards for the frameworks * Removes old snippets * New AI agents side menu section * WIP adding new ai agent pages * Better overview page * More copy added to the agent example pages * Copy improvements * Removes “Creating a project” page and side menu section * Fixes broken links * Updates to the latest Mintlify version, fixes issues, changes theme * Adds descriptions to the main dropdown menu items * Reformatted Introduction docs ‘landing page’ * Retry heartbeat timeouts by putting back in the queue (#1689) * If there’s a heartbeat error and no attempts we put it back in the queue to try again * When nacking, return whether it was put back in the queue or not * Try and nack, if it fails then fail the run * Consolidated switch statement * Fail executing/retrying runs * OOM retrying on larger machines (#1691) * OOM retrying on larger machines * Create forty-windows-shop.md * Update forty-windows-shop.md * Only retry again if the machine is different from the original * Kubernetes OOMs appear as non-zero sigkills, adding support for treating these as OOMs * Complete the original attempt span if retrying due to an OOM * Revert "Complete the original attempt span if retrying due to an OOM" This reverts commit 5f652c6. * chore: Update version for release (#1666) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * Release 3.3.14 * Set machine when triggering docs * Batch queue runs that are waiting for deploy (#1693) * Detect ffmpeg OOM errors, added manual OutOfMemoryError (#1694) * Detect ffmpeg OOM errors, added manual OutOfMemoryError * Create eighty-spies-knock.md * Improved the machines docs, including the new OutOfMemoryError * chore: Update version for release (#1695) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * Release 3.3.15 * Create new partitioned TaskEvent table, and switch to it gradually as new runs are created (#1696) * Create new partitioned TaskEvent table, and switch to it gradually as new runs are created * Add env var for partition window in seconds * Make startCreatedAt required in task event store * Don't create an attempt if the run is final, batchTriggerAndWait bad continue fix (#1698) * WIP fix for ResumeAttemptService selecting the wrong attempt (which has no error or output) * Don’t create an attempt if the run is already in a final status * Don’t get all the columns for the query. Improved the logging. * Added a log to the batch example * Filter out the undefined values * Fix missing logs on child runs by using the root task run createdAt if it exists (#1697) * Provider changes to support image cache (#1700) * add env var for additional pull secrets * make static images configurable * optional image prefixes * optional labels with sample rates * add missing core paths * remove excessive logs * Fix run container exits after OOM retries (#1701) * remove unused imports * tell run to exit before force requeue * handle exit for case where we already retried after oom * improve retry span and add machine props * don't try to exit run in dev * Upgrade local dev to use electric beta.15 (#1699) * Text fixes * Removed pnpm files --------- Co-authored-by: Matt Aitken <matt@mattaitken.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Eric Allam <eric@trigger.dev> Co-authored-by: nicktrn <55853254+nicktrn@users.noreply.github.com>
Summary by CodeRabbit