Automatic renaming of snippet and assets on file rename

[charlespwd]

What are you adding in this PR?

automatic-rename-refactor.mp4

• When snippets/*.liquid files are renamed, we ask the client to do the following:
  • update all the {% render 'oldName' %} calls to {% render 'newName' %}
• When assets/*(.liquid)? files are renamed, we ask the client to do the following:
  • update all the {{ 'oldName.ext' | asset_url }} calls to {{ 'newName.ext' | asset_url }}
• Add a preload method to the DocumentManager
  • this makes it possible for files that are not open to be parsed and be accesible
  • we won't reparse files that are already parsed
  • files from disk are cached in the DocumentManager with a version of undefined
• Add an optional second argument to DocumentManager#theme
  • includeFilesFromDisk will make it so the theme method returns all the files in the theme
  • the default is still to only return all the open files
  • that way theme check keeps running on the open files only.

How it works

• The server listens for workspace/didRenameFiles notifications from the language client
• The server preloads the theme and identify what TextDocumentEdit we need to apply to modify the theme
• The server sends a workspace/applyEdit request to the client
• We're done.

sequenceDiagram
    participant C as Client
    participant S as Server
    C->>S: Notification "workspace/didRenameFiles"
    activate S
    S->>S: Find relevant changes
    S->>+C: Request "workspace/applyEdit"
    activate C
    C->>-S: Response "applied!"
    deactivate S

Loading

What's next? Any followup issues?

• Update theme block references when theme blocks are renamed #547
• Update section references when sections are renamed #546
• Renaming a section group should update references to that section group #548

Before you deploy

• I included a minor bump changeset

[aswamy]

🎩 complete (we already discussed about the limitations on auto-saving after renaming files on Node)

minor comments, but nothing blocking

[aswamy]

I couldn't tophat this, but could we also attempt to rename two files at once in both handler's test cases? From the code, it appears as if that's possible.

[charlespwd]

Tbh, I put that in to satisfy typescript. You'd get an array when you rename folders, but if you did then the condition isSnippet() for both the old and new uri would never be true.

I'm not 100% sure how else you could get multiple snippet renames in one UI operation.

[aswamy]

Suggested change

	it('handles .js.liquid files', async (ext) => {
	it('handles when asset file is renamed and has added a `.liquid` extension to it', async (ext) => {

[aswamy]

Suggested change

	it('handles .css.liquid files', async () => {
	it('handles when asset file with `.liquid` extension is renamed', async () => {