chore(deps-dev): bump astro, @astrojs/mdx and astro-auto-import#3700
chore(deps-dev): bump astro, @astrojs/mdx and astro-auto-import#3700dependabot[bot] wants to merge 4 commits into
Conversation
✅ Deploy Preview for boosted ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
@dependabot recreate |
a9de4db to
c3af25e
Compare
c3af25e to
e19514d
Compare
This comment was marked as outdated.
This comment was marked as outdated.
There was a problem hiding this comment.
Pull request overview
Updates the documentation site tooling to Astro 7 / MDX 7 and migrates the docs rendering pipeline accordingly (content loading, rendering, and shortcode wiring), with a small CSS adjustment and bundle size threshold update.
Changes:
- Bump Astro + related integrations (notably
@astrojs/mdx) and adapt the site to Astro 7 content/rendering APIs. - Replace the previous global shortcode auto-import mechanism with explicit shortcode exports and passing shortcodes into rendered content.
- Update docs/content routing to use an
id-based slug helper and add content loaders viaastro/loadersglob.
Reviewed changes
Copilot reviewed 17 out of 21 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
site/static/orange/docs/[version]/assets/fonts/solaris-icons.css |
Adjust font URL to point to the local .woff2 file. |
site/src/types/auto-import.d.ts |
Remove generated global auto-import type declarations (auto-import mechanism removed). |
site/src/pages/[brand]/docs/[version]/[...slug].astro |
Switch to astro:content render() and pass shortcodes via components. |
site/src/pages/[brand]/[...alias].astro |
Use getDocsPageSlug() for alias path generation. |
site/src/pages/[...alias].astro |
Use getDocsPageSlug() for alias path generation. |
site/src/libs/data.ts |
Update URL validation schema to z.url(). |
site/src/libs/content.ts |
Add getDocsPageSlug() helper for stripping .md/.mdx from entry IDs. |
site/src/libs/config.ts |
Update multiple URL fields to validate with z.url(). |
site/src/libs/astro.ts |
Remove auto-import integration and move markdown processor config responsibilities out. |
site/src/content/docs/components.mdx |
Explicitly import ComponentCard from shortcodes. |
site/src/content.config.ts |
Add glob() loaders for docs and callouts collections. |
site/src/components/shortcodes/index.ts |
New barrel file exporting all shortcodes for easy import/spread. |
site/src/components/shortcodes/Callout.astro |
Use render(entry) from astro:content instead of entry.render(). |
site/src/components/DocsSidebar.astro |
Make sidebar map as const, fix typing, and match docs pages via id-derived slug. |
site/astro.config.ts |
Configure markdown processor via unified() with existing remark/rehype plugins. |
scss/tests/snapshot-tests/__snapshots__/ouds-web.css |
Snapshot update reflecting generated CSS changes. |
scss/tests/snapshot-tests/__snapshots__/ouds-web-bootstrap.css |
Snapshot update reflecting generated CSS changes. |
scss/_alert.scss |
Add width rules affecting alert container/text layout. |
package.json |
Update Astro-related devDependencies; adjust css-snapshot-refresh command. |
.bundlewatch.config.json |
Raise CSS bundle size threshold slightly. |
Files not reviewed (2)
- scss/tests/snapshot-tests/snapshots/ouds-web-bootstrap.css: Generated file
- scss/tests/snapshot-tests/snapshots/ouds-web.css: Generated file
| flex-direction: column; | ||
| gap: $ouds-alert-space-row-gap-action; | ||
| align-items: flex-start; | ||
| justify-content: center; // To handle weird edge cases in de-zoom one line title alert | ||
| width: 20%; | ||
| min-height: $ouds-alert-size-min-height; // To handle weird edge cases in de-zoom one line title alert |
MaxLardenois
left a comment
There was a problem hiding this comment.
We need to integrate work done for the changelog links, otherwise ok!
|
|
||
| const sidebar = sidebarMap[slug] ? getData(`sidebar-${sidebarMap[slug]}`) : [] | ||
| const sidebarKey = sidebarMap[slug as keyof typeof sidebarMap] | ||
| const sidebar = sidebarKey ? getData(`sidebar-${sidebarKey}`) : [] |
There was a problem hiding this comment.
Ah, I rewrote this too in the Scrollspy update. I started from the actual data definition to get the sidebars and have a hard typing. We can keep this and see what we do on the scrollspy side
There was a problem hiding this comment.
It's only some warnings that I got from the tsconfig, I can revert them and wait for Scrollspy
There was a problem hiding this comment.
Yes same, just I fixed them with an other solution but we can discuss on the Scrollspy PR
| configurePrism() | ||
|
|
||
| return [ | ||
| ouds_web_auto_import(), |
There was a problem hiding this comment.
question: Was that discouraged by Astro or simply no longer possible?
There was a problem hiding this comment.
|
|
||
| import { componentsDetails } from '../../../data/components-details' | ||
| import { getVersionedDocsPath } from '@libs/path' | ||
| import { ComponentCard } from '@components/shortcodes' |
There was a problem hiding this comment.
Is it not made available everywhere by what you added in [...slug].astro?
There was a problem hiding this comment.
I thought, but I had some errors while not having it, I don't know why tbh
| integrations: [oudsWeb()], | ||
| markdown: { | ||
| smartypants: false, | ||
| processor: unified({ |
There was a problem hiding this comment.
I do not know if this was a mandatory change but I think it's clearer to have that config here 👍
Will need to integrate the modifications that @duvalale did for the changelog links
Bumps astro, @astrojs/mdx and astro-auto-import. These dependencies needed to be updated together.
Updates
astrofrom 5.13.5 to 7.1.3Release notes
Sourced from astro's releases.
... (truncated)
Changelog
Sourced from astro's changelog.
... (truncated)
Commits
cf9ffc7[ci] release (#17455)630b382Useos.availableParallelism()for image optimization queue to fix OOM in CP...8a7f7e0[ci] release (#17411)ebbd793chore(deps): updateastro-embed(#17436)311e42c[ci] format1d62202chore(deps): updateneotraverseto v1 (#17446)a5f7230fix(astro): updatecookieto v2 (#17445)91992effix(prefetch): swallow fetch-fallback rejection to avoid unhandled 'Load fail...a89c137fix(astro): propagate mutated locals to custom error pages (#17402)91c645a[ci] release (#17403)Maintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for astro since your current version.
Updates
@astrojs/mdxfrom 4.3.4 to 7.0.3Release notes
Sourced from @astrojs/mdx's releases.
... (truncated)
Changelog
Sourced from @astrojs/mdx's changelog.
... (truncated)
Commits
dec7692[ci] release (#17338)64b0d66fix(satteri): route highlighted code blocks through MDX pre components (#17341)a86160e[ci] release (#17256)7d8ec1f[ci] release (#17237)02b73b0fix: peer deps of packages (#17249)0a054e7chore(head): normalize html charset declarations (#17197)2bfb179chore: simpler package.json types (#17229)dc260daRemove legacy "X-UA-Compatible" meta tags (#17162)f55ba4c[ci] release (#17132)7725a61[ci] formatMaintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for
@astrojs/mdxsince your current version.Updates
astro-auto-importfrom 0.4.4 to 0.5.2Release notes
Sourced from astro-auto-import's releases.
Changelog
Sourced from astro-auto-import's changelog.
Commits
e45717eVersion Packages (#159)965278fImproved Astro 7 support (#158)d99d066Update dependency typescript to ^6.0.3 (#142)ee64877Update dependency@types/nodeto ^20.19.43 (#149)9678a1aUpdate astro (#141)dc91d54Update dependency acorn to ^8.17.0 (#155)c3568deUpdate dependency typescript to v6 (#136)0f39c97Update dependency@types/nodeto ^20.19.39 (#138)0488a44Update dependency astro to ^6.1.5 (#139)12979d2Update astro (#133)Maintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for astro-auto-import since your current version.
Closes #3622