Skip to content

feat: support React live molecule props#21

Merged
aose-yuu merged 3 commits into
mainfrom
codex/live-props
May 16, 2026
Merged

feat: support React live molecule props#21
aose-yuu merged 3 commits into
mainfrom
codex/live-props

Conversation

@aose-yuu
Copy link
Copy Markdown
Contributor

@aose-yuu aose-yuu commented May 16, 2026

Purpose

Support live props sources for React molecules while updating the adapter to the core 0.7 peer range.

Summary

  • Adds the useMolecule(factory, () => props, deps) contract for live React props without changing object props snapshot behavior.
  • Requires dependency lists for React props getters and keeps getter-without-deps rejected at both type and runtime boundaries.
  • Marks the package output as Client Component code and documents the React-specific props and SSR constraints.
  • Covers StrictMode, SSR cleanup, stable deps, optional props, and callable prop object cases.

Verification

  • mise run ci

@aose-yuu aose-yuu marked this pull request as ready for review May 16, 2026 15:09
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a6b5a26ad7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/useMolecule.ts
@aose-yuu aose-yuu merged commit c04a0c8 into main May 16, 2026
1 check passed
@aose-yuu aose-yuu deleted the codex/live-props branch May 16, 2026 15:24
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.

1 participant