Skip to content

Trim duplicated Billing and Organizations skill details - #118

Merged
SarahSoutoul merged 9 commits into
mainfrom
ss/DOCS-12373-content-cleanup
Nov 11, 2026
Merged

SarahSoutoul merged 9 commits into
mainfrom
ss/DOCS-12373-content-cleanup

Conversation

@SarahSoutoul

@SarahSoutoul SarahSoutoul commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

What does this solve? What changed?

This is the content follow-up to the merged clerk/skills#117 for DOCS-12373. The formatter and one-time reformat are now on main. They brought clerk-billing under the 5,000-estimated-token limit, but left only nine tokens of room. This PR removes repeated instructions and moves the complete Stripe event comparison into a reference. It now targets main, and its diff contains only the three Billing and Organizations content files.

  • Keep the Clerk-vs-Stripe webhook warning in clerk-billing/SKILL.md, and direct agents to the existing webhook reference for the event catalog, payloads, and handler. Move the complete 16-row Stripe-to-Clerk event comparison there, including events without a Stripe equivalent.
  • Replace the duplicated default-role table in clerk-orgs/SKILL.md with a link to its existing roles reference. Remove the final workflow list because its steps are already covered by the Quick Start and linked references.

Deadline

No fixed deadline. Review the instruction changes and merge this PR into main.

Other resources

Checklist

  • Diff against main contains only the three Billing and Organizations content files.
  • pnpm run check passes, including formatting and the skill-length check.
  • pnpm test passes (152 tests).
  • clerk-billing is 4,318 and clerk-orgs is 4,738 estimated tokens, both below 5,000.
  • Review the instruction changes and merge this PR into main.

@SarahSoutoul SarahSoutoul self-assigned this Nov 6, 2026
@SarahSoutoul
SarahSoutoul marked this pull request as ready for review November 6, 2026 19:54
@SarahSoutoul
SarahSoutoul requested review from a team as code owners November 6, 2026 19:54
@SarahSoutoul
SarahSoutoul changed the base branch from ss/DOCS-12092 to main November 10, 2026 22:41
@manovotny manovotny self-assigned this Nov 10, 2026
@manovotny

Copy link
Copy Markdown
Collaborator

I opened #119 to fix three errors in the references this PR now points to, checked against clerk_go. It doesn't touch any lines here, so the two can merge in either order.

Comment thread skills/clerk-orgs/SKILL.md Resolved
Comment thread skills/clerk-billing/SKILL.md Resolved
@manovotny

Copy link
Copy Markdown
Collaborator

Pushed some changes directly in 3b075e9 and 7b87821, now that #119 has merged.

  • Merged main so the branch has Correct billing and org references against clerk_go #119's reference fixes
  • Changed "The table lists all 16 billing events" to "The table below lists all billing events," so the count can't go stale
  • Moved the naming rule up next to "Subscribe to…" and the subscription.canceled note below the table, so the table follows its intro directly

@SarahSoutoul
SarahSoutoul merged commit f17ee71 into main Nov 11, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants