Skip to content

fix(deepagents): omit middleware trace inputs - #837

Merged
Hunter Lovell (hntrl) merged 2 commits into
mainfrom
hunter/omit-middleware-trace-inputs
Sep 6, 2026
Merged

Hunter Lovell (hntrl) merged 2 commits into
mainfrom
hunter/omit-middleware-trace-inputs

Conversation

@hntrl

@hntrl Hunter Lovell (hntrl) commented Sep 6, 2026

Copy link
Copy Markdown
Member

(in an rc release, using this to test in a real deployment)

  • adopts a policy of dropping inputs for built in middleware -- we seldom get value out of these input values and for large traces this can cause issues

see langchain-ai/langchainjs#11568 and langchain-ai/langgraphjs#2794

@vercel

vercel Bot commented Sep 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
deepagentsjs-ui Ready Ready Preview Sep 6, 2026 3:11am UTC

Request Review

@changeset-bot

changeset-bot Bot commented Sep 6, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: ef7ce9c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
deepagents Patch
@langchain/quickjs Patch
deepagents-acp Patch
@deepagents/evals Patch

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

@socket-security

socket-security Bot commented Sep 6, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Added@​langchain/​langgraph@​1.4.15-rc.0991009297100
Addedlangchain@​1.5.11-rc.0991009396100

View full report

@pkg-pr-new

pkg-pr-new Bot commented Sep 6, 2026

Copy link
Copy Markdown

Open in StackBlitz

deepagents-acp

npm i https://pkg.pr.new/deepagents-acp@837

deepagents

npm i https://pkg.pr.new/deepagents@837

@langchain/sandbox-standard-tests

npm i https://pkg.pr.new/@langchain/sandbox-standard-tests@837

@langchain/daytona

npm i https://pkg.pr.new/@langchain/daytona@837

@langchain/deno

npm i https://pkg.pr.new/@langchain/deno@837

@langchain/modal

npm i https://pkg.pr.new/@langchain/modal@837

@langchain/node-vfs

npm i https://pkg.pr.new/@langchain/node-vfs@837

@langchain/quickjs

npm i https://pkg.pr.new/@langchain/quickjs@837

commit: ef7ce9c

@open-swe open-swe Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Open SWE Review: No issues found

Open SWE reviewed this PR and found no potential bugs to report.

Open in WebView Open SWE trace

Comment thread .changeset/omit-middleware-trace-inputs.md Outdated
Signed-off-by: Hunter Lovell <40191806+hntrl@users.noreply.github.com>
@hntrl
Hunter Lovell (hntrl) merged commit 22cc873 into main Sep 6, 2026
22 checks passed
@hntrl
Hunter Lovell (hntrl) deleted the hunter/omit-middleware-trace-inputs branch September 6, 2026 03:15
Hunter Lovell (hntrl) added a commit that referenced this pull request Sep 6, 2026
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.

⚠️⚠️⚠️⚠️⚠️⚠️

`main` is currently in **pre mode** so this branch has prereleases
rather than normal releases. If you want to exit prereleases, run
`changeset pre exit` on `main`.

⚠️⚠️⚠️⚠️⚠️⚠️

# Releases
## deepagents-acp@0.1.30-rc.0

### Patch Changes

- Updated dependencies
[[`22cc873`](22cc873)]:
  - deepagents@1.13.4-rc.0
## deepagents@1.13.4-rc.0

### Patch Changes

- [#837](#837)
[`22cc873`](22cc873)
Thanks [@hntrl](https://github.com/hntrl)! - fix(middleware): omit
repeated conversation and state inputs from owned middleware lifecycle
traces

Keep middleware spans and outputs, and preserve model/tool tracing and
user-supplied middleware policies. Apply the defaults to standalone
middleware, subagents, and QuickJS cleanup. Lifecycle chain events also
receive the omitted inputs.
## @langchain/daytona@0.2.3-rc.0

### Patch Changes

- Updated dependencies
[[`22cc873`](22cc873)]:
  - deepagents@1.13.4-rc.0
## @langchain/deno@0.2.4-rc.0

### Patch Changes

- Updated dependencies
[[`22cc873`](22cc873)]:
  - deepagents@1.13.4-rc.0
## @langchain/modal@0.1.7-rc.0

### Patch Changes

- Updated dependencies
[[`22cc873`](22cc873)]:
  - deepagents@1.13.4-rc.0
## @langchain/node-vfs@0.2.4-rc.0

### Patch Changes

- Updated dependencies
[[`22cc873`](22cc873)]:
  - deepagents@1.13.4-rc.0
## @langchain/quickjs@0.6.3-rc.0

### Patch Changes

- [#837](#837)
[`22cc873`](22cc873)
Thanks [@hntrl](https://github.com/hntrl)! - fix(middleware): omit
repeated conversation and state inputs from owned middleware lifecycle
traces

Keep middleware spans and outputs, and preserve model/tool tracing and
user-supplied middleware policies. Apply the defaults to standalone
middleware, subagents, and QuickJS cleanup. Lifecycle chain events also
receive the omitted inputs.

- Updated dependencies
[[`22cc873`](22cc873)]:
  - deepagents@1.13.4-rc.0
## @langchain/sandbox-standard-tests@2.0.1-rc.0

### Patch Changes

- Updated dependencies
[[`22cc873`](22cc873)]:
  - deepagents@1.13.4-rc.0
## @deepagents/evals@0.0.29-rc.0

### Patch Changes

- Updated dependencies
[[`22cc873`](22cc873)]:
  - deepagents@1.13.4-rc.0

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Hunter Lovell <hunter@hntrl.io>
Baihao Wang (byhow) added a commit that referenced this pull request Sep 9, 2026
## Summary

Reverts #837 (`fix(deepagents): omit middleware trace inputs`).

That change adopted `tracePolicy` / `omitPayload`, which only exist in
prerelease langchain, so it moved the workspace onto rc dependency
ranges — `langchain` `^1.5.10` to `^1.5.11-rc.0`, `@langchain/langgraph`
`^1.4.10` to `^1.4.15-rc.0`, plus a new `langchain: ^1.5.11-rc.0` peer
on `@langchain/quickjs`. A stable release cannot depend on rc packages,
which is what blocks the release.

Reverting restores the stable ranges and removes the `tracePolicy`
usages, the `trace_policy.test.ts` suite, and the accompanying
changeset.

Pairs with #845, which exits rc prerelease mode. Both are needed before
a stable release can go out.

One manual conflict resolution, in
`libs/providers/quickjs/package.json`: #838 bumped the `deepagents` peer
to `>=1.13.4-rc.0` after #837 landed, so the peer block conflicted. I
kept #838's bump and dropped only the `langchain` peer that #837 added —
reverting #837 should not undo the release versioning done by #838.

## Testing

- `pnpm install --frozen-lockfile` is clean, and `langchain` resolves
back to `1.5.10`.
- `pnpm build` passes.
- `pnpm test` passes: 43 files in `libs/deepagents` and 4 in `libs/acp`,
no type errors, lint and format clean. The deepagents file count drops
by one because the revert removes `trace_policy.test.ts`.
- With this and #845 applied together, `changeset status` reports a
clean patch bump across all packages, so the pair leaves the release
plan in a valid state.
Baihao Wang (byhow) added a commit that referenced this pull request Sep 9, 2026
## Summary

Runs `changeset pre exit` so the next release publishes stable versions
rather than `-rc` ones. `.changeset/pre.json` moves to `"mode": "exit"`;
the release workflow deletes the file when it next versions, and the
open release PR (#843) will regenerate as a stable `chore: version
packages`.

Pairs with #844, which reverts #837. #837 is what pulled rc dependency
ranges into the tree, so both are needed before a stable release can go
out.

## Testing

No code change — `.changeset/pre.json` only.
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