← Back to PRs

#22026: fix(summarize): support nodejs install by default, and brew install on darwin

by austingamez91 open 2026-02-20 16:59 View on GitHub →
size: XS
## Summary Describe the problem and fix in 2–5 bullets: - Problem: On all openclaw installations, the summarize skill relies on brew. This limits portability. I have changed the skill to default to nodejs, and use brew when on darwin. Tested on both linux and darwin devices. Screenshots below. - Why it matters: this increases portability and allows for cleaner "out-of-the-box" installations. - What changed: skills/summarize/SKILL.md was edited to default to nodejs, while still using brew on darwin - What did NOT change (scope boundary): No runtime logic was changed, and brew was not removed entirely. It is now used for darwin install only ## Change Type (select all) - [X] Bug fix - [ ] Feature - [ ] Refactor - [ ] Docs - [ ] Security hardening - [X] Chore/infra ## Scope (select all touched areas) - [ ] Gateway / orchestration - [X] Skills / tool execution - [ ] Auth / tokens - [ ] Memory / storage - [ ] Integrations - [ ] API / contracts - [ ] UI / DX - [ ] CI/CD / infra ## Linked Issue/PR - Closes # - Related # ## User-visible / Behavior Changes users on windows or linux will receive nodejs installation of summarize. ## Security Impact (required) - New permissions/capabilities? (`Yes/No`): No - Secrets/tokens handling changed? (`Yes/No`): No - New/changed network calls? (`Yes/No`): No - Command/tool execution surface changed? (`Yes/No`): No - Data access scope changed? (`Yes/No`): No - If any `Yes`, explain risk + mitigation: ## Repro + Verification ### Environment Tested in two environments: OS: macOS Runtime/container: Node 22+, pnpm Model/provider: N/A Integration/channel (if any): N/A Relevant config (redacted): default In the above Environment, brew will still be used Environment OS: linux (pop_os) Runtime/container: Node 22+, pnpm Model/provider: N/A Integration/channel (if any): N/A Relevant config (redacted): default repo config In the above environment, nodejs package managers will be used instead of brew ### Steps Linux: 1. install openclaw from source on linux 2. add summarize skill 3. verify nodejs package manager is used for install MacOS: 1. install openclaw from source on macOS 2. Add summarize skill 3. Verify that brew is used for installation ### Expected - linux installs will use nodejs package manager for the installation of summarize - macOS installs will use brew for the installation of summarize ### Actual - linux installs use nodejs package manager for the installation of summarize - macOS installs use brew for the installation of summarize ## Evidence Attach at least one: - [ ] Failing test/log before + passing after - [ ] Trace/log snippets - [X] Screenshot/recording - [ ] Perf numbers (if relevant) <img width="3489" height="1842" alt="screenshot-linux" src="https://github.com/user-attachments/assets/bb1aca72-ce75-4656-9bb7-37efad8aaea9" /> <img width="2662" height="1312" alt="screenshot-darwin" src="https://github.com/user-attachments/assets/2e46d7db-3f40-447d-a661-46bf7e23de87" /> ## Human Verification (required) What you personally verified (not just CI), and how: - Verified scenarios: Verified openclaw onboard will use nodejs to install summarize on linux. Verified it will use brew on macOS. - Edge cases checked: - What you did **not** verify: I did not directly verify on Windows, because windows will also default to nodejs install. So testing on Linux should be sufficient. ## Compatibility / Migration - Backward compatible? (`Yes/No`): Yes - Config/env changes? (`Yes/No`): No - Migration needed? (`Yes/No`): No - If yes, exact upgrade steps: ## Failure Recovery (if this breaks) - How to disable/revert this change quickly: revert skills/summarize/SKILL.md to previous state, which only referenced brew - Files/config to restore: None - Known bad symptoms reviewers should watch for: None ## Risks and Mitigations List only real risks for this PR. Add/remove entries as needed. If none, write `None`. - Risk: - None

Most Similar PRs