Version 0.20260714.1#529
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR bumps the application version to 0.20260714.1, updates a few dependencies, and migrates several Bun-run utility scripts from .js usage to .ts, including adding new maintenance scripts for Electron installation, cleaning, and dependency updates.
Changes:
- Bump root and renderer package versions and update dependency ranges (and lockfiles).
- Switch build/typecheck/version/archive/maintenance scripts to run the
.tssources directly under Bun. - Add new
ensure.electron.ts,clean.ts, and a more interactivedependencies.update.ts(and remove the old.jsupdater).
Reviewed changes
Copilot reviewed 9 out of 13 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| src/renderer/scripts/version.new.ts | Adds TS typing assertions while keeping the version-bump logic. |
| src/renderer/scripts/type.check.ts | Tightens typings for the Module._resolveFilename hook used by vue-tsc/TS6. |
| src/renderer/scripts/libopencor.ts | Minor adjustment to Bun spawn usage for building the native module. |
| src/renderer/scripts/generate.version.ts | Reads version from package.json with a typed parse and writes version.json. |
| src/renderer/scripts/ensure.electron.ts | New postinstall helper to explicitly run Electron’s installer when using Bun. |
| src/renderer/scripts/dependencies.update.ts | New interactive dependency update checker/updater script for root + renderer. |
| src/renderer/scripts/dependencies.update.js | Removes the old non-interactive dependency update script. |
| src/renderer/scripts/clean.ts | New Bun clean script removing build outputs and node_modules folders. |
| src/renderer/scripts/archive.web.ts | Typed parse for version when generating the web tarball name. |
| src/renderer/package.json | Updates scripts to use .ts, bumps renderer version, updates dependency ranges. |
| src/renderer/bun.lock | Lockfile updates for renderer dependency changes. |
| package.json | Updates scripts to use .ts, bumps root version, updates dependency ranges. |
| bun.lock | Lockfile updates for root dependency changes. |
Comments suppressed due to low confidence (1)
src/renderer/scripts/type.check.ts:19
Module._resolveFilenamecan be invoked with a null/undefinedparentby Node internals. Typing it asModuleis too strict and can hide real call patterns (or force unsafe casts later).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
agarny
force-pushed
the
cleaning-up
branch
2 times, most recently
from
July 14, 2026 05:35
90516fc to
de21305
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.