Repository navigation
Conversation
🦋 Changeset detectedLatest commit: daab0dc The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 59 minutes. View limit details
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 |
Recognize
macroas a keyword in declarations such aspublic macro m()andpublic macro m<T>(). The existing contextual-keyword rule misses these short lowercase names.Keep identifiers (
let macro = value) and member calls (value.macro()) unchanged. Add a focused regression test and patch changeset.This isolates the Swift fix previously included in #26. It was branched from main at
0541f2fand has no PowerShell dependency.Validation on Node 24.19.0:
pnpm run verifypasses: typecheck, build, docs, skills, package lint, package/packed tests, 217 tests, size budgets, and benchmarks.pnpm run changeset:statusreports a patch;git diff --checkpasses.CI for
daab0dcpassed verification and the Node 18, 20, 22, 24, and 26 runtime checks.