Repository navigation
Conversation
…oduction released widgets
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
L03TJ3
left a comment
There was a problem hiding this comment.
@blueogin could you verify the intended Wallet integration shape here and, if needed, follow up with Copilot to align the automation instructions?
At current head 940cfb7, the mismatch is already present within this PR:
- The generated task prompt asks a new integration to add both static entry loaders and follow the custom agent exactly. That prompt is sent to Agent Tasks with main as its base.
- The custom-agent instructions also require loaders for both entries.
- The documentation example stores both as entries.react and entries.webComponent.
- But the actual registry type in this same head uses a single entry selected by integrationMode: Web Component by default, or React explicitly.
This is an instruction/example inconsistency affecting generated integrations, not evidence of an existing Wallet runtime failure. The package descriptor can still advertise both supported exports without Wallet storing both loaders. Merging main into this branch alone would not automatically reconcile these instructions, since the single-entry registry is already here.
If the current single-entry host model is intended, please align the prompt, custom-agent instructions and documentation example, then validate the example and a generated new-widget integration against the current registry types/build/tests. If dual loaders are intended instead, please clarify the corresponding host/registry change needed before enabling the automation.
Description
<wip, description coming shortly)
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
About # (link your issue here)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Checklist: