Add skill: adspower/adspower-browser#422
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThis PR adds the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
categories/browser-and-automation.md (1)
5-5:⚠️ Potential issue | 🔴 CriticalUpdate the skill count to 323.
The count should be incremented from 322 to 323 to reflect the addition of
adspower-browser.🔢 Proposed fix
-**322 skills** +**323 skills**🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@categories/browser-and-automation.md` at line 5, Update the skills header to reflect the new total by changing the text "**322 skills**" to "**323 skills**" in the categories/browser-and-automation.md entry, since `adspower-browser` was added; ensure the single-line count is incremented from 322 to 323.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@categories/browser-and-automation.md`:
- Line 314: The "adspower-browser" entry is out of alphabetical order and
duplicated; locate the entry text "adspower-browser" and move that line so it
appears after the "admet-prediction" entry and before the "Agent Browser" entry,
then remove the duplicate occurrence at its current location (the one appended
at the end) to keep the list alphabetized and deduplicated.
In `@README.md`:
- Line 283: The README preview section has the 'adspower-browser' entry placed
at the end instead of alphabetically; move the line containing "adspower-browser
- Use when the user asks to create or manage AdsPower browsers, groups, tags,
proxies, or check status via AdsPower Local API." so it appears after the
"admet-prediction" entry and before the "Agent Browser" entry, and remove the
duplicate occurrence that remains at the old location; update only the ordering
in the preview list (no text changes to the entry).
---
Outside diff comments:
In `@categories/browser-and-automation.md`:
- Line 5: Update the skills header to reflect the new total by changing the text
"**322 skills**" to "**323 skills**" in the categories/browser-and-automation.md
entry, since `adspower-browser` was added; ensure the single-line count is
incremented from 322 to 323.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 21d043e0-f846-4b77-94d1-78acfeb57e19
📒 Files selected for processing (2)
README.mdcategories/browser-and-automation.md
| - [airtable-automation](https://clawskills.sh/skills/sohamganatra-airtable-automation) - Automate Airtable tasks via Rube MCP (Composio) | ||
| - [airtable-participants](https://clawskills.sh/skills/austinmao-airtable-participants) - Read and query retreat participant data from the Ceremonia Airtable base. | ||
| - [ak-rss-24h-brief](https://clawskills.sh/skills/seandong-ak-rss-24h-brief) - Read RSS/Atom feeds from an OPML list, fetch articles from the last N hours, and generate a Chinese categorized. | ||
| - [adspower-browser](https://github.com/openclaw/skills/tree/main/skills/adspower/adspower-browser) - Use when the user asks to create or manage AdsPower browsers, groups, tags, proxies, or check status via AdsPower Local API. |
There was a problem hiding this comment.
Place the entry in alphabetical order.
The adspower-browser entry should be positioned alphabetically in the preview section, not appended at the end. It should appear after admet-prediction (line 264) and before Agent Browser (line 265).
📝 Proposed fix
Move this line to its correct alphabetical position:
- [admet-prediction](https://clawskills.sh/skills/huifer-admet-prediction) - ADMET (Absorption, Distribution, Metabolism, Excretion, Toxicity) prediction for drug candidates.
+- [adspower-browser](https://github.com/openclaw/skills/tree/main/skills/adspower/adspower-browser) - Use when the user asks to create or manage AdsPower browsers, groups, tags, proxies, or check status via AdsPower Local API.
- [Agent Browser](https://clawskills.sh/skills/thesethrose-agent-browser) - A fast Rust-based headless browser automation CLI.And remove the duplicate entry from line 283.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` at line 283, The README preview section has the 'adspower-browser'
entry placed at the end instead of alphabetically; move the line containing
"adspower-browser - Use when the user asks to create or manage AdsPower
browsers, groups, tags, proxies, or check status via AdsPower Local API." so it
appears after the "admet-prediction" entry and before the "Agent Browser" entry,
and remove the duplicate occurrence that remains at the old location; update
only the ordering in the preview list (no text changes to the entry).
Summary
Add
adspower/adspower-browserto the Browser & Automation category.Skill Info
adspower/adspower-browserWhy It Fits
adspower-browserUse when the user asks to create or manage AdsPower browsers, groups, tags, proxies, or automation via AdsPower Local API.Notes
This skill is maintained by the AdsPower team and is already published in the official OpenClaw skills registry.
Summary by CodeRabbit