Skip to content

Conversation

@ommeirelles
Copy link
Contributor

@ommeirelles ommeirelles commented Nov 11, 2025

  • Fixing storybook configurations.
  • replacing jest for vitest in cli and core packages
  • ci pack with pnpm to replace catalog: versions in package.json
  • fixing size script in sdk

@ommeirelles ommeirelles requested a review from a team as a code owner November 11, 2025 19:25
@ommeirelles ommeirelles requested review from lariciamota and lemagnetic and removed request for a team November 11, 2025 19:25
@codesandbox-ci
Copy link

codesandbox-ci bot commented Nov 13, 2025

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Copy link
Member

@eduardoformiga eduardoformiga left a comment

Choose a reason for hiding this comment

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

Lots of files changed, and I had a lot of questions hahah

ommeirelles and others added 2 commits December 2, 2025 11:11
- faststore/cli command to calculate graphql chaced queries


You can test it by running the cli on the starter or on the core
mannualy.

Ex:
- Running on core
```
  $ pnpm i
  $ cd ./packages/core
  $ (../cli & pnpm build)
  $ node ../cli/bin/run generate-types
  $ node ../cli/bin/run cache-graphql
```

Builds the cli, and regenerated the types to create the
persisted-documents.json file and run the cache-graphql.

After running it you should see the discovery.config.default.js changed
to include all the queries names that would be cached.

---------

Co-authored-by: Fanny Chien <[email protected]>
Copy link
Contributor

@lemagnetic lemagnetic left a comment

Choose a reason for hiding this comment

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

LGTM! Builds and tests passing

@ommeirelles ommeirelles merged commit cd5cc30 into chore/upgrade-next Dec 3, 2025
4 of 5 checks passed
@ommeirelles ommeirelles deleted the fix/storybook branch December 3, 2025 18:45
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.

4 participants