[PR #493] [CLOSED] wrong pull request, please ignore it, sorry. #3342

Closed
opened 2026-07-15 02:02:44 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/mountain-loop/yaak/pull/493
Author: @baofeidyz
Created: 7/3/2026
Status: Closed

Base: mainHead: main


📝 Commits (10+)

  • f6d78b9 Add SSE response summary helpers
  • ce1dacb Merge branch 'mountain-loop:main' into main
  • 0fc4da7 Fix SSE summary custom input and copy
  • 884e3b8 Merge branch 'mountain-loop:main' into main
  • dd2d7bb Merge remote-tracking branch 'origin/main' into pr-466-sse-summary
  • 0c5f1f0 Improve SSE extracted text controls
  • 59685d2 Merge branch 'main' into main
  • 97bcb0b Merge branch 'main' into main
  • 21b00e4 Simplify SSE JSON candidate detection
  • a074340 Clamp split layout ratios before render

📊 Changes

13 files changed (+794 additions, -108 deletions)

View changed files

📝 apps/yaak-client/components/core/Editor/Editor.tsx (+18 -4)
📝 apps/yaak-client/components/core/EventViewer.tsx (+73 -40)
📝 apps/yaak-client/components/responseViewers/EventStreamViewer.tsx (+317 -45)
📝 apps/yaak-client/hooks/useResponseBodyEventSource.ts (+6 -1)
apps/yaak-client/hooks/useResponseBodySseSummary.ts (+18 -0)
apps/yaak-client/hooks/useSseSummaryResultKeyPath.ts (+98 -0)
📝 apps/yaak-client/lib/responseBody.ts (+32 -4)
📝 crates/yaak-sse/index.ts (+1 -0)
📝 crates/yaak-sse/package.json (+7 -1)
crates/yaak-sse/summary.test.ts (+51 -0)
crates/yaak-sse/summary.ts (+131 -0)
📝 package-lock.json (+4 -1)
📝 packages/ui/src/components/SplitLayout.tsx (+38 -12)

📄 Description

Summary

Submission

  • This PR is a bug fix.
  • If this PR is not a bug fix, I linked the feedback item where @gschier explicitly gave me permission to work on it.
  • I have read and followed CONTRIBUTING.md.
  • I tested this change locally.
  • I added or updated tests when reasonable.
  • I added screenshots or recordings for UI changes when reasonable.

Explicit permission feedback item (required if not a bug fix):


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/mountain-loop/yaak/pull/493 **Author:** [@baofeidyz](https://github.com/baofeidyz) **Created:** 7/3/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (10+) - [`f6d78b9`](https://github.com/mountain-loop/yaak/commit/f6d78b971fec954039057f5cacac40359b4d7be9) Add SSE response summary helpers - [`ce1dacb`](https://github.com/mountain-loop/yaak/commit/ce1dacb4ec5f1d5f8866b66d6ae3caa4ca7f3cef) Merge branch 'mountain-loop:main' into main - [`0fc4da7`](https://github.com/mountain-loop/yaak/commit/0fc4da7bc4044ce71b30aeb294a22d1cfc02acf6) Fix SSE summary custom input and copy - [`884e3b8`](https://github.com/mountain-loop/yaak/commit/884e3b85ef4e75a55db67dddfcf62805713849bf) Merge branch 'mountain-loop:main' into main - [`dd2d7bb`](https://github.com/mountain-loop/yaak/commit/dd2d7bbdff7159d07dd56e760db6ad43ffd51315) Merge remote-tracking branch 'origin/main' into pr-466-sse-summary - [`0c5f1f0`](https://github.com/mountain-loop/yaak/commit/0c5f1f0cbe4d3e28d4819d14943b64d0ab200c3b) Improve SSE extracted text controls - [`59685d2`](https://github.com/mountain-loop/yaak/commit/59685d2b8d3073cd392a56008e84b4a924f645ae) Merge branch 'main' into main - [`97bcb0b`](https://github.com/mountain-loop/yaak/commit/97bcb0b73e6942d1b711b0932de92cad53368b16) Merge branch 'main' into main - [`21b00e4`](https://github.com/mountain-loop/yaak/commit/21b00e453db538f3c8ecb664392199cdee135e88) Simplify SSE JSON candidate detection - [`a074340`](https://github.com/mountain-loop/yaak/commit/a074340cdb1e8e366d5e3e140852f39a600737bc) Clamp split layout ratios before render ### 📊 Changes **13 files changed** (+794 additions, -108 deletions) <details> <summary>View changed files</summary> 📝 `apps/yaak-client/components/core/Editor/Editor.tsx` (+18 -4) 📝 `apps/yaak-client/components/core/EventViewer.tsx` (+73 -40) 📝 `apps/yaak-client/components/responseViewers/EventStreamViewer.tsx` (+317 -45) 📝 `apps/yaak-client/hooks/useResponseBodyEventSource.ts` (+6 -1) ➕ `apps/yaak-client/hooks/useResponseBodySseSummary.ts` (+18 -0) ➕ `apps/yaak-client/hooks/useSseSummaryResultKeyPath.ts` (+98 -0) 📝 `apps/yaak-client/lib/responseBody.ts` (+32 -4) 📝 `crates/yaak-sse/index.ts` (+1 -0) 📝 `crates/yaak-sse/package.json` (+7 -1) ➕ `crates/yaak-sse/summary.test.ts` (+51 -0) ➕ `crates/yaak-sse/summary.ts` (+131 -0) 📝 `package-lock.json` (+4 -1) 📝 `packages/ui/src/components/SplitLayout.tsx` (+38 -12) </details> ### 📄 Description ## Summary <!-- Describe the bug and the fix in 1-3 sentences. --> ## Submission - [ ] This PR is a bug fix. - [ ] If this PR is not a bug fix, I linked the feedback item where @gschier explicitly gave me permission to work on it. - [ ] I have read and followed [`CONTRIBUTING.md`](CONTRIBUTING.md). - [ ] I tested this change locally. - [ ] I added or updated tests when reasonable. - [ ] I added screenshots or recordings for UI changes when reasonable. Explicit permission feedback item (required if not a bug fix): <!-- https://yaak.app/feedback/... --> ## Related <!-- Link related issues, discussions, or feedback items. --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-07-15 02:02:44 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/yaak#3342