fix(types): resolve pre-existing type errors on release/2.0

[alfredo1996]

Summary

• Fix implicit any parameter types in plugin-hardening.test.ts (4 occurrences of (d) => d changed to (d: unknown) => d)
• Add missing chart component stubs (GanttChart, CirclePackingChart, ChoroplethChart) to the @neoboard/components mock in chart-helpers.test.ts, which caused next/dynamic import resolution errors

These are pre-existing type errors on the release/2.0 branch that block CI for all PRs targeting it (#627, #628, #629, #630).

Test plan

• npx tsc --noEmit -p app/tsconfig.json passes with zero errors
• npm -w app run test -- --run passes (167 files, 2236 tests, 0 failures)

🤖 Generated with Claude Code

Summary by CodeRabbit

• Tests
  • Enhanced test infrastructure by adding mocked exports for additional chart component types in plugin test modules, expanding component coverage in validation testing scenarios.
  • Strengthened chart plugin test fixtures by applying explicit and consistent input parameter type annotations to transform callbacks across all validation and warning test cases.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 87897b52-5510-4845-815d-57d0d242f22b

📥 Commits

Reviewing files that changed from the base of the PR and between 2b5ca1f and d21eede.

📒 Files selected for processing (2)

• app/src/lib/__tests__/plugin/chart-helpers.test.ts
• app/src/plugins/__tests__/plugin-hardening.test.ts

---

Walkthrough

Test files updated to expand mocked chart component exports and refine transform callback type annotations in plugin test fixtures. Both changes are test-only with no impact on production code or exported APIs.

Changes

Cohort / File(s)	Summary
Plugin test fixtures app/src/lib/__tests__/plugin/chart-helpers.test.ts	Added stubbed exports (GanttChart, CirclePackingChart, ChoroplethChart) to mocked @neoboard/components module for test module resolution.
Plugin hardening tests app/src/plugins/__tests__/plugin-hardening.test.ts	Updated defineChartPlugin transform callbacks in multiple test fixtures to explicitly type input parameter as unknown.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• fix: harden plugin infrastructure — safety and validation #615: Updates to the same test files and plugin/transform codepaths with hardening and validation changes.
• refactor(plugins): plugin system overhaul — phases 1-4 + 6 integrated #428: Plugin-system refactor introducing chart-helpers and stricter plugin typing.
• feat(plugins): external chart plugin loading via manifest (#423) #589: Tests mocking @neoboard/components during external-plugin bootstrap.

Suggested labels

pkg:app, area:charts

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main changes: fixing type errors in test files on the release/2.0 branch, which matches the modifications shown in the raw summary.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/release-2.0-type-errors

---

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

_{Review rate limit: 0/1 reviews remaining, refill in 60 minutes.}

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud