[Release] v1.2.4

[Power-Maverick]

Summary

Releasing v1.2.4

Type of change

• New feature
• Bug fix
• Refactor (no functional change)
• Documentation
• Chore / maintenance (dependency update, build, config)
• Test addition / improvement

Changes

Architecture checklist

Packages (types & validation)

• Not applicable — no changes to packages/

If you did change a package:

• @pptb/types (types): type definitions updated and version bumped in packages/types/package.json
• @pptb/validate (validation): validation rules updated and version bumped in packages/validation/package.json

Code quality

• pnpm run typecheck passes with 0 errors (warnings are acceptable)
• pnpm run lint passes with 0 errors (warnings are acceptable)
• pnpm run build completes successfully

Testing

• pnpm run test:unit passes (for changes to src/main/, src/common/, or src/renderer/ utilities)
• pnpm run test:e2e passes (for UI / navigation / end-to-end flows)
• Manually tested in the running app (pnpm run dev)

Scenario tested:

Screenshots / recordings

Breaking changes

• No breaking changes
• Yes — describe impact and migration path below:

Reviewer notes

• I have added appropriate unit and/or e2e tests for this change
• I have resolved all GitHub Copilot review comments
• I have followed the guidelines in CONTRIBUTING.md

[github-actions]

✅ Preflight checks passed

All preflight checks required for merging into main are currently passing.

Checked at commit cf1abc1 — view run

[github-actions]

Bundle Size Report 📦

Bundle	Size
Main Process	1.32 MB
Renderer JS	612.59 KB
Renderer CSS	101.13 KB
Total	2.01 MB

Bundle Analysis Reports

The detailed bundle analysis reports are available in the workflow artifacts:

• 📊 Main Process: stats-main.html
• 📊 Renderer Process: stats-renderer.html

Download the artifacts from the workflow run to view interactive visualizations.

---

Bundle size tracking is now active! This helps prevent bundle bloat.

[Copilot]

Pull request overview

Copilot reviewed 116 out of 129 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (11)

src/renderer/modals/toolDetail/controller.ts:169

• Avoid console.error in production modal scripts (project guideline is to use the app logger/Sentry instead of console). Here the error is not otherwise used, so this can be removed while still showing the user-facing fallback text.
  src/renderer/modals/selectMultiConnection/controller.ts:351
• Avoid console.error in production modal scripts. The caught error isn't used here beyond logging, so remove the console call and just restore the button state.
  src/renderer/modals/selectMultiConnection/controller.ts:488
• Avoid console.error in production modal scripts. If you need user-visible feedback here, prefer wiring an inline feedback element; otherwise remove the console logging line to comply with the repo's no-console policy.
  packages/types/package.json:28
• @pptb/types depends on @pptb/validate but the range ^0.0.2 will never resolve to the newly introduced @pptb/validate@1.0.0 (it is restricted to <0.0.3). This will cause consumers to install an older validate package or fail to resolve the expected CLI/export behavior.
  test-results/.last-run.json:4
• Playwright/Jest outputs under test-results/ should be treated as local/CI artifacts and not checked into git. Please remove this tracked file (e.g. git rm --cached test-results/.last-run.json) and add an ignore rule for test-results/ (and optionally playwright-report/).
  packages/validation/src/cli.ts:52
• Replace remaining console.* calls in this helper with stdout/stderr writes to keep CLI output deterministic and avoid console usage.
  packages/validation/src/cli.ts:57
• Avoid console.error in the top-level error handler; write to stderr explicitly instead.
  src/renderer/modals/selectConnection/controller.ts:33
• Avoid console.warn in production modal scripts. If modalBridge is missing, just return without logging to the console.

This issue also appears on line 420 of the same file.
src/renderer/modals/selectConnection/controller.ts:422

• Avoid console.warn in production modal scripts. If modalBridge.onMessage isn't available, there's nothing actionable to do here besides skipping subscription.
  src/renderer/modals/selectMultiConnection/controller.ts:39
• Avoid console.warn in production modal scripts. If modalBridge is missing, just return without logging to the console.

This issue also appears on line 495 of the same file.
src/renderer/modals/selectMultiConnection/controller.ts:497

• Avoid console.warn in production modal scripts. If modalBridge.onMessage isn't available, skip wiring handlers without writing to the console.

[Copilot]

Pull request overview

Copilot reviewed 116 out of 129 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (4)

src/renderer/modals/toolDetail/controller.ts:169

• Avoid direct console.error in the modal script. This project centralizes logging/telemetry via utilities exposed to modals; emitting to the console here makes it harder to manage logging consistently and can be noisy for users running with DevTools open. Consider using the modal-exposed notification utility (or another centralized mechanism) instead of console.error.
  src/renderer/modals/selectMultiConnection/controller.ts:488
• Avoid console.error in the modal script for auth failures. This is user-visible noise in DevTools and bypasses the centralized modal utilities that already exist for communicating issues back to the user.
  packages/types/package.json:28
• @pptb/types now depends on @pptb/validate, but the version range ("^0.0.2") does not match the workspace package added in this PR (packages/validation is version 1.0.0). With pnpm workspaces this range will not be satisfied by the local package, so installs may pull an older registry version (or fail) instead of linking the workspace package.
  test-results/.last-run.json:4
• This looks like a generated Playwright artifact (test output). Committing it will create noisy diffs and potential merge conflicts; it should be removed from version control and ignored (e.g., ignore the test-results/ directory).

[Copilot]

Pull request overview

Copilot reviewed 115 out of 128 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (6)

src/renderer/modals/toolDetail/controller.ts:167

• Avoid using console.error in production modal scripts (repo guideline). The UI already falls back to a generic README failure message; remove the console error to keep production logs clean and consistent with the app’s logging/telemetry approach.
  src/renderer/modals/selectMultiConnection/controller.ts:348
• Avoid using console.error in production modal scripts. Use the existing modal-exposed toolboxAPI.utils.showNotification (or similar app logging mechanism) so failures are surfaced without emitting console errors in shipped code.
  src/renderer/modals/selectMultiConnection/controller.ts:487
• Avoid using console.error in production modal scripts. Surface auth failures via the existing notification utility (or app logging) rather than writing to the devtools console.
  packages/types/package.json:28
• The workspace now defines @pptb/validate as version 1.0.0 (packages/validation/package.json), but @pptb/types depends on ^0.0.2. In a pnpm workspace this range mismatch prevents linking the local package and instead pulls from the registry, which can break the build/release if 0.0.2 isn’t published (or is the wrong code). Align the dependency range with the workspace package version.
  src/renderer/modals/selectConnection/controller.ts:421
• Avoid using console.warn in production modal scripts. If modalBridge.onMessage is unavailable, silently skip registration rather than logging to the console.
  src/renderer/modals/selectMultiConnection/controller.ts:496
• Avoid using console.warn in production modal scripts. If modalBridge.onMessage is unavailable, silently skip registration rather than writing to the console.

[Copilot]

Pull request overview

Copilot reviewed 115 out of 128 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (4)

src/renderer/modals/toolDetail/controller.ts:169

• Avoid using console.error in production modal scripts; this repo’s logging guidelines prohibit console.*. Since the error is already handled via UI fallback, remove the console call (and rename the unused catch binding to satisfy eslint).
  src/renderer/modals/selectMultiConnection/controller.ts:351
• Avoid console.error in production modal controller scripts. Remove the console call and rename the unused catch binding to satisfy eslint; UI state restoration is already handled here.
  src/renderer/modals/selectMultiConnection/controller.ts:488
• Avoid console.error in production modal controller scripts. If no user-visible feedback is needed here, drop the console logging and rely on the existing connect button reset / UI re-rendering paths.
  packages/types/package.json:28
• @pptb/types depends on @pptb/validate at ^0.0.2, but the workspace package is version 1.0.0. This range won’t satisfy the local package and will force pnpm to fetch an old/nonexistent version from the registry, breaking installs/builds. Update the dependency range to match the validate package version.