Skip to content

fix(sessions): render string tool inputs as multiline content - #13

Open
KinomotoMio wants to merge 1 commit into
tommy0103:mainfrom
KinomotoMio:fix/apply-patch-input-rendering
Open

fix(sessions): render string tool inputs as multiline content#13
KinomotoMio wants to merge 1 commit into
tommy0103:mainfrom
KinomotoMio:fix/apply-patch-input-rendering

Conversation

@KinomotoMio

Copy link
Copy Markdown
Collaborator

Summary

  • render top-level string tool inputs with the existing multiline content viewer
  • keep object-valued generic tool inputs on the existing field-grid path
  • add regression coverage for Codex apply_patch, HTML escaping, and object fallback behavior

Scope

This is intentionally limited to the Sessions presentation layer. Codex parsing, persistence, raw input display, and renderer styles are unchanged.

Verification

  • node --experimental-test-module-mocks --test tests/app-tool-renderer.test.mjs (11 passed)
  • npm test (277 passed)
  • npm run typecheck
  • npm run lint (0 errors; 4 pre-existing warnings)
  • cd app && npx electron-vite build
  • real captured apply_patch input: 6,878 characters / 412 lines rendered as 1 content block and 0 character field rows

Closes #11

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.

[Sessions] apply_patch input renders as one row per character

1 participant