Skip to content

refactor(core): zero circulars + esm ready #10770

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 74 commits into
base: main
Choose a base branch
from

Conversation

NathanWalker
Copy link
Contributor

@NathanWalker NathanWalker commented Jul 25, 2025

  • resolves all 190+ circulars
  • provides esm ready distribution
  • provides bundling support for esm
  • provides esm hooks

TODO:

  • Isolate webpack changes into separate pull request
  • Remove extra test logging
  • Check if @nativescript/hook needs to conditionally create .js or .mjs based on bundling settings if possible (for backwards compat of <9 cli with >=9 core)

Notes:
@nativescript/webpack can be released as a patch as it's backwards compat now. It's been on a 5.1.x esm tag while testing however it's proven fully backwards compatible so a patch would be fine and likely recommended so users don't need to think about cjs v esm.

Copy link

nx-cloud bot commented Jul 25, 2025

🤖 Nx Cloud AI Fix Eligible

An automatically generated fix could have helped fix failing tasks for this run, but Self-healing CI is disabled for this workspace. Visit workspace settings to enable it and get automatic fixes in future runs.

To disable these notifications, a workspace admin can disable them in workspace settings.


View your CI Pipeline Execution ↗ for commit 85ae1b0

Command Status Duration Result
nx test apps-automated -c=ios ❌ Failed 2m 31s View ↗
nx run-many --target=test --configuration=ci --... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-08-07 23:59:21 UTC

@NathanWalker NathanWalker changed the title refactor: zero circulars and esm ready refactor(core): zero circulars + esm ready Jul 25, 2025
@NathanWalker NathanWalker added this to the 9.0 milestone Jul 30, 2025
dependabot bot and others added 15 commits July 29, 2025 17:44
Bumps [postcss](https://github.com/postcss/postcss) to 8.5.6 and updates ancestor dependency [vue-loader](https://github.com/vuejs/vue-loader). These dependencies need to be updated together.

Updates `postcss` from 7.0.39 to 8.5.6
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss@7.0.39...8.5.6)

Updates `vue-loader` from 15.9.8 to 17.4.2
- [Release notes](https://github.com/vuejs/vue-loader/releases)
- [Changelog](https://github.com/vuejs/vue-loader/blob/main/CHANGELOG.md)
- [Commits](vuejs/vue-loader@v15.9.8...v17.4.2)

---
updated-dependencies:
- dependency-name: postcss
  dependency-version: 8.5.6
  dependency-type: indirect
- dependency-name: vue-loader
  dependency-version: 17.4.2
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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