Description
Multiple TypeScript type errors exist in the apps/web/charts/analyze directory. These errors prevent clean builds and indicate potential runtime issues with ECharts configurations.
Affected Files
apps/web/charts/analyze/org/activity-open-to-close/visualization.ts
apps/web/charts/analyze/org/activity-open-to-first-response/visualization.ts
apps/web/charts/analyze/org/engagement-scatter/visualization.ts
apps/web/charts/analyze/org/pull-requests-open-to-review/visualization.ts
apps/web/charts/analyze/org/recent-pr-review-stats/visualization.ts
apps/web/charts/analyze/org/recent-stats/visualization.ts
Errors
1. splitNumber property does not exist on YAxisOption
- Files: activity-open-to-close, activity-open-to-first-response, pull-requests-open-to-review
- Error:
Object literal may only specify known properties, and 'splitNumber' does not exist in type...
- Location: Y-axis configuration (lines 105-106)
- Fix: Remove
splitNumber property or update ECharts type definitions
2. Implicit any type for formatter parameters
- Files: activity-open-to-close, engagement-scatter, pull-requests-open-to-review
- Error:
Parameter 'params' implicitly has an 'any' type
- Location: Tooltip/formatter functions (lines 156, 179)
- Fix: Add explicit type annotation:
params: TooltipFormatterParams
3. data property does not exist on TopLevelFormatterParams
- File: engagement-scatter/visualization.ts
- Error:
Property 'data' does not exist on type 'TopLevelFormatterParams'
- Location: Line 144
- Fix: Use correct property access or type guard
4. decal type mismatch in AriaOption
- Files: recent-pr-review-stats, recent-stats
- Error:
Type 'false | { enabled: true; decal: { show: true; }; }' is not assignable to type 'AriaOption | undefined'
- Location: Lines 73, 158
- Fix: Update aria configuration to match ECharts AriaOption type
Impact
- TypeScript builds fail with errors
- Potential runtime issues if ECharts receives invalid configuration
- Developer experience degraded by type errors
Suggested Fix
- Review ECharts type definitions and update configurations accordingly
- Add explicit type annotations for formatter parameters
- Validate aria/decal configurations against current ECharts API
Description
Multiple TypeScript type errors exist in the
apps/web/charts/analyzedirectory. These errors prevent clean builds and indicate potential runtime issues with ECharts configurations.Affected Files
apps/web/charts/analyze/org/activity-open-to-close/visualization.tsapps/web/charts/analyze/org/activity-open-to-first-response/visualization.tsapps/web/charts/analyze/org/engagement-scatter/visualization.tsapps/web/charts/analyze/org/pull-requests-open-to-review/visualization.tsapps/web/charts/analyze/org/recent-pr-review-stats/visualization.tsapps/web/charts/analyze/org/recent-stats/visualization.tsErrors
1.
splitNumberproperty does not exist on YAxisOptionObject literal may only specify known properties, and 'splitNumber' does not exist in type...splitNumberproperty or update ECharts type definitions2. Implicit
anytype for formatter parametersParameter 'params' implicitly has an 'any' typeparams: TooltipFormatterParams3.
dataproperty does not exist on TopLevelFormatterParamsProperty 'data' does not exist on type 'TopLevelFormatterParams'4.
decaltype mismatch in AriaOptionType 'false | { enabled: true; decal: { show: true; }; }' is not assignable to type 'AriaOption | undefined'Impact
Suggested Fix