Skip to content

release: fill the 0.0.39 offline release fallback digests - #60

Merged
gitcommit90 merged 1 commit into
mainfrom
release/0.0.39-fallback-digests
Aug 3, 2026
Merged

release: fill the 0.0.39 offline release fallback digests#60
gitcommit90 merged 1 commit into
mainfrom
release/0.0.39-fallback-digests

Conversation

@gitcommit90

@gitcommit90 gitcommit90 commented Aug 3, 2026

Copy link
Copy Markdown
Owner

The offline release fallback is served when GitHub's release API is unreachable, and install.sh verifies the digest it returns. Those digests are the digests of the 0.0.39 release commit's own artifacts, so they could not exist in that commit — this is the follow-up that fills them in, after which the site is redeployed.

Until now the placeholder was deliberately not 64 hex characters, so latestLinuxRelease() rejected it and /api/releases/linux/latest answered 503 — failing closed rather than handing an installer a digest that could not match what it downloads.

Verified locally against what the build lanes reported:

Artifact SHA-256 Bytes
1Helm-0.0.39-arm64.dmg de381468…fd0b 504,397,451
1Helm-0.0.39-mac-arm64.zip a35ba43a…4547 301,207,402
1Helm-0.0.39-linux-node.tgz ac54f11c…b077 401,004,908

npm run ci: 122 tests, 120 pass, 0 fail.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Bug Fixes
    • Updated release fallback information with the correct SHA-256 checksums for macOS and Linux version 0.0.39 artifacts.
    • Removed outdated placeholder digest information and macOS-specific fallback messaging.

The website's offline fallback is served when GitHub's release API is
unreachable, and install.sh verifies the digest it returns. Those digests
are the digests of the 0.0.39 release commit's own artifacts, so they could
not exist in that commit.

Until now the placeholder was deliberately not 64 hex characters, so
latestLinuxRelease() rejected it and /api/releases/linux/latest answered
503 - an installer was told no release was available rather than handed a
digest that could not match what it downloaded. The artifacts now exist and
have been verified locally against the values reported by the macOS and
Linux build lanes:

  1Helm-0.0.39-arm64.dmg        de381468…  504,397,451
  1Helm-0.0.39-mac-arm64.zip    a35ba43a…  301,207,402
  1Helm-0.0.39-linux-node.tgz   ac54f11c…  401,004,908

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 3cc22e95-eb80-4706-af5d-0b7ea02b2a15

📥 Commits

Reviewing files that changed from the base of the PR and between 3da8dcd and 4d02a23.

📒 Files selected for processing (1)
  • site/server.mjs

📝 Walkthrough

Walkthrough

The release fallback in site/server.mjs now uses concrete SHA-256 digests for the 0.0.39 macOS and Linux artifacts. The pending-digest placeholder and macOS-specific fallback explanation were removed.

Changes

Release fallback metadata

Layer / File(s) Summary
Complete fallback artifact digests
site/server.mjs
The 0.0.39 fallback entries now contain concrete SHA-256 digests. The fallback documentation was updated. Release validation behavior remains unchanged.

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

Possibly related PRs

  • gitcommit90/1Helm#43: Both changes update release fallback and artifact digest handling in site/server.mjs.
  • gitcommit90/1Helm#44: Both changes update hard-coded release fallback metadata and artifact SHA-256 digests.
  • gitcommit90/1Helm#58: Both changes update hard-coded release fallback metadata with artifact SHA-256 digests.
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the change and verification results but omits the required type, release notes, and verification checklist sections. Complete the template sections for change type, release notes, checklist items, and any applicable documentation or post-merge requirements.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: adding the 0.0.39 offline release fallback digests.
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/0.0.39-fallback-digests

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

@gitcommit90
gitcommit90 merged commit 4a13b6d into main Aug 3, 2026
6 checks passed
@gitcommit90
gitcommit90 deleted the release/0.0.39-fallback-digests branch August 3, 2026 01:32
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.

1 participant