prefix function

[jycouet]

Fixes #751, closes #597

Since SvelteKit 1.0.0-next.573, we are not allowed to export non-Kit functions from +page.js, or it has to start with "_".

To help everyone out, please make sure your PR does the following:

• Update the first line to point to the ticket that this PR fixes
• Add a message that clearly describes the fix
• If applicable, add a test that would fail without this fix
• Make sure the unit and integration tests pass locally with pnpm run tests and cd integration && pnpm run tests
• Includes a changeset if your fix affects the user with pnpm changeset

[changeset-bot]

🦋 Changeset detected

Latest commit: 293c790

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

This PR includes changesets to release 3 packages

Name	Type
houdini-svelte	Minor
houdini-react	Minor
houdini	Minor

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

[vercel]

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

Name	Status	Preview	Updated
docs	✅ Ready (Inspect)	Visit Preview	Dec 9, 2022 at 10:30PM (UTC)
docs-next	✅ Ready (Inspect)	Visit Preview	Dec 9, 2022 at 10:30PM (UTC)

[sjcobb2022]

Just a quick comment. Wouldn't this effect/fix #597?

Having the type without the _ prefix prevents the collision.

[jycouet]

Just a quick comment. Wouldn't this effect/fix #597?

Having the type without the _ prefix prevents the collision.

Yes, you are right. We will have no more collisions with this fix.

You will be able to do:

import { FooVariables } from '$houdini'

export const _FooVariables: FooVariables = () => { ...}

---

Just FYI, I will probably continue to use:

import { FooVariables as V } from '$houdini'

export const _FooVariables: V = () => { ...}

To have as less "FooVariables" as possible, but I guess that it's only a preference