Skip to content

fix: session resume with arg handled incorrectly#145

Merged
codefromthecrypt merged 3 commits intoblock:mainfrom
lamchau:lam/fix-session-resume
Oct 13, 2024
Merged

fix: session resume with arg handled incorrectly#145
codefromthecrypt merged 3 commits intoblock:mainfrom
lamchau:lam/fix-session-resume

Conversation

@lamchau
Copy link
Contributor

@lamchau lamchau commented Oct 12, 2024

This patch addresses 2 bugs:

  • goose session resume <session_name> which is used by the IntelliJ plugin to break
  • improper handling of case sensitive menu options used by @click.

@lamchau lamchau requested review from Kvadratni, baxen and lifeizhou-ap and removed request for lifeizhou-ap October 12, 2024 00:08
@lamchau lamchau force-pushed the lam/fix-session-resume branch from 6ed3cfc to 6b361be Compare October 12, 2024 00:12
@codefromthecrypt codefromthecrypt merged commit 6ea6d14 into block:main Oct 13, 2024
lukealvoeiro added a commit that referenced this pull request Oct 17, 2024
* main: (23 commits)
  feat: Run with resume session (#153)
  refactor: move langfuse wrapper to a module in exchange instead of a package (#138)
  docs: add subheaders to the 'Other ways to run Goose' section (#155)
  fix: Remove tools from exchange when summarizing files (#157)
  chore: use primitives instead of typing imports and fixes completion … (#149)
  chore: make vcr tests pretty-print JSON (#146)
  chore(release): goose 0.9.5 (#159)
  chore(release): exchange 0.9.5 (#158)
  chore: updates ollama default model from mistral-nemo to qwen2.5 (#150)
  feat: add vision support for Google (#141)
  fix: session resume with arg handled incorrectly (#145)
  docs: add release instructions to CONTRIBUTING.md (#143)
  docs: add link to action, IDE words (#140)
  docs: goosehints doc fix only (#142)
  chore(release): release 0.9.4 (#136)
  revert: "feat: add local langfuse tracing option  (#106)" (#137)
  feat: add local langfuse tracing option  (#106)
  feat: add groq provider (#134)
  feat: add a deep thinking reasoner model (o1-preview/mini) (#68)
  fix: use concrete SessionNotifier (#135)
  ...
@lamchau lamchau deleted the lam/fix-session-resume branch October 24, 2024 11:42
cbruyndoncx pushed a commit to cbruyndoncx/goose that referenced this pull request Jul 20, 2025
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

Comments