Skip to content

Refactor: Share ExpertSelector across wizards#35

Merged
FL4TLiN3 merged 2 commits intomainfrom
refactor/share-expert-selector-26
Dec 4, 2025
Merged

Refactor: Share ExpertSelector across wizards#35
FL4TLiN3 merged 2 commits intomainfrom
refactor/share-expert-selector-26

Conversation

@FL4TLiN3
Copy link
Contributor

@FL4TLiN3 FL4TLiN3 commented Dec 4, 2025

Summary

  • Extract shared WizardExpertSelector component from duplicated implementations
  • Add ExpertChoice type to centralized types
  • Replace inline ExpertSelector implementations in all wizard apps (Publish, Status, Tag, Unpublish)

Closes #26

Test plan

  • Verify pnpm typecheck passes
  • Verify pnpm format-and-lint passes
  • Verify pnpm build passes
  • Verify E2E tests for wizard functionality

Note

Introduce shared WizardExpertSelector and replace duplicated expert selection UIs in Publish, Status, Tag, and Unpublish apps.

  • Components:
    • Add WizardExpertSelector in packages/tui/src/components/wizard-expert-selector.tsx and export via packages/tui/src/components/index.ts.
  • Apps:
    • Replace inline expert selectors with WizardExpertSelector in:
      • packages/tui/apps/publish/app.tsx (adds handleSelect to exit after selection).
      • packages/tui/apps/status/app.tsx (uses shared selector for expert selection step).
      • packages/tui/apps/tag/app.tsx (uses shared selector for expert selection step).
      • packages/tui/apps/unpublish/app.tsx (uses shared selector for expert selection step).

Written by Cursor Bugbot for commit f591e9f. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Dec 4, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
perstack Ready Ready Preview Comment Dec 4, 2025 5:15am

@codecov
Copy link

codecov bot commented Dec 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@FL4TLiN3 FL4TLiN3 force-pushed the refactor/share-expert-selector-26 branch from ea07b7c to f591e9f Compare December 4, 2025 05:14
@FL4TLiN3 FL4TLiN3 merged commit 7004faf into main Dec 4, 2025
8 of 9 checks passed
@FL4TLiN3 FL4TLiN3 deleted the refactor/share-expert-selector-26 branch December 8, 2025 19: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.

Refactor: Share ExpertSelector across wizards

1 participant