Skip to content

Upgrading some dependencies#2952

Merged
RobinTail merged 10 commits intomasterfrom
deps-2025-09-20alt
Sep 20, 2025
Merged

Upgrading some dependencies#2952
RobinTail merged 10 commits intomasterfrom
deps-2025-09-20alt

Conversation

@RobinTail
Copy link
Copy Markdown
Owner

@RobinTail RobinTail commented Sep 20, 2025

instead of #2949

Summary by CodeRabbit

  • Chores
    • Upgraded development tooling dependency to latest patch release to keep tooling current and improve build reliability.
    • No user-facing changes; features, performance, and data remain unaffected.
    • No configuration or action required; existing workflows remain unchanged.
    • Intended for developers and CI environments only, ensuring compatibility with recent Node and ecosystem updates across platforms.

@RobinTail RobinTail added the dependencies Pull requests that update a dependency file label Sep 20, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Sep 20, 2025

Walkthrough

Updated the development dependency tsdown in package.json from ^0.15.0 to ^0.15.3. No other files or logic were changed.

Changes

Cohort / File(s) Summary of Changes
Dev dependency bump
package.json
Updated devDependencies.tsdown from ^0.15.0 to ^0.15.3.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I twitch my ears at version three,
A tiny hop from nought to .3—
tsdown polished, devs agree,
Carrot-shaped semvers please me! 🥕
No code paths changed, just tidied tea,
Onward we bound—dependency-free!

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title Check ❓ Inconclusive The title "Upgrading some dependencies" correctly indicates this PR updates dependencies but is too generic to be useful; the changeset shows a single dev dependency (tsdown) bumped from ^0.15.0 to ^0.15.3, so the title should identify the specific package and version to be clear in the project history. Rename the PR to a specific, concise title such as "chore(deps): bump tsdown from ^0.15.0 to ^0.15.3" or "Upgrade tsdown to ^0.15.3" so reviewers and future readers immediately see the primary change.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch deps-2025-09-20alt

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedtsdown@​0.15.2 ⏵ 0.15.398 +110092 +196 +1100

View full report

Copy link
Copy Markdown
Owner Author

@RobinTail RobinTail left a comment

Choose a reason for hiding this comment

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

ok

@RobinTail RobinTail merged commit d065382 into master Sep 20, 2025
12 of 13 checks passed
@RobinTail RobinTail deleted the deps-2025-09-20alt branch September 20, 2025 08:52
@coveralls-official
Copy link
Copy Markdown

Coverage Status

coverage: 100.0%. remained the same
when pulling adb848b on deps-2025-09-20alt
into 648be8f on master.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 648be8f and adb848b.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • package.json (1 hunks)

Comment thread package.json
"husky": "^9.0.5",
"prettier": "3.6.2",
"tsdown": "^0.15.0",
"tsdown": "^0.15.3",
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.

⚠️ Potential issue

Invalid version range: ^0.15.3 isn’t published on npm — installs will fail

The latest npm release of tsdown is 0.14.2 as of September 20, 2025. Either pin to the latest npm version or switch to the JSR package. (npmjs.com)

Apply this fix (npm path):

-    "tsdown": "^0.15.3",
+    "tsdown": "^0.14.2",

Alternative (JSR path, if you intended 0.15.x+ from JSR): replace the dependency key to use JSR and adjust the version accordingly, e.g. "jsr:@sxzz/tsdown": "^0.15.3". (jsr.io)

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"tsdown": "^0.15.3",
"tsdown": "^0.14.2",
🤖 Prompt for AI Agents
In package.json around line 27, the dependency "tsdown": "^0.15.3" references a
version not published to npm and will fail to install; either change it to the
latest published npm version by replacing the entry with "tsdown": "0.14.2" (pin
to the exact published version) or, if you intended the newer JSR release,
replace the key to the JSR package form and version (e.g. "jsr:@sxzz/tsdown":
"^0.15.3") and update lockfiles/install commands accordingly.

⚠️ Potential issue

🧩 Analysis chain

Ensure Node >= 20.19 in CI if upgrading tsdown

tsdown requires Node.js 20.19+; add an engines.node constraint and verify workflow Node versions to prevent CI breakage. (tsdown.dev)

Run to verify the registry version and CI Node versions:


🏁 Script executed:

#!/bin/bash
# Check npm dist-tags and latest version for tsdown
curl -s https://registry.npmjs.org/tsdown | jq '.["dist-tags"], .["versions"] | keys | sort | last'

# Inspect GitHub Actions workflows for setup-node versions
fd -H ".github/workflows" -t d | xargs -I{} rg -nP -C2 'uses:\s*actions/setup-node@|node-version:\s*["'\'']?(\d+(\.\d+)*)' .github/workflows

Length of output: 390


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "==== package.json (first 200 lines) ===="
nl -ba package.json | sed -n '1,200p' || true

echo
echo "==== tsdown dependency lines ===="
rg -n --hidden -S '"tsdown"\s*:\s*".*"' package.json || true

echo
echo "==== engines field in package.json ===="
jq '.engines' package.json || true

echo
echo "==== repo occurrences of 'tsdown' (excluding node_modules) ===="
rg -n --hidden -S '\btsdown\b' --glob '!node_modules' || true

echo
echo "==== check for node version files (.nvmrc, .node-version, .tool-versions) ===="
[ -f .nvmrc ] && echo ".nvmrc:" && cat .nvmrc || true
[ -f .node-version ] && echo ".node-version:" && cat .node-version || true
[ -f .tool-versions ] && echo ".tool-versions:" && sed -n '1,50p' .tool-versions || true

echo
echo "==== GitHub Actions workflows (if any) and Node setup usage ===="
if [ -d .github/workflows ]; then
  ls -la .github/workflows || true
  rg -n -C2 'actions/setup-node|node-version' .github/workflows || true
else
  echo "No .github/workflows directory found"
  rg -n -C2 'actions/setup-node|node-version' || true
fi

Length of output: 3983


Add engines.node >=20.19.0 to package.json; confirm tsdown version

  • package.json (line 27): add "engines": { "node": ">=20.19.0" } — package.json currently has no engines field.
  • CI: .github/workflows/node.js.yml already tests Node 20.19.0 in its matrix; other workflows use node-version: 22 — CI coverage is sufficient.
  • package.json lists "tsdown": "^0.15.3" but the npm registry shows latest 0.9.9 — confirm this is intentional (private registry or typo) and fix/pin the dependency if needed.
🤖 Prompt for AI Agents
In package.json around line 27, add an "engines" entry with "node": ">=20.19.0"
to enforce the minimum Node version used in CI; also verify the "tsdown"
dependency version — if it was a typo (npm shows 0.9.9) replace "^0.15.3" with
the intended published version (e.g., "0.9.9") or pin the private-registry
package by adding a comment/registry override in your lockfile or .npmrc; after
updating, run npm install to update lockfile and ensure CI workflows continue to
pass.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant