Skip to content

Comments

refactor: change open recipe in new window to pass recipe id#7392

Open
zanesq wants to merge 1 commit intomainfrom
zane/recipe-id
Open

refactor: change open recipe in new window to pass recipe id#7392
zanesq wants to merge 1 commit intomainfrom
zane/recipe-id

Conversation

@zanesq
Copy link
Collaborator

@zanesq zanesq commented Feb 20, 2026

Summary

Use recipeId instead of encoded deeplinks when opening a new window

For internal flows (open recipe in new window, save & run), we now pass just the recipe manifest id string through the IPC chain instead of encoding the full recipe into a deeplink. The server's existing recipe_id + load_recipe_by_id support handles the lookup.

Also refactored createChatWindow from 6 positional optional params to an options object while I was in there.

Before:

window.electron.createChatWindow(undefined, undefined, undefined, undefined, undefined, savedId);

After:

window.electron.createChatWindow({ recipeId });

Verified locally with recipes in new windows from the ui and from deeplinks.

@zanesq zanesq requested a review from lifeizhou-ap February 20, 2026 17:44
@zanesq zanesq changed the title chore: refactor new window to pass recipe id refactor: change open recipe in new window to pass recipe id Feb 20, 2026
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