Repository navigation
[Hotfix] Cherry-pick to release-v0.179.3 - #1093
Closed
truefoundry-saas wants to merge 5 commits into
Closed
truefoundry-saas wants to merge 5 commits into
truefoundry-saas wants to merge 5 commits into
Conversation
🦋 Changeset detectedLatest commit: 00ad3c2 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
…de tenant id (#1036) * chore: accept PostHog Vite build args in Dockerfile Allow SaaS image builds to bake in analytics via --build-arg without affecting default OSS images. Co-authored-by: Cursor <cursoragent@cursor.com> * refactor: update analytics event names to include 'trueforge' namespace * refactor: enhance analytics tracking with tenant ID and update session state structure * feat: prefix AnalyticsEvents names with 'trueforge.' for better event namespacing * refactor: simplify SessionState type definition for clarity --------- Co-authored-by: Cursor <cursoragent@cursor.com> (cherry picked from commit 7f2be59)
* Wire posthog creds during build * nit: env ordering (cherry picked from commit 69f0990)
* feat: fine-grained tool streaming via TFY beta header TrueFoundry uses anthropic_beta instead of mutating tool defs; native Anthropic keeps call-level toolStreaming. Co-authored-by: Cursor <cursoragent@cursor.com> * refactor: inline TFY fine-grained tool streaming beta string Co-authored-by: Cursor <cursoragent@cursor.com> * fix: use anthropic-beta header name for TFY tool streaming Match Anthropic's wire header spelling instead of anthropic_beta. Co-authored-by: Cursor <cursoragent@cursor.com> * chore: drop TFY beta header unit test; note deprecation Co-authored-by: Cursor <cursoragent@cursor.com> * feat: report invalid tool-arg JSON as a distinct parse error Fine-grained streaming can leave incomplete arguments; surface parse_error without echoing the raw payload. Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Cursor <cursoragent@cursor.com> (cherry picked from commit 248e768)
cursor
Bot
requested review from
chiragjn,
debajyoti-truefoundry,
govindavashishtha,
heerambavi1998,
kumarsks619,
qaifi-tf,
sajal-truefoundry,
sayan-truefoundry,
thesujai and
vinit-truefoundry
as code owners
October 19, 2026 10:13
truefoundry-saas
deleted the
cherry-pick/735ce6c7-5f21-475b-9b37-e1109508fa67/release-v0.179.3
branch
October 19, 2026 10:16
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated cherry-pick for release request
735ce6c7-5f21-475b-9b37-e1109508fa67.Source PRs (merged to main):
Warning
This PR does not contain the fix yet. The cherry-pick conflicted, so
automation stopped at the first conflicting commit and left the rest out.
GitHub therefore shows this PR as mergeable, with no conflict to resolve —
it currently carries no changes at all (an empty placeholder commit).
Paste the prompt below into any coding agent that has this repository open.
It checks out this branch, applies the missing commits, and pushes.
Do not merge the PR. The release request checks every commit within a few
minutes (or use "Sync status" on it). The PR is merged when the request is
approved — or, if it is already approved, when a QA member or engineering
admin uses "Retry merge".
Agent prompt — copy from here:
Conflicting file(s):
packages/frontend/src/authSession.tsCommits still to apply (starting at the conflict):
7f2be59e6e8beaa91d67aa9e6338203a519beed769f099012ee004b79c4bd9da0e7e5725b804cdc0248e7683b4d3f96b5fd0bb54e875989ad1c148bcDo not merge manually — this PR is merged automatically when the release request is approved. If you do merge it yourself, use "Sync status" on the request so it stops showing as pending.
Note
Medium Risk
Renaming analytics events is a breaking contract for hosts on old names; LLM header/tool-streaming and stricter tool-arg parsing change agent behavior on incomplete streamed JSON.
Overview
Analytics & product telemetry: Public
AnalyticsEventsstrings are renamed with atrueforge.prefix so host sinks like PostHog can namespace TrueForge events. The harness app wires PostHog at build time (VITE_POSTHOG_*in Docker and release workflow) and only attacheshostAnalyticsafter auth, enriching each capture withtenantfrom the session.Agent runtime (LLM + tools): TrueFoundry OpenAI-compatible calls add the
anthropic-beta: fine-grained-tool-streaming-2025-05-14header; Anthropic provider options always settoolStreaming: true. Tool execution parsesfunction.argumentsbeforecallTool—invalid or non-object JSON returns a structured parse error without invoking the tool (new test coverage).OpenAPI/docs version bumps to 0.179.2; patch changesets document the UI and core package bumps.
Reviewed by Cursor Bugbot for commit 00ad3c2. Bugbot is set up for automated code reviews on this repo. Configure here.