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

[Bug]: Unexpected end of input error for JS files that contain curly apostrophes #4535

Closed
2 tasks done
nikitaourazbaev opened this issue Sep 26, 2024 · 2 comments
Closed
2 tasks done
Assignees
Labels
Area: @shopify/theme @shopify/theme package issues #gsd:40767 Fortify local development experience for Liquid themes Theme Sev-2 second highest severity for theme related bug Type: Bug Something isn't working

Comments

@nikitaourazbaev
Copy link

Please confirm that you have:

  • Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
  • Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?

Theme

Expected behavior

The file synced should be the same as the local file

Actual behavior

The file does not get synced correctly, resulting in an “Uncaught SyntaxError: Unexpected end of input”

This seems to be related to #4525 and gets fixed with npm i -g @shopify/[email protected]

Verbose output

2024-09-26T14:27:54.368Z: ← 200 (request_id: 2c1d0c57-3b07-462f-b6a7-f5bbb02e5f15-1727360873)
2024-09-26T14:27:54.832Z: Request to https://theme-kit-access.shopifyapps.com/cli/admin/api/unstable/themes/128731349105/assets/bulk.json completed in 952 ms
With response headers:

  • cache-control: no-cache
  • content-type: application/json; charset=utf-8
  • server-timing: processing;dur=847, socket_queue;dur=6.913, cfRequestDuration;dur=915.999889
  • x-request-id: dd4d5f17-d170-4099-8297-3f645fd92eee-1727360873

• 10:27:54 Synced » update assets/utils.js

Reproduction steps

  1. This file works: https://gist.github.com/nikitaourazbaev/99ea8d0c071662f82d3e1c5c0b585346#file-will-work-js
  2. This file won’t work – only difference is the apostrophe after the word Shopify: https://gist.github.com/nikitaourazbaev/99ea8d0c071662f82d3e1c5c0b585346#file-wont-work-js

Operating System

macOS Ventura 14.7

Shopify CLI version (check your project's package.json if you're not sure)

3.67.2

Shell

Warp, fish

Node version (run node -v if you're not sure)

v20.11.1

What language and version are you using in your application?

Liquid

@nikitaourazbaev nikitaourazbaev added the Type: Bug Something isn't working label Sep 26, 2024
@craigmichaelmartin craigmichaelmartin added the Area: @shopify/theme @shopify/theme package issues label Sep 26, 2024
@karreiro karreiro added Theme Sev-2 second highest severity for theme related bug #gsd:40767 Fortify local development experience for Liquid themes labels Oct 1, 2024
@montalvomiguelo
Copy link
Contributor

montalvomiguelo commented Oct 2, 2024

a fix for this is included in 3.68.0 🚀

@frandiox
Copy link
Contributor

frandiox commented Oct 3, 2024

@nikitaourazbaev Thanks for the reproduction files! As mentioned above, this is fixed in 3.68.0 (I've tested your files with and without the fix).

Btw @montalvomiguelo thanks for notifying everyone about the release 🙌

@frandiox frandiox closed this as completed Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: @shopify/theme @shopify/theme package issues #gsd:40767 Fortify local development experience for Liquid themes Theme Sev-2 second highest severity for theme related bug Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants