Skip to content

[WIP] Server Components#776

Draft
schultek wants to merge 24 commits intonextfrom
feat/server-components
Draft

[WIP] Server Components#776
schultek wants to merge 24 commits intonextfrom
feat/server-components

Conversation

@schultek
Copy link
Copy Markdown
Owner

Description

Add support for Server Components to Jaspr.

Type of Change

  • ❌ Breaking change
  • ✨ New feature or improvement

Kilian Schulte and others added 21 commits June 14, 2024 16:39
# Conflicts:
#	packages/jaspr/lib/src/browser/browser_binding.dart
#	packages/jaspr/lib/src/browser/clients.dart
#	packages/jaspr/lib/src/browser/js_data.dart
#	packages/jaspr/lib/src/foundation/sync.dart
#	packages/jaspr/lib/src/framework/render_element.dart
#	packages/jaspr/lib/src/server/document/document.dart
#	packages/jaspr/lib/src/server/document/document_base.dart
#	packages/jaspr/lib/src/server/document/document_binding.dart
#	packages/jaspr/lib/src/server/document/document_file.dart
#	packages/jaspr/lib/src/server/markup_renderer.dart
#	packages/jaspr_builder/lib/src/client/client_part_builder.dart
# Conflicts:
#	packages/jaspr/lib/src/browser/clients.dart
#	packages/jaspr_builder/lib/src/client/client_module_builder.dart
# Conflicts:
#	packages/jaspr/lib/browser.dart
#	packages/jaspr/lib/src/browser/clients.dart
#	packages/jaspr/lib/src/components/raw_text/raw_text_web.dart
#	packages/jaspr/lib/src/foundation/options.dart
#	packages/jaspr/lib/src/server/adapters/client_component_adapter.dart
#	packages/jaspr/lib/src/server/adapters/client_script_adapter.dart
#	packages/jaspr/lib/src/server/adapters/sync_script_adapter.dart
#	packages/jaspr/lib/src/server/server_binding.dart
#	packages/jaspr_builder/lib/src/client/client_module_builder.dart
#	packages/jaspr_builder/lib/src/client/client_registry_builder.dart
# Conflicts:
#	packages/jaspr/lib/src/browser/clients.dart
#	packages/jaspr/lib/src/components/raw_text/raw_text_web.dart
#	packages/jaspr/lib/src/foundation/marker_utils.dart
#	packages/jaspr/lib/src/foundation/options.dart
#	packages/jaspr/lib/src/server/adapters/client_component_adapter.dart
#	packages/jaspr/lib/src/server/adapters/client_script_adapter.dart
#	packages/jaspr/lib/src/server/server_binding.dart
#	packages/jaspr/test/browser/hydration/hydration_browser_test.dart
#	packages/jaspr_builder/lib/src/client/client_module_builder.dart
#	packages/jaspr_builder/lib/src/codec/codecs.dart
# Conflicts:
#	packages/jaspr/lib/src/browser/clients.dart
#	packages/jaspr/lib/src/client/custom_node_component.dart
#	packages/jaspr/lib/src/dom/raw_text/raw_text_web.dart
#	packages/jaspr/lib/src/foundation/options.dart
#	packages/jaspr/lib/src/server/adapters/client_component_adapter.dart
#	packages/jaspr/lib/src/server/adapters/client_script_adapter.dart
#	packages/jaspr/lib/src/server/server_binding.dart
#	packages/jaspr/test/client/hydration/hydration_browser_test.dart
#	packages/jaspr_builder/lib/src/client/client_module_builder.dart
#	packages/jaspr_builder/lib/src/codec/codec_module_builder.dart
#	packages/jaspr_builder/test/client/sources/client_basic.dart
#	packages/jaspr_builder/test/client/sources/client_model_class.dart
#	packages/jaspr_builder/test/client/sources/client_model_extension.dart
#	packages/jaspr_builder/test/sync/sources/sync_model_class.dart
#	packages/jaspr_builder/test/sync/sources/sync_model_extension.dart
# Conflicts:
#	packages/jaspr/lib/src/client/component_anchors.dart
#	packages/jaspr/lib/src/server/adapters/client_component_adapter.dart
#	packages/jaspr/lib/src/server/server_binding.dart
#	packages/jaspr_builder/lib/src/client/client_module_builder.dart
#	packages/jaspr_builder/lib/src/codec/codecs.dart
#	packages/jaspr_builder/lib/src/sync/sync_mixins_builder.dart
#	packages/jaspr_builder/test/client/sources/bundle.dart
#	packages/jaspr_builder/test/client/sources/client_basic.dart
#	packages/jaspr_builder/test/client/sources/client_model_class.dart
#	packages/jaspr_builder/test/client/sources/client_model_extension.dart
#	packages/jaspr_builder/test/codec/sources/bundle.dart
#	packages/jaspr_builder/test/codec/sources/model_class.dart
#	packages/jaspr_builder/test/codec/sources/model_extension.dart
#	packages/jaspr_builder/test/sync/sources/sync_basic.dart
#	packages/jaspr_builder/test/sync/sources/sync_multi.dart
@docs-page
Copy link
Copy Markdown

docs-page Bot commented Mar 16, 2026

To view this pull requests documentation preview, visit the following URL:

docs.page/schultek/jaspr~776

Documentation is deployed and generated using docs.page.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 16, 2026

Package Version Report

The following packages have been updated:
jaspr_test : 0.23.0 -> 0.24.0
jaspr : 0.23.0 -> 0.24.0
jaspr_builder : 0.23.0 -> 0.24.0
jaspr_cli : 0.23.0+3 -> 0.24.0
jaspr_router : 0.8.2 -> 0.8.3
jaspr_content : 0.5.2 -> 0.5.3
jaspr_flutter_embed : 0.4.11 -> 0.4.12
jaspr_riverpod : 0.4.5 -> 0.4.6
jaspr_serverpod : 0.6.1 -> 0.6.2

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 16, 2026

Codecov Report

❌ Patch coverage is 62.38361% with 202 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.10%. Comparing base (ee742ff) to head (f5bbd92).

Files with missing lines Patch % Lines
...ckages/jaspr/lib/src/client/component_anchors.dart 52.23% 32 Missing ⚠️
...kages/jaspr/lib/src/client/slotted_child_view.dart 70.00% 30 Missing ⚠️
packages/jaspr/lib/src/framework/build_owner.dart 0.00% 21 Missing ⚠️
packages/jaspr/lib/src/client/client_binding.dart 37.50% 20 Missing ⚠️
.../src/server/adapters/client_component_adapter.dart 41.93% 18 Missing ⚠️
packages/jaspr/lib/src/server/server_handler.dart 15.00% 17 Missing ⚠️
packages/jaspr/lib/src/framework/components.dart 90.81% 9 Missing ⚠️
packages/jaspr/lib/src/framework/framework.dart 22.22% 7 Missing ⚠️
packages/jaspr/lib/src/client/client_app.dart 50.00% 6 Missing ⚠️
...es/jaspr/lib/src/framework/stateful_component.dart 0.00% 6 Missing ⚠️
... and 12 more
Additional details and impacted files
@@            Coverage Diff             @@
##             next     #776      +/-   ##
==========================================
- Coverage   68.52%   68.10%   -0.43%     
==========================================
  Files         212      214       +2     
  Lines       11857    12251     +394     
==========================================
+ Hits         8125     8343     +218     
- Misses       3732     3908     +176     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Contributor

@parlough parlough left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exciting!! I haven't tried it out yet, but some small questions and a few minor suggestions to consider:

Comment thread packages/jaspr/lib/src/client/component_anchors.dart Outdated
Comment thread packages/jaspr/lib/src/client/component_anchors.dart Outdated
Comment thread packages/jaspr/lib/src/client/options.dart Outdated
Comment thread packages/jaspr/lib/src/client/options.dart Outdated
Comment thread packages/jaspr/lib/src/client/options.dart Outdated
Comment on lines +72 to +73
for (var i = 0; i < _adapters.length; i++) {
final r = _adapters[i].prepare();

This comment was marked as resolved.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't use foreach because new adapters can be added during .prepare(). I will add a comment.

Comment thread packages/jaspr/lib/src/server/server_binding.dart Outdated
Comment on lines +117 to +118
rootRenderObject.node = newNode;
rootRenderObject.toHydrate = [...newNode.childNodes.toIterable()];
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe encapsulate this logic in a new method on RootDomRenderObject, so node doesn't need to have a public setter.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Also I noticed I could remove some dead code in RootDomRenderObject.


/// Reloads the current page.
void reload([String? path]) {
throw UnimplementedError('Should be overridden on client and server.');
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems the client's AppContext extension doesn't have this method. Is that intended?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah no, that is leftover from I feature I was experimenting with but decided against for the initial version of server components. Will remove.

} else if (allowComponent &&
type.element.name == 'Component' &&
type.element.library.identifier == 'package:jaspr/src/framework/framework.dart') {
return (decoder: 'p.mount($argument)', cast: 'String');
Copy link
Copy Markdown
Contributor

@parlough parlough Mar 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What if the component parameter is nullable? Should this consider the nullability suffix like the above handling?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch

@schultek schultek force-pushed the feat/server-components branch from 38c987f to e46ec38 Compare March 18, 2026 14:11
@schultek schultek changed the base branch from main to next April 20, 2026 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants