Skip to content

Fix beta packaging paths and ignore local worktrees#63

Merged
joalves merged 3 commits intomainfrom
fix/beta-packaging-paths
Apr 24, 2026
Merged

Fix beta packaging paths and ignore local worktrees#63
joalves merged 3 commits intomainfrom
fix/beta-packaging-paths

Conversation

@joalves
Copy link
Copy Markdown
Contributor

@joalves joalves commented Apr 23, 2026

Summary

  • stop Babel from embedding absolute runtime helper paths in published artifacts
  • add @babel/runtime to production dependencies so published lib/ and es/ helpers resolve for consumers
  • bump the package version to 1.14.0-beta.1
  • make package.json the only version source and generate src/version.ts automatically before compile and test
  • ignore .claude/ in Jest and .claude/worktrees in Git so local worktrees do not affect package tests

Testing

  • npm test -- --runInBand

Summary by CodeRabbit

  • Chores
    • Released version 1.14.0-beta.1
    • Introduced a new runtime dependency (@babel/runtime)
    • Restructured the build pipeline automation: code generation now executes automatically before compilation and testing, rather than as a separate explicit step in the build sequence
    • Removed a previously exported version constant from the public API

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1208b969-b0b5-404a-af93-69435556d2d5

📥 Commits

Reviewing files that changed from the base of the PR and between 1f4360a and 2ab1799.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json
🚧 Files skipped from review as they are similar to previous changes (1)
  • package.json

Walkthrough

This PR bumps package.json from 1.14.0-beta.0 to 1.14.0-beta.1, removes the exported SDK_VERSION constant (src/version.ts), adds @babel/runtime as a dependency, adjusts build/test scripts to run generate-version via precompile and pretest hooks, changes Babel's @babel/plugin-transform-runtime absoluteRuntime from true to false, adds <rootDir>/.claude/ to Jest ignore patterns, and updates .gitignore to exclude .claude/worktrees and src/version.ts.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • marcio-absmartly
  • mario-silva

Poem

🐇 I hopped through code with tiny feet,
Updated versions, neat and sweet,
A file removed, a helper found,
.claude tucked safe beneath the ground,
Build and tests now hum and meet ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarises the main changes: fixing Babel runtime configuration and ignoring local worktrees in Git and Jest.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/beta-packaging-paths

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

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@babel.config.js`:
- Line 31: The Babel config uses helpers that import from `@babel/runtime`
(absoluteRuntime: false and helpers: true), so add `@babel/runtime` to the
package's production dependencies in package.json (e.g., under "dependencies")
before publishing; ensure the correct semver (match your peer tooling) and run
npm install / yarn add --save to include it so consumers of the published lib
have the runtime helpers available.
🪄 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: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c87ce596-65b2-4166-b5fe-0be38d926f04

📥 Commits

Reviewing files that changed from the base of the PR and between 5cc1617 and e47df6b.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (5)
  • .gitignore
  • babel.config.js
  • jest.config.js
  • package.json
  • src/version.ts

Comment thread babel.config.js
@joalves joalves merged commit 816d361 into main Apr 24, 2026
3 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