chore(deps): upgrade dependencies for Astro 7 (DOCSDEV-75) - #240
Infi-Knight wants to merge 2 commits into
Conversation
Phase 3a of DOCSDEV-75. This site carries the most major bumps in the cycle, including the only MDX integration in the fleet. Core: - astro 6.3.8 to 7.2.10 - @astrojs/starlight 0.39.2 to 0.42.0 - @astrojs/markdown-remark added at 7.3.0 - @astrojs/mdx 5.0.6 to 8.0.0 - @astrojs/netlify 7.0.7 to 8.2.5 - starlight-links-validator 0.24.0 to 0.26.0 - sharp 0.34.5 to 0.35.4 Runtime and content: - @linear/sdk 82.0.0 to 92.0.0 - @netlify/blobs 10.7.4 to 11.0.2 - @netlify/functions 5.2.0 to 6.0.0 - markdown-it 14.2.0 to 15.0.1 - node-html-parser 7.1.0 to 9.0.2 - html-to-text 10.0.0 to 10.0.1 Tooling: - eslint 10.2.1 to 10.9.1 - eslint-plugin-astro 1.7.0 to 3.1.0 - astro-eslint-parser 1.4.0 to 3.1.0 - typescript-eslint and @typescript-eslint/parser 8.59.0 to 8.69.0 - globals 17.5.0 to 17.11.0 - prettier 3.8.3 to 3.9.6 - eslint-plugin-jsx-a11y added at 6.10.2 Removed @astrojs/node. It was declared but never imported. The config uses the Netlify adapter with output: 'static'. Astro 7 makes Sätteri the default markdown processor. This cycle keeps every Starlight site on the unified processor. This site configures no plugins, so its unified() call takes no arguments. Enabled the eslint-plugin-astro accessibility rules through flat/jsx-a11y-recommended. This site reports 0 findings. Prettier 3.9.6 normalizes markdown italic markers from *text* to _text_. That reformats 32 lines across 4 content files. The rendered output does not change. @interledger/docs-design-system stays at 0.13.0. Version 0.14.0 is not published yet. This site uses the design system for CSS files only, so it does not need 0.14.0 to build. A follow-up commit bumps it.
✅ Deploy Preview for interledger-org-developers ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Version 0.14.0 is published. It carries the Astro 7 mermaid import fix and the MermaidWrapper click-target fix. This site uses the design system for CSS files only, so the bump changes no rendered output here.
Anca2022
left a comment
Anca2022
left a comment
There was a problem hiding this comment.
Not blocking the PR, but I’m wondering if we need to keep this repo up to date going forward, or if this is the last time we’re planning to update it since it’s been migrated and is no longer deployed.
|
We should not be maintaining this code anymore. It is now deprecated. Sorry i should have pointed that out in the recurring issue. It's now been updated. |
|
Anca, you asked whether we keep this repo up to date going forward. Sarah answered it on DOCSDEV-76 today:
So I suggest we close this PR rather than merge it. The next upgrade cycle, DOCSDEV-76, no longer lists this repo. The site is already served from elsewhere, which I found before reading her comment:
That also explains the broken deploy preview here. The 404 is Thanks for asking the question. It saved the review time on the other PRs. |
3 participants
Dependency changes
Core:
astro6.3.8 to 7.2.10@astrojs/starlight0.39.2 to 0.42.0@astrojs/markdown-remarkadded at 7.3.0@interledger/docs-design-system0.13.0 to 0.14.0@astrojs/mdx5.0.6 to 8.0.0@astrojs/netlify7.0.7 to 8.2.5starlight-links-validator0.24.0 to 0.26.0sharp0.34.5 to 0.35.4Runtime and content:
@linear/sdk82.0.0 to 92.0.0@netlify/blobs10.7.4 to 11.0.2@netlify/functions5.2.0 to 6.0.0markdown-it14.2.0 to 15.0.1node-html-parser7.1.0 to 9.0.2html-to-text10.0.0 to 10.0.1Tooling:
eslintto 10.9.1,eslint-plugin-astroto 3.1.0,astro-eslint-parserto 3.1.0,typescript-eslintto 8.69.0,globalsto 17.11.0,prettierto 3.9.6,eslint-plugin-jsx-a11yadded at 6.10.2.Removed
@astrojs/nodeIt was declared at
^10.1.2but never imported.astro.config.mjsusesnetlify()withoutput: "static". Removing it avoids a pointless major bump.Markdown processor
Astro 7 makes Sätteri the default. Sätteri does not run remark or rehype plugins, and Starlight follows the same default. This cycle keeps every Starlight site on the
unified()processor. This site configures no plugins, so itsunified()call takes no arguments.@astrojs/mdxgoes to 8.0.0 because Starlight 0.42 itself depends on@astrojs/mdx ^8and@astrojs/markdown-satteri ^0.4. An earlier plan draft stopped at MDX 7.0.8, because Starlight 0.41 usedmdx ^7andsatteri ^0.3and MDX 8 conflicted with it. Starlight 0.42 removed that conflict.Accessibility rules
eslint-plugin-astro3 exposes accessibility rule sets that version 1 did not. The config now extendsflat/jsx-a11y-recommended, which turns on 31 rules. This site reports 0 findings.Prettier reformats 4 content files
Prettier 3.9.6 normalizes markdown italic markers from
*text*to_text_. That is 32 lines across 4 content files. Markdown renders*x*and_x_identically, so the output does not change. I verified that by comparing rendered text against production.Test plan
bun installclean, only the known stale jsx-a11y peer warningbun run buildpassed. 78 HTML files, SSR function generated,_redirectsemittedstarlight-links-validator0.26.0 reports all internal links valid, and the/participation-guidelinesexclusion still appliesbun run lintexits 0 at--max-warnings=0node-html-parser9 ran during the buildmarkdown-it15 andhtml-to-text10. Href stripping, image skipping and figure skipping all still work.mainhas one. My diff touches only lines 36 to 54 of that file.@interledger/docs-design-systembumped to^0.14.0and rebuilt (commitd2e05a3)@linear/sdk92 and@netlify/blobs11 did not run during the build.sync-nowfunction responds (@netlify/functions6)scripts/sync-blob.tsruns on production only, not on previews.Refs: DOCSDEV-75