Skip to content

Conversation

@DOsinga
Copy link
Collaborator

@DOsinga DOsinga commented Oct 17, 2025

This makes one window sort of workable for new chats, resume chats and recipes. Also brings back recipe acceptance and security scanning to BaseChat2

Things to do

  • check on why starting from hub flickers
  • allow to start a chat from pair if no chat is open yet
  • hub starts also seem to render the conversation twice
  • hook up recipe variables entry
  • centralize recipe launching
  • find other places where we still open a new window
  • integrate with @lifeizhou-ap 's work for creating recipes with system prompts the right way

@DOsinga DOsinga requested a review from a team as a code owner October 17, 2025 18:19
@zanesq
Copy link
Collaborator

zanesq commented Oct 17, 2025

Will look into the following in another branch off this one.

  • check on why starting from hub flickers
  • hub starts also seem to render the conversation twice

While it should be default to launch in same window we should continue to support open in a new window for sessions and recipes and add another button for that option (users might not want to always overwrite their current chat window) -- can follow up with this later

@DOsinga
Copy link
Collaborator Author

DOsinga commented Oct 17, 2025

supporting multiple windows seems fine, but we then do need to make sure it talks to the same goosed still

@DOsinga DOsinga mentioned this pull request Oct 17, 2025
2 tasks
@zanesq
Copy link
Collaborator

zanesq commented Oct 17, 2025

tried a new chat on this branch and getting

12:56:38.554 › goosed stderr for port 58183 and dir /Users/zane/Development/goose-main:   2025-10-17T19:56:38.554157Z ERROR goosed::routes::reply: Failed to read session for : Session not found
    at crates/goose-server/src/routes/reply.rs:240
image

@DOsinga
Copy link
Collaborator Author

DOsinga commented Oct 17, 2025

oh is this when you start a chat from pair? I think we might have to still implement that too!

@zanesq
Copy link
Collaborator

zanesq commented Oct 17, 2025

ah yes from pair, I thought that was working when we were sharing earlier but must have been mistaken, its working from hub can use that for now 👍

@DOsinga
Copy link
Collaborator Author

DOsinga commented Oct 17, 2025

only if a conversation is already loaded. can look into this next week

@zanesq
Copy link
Collaborator

zanesq commented Oct 21, 2025

@DOsinga fixed above issue when new chat in pair and reload

@zanesq zanesq merged commit d836a10 into main Oct 21, 2025
14 checks passed
@zanesq zanesq deleted the next-camp branch October 21, 2025 22:00
wpfleger96 added a commit that referenced this pull request Oct 22, 2025
* main: (33 commits)
  Add Web Accessibility Auditor recipe to cookbook (#5318)
  To do mcp tutorial (#5317)
  workflows: add a manual trigger option to pr-smoke-test (#5302)
  documenting `goose recipe list` command (#5278)
  add a system prompt snapshot test (#5305)
  fix: handle non exist subrecipe path (#5287)
  Next camp (#5237)
  more lowercasing of TabItem labels (#5307)
  modified docs/tutorials/cicd Github Action's install path to follow download_cli script (#5240)
  Fix artifact download to work across workflow runs (#5304)
  Added extension search (#5283)
  docs: lowercase 'goose' in TabItem labels for consistency (#5297)
  feat(prompts): add format to save code snippet (#5007)
  fix: use Windows-compatible default path for CLI installation (#5221)
  feat: add Test Coverage Optimizer recipe (#5118) (#5272)
  Upgrade node to fix canary (#5301)
  Remove reliance on localstorage for pendingScheduleDeepLink when scheduling a recipe (#5290)
  Add historical tracking with trend indicators using artifacts (#5295)
  roll back vite and electron package upgrades breaking canary win and linux (#5292)
  Revert "Revert "Rewrite extension management tools"" (#5273)
  ...
BlairAllan pushed a commit to BlairAllan/goose that referenced this pull request Oct 25, 2025
Co-authored-by: Douwe Osinga <[email protected]>
Co-authored-by: Zane <[email protected]>
Co-authored-by: Zane Staggs <[email protected]>
Signed-off-by: Blair Allan <[email protected]>
BlairAllan pushed a commit to BlairAllan/goose that referenced this pull request Nov 29, 2025
Co-authored-by: Douwe Osinga <[email protected]>
Co-authored-by: Zane <[email protected]>
Co-authored-by: Zane Staggs <[email protected]>
Signed-off-by: Blair Allan <[email protected]>
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.

3 participants