Skip to content

Select desktop archives precisely in published-package verification - #1976

Open
currentbits wants to merge 1 commit into
mainfrom
coder/nightly-package-verifier
Open

currentbits wants to merge 1 commit into
mainfrom
coder/nightly-package-verifier

Conversation

@currentbits

Copy link
Copy Markdown
Owner

Published releases contain both desktop and headless server Linux archives. The package verifier matched their shared -linux-x64.tar.gz suffix and failed before downloading either. Select the exact desktop asset name for the requested release instead.

Also record Windows tree-termination output and final child state in the verification artifact, so a cleanup failure after successful startup/embedding has actionable evidence.

Validation: the published nightly reproduces two suffix matches and one exact desktop match. Native Linux and Windows verification both passed against nightly-202611091044-5f59a825 in workflow https://github.com/currentbits/solenta/actions/runs/42606820529, including download checksum, metadata, isolated startup, storage and real embeddings. No application binaries changed. The existing main sitemap-date failure is tracked separately and is unrelated to this verifier change.

Refs #1975.

Created with GPT-6.1 Sol in Codex.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-09T11:18:01.962437Z ad116c8 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

This branch has not been deployed

No deployments
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