[feat][UX/UI] Hide mapping and preview until a test set is selected

[NamHT4Devlop]

Summary

Closes #4616

Steps 2–4 of the Add to test set drawer (DataPreviewEditor, MappingSection, PreviewSection) are now hidden until the user either:

• selects an existing test set revision (selectedRevisionId is truthy), or
• switches to creating a new test set (isNewTestset === true).

Before this change all three sections rendered immediately on drawer open, showing mapping controls that had no effect until a test set was chosen — which was confusing.

Changes

web/oss/src/components/SharedDrawers/AddToTestsetDrawer/TestsetDrawer.tsx

Wrapped the three lower sections in a single conditional:

{(!!drawer.selectedRevisionId || drawer.isNewTestset) && (
    <>
        <DataPreviewEditor ... />
        <MappingSection ... />
        <PreviewSection ... />
    </>
)}

Demo

Before — Steps 2–4 (data preview, column mapping, row preview) are always visible even with no test set selected, showing empty/unusable controls:

After — Only Step 1 (test set selector) is shown initially. Steps 2–4 appear only once a test set revision is selected or a new test set is being created:

How to test

1. Open the Add to test set drawer from a trace span.
2. Confirm only the TestsetSelector (step 1) is visible initially.
3. Select an existing test set — steps 2–4 should appear.
4. Clear the selection — steps 2–4 should hide again.
5. Choose + New test set — steps 2–4 should appear.

Type checks

pnpm run types:check in web/oss/ — no new errors introduced (all pre-existing).

[vercel]

@NamHT4Devlop is attempting to deploy a commit to the agenta projects Team on Vercel.

A member of the Team first needs to authorize it.

[github-actions]

✅ Thanks @NamHT4Devlop! This PR now meets the contribution requirements and has been reopened. A maintainer will review it soon.

[CLAassistant]

All committers have signed the CLA.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 4b9212aa-3fbe-43d1-aaca-81226bfb4ea2

📥 Commits

Reviewing files that changed from the base of the PR and between 399602a and 0077874.

📒 Files selected for processing (1)

• web/oss/src/components/SharedDrawers/AddToTestsetDrawer/TestsetDrawer.tsx

Disabled knowledge base sources:

• Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

---

📝 Walkthrough

Summary by CodeRabbit

• Bug Fixes
  • Improved testset drawer functionality to conditionally display content sections including data preview, mapping, and preview areas. These sections now appear only after a test revision is selected or when creating a new testset, preventing incomplete information from displaying prematurely.

Walkthrough

TestsetDrawer now wraps DataPreviewEditor, MappingSection, and PreviewSection in a conditional that only renders them when drawer.selectedRevisionId is set or drawer.isNewTestset is true, hiding steps 2–4 until a testset is selected or created.

Changes

Conditional rendering of drawer content steps

Layer / File(s)	Summary
Gate content steps on testset/revision selection web/oss/src/components/SharedDrawers/AddToTestsetDrawer/TestsetDrawer.tsx	DataPreviewEditor, MappingSection, and PreviewSection are now wrapped in a condition checking drawer.selectedRevisionId || drawer.isNewTestset, preventing them from rendering before a testset is selected or a new one is being created.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}