Skip to content

chore: deprecate visualization packages#567

Merged
cb-ekuersch merged 15 commits intocds-v9from
deprecated-viz
Apr 1, 2026
Merged

chore: deprecate visualization packages#567
cb-ekuersch merged 15 commits intocds-v9from
deprecated-viz

Conversation

@cb-ekuersch
Copy link
Copy Markdown
Contributor

@cb-ekuersch cb-ekuersch commented Mar 31, 2026

What changed? Why?

In an effort to simplify the CDS landscape, v9 will consolidate the visualization packages with the core web/mobile packages. This will reduce setup complexity and make package version numbers easier to track as the visualization packages maintain their own versions separate from the rest of CDS.

Closes:

Root cause (required for bugfixes)

UI changes

iOS Old iOS New
old screenshot new screenshot
Android Old Android New
old screenshot new screenshot
Web Old Web New
old screenshot new screenshot

Testing

How has it been tested?

  • Unit tests
  • Interaction tests
  • Pseudo State tests
  • Manual - Web
  • Manual - Android (Emulator / Device)
  • Manual - iOS (Emulator / Device)

Testing instructions

Illustrations/Icons Checklist

Required if this PR changes files under packages/illustrations/** or packages/icons/**

  • verified visreg changes with Terran (include link to visreg run/approval)
  • all illustration/icons names have been reviewed by Dom and/or Terran

Change management

type=routine
risk=low
impact=sev5

automerge=false

@cb-ekuersch cb-ekuersch marked this pull request as draft March 31, 2026 01:00
cb-ekuersch and others added 14 commits March 31, 2026 12:35
- Change @deprecationExpectedRemoval v9 → v10 in visualization bridge shims
- Remove "warning" field from all 30 chart/sparkline metadata JSON files
  (import path already communicates the new location)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…cations

Point source links to packages/web/src/visualizations and
packages/mobile/src/visualizations instead of the deprecated
visualization packages.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sparkline components are deprecated in favor of LineChart.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coinbase/cds-web/visualizations/chart and
@coinbase/cds-mobile/visualizations/chart are subpaths, not packages.
Imports from these paths are already covered by the @coinbase/cds-web
and @coinbase/cds-mobile package entries.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
These Code Connect files should not have been included when copying
sparkline source into cds-web and cds-mobile.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@deprecated annotations on export * re-exports don't surface to
consumers, so they have no effect and can be removed.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Restores the .figma.tsx files that were in web-visualization and
mobile-visualization, placing them in their new home under
packages/web and packages/mobile, with import strings updated to
the new subpath locations.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Remove web-visualization and mobile-visualization entryPoints from docgen.config.js
- Prefix chart and sparkline sourceFiles with visualizations/ to resolve from cds-web/cds-mobile
- Update all :docgen/ import paths in chart/sparkline MDX files from web-visualization/* and mobile-visualization/* to web/visualizations/* and mobile/visualizations/*
- Update live component imports in example MDX files to new subpath locations

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Remove web-visualization and mobile-visualization from tsconfig references
  in docs, storybook, test-expo, and vite-app
- Remove visualization packages from storybook project.json inputs
- Remove web-visualization from storybook visreg shouldRun check
- Remove visualization packages from eslint.config.mjs react19 rules
- Update figma.config.web/mobile.json to only include web/mobile src
- Simplify prepareRoutes.ts glob to mobile only, clean up comments
- Remove visualization package references from eslint-plugin-internal tests,
  update test fixture to use new subpath import and file path
- Update templates to use new subpath imports and remove visualization deps
- Sort dependencies in packages/mobile and packages/web package.json

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@cb-ekuersch cb-ekuersch marked this pull request as ready for review April 1, 2026 15:22
@cb-ekuersch cb-ekuersch requested a review from hcopp April 1, 2026 15:24
@cb-ekuersch cb-ekuersch merged commit 5b4d737 into cds-v9 Apr 1, 2026
27 checks passed
@cb-ekuersch cb-ekuersch deleted the deprecated-viz branch April 1, 2026 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants