Reviewing files that changed from the base of the PR and between 10d58e9 and 820bb7a.

• src/polar/Pie.tsx (3 hunks)
• src/util/ActiveShapeUtils.tsx (2 hunks)
• storybook/stories/Examples/Pie/CustomActiveShapePieChart.stories.tsx (3 hunks)
• storybook/stories/Examples/Pie/PieWithCells.stories.tsx (2 hunks)
• test/polar/Pie.spec.tsx (2 hunks)

• GitHub Check: Build, Test, Pack
• GitHub Check: Analyze (javascript-typescript)

src/util/ActiveShapeUtils.tsx (2)

27-34: Index added to ShapeProps looks consistent and safe

Adding index?: string | number to ShapeProps matches how Shape is now used in Pie and other charts and stays backward‑compatible since it’s optional and intersected with ExtraProps. No issues here.

---

90-95: Passing props.index into function shape is the right extension

Calling function options as option(props, props.index) correctly exposes the index to shape/activeShape callbacks while remaining non‑breaking for existing single‑argument handlers that will simply ignore the second parameter.

src/polar/Pie.tsx (2)

159-160: PieSectorShapeProps/PieShape definitions align with Shape usage

Defining PieSectorShapeProps as PieSectorDataItem & { isActive: boolean; index: number } and PieShape as a two‑argument function matches what PieSectors passes into Shape and what Shape forwards to user callbacks; this gives both props.index and the numeric index parameter reliably.

---

545-573: Index forwarding into Shape is correct and matches the new API

Passing index={i} into <Shape option={shape ?? sectorOptions} ...> ensures both shape and activeShape/inactiveShape functions can receive a numeric index via the second argument and via props.index, which matches the updated tests and story usage.

storybook/stories/Examples/Pie/CustomActiveShapePieChart.stories.tsx (1)

22-80: Story correctly demonstrates new Pie.shape behavior

Using renderActiveShape as (props: PieSectorDataItem & { isActive: boolean }) => ReactElement and wiring it via shape={renderActiveShape} matches the new Pie shape API; the isActive-based branch plus the plain <Sector {...props} /> fallback is a good, minimal example.

Also applies to: 85-85

storybook/stories/Examples/Pie/PieWithCells.stories.tsx (1)

3-3: Interactive highlighting via shape and click index looks solid

Tracking activeIndices driven by onClick={(_, index) => { ... }} and using shape={(props, index) => ...} to conditionally render <Sector> with a different fill is a clean demonstration of the new numeric index being passed through; the toggle logic on activeIndices is straightforward and correct.

Also applies to: 21-42

test/polar/Pie.spec.tsx (1)

335-412: Expectation on activeShape call correctly asserts index propagation

Updating expect(activeShape).toHaveBeenCalledWith(...) to include the index: 0 field in the props object and the explicit second argument 0 matches the new behavior where Shape forwards both props.index and the numeric index parameter to activeShape. This provides good coverage that the bug fix (missing index) is actually wired through.