chore: update dependencies and fix captive portal build#487
Merged
Conversation
Update frontend dependencies (package.json, lockfile, .nvmrc, workspace, svelte-core submodule) and fix @openshock/svelte-core import specifiers that used unexported ./...index.ts subpaths, breaking `pnpm run build`. Rewrite imports to the package's exported subpaths: - components/ui/<X>/index.ts -> ui/<X> - components/stepper/index.ts -> stepper - components/index.ts -> components - typeguards/index.ts -> typeguards - state/<X>.svelte.ts -> state/<X>.svelte
Contributor
Change file check
If this change should show up in the release notes, add one with |
|
Ready to review this PR? Stage has broken it down into 4 individual chapters for you:
Chapters generated by Stage for commit e45ff5f on Jul 7, 2026 9:30am UTC. |
Member
|
but why |
Contributor
Author
why what? |
…rborepo - Resolve package.json/pnpm-lock.yaml conflicts (take higher dep versions) - Bump svelte-core submodule to 89d2853 (Frontend's pin; source-based exports) - Drop Turborepo: remove turbo dependency, consume svelte-core from source - Keep frontend imports on the components/ui/<X>/index.ts style (matches Frontend) - build stays plain 'vite build' (no cross-env) Verified: svelte-check 0 errors, vite build passes.
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the frontend toolchain/dependency set (Node, pnpm, and various frontend devDependencies) and adjusts pnpm workspace/lockfile overrides to keep installs/builds consistent across the workspace.
Changes:
- Bumped frontend devDependencies (Vite, Tailwind, Svelte tooling, etc.) and refreshed
pnpm-lock.yaml. - Updated Node/pnpm tooling expectations via
frontend/.nvmrc,frontend/package.jsonengines, andpackageManager. - Added pnpm overrides for
cookieandts-deepmergeplus minimum-release-age exceptions inpnpm-workspace.yaml.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| frontend/pnpm-workspace.yaml | Adds minimum-release-age exclusions and overrides for cookie/ts-deepmerge. |
| frontend/pnpm-lock.yaml | Updates lockfile to reflect new dependency graph and overrides. |
| frontend/package.json | Bumps frontend devDependencies; updates Node/pnpm requirements and package manager pin. |
| frontend/.nvmrc | Updates the pinned Node version used for local development. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
f85a2ab resolves directory subpaths for TS/Node consumers, so imports can reference the package's directory subpaths directly (e.g. @openshock/svelte-core/components/ui/button) without /index.
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.
Update frontend dependencies (package.json, lockfile, .nvmrc, workspace, svelte-core submodule) and fix @openshock/svelte-core import specifiers that used unexported ./...index.ts subpaths, breaking
pnpm run build.Rewrite imports to the package's exported subpaths: