Skip to content

Fix Swift macro declarations with short names - #33

Open
kahwee wants to merge 1 commit into
TanStack:mainfrom
kahwee:fix/swift-short-macro
Open

kahwee wants to merge 1 commit into
TanStack:mainfrom
kahwee:fix/swift-short-macro

Conversation

@kahwee

@kahwee kahwee commented Oct 21, 2026 •

Copy link
Copy Markdown

Recognize macro as a keyword in declarations such as public macro m() and public 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 0541f2f and has no PowerShell dependency.

Validation on Node 24.19.0:

  • The regression test fails on unchanged main and passes with the fix.
  • pnpm run verify passes: typecheck, build, docs, skills, package lint, package/packed tests, 217 tests, size budgets, and benchmarks.
  • pnpm run changeset:status reports a patch; git diff --check passes.

CI for daab0dc passed verification and the Node 18, 20, 22, 24, and 26 runtime checks.

@changeset-bot

changeset-bot Bot commented Oct 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: daab0dc

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@tanstack/highlight Patch

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

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-21T02:06:28.816792Z daab0dc PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@coderabbitai

coderabbitai Bot commented Oct 21, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You'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.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ba4d03c5-8580-470c-81f7-a65aac505189
📥 Commits

Reviewing files that changed from the base of the PR and between 0541f2f and daab0dc.

📒 Files selected for processing: 3
  • .changeset/swift-short-macro.md
  • src/languages/swift.ts
  • test/swift.test.ts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant