[Xeonacid]

Cross-compile the static binary with the other release targets and ship the zip plus the npm optional package.

Summary by CodeRabbit

• New Features
  • Added support for Linux and Android devices with RISC-V 64-bit processors, including platform-specific downloads and packages.

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

The build workflow adds Linux RISC-V 64-bit compilation, artifact packaging, and npm publishing. The npm package adds platform metadata, and platform selection now routes Linux and Android on riscv64 to the new binary package.

Changes: Linux RISC-V 64-bit support

Layer / File(s)	Summary
Platform package and selection publish/npm-linux-riscv64/package.json, publish/npm/index.js, publish/npm/package.json	Adds metadata for the Linux RISC-V package. The platform selector and optional dependencies include the package.
Build, release, and publish .github/workflows/build.yml	Adds linux/riscv64 cross-compilation, includes the binary in build and release artifacts, and adds the platform to the npm publishing workflow.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Suggested reviewers: gmegidish

Merge Risk: Minimal · previous review 1ad37

No merge-blocking risk is established for the Linux/riscv64 support changes.

Pre-merge checks: 5

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title "ci: add linux/riscv64 build and publish" matches the main change. The PR adds Linux RISC-V 64-bit cross-compilation to the build workflow, includes the binary in the Linux artifact, creates…
Docstring Coverage		No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title "ci: add linux/riscv64 build and publish" matches the main change. The PR adds Linux RISC-V 64-bit cross-compilation to the build workflow, includes the binary in the Linux artifact, creates a release archive, and publishes a new npm platform package (@mobilenext/mobilecli-linux-riscv64) as an optional dependency. The title is concise and specific.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 1 files. (3 skipped: 3 unsupported.)

Finishing Touches

Generate unit tests (beta)

• Create a new PR

Tip

🎯 Customize your risk assessments

Guide how CodeRabbit assesses merge risk and security architecture risk. Highlight sensitive changes and explain your security boundaries using Merge Risk Instructions and Security Risk Instructions in your review settings. Security architecture risk assessments require the Advanced plan or above.

✅ Automatic approvals, with your rules

Let CodeRabbit approve pull requests once required threads are resolved, pre-merge checks pass, and your risk limits are met. Enable Automatic approvals in review settings, with overrides for teams or file paths.

Recent review info

Run configuration

• Source mode: Auto
• Repository: mobile-next/mobilecli
• Branch: main
• File: .coderabbit.yaml
• Commit: 83e47ca
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 545b483c-68b9-4374-ba63-b5e871bca141

Commits

Reviewing files that changed from the base of the PR and between 83e47ca and 1ad37aa.

Files selected for processing: 4

• .github/workflows/build.yml
• publish/npm-linux-riscv64/package.json
• publish/npm/index.js
• publish/npm/package.json

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

[gmegidish]

@Xeonacid do we have people using linux-riscv64 for mobile qa and development? that's awesome! can you tell me how you're using this?

[Xeonacid]

@Xeonacid do we have people using linux-riscv64 for mobile qa and development? that's awesome! can you tell me how you're using this?

Yeah! I connected a RISC-V board to my Android phone and controlled it from there. The mobilecli binary compiles smoothly on linux/riscv64, but since there is no Android NDK for riscv64, I can only compile the Android agent on x86_64. :(