feat(opencli): add Xiaohongshu ask via upstream 1.8.4

[Astro-Han]

Summary

Bumps @jackwener/opencli from 1.8.3 to 1.8.4 so PawWork gets the upstream xiaohongshu/ask adapter from the published package.

There is no related issue; this updates PawWork's pinned OpenCLI integration after OpenCLI released v1.8.4 with the Xiaohongshu 点点 command.

Why

The original PR used a PawWork-local overlay because OpenCLI had not published the adapter yet. OpenCLI v1.8.4 is now published and its npm package includes both clis/xiaohongshu/ask.js and the manifest entry, so the local overlay has been removed in favor of the dependency bump.

Related Issue

No issue.

Human Review Status

Pending

Review Focus

Please focus on whether the final diff stays limited to the OpenCLI dependency bump and public adapter discovery coverage, and whether xiaohongshu/ask is verified through both source tests and the built embedded-server artifact.

Risk Notes

This changes PawWork's pinned OpenCLI package from 1.8.3 to 1.8.4. The previous local overlay source and overlay-only tests were removed because the adapter now comes from the upstream package.

No visible UI changed, so no screenshots are included. The desktop embedded-server packaging path is covered by the built-node OpenCLI adapter test.

How To Verify

bun install --frozen-lockfile: passed
bun test ./test/opencli/adapter-registry.test.ts ./test/tool/opencli-tools.test.ts ./test/opencli/adapter-capabilities.test.ts --timeout 30000: 22 passed
bun test ./test/server/built-node-opencli-adapters.test.ts --timeout 120000: 1 passed
git diff --check: passed

Screenshots or Recordings

Not required; no visible UI changed.

Checklist

How to use this checklist:

• Tick a box by replacing [ ] with [x]. Do not edit, add, or remove items.
• The bot-applied label items can only be honestly ticked AFTER the PR is opened and the labeler / priority-triage bots have run — return to the PR description and tick them then.
• Most items are required. The few that are conditional are explicitly marked (conditional); for those, leave unticked if they truly do not apply and explain why in Risk Notes. All other items must be ticked before requesting human review.

• Type label — this PR carries exactly one of bug, enhancement, task, documentation. Type labels are author-added; the labeler bot does NOT assign them. Add the label in the GitHub UI, then tick this.
• Routing labels — this PR carries at least one of app, ui, platform, harness, ci. The labeler bot assigns these on PR open based on changed paths. Confirm the bot's choice (or override if wrong), then tick this.
• Priority label — this PR carries exactly one of P0, P1, P2, P3. The priority-triage bot suggests one on PR open. Confirm or override, then tick this.
• Human Review Status above is set to Pending, Approved by @<reviewer>, or Not required: <reason> (default is Pending; "not required" is restricted to bot-authored low-risk PRs).
• I linked the related issue, or stated in Summary why there is no issue.
• I described the review focus and any meaningful risks.
• I replaced the example block in How To Verify with the real verification steps and the key result for each.
• I did not introduce unrelated refactors, dependencies, generated files, or file changes beyond the stated scope.
• (conditional) I manually checked visible UI or copy changes when needed, with screenshots or recordings. Leave unticked only if no visible UI or copy changed.
• (conditional) I considered macOS and Windows impact for platform, packaging, updater, signing, paths, shell, or permissions changes. Leave unticked only if no platform/packaging surface was touched.
• (conditional) I called out docs, release notes, dependencies, permissions, credentials, deletion behavior, generated content, or local file changes when relevant. Leave unticked only if none of those surfaces was touched.
• I reviewed the final diff for unrelated changes and suspicious dependency changes.
• I am targeting dev, and my PR title and commit messages use Conventional Commits in English.

Summary by CodeRabbit

• Chores

  • Updated an OpenCLI-related dependency to the latest patch version.

• Tests

  • Enhanced test coverage for multiple OpenCLI tool queries, including updated assertions for a nested output structure.
  • Added a live test to verify adapter discovery and to validate rendered command output and argument metadata (including default values).

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 746b6b0f-99d5-4510-bc4a-d43e9a1387f9

📥 Commits

Reviewing files that changed from the base of the PR and between 76a2655 and 6cf7f9d.

⛔ Files ignored due to path filters (1)

• bun.lock is excluded by !**/*.lock

📒 Files selected for processing (1)

• packages/opencode/package.json

🚧 Files skipped from review as they are similar to previous changes (1)

• packages/opencode/package.json

---

📝 Walkthrough

Walkthrough

Bumps @jackwener/opencli from 1.8.3 to 1.8.4 in packages/opencode/package.json. Updates the built-node adapter integration test to execute two queries and assert a nested account/xiaohongshu output shape. Adds a new live test verifying OpenCliSearchTool discovers the Xiaohongshu "ask" adapter and renders the expected argument metadata.

Changes

opencli 1.8.4 bump and Xiaohongshu test coverage

Layer / File(s)	Summary
@jackwener/opencli dependency bump packages/opencode/package.json	@jackwener/opencli bumped from 1.8.3 to 1.8.4.
Dual-query nested output assertions and Xiaohongshu live test packages/opencode/test/server/built-node-opencli-adapters.test.ts, packages/opencode/test/tool/opencli-tools.test.ts	Embedded server test runs two separate opencli_search queries and asserts a nested output.account.* / output.xiaohongshu.* shape with title, output, and non-zero metadata.count. New live test in opencli-tools.test.ts validates OpenCliSearchTool discovers the Xiaohongshu ask adapter and renders query type, timeout, and source-limit argument metadata.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• Astro-Han/pawwork#1272: Introduced the bundled opencli_search adapter registry tooling and <opencli_command ...> rendering that the Xiaohongshu adapter assertions in this PR directly depend on.

Suggested labels

app

Poem

🐇 Hop, hop, a version bumps along,
From three to four, a tiny song!
Xiaohongshu ask now joins the test,
Nested outputs put to the quest.
One bunny, two queries — all the rest! 🌸

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main change: adding Xiaohongshu ask support via the upstream OpenCLI dependency version bump to 1.8.4.
Description check	✅ Passed	The PR description covers all major template sections including Summary, Why, Related Issue, Human Review Status, Review Focus, Risk Notes, How To Verify, and a completed Checklist with all required items addressed.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch codex/opencli-xhs-ask

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Suggested priority: P2 (includes non-doc, non-test paths outside the low-risk bucket).

P1/P0 are reserved for maintainer confirmation. Please relabel manually if this is a release blocker, security issue, data-loss risk, or updater/runtime failure.

[gemini-code-assist]

Code Review

This pull request introduces a new adapter override for Xiaohongshu's "ask" command (xiaohongshu-ask.ts), which interacts with the site's webpack conversation store to query "点点" and retrieve answers along with citation sources. The adapter is registered in adapter-registry.ts, and corresponding integration tests have been added to verify its discovery and functionality. I have no feedback to provide as there are no review comments and the implementation is complete.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.