Skip to content

feat(logs): add copy-all button to request detail modal [defer to 3.8.51] - #11083

Merged
diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.51from
AndrianBalanescu:feat/request-detail-copy-all
Aug 26, 2026
Merged

diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.51from
AndrianBalanescu:feat/request-detail-copy-all

Conversation

@AndrianBalanescu

Copy link
Copy Markdown
Contributor

Summary

  • Adds a Copy all button to the request detail modal header so operators can copy the entire multi-stage transcript (client/provider payloads, stream chunks, or legacy request/response) in one click instead of copying each accordion card manually.
  • Extracts buildCopyAllText() as a pure, exported helper in src/shared/components/RequestLoggerDetail.tsx.
  • Adds copyAll and copiedAll translation strings across all 43 supported locales.
  • Adds comprehensive unit tests for buildCopyAllText() in tests/unit/request-logger-detail-copy-all.test.ts.

Testing

  • node --import tsx/esm --test tests/unit/request-logger-detail-copy-all.test.ts passes (4/4).
  • ESLint checks clean for changed files.

@diegosouzapw diegosouzapw changed the title feat(logs): add copy-all button to request detail modal feat(logs): add copy-all button to request detail modal [defer to 3.8.51] Aug 22, 2026
@diegosouzapw diegosouzapw mentioned this pull request Aug 23, 2026
@diegosouzapw
diegosouzapw changed the base branch from release/v3.8.50 to release/v3.8.51 August 24, 2026 01:34
@diegosouzapw

Copy link
Copy Markdown
Owner

Solid little UX win — extracting buildCopyAllText as a pure exported helper was the right call; we ran your 4 tests on the current release/v3.8.51 tip and they pass, and the copy path reuses the modal's existing onCopy so clipboard fallback/XSS concerns are inherited-solved. Before merge: please add a changelog.d/features entry like the other PRs in this cycle — the Merge integrity job is red on the PR and the missing entry is the likely owner. Also note the 43 locale entries carry English 'Copy all'/'Copied all' placeholders; keys are present so the coverage gate is happy, but a translation pass (or the usual sync flow) would be welcome as a follow-up. The live modal click-through (clipboard write in a real browser) hasn't been exercised by us — if you have a screenshot/clip of it working, drop it in the thread.

@diegosouzapw
diegosouzapw merged commit 5984c43 into diegosouzapw:release/v3.8.51 Aug 26, 2026
9 of 16 checks passed
diegosouzapw added a commit that referenced this pull request Aug 26, 2026
…11636)

Merged via /merge-batch (correção de lacuna, 2026-08-26). check:file-size confirmado OK após o rebaseline.
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…#11083)

Merged via /merge-batch (lote 2026-08-26 batch 2, v3.8.51). Boarded no worktree combinado junto com outras ~20 PRs; validação única: typecheck/complexity/cognitive-complexity/changelog-integrity verdes, file-size rebaseado onde necessário (crescimento legítimo), lint com os mesmos 228 achados pré-existentes confirmados via sonda contra o tip puro (não introduzidos por este lote), e 292 testes focados (unit) + 18 (vitest) passando. Obrigado pela contribuição.
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
…apw#11631/diegosouzapw#11628 growth (diegosouzapw#11636)

Merged via /merge-batch (correção de lacuna, 2026-08-26). check:file-size confirmado OK após o rebaseline.
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.

2 participants