Skip to content

Upgrade dependencies - #207

Merged
cyanzhong merged 1 commit into
mainfrom
develop
Oct 8, 2026
Merged

cyanzhong merged 1 commit into
mainfrom
develop

Conversation

@cyanzhong

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The upgrade duplicates KaTeX in the production bundle and introduces unconfigured third-party registry URLs.

1 open finding
What changed in this PR

Upgrades core dependencies and adapts build, typing, rendering, and CI integration.

Changes:

  • Upgrades Vite, TypeScript, KaTeX, YAML, and related dependencies.
  • Migrates build configuration to Rolldown and adds type checking.
  • Adds KaTeX and bundle validation tests.
File Description
.github/​workflows/​lint-and-test.yml Runs type checking in CI.
README.md Documents type checking.
package.json Upgrades dependencies and build scripts.
src/​features/​frontMatter.ts Updates Markdown-It type import.
src/​quicklook/​shim.ts Generalizes build-banner documentation.
src/​render.ts Shares KaTeX 0.19 with the Markdown plugin.
styles/​katex.css Synchronizes KaTeX 0.19 styles.
tests/​build.test.ts Validates styles and generated bundles.
tests/​render.test.ts Tests KaTeX 0.19 rendering.
tsconfig.json Expands TypeScript checking scope.
vite.config.mts Migrates banner configuration to Rolldown.
yarn.lock Locks upgraded dependency versions.

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

Comment thread package.json
@cyanzhong
cyanzhong force-pushed the develop branch 2 times, most recently from 44650e6 to a303964 Compare October 8, 2026 03:15
@cyanzhong
cyanzhong requested a balanced review from Copilot October 8, 2026 03:20

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The lockfile unexpectedly routes many packages through an unconfigured third-party registry mirror.

0 open findings

1 resolved since last review

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The Vite configuration currently drops host-provided externals by defining conflicting legacy and new bundler option keys.

1 open finding

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

Comment thread vite.config.mts
@cyanzhong
cyanzhong merged commit 071e411 into main Oct 8, 2026
2 checks passed
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.

2 participants