Skip to content

feat: streamline human review detail previews - #5573

Merged
steve8708 merged 10 commits into
mainfrom
steve8708/changes-100029
Sep 23, 2026
Merged

feat: streamline human review detail previews#5573
steve8708 merged 10 commits into
mainfrom
steve8708/changes-100029

Conversation

@steve8708

@steve8708 steve8708 commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Reduce Human Review detail-view density by moving written feedback and instruction drafts into focused popovers.
  • Add an opt-in output preview for text, charts, tables, images, and design payloads.
  • Keep the preview bounded and safe, with localized controls and parser coverage.

Validation

  • pnpm --filter @agent-native/core exec vitest run src/client/observability/OutputPreview.spec.ts
  • pnpm --filter @agent-native/core exec tsc --noEmit --pretty false
  • pnpm guards
  • Verified the compact table, sparse detail, preview, feedback popover, and instruction popover in the running Factory app.

steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
5605c25 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=2

expires_at=1790030525

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
cd776bd 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=3

expires_at=1790030565

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
185f31c 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=4

expires_at=1790030812

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
ae5d4a2 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=5

expires_at=1790030914

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
65ab928 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=6

expires_at=1790031112

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
d4047dc 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=7

expires_at=1790031299

legacy_version=absent

legacy_retired=true
builder-io-integration[bot]

This comment was marked as outdated.

Sign in to view

@github-actions

github-actions Bot commented Sep 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Here's a visual recap of what changed:

Visual recap

Open the full interactive recap

steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
15d45b9 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=8

expires_at=1790031535

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
a0d69dd 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=9

expires_at=1790031982

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
1a41a46 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=10

expires_at=1790032160

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
e66219b 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=11

expires_at=1790032344

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
e8218c7 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=12

expires_at=1790032520

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
03d7ff0 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=13

expires_at=1790032731

legacy_version=absent

legacy_retired=true
builder-io-integration[bot]

This comment was marked as outdated.

Sign in to view

steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
be4de9a 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=14

expires_at=1790032941

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
9ee6995 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=15

expires_at=1790033313

legacy_version=absent

legacy_retired=true
builder-io-integration[bot]

This comment was marked as outdated.

Sign in to view

steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
241d373 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=16

expires_at=1790033541

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
ed7d080 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=17

expires_at=1790033961

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
3abeb70 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=18

expires_at=1790034170

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
23dbafb 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=19

expires_at=1790034380

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
d7c3c3b 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=20

expires_at=1790034592

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
17db242 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=21

expires_at=1790034800

legacy_version=absent

legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 21, 2026
@steve8708
babysit lease: PR #5573
4eb7548 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=22

expires_at=1790035032

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
995c45d 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=44

expires_at=1790039666

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
7e1cdbf 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=45

expires_at=1790039874

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
7c29f0d 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=46

expires_at=1790040460

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
eb28c61 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=47

expires_at=1790040665

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
ac8a0e7 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=48

expires_at=1790040938

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
2d783c0 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=49

expires_at=1790041176

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
a75492d 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=50

expires_at=1790041385

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
33f2b6a 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=51

expires_at=1790041673

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
2d4be53 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=52

expires_at=1790041895

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
50a589f 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=53

expires_at=1790042072

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
3c4ae4d 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=54

expires_at=1790042282

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
7cd3c31 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=55

expires_at=1790042494

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
f4ad327 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=56

expires_at=1790042704

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
a453340 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=57

expires_at=1790042914

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
0b1c9c3 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=58

expires_at=1790043097

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
852794f 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=59

expires_at=1790043275

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
65cf295 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=60

expires_at=1790043486

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
e263076 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=61

expires_at=1790043723

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
a14e0f2 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=62

expires_at=1790043933

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
7debd1b 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=63

expires_at=1790044113

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
f033ec3 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=64

expires_at=1790044322

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
205d660 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=65

expires_at=1790044535

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
5827c56 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=66

expires_at=1790044714

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
7ceaba5 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=67

expires_at=1790044892

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
ad25478 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=68

expires_at=1790045073

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
9679eaa 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=69

expires_at=1790045282

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
c8882a3 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=70

expires_at=1790045493

legacy_version=absent
legacy_retired=true
steve8708 added a commit that referenced this pull request Sep 22, 2026
@steve8708
babysit lease: PR #5573
59a272f 
owner_thread=01a0b1bc-8b00-7842-8738-96778a88ede4

lease_version=71

expires_at=1790045702

legacy_version=absent
legacy_retired=true
@steve8708
chore: publish branch work in scripts/check-function-size-baseline.mj…
0f6d63d 
…s, scripts/check-function-size-baseline.test.ts (2 files)
@steve8708
Merge remote-tracking branch 'origin/main' into steve8708/changes-100029
d69779d 
# Conflicts:
#	scripts/check-function-size-baseline.mjs
#	scripts/check-function-size-baseline.test.ts

This branch was successfully deployed

No deployments
pr-5573-content — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
pr-5573-slides — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
pr-5573-fw — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
pr-5573-plan — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
pr-5573-design — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
pr-5573-calendar — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
pr-5573-mail — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
pr-5573-forms — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
pr-5573-analytics — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
pr-5573-clips — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
pr-5573-starter — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
pr-5573-dispatch — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
pr-5573-assets — d69779dc Deployed Sep 23, 2026 by github-actions[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@builder-io-integration builder-io-integration[bot] builder-io-integration[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@steve8708