chore(release): 📦 version packages #550
Conversation
WalkthroughRemoved many .changeset metadata files and applied coordinated version bumps and changelog updates across packages and apps; no source code, runtime logic, or public API signatures were changed. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes
Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (59)
💤 Files with no reviewable changes (25)
✅ Files skipped from review due to trivial changes (3)
🚧 Files skipped from review as they are similar to previous changes (17)
🧰 Additional context used📓 Path-based instructions (3)packages/**/*📄 CodeRabbit inference engine (.cursor/rules/monorepo-setup.mdc)
Files:
**/package.json📄 CodeRabbit inference engine (CLAUDE.md)
Files:
apps/**/*📄 CodeRabbit inference engine (.cursor/rules/monorepo-setup.mdc)
Files:
🧠 Learnings (9)📚 Learning: 2025-10-14T22:05:16.978ZApplied to files:
📚 Learning: 2025-07-22T11:50:39.243ZApplied to files:
📚 Learning: 2025-07-22T11:50:39.243ZApplied to files:
📚 Learning: 2025-07-23T10:53:00.058ZApplied to files:
📚 Learning: 2025-07-22T12:20:03.175ZApplied to files:
📚 Learning: 2025-07-22T21:59:25.155ZApplied to files:
📚 Learning: 2025-07-23T10:53:08.297ZApplied to files:
📚 Learning: 2025-07-22T21:59:36.674ZApplied to files:
📚 Learning: 2025-07-22T11:50:39.243ZApplied to files:
🪛 LanguageToolpackages/core/CHANGELOG.md[grammar] ~9-~9: Use a hyphen to join words. (QB_NEW_EN_HYPHEN) [style] ~26-~26: In American English, abbreviations like “etc.” require a period. (ETC_PERIOD) [grammar] ~43-~43: Ensure spelling is correct (QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1) [grammar] ~43-~43: Ensure spelling is correct (QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1) packages/mdx-mermaid/CHANGELOG.md[style] ~10-~10: In American English, abbreviations like “etc.” require a period. (ETC_PERIOD) packages/config/eslint-config/CHANGELOG.md[style] ~41-~41: Using many exclamation marks might seem excessive (in this case: 3 exclamation marks for a text that’s 1623 characters long) (EN_EXCESSIVE_EXCLAMATION) [uncategorized] ~42-~42: “React” is a proper noun and needs to be capitalized. (A_GOOGLE) packages/email-templates/CHANGELOG.md[style] ~10-~10: In American English, abbreviations like “etc.” require a period. (ETC_PERIOD) packages/commerce-next/CHANGELOG.md[grammar] ~9-~9: Use a hyphen to join words. (QB_NEW_EN_HYPHEN) [style] ~21-~21: In American English, abbreviations like “etc.” require a period. (ETC_PERIOD) packages/next/CHANGELOG.md[style] ~10-~10: In American English, abbreviations like “etc.” require a period. (ETC_PERIOD) packages/ui/CHANGELOG.md[style] ~10-~10: In American English, abbreviations like “etc.” require a period. (ETC_PERIOD) packages/utils-email/CHANGELOG.md[style] ~10-~10: In American English, abbreviations like “etc.” require a period. (ETC_PERIOD) 🔇 Additional comments (7)
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 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (8)
.changeset/odd-lines-care.md(0 hunks)apps/astro-party/package.json(1 hunks)apps/course-builder-web/package.json(1 hunks)apps/egghead/package.json(1 hunks)apps/epic-react/package.json(1 hunks)apps/go-local-first/package.json(1 hunks)packages/ui/CHANGELOG.md(1 hunks)packages/ui/package.json(1 hunks)
💤 Files with no reviewable changes (1)
- .changeset/odd-lines-care.md
🧰 Additional context used
📓 Path-based instructions (3)
**/package.json
📄 CodeRabbit inference engine (CLAUDE.md)
**/package.json: When adding dependencies to package.json files, ensure all packages use consistent dependency versions and dependencies are sorted alphabetically.
When updating package.json files to add dependencies, use string replacement to add dependencies and maintain alphabetical order. Do not replace entire sections, just add the new line.
Files:
apps/course-builder-web/package.jsonpackages/ui/package.jsonapps/go-local-first/package.jsonapps/astro-party/package.jsonapps/egghead/package.jsonapps/epic-react/package.json
apps/**/*
📄 CodeRabbit inference engine (.cursor/rules/monorepo-setup.mdc)
Apps are located in the /apps directory
Files:
apps/course-builder-web/package.jsonapps/go-local-first/package.jsonapps/astro-party/package.jsonapps/egghead/package.jsonapps/epic-react/package.json
packages/**/*
📄 CodeRabbit inference engine (.cursor/rules/monorepo-setup.mdc)
Packages are located in the /packages directory
Files:
packages/ui/CHANGELOG.mdpackages/ui/package.json
🔇 Additional comments (5)
packages/ui/package.json (1)
3-3: Package version bumped to 2.0.8 — aligned with app consumers.
Nothing else changed; no API surface implications here. Ensure the changelog entry matches this version (it should if generated by Changesets).apps/egghead/package.json (1)
37-37: LGTM: @coursebuilder/ui upgraded to ^2.0.8 with ordering intact.
Aligned with the workspace and the UI package version.apps/astro-party/package.json (1)
55-55: LGTM: @coursebuilder/ui bumped to ^2.0.8; grouping/order within @coursebuilder/ remains consistent.*apps/go-local-first/package.json (1)
58-58: LGTM: Version spec updated to ^2.0.8 and ordering preserved.packages/ui/CHANGELOG.md (1)
3-11: Changelog entry 2.0.8 reads well and follows existing format.Links, attribution, and “Patch Changes” section match prior entries. No further edits needed.
github-actions
bot
force-pushed
the
changeset-release/main
branch
from
5235dc0 to
d21f825
Compare
github-actions
bot
force-pushed
the
changeset-release/main
branch
from
86e6fd1 to
52b4364
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
apps/egghead/package.json (1)
37-37: Consider switching to the workspace protocol for internal packages.To guarantee local linking during development and auto-follow workspace version bumps, consider using the workspace range instead of a plain caret. This also keeps internal deps consistent with other @coursebuilder/* entries that already use workspace:^.
- "@coursebuilder/ui": "^2.0.8", + "@coursebuilder/ui": "workspace:^",
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (8)
.changeset/odd-lines-care.md(0 hunks)apps/astro-party/package.json(1 hunks)apps/course-builder-web/package.json(1 hunks)apps/egghead/package.json(1 hunks)apps/epic-react/package.json(1 hunks)apps/go-local-first/package.json(1 hunks)packages/ui/CHANGELOG.md(1 hunks)packages/ui/package.json(1 hunks)
💤 Files with no reviewable changes (1)
- .changeset/odd-lines-care.md
✅ Files skipped from review due to trivial changes (2)
- packages/ui/CHANGELOG.md
- packages/ui/package.json
🚧 Files skipped from review as they are similar to previous changes (4)
- apps/astro-party/package.json
- apps/course-builder-web/package.json
- apps/go-local-first/package.json
- apps/epic-react/package.json
🧰 Additional context used
📓 Path-based instructions (2)
**/package.json
📄 CodeRabbit inference engine (CLAUDE.md)
**/package.json: When adding dependencies to package.json files, ensure all packages use consistent dependency versions and dependencies are sorted alphabetically.
When updating package.json files to add dependencies, use string replacement to add dependencies and maintain alphabetical order. Do not replace entire sections, just add the new line.
Files:
apps/egghead/package.json
apps/**/*
📄 CodeRabbit inference engine (.cursor/rules/monorepo-setup.mdc)
Apps are located in the /apps directory
Files:
apps/egghead/package.json
🔇 Additional comments (2)
apps/egghead/package.json (2)
37-37: Version bump looks good and matches the release intent.Upgrading @coursebuilder/ui to ^2.0.8 aligns with the release PR. No ordering issues in dependencies; the entry remains alphabetically positioned within the @coursebuilder/* block.
37-37: No Accordion usage detected – safe to upgradeI ran searches across apps/egghead and found numerous imports from @coursebuilder/ui but zero occurrences of an Accordion import or usage. Since there are no direct references to the Accordion component in this codebase, the internal upgrade poses no breaking risk here.
• Verified direct imports: all usages are of Button, Input, ScrollArea, Card, etc., but not Accordion
• Confirmed no “Accordion” symbol anywhere outside node_modulesNo further action required.
apps/egghead/package.json
Outdated
| "@coursebuilder/next": "workspace:^", | ||
| "@coursebuilder/nodash": "workspace:^", | ||
| "@coursebuilder/ui": "^2.0.7", | ||
| "@coursebuilder/ui": "^2.0.8", |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
Verify monorepo consistency for @coursebuilder/ui and catch any stragglers.
Ensure all apps/packages that depend on @coursebuilder/ui were bumped to ^2.0.8 (or use the workspace protocol) to avoid mixed versions at publish time.
Run this script to list and validate versions across the repo:
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "Scanning for @coursebuilder/ui versions in package.json files..."
rg -nP '"@coursebuilder/ui"\s*:\s*"(.*?)"' \
-g 'apps/**/package.json' -g 'packages/**/package.json' -g '!**/node_modules/**' -C0 || true
echo
echo "Unique versions found:"
VERSIONS=$(rg -nP '"@coursebuilder/ui"\s*:\s*"(.*?)"' \
-g 'apps/**/package.json' -g 'packages/**/package.json' -g '!**/node_modules/**' \
| sed -nE 's/.*"@coursebuilder\/ui"\s*:\s*"([^"]+)".*/\1/p' | sort -u)
echo "$VERSIONS"
echo
echo "Checking for nonconforming versions (expected: ^2.0.8 or workspace:^)..."
BAD=$(echo "$VERSIONS" | rg -v '^\^?2\.0\.8$|^workspace:\^$' || true)
if [ -n "$BAD" ]; then
echo "Found nonconforming versions:"
echo "$BAD"
exit 1
fi
echo "All good."Length of output: 1827
Ensure consistent @coursebuilder/ui versions across the monorepo
The scan uncovered two types of mismatches that must be fixed before publishing:
• packages/commerce-next/package.json (line 80)
- "@coursebuilder/ui": "^2.0.7",
+ "@coursebuilder/ui": "^2.0.8",• Five apps are using the wildcard Workspace protocol instead of the caret‐pinned Workspace protocol; update each of the following from
- "@coursebuilder/ui": "workspace:*",
+ "@coursebuilder/ui": "workspace:^",– apps/epic-web/package.json (line 63)
– apps/cursor-pro/package.json (line 63)
– apps/epicdev-ai/package.json (line 63)
– apps/ai-hero/package.json (line 63)
– apps/craft-of-ui/package.json (line 63)
These edits ensure every consumer is either pinned to ^2.0.8 or correctly using the workspace:^ protocol—preventing mixed versions at publish time.
Committable suggestion skipped: line range outside the PR's diff.
🤖 Prompt for AI Agents
In apps/egghead/package.json around line 37 (and also address the other reported
locations), the @coursebuilder/ui version is inconsistent across the repo;
update packages/commerce-next/package.json (line 80) to use the pinned version
"^2.0.8" and for the five apps listed (apps/epic-web/package.json line 63,
apps/cursor-pro/package.json line 63, apps/epicdev-ai/package.json line 63,
apps/ai-hero/package.json line 63, apps/craft-of-ui/package.json line 63)
replace the wildcard workspace protocol with the caret-pinned workspace protocol
(use "workspace:^2.0.8") so every consumer is either pinned to ^2.0.8 or uses
workspace:^2.0.8 consistently.
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
packages/utils-resource/package.json (1)
29-29: Update @coursebuilder/core dependency to concrete semver for release consistency.The CHANGELOG declares this package now depends on @coursebuilder/[email protected] (lines 3–17 of packages/utils-resource/CHANGELOG.md), but package.json still uses the workspace protocol. Replace the workspace reference with a concrete semver range to match the published changelog.
"dependencies": { - "@coursebuilder/core": "workspace:^", + "@coursebuilder/core": "^1.1.0", "zod": "^3.22.4" },
♻️ Duplicate comments (14)
packages/utils-email/CHANGELOG.md (1)
9-10: Add the missing period after "etc."The parenthetical abbreviation should end with "etc." rather than "etc)". This issue was previously flagged in reviews; please ensure correct punctuation in the new changelog entry.
Apply this diff to fix the punctuation:
- update related deps (react, react-dom, eslint, etc) + update related deps (react, react-dom, eslint, etc.)packages/mdx-mermaid/CHANGELOG.md (1)
9-10: Apply pending grammar and style fixes from the previous review.This mirrors the style issues flagged in the past review. Lines 9-10 need minor corrections for consistency and clarity:
- Capitalize product name "Next.js"
- Expand "deps" to "dependencies"
- Add period after "etc."
- Optionally wrap package names in backticks
Apply this diff to align with changelog conventions:
- Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade next to beta, - update related deps (react, react-dom, eslint, etc) + Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade Next.js to beta, + update related dependencies (`react`, `react-dom`, `eslint`, etc.).packages/config/eslint-config/CHANGELOG.md (1)
41-42: Capitalize the proper noun "React".Line 42 contains lowercase "react" that should be capitalized as "React" per standard English capitalization rules for proper nouns. This issue was flagged in a prior review and remains unresolved.
Apply this diff to fix the capitalization:
- Thanks [@joelhooks](https://github.com/joelhooks)! - Bumped to the canary - version of react ☠️ + Thanks [@joelhooks](https://github.com/joelhooks)! - Bumped to the canary + version of React ☠️packages/next/CHANGELOG.md (1)
9-10: Add period after "etc" abbreviation.American English requires a period after the abbreviation "etc."
- update related deps (react, react-dom, eslint, etc) + update related deps (react, react-dom, eslint, etc.)cli/CHANGELOG.md (1)
9-10: Add period after "etc" abbreviation.American English requires a period after the abbreviation "etc."
- update related deps (react, react-dom, eslint, etc) + update related deps (react, react-dom, eslint, etc.)packages/ui/CHANGELOG.md (1)
9-10: Add period to "etc." for proper punctuation.Line 10 ends "etc)" but should be "etc.)" to follow standard English grammar rules for abbreviations.
- update related deps (react, react-dom, eslint, etc) + update related deps (react, react-dom, eslint, etc.)packages/commerce-next/CHANGELOG.md (3)
20-21: Add period to "etc." for proper punctuation.Line 21 ends "etc)" but should be "etc.)" to follow standard English grammar rules for abbreviations.
- update related deps (react, react-dom, eslint, etc) + update related deps (react, react-dom, eslint, etc.)
38-39: Fix contraction: "dont" → "don't".Line 38 has the misspelled contraction "dont" which should be "don't" (with apostrophe).
- Thanks [@vojtaholik](https://github.com/vojtaholik)! - dont require - verify-login redirect + Thanks [@vojtaholik](https://github.com/vojtaholik)! - don't require + verify-login redirect
9-14: Use hyphen for compound adjective: "fixed-amount discount".When "fixed amount" modifies a noun (discount), it should be hyphenated as "fixed-amount" to follow compound adjective rules.
- Thanks [@zacjones93](https://github.com/zacjones93)! - Add fixed amount - discount functionality for merchant coupons with proper dollar-based pricing + Thanks [@zacjones93](https://github.com/zacjones93)! - Add fixed-amount + discount functionality for merchant coupons with proper dollar-based pricingpackages/core/CHANGELOG.md (4)
9-14: Use hyphen for compound adjective: "fixed-amount discount".When "fixed amount" modifies a noun (discount), it should be hyphenated as "fixed-amount" to follow compound adjective rules.
- Thanks [@zacjones93](https://github.com/zacjones93)! - Add fixed amount - discount functionality for merchant coupons with proper dollar-based pricing + Thanks [@zacjones93](https://github.com/zacjones93)! - Add fixed-amount + discount functionality for merchant coupons with proper dollar-based pricing
25-26: Add period to "etc." for proper punctuation.Line 26 ends "etc)" but should be "etc.)" to follow standard English grammar rules for abbreviations.
- update related deps (react, react-dom, eslint, etc) + update related deps (react, react-dom, eslint, etc.)
38-39: Fix contraction: "dont" → "don't".Line 38 has the misspelled contraction "dont" which should be "don't" (with apostrophe).
- Thanks [@vojtaholik](https://github.com/vojtaholik)! - dont require - verify-login redirect + Thanks [@vojtaholik](https://github.com/vojtaholik)! - don't require + verify-login redirect
42-43: Capitalize proper noun: "inngest" → "Inngest".Line 43 refers to the Inngest service, which is a proper noun and should be capitalized.
- Thanks [@vojtaholik](https://github.com/vojtaholik)! - limit retries on - remove-completed-video inngest function + Thanks [@vojtaholik](https://github.com/vojtaholik)! - limit retries on + remove-completed-video Inngest functionpackages/email-templates/CHANGELOG.md (1)
9-10: Add period to "etc." for proper punctuation.Line 10 ends "etc)" but should be "etc.)" to follow standard English grammar rules for abbreviations.
- update related deps (react, react-dom, eslint, etc) + update related deps (react, react-dom, eslint, etc.)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (58)
.changeset/clear-regions-search.md(0 hunks).changeset/common-boats-fold.md(0 hunks).changeset/cruel-symbols-grin.md(0 hunks).changeset/cute-ghosts-learn.md(0 hunks).changeset/five-nights-bathe.md(0 hunks).changeset/fixed-discount-feature.md(0 hunks).changeset/great-doors-attack.md(0 hunks).changeset/lazy-banks-film.md(0 hunks).changeset/odd-lines-care.md(0 hunks).changeset/plenty-dodos-search.md(0 hunks).changeset/quiet-bags-raise.md(0 hunks).changeset/rude-peaches-like.md(0 hunks).changeset/salty-birds-sell.md(0 hunks).changeset/shaky-crews-wonder.md(0 hunks).changeset/slow-eels-behave.md(0 hunks).changeset/smart-candles-vanish.md(0 hunks).changeset/spicy-kids-count.md(0 hunks).changeset/stale-forks-look.md(0 hunks).changeset/stupid-beans-move.md(0 hunks).changeset/stupid-parts-end.md(0 hunks).changeset/tall-ideas-sip.md(0 hunks).changeset/tame-rings-show.md(0 hunks).changeset/tasty-keys-post.md(0 hunks).changeset/tender-things-cheer.md(0 hunks)apps/astro-party/package.json(1 hunks)apps/course-builder-web/package.json(1 hunks)apps/epic-react/package.json(1 hunks)apps/go-local-first/package.json(1 hunks)cli/CHANGELOG.md(1 hunks)cli/package.json(1 hunks)packages/adapter-drizzle/CHANGELOG.md(1 hunks)packages/adapter-drizzle/package.json(2 hunks)packages/astro/CHANGELOG.md(1 hunks)packages/astro/package.json(1 hunks)packages/commerce-next/CHANGELOG.md(1 hunks)packages/commerce-next/package.json(2 hunks)packages/config/eslint-config/CHANGELOG.md(2 hunks)packages/config/eslint-config/package.json(1 hunks)packages/core/CHANGELOG.md(1 hunks)packages/core/package.json(2 hunks)packages/email-templates/CHANGELOG.md(1 hunks)packages/email-templates/package.json(1 hunks)packages/mdx-mermaid/CHANGELOG.md(1 hunks)packages/mdx-mermaid/package.json(1 hunks)packages/next/CHANGELOG.md(1 hunks)packages/next/package.json(2 hunks)packages/nodash/CHANGELOG.md(1 hunks)packages/nodash/package.json(1 hunks)packages/survey/CHANGELOG.md(1 hunks)packages/survey/package.json(1 hunks)packages/ui/CHANGELOG.md(1 hunks)packages/ui/package.json(2 hunks)packages/utils-email/CHANGELOG.md(1 hunks)packages/utils-email/package.json(1 hunks)packages/utils-resource/CHANGELOG.md(1 hunks)packages/utils-resource/package.json(1 hunks)packages/utils-search/CHANGELOG.md(1 hunks)packages/utils-search/package.json(1 hunks)
💤 Files with no reviewable changes (24)
- .changeset/stale-forks-look.md
- .changeset/cute-ghosts-learn.md
- .changeset/tender-things-cheer.md
- .changeset/five-nights-bathe.md
- .changeset/salty-birds-sell.md
- .changeset/clear-regions-search.md
- .changeset/plenty-dodos-search.md
- .changeset/tasty-keys-post.md
- .changeset/slow-eels-behave.md
- .changeset/great-doors-attack.md
- .changeset/cruel-symbols-grin.md
- .changeset/shaky-crews-wonder.md
- .changeset/rude-peaches-like.md
- .changeset/tall-ideas-sip.md
- .changeset/spicy-kids-count.md
- .changeset/fixed-discount-feature.md
- .changeset/lazy-banks-film.md
- .changeset/stupid-parts-end.md
- .changeset/odd-lines-care.md
- .changeset/smart-candles-vanish.md
- .changeset/quiet-bags-raise.md
- .changeset/stupid-beans-move.md
- .changeset/common-boats-fold.md
- .changeset/tame-rings-show.md
✅ Files skipped from review due to trivial changes (2)
- packages/utils-search/CHANGELOG.md
- packages/nodash/package.json
🚧 Files skipped from review as they are similar to previous changes (14)
- apps/epic-react/package.json
- packages/mdx-mermaid/package.json
- packages/email-templates/package.json
- cli/package.json
- apps/astro-party/package.json
- packages/config/eslint-config/package.json
- packages/commerce-next/package.json
- packages/next/package.json
- packages/core/package.json
- packages/survey/CHANGELOG.md
- packages/nodash/CHANGELOG.md
- packages/survey/package.json
- packages/utils-email/package.json
- apps/go-local-first/package.json
🧰 Additional context used
📓 Path-based instructions (4)
packages/**/*
📄 CodeRabbit inference engine (.cursor/rules/monorepo-setup.mdc)
Packages are located in the /packages directory
Files:
packages/astro/CHANGELOG.mdpackages/ui/package.jsonpackages/utils-resource/package.jsonpackages/adapter-drizzle/CHANGELOG.mdpackages/utils-resource/CHANGELOG.mdpackages/commerce-next/CHANGELOG.mdpackages/ui/CHANGELOG.mdpackages/adapter-drizzle/package.jsonpackages/astro/package.jsonpackages/mdx-mermaid/CHANGELOG.mdpackages/utils-search/package.jsonpackages/config/eslint-config/CHANGELOG.mdpackages/core/CHANGELOG.mdpackages/email-templates/CHANGELOG.mdpackages/next/CHANGELOG.mdpackages/utils-email/CHANGELOG.md
**/package.json
📄 CodeRabbit inference engine (CLAUDE.md)
**/package.json: When adding dependencies to package.json files, ensure all packages use consistent dependency versions and dependencies are sorted alphabetically.
When updating package.json files to add dependencies, use string replacement to add dependencies and maintain alphabetical order. Do not replace entire sections, just add the new line.
Files:
packages/ui/package.jsonpackages/utils-resource/package.jsonpackages/adapter-drizzle/package.jsonpackages/astro/package.jsonapps/course-builder-web/package.jsonpackages/utils-search/package.json
packages/utils-*/package.json
📄 CodeRabbit inference engine (CLAUDE.md)
When creating utility packages that interact with framework-specific libraries, keep framework-specific dependencies (React, Next.js, etc.) as peer dependencies.
Files:
packages/utils-resource/package.jsonpackages/utils-search/package.json
apps/**/*
📄 CodeRabbit inference engine (.cursor/rules/monorepo-setup.mdc)
Apps are located in the /apps directory
Files:
apps/course-builder-web/package.json
🧠 Learnings (8)
📚 Learning: 2025-07-22T11:50:39.243Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-07-22T11:50:39.243Z
Learning: Applies to **/package.json : When adding dependencies to package.json files, ensure all packages use consistent dependency versions and dependencies are sorted alphabetically.
Applied to files:
packages/utils-resource/CHANGELOG.mdpackages/adapter-drizzle/package.jsonapps/course-builder-web/package.json
📚 Learning: 2025-07-22T11:50:39.243Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-07-22T11:50:39.243Z
Learning: Applies to **/package.json : When updating package.json files to add dependencies, use string replacement to add dependencies and maintain alphabetical order. Do not replace entire sections, just add the new line.
Applied to files:
packages/utils-resource/CHANGELOG.mdpackages/adapter-drizzle/package.jsonpackages/mdx-mermaid/CHANGELOG.mdapps/course-builder-web/package.json
📚 Learning: 2025-10-14T22:05:16.978Z
Learnt from: zacjones93
Repo: badass-courses/course-builder PR: 578
File: apps/astro-party/src/trpc/api/routers/pricing.ts:140-141
Timestamp: 2025-10-14T22:05:16.978Z
Learning: The fixed discount feature (amountDiscount support) was only implemented in the ai-hero app. Other apps in the monorepo (astro-party, code-with-antonio, course-builder-web, epicdev-ai, craft-of-ui, cursor-pro, epic-web, go-local-first) still only support percentage discounts in their application code, even though the core pricing infrastructure supports both types.
Applied to files:
packages/commerce-next/CHANGELOG.mdpackages/core/CHANGELOG.md
📚 Learning: 2025-10-10T17:13:02.161Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: apps/code-with-antonio/.cursor/rules/component-system.mdc:0-0
Timestamp: 2025-10-10T17:13:02.161Z
Learning: Applies to apps/code-with-antonio/packages/ui/**/*.{ts,tsx} : Keep shared, reusable UI components under the monorepo package at /packages/ui
Applied to files:
apps/course-builder-web/package.json
📚 Learning: 2025-07-22T11:50:39.243Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-07-22T11:50:39.243Z
Learning: Applies to packages/utils-*/package.json : When creating utility packages that interact with framework-specific libraries, keep framework-specific dependencies (React, Next.js, etc.) as peer dependencies.
Applied to files:
apps/course-builder-web/package.json
📚 Learning: 2025-07-23T10:53:00.058Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: apps/epicdev-ai/.cursor/rules/component-system.mdc:0-0
Timestamp: 2025-07-23T10:53:00.058Z
Learning: Applies to apps/epicdev-ai/**/*.tsx : Use base React components when possible
Applied to files:
packages/config/eslint-config/CHANGELOG.md
📚 Learning: 2025-07-22T12:20:03.175Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: apps/craft-of-ui/.cursor/rules/component-system.mdc:0-0
Timestamp: 2025-07-22T12:20:03.175Z
Learning: Applies to apps/craft-of-ui/**/*.tsx : Use base React components when possible
Applied to files:
packages/config/eslint-config/CHANGELOG.md
📚 Learning: 2025-07-22T21:59:25.155Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: apps/epic-web/.cursor/rules/component-system.mdc:0-0
Timestamp: 2025-07-22T21:59:25.155Z
Learning: Applies to apps/epic-web/**/*.tsx : Use base React components when possible
Applied to files:
packages/config/eslint-config/CHANGELOG.md
🪛 LanguageTool
packages/commerce-next/CHANGELOG.md
[grammar] ~9-~9: Use a hyphen to join words.
Context: ...ps://github.com/zacjones93)! - Add fixed amount discount functionality for merc...
(QB_NEW_EN_HYPHEN)
[style] ~21-~21: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) - [#588](https://github.com/badass-co...
(ETC_PERIOD)
packages/ui/CHANGELOG.md
[style] ~10-~10: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) - [#588](https://github.com/badass-co...
(ETC_PERIOD)
packages/mdx-mermaid/CHANGELOG.md
[style] ~10-~10: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) ## 0.1.1 ### Patch Changes - [#534]...
(ETC_PERIOD)
packages/config/eslint-config/CHANGELOG.md
[style] ~41-~41: Using many exclamation marks might seem excessive (in this case: 3 exclamation marks for a text that’s 1623 characters long)
Context: ...joelhooks](https://github.com/joelhooks)! - Bumped to the canary version of rea...
(EN_EXCESSIVE_EXCLAMATION)
[uncategorized] ~42-~42: “React” is a proper noun and needs to be capitalized.
Context: ...s)! - Bumped to the canary version of react ☠️ - Updated dependencies [[`ecd40cb...
(A_GOOGLE)
packages/core/CHANGELOG.md
[grammar] ~9-~9: Use a hyphen to join words.
Context: ...ps://github.com/zacjones93)! - Add fixed amount discount functionality for merc...
(QB_NEW_EN_HYPHEN)
[style] ~26-~26: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) - [1b92a3a](https://github.com/bada...
(ETC_PERIOD)
[grammar] ~43-~43: Ensure spelling is correct
Context: ...mit retries on remove-completed-video inngest function - [#568](https://github.com/b...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
[grammar] ~43-~43: Ensure spelling is correct
Context: ... remove-completed-video inngest function - [#568](https://github.com/badass-courses/...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
packages/email-templates/CHANGELOG.md
[style] ~10-~10: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) ## 1.0.6 ### Patch Changes - [#534]...
(ETC_PERIOD)
packages/next/CHANGELOG.md
[style] ~10-~10: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) - [#589](https://github.com/badass-co...
(ETC_PERIOD)
packages/utils-email/CHANGELOG.md
[style] ~10-~10: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) ## 1.0.1 ### Patch Changes - [#534]...
(ETC_PERIOD)
🔇 Additional comments (10)
packages/config/eslint-config/CHANGELOG.md (2)
3-11: ✅ Well-formatted new release entry.The new 0.1.1 patch section is properly structured with PR link, commit hash, author attribution, and clear description of the bump from beta to stable.
12-25: ✅ Properly reformatted version section.The 0.1.0 section has been correctly expanded with proper PR reference, commit link, author attribution, and updated dependency tracking.
packages/adapter-drizzle/CHANGELOG.md (1)
3-40: Changelog entry looks good.The 1.0.5 patch version entry follows the established Changesets format consistently and accurately documents all referenced changes from the PR objectives, including cohort/workshop adapter extraction, bug fixes, dependency bumps, and new features. All PR numbers and commit hashes are properly formatted with appropriate attribution.
packages/utils-search/package.json (1)
3-3: Version bump looks good.Patch-level version bump with no dependency or configuration changes.
packages/astro/CHANGELOG.md (1)
3-8: Changelog entry looks good.Properly formatted Changesets entry with correct commit references and dependency declaration.
packages/astro/package.json (1)
4-4: Version bump to 1.0.0 approved.Reflects the package stabilization documented in the changelog. No dependency or configuration changes.
packages/ui/package.json (1)
3-3: Version bump and dependency update approved.Patch version bump with correct minor-version update to @coursebuilder/[email protected]. Dependencies remain sorted alphabetically.
Also applies to: 20-20
packages/utils-resource/CHANGELOG.md (1)
3-17: Changelog entry is properly formatted.Standard Changesets entry with correct commit references and dependency declaration. (See critical issue flagged on package.json for dependency version alignment.)
packages/adapter-drizzle/package.json (1)
3-3: ✅ Version and dependency updates are consistent and well-ordered.The patch version bump (1.0.4 → 1.0.5) and dependency updates to @coursebuilder/[email protected] and @coursebuilder/[email protected] align with the PR objectives. Dependencies remain alphabetically sorted within their sections.
Also applies to: 46-46, 54-54
apps/course-builder-web/package.json (1)
45-56: ✅ Dependency updates are coordinated and properly ordered.All @coursebuilder/* dependencies have been updated to versions matching the release ([email protected], commerce-next@^0.1.0, [email protected], email-templates@^1.0.7, ui@^2.0.8, [email protected]). Dependencies remain alphabetically sorted within their section.
| stable | ||
|
|
There was a problem hiding this comment.
Capitalize proper noun: "inngest" → "Inngest".
Line 43 refers to the Inngest service, which is a proper noun and should be capitalized.
- Thanks [@vojtaholik](https://github.com/vojtaholik)! - limit retries on
- remove-completed-video inngest function
+ Thanks [@vojtaholik](https://github.com/vojtaholik)! - limit retries on
+ remove-completed-video Inngest functionCommittable suggestion skipped: line range outside the PR's diff.
🤖 Prompt for AI Agents
In packages/commerce-next/CHANGELOG.md around lines 42 to 43, the changelog
mentions the service name "inngest" in lowercase; update that occurrence to the
capitalized proper noun "Inngest" so the product name is correctly capitalized.
6511751 to
b660ca3
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (10)
packages/email-templates/CHANGELOG.md (1)
9-10: Add period after "etc" abbreviation.Line 10 is missing a period after the "etc" abbreviation. This is the same punctuation issue flagged in the previous review but appears to have persisted.
Apply this diff:
Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade next to beta, - update related deps (react, react-dom, eslint, etc) + update related deps (react, react-dom, eslint, etc.)packages/utils-email/CHANGELOG.md (1)
10-10: Fix missing period after "etc." abbreviation.This punctuation issue was flagged in previous reviews. Line 10 should end with "etc.)" instead of "etc)" for correct American English style.
- update related deps (react, react-dom, eslint, etc) + update related deps (react, react-dom, eslint, etc.)packages/mdx-mermaid/CHANGELOG.md (1)
9-10: Fix style issues in changelog entry (previously flagged).This changelog entry still contains the style issues flagged in the past review. Please apply these corrections:
- Capitalize the product name: "next" → "Next.js"
- Replace abbreviation with full term: "deps" → "dependencies"
- Add required period to abbreviation: "etc" → "etc."
- Optionally wrap package names in backticks for consistency
- Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade next to beta, - update related deps (react, react-dom, eslint, etc) + Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade Next.js to beta, + update related dependencies (`react`, `react-dom`, `eslint`, etc.).cli/CHANGELOG.md (1)
9-10: Punctuation nit: add period after "etc." — same issue flagged in prior review.This appears to be the same unresolved punctuation issue previously flagged. The text should read "(react, react-dom, eslint, etc.)" with a period after the abbreviation.
- Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade next to beta, - update related deps (react, react-dom, eslint, etc) + Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade next to beta, + update related deps (react, react-dom, eslint, etc.)packages/next/CHANGELOG.md (1)
9-10: Add period after "etc" to complete abbreviation.Line 10 ends with "etc)" but should be "etc.)" to follow American English punctuation conventions for abbreviations.
- Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade next to beta, - update related deps (react, react-dom, eslint, etc) + Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade next to beta, + update related deps (react, react-dom, eslint, etc.)packages/ui/CHANGELOG.md (1)
9-10: Add period after "etc" abbreviation.Line 10 should read "etc." instead of "etc)" to complete the abbreviation properly.
- Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade next to beta, - update related deps (react, react-dom, eslint, etc) + Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade next to beta, + update related deps (react, react-dom, eslint, etc.)packages/commerce-next/CHANGELOG.md (3)
9-14: Hyphenate compound modifier: "fixed amount discount" → "fixed-amount discount".Line 9 uses the unhyphenated form "fixed amount discount" but should use a hyphenated compound adjective.
- Thanks [@zacjones93](https://github.com/zacjones93)! - Add fixed amount - discount functionality for merchant coupons with proper dollar-based pricing + Thanks [@zacjones93](https://github.com/zacjones93)! - Add fixed-amount + discount functionality for merchant coupons with proper dollar-based pricing
20-21: Add period after "etc" abbreviation.Line 21 should read "etc." instead of "etc)" to follow American English punctuation conventions.
- Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade next to beta, - update related deps (react, react-dom, eslint, etc) + Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade next to beta, + update related deps (react, react-dom, eslint, etc.)
49-51: Correct the contraction: "dont" → "don't".Line 50 uses the incorrect spelling "dont" instead of the proper contraction "don't".
- Thanks [@vojtaholik](https://github.com/vojtaholik)! - dont require - verify-login redirect + Thanks [@vojtaholik](https://github.com/vojtaholik)! - don't require + verify-login redirectapps/astro-party/package.json (1)
54-55: Replace workspace protocol with published versions.In a release PR, dependency entries should reference published semver versions, not workspace references. Lines 54-55 use workspace protocol while all other @coursebuilder/* dependencies reference published versions. Update these entries to use the released versions:
- "@coursebuilder/next": "workspace:^", - "@coursebuilder/nodash": "workspace:^", + "@coursebuilder/next": "^0.0.29", + "@coursebuilder/nodash": "^0.0.2",
🧹 Nitpick comments (5)
packages/core/CHANGELOG.md (4)
9-10: Hyphenate compound adjective: "fixed amount" → "fixed-amount".When used as a compound adjective before a noun, use a hyphen to join the words.
- Thanks [@zacjones93](https://github.com/zacjones93)! - Add fixed amount + Thanks [@zacjones93](https://github.com/zacjones93)! - Add fixed-amount
25-26: Add period to abbreviation: "etc)" → "etc.)".In American English, abbreviations such as "etc." require a period.
Thanks [@vojtaholik](https://github.com/vojtaholik)! - upgrade next to beta, - update related deps (react, react-dom, eslint, etc) + update related deps (react, react-dom, eslint, etc.)
38-39: Correct contraction: "dont" → "don't".Add the apostrophe to the contraction.
Thanks [@vojtaholik](https://github.com/vojtaholik)! - dont require + Thanks [@vojtaholik](https://github.com/vojtaholik)! - don't require
42-43: Capitalize proper noun: "inngest" → "Inngest".Inngest is a service name and should be capitalized.
Thanks [@vojtaholik](https://github.com/vojtaholik)! - limit retries on - remove-completed-video inngest function + remove-completed-video Inngest functionpackages/config/eslint-config/CHANGELOG.md (1)
41-42: Capitalize proper noun: "react" → "React".React is a proper noun (framework name) and should be capitalized.
Thanks [@joelhooks](https://github.com/joelhooks)! - Bumped to the canary - version of react ☠️ + version of React ☠️
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (59)
.changeset/clear-regions-search.md(0 hunks).changeset/common-boats-fold.md(0 hunks).changeset/cruel-symbols-grin.md(0 hunks).changeset/cute-ghosts-learn.md(0 hunks).changeset/five-nights-bathe.md(0 hunks).changeset/fixed-discount-feature.md(0 hunks).changeset/great-doors-attack.md(0 hunks).changeset/lazy-banks-film.md(0 hunks).changeset/odd-lines-care.md(0 hunks).changeset/plenty-dodos-search.md(0 hunks).changeset/quiet-bags-raise.md(0 hunks).changeset/rude-peaches-like.md(0 hunks).changeset/salty-birds-sell.md(0 hunks).changeset/shaky-crews-wonder.md(0 hunks).changeset/slow-eels-behave.md(0 hunks).changeset/smart-candles-vanish.md(0 hunks).changeset/spicy-kids-count.md(0 hunks).changeset/stale-forks-look.md(0 hunks).changeset/stupid-beans-move.md(0 hunks).changeset/stupid-parts-end.md(0 hunks).changeset/tall-ideas-sip.md(0 hunks).changeset/tame-rings-show.md(0 hunks).changeset/tangy-candles-hammer.md(0 hunks).changeset/tasty-keys-post.md(0 hunks).changeset/tender-things-cheer.md(0 hunks)apps/astro-party/package.json(1 hunks)apps/course-builder-web/package.json(1 hunks)apps/epic-react/package.json(1 hunks)apps/go-local-first/package.json(1 hunks)cli/CHANGELOG.md(1 hunks)cli/package.json(1 hunks)packages/adapter-drizzle/CHANGELOG.md(1 hunks)packages/adapter-drizzle/package.json(2 hunks)packages/astro/CHANGELOG.md(1 hunks)packages/astro/package.json(1 hunks)packages/commerce-next/CHANGELOG.md(1 hunks)packages/commerce-next/package.json(2 hunks)packages/config/eslint-config/CHANGELOG.md(2 hunks)packages/config/eslint-config/package.json(1 hunks)packages/core/CHANGELOG.md(1 hunks)packages/core/package.json(2 hunks)packages/email-templates/CHANGELOG.md(1 hunks)packages/email-templates/package.json(1 hunks)packages/mdx-mermaid/CHANGELOG.md(1 hunks)packages/mdx-mermaid/package.json(1 hunks)packages/next/CHANGELOG.md(1 hunks)packages/next/package.json(2 hunks)packages/nodash/CHANGELOG.md(1 hunks)packages/nodash/package.json(1 hunks)packages/survey/CHANGELOG.md(1 hunks)packages/survey/package.json(1 hunks)packages/ui/CHANGELOG.md(1 hunks)packages/ui/package.json(2 hunks)packages/utils-email/CHANGELOG.md(1 hunks)packages/utils-email/package.json(1 hunks)packages/utils-resource/CHANGELOG.md(1 hunks)packages/utils-resource/package.json(1 hunks)packages/utils-search/CHANGELOG.md(1 hunks)packages/utils-search/package.json(1 hunks)
💤 Files with no reviewable changes (25)
- .changeset/spicy-kids-count.md
- .changeset/lazy-banks-film.md
- .changeset/slow-eels-behave.md
- .changeset/smart-candles-vanish.md
- .changeset/stupid-beans-move.md
- .changeset/quiet-bags-raise.md
- .changeset/common-boats-fold.md
- .changeset/cute-ghosts-learn.md
- .changeset/stupid-parts-end.md
- .changeset/stale-forks-look.md
- .changeset/five-nights-bathe.md
- .changeset/cruel-symbols-grin.md
- .changeset/plenty-dodos-search.md
- .changeset/clear-regions-search.md
- .changeset/tall-ideas-sip.md
- .changeset/tangy-candles-hammer.md
- .changeset/fixed-discount-feature.md
- .changeset/tender-things-cheer.md
- .changeset/rude-peaches-like.md
- .changeset/tame-rings-show.md
- .changeset/tasty-keys-post.md
- .changeset/odd-lines-care.md
- .changeset/salty-birds-sell.md
- .changeset/shaky-crews-wonder.md
- .changeset/great-doors-attack.md
✅ Files skipped from review due to trivial changes (3)
- packages/nodash/CHANGELOG.md
- packages/utils-search/package.json
- packages/utils-resource/CHANGELOG.md
🚧 Files skipped from review as they are similar to previous changes (15)
- packages/survey/package.json
- cli/package.json
- packages/utils-search/CHANGELOG.md
- packages/commerce-next/package.json
- packages/mdx-mermaid/package.json
- packages/utils-resource/package.json
- packages/astro/package.json
- packages/adapter-drizzle/CHANGELOG.md
- packages/survey/CHANGELOG.md
- apps/course-builder-web/package.json
- packages/core/package.json
- packages/astro/CHANGELOG.md
- packages/adapter-drizzle/package.json
- packages/email-templates/package.json
- packages/ui/package.json
🧰 Additional context used
📓 Path-based instructions (4)
**/package.json
📄 CodeRabbit inference engine (CLAUDE.md)
**/package.json: When adding dependencies to package.json files, ensure all packages use consistent dependency versions and dependencies are sorted alphabetically.
When updating package.json files to add dependencies, use string replacement to add dependencies and maintain alphabetical order. Do not replace entire sections, just add the new line.
Files:
packages/nodash/package.jsonapps/go-local-first/package.jsonapps/epic-react/package.jsonpackages/next/package.jsonpackages/utils-email/package.jsonpackages/config/eslint-config/package.jsonapps/astro-party/package.json
packages/**/*
📄 CodeRabbit inference engine (.cursor/rules/monorepo-setup.mdc)
Packages are located in the /packages directory
Files:
packages/nodash/package.jsonpackages/ui/CHANGELOG.mdpackages/next/package.jsonpackages/mdx-mermaid/CHANGELOG.mdpackages/utils-email/CHANGELOG.mdpackages/utils-email/package.jsonpackages/config/eslint-config/package.jsonpackages/commerce-next/CHANGELOG.mdpackages/config/eslint-config/CHANGELOG.mdpackages/core/CHANGELOG.mdpackages/email-templates/CHANGELOG.mdpackages/next/CHANGELOG.md
apps/**/*
📄 CodeRabbit inference engine (.cursor/rules/monorepo-setup.mdc)
Apps are located in the /apps directory
Files:
apps/go-local-first/package.jsonapps/epic-react/package.jsonapps/astro-party/package.json
packages/utils-*/package.json
📄 CodeRabbit inference engine (CLAUDE.md)
When creating utility packages that interact with framework-specific libraries, keep framework-specific dependencies (React, Next.js, etc.) as peer dependencies.
Files:
packages/utils-email/package.json
🧠 Learnings (9)
📚 Learning: 2025-07-22T11:50:39.243Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-07-22T11:50:39.243Z
Learning: Applies to **/package.json : When adding dependencies to package.json files, ensure all packages use consistent dependency versions and dependencies are sorted alphabetically.
Applied to files:
packages/nodash/package.jsonapps/go-local-first/package.jsonpackages/config/eslint-config/package.jsonapps/astro-party/package.json
📚 Learning: 2025-07-22T11:50:39.243Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-07-22T11:50:39.243Z
Learning: Applies to **/package.json : When updating package.json files to add dependencies, use string replacement to add dependencies and maintain alphabetical order. Do not replace entire sections, just add the new line.
Applied to files:
apps/go-local-first/package.jsonpackages/mdx-mermaid/CHANGELOG.mdapps/astro-party/package.json
📚 Learning: 2025-07-22T11:50:39.243Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-07-22T11:50:39.243Z
Learning: Applies to packages/utils-*/package.json : When creating utility packages that interact with framework-specific libraries, keep framework-specific dependencies (React, Next.js, etc.) as peer dependencies.
Applied to files:
apps/go-local-first/package.jsonapps/epic-react/package.jsonapps/astro-party/package.json
📚 Learning: 2025-07-23T10:53:08.297Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: apps/epicdev-ai/.cursor/rules/jsdoc-comments.mdc:0-0
Timestamp: 2025-07-23T10:53:08.297Z
Learning: Applies to apps/epicdev-ai/**/*.{ts,tsx} : Add JSDoc comments to exported methods and React components in TypeScript files
Applied to files:
apps/epic-react/package.json
📚 Learning: 2025-07-22T21:59:36.674Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: apps/epic-web/.cursor/rules/jsdoc-comments.mdc:0-0
Timestamp: 2025-07-22T21:59:36.674Z
Learning: Applies to apps/epic-web/**/*.{ts,tsx} : Add JSDoc comments to exported methods and React components
Applied to files:
apps/epic-react/package.json
📚 Learning: 2025-10-14T22:05:16.978Z
Learnt from: zacjones93
Repo: badass-courses/course-builder PR: 578
File: apps/astro-party/src/trpc/api/routers/pricing.ts:140-141
Timestamp: 2025-10-14T22:05:16.978Z
Learning: The fixed discount feature (amountDiscount support) was only implemented in the ai-hero app. Other apps in the monorepo (astro-party, code-with-antonio, course-builder-web, epicdev-ai, craft-of-ui, cursor-pro, epic-web, go-local-first) still only support percentage discounts in their application code, even though the core pricing infrastructure supports both types.
Applied to files:
packages/commerce-next/CHANGELOG.mdpackages/core/CHANGELOG.md
📚 Learning: 2025-07-23T10:53:00.058Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: apps/epicdev-ai/.cursor/rules/component-system.mdc:0-0
Timestamp: 2025-07-23T10:53:00.058Z
Learning: Applies to apps/epicdev-ai/**/*.tsx : Use base React components when possible
Applied to files:
packages/config/eslint-config/CHANGELOG.md
📚 Learning: 2025-07-22T12:20:03.175Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: apps/craft-of-ui/.cursor/rules/component-system.mdc:0-0
Timestamp: 2025-07-22T12:20:03.175Z
Learning: Applies to apps/craft-of-ui/**/*.tsx : Use base React components when possible
Applied to files:
packages/config/eslint-config/CHANGELOG.md
📚 Learning: 2025-07-22T21:59:25.155Z
Learnt from: CR
Repo: badass-courses/course-builder PR: 0
File: apps/epic-web/.cursor/rules/component-system.mdc:0-0
Timestamp: 2025-07-22T21:59:25.155Z
Learning: Applies to apps/epic-web/**/*.tsx : Use base React components when possible
Applied to files:
packages/config/eslint-config/CHANGELOG.md
🪛 LanguageTool
packages/ui/CHANGELOG.md
[style] ~10-~10: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) - [#588](https://github.com/badass-co...
(ETC_PERIOD)
packages/mdx-mermaid/CHANGELOG.md
[style] ~10-~10: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) ## 0.1.1 ### Patch Changes - [#534]...
(ETC_PERIOD)
packages/utils-email/CHANGELOG.md
[style] ~10-~10: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) ## 1.0.1 ### Patch Changes - [#534]...
(ETC_PERIOD)
packages/commerce-next/CHANGELOG.md
[grammar] ~9-~9: Use a hyphen to join words.
Context: ...ps://github.com/zacjones93)! - Add fixed amount discount functionality for merc...
(QB_NEW_EN_HYPHEN)
[style] ~21-~21: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) - [#588](https://github.com/badass-co...
(ETC_PERIOD)
packages/config/eslint-config/CHANGELOG.md
[style] ~41-~41: Using many exclamation marks might seem excessive (in this case: 3 exclamation marks for a text that’s 1623 characters long)
Context: ...joelhooks](https://github.com/joelhooks)! - Bumped to the canary version of rea...
(EN_EXCESSIVE_EXCLAMATION)
[uncategorized] ~42-~42: “React” is a proper noun and needs to be capitalized.
Context: ...s)! - Bumped to the canary version of react ☠️ - Updated dependencies [[`ecd40cb...
(A_GOOGLE)
packages/core/CHANGELOG.md
[grammar] ~9-~9: Use a hyphen to join words.
Context: ...ps://github.com/zacjones93)! - Add fixed amount discount functionality for merc...
(QB_NEW_EN_HYPHEN)
[style] ~26-~26: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) - [1b92a3a](https://github.com/bada...
(ETC_PERIOD)
[grammar] ~43-~43: Ensure spelling is correct
Context: ...mit retries on remove-completed-video inngest function - [#568](https://github.com/b...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
[grammar] ~43-~43: Ensure spelling is correct
Context: ... remove-completed-video inngest function - [#568](https://github.com/badass-courses/...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
packages/email-templates/CHANGELOG.md
[style] ~10-~10: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) ## 1.0.6 ### Patch Changes - [#534]...
(ETC_PERIOD)
packages/next/CHANGELOG.md
[style] ~10-~10: In American English, abbreviations like “etc.” require a period.
Context: ...related deps (react, react-dom, eslint, etc) - [#589](https://github.com/badass-co...
(ETC_PERIOD)
🔇 Additional comments (6)
packages/nodash/package.json (1)
3-3: Version bump is correct and properly aligned with the release.The version is correctly updated to 0.0.2 as part of the Changesets release process. DevDependencies remain alphabetically sorted.
packages/config/eslint-config/package.json (1)
3-3: Version bump aligns with release schedule.Straightforward patch version bump for this package in the coordinated release.
packages/next/package.json (1)
3-3: Coordinated version and dependency updates look good.Version bump to 0.0.29 and core dependency update to 1.1.0 are properly coordinated with the release.
Also applies to: 37-37
packages/utils-email/package.json (1)
3-3: Patch version bump is coordinated with release.Version updated to 1.0.2 as part of the coordinated package releases.
apps/epic-react/package.json (1)
26-27: Dependency updates are properly coordinated and sorted.All @coursebuilder package updates (core→1.1.0, ui→^2.0.8, adapter-drizzle→1.0.5, utils-email→1.0.2) maintain alphabetical ordering and align with the overall release strategy.
Also applies to: 30-30, 34-34
apps/go-local-first/package.json (1)
56-57: Revert to the existing code—the review comment is incorrect.The current use of
workspace:^for@coursebuilder/nextand@coursebuilder/nodashis correct and consistent with the monorepo's established dependency strategy. All other apps in the codebase (code-with-antonio, ai-hero, epic-web, epicdev-ai, egghead, epic-react) useworkspace:^for these same internal packages. Theworkspace:protocol is the proper pattern for monorepo dependency management, enabling coordinated releases and local development. Publishing these as external versions would contradict the monorepo's structure.Likely an incorrect or invalid review comment.
github-actions
bot
force-pushed
the
changeset-release/main
branch
from
8f4f0cd to
caddf0f
Compare
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub. |
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
@coursebuilder/[email protected]
Minor Changes
f11c60fThanks @zacjones93! - Add fixed amount
discount functionality for merchant coupons with proper dollar-based pricing
throughout the pricing system. This includes converting fixed discount amounts
from cents to dollars (breaking change), displaying fixed discount amounts in
pricing components, and adding shared pricing UI components for discount
visualization.
Patch Changes
#579
5d94d4eThanks @vojtaholik! - upgrade next to beta,
update related deps (react, react-dom, eslint, etc)
#588
43405a9Thanks @vojtaholik! - bump mux
#600
1660ec0Thanks @vojtaholik! - use client in pricing
root
9a1d398Thanks @vojtaholik! - style tweaks
#595
e58242eThanks @vojtaholik! - bump mux player
#589
f8dffc9Thanks @vojtaholik! - bump next from beta to
stable
#596
896c68cThanks @vojtaholik! - add QuestionResponses
table
79764deThanks @vojtaholik! - dont require
verify-login redirect
78595efThanks @vojtaholik! - bump mux player
8267b98Thanks @vojtaholik! - chore: make sure
availableCoupons is array
Updated dependencies
[
fff302b,5d94d4e,1b92a3a,f11c60f,43405a9,4134b05,14a6804,a4dce03,9a1d398,e58242e,9368a4c,896c68c,79764de,22fe75f,78595ef,a2a97ca,b2fd748,8267b98,942042c,06a993f]:@coursebuilder/[email protected]
Minor Changes
f11c60fThanks @zacjones93! - Add fixed amount
discount functionality for merchant coupons with proper dollar-based pricing
throughout the pricing system. This includes converting fixed discount amounts
from cents to dollars (breaking change), displaying fixed discount amounts in
pricing components, and adding shared pricing UI components for discount
visualization.
Patch Changes
#587
fff302bThanks @vojtaholik! - extract cohort and
workshop creation to adapter and add tests
#579
5d94d4eThanks @vojtaholik! - upgrade next to beta,
update related deps (react, react-dom, eslint, etc)
1b92a3aThanks @vojtaholik! - add openEnrollment and
closeEnrollment fields to cohort creation
4134b05Thanks @vojtaholik! - add optional retryCount
to VideoStatusCheckEvent schema and enhance error handling in
removeCompletedVideo function
79764deThanks @vojtaholik! - dont require
verify-login redirect
22fe75fThanks @vojtaholik! - limit retries on
remove-completed-video inngest function
#568
a2a97caThanks @vojtaholik! - ensure self-paced
workshop purchasers have org id by redirecting to /verify-login before
checkout
8267b98Thanks @vojtaholik! - chore: make sure
availableCoupons is array
Updated dependencies
[
5d94d4e,a4dce03]:[email protected]
Patch Changes
#579
5d94d4eThanks @vojtaholik! - upgrade next to beta,
update related deps (react, react-dom, eslint, etc)
#589
f8dffc9Thanks @vojtaholik! - bump next from beta to
stable
@coursebuilder/[email protected]
Patch Changes
#587
fff302bThanks @vojtaholik! - extract cohort and
workshop creation to adapter and add tests
783e546Thanks @vojtaholik! - fix getProgressForUser
to consider sections
1b92a3aThanks @vojtaholik! - add openEnrollment and
closeEnrollment fields to cohort creation
#588
43405a9Thanks @vojtaholik! - bump mux
#590
cf94c52Thanks @vojtaholik! - bump instantsearch
#589
f8dffc9Thanks @vojtaholik! - bump next from beta to
stable
#596
896c68cThanks @vojtaholik! - add QuestionResponses
table
Updated dependencies
[
a4dce03]:@coursebuilder/[email protected]
Patch Changes
fff302b,5d94d4e,1b92a3a,f11c60f,4134b05,79764de,22fe75f,a2a97ca,8267b98]:@coursebuilder/[email protected]
Patch Changes
f8dffc9Thanks @vojtaholik! - bump next from beta to
stable
@coursebuilder/[email protected]
Patch Changes
5d94d4eThanks @vojtaholik! - upgrade next to beta,
update related deps (react, react-dom, eslint, etc)
@coursebuilder/[email protected]
Patch Changes
5d94d4eThanks @vojtaholik! - upgrade next to beta,
update related deps (react, react-dom, eslint, etc)
@coursebuilder/[email protected]
Patch Changes
#579
5d94d4eThanks @vojtaholik! - upgrade next to beta,
update related deps (react, react-dom, eslint, etc)
#589
f8dffc9Thanks @vojtaholik! - bump next from beta to
stable
Updated dependencies
[
fff302b,5d94d4e,1b92a3a,f11c60f,4134b05,79764de,22fe75f,a2a97ca,8267b98]:@coursebuilder/[email protected]
Patch Changes
a4dce03Thanks @vojtaholik! - add uniqBy
@coursebuilder/[email protected]
Patch Changes
#592
b9ebdd3Thanks @vojtaholik! - init survey package
#593
9368a4cThanks @vojtaholik! - surveys
Updated dependencies
[
5d94d4e,43405a9,14a6804,9a1d398,e58242e,9368a4c,896c68c,78595ef,b2fd748,942042c,06a993f]:@coursebuilder/[email protected]
Patch Changes
#579
5d94d4eThanks @vojtaholik! - upgrade next to beta,
update related deps (react, react-dom, eslint, etc)
#588
43405a9Thanks @vojtaholik! - bump mux
#549
14a6804Thanks @vojtaholik! - upgrade accordion to
latest
9a1d398Thanks @vojtaholik! - style tweaks
#595
e58242eThanks @vojtaholik! - bump mux player
#593
9368a4cThanks @vojtaholik! - surveys
#596
896c68cThanks @vojtaholik! - add QuestionResponses
table
78595efThanks @vojtaholik! - bump mux player
#573
b2fd748Thanks @vojtaholik! - add navigation menu
component
942042cThanks @vojtaholik! - add workshop creation
dialog and form components
06a993fThanks @vojtaholik! - add sidebar
Updated dependencies
[
fff302b,5d94d4e,1b92a3a,f11c60f,4134b05,79764de,22fe75f,a2a97ca,8267b98]:@coursebuilder/[email protected]
Patch Changes
5d94d4eThanks @vojtaholik! - upgrade next to beta,
update related deps (react, react-dom, eslint, etc)
@coursebuilder/[email protected]
Patch Changes
[
fff302b,5d94d4e,1b92a3a,f11c60f,4134b05,79764de,22fe75f,a2a97ca,8267b98]:@coursebuilder/[email protected]
Patch Changes
#590
cf94c52Thanks @vojtaholik! - bump instantsearch
#568
a2a97caThanks @vojtaholik! - ensure self-paced
workshop purchasers have org id by redirecting to /verify-login before
checkout
Summary by CodeRabbit
New Features
Chores