fix: deconflict multiple snippets of the same name

[trueadm]

Fixes #12186

[changeset-bot]

🦋 Changeset detected

Latest commit: 1c99218

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
svelte	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[Rich-Harris]

I don't follow where exactly, but this is causing additional effects to be created. Using the test in this PR:

export default function App($$anchor) {
  // ...

  var div = $.sibling($.sibling(button, true));

  {
    var div_1 = $.child(div);

    {
      // ...

+     $.template_effect(() => {});
    }

    var div_2 = $.sibling($.sibling(div_1, true));

    {
      // ...

+     $.template_effect(() => {});
    }

+   $.template_effect(() => {});
  }

  $.append($$anchor, fragment);
}

$.delegate(["click"]);

Also, how complicated would it be to only create the block statement in cases where a snippet is declared?

[trueadm]

I don't follow where exactly, but this is causing additional effects to be created. Using the test in this PR:

export default function App($$anchor) {
  // ...

  var div = $.sibling($.sibling(button, true));

  {
    var div_1 = $.child(div);

    {
      // ...

+     $.template_effect(() => {});
    }

    var div_2 = $.sibling($.sibling(div_1, true));

    {
      // ...

+     $.template_effect(() => {});
    }

+   $.template_effect(() => {});
  }

  $.append($$anchor, fragment);
}

$.delegate(["click"]);

Also, how complicated would it be to only create the block statement in cases where a snippet is declared?

Fixed!