Skip to content

Merge upstream v0.0.10 + fix installer on Linux#8

Merged
aaditagrawal merged 2 commits intomainfrom
merge/upstream-main-5
Mar 12, 2026
Merged

Merge upstream v0.0.10 + fix installer on Linux#8
aaditagrawal merged 2 commits intomainfrom
merge/upstream-main-5

Conversation

@aaditagrawal
Copy link
Copy Markdown
Owner

@aaditagrawal aaditagrawal commented Mar 11, 2026 •
edited by coderabbitai bot
Loading

Summary

  • Cherry-picks upstream chore(release): prepare v0.0.10 (version bumps across packages)
  • Fixes the installer script silently exiting on non-WSL Linux systems

Installer bug details

detect_os() ended with $IS_WSL && log_info ... — on non-WSL systems this expands to the literal false command. As the last statement in the function, its exit code 1 propagated up, and set -e killed the script silently right after OS detection. Added return 0 to ensure the function always exits cleanly.

Test plan

  • Run installer on Fedora/Debian (non-WSL) Linux — should proceed past OS detection
  • Run installer on WSL — should still show WSL info line
  • Run installer on macOS — should work as before
  • Verify package versions match upstream v0.0.10

Summary by CodeRabbit

  • Chores
    • Version bumped to 0.0.10 across desktop, server, web, and contracts packages.
    • Minor improvements made to the installation script return handling.

github-actions bot and others added 2 commits March 12, 2026 02:23
@github-actions @aaditagrawal
chore(release): prepare v0.0.10
17928cf
@aaditagrawal
fix(installer): prevent silent exit on non-WSL Linux
bcf840b 
detect_os() ended with `$IS_WSL && log_info ...` which expands to
the literal `false` command on non-WSL systems. As the last statement
in the function its exit code 1 propagated, and `set -e` killed the
script silently after OS detection.

Add `return 0` to ensure the function always exits cleanly.
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 11, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c1ad60cd-e464-4438-9a68-7e6117311c71

📥 Commits

Reviewing files that changed from the base of the PR and between b0d18dc and bcf840b.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (5)
  • apps/desktop/package.json
  • apps/server/package.json
  • apps/web/package.json
  • packages/contracts/package.json
  • scripts/install.sh
📝 Walkthrough

Walkthrough

The pull request bumps the application version across four package.json files in the monorepo from 0.0.9 to 0.0.10, and adds an explicit return statement to the detect_os function in the installation script with minor formatting adjustments.

Changes

Cohort / File(s) Summary
Version Bumps
apps/desktop/package.json, apps/server/package.json, apps/web/package.json, packages/contracts/package.json		 Version field updated from 0.0.9 to 0.0.10 across all four packages.
Install Script Update
scripts/install.sh		 Added explicit return 0 statement at the end of the detect_os function and minor formatting adjustments to log_info lines.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Version bumps hop along with care,
From nine to ten, a fresh new air!
Install script returns with pride so clear,
All polished up for the next frontier! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely captures the two main changes: merging upstream v0.0.10 version bumps and fixing a Linux installer bug.
Description check ✅ Passed The description follows the template structure, includes clear summary, detailed problem explanation, and test plan, though it uses checkboxes instead of checking them.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch merge/upstream-main-5

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

@github-actions github-actions bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Mar 11, 2026
@aaditagrawal aaditagrawal merged commit 83d5f9f into main Mar 12, 2026
4 of 5 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Mar 12, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@aaditagrawal