Skip to content

feat(core): optimize unity-cli runtime path and daemon workflow#34

Merged
github-actions[bot] merged 4 commits into
developfrom
feature/optimize-cli
Feb 23, 2026
Merged

feat(core): optimize unity-cli runtime path and daemon workflow#34
github-actions[bot] merged 4 commits into
developfrom
feature/optimize-cli

Conversation

@akiojin

@akiojin akiojin commented Feb 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • Renamed legacy MCP-facing assets, handlers, and docs to unity-cli naming so repository semantics match runtime behavior and reduce maintenance confusion.
  • Added and refined daemon-based execution paths (unityd control commands, transport-aware auto fallback, and batch invocation entrypoints) to reduce per-command connection overhead in LLM workflows.
  • Improved execution throughput with Unity-side command queue draining and release-profile optimizations while aligning docs/specs to always-auto daemon behavior.

Changes

  • src/main.rs, src/unityd.rs, src/cli.rs, src/config.rs, src/transport.rs, src/lspd.rs: wired daemon commands, auto-mode transport fallback behavior, batch routing, and related runtime/tests updates.
  • UnityCliBridge/Packages/unity-cli-bridge/Editor/Core/UnityCliBridgeHost.cs and bridge handlers/models: renamed legacy MCP symbols and updated command-processing flow to drain queued requests in one processing cycle.
  • Cargo.toml: enabled release optimizations (lto, codegen-units = 1, strip) to improve binary startup/runtime characteristics.
  • docs/development.md and specs/SPEC-a7206673/*: documented always-auto daemon usage, added CLI examples (unityd, batch), and synchronized TDD/spec artifacts.

Testing

  • cargo fmt --all -- --check — passed with exit code 0.
  • cargo clippy --all-targets -- -D warnings — passed with exit code 0.
  • cargo test --all-targets — passed (51 passed, 0 failed).

Related Issues / Links

Checklist

  • Tests added/updated
  • Lint/format passed (cargo clippy, cargo fmt, svelte-check) — cargo clippy/cargo fmt passed; svelte-check N/A (no Svelte targets)
  • Documentation updated (if user-facing change)
  • Migration/backfill plan included (if schema/data change) — N/A: no schema/data migration
  • CHANGELOG impact considered (breaking change flagged in commit) — considered; no dedicated CHANGELOG file in this repository

Context

Risk / Impact

  • Affected areas: Rust CLI command dispatch, daemon transport/runtime mode handling, Unity bridge request processing loop, and skill/documentation naming consistency.
  • Rollback plan: Revert PR commits on feature/optimize-cli (or cherry-pick revert specific runtime commits) and republish the previous known-good CLI/UPM package revisions.

@github-actions
github-actions Bot enabled auto-merge February 23, 2026 12:30
@coderabbitai

coderabbitai Bot commented Feb 23, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/optimize-cli

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions
github-actions Bot merged commit a65fdef into develop Feb 23, 2026
11 checks passed
@akiojin
akiojin deleted the feature/optimize-cli branch February 23, 2026 13:29
akiojin pushed a commit that referenced this pull request Mar 3, 2026
feat(core): optimize unity-cli runtime path and daemon workflow
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