feat(ui): inventory related-node chips in one module (v0.76.0) - #230
Conversation
Related-node walk chips share RelatedNodeChip and a Storybook inventory. After seed, Ada West still names Demo Corp; Priya Nair stays side-only so two orgs never invent a primary. Click a chip to continue the walk or open that post. Stacked on #92 @ 9bb5829. Do not mix into #74. Independent exact-head APPROVE required.
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
Exact head Buyer: Ada West, Demo Corp (Our side); Priya Nair stays side-only. Click a chip to continue the walk or open that post. No commits added to #92. Independent exact-head APPROVE required. This comment is not an approval. Do not mix into #74. Do not self-approve. Owner/self COMMENT is not APPROVE. |
|
Warning Review limit reached
Next review available in: 9 minutes Limit details: You’ve used all 1 included review currently available under your plan. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (14)
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. Comment |
|
@coderabbitai review |
|
6db8b72
into
feat/related-node-person-side-labels-main
Buyer impact
Stacked on #92 @
9bb5829. Related-node walk chips shareRelatedNodeChipand a Storybook inventory (ADR 0014). After seed, walking from Demo Corp still shows Ada West, Demo Corp (Our side). Priya Nair stays Priya Nair (Counterparty) — two orgs never invent a primary. Click a chip to continue the walk or open that post.relatedNodeCaption.ts(WCAG 2.5.3).RelatedNodeChip.stories.tsxinventories the four buyer states. Host with Storybook 10 when the later token stack lands; vitest locks the same behavior now.OurSidePersoncomponent is synchronized to Ada West, Demo Corp (Our side) so the design inventory and current feat(ui): show business context on related-node chips (v0.75.0) #92/Storybook contract no longer disagree.affiliation_ambiguousor "multiple organizations" next-action copy (feat(ui): mark plural affiliations on related-node chips (v0.76.0) #123 / feat(ui): name next action on plural affiliation chips (v0.77.0) #192).Exact head and repair
Current head:
dc8fb15fe2eb5eedfb31db57e671c4b1d6eb443confeat/related-node-storybook-chips-v0760.Base:
feat/related-node-person-side-labels-main(#92) @9bb5829f706d4eb98420e8ac4586e7ac4200223e.The original story objects widened
args.actionto plainstring. Becausetsconfig.app.jsonincludes all ofsrcand the existingRelatedNodeChip.test.tsxpasses those story args into the typedRelatedNodeChipActionprop,tsc -bhas a compile-time contract failure even though Vitest can transpile the files. Current head fixes the owning Storybook boundary by constraining story args withsatisfies RelatedNodeStoryArgs; it does not weaken the component action union or add Storybook runtime dependencies.No predecessor-head check or review evidence transfers to this head. The Figma synchronization is design-source alignment, not merge evidence.
Review gate
Not in this slice
Valkey outbox is #229. RankWeave landed as #220. TEPP is #214. naruon is #217. ThreadWeave is #219. Keyverse is #221. Orchestrator is #228. Plural next-action is #192. Strix ModelBehaviorError classifier is ContextualWisdomLab/.github#1091.