Skip to content

fix(design-data): decompose stack-item background-highlight fusion (284) - #1317

Merged
GarthDB merged 1 commit into
mainfrom
fix/decompose-stack-item-background-highlight-284
Jul 30, 2026
Merged

fix(design-data): decompose stack-item background-highlight fusion (284)#1317
GarthDB merged 1 commit into
mainfrom
fix/decompose-stack-item-background-highlight-284

Conversation

@GarthDB

@GarthDB GarthDB commented Jul 30, 2026

Copy link
Copy Markdown
Member

Description

Closes out epic spectrum-design-data-284 (fused name.property decomposition). This was
the last remaining fused property in the whole corpus, found via a re-run of the
tools/token-mapping-analyzer residue analyzer after #1315 merged.

  • packages/design-data/tokens/color-aliases.tokens.json: split the
    stack-item-selected-background-color-highlight token's fused stack-item-background-color
    property into object:"background" + property:"color". No component field was added —
    the legacy reference data (packages/tokens/src) for this cross-component alias never
    tagged it with one (unlike sibling stack-item-background-color-* tokens in
    color-component.tokens.json, which do carry component:"stack-item" because their
    legacy reference does too). Adding component here caused a real legacy-verify
    mismatch, so stack-item stays only in the pinned legacyKey.

Related Issue

Closes spectrum-design-data-284.

Motivation and Context

node tools/token-mapping-analyzer/src/index.js now reports Fused properties (active): 0
— every name.property in the corpus is atomic.

How Has This Been Tested?

  • cargo run -p design-data-cli -- migrate legacy-verify — legacy output OK, byte-identical.
  • cargo run -p design-data-cli -- migrate roundtrip-verify — roundtrip OK.
  • moon run design-data:validate sdk:test sdk:lint — all green.
  • moon run :test (JS/AVA) — all green except the pre-existing sdk-wasm:test parity
    failure (Dataset.embedded() query result .raw is a plain object, not a Map), already
    confirmed unrelated (fails identically on main).
  • Re-ran the token-mapping-analyzer residue report to confirm zero fused properties remain.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Last remaining fused name.property in the corpus: the stack-item
selected/highlight background-color alias fused an object segment
(background) into its property string.

- packages/design-data/tokens/color-aliases.tokens.json: split
  stack-item-background-color into object:"background" +
  property:"color". No component field added -- the legacy reference
  data for this cross-component alias never tagged it with one, so
  legacyKey is the only place "stack-item" is preserved.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented Jul 30, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: c8066f6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 10 packages
Name Type
@adobe/spectrum-design-data Patch
@adobe/spectrum-component-api-schemas Patch
@adobe/design-system-registry Patch
@adobe/design-data-agent-mcp Patch
@adobe/design-data-mcp Patch
markdown-generator Patch
@adobe/token-corpus-migrate Patch
@adobe/token-naming-audit Patch
component-options-editor Patch
@adobe/spectrum-design-data-mcp Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions

Copy link
Copy Markdown
Contributor

🎨 Token Changes Report

Tokens Changed (0)

Original Branch: main


This comment was automatically generated by the token diff tool. 🤖

@github-actions

Copy link
Copy Markdown
Contributor

🧩 Component Schema Changes Report

No component schema changes detected.

This comment was automatically generated by the component schema diff tool. 🤖

@github-actions

Copy link
Copy Markdown
Contributor

Run report for c8066f6e

Total time: 2m 34s | Comparison time: 3m 20s | Estimated savings: 46.6s (23.2% faster)

Action Time Status Info
⬛️ SetupProto(0.57.4) 0ms Skipped
🟩 SyncWorkspace 10.4ms Passed
🟩 SyncProject(sdk) 0.4ms Passed
🟩 SyncProject(design-data) 1ms Passed
⬛️ SetupToolchain(node:20.17.0) 1.1s Skipped
⬛️ SetupToolchain(rust:1.88.0) 1.8s Skipped
⬛️ SetupToolchain(pnpm:10.17.1) 696.9ms Skipped
⬛️ SetupToolchain(javascript) 0ms Skipped
🟩 SetupEnvironment(javascript) 0.8ms Passed
⬛️ SetupEnvironment(javascript, sdk) 0.9ms Skipped
⬛️ InstallDependencies(javascript) 0.1ms Skipped
🟩 SetupEnvironment(rust, sdk) 156.9ms Passed
⬛️ InstallDependencies(rust, sdk) 2.5ms Skipped
🟩 RunTask(design-data:validate-registry) 223.2ms Passed
🟩 InstallDependencies(javascript, sdk) 6s Passed
🟩 RunTask(sdk:codegen-check) 68.8ms Passed
🟩 RunTask(sdk:build) 1m 6s Passed
🟩 RunTask(design-data:legacy-output) 1.1s Passed
🟩 RunTask(design-data:roundtrip-verify) 1.5s Passed
🟩 RunTask(design-data:validate-dataset) 6.4s Passed
And 2 more...
Expanded report
Action Time Status Info
🟩 RunTask(design-data:validate) 2.5s Passed
🟩 RunTask(sdk:test) 2m 26s Passed SLOW
Changed files
.changeset/decompose-stack-item-background-highlight.md
packages/design-data/tokens/color-aliases.tokens.json

@GarthDB
GarthDB merged commit f7f8bef into main Jul 30, 2026
10 checks passed
@GarthDB
GarthDB deleted the fix/decompose-stack-item-background-highlight-284 branch July 30, 2026 04:39
@GarthDB GarthDB mentioned this pull request Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant