sourceRef.current in useMemo may produce stale matchedTemplate

オープン

#36 2026年3月24日に作成コメント 0 件リアクション 0 件担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度: 3/5
見積もり時間: 1〜2日
初心者へのやさしさ: 55/100

issue の種類: バグ
明瞭さ: おおむね明確
活発さ: 静か
技術スタック: typescript
領域: frontend

調査の方向性

Start in apps/app/src/components/generative-ui/save-template-overlay.tsx and inspect the matchedTemplate useMemo, its dependency array, and how sourceRef.current is set in the apply flow. Choose a reactive source-template representation, then verify that the matched template badge updates when the source ref changes after the last html or templates change.

索引モデルが issue の本文から書いたものです。

説明

bug

Problem

In save-template-overlay.tsx, the matchedTemplate useMemo reads sourceRef.current to find the source template from the apply flow. However, React refs don't trigger re-renders when they change, so the memo's dependency array ([html, agent.state?.templates]) won't capture changes to sourceRef.current.

In practice this works because html or templates changes trigger recomputation, but it's technically not guaranteed — if the ref is set after the last html/templates change, the matched template badge could show stale data.

Suggested fix

Either:

Store the source template info in state instead of a ref (triggers re-render on change)
Add the source ID as an explicit dependency by extracting it to a state variable

Files

apps/app/src/components/generative-ui/save-template-overlay.tsx

From PR #20 review

主要言語: TypeScript
スター: 1.6k
フォーク: 202
PR マージ指標: 30日以内にマージされた PR はありません

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
リポジトリをフォークし、ブランチを切って変更します。
issue 番号を参照したプルリクエストを送ります。

CopilotKit/OpenGenerativeUI のほかの issue

Script dedup key truncation may cause collisions オープン

bug

難易度 2/5 1〜3時間初心者へのやさしさ 72/100

CopilotKit/OpenGenerativeUI#37 · 2026年3月24日に作成
Deduplicate progressive rendering pattern between playbook and mechanical skill オープン

enhancement

難易度 2/5 1〜3時間初心者へのやさしさ 72/100

CopilotKit/OpenGenerativeUI#29 · 2026年3月23日に作成
Agent loop stops after first tool call with non-Claude models (GLM-5): generateSandboxedUi never invoked オープン

難易度 4/5 3〜5日初心者へのやさしさ 48/100

CopilotKit/OpenGenerativeUI#87 · コメント 2 件 · 2026年7月18日に作成
Replace custom widgetRenderer with CopilotKit Open Generative UI オープン

難易度 5/5 1週間以上初心者へのやさしさ 35/100

CopilotKit/OpenGenerativeUI#82 · 2026年4月30日に作成
Edit existing HTML instead of regenerating from scratch オープン

難易度 4/5 3〜5日初心者へのやさしさ 45/100

CopilotKit/OpenGenerativeUI#77 · 2026年4月6日に作成

CopilotKit/OpenGenerativeUI の issue をすべて見る

似ている issue

portal pins drifted from wallet main オープン

難易度 2/5 1〜3時間初心者へのやさしさ 72/100

0xMiden/bridge-portal#132 · 2026年9月21日に作成
Cloudflare scaffold recommends the keyless cloudflare/... model but never declares the AI binding it requires オープン

難易度 2/5 1〜3時間初心者へのやさしさ 78/100

withastro/flue#739 · 2026年9月21日に作成
getEcashRowColors throws on the default theme: buttonGradient is an array, blendHexColors expects a string オープン

難易度 2/5 1〜3時間初心者へのやさしさ 82/100

ZeusLN/zeus#4687 · 2026年9月21日に作成
Do not render a phantom EOF line in read output オープン

area:tools bug good first issue help wanted priority:P2

難易度 2/5 1〜3時間初心者へのやさしさ 90/100

TaewoooPark/Motifcode#14 · 2026年9月21日に作成
Antigravity CLI: PostToolUse hook misidentified as PreToolUse due to `toolCall` check, causing protojson error オープン

bug

難易度 2/5 1〜3時間初心者へのやさしさ 84/100

newrelic-experimental/preflight#793 · コメント 1 件 · 2026年9月21日に作成

TypeScript の issue をもっと見る