Skip to content

chore(deps): update dependencies#204

Merged
ocavue merged 4 commits intomasterfrom
ocavue/deps62
Feb 10, 2026
Merged

chore(deps): update dependencies#204
ocavue merged 4 commits intomasterfrom
ocavue/deps62

Conversation

@ocavue
Copy link
Owner

@ocavue ocavue commented Feb 10, 2026

No description provided.

@changeset-bot
Copy link

changeset-bot bot commented Feb 10, 2026

⚠️ No Changeset found

Latest commit: cd44884

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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 10, 2026

  • example-playground

    npm i https://pkg.pr.new/ocavue/astrobook@204
    
    npm i https://pkg.pr.new/ocavue/astrobook/@astrobook/core@204
    
    npm i https://pkg.pr.new/ocavue/astrobook/@astrobook/types@204
    
    npm i https://pkg.pr.new/ocavue/astrobook/@astrobook/ui@204
    

commit: cd44884

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 10, 2026

Deploying astrobook with  Cloudflare Pages  Cloudflare Pages

Latest commit: cd44884
Status: ✅  Deploy successful!
Preview URL: https://d6062ffb.astrobook.pages.dev
Branch Preview URL: https://ocavue-deps62.astrobook.pages.dev

View logs

@ocavue ocavue enabled auto-merge (squash) February 10, 2026 00:56
@ocavue ocavue requested a review from Copilot February 10, 2026 00:57
@ocavue ocavue disabled auto-merge February 10, 2026 00:57
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the monorepo’s dependency set (notably lint/tooling and build tooling), with small follow-up changes in @astrobook/core to accommodate the updated build pipeline and public API typing.

Changes:

  • Bumped tooling dependencies (e.g. @ocavue/eslint-config, tsdown), updating pnpm-lock.yaml accordingly.
  • Updated @astrobook/core build config (tsdown.config.ts) and package metadata (peerDependencies) for Astro usage.
  • Exposed IntegrationOptions from @astrobook/core’s public entrypoint and applied a small internal refactor (letconst).

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
pnpm-lock.yaml Large dependency graph update reflecting bumped tooling/build deps.
package.json Bumps root dev dependency @ocavue/eslint-config.
packages/core/package.json Adds optional astro peer dependency; bumps tsdown.
packages/core/tsdown.config.ts Adds inlineOnly option for tsdown build.
packages/core/src/astro-integration.ts Re-exports IntegrationOptions type.
packages/core/src/index.ts Exports IntegrationOptions from public entrypoint.
packages/core/src/virtual-module/vite-plugin.ts Minor refactor (let to const).
packages/astrobook/README.md Reorders imports in examples for consistency.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 3 to 14
const config: unknown = defineConfig({
entry: {
index: 'src/index.ts',
client: 'src/client.ts',
},
format: ['esm'],
clean: true,
dts: true,
fixedExtension: false,
external: [/^virtual:/],
inlineOnly: ['just-kebab-case'],
})
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

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

defineConfig(...) is being assigned to const config: unknown, which disables type-checking of the config object. Since this change adds a new option (inlineOnly), it would be safer to let TS infer the type or explicitly type the config so typos/unsupported options are caught at build time.

Copilot uses AI. Check for mistakes.
Comment on lines +445 to +447
'@babel/generator@8.0.0-rc.1':
resolution: {integrity: sha512-3ypWOOiC4AYHKr8vYRVtWtWmyvcoItHtVqF8paFax+ydpmUdPsJpLBkBBs5ItmhdrwC3a0ZSqqFAdzls4ODP3w==}
engines: {node: ^20.19.0 || >=22.12.0}
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

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

The lockfile now pulls in pre-release toolchain dependencies (e.g. @babel/*@8.0.0-rc.1). If this is coming from tsdown/rolldown-plugin-dts, it increases the risk of build instability; consider pinning to non-pre-release versions (or adding pnpm overrides) if a stable dependency chain is available.

Copilot uses AI. Check for mistakes.
Comment on lines +445 to +447
'@babel/generator@8.0.0-rc.1':
resolution: {integrity: sha512-3ypWOOiC4AYHKr8vYRVtWtWmyvcoItHtVqF8paFax+ydpmUdPsJpLBkBBs5ItmhdrwC3a0ZSqqFAdzls4ODP3w==}
engines: {node: ^20.19.0 || >=22.12.0}
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

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

Newly added dependencies (e.g. @babel/generator@8.0.0-rc.1) require Node ^20.19.0 || >=22.12.0, but the repo doesn't declare a root engines.node (and there's no .nvmrc/.node-version). This can lead to confusing install failures for contributors on older Node 20.x; consider documenting/enforcing the minimum supported Node version in package.json.

Copilot uses AI. Check for mistakes.
"yoctocolors": "^2.1.2"
},
"peerDependencies": {
"astro": "*"
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

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

@astrobook/core declares astro as an optional peer dependency with version "*", which is inconsistent with the rest of the repo (e.g. packages/types and packages/astrobook use >=4.0.0). Using * can allow unsupported/breaking Astro majors; consider setting an explicit supported range (likely >=4.0.0 to match the other packages).

Suggested change
"astro": "*"
"astro": ">=4.0.0"

Copilot uses AI. Check for mistakes.
@ocavue ocavue merged commit 50ab103 into master Feb 10, 2026
23 checks passed
@ocavue ocavue deleted the ocavue/deps62 branch February 10, 2026 01:08
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