Skip to content

chore: upgrade bundled CLIs and project dependencies - #408

Merged
alwaysmavs merged 1 commit into
mainfrom
codex/upgrade-cli-dependencies
Oct 27, 2026
Merged

alwaysmavs merged 1 commit into
mainfrom
codex/upgrade-cli-dependencies

Conversation

@alwaysmavs

Copy link
Copy Markdown
Contributor

Upgrade OOCLI, DingTalk CLI, pnpm, agent adapters, Electron, and project dependencies. Update bundle integrity pins and documentation, and adapt clipboard and renderer types to the new APIs. Keep Effect pinned to the version required by Open Flow.

Validation: frozen lockfile installation, type checking, lint, formatting, and production build passed. 423 test files and 3505 tests passed, with 4 tests skipped. Electron 45 startup and bundled CLI versions were verified locally.

@coderabbitai

coderabbitai Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI; Enforced overrides: Organization
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 55b2bba8-b2ac-400c-897a-d6cfd652acef
Commits

Reviewing files that changed from the base of the PR and between 19cbd5d and bf9f9e4.

Files ignored due to path filters: 1
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
Files selected for processing: 27
  • .github/workflows/release.yml
  • README.es.md
  • README.ja.md
  • README.ko.md
  • README.md
  • README.zh-CN.md
  • THIRD_PARTY_NOTICES.md
  • docs/ai/agent-adapter.md
  • docs/ai/bootstrap.md
  • docs/architecture.md
  • docs/development.md
  • docs/open-source-plan.md
  • docs/project-overview.md
  • electron/agent/acp/translator.test.ts
  • electron/agent/external/oo-capability-contract.ts
  • electron/agent/oo-version.ts
  • electron/chat/node.ts
  • electron/clipboard-handler.test.ts
  • electron/clipboard-handler.ts
  • package.json
  • pnpm-workspace.yaml
  • resources/skill-lock/oo.json
  • scripts/claude-agent-acp.test.ts
  • scripts/dingtalk-cli.ts
  • scripts/oo-upgrade-review-core.test.ts
  • scripts/open-source-readiness.test.ts
  • src/components/ai-elements/mermaid-renderer.tsx
Files with no reviewable changes: 1
  • electron/agent/acp/translator.test.ts

Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

Summary by CodeRabbit

  • Bug Fixes
    • Improved clipboard handling so image copies and asynchronous text-copy errors are handled correctly.
  • Updates
    • Updated the app’s bundled tools and supporting libraries, including OpenCode, agent integrations, and the DingTalk CLI.
    • Updated the package manager and documented setup requirements to pnpm 12.15.0.
  • Documentation
    • Refreshed version references and development guidance across the project documentation.

Walkthrough

The pull request updates dependency and tool version pins and aligns related documentation and checks. It adds team.list to the OO capability contract and required-operation metadata. Clipboard handling now returns the text-write promise, and local image copying uses clipboard.write with a PNG ClipboardItem. The Mermaid renderer derives its configuration type from the plugin API.

Possibly related PRs

  • oomol-lab/wanta#405: Updates the same bundled oo CLI and DingTalk CLI versions that this pull request advances.

Priority: Not assessed

Merge Risk: Minimal · up to bf9f9

The dependency, contract, clipboard, and type updates have no established merge-blocking risk.

Pre-merge checks: Passed 3 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Description check Warning The description provides a concise summary and reports validation results, but it omits the required ## Summary, ## Verification, and ## Safety and Compatibility structure and does not address t… Rewrite the description using the repository template. Include the required headings, verification checklist results, runtime/UI verification status, and explicit responses for each Safety and Compatibility item.
Passed Passed checks: 3 passed
Check name Status Explanation
Title check Passed The title uses the required <type>: <subject> format with an optional scope, uses English, and accurately summarizes the dependency and CLI upgrades.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Description check

Explanation

The description provides a concise summary and reports validation results, but it omits the required ## Summary, ## Verification, and ## Safety and Compatibility structure and does not address the required safety and compatibility checks.

Resolution

Rewrite the description using the repository template. Include the required headings, verification checklist results, runtime/UI verification status, and explicit responses for each Safety and Compatibility item.

Finishing Touches
✨ Simplify code
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@alwaysmavs
alwaysmavs enabled auto-merge (squash) October 27, 2026 07:14
@alwaysmavs
alwaysmavs merged commit 5a52ddc into main Oct 27, 2026
2 checks passed
@alwaysmavs
alwaysmavs deleted the codex/upgrade-cli-dependencies branch October 27, 2026 07:17
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