feat(#30): Update OpenClaw skills with formatting rules + poster/image docs#42
feat(#30): Update OpenClaw skills with formatting rules + poster/image docs#42
Conversation
… docs
- partiful-events: added poster/image flags, formatting rules (plain text, full year dates, AM/PM check)
- partiful-shared: added formatting rules, schema introspection docs
- partiful-posters: new skill for poster catalog browsing
- Replaces old browser-automation-only skill with CLI-first pattern
- Follows gws naming convention: partiful-{resource}
Closes #30
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdated Partiful CLI documentation across three SKILL.md files: replaced event-list flags with Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 4
🧹 Nitpick comments (1)
skills/partiful-posters/SKILL.md (1)
40-48: Clarify auth boundary forevents createexamples.Line 8 says “No auth required,” which is true for poster catalog commands, but Lines 42 and 47 invoke
events create(which still requires Partiful auth). Add one explicit note here to avoid operator confusion.Suggested doc tweak
## Usage with Events Once you find a poster ID, pass it to `events create` or `events update`: +> Note: Poster catalog commands are public, but `events create` / `events update` still require Partiful authentication. ```bash partiful events create --title "Party" --date "2026-04-05T19:00" --poster "piscesairbrush.png"</details> <details> <summary>🤖 Prompt for AI Agents</summary>Verify each finding against the current code and only fix it if needed.
In
@skills/partiful-posters/SKILL.mdaround lines 40 - 48, The examples for
passing a poster ID to events create and using --poster-search are unclear about
authentication: clarify that the "No auth required" note applies only to the
poster catalog commands and not to events create/update; update the text around
theevents createexamples (references: the events create command and the
--poster-search flag) to add a short explicit note like "events create/update
require Partiful authentication" so operators know they must be authenticated
before running those commands.</details> </blockquote></details> </blockquote></details> <details> <summary>🤖 Prompt for all review comments with AI agents</summary>Verify each finding against the current code and only fix it if needed.
Inline comments:
In@skills/partiful-events/SKILL.md:
- Around line 103-116: The fenced code blocks showing the date/time examples and
the description examples in SKILL.md are missing explicit fence languages;
update the backtick fences for both examples (the block with "✅ 2026-04-01T19:00
..." and the block with the description examples containing "🎮 Game Night!" and
"Game Night!") to include a language such as text (e.g., ```text) so
markdownlint rule MD040 is satisfied.- Around line 95-98: The docs list an unavailable command; remove or relocate
the "Share Link (+share)" entry from SKILL.md so it matches the implemented
commands in registerEventsCommands (which only exposes
list/get/create/update/cancel), or if +share exists elsewhere, move the snippet
to the correct skill/command doc; update the README so the documented commands
exactly match the registerEventsCommands API.- Around line 65-70: Update the auth table in SKILL.md to correctly state that
poster flags used by the event commands require Partiful authentication: change
the "No" entries for--poster <id>and--poster-search <query>to indicate
Partiful auth required (or rephrase the table header to "Partiful auth required
for event mutations"), and clarify that--image <path|url>requires both
Partiful auth for event create/update and an additional Firebase token for
custom uploads; ensure the Notes column mentions the 10MB limit remains.In
@skills/partiful-shared/SKILL.md:
- Around line 73-86: Update the two fenced code blocks that show the date
examples and the "Descriptions — Plain Text Only" examples by adding the
language identifier "text" to the opening fences (changetotext) so they
satisfy MD040; target the fenced block containing the date lines (the ✅/❌
examples) and the fenced block containing the plain-text description examples
(the quoted game night lines).
Nitpick comments:
In@skills/partiful-posters/SKILL.md:
- Around line 40-48: The examples for passing a poster ID to events create and
using --poster-search are unclear about authentication: clarify that the "No
auth required" note applies only to the poster catalog commands and not to
events create/update; update the text around theevents createexamples
(references: the events create command and the --poster-search flag) to add a
short explicit note like "events create/update require Partiful authentication"
so operators know they must be authenticated before running those commands.</details> --- <details> <summary>ℹ️ Review info</summary> <details> <summary>⚙️ Run configuration</summary> **Configuration used**: defaults **Review profile**: CHILL **Plan**: Pro **Run ID**: `d26f8d94-e0e3-4908-9cc4-7a8d3d998e04` </details> <details> <summary>📥 Commits</summary> Reviewing files that changed from the base of the PR and between 899898cc3af30086e28ddd6ca0858db2541d7ff2 and 495de7cfc0b0ad73d84fe0084d2f75596854c87b. </details> <details> <summary>📒 Files selected for processing (3)</summary> * `skills/partiful-events/SKILL.md` * `skills/partiful-posters/SKILL.md` * `skills/partiful-shared/SKILL.md` </details> </details> <!-- This is an auto-generated comment by CodeRabbit for review status -->
- Auth table: clarified that all event mutations need Partiful auth, custom upload additionally needs Firebase token - Removed undocumented events +share helper (not implemented) - Added 'text' language identifier to all formatting example code blocks (MD040)
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@skills/partiful-shared/SKILL.md`:
- Line 15: Update the SKILL.md wording "Phone-based authentication in **E.164
format**." to reflect actual behavior: indicate that E.164 is supported but
non-`+` US numbers will be normalized by prepending `+1` (e.g., "E.164 (US
numbers without a leading '+' will be normalized with +1)"), so users aren’t
misled about accepted input; ensure the sentence referencing phone auth clearly
mentions the automatic +1 normalization used by the login command.
- Around line 18-19: The wording in SKILL.md that reads "Auto-retrieves the code
from iMessage (via `imsg`)" incorrectly implies macOS-only support; update that
phrase to mention SMS auto-retrieval across platforms by replacing it with
something like "Auto-retrieves the code from SMS sources (e.g., iMessage via
`imsg` on macOS or Termux SMS on Android)" so both iMessage and Termux are
clearly represented; locate the exact text "Auto-retrieves the code from
iMessage (via `imsg`)" and replace it with a broader platform-agnostic
description that references `imsg` and Termux where appropriate.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 74e8d98c-1e21-4b95-91c5-7bdaf35a8208
📒 Files selected for processing (2)
skills/partiful-events/SKILL.mdskills/partiful-shared/SKILL.md
🚧 Files skipped from review as they are similar to previous changes (1)
- skills/partiful-events/SKILL.md
- E.164 format is preferred but US numbers without +1 are auto-normalized - SMS auto-retrieval supports macOS (imsg) and Android (termux-sms-list), not just macOS
1 participant
Closes #30
What
Updates all OpenClaw skills to the CLI-first pattern (replacing old browser-automation skill) and adds the formatting/date guidelines from #30.
Changes
partiful-events/SKILL.md
--poster,--poster-search,--image)partiful-shared/SKILL.md
partiful-posters/SKILL.md (new)
OpenClaw Integration
Old
skills/partiful/(single browser-automation skill) replaced by 5 skills following the gws naming convention:partiful-shared— auth, flags, formatting rulespartiful-events— CRUD + posters/imagespartiful-guests— RSVPs, invites, watch, exportpartiful-blasts— text blastspartiful-posters— poster catalogSummary by CodeRabbit