Skip to content

Prepare Ceiling 1.5.17 - #181

Merged
tsouth89 merged 1 commit into
mainfrom
release/1.5.17
Jul 29, 2026
Merged

Prepare Ceiling 1.5.17#181
tsouth89 merged 1 commit into
mainfrom
release/1.5.17

Conversation

@tsouth89

@tsouth89 tsouth89 commented Jul 29, 2026

Copy link
Copy Markdown
Owner

Version bump and dated changelog for 1.5.17.

Version

version.env (plus BUILD_NUMBER 118 → 119), both Rust manifests, Cargo.lock, package.json, and tauri.conf.json all move to 1.5.17.

What ships

Also: release-doctor could never pass before the tag existed

gh release view reporting "release not found" is the expected answer on every pre-tag run, but under $ErrorActionPreference = "Stop" a native command writing to stderr raises a terminating error. The doctor died there and never printed its summary, so local-check.ps1 -All -Version <version> — step 3 of docs/RELEASING.md, which runs before the tag is created — always failed, and a genuine failure was indistinguishable from the normal case.

The block already redirected stderr and branched on $LASTEXITCODE; it just never got the chance to. Scoping ErrorActionPreference around the call lets the exit code decide, as intended.

Before: died at line 239, no summary, exit 1.
After: 3 warning(s). Release doctor passed. exit 0, with the warnings being the expected pre-tag ones (no local tag, no local assets dir).

Validation

scripts\local-check.ps1 -All -Version 1.5.17 passes end to end, exit 0:

  • Rust format, both per-manifest clippy -D warnings gates
  • 772 shared + 459 desktop Rust tests
  • 52 frontend test files, frontend build
  • Release doctor: all five version files at 1.5.17, identifier io.github.tsouth89.ceiling, changelog mentions 1.5.17

Note before tagging

This is the first tag to exercise Microsoft Store submission for real. MICROSOFT_STORE_SUBMISSION_ENABLED is true, set right after the validation-only run succeeded, so the gate in #177 was honoured — but the automated submission path itself has not run in a real release.

Summary by CodeRabbit

  • New Features

    • Added Simplified Chinese language support.
    • Added in-app provider login.
    • Windows notifications now remain available in the notification center and display the Ceiling name and logo.
  • Release

    • Updated the application to version 1.5.17.

Bump the five version files plus BUILD_NUMBER, and date the changelog
section covering everything merged since v1.5.16: Windows notification
delivery (#180), Simplified Chinese (#113, #130), provider login flows
(#179), multi-account API value scans (#176), the Cursor Auto/API strip
fix (#175), and Store submission automation (#177, #178).

Also stop release-doctor from dying on its own GitHub check. Under
$ErrorActionPreference = "Stop" a native command writing to stderr raises
a terminating error, so `gh release view` reporting "release not found"
killed the script before it printed a summary. That is the expected
answer for every run before the tag exists, which meant the validation
step RELEASING.md requires could never pass at the point it is meant to
run, and a real failure looked exactly like the normal one. The block
already redirected stderr and branched on $LASTEXITCODE; it just never
got the chance. Scope ErrorActionPreference around the call so the exit
code decides, as intended.
Copilot AI review requested due to automatic review settings July 29, 2026 23:12
@cloudflare-workers-and-pages

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
ceiling 0d7d04c Commit Preview URL

Branch Preview URL
Jul 29 2026, 11:13 PM

@coderabbitai

coderabbitai Bot commented Jul 29, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 98094fcb-6f60-4008-bdea-839c8f948bdb

📥 Commits

Reviewing files that changed from the base of the PR and between 3b14425 and 0d7d04c.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (7)
  • CHANGELOG.md
  • apps/desktop-tauri/package.json
  • apps/desktop-tauri/src-tauri/Cargo.toml
  • apps/desktop-tauri/src-tauri/tauri.conf.json
  • rust/Cargo.toml
  • scripts/release-doctor.ps1
  • version.env

📝 Walkthrough

Walkthrough

Ceiling 1.5.17 release metadata is synchronized across changelog, package manifests, Tauri configuration, Rust metadata, and build variables. The release doctor now handles missing GitHub releases without terminating while preserving exit-code checks.

Changes

Release preparation

Layer / File(s) Summary
Synchronize release metadata
CHANGELOG.md, apps/desktop-tauri/package.json, apps/desktop-tauri/src-tauri/Cargo.toml, apps/desktop-tauri/src-tauri/tauri.conf.json, rust/Cargo.toml, version.env
Version references are updated to 1.5.17, the build number advances to 119, and the changelog records the release contents.
Harden release existence checks
scripts/release-doctor.ps1
The script temporarily sets $ErrorActionPreference to Continue around gh release view, restores it in finally, and continues using $LASTEXITCODE for branching.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Suggested reviewers: copilot, finesssee

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the release-prep version bump for Ceiling 1.5.17.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch release/1.5.17

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

Copilot AI left a comment

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.

Pull request overview

Prepares the Ceiling 1.5.17 release by bumping versions across the Rust + Tauri + frontend surfaces, adding a dated changelog entry, and fixing release-doctor.ps1 so pre-tag runs don’t terminate early when gh release view reports the expected “release not found” case.

Changes:

  • Bump version references to 1.5.17 (and BUILD_NUMBER 119) across release/version files and manifests.
  • Add a 1.5.17 dated entry to CHANGELOG.md.
  • Scope PowerShell $ErrorActionPreference around gh release view so the script can continue and print its summary on expected pre-tag stderr output.

Reviewed changes

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

Show a summary per file
File Description
version.env Bumps marketing version to 1.5.17 and build number to 119.
scripts/release-doctor.ps1 Prevents expected pre-tag gh release view stderr from terminating the script under ErrorActionPreference=Stop.
rust/Cargo.toml Updates shared Rust crate version to 1.5.17.
CHANGELOG.md Adds the 1.5.17 release entry dated 2026-07-29.
Cargo.lock Updates locked package versions for the two workspace crates to 1.5.17.
apps/desktop-tauri/src-tauri/tauri.conf.json Updates Tauri app version to 1.5.17.
apps/desktop-tauri/src-tauri/Cargo.toml Updates Tauri Rust crate version to 1.5.17.
apps/desktop-tauri/package.json Updates frontend package version to 1.5.17.

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

@tsouth89
tsouth89 merged commit 638f2df into main Jul 29, 2026
12 checks passed
@tsouth89
tsouth89 deleted the release/1.5.17 branch July 29, 2026 23:16
@coderabbitai coderabbitai Bot mentioned this pull request Aug 16, 2026
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