Skip to content
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

fix(cli): remove comments from moviedb template, hide location field #6488

Merged
merged 1 commit into from
Apr 26, 2024

Conversation

rexxars
Copy link
Member

@rexxars rexxars commented Apr 25, 2024

Description

There's an issue I am struggling to find the cause of, where npm create sanity@latest will crash when trying to create the configuration file for the movie db template.

Reading the stack trace, it appears as part of the comments handling in recast, but I am struggling to easily reproduce it in testing. I modified the template to not include any comments, by simply removing the google maps input references. It was a bit of an oversight that it had stayed commented out since v3 was launched, and it looks a bit unprofessional in my opinion.

I also chose to hide the location field in the schema, since showing plain lat/lng fields seems... not very user friendly, but the data exists in the tarball that we provide, so I'd rather not show warnings on data for missing fields.

We can reintroduce the google maps plugin (or similar) later, but we should make sure it works with little to no configuration. Currently it's a bit of a setup in terms of getting the necessary API keys etc.

What to review

The changes appears safe and sane

Testing

This one is hard to test - I wasn't actually able to easily reproduce outside of modifying the built version in my local node_modules, unfortunately. It may be that it was built with a dependency that had a bug that I am not seeing locally for some reason.

Regardless, the changes here are somewhat useful even without the bugfix, so we can do a release and see if it goes away after these changes.

Notes for release

  • Fixes an issue that caused npm create sanity to crash when using the movie database template

@rexxars rexxars requested a review from a team as a code owner April 25, 2024 23:09
@rexxars rexxars requested review from cngonzalez and removed request for a team April 25, 2024 23:09
Copy link

vercel bot commented Apr 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
performance-studio ✅ Ready (Inspect) Visit Preview Apr 25, 2024 11:09pm
test-next-studio ❌ Failed (Inspect) Apr 25, 2024 11:09pm
test-studio ✅ Ready (Inspect) Visit Preview Apr 25, 2024 11:09pm
1 Ignored Deployment
Name Status Preview Updated (UTC)
studio-workshop ⬜️ Ignored (Inspect) Apr 25, 2024 11:09pm

Copy link
Contributor

No changes to documentation

Copy link
Contributor

Component Testing Report Updated Apr 25, 2024 11:17 PM (UTC)

File Status Duration Passed Skipped Failed
comments/CommentInput.spec.tsx ✅ Passed (Inspect) 40s 15 0 0
formBuilder/ArrayInput.spec.tsx ✅ Passed (Inspect) 6s 3 0 0
formBuilder/inputs/PortableText/Annotations.spec.tsx ✅ Passed (Inspect) 25s 6 0 0
formBuilder/inputs/PortableText/copyPaste/CopyPaste.spec.tsx ✅ Passed (Inspect) 13s 4 2 0
formBuilder/inputs/PortableText/Decorators.spec.tsx ✅ Passed (Inspect) 13s 6 0 0
formBuilder/inputs/PortableText/FocusTracking.spec.tsx ✅ Passed (Inspect) 35s 15 0 0
formBuilder/inputs/PortableText/Input.spec.tsx ✅ Passed (Inspect) 1m 3s 15 0 0
formBuilder/inputs/PortableText/ObjectBlock.spec.tsx ✅ Passed (Inspect) 1m 4s 18 0 0
formBuilder/inputs/PortableText/RangeDecoration.spec.tsx ✅ Passed (Inspect) 20s 9 0 0
formBuilder/inputs/PortableText/Styles.spec.tsx ✅ Passed (Inspect) 14s 6 0 0
formBuilder/inputs/PortableText/Toolbar.spec.tsx ❌ Failed (Inspect) 26s 8 0 1

Copy link
Contributor

@binoy14 binoy14 left a comment

Choose a reason for hiding this comment

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

makes sense to me, thanks!

@rexxars rexxars added this pull request to the merge queue Apr 26, 2024
Merged via the queue into next with commit 80f8daf Apr 26, 2024
36 of 39 checks passed
@rexxars rexxars deleted the fix/create-sanity-issue branch April 26, 2024 21:48
github-merge-queue bot added a commit that referenced this pull request Apr 29, 2024
* fix: statusButton requires provided aria-label

* fix: commenting on the new translation mapping added

* chore(deps): dedupe pnpm-lock.yaml (#6463)

Co-authored-by: cngonzalez <[email protected]>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.4 (#6460)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(cli): mock matchMedia window function (#6472)

* chore: add perf runner to workspace (#6427)

* fix(typegen): fixes a bug where we imported the wrong relative path (#6457)

* chore(deps): update dependency @sanity/tsdoc to v1.0.39 (#6464)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* feat(typegen): also search for queries in app and sanity folders (#6475)

* feat(typegen): also search for queries in app and sanity folders

* feat(typegen): format generated types with prettier

* chore: upgrade groq-js (#6474)

* chore: upgrade groq-js

* chore: always bump groq-js

* chore(deps): dedupe pnpm-lock.yaml (#6476)

Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/visual-editing to v1.8.15 (#6465)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/react-loader to v1.9.14 (#6466)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency date-fns-tz to v2.0.1 (#6467)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency i18next to v23.11.2 (#6468)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency zod to v3.23.4 (#6470)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency next to v14.2.2 (#6469)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency react-focus-lock to v2.12.1 (#6471)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.6 (#6482)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update react monorepo (#6484)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/tsdoc to v1.0.40 (#6483)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update dependency @sanity/presentation to v1.12.8 (#6485)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update react monorepo to ^18.3.0 (#6490)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @types/react-is to v18.3.0 (#6491)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: setup next test that embeds the test studio (#6487)

* chore: setup next test that embeds the test studio

* chore: support HMR same way test-studio does

* refactor: replace deprecated `empty()` with `EMPTY`

* refactor: shorter `pnpm dev:next-studio` alias

* chore: add `dev:turbo-studio` to test with turbopack

* chore: remove unused

* fix(test): remove hydrateroot warning test (#6494)

* chore(deps): dedupe pnpm-lock.yaml (#6493)

Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>

* feat: set print width to 40 (#6068)

* chore(deps): update dependency @sanity/pkg-utils to v6.8.7 (#6496)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency turbo to ^1.13.3 (#6497)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/logos to v2.1.11 (#6498)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/react-loader to v1.9.15 (#6499)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(cli): remove comments from moviedb template, hide location field (#6488)

* fix(search): revert to old search API for now (#6503)

* v3.39.1

* chore(deps): dedupe pnpm-lock.yaml (#6463)

Co-authored-by: cngonzalez <[email protected]>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.4 (#6460)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: add perf runner to workspace (#6427)

* chore(deps): update dependency @sanity/tsdoc to v1.0.39 (#6464)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: upgrade groq-js (#6474)

* chore: upgrade groq-js

* chore: always bump groq-js

* chore(deps): dedupe pnpm-lock.yaml (#6476)

Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/react-loader to v1.9.14 (#6466)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency i18next to v23.11.2 (#6468)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency zod to v3.23.4 (#6470)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency next to v14.2.2 (#6469)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency react-focus-lock to v2.12.1 (#6471)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.6 (#6482)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update react monorepo (#6484)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/tsdoc to v1.0.40 (#6483)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update dependency @sanity/presentation to v1.12.8 (#6485)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update react monorepo to ^18.3.0 (#6490)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: setup next test that embeds the test studio (#6487)

* chore: setup next test that embeds the test studio

* chore: support HMR same way test-studio does

* refactor: replace deprecated `empty()` with `EMPTY`

* refactor: shorter `pnpm dev:next-studio` alias

* chore: add `dev:turbo-studio` to test with turbopack

* chore: remove unused

* chore(deps): dedupe pnpm-lock.yaml (#6493)

Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.7 (#6496)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/logos to v2.1.11 (#6498)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/react-loader to v1.9.15 (#6499)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/visual-editing to v1.8.16 (#6500)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update dependency @sanity/presentation to v1.12.10 (#6501)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* feat(core): implement presence cursors (#6081)

* feat(portable-text-editor): improve range decorations perf by comparing prev values

* test(portable-text-editor): add tests for range decorations rendering

* fixup! feat(portable-text-editor): improve range decorations perf by comparing prev values

* test(portable-text-editor): improve some minor test things

* fix(core): update z-index on PTE activate overlay

* feat(core): extend presence data model with `selection`

* feat(structure): include `selection` in presence data

* feat(core): implement presence cursors in PTE

* test(core): add presence cursor workshop story

* refactor(core): use `getTheme_v2` to access theme values

* fix(core): introduce `OnPathFocusPayload` to improve typing

* fix(form/inputs): perf optimization for PT-input decorators

* refactor(form/inputs): also display user presence when user is selecting a range, but only the focus point of it

* fix(core): render presence cursors inline instead of in a portal

* fix(core): broken workshop story

* feat(core): reset presence selection on blur in PTE

* fix(structure): add accidentially removed setFocusPath (DocumentPaneProvider)

This seems to have been lost in a rebase.

* fix(core): use `useFormFieldPresence` in `usePresenceCursorDecorations`

* fix(core): set `focusPath` when receiving a mutation event if there are pending patches

* fix(form/inputs): break don't return

* fix(form/inputs): remove ref that should not be set here

This ref is supposed to be set elsewhere (on the PTE Editable component)
See eca960e

* fix(form/inputs): sort hook deps list

* fix(form/inputs): reconcile presence decorations for PT-Input

Make sure these objects stays as stable as possible in order to not
redraw any range decorations unnecessary.

* fix(form/inputs): return early if focusPath is already selected in PT-Input

* refactor(form/inputs): throttle reporting of focusPath and presence updates for PT-input

* fix(core): import of `FormNodePresence`

* fix(form/inputs): remove lastActive as dep for PT-presence range decorations uniqueness

We are not using this value anyway, and it complicates the reconciliation of the presence decorators

* test(core): add presence cursor workshop story

* fix(core): remove unnecessary z-index in PTE activate overlay

* fix(core): prevent presence cursor user name from being selected

* fix(form/inputs): fix issue where perf opt on focusPath tracking broke tests

This perf opt. broke some tests. It's not clear if it is a problem with the test
or the func. so restoring the old behaviour for now.

* test(core): update snapshots in `Studio.test`

* refactor(core): move presence decorations according to user edits

Use a own state for this.

* refactor(core/form): remove complexity from PortableTextInput

Handle debouncing in the DocumentProvider instead. Setting focusPath must be synchronous

* refactor(structure): announce presence throttled

When we introduced presence in the PortableTextInput, we risk calling the presence updates very often.
There should be no reason for not doing this throttled, as long as we have leading true.

* refactor(core): clean up presence cursors code + add comments

---------

Co-authored-by: Per-Kristian Nordnes <[email protected]>

* fix(deps): Update react monorepo (#6502)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lint): add certain unicorn eslint rules (#6352)

* chore(lint): add certain unicorn eslint rules

* chore: fix lint errors

* test: increase timeout for cross-dataset reference test

* chore: fix incorrect path used in workshop story

* chore(deps): dedupe pnpm-lock.yaml (#6506)

Co-authored-by: rexxars <[email protected]>

* fix(structure): uncaught error while swapping images in array in PTE (#6399)

* chore(test-studio): add schema for fix

* fix(structure): console error for when selection is zero

* chore(test-studio): update test schema

* chore(core): remove try catch

* test(pte): add test for usecase

* chore(sanity): remove ? since it will always be defined

Co-authored-by: Pedro Bonamin <[email protected]>

* chore(test): fix test file to have .spec.

---------

Co-authored-by: Pedro Bonamin <[email protected]>

* chore(deps): update dependency @types/react to v18.3.1 (#6507)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* v3.39.1

* chore(deps): dedupe pnpm-lock.yaml (#6463)

Co-authored-by: cngonzalez <[email protected]>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.4 (#6460)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: add perf runner to workspace (#6427)

* chore(deps): update dependency @sanity/tsdoc to v1.0.39 (#6464)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: upgrade groq-js (#6474)

* chore: upgrade groq-js

* chore: always bump groq-js

* chore(deps): dedupe pnpm-lock.yaml (#6476)

Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/visual-editing to v1.8.15 (#6465)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/react-loader to v1.9.14 (#6466)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency i18next to v23.11.2 (#6468)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency zod to v3.23.4 (#6470)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency next to v14.2.2 (#6469)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency react-focus-lock to v2.12.1 (#6471)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.6 (#6482)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update react monorepo (#6484)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/tsdoc to v1.0.40 (#6483)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update dependency @sanity/presentation to v1.12.8 (#6485)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update react monorepo to ^18.3.0 (#6490)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: setup next test that embeds the test studio (#6487)

* chore: setup next test that embeds the test studio

* chore: support HMR same way test-studio does

* refactor: replace deprecated `empty()` with `EMPTY`

* refactor: shorter `pnpm dev:next-studio` alias

* chore: add `dev:turbo-studio` to test with turbopack

* chore: remove unused

* chore(deps): dedupe pnpm-lock.yaml (#6493)

Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.7 (#6496)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/logos to v2.1.11 (#6498)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/react-loader to v1.9.15 (#6499)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/visual-editing to v1.8.16 (#6500)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update dependency @sanity/presentation to v1.12.10 (#6501)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* feat(core): implement presence cursors (#6081)

* feat(portable-text-editor): improve range decorations perf by comparing prev values

* test(portable-text-editor): add tests for range decorations rendering

* fixup! feat(portable-text-editor): improve range decorations perf by comparing prev values

* test(portable-text-editor): improve some minor test things

* fix(core): update z-index on PTE activate overlay

* feat(core): extend presence data model with `selection`

* feat(structure): include `selection` in presence data

* feat(core): implement presence cursors in PTE

* test(core): add presence cursor workshop story

* refactor(core): use `getTheme_v2` to access theme values

* fix(core): introduce `OnPathFocusPayload` to improve typing

* fix(form/inputs): perf optimization for PT-input decorators

* refactor(form/inputs): also display user presence when user is selecting a range, but only the focus point of it

* fix(core): render presence cursors inline instead of in a portal

* fix(core): broken workshop story

* feat(core): reset presence selection on blur in PTE

* fix(structure): add accidentially removed setFocusPath (DocumentPaneProvider)

This seems to have been lost in a rebase.

* fix(core): use `useFormFieldPresence` in `usePresenceCursorDecorations`

* fix(core): set `focusPath` when receiving a mutation event if there are pending patches

* fix(form/inputs): break don't return

* fix(form/inputs): remove ref that should not be set here

This ref is supposed to be set elsewhere (on the PTE Editable component)
See eca960e

* fix(form/inputs): sort hook deps list

* fix(form/inputs): reconcile presence decorations for PT-Input

Make sure these objects stays as stable as possible in order to not
redraw any range decorations unnecessary.

* fix(form/inputs): return early if focusPath is already selected in PT-Input

* refactor(form/inputs): throttle reporting of focusPath and presence updates for PT-input

* fix(core): import of `FormNodePresence`

* fix(form/inputs): remove lastActive as dep for PT-presence range decorations uniqueness

We are not using this value anyway, and it complicates the reconciliation of the presence decorators

* test(core): add presence cursor workshop story

* fix(core): remove unnecessary z-index in PTE activate overlay

* fix(core): prevent presence cursor user name from being selected

* fix(form/inputs): fix issue where perf opt on focusPath tracking broke tests

This perf opt. broke some tests. It's not clear if it is a problem with the test
or the func. so restoring the old behaviour for now.

* test(core): update snapshots in `Studio.test`

* refactor(core): move presence decorations according to user edits

Use a own state for this.

* refactor(core/form): remove complexity from PortableTextInput

Handle debouncing in the DocumentProvider instead. Setting focusPath must be synchronous

* refactor(structure): announce presence throttled

When we introduced presence in the PortableTextInput, we risk calling the presence updates very often.
There should be no reason for not doing this throttled, as long as we have leading true.

* refactor(core): clean up presence cursors code + add comments

---------

Co-authored-by: Per-Kristian Nordnes <[email protected]>

* fix(deps): Update react monorepo (#6502)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: fix incorrect path used in workshop story

* chore(deps): dedupe pnpm-lock.yaml (#6506)

Co-authored-by: rexxars <[email protected]>

* chore(deps): update dependency @types/react to v18.3.1 (#6507)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

---------

Co-authored-by: ecospark[bot] <128108030+ecospark[bot]@users.noreply.github.com>
Co-authored-by: cngonzalez <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Sindre Gulseth <[email protected]>
Co-authored-by: Bjørge Næss <[email protected]>
Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>
Co-authored-by: Cody Olsen <[email protected]>
Co-authored-by: Rostislav Melkumyan <[email protected]>
Co-authored-by: Espen Hovlandsdal <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Herman Wikner <[email protected]>
Co-authored-by: Per-Kristian Nordnes <[email protected]>
Co-authored-by: rexxars <[email protected]>
Co-authored-by: RitaDias <[email protected]>
Co-authored-by: Pedro Bonamin <[email protected]>
cngonzalez added a commit that referenced this pull request May 8, 2024
* fix: statusButton requires provided aria-label

* fix: commenting on the new translation mapping added

* chore(deps): dedupe pnpm-lock.yaml (#6463)

Co-authored-by: cngonzalez <[email protected]>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.4 (#6460)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(cli): mock matchMedia window function (#6472)

* chore: add perf runner to workspace (#6427)

* fix(typegen): fixes a bug where we imported the wrong relative path (#6457)

* chore(deps): update dependency @sanity/tsdoc to v1.0.39 (#6464)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* feat(typegen): also search for queries in app and sanity folders (#6475)

* feat(typegen): also search for queries in app and sanity folders

* feat(typegen): format generated types with prettier

* chore: upgrade groq-js (#6474)

* chore: upgrade groq-js

* chore: always bump groq-js

* chore(deps): dedupe pnpm-lock.yaml (#6476)

Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/visual-editing to v1.8.15 (#6465)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/react-loader to v1.9.14 (#6466)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency date-fns-tz to v2.0.1 (#6467)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency i18next to v23.11.2 (#6468)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency zod to v3.23.4 (#6470)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency next to v14.2.2 (#6469)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency react-focus-lock to v2.12.1 (#6471)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.6 (#6482)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update react monorepo (#6484)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/tsdoc to v1.0.40 (#6483)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update dependency @sanity/presentation to v1.12.8 (#6485)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update react monorepo to ^18.3.0 (#6490)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @types/react-is to v18.3.0 (#6491)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: setup next test that embeds the test studio (#6487)

* chore: setup next test that embeds the test studio

* chore: support HMR same way test-studio does

* refactor: replace deprecated `empty()` with `EMPTY`

* refactor: shorter `pnpm dev:next-studio` alias

* chore: add `dev:turbo-studio` to test with turbopack

* chore: remove unused

* fix(test): remove hydrateroot warning test (#6494)

* chore(deps): dedupe pnpm-lock.yaml (#6493)

Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>

* feat: set print width to 40 (#6068)

* chore(deps): update dependency @sanity/pkg-utils to v6.8.7 (#6496)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency turbo to ^1.13.3 (#6497)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/logos to v2.1.11 (#6498)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/react-loader to v1.9.15 (#6499)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(cli): remove comments from moviedb template, hide location field (#6488)

* fix(search): revert to old search API for now (#6503)

* v3.39.1

* chore(deps): dedupe pnpm-lock.yaml (#6463)

Co-authored-by: cngonzalez <[email protected]>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.4 (#6460)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: add perf runner to workspace (#6427)

* chore(deps): update dependency @sanity/tsdoc to v1.0.39 (#6464)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: upgrade groq-js (#6474)

* chore: upgrade groq-js

* chore: always bump groq-js

* chore(deps): dedupe pnpm-lock.yaml (#6476)

Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/react-loader to v1.9.14 (#6466)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency i18next to v23.11.2 (#6468)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency zod to v3.23.4 (#6470)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency next to v14.2.2 (#6469)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency react-focus-lock to v2.12.1 (#6471)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.6 (#6482)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update react monorepo (#6484)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/tsdoc to v1.0.40 (#6483)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update dependency @sanity/presentation to v1.12.8 (#6485)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update react monorepo to ^18.3.0 (#6490)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: setup next test that embeds the test studio (#6487)

* chore: setup next test that embeds the test studio

* chore: support HMR same way test-studio does

* refactor: replace deprecated `empty()` with `EMPTY`

* refactor: shorter `pnpm dev:next-studio` alias

* chore: add `dev:turbo-studio` to test with turbopack

* chore: remove unused

* chore(deps): dedupe pnpm-lock.yaml (#6493)

Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.7 (#6496)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/logos to v2.1.11 (#6498)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/react-loader to v1.9.15 (#6499)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/visual-editing to v1.8.16 (#6500)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update dependency @sanity/presentation to v1.12.10 (#6501)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* feat(core): implement presence cursors (#6081)

* feat(portable-text-editor): improve range decorations perf by comparing prev values

* test(portable-text-editor): add tests for range decorations rendering

* fixup! feat(portable-text-editor): improve range decorations perf by comparing prev values

* test(portable-text-editor): improve some minor test things

* fix(core): update z-index on PTE activate overlay

* feat(core): extend presence data model with `selection`

* feat(structure): include `selection` in presence data

* feat(core): implement presence cursors in PTE

* test(core): add presence cursor workshop story

* refactor(core): use `getTheme_v2` to access theme values

* fix(core): introduce `OnPathFocusPayload` to improve typing

* fix(form/inputs): perf optimization for PT-input decorators

* refactor(form/inputs): also display user presence when user is selecting a range, but only the focus point of it

* fix(core): render presence cursors inline instead of in a portal

* fix(core): broken workshop story

* feat(core): reset presence selection on blur in PTE

* fix(structure): add accidentially removed setFocusPath (DocumentPaneProvider)

This seems to have been lost in a rebase.

* fix(core): use `useFormFieldPresence` in `usePresenceCursorDecorations`

* fix(core): set `focusPath` when receiving a mutation event if there are pending patches

* fix(form/inputs): break don't return

* fix(form/inputs): remove ref that should not be set here

This ref is supposed to be set elsewhere (on the PTE Editable component)
See eca960e

* fix(form/inputs): sort hook deps list

* fix(form/inputs): reconcile presence decorations for PT-Input

Make sure these objects stays as stable as possible in order to not
redraw any range decorations unnecessary.

* fix(form/inputs): return early if focusPath is already selected in PT-Input

* refactor(form/inputs): throttle reporting of focusPath and presence updates for PT-input

* fix(core): import of `FormNodePresence`

* fix(form/inputs): remove lastActive as dep for PT-presence range decorations uniqueness

We are not using this value anyway, and it complicates the reconciliation of the presence decorators

* test(core): add presence cursor workshop story

* fix(core): remove unnecessary z-index in PTE activate overlay

* fix(core): prevent presence cursor user name from being selected

* fix(form/inputs): fix issue where perf opt on focusPath tracking broke tests

This perf opt. broke some tests. It's not clear if it is a problem with the test
or the func. so restoring the old behaviour for now.

* test(core): update snapshots in `Studio.test`

* refactor(core): move presence decorations according to user edits

Use a own state for this.

* refactor(core/form): remove complexity from PortableTextInput

Handle debouncing in the DocumentProvider instead. Setting focusPath must be synchronous

* refactor(structure): announce presence throttled

When we introduced presence in the PortableTextInput, we risk calling the presence updates very often.
There should be no reason for not doing this throttled, as long as we have leading true.

* refactor(core): clean up presence cursors code + add comments

---------

Co-authored-by: Per-Kristian Nordnes <[email protected]>

* fix(deps): Update react monorepo (#6502)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lint): add certain unicorn eslint rules (#6352)

* chore(lint): add certain unicorn eslint rules

* chore: fix lint errors

* test: increase timeout for cross-dataset reference test

* chore: fix incorrect path used in workshop story

* chore(deps): dedupe pnpm-lock.yaml (#6506)

Co-authored-by: rexxars <[email protected]>

* fix(structure): uncaught error while swapping images in array in PTE (#6399)

* chore(test-studio): add schema for fix

* fix(structure): console error for when selection is zero

* chore(test-studio): update test schema

* chore(core): remove try catch

* test(pte): add test for usecase

* chore(sanity): remove ? since it will always be defined

Co-authored-by: Pedro Bonamin <[email protected]>

* chore(test): fix test file to have .spec.

---------

Co-authored-by: Pedro Bonamin <[email protected]>

* chore(deps): update dependency @types/react to v18.3.1 (#6507)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* v3.39.1

* chore(deps): dedupe pnpm-lock.yaml (#6463)

Co-authored-by: cngonzalez <[email protected]>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.4 (#6460)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: add perf runner to workspace (#6427)

* chore(deps): update dependency @sanity/tsdoc to v1.0.39 (#6464)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: upgrade groq-js (#6474)

* chore: upgrade groq-js

* chore: always bump groq-js

* chore(deps): dedupe pnpm-lock.yaml (#6476)

Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/visual-editing to v1.8.15 (#6465)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/react-loader to v1.9.14 (#6466)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency i18next to v23.11.2 (#6468)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency zod to v3.23.4 (#6470)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency next to v14.2.2 (#6469)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency react-focus-lock to v2.12.1 (#6471)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.6 (#6482)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update react monorepo (#6484)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/tsdoc to v1.0.40 (#6483)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update dependency @sanity/presentation to v1.12.8 (#6485)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update react monorepo to ^18.3.0 (#6490)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: setup next test that embeds the test studio (#6487)

* chore: setup next test that embeds the test studio

* chore: support HMR same way test-studio does

* refactor: replace deprecated `empty()` with `EMPTY`

* refactor: shorter `pnpm dev:next-studio` alias

* chore: add `dev:turbo-studio` to test with turbopack

* chore: remove unused

* chore(deps): dedupe pnpm-lock.yaml (#6493)

Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/pkg-utils to v6.8.7 (#6496)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/logos to v2.1.11 (#6498)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(lockfile): update dependency @sanity/react-loader to v1.9.15 (#6499)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore(deps): update dependency @sanity/visual-editing to v1.8.16 (#6500)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix(deps): update dependency @sanity/presentation to v1.12.10 (#6501)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* feat(core): implement presence cursors (#6081)

* feat(portable-text-editor): improve range decorations perf by comparing prev values

* test(portable-text-editor): add tests for range decorations rendering

* fixup! feat(portable-text-editor): improve range decorations perf by comparing prev values

* test(portable-text-editor): improve some minor test things

* fix(core): update z-index on PTE activate overlay

* feat(core): extend presence data model with `selection`

* feat(structure): include `selection` in presence data

* feat(core): implement presence cursors in PTE

* test(core): add presence cursor workshop story

* refactor(core): use `getTheme_v2` to access theme values

* fix(core): introduce `OnPathFocusPayload` to improve typing

* fix(form/inputs): perf optimization for PT-input decorators

* refactor(form/inputs): also display user presence when user is selecting a range, but only the focus point of it

* fix(core): render presence cursors inline instead of in a portal

* fix(core): broken workshop story

* feat(core): reset presence selection on blur in PTE

* fix(structure): add accidentially removed setFocusPath (DocumentPaneProvider)

This seems to have been lost in a rebase.

* fix(core): use `useFormFieldPresence` in `usePresenceCursorDecorations`

* fix(core): set `focusPath` when receiving a mutation event if there are pending patches

* fix(form/inputs): break don't return

* fix(form/inputs): remove ref that should not be set here

This ref is supposed to be set elsewhere (on the PTE Editable component)
See eca960e

* fix(form/inputs): sort hook deps list

* fix(form/inputs): reconcile presence decorations for PT-Input

Make sure these objects stays as stable as possible in order to not
redraw any range decorations unnecessary.

* fix(form/inputs): return early if focusPath is already selected in PT-Input

* refactor(form/inputs): throttle reporting of focusPath and presence updates for PT-input

* fix(core): import of `FormNodePresence`

* fix(form/inputs): remove lastActive as dep for PT-presence range decorations uniqueness

We are not using this value anyway, and it complicates the reconciliation of the presence decorators

* test(core): add presence cursor workshop story

* fix(core): remove unnecessary z-index in PTE activate overlay

* fix(core): prevent presence cursor user name from being selected

* fix(form/inputs): fix issue where perf opt on focusPath tracking broke tests

This perf opt. broke some tests. It's not clear if it is a problem with the test
or the func. so restoring the old behaviour for now.

* test(core): update snapshots in `Studio.test`

* refactor(core): move presence decorations according to user edits

Use a own state for this.

* refactor(core/form): remove complexity from PortableTextInput

Handle debouncing in the DocumentProvider instead. Setting focusPath must be synchronous

* refactor(structure): announce presence throttled

When we introduced presence in the PortableTextInput, we risk calling the presence updates very often.
There should be no reason for not doing this throttled, as long as we have leading true.

* refactor(core): clean up presence cursors code + add comments

---------

Co-authored-by: Per-Kristian Nordnes <[email protected]>

* fix(deps): Update react monorepo (#6502)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* chore: fix incorrect path used in workshop story

* chore(deps): dedupe pnpm-lock.yaml (#6506)

Co-authored-by: rexxars <[email protected]>

* chore(deps): update dependency @types/react to v18.3.1 (#6507)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

---------

Co-authored-by: ecospark[bot] <128108030+ecospark[bot]@users.noreply.github.com>
Co-authored-by: cngonzalez <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Sindre Gulseth <[email protected]>
Co-authored-by: Bjørge Næss <[email protected]>
Co-authored-by: github-merge-queue[bot] <118344674+github-merge-queue[bot]@users.noreply.github.com>
Co-authored-by: Cody Olsen <[email protected]>
Co-authored-by: Rostislav Melkumyan <[email protected]>
Co-authored-by: Espen Hovlandsdal <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Herman Wikner <[email protected]>
Co-authored-by: Per-Kristian Nordnes <[email protected]>
Co-authored-by: rexxars <[email protected]>
Co-authored-by: RitaDias <[email protected]>
Co-authored-by: Pedro Bonamin <[email protected]>
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