Skip to content

refactor(angular-query): migrate from tsup to vite #9281

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

Merged
merged 3 commits into from
Jun 17, 2025

Conversation

arnoud-dv
Copy link
Collaborator

@arnoud-dv arnoud-dv commented Jun 17, 2025

Allows for more flexibility in build process.

Adds prepack and postpack scripts to publish type declaration files in package root, a prerequisite to support subpath exports while also supporting applications using "moduleResolution": "node".

Fixes moduleResolution: node16

Copy link

nx-cloud bot commented Jun 17, 2025

View your CI Pipeline Execution ↗ for commit e6a1444.

Command Status Duration Result
nx affected --targets=test:sherif,test:knip,tes... ✅ Succeeded 1m 23s View ↗
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 10s View ↗

☁️ Nx Cloud last updated this comment at 2025-06-17 17:58:49 UTC

Copy link

pkg-pr-new bot commented Jun 17, 2025

More templates

@tanstack/angular-query-devtools-experimental

npm i https://pkg.pr.new/@tanstack/angular-query-devtools-experimental@9281

@tanstack/angular-query-experimental

npm i https://pkg.pr.new/@tanstack/angular-query-experimental@9281

@tanstack/eslint-plugin-query

npm i https://pkg.pr.new/@tanstack/eslint-plugin-query@9281

@tanstack/query-async-storage-persister

npm i https://pkg.pr.new/@tanstack/query-async-storage-persister@9281

@tanstack/query-broadcast-client-experimental

npm i https://pkg.pr.new/@tanstack/query-broadcast-client-experimental@9281

@tanstack/query-core

npm i https://pkg.pr.new/@tanstack/query-core@9281

@tanstack/query-devtools

npm i https://pkg.pr.new/@tanstack/query-devtools@9281

@tanstack/query-persist-client-core

npm i https://pkg.pr.new/@tanstack/query-persist-client-core@9281

@tanstack/query-sync-storage-persister

npm i https://pkg.pr.new/@tanstack/query-sync-storage-persister@9281

@tanstack/react-query

npm i https://pkg.pr.new/@tanstack/react-query@9281

@tanstack/react-query-devtools

npm i https://pkg.pr.new/@tanstack/react-query-devtools@9281

@tanstack/react-query-next-experimental

npm i https://pkg.pr.new/@tanstack/react-query-next-experimental@9281

@tanstack/react-query-persist-client

npm i https://pkg.pr.new/@tanstack/react-query-persist-client@9281

@tanstack/solid-query

npm i https://pkg.pr.new/@tanstack/solid-query@9281

@tanstack/solid-query-devtools

npm i https://pkg.pr.new/@tanstack/solid-query-devtools@9281

@tanstack/solid-query-persist-client

npm i https://pkg.pr.new/@tanstack/solid-query-persist-client@9281

@tanstack/svelte-query

npm i https://pkg.pr.new/@tanstack/svelte-query@9281

@tanstack/svelte-query-devtools

npm i https://pkg.pr.new/@tanstack/svelte-query-devtools@9281

@tanstack/svelte-query-persist-client

npm i https://pkg.pr.new/@tanstack/svelte-query-persist-client@9281

@tanstack/vue-query

npm i https://pkg.pr.new/@tanstack/vue-query@9281

@tanstack/vue-query-devtools

npm i https://pkg.pr.new/@tanstack/vue-query-devtools@9281

commit: e6a1444

Copy link

codecov bot commented Jun 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.37%. Comparing base (bd3ac49) to head (e6a1444).
Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #9281   +/-   ##
=======================================
  Coverage   45.37%   45.37%           
=======================================
  Files         207      207           
  Lines        8276     8276           
  Branches     1864     1865    +1     
=======================================
  Hits         3755     3755           
  Misses       4080     4080           
  Partials      441      441           
Components Coverage Δ
@tanstack/angular-query-devtools-experimental ∅ <ø> (∅)
@tanstack/angular-query-experimental 85.00% <ø> (ø)
@tanstack/eslint-plugin-query 83.24% <ø> (ø)
@tanstack/query-async-storage-persister 43.85% <ø> (ø)
@tanstack/query-broadcast-client-experimental 24.39% <ø> (ø)
@tanstack/query-codemods 0.00% <ø> (ø)
@tanstack/query-core 98.15% <ø> (ø)
@tanstack/query-devtools 3.55% <ø> (ø)
@tanstack/query-persist-client-core 79.47% <ø> (ø)
@tanstack/query-sync-storage-persister 84.61% <ø> (ø)
@tanstack/query-test-utils 77.77% <ø> (ø)
@tanstack/react-query 95.93% <ø> (ø)
@tanstack/react-query-devtools 10.00% <ø> (ø)
@tanstack/react-query-next-experimental ∅ <ø> (∅)
@tanstack/react-query-persist-client 100.00% <ø> (ø)
@tanstack/solid-query 78.13% <ø> (ø)
@tanstack/solid-query-devtools ∅ <ø> (∅)
@tanstack/solid-query-persist-client 100.00% <ø> (ø)
@tanstack/svelte-query 88.07% <ø> (ø)
@tanstack/svelte-query-devtools ∅ <ø> (∅)
@tanstack/svelte-query-persist-client 100.00% <ø> (ø)
@tanstack/vue-query 71.10% <ø> (ø)
@tanstack/vue-query-devtools ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@arnoud-dv arnoud-dv changed the title chore(angular-query): migrate from tsup to vite refactor(angular-query): migrate from tsup to vite Jun 17, 2025
@arnoud-dv arnoud-dv force-pushed the chore/angular-query-build-vite branch from 862240b to e6a1444 Compare June 17, 2025 17:56
@arnoud-dv arnoud-dv merged commit 1c7c276 into TanStack:main Jun 17, 2025
7 checks passed
@arnoud-dv arnoud-dv deleted the chore/angular-query-build-vite branch June 17, 2025 18:03
@hakimio
Copy link

hakimio commented Jun 17, 2025

@arnoud-dv the production bundle seems to have incorrect types config.

image

@arnoud-dv
Copy link
Collaborator Author

@hakimio do you mean it should be

"types": "./index.d.ts"

@hakimio
Copy link

hakimio commented Jun 17, 2025

@arnoud-dv no, I mean that index.d.ts no longer exists in the root dir - it was moved to dist/types, but the production bundle package.json was not updated correctly (like shown in the screenshot above).

@arnoud-dv
Copy link
Collaborator Author

There's a prepack lifecycle script that makes sure the type declarations are linked from the package root at the moment the package is published. I want to add sub path exports and to support the still widely used "moduleResolution": "node", type declarations files should follow the same path structure.

@hakimio
Copy link

hakimio commented Jun 17, 2025

Ok, let's do it step-by-step:

  1. download the latest @tanstack/angular-query-experimental package from npm:
npm pack @tanstack/angular-query-experimental
  1. Open the archive and inspect its contents.
  2. Does it have index.d.ts in the root dir? No.
  3. Now open package.json and check what it says:
"exports": {
    ".": {
      "types": "./index.d.ts",
      "default": "./dist/index.mjs"
    }
  }

It points to non-existant index.d.ts file.

arnoud-dv added a commit that referenced this pull request Jun 17, 2025
@arnoud-dv
Copy link
Collaborator Author

arnoud-dv commented Jun 17, 2025

I see - I'll revert it for now.

Tested this very well, preview package from the PR build was working perfectly on all supported Angular versions and when looking at its files contents it was also good. But probably the actual publish script behaviour is different. I'll look at this later.

Thanks for reporting the issue!

arnoud-dv added a commit that referenced this pull request Jun 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants