Skip to content

[pull] main from lynx-family:main#566

Merged
pull[bot] merged 4 commits intocolinaaa:mainfrom
lynx-family:main
Mar 30, 2026
Merged

[pull] main from lynx-family:main#566
pull[bot] merged 4 commits intocolinaaa:mainfrom
lynx-family:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Mar 30, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

PupilTong and others added 4 commits March 30, 2026 20:08
#2399)

<!--
  Thank you for submitting a pull request!

We appreciate the time and effort you have invested in making these
changes. Please ensure that you provide enough information to allow
others to review your pull request.

Upon submission, your pull request will be automatically assigned with
reviewers.

If you want to learn more about contributing to this project, please
visit:
https://github.com/lynx-family/lynx-stack/blob/main/CONTRIBUTING.md.
-->

<!-- The AI summary below will be auto-generated - feel free to replace
it with your own. -->

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

## Release Notes

* **Bug Fixes**
* Fixed `queryComponent` fallback behavior when processing evaluation
results is unavailable
  * Improved event handling to correctly process non-bubbling events

* **Tests**
* Added end-to-end tests for component query execution with and without
result processing
  * Added test coverage for non-bubbling event propagation scenarios
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

## Checklist

<!--- Check and mark with an "x" -->

- [ ] Tests updated (or not required).
- [ ] Documentation updated (or not required).
- [ ] Changeset added, and when a BREAKING CHANGE occurs, it needs to be
clearly marked (or not required).
<!--
  Thank you for submitting a pull request!

We appreciate the time and effort you have invested in making these
changes. Please ensure that you provide enough information to allow
others to review your pull request.

Upon submission, your pull request will be automatically assigned with
reviewers.

If you want to learn more about contributing to this project, please
visit:
https://github.com/lynx-family/lynx-stack/blob/main/CONTRIBUTING.md.
-->

<!-- The AI summary below will be auto-generated - feel free to replace
it with your own. -->

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Tests**
* Added validation tests to ensure CSS custom properties (variables) are
correctly preserved in inline style color declarations, supporting both
standard variables and fallback values.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

## Checklist

<!--- Check and mark with an "x" -->

- [ ] Tests updated (or not required).
- [ ] Documentation updated (or not required).
- [ ] Changeset added, and when a BREAKING CHANGE occurs, it needs to be
clearly marked (or not required).
This extracts the misc micro-trims from `bundle-size` into a standalone
slice.

- simplify `BackgroundSnapshotInstance.childNodes()` by iterating
directly until `null`
- drop redundant guards in commit patch assembly now that
`snapshotPatch` is guaranteed after the early return

Validation: `git diff --check`

`pnpm exec biome check` could not run in this worktree because `biome`
is not installed locally.

issue: #17 

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Refactor**
  * Optimized internal conditional logic for improved code efficiency.
  * Enhanced child node traversal handling for better performance.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
This slices the hook callback dedup optimization out of the bundle-size
work.

Changes:
- Share the render hook wrapper in
packages/react/runtime/src/lifecycle/isRendering.ts
- Share the timing hook wrapper in
packages/react/runtime/src/lynx/performance.ts
- Remove the now-unused preact VNode type import

Validation:
- git diff --check

Commit: e9c7554
issue: #17

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Refactor**
* Internal code consolidation and cleanup for improved maintainability.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
@pull pull bot locked and limited conversation to collaborators Mar 30, 2026
@pull pull bot added the ⤵️ pull label Mar 30, 2026
@pull pull bot merged commit 495ad36 into colinaaa:main Mar 30, 2026
@pull pull bot added the ⤵️ pull label Mar 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants