Skip to content

chore(platform): improve local Convex dev setup#424

Merged
Israeltheminer merged 1 commit into
mainfrom
chore/dev-setup-improvements
Feb 10, 2026
Merged

chore(platform): improve local Convex dev setup#424
Israeltheminer merged 1 commit into
mainfrom
chore/dev-setup-improvements

Conversation

@Israeltheminer

@Israeltheminer Israeltheminer commented Feb 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Improves dev.mjs script to reuse anonymous Convex deployments for local development

Test plan

  • npm run dev --workspace=@tale/platform starts correctly with Convex local backend

Summary by CodeRabbit

  • Bug Fixes

    • Fixed unnecessary recreation of local deployments on every development run.
  • Refactor

    • Improved local deployment environment handling to preserve state across development restarts.
    • Enhanced deployment state management for faster backend reconnection.

Reuse anonymous Convex deployments across restarts instead of creating a
new one each time. Remove unused `runConvexEnvRemove` helper.
@Israeltheminer Israeltheminer force-pushed the chore/dev-setup-improvements branch from f93940b to 10113ad Compare February 10, 2026 16:48
@coderabbitai

coderabbitai Bot commented Feb 10, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The changes refine how CONVEX_DEPLOYMENT is handled during local development in the Convex dev script. The implementation now detects whether the current deployment is a local anonymous deployment (identified by "anonymous:" prefix) and preserves it across restarts instead of clearing it. Additionally, the environment setup is modified to only set CONVEX_AGENT_MODE = 'anonymous' when no local deployment exists, and the constructed environment is passed directly to the Convex dev process rather than naively overriding spawn environment variables.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/dev-setup-improvements

Comment @coderabbitai help to get the list of available commands and usage tips.

@Israeltheminer Israeltheminer merged commit 9e03300 into main Feb 10, 2026
7 of 8 checks passed
@Israeltheminer Israeltheminer deleted the chore/dev-setup-improvements branch February 10, 2026 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant