Skip to content

Update detsys-ts: Merge pull request #116 from DeterminateSystems/dependabot/github_actions/actions-deps-76468cb07f#226

Merged
lucperkins merged 2 commits intoDeterminateSystems:mainfrom
detsys-pr-bot:detsys-ts-update-0e3d1accf3c5dab8b18274658ede653aa016b6cd
Dec 1, 2025
Merged

Update detsys-ts: Merge pull request #116 from DeterminateSystems/dependabot/github_actions/actions-deps-76468cb07f#226
lucperkins merged 2 commits intoDeterminateSystems:mainfrom
detsys-pr-bot:detsys-ts-update-0e3d1accf3c5dab8b18274658ede653aa016b6cd

Conversation

@detsys-pr-bot
Copy link

@detsys-pr-bot detsys-pr-bot commented Dec 1, 2025

Bump detsys-ts to the latest HEAD and regenerate application bundle.

Summary by CodeRabbit

  • Chores
    • Updated development dependencies to their latest versions.

✏️ Tip: You can customize this high-level summary in your review settings.

…om DeterminateSystems/dependabot/github_actions/actions-deps-76468cb07f` (`0e3d1accf3c5dab8b18274658ede653aa016b6cd`)
@coderabbitai
Copy link

coderabbitai bot commented Dec 1, 2025

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • flake.lock is excluded by !**/*.lock

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

package.json devDependencies updated: prettier from ^3.6.2 to ^3.7.3 and tsup from ^8.5.0 to ^8.5.1. No runtime behavior changes or API alterations detected.

Changes

Cohort / File(s) Summary
Dependency Updates
package.json
Updated prettier to ^3.7.3 and tsup to ^8.5.1

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Verify that prettier and tsup patch/minor version updates contain no breaking changes
  • Confirm devDependency updates pose no compatibility issues with existing toolchain

Suggested reviewers

  • grahamc
  • lucperkins

Poem

🐰 Hop, hop, hooray! A rabbit's delight,
Prettier formats and tsup builds bright,
Dependencies updated with care,
Fresh versions float through the air! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title references merging PR #116 from a dependency update, but the actual changes shown are only package.json devDependency updates (prettier and tsup versions). Consider a more concise, specific title that accurately reflects the main change, such as 'Update devDependencies: prettier and tsup' or 'Bump detsys-ts dependency'.
✅ 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 changed files to evaluate docstring coverage. Skipping docstring coverage check.

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.

Copy link

@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: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 834c491 and 836f399.

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

Comment on lines +42 to +43
"prettier": "^3.7.3",
"tsup": "^8.5.1",
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🌐 Web query:

prettier npm 3.7.3 December 2025

💡 Result:

Yes — Prettier v3.7.x is released and v3.7.3 is available (build artifacts/tarball indexed). Sources: Prettier 3.7 blog/GitHub release activity and a 3.7.3 package artifact index. [1][2][3]

Sources

  • Prettier blog (3.7.0 announcement, Nov 27, 2025). [1]
  • Prettier GitHub (release/milestone activity for 3.7). [2]
  • Prettier 3.7.3 archive/index (prettier-3.7.3.tar.gz listed). [3]

🌐 Web query:

tsup npm 8.5.1 December 2025

💡 Result:

There is no tsup v8.5.1 on npm as of December 1, 2025. The latest published tsup release on npm is v8.1.1 (published earlier), and the tsup repo notes the project is no longer actively maintained (they recommend migrating to tsdown). [1][2]

Sources:
[1] npm — tsup v8.1.1.
[2] egoist/tsup GitHub README (projects not actively maintained; migration to tsdown).


tsup version ^8.5.1 does not exist on npm—correct before merge.

prettier ^3.7.3 is valid and current. However, tsup v8.5.1 has never been published to npm; the latest available version is 8.1.1. Additionally, tsup is no longer actively maintained and the project recommends migration to tsdown. Update the tsup version to a valid release (8.1.1 or evaluate migration) to prevent installation failures.

🤖 Prompt for AI Agents
In package.json around lines 42-43, the specified tsup version "^8.5.1" is
invalid on npm and will break installs; update the dependency to a valid
published release (e.g., "tsup": "^8.1.1") or replace it by migrating to the
recommended successor (tsdown) if you want to follow upstream guidance; after
changing package.json run your package manager to update lockfile (npm install /
npm ci or yarn install) and verify builds/tests pass.

@lucperkins lucperkins enabled auto-merge December 1, 2025 19:44
@lucperkins lucperkins merged commit 1a51c18 into DeterminateSystems:main Dec 1, 2025
3 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