Skip to content

Remote Functions Exploration PR #13957

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 96 commits into
base: main
Choose a base branch
from
Open

Remote Functions Exploration PR #13957

wants to merge 96 commits into from
+4,946 −350

Conversation

dummdidumm
Copy link
Member

This PR is full of exploratory commits, changing directions of the implementation multiple times. As such it's of limited use, and it will be squash-merged into the remote-functions branch which can then start from a clean state to iron out bugs, tweak the final design, etc.

trueadm and others added 30 commits March 12, 2025 22:29
@trueadm
initial commit
3684cbe
@Rich-Harris
various fixes
33fecc0
@Rich-Harris
tweak demo
89894e7
@Rich-Harris
fix
10cd660
@Rich-Harris
remove acorn-typescript
9ea9af4
@Rich-Harris
simplify
0b6eea0
@Rich-Harris
fix
0fdcb1e
@Rich-Harris
generate, don't transform
afd66e0
@Rich-Harris
use an x- header
c3a4651
@Rich-Harris
regenerate manifest when remote files are added/removed
b49d381
@Rich-Harris
move RPC logic out of page, it belongs elsewhere
e5fbb1c
@Rich-Harris
use _app/remote/x
c6271ec
@Rich-Harris
tighten up error handling
9b3cb3d
@Rich-Harris
unused
ed95600
@Rich-Harris
CSRF
e61faee
@Rich-Harris
smaller payloads
e504a8b
@Rich-Harris
belt and braces
939cd49
@Rich-Harris
don't use 204, that only applies to DELETE/PUT requests
d0a1234
@Rich-Harris
turn remote_call into a factory - more compact and gives us a lot mor…
bfe200c 
…e options
@dummdidumm
Merge branch 'main' into rpc
1a823ba
@dummdidumm
analyze exports of remotes dynamically, add query/action/formAction a…
c37964e 
…s different types of remotes
@dummdidumm
update playground
ad73a46
@dummdidumm
POC: hydrate query results
8900ba3
@dummdidumm
remote form actions that hydrate and work without JS
0f26cbc
@dummdidumm
conditional
09f81ab
@dummdidumm
load fn WIP
b7bf021
@dummdidumm
rerun query on invalidation
047e564
@dummdidumm
make it possible to call invalidate in rpc functions
d96133f
@dummdidumm
fix
13fe672
@dummdidumm
adjust form API
99c8f44
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Jul 4, 2025

⚠️ No Changeset found

Latest commit: 3de122d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@sveltejs sveltejs locked as too heated and limited conversation to collaborators Jul 4, 2025
paoloricciuti
paoloricciuti reviewed Jul 6, 2025
View reviewed changes
@dummdidumm @paoloricciuti
Update packages/kit/src/runtime/app/server/remote.js
3b2a636 
Co-authored-by: Paolo Ricciuti <ricciutipaolo@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@paoloricciuti paoloricciuti paoloricciuti left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dummdidumm @paoloricciuti @trueadm @Rich-Harris