Consolidate Partiful skills into one progressive skill - #71
Conversation
|
Warning Review limit reached
Next review available in: 30 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (7)
📝 WalkthroughWalkthroughThe PR consolidates separate Partiful skills into one routed skill with reference documentation, removes the ChangesPartiful skill migration
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@skills/partiful/references/auth-output-and-safety.md`:
- Line 37: Fix the `--format` row in the Markdown table by escaping the embedded
pipe characters in the format alternatives, or replace them with wording that
avoids pipes. Preserve the documented JSON default and keep the cell as a single
valid table column.
In `@skills/partiful/references/events.md`:
- Line 27: Update the default timezone statement in the events reference to use
the canonical IANA identifier America/Los_Angeles instead of the ambiguous
“Pacific” wording. Preserve the existing date and description guidance.
- Line 21: Update the event description argument in the command example to use
Bash ANSI-C quoting, ensuring the embedded newline escapes become actual line
breaks rather than literal “\n” text.
🪄 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 Plus
Run ID: fcf41592-aba0-4a1a-9dba-674bbe966e0f
📒 Files selected for processing (17)
docs/plans/2026-03-26-distribution-plan.mdpackage.jsonskills/partiful-blasts/SKILL.mdskills/partiful-events/SKILL.mdskills/partiful-guests/SKILL.mdskills/partiful-posters/SKILL.mdskills/partiful-shared/SKILL.mdskills/partiful/SKILL.mdskills/partiful/references/auth-output-and-safety.mdskills/partiful/references/events.mdskills/partiful/references/guests-and-rsvps.mdskills/partiful/references/posters-and-images.mdskills/partiful/references/text-blasts.mdsrc/cli.tssrc/commands/setup.tstests/setup.test.jstests/skill-structure.test.js
💤 Files with no reviewable changes (8)
- skills/partiful-blasts/SKILL.md
- src/commands/setup.ts
- skills/partiful-events/SKILL.md
- skills/partiful-posters/SKILL.md
- skills/partiful-shared/SKILL.md
- tests/setup.test.js
- skills/partiful-guests/SKILL.md
- src/cli.ts
|
|
||
| | Flag | Purpose | | ||
| |---|---| | ||
| | `--format <json|table|csv|ndjson>` | Output format; JSON is default | |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Fix the malformed --format table cell.
Line [37] contains unescaped pipe characters inside a Markdown table cell, causing MD056 and potentially rendering the row as multiple columns. Use wording that avoids embedded pipes or escape them.
Proposed fix
-| `--format <json|table|csv|ndjson>` | Output format; JSON is default |
+| `--format <json, table, csv, or ndjson>` | Output format; JSON is default |📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| | `--format <json|table|csv|ndjson>` | Output format; JSON is default | | |
| | `--format <json, table, csv, or ndjson>` | Output format; JSON is default | |
🧰 Tools
🪛 markdownlint-cli2 (0.23.0)
[warning] 37-37: Table column count
Expected: 2; Actual: 5; Too many cells, extra data will be missing
(MD056, table-column-count)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@skills/partiful/references/auth-output-and-safety.md` at line 37, Fix the
`--format` row in the Markdown table by escaping the embedded pipe characters in
the format alternatives, or replace them with wording that avoids pipes.
Preserve the documented JSON default and keep the cell as a single valid table
column.
Source: Linters/SAST tools
| @@ -0,0 +1,63 @@ | |||
| # Auth, Output, and Safety | |||
There was a problem hiding this comment.
should we move auth to it's own skill ?
| partiful events interested <event-id> --remove | ||
| ``` | ||
|
|
||
| `explore rsvp` and `explore interested` are equivalent aliases. Ticketed events and host questionnaires cannot be completed through the CLI; use Partiful directly. |
There was a problem hiding this comment.
make a pr and verify if host questionnaries can be completed or not
| @@ -0,0 +1,36 @@ | |||
| --- | |||
| name: partiful | |||
| description: Use when managing Partiful from the CLI, including authentication, events, RSVPs, guests, invitations, cohosts, contacts, posters, images, templates, exports, bulk operations, and text blasts. | |||
There was a problem hiding this comment.
use when managing any partiful event. use when the user wants to check partiful, see what's events to go to, create events for people,etc. needs another pass on this description to be better
|
|
||
| | Task | Read | | ||
| |---|---| | ||
| | Login, auth status, diagnostics, output formats, global flags, schema, errors, security | [Auth, output, and safety](references/auth-output-and-safety.md) | |
There was a problem hiding this comment.
determine if we need to move authentication to its own referece
| |---|---| | ||
| | Login, auth status, diagnostics, output formats, global flags, schema, errors, security | [Auth, output, and safety](references/auth-output-and-safety.md) | | ||
| | List, inspect, create, update, cancel, clone, template, or bulk-manage events | [Events](references/events.md) | | ||
| | RSVP, express interest, list/export/watch guests, invite people, find contacts, or manage cohosts | [Guests and RSVPs](references/guests-and-rsvps.md) | |
There was a problem hiding this comment.
determine if we seperate this into different markdown files (i think yes)
1 participant
Closes #70.
What changed
partiful-*skills with one model-invokablepartifulskill.partiful setup openclawcommand, registration, tests, and package keyword.Verification
npm test: 210 passed, 6 skippednpm run typecheck: passednpm pack --dry-run: exactlyskills/partiful/SKILL.mdplus five referencesSummary by CodeRabbit
New Features
Documentation
Removed