Skip to content

fix(release): preserve PR notes and validate WinGet access - #658

Open
pascalandr wants to merge 2 commits into
NeuralNomadsAI:devfrom
pascalandr:fix/release-notes-winget-automation
Open

fix(release): preserve PR notes and validate WinGet access#658
pascalandr wants to merge 2 commits into
NeuralNomadsAI:devfrom
pascalandr:fix/release-notes-winget-automation

Conversation

@pascalandr

Copy link
Copy Markdown
Contributor

Summary

  • prepend the merged stable release PR body to GitHub-generated release notes
  • keep nightly prerelease notes unchanged
  • verify the WinGet token can create and delete branches in the configured fork before invoking Komac
  • document writable PAT and GitHub CLI OAuth token options

Root cause

The v0.19.0 WinGet job reached Komac successfully, but WINGET_GITHUB_TOKEN could not execute CreateRef in pascalandr/winget-pkgs. The repository secret has been replaced with the currently authenticated writable GitHub CLI token.

Validation

Manual WinGet submission: microsoft/winget-pkgs#423225

Prepend the merged stable release pull request body to GitHub's generated notes so user-facing highlights are present from the moment a release is created. Prerelease behavior remains unchanged.

Validate that the configured WinGet token can create and delete a temporary branch in the contributor fork before invoking Komac. This catches the exact permission failure that blocked v0.19.0 and documents supported writable token types.

Validated with actionlint, a live associated-PR lookup for the v0.19.0 merge commit, a temporary fork branch create/delete check, and winget manifest validation for the manual 0.19.0 submission.
Run the Windows Rust test binary with one test thread so process-election tests do not compete for PowerShell/CIM process identity probes on slower GitHub runners.

The Electron and Tauri startup scenario remains concurrent inside its integration test; only unrelated Rust tests stop running beside it.

Validated with actionlint and cargo test --locked -- --test-threads=1 on Windows (86 passed).
@github-actions

Copy link
Copy Markdown

PR builds are available as GitHub Actions artifacts:

https://github.com/NeuralNomadsAI/CodeNomad/actions/runs/32718172386

Artifacts expire in 7 days.
Artifacts:

  • pr-658-acfe5e12cea0c2633ec494410b1dfa799d9fb098-tauri-macos
  • pr-658-acfe5e12cea0c2633ec494410b1dfa799d9fb098-tauri-windows
  • pr-658-acfe5e12cea0c2633ec494410b1dfa799d9fb098-tauri-linux
  • pr-658-acfe5e12cea0c2633ec494410b1dfa799d9fb098-electron-macos
  • pr-658-acfe5e12cea0c2633ec494410b1dfa799d9fb098-tauri-macos-arm64
  • pr-658-acfe5e12cea0c2633ec494410b1dfa799d9fb098-electron-linux
  • pr-658-acfe5e12cea0c2633ec494410b1dfa799d9fb098-electron-windows

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