Skip to content

fix(context-pivot): restore visibility on session start - #745

Open
outlier27-cell wants to merge 1 commit into
openpi-dev:mainfrom
outlier27-cell:fix/issue-744-context-pivot-session-start
Open

outlier27-cell wants to merge 1 commit into
openpi-dev:mainfrom
outlier27-cell:fix/issue-744-context-pivot-session-start

Conversation

@outlier27-cell

Copy link
Copy Markdown
Contributor

Problem

context-pivot ignored the ExtensionContext supplied at session_start, so a resumed or reloaded session already above the 30k threshold hid the tool until the next agent turn.

Closes #744

Value

Context-pivot availability now immediately reflects the active Session's known context usage.

Approach

  • Pass the existing session-start context through the existing visibility helper.
  • Extend the threshold lifecycle test to cover a 30k-token session start.

Validation

  • node --test --experimental-strip-types tests/extensions/context-pivot/index.test.ts
  • bun run check
  • Exact-head GitHub CI is tracked for the final cross-platform result.

Impact

No tool semantics or threshold changes. This only removes the post-resume visibility delay.

@outlier27-cell

Copy link
Copy Markdown
Contributor Author

@tt-a1i 请审核此 PR。当前精确提交的 GitHub CI 已全部通过,修改范围与回归测试均已在 PR 说明中列明;现仅等待维护者审核。

This branch has not been deployed

No deployments
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.

bug(context-pivot): restore tool visibility at session start

1 participant