Skip to content

Comments

refactor: move langfuse wrapper to a module in exchange instead of a package#138

Merged
ahau-square merged 17 commits intomainfrom
sm/mv-langfuse-wrapper
Oct 16, 2024
Merged

refactor: move langfuse wrapper to a module in exchange instead of a package#138
ahau-square merged 17 commits intomainfrom
sm/mv-langfuse-wrapper

Conversation

@salman1993
Copy link
Contributor

@salman1993 salman1993 commented Oct 10, 2024

have the Langfuse wrapper as module in the exchange package. This allows us to not release langfuse_wrapper as a package to pypi.

working locally for me:
Screenshot 2024-10-10 at 7 06 22 PM

-> see these traces
Screenshot 2024-10-10 at 7 06 32 PM

improvement: we should start a langfuse session to batch all related traces for a session. currently, there are multiple traces for each turn -> https://langfuse.com/docs/tracing-features/sessions

@ahau-square
Copy link
Contributor

I like the justfile addition!

* origin/main:
  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)
@ahau-square ahau-square merged commit 8cf7b9f into main Oct 16, 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 sm/mv-langfuse-wrapper branch October 24, 2024 11:44
ahau-square pushed a commit that referenced this pull request May 2, 2025
…package (#138)

Co-authored-by: Alice Hau <ahau@squareup.com>
cbruyndoncx pushed a commit to cbruyndoncx/goose that referenced this pull request Jul 20, 2025
…package (block#138)

Co-authored-by: Alice Hau <ahau@squareup.com>
Ghenghis referenced this pull request in Ghenghis/Super-Goose Feb 5, 2026
This removes the goose/temp/ directory which contained old test projects
with vulnerable dependencies that were incorrectly tracked in git.

Fixes Dependabot alerts:
- #147-162: Python dependencies in watchflow-main/uv.lock (langchain, aiohttp, urllib3, etc.)
- #138-139: esbuild vulnerabilities in vibes-cli temp files

The @modelcontextprotocol/sdk in ui/desktop is already updated to 1.26.0
which fixes alert #167.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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.

2 participants