Skip to content

chore(deps): vite 8 and the marko toolchain - #926

Open
LuLaValva wants to merge 5 commits into
llavalva-M451495HWX-evo-deps-a7929rfrom
llavalva-M451495HWX-evo-deps-a7929r-marko-vite
Open

chore(deps): vite 8 and the marko toolchain#926
LuLaValva wants to merge 5 commits into
llavalva-M451495HWX-evo-deps-a7929rfrom
llavalva-M451495HWX-evo-deps-a7929r-marko-vite

Conversation

@LuLaValva

Copy link
Copy Markdown
Member

Updates the interlocked vite/marko cluster: vite 8, @marko/vite 6, @marko/run 0.11, @storybook/marko-vite 5, vite-css-modules 1.16, marko 6.3.48, and @marko/type-check 3. The new type-checker actually checks the docs site, requiring: scss module imports switched to default imports, typing for $global.meta/$global.url, declarations for @ebay/skin and @docsearch/css side-effect imports, migrating the root tsconfig off the removed node10 resolution, and a snapshot refresh for marko's new compiled output. ebayui-core's nested marko 5 stays lock-pinned at 5.38.38 — marko 5.39's type changes break its type-check (filed in agent-feedback).

🤖 Generated with Claude Code

@changeset-bot

changeset-bot Bot commented Sep 3, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: e357d37

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@LuLaValva
LuLaValva force-pushed the llavalva-M451495HWX-evo-deps-a7929r-marko-vite branch 2 times, most recently from fdc69a6 to 1dbec5e Compare September 3, 2026 20:58
@LuLaValva

Copy link
Copy Markdown
Member Author

Merge-order note: this PR and #924 (typescript 6) are each green independently, but once both are merged, ebayui-core's tsconfig.json needs "ignoreDeprecations": "6.0" added (the new type-checker running on TS 6 hard-errors on its node10 moduleResolution). The option is invalid under TS 5.9, so it can't be added in either PR alone — it's a one-line follow-up for whichever lands second. Ping Claude to do it.

@LuLaValva
LuLaValva force-pushed the llavalva-M451495HWX-evo-deps-a7929r-marko-vite branch from 6ff93a4 to bb04a9c Compare September 3, 2026 21:27
LuLaValva and others added 2 commits September 3, 2026 14:48
vite 8, @marko/vite 6, @marko/run 0.11, @storybook/marko-vite 5,
vite-css-modules 1.16, marko 6.3.48, and @marko/type-check 3, which
actually type-checks the docs site. Site changes: regenerate
.marko-run/routes.d.ts with @marko/run 0.11 (restores typed
$global.meta), switch scss module imports to default imports with
exportMode 'both' as a workaround for the type-checker ignoring the
generated scss declarations (filed in agent-feedback), declare
@eBay/skin and @docsearch/css side-effect imports, migrate the root
tsconfig off node10 resolution, and refresh a snapshot for marko's new
compiled output. ebayui-core keeps marko 5.38 pinned in the lock; 5.39's
type changes break its type-check (filed in agent-feedback).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@marko/type-check 3 nests TypeScript 6, which hard-errors on the
package's CommonJS-era node10 moduleResolution.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@LuLaValva
LuLaValva force-pushed the llavalva-M451495HWX-evo-deps-a7929r-marko-vite branch from bb04a9c to 680b550 Compare September 3, 2026 21:52
@LuLaValva

Copy link
Copy Markdown
Member Author

Correction to my earlier merge-order note: no coupling with #924 after all. @marko/type-check 3 nests its own TypeScript 6, so the ignoreDeprecations: "6.0" for ebayui-core ships in this PR (it's valid whenever the new checker is present, regardless of the workspace TypeScript version). The earlier observation came from a stale node_modules. Both PRs can merge in any order with no follow-up.

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

PR Preview Deployed

Websiteevo-markoevo-reactebayui-coreebayui-core-reactskin

commit e357d37

LuLaValva and others added 3 commits September 3, 2026 15:22
Nothing in the toolchain reads the generated *.module.scss.d.ts files:
no .ts file imports a scss module and the marko type-checker does not
consult sibling declarations (see agent-feedback). They only churn on
every site build. Re-enable generateSourceTypes when the checker learns
to use them.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Use module preserve with bundler resolution instead of silencing the
TS 6 deprecation, and add the types export condition to the @internal
shim packages — they ship declarations, but exports maps without a
types condition hide them from modern resolution.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
.marko-run and the skin dist output are build artifacts; marking them
linguist-generated collapses them in GitHub review and drops them from
diff stats.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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