Conversation
Bumps [@sveltejs/vite-plugin-svelte](https://github.com/sveltejs/vite-plugin-svelte/tree/HEAD/packages/vite-plugin-svelte) from 6.2.4 to 7.0.0. - [Release notes](https://github.com/sveltejs/vite-plugin-svelte/releases) - [Changelog](https://github.com/sveltejs/vite-plugin-svelte/blob/main/packages/vite-plugin-svelte/CHANGELOG.md) - [Commits](https://github.com/sveltejs/vite-plugin-svelte/commits/@sveltejs/vite-plugin-svelte@7.0.0/packages/vite-plugin-svelte) --- updated-dependencies: - dependency-name: "@sveltejs/vite-plugin-svelte" dependency-version: 7.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
…/Site/sveltejs/vite-plugin-svelte-7.0.0 Bump @sveltejs/vite-plugin-svelte from 6.2.4 to 7.0.0 in /Site
follows on from #445
Contributor
There was a problem hiding this comment.
Pull request overview
Merges main into the forms branch, bringing in updated client (rbxclient) URL handling around the new DomainInsecure config, dependency bumps for the Site build toolchain, and expanded documentation for the config layout and Client TLS approach.
Changes:
- Updated several rbxclient game routes to use
config.DomainInsecureover HTTP for client-facing endpoints. - Upgraded Site dependencies (Biome, SvelteKit, Vite plugin, Svelte, and Vite) and updated
bun.lock. - Expanded Docs: clarified config property grouping, documented
DomainInsecure, and added new design pages.
Reviewed changes
Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| Site/src/routes/(rbxclient)/game/join/+server.ts | Switches join-script URLs to use DomainInsecure (HTTP) for client endpoints. |
| Site/src/routes/(rbxclient)/game/host/+server.ts | Switches host-script base URL and server presence URL to DomainInsecure (HTTP). |
| Site/src/routes/(rbxclient)/game/gameserver/+server.ts | Updates host-script base URL substitution to use DomainInsecure. |
| Site/src/routes/(rbxclient)/game/[id=asset]/serve/+server.ts | Updates dedicated host script URLs to use DomainInsecure (HTTP). |
| Site/src/routes/(rbxclient)/asset/characterfetch/[username]/+server.ts | Alters generated character appearance URLs (now referencing Domain). |
| Site/package.json | Dependency upgrades including Vite to ^8.0.0 and SvelteKit/plugin bumps. |
| Site/bun.lock | Lockfile updates reflecting dependency changes. |
| Docs/src/content/docs/guides/config.mdoc | Documents DomainInsecure and adds grouped sections for Database/Economy. |
| Docs/src/content/docs/design/mercury3.md | Adds a new design overview page for Mercury 3 vs Mercury Core. |
| Docs/src/content/docs/design/client-tls.md | Adds a new design page explaining TLS/HTTPS constraints for clients. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Comment on lines
+23
to
+25
| let charApp = `http://${config.Domain}/asset/bodycolours/${username}?` | ||
| for (const asset of user.wearing) | ||
| charApp += `;http://${config.DomainInsecure}/asset?id=${asset}` | ||
| charApp += `;http://${config.Domain}/asset?id=${asset}` |
|
|
||
| Whether to automatically start the database server when starting the Site. Requires that [SurrealDB](/install/surrealdb) is installed and available on the command line or in your system's PATH as `surreal`. | ||
|
|
||
| This is recommended to be enabled in all cases except when debugging database issues, or when managing the database server separately from the Site, surch as when using a container manager. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.