Skip to content

fix(core): cap session shell output - #45136

Open
Oxygen56 wants to merge 1 commit into
anomalyco:v2from
Oxygen56:session-shell-cap
Open

fix(core): cap session shell output#45136
Oxygen56 wants to merge 1 commit into
anomalyco:v2from
Oxygen56:session-shell-cap

Conversation

@Oxygen56

Copy link
Copy Markdown

Issue for this PR

Closes #45099

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Session shell commands now use the same 50 KiB preview limit as regular shell output. Large command output remains file-backed for paging, while the model-facing session history receives only the bounded preview.

How did you verify your code works?

  • A focused before/after check reproduces the old oversized capture and confirms the new model-facing shell output stays within 50 KiB.
  • TypeScript syntax and diff checks pass.
  • The package test and typecheck commands could not run locally because Bun is unavailable; pull-request CI should run them.

Screenshots / recordings

Not applicable.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@github-actions

Copy link
Copy Markdown
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions

Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

Potential duplicate found:

Recommendation: Check if PR #45109 already implements the 50 KiB capping logic or if these PRs address complementary aspects of the same issue (#45099).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant