Skip to content

fix: repair main build after cross-PR merge skew#1020

Merged
wpfleger96 merged 1 commit into
mainfrom
worktree-wpfleger+fix-main-ci-merge-skew
Jun 12, 2026
Merged

fix: repair main build after cross-PR merge skew#1020
wpfleger96 merged 1 commit into
mainfrom
worktree-wpfleger+fix-main-ci-merge-skew

Conversation

@wpfleger96

Copy link
Copy Markdown
Collaborator

Two independent clippy/compile errors broke main (87e45c6) and the local pre-push hook, blocking just release for v0.3.19.

Both are cross-PR merge skew: CI for each contributing PR ran before its partner merged, so neither caught the conflict. The error_outcome_emission_tests::test_config() initializer was added by #1010 before #1017 introduced turn_liveness_secs: u64 on Config; #1017 updated the two sibling test configs but missed this one. The manual_is_multiple_of clippy lint in legacy_storage.rs came from #1016 but wasn't exercised by GitHub CI for that PR — only by the local pre-push hook.

  • Add missing turn_liveness_secs: 10 to error_outcome_emission_tests::test_config() in crates/buzz-acp/src/lib.rs (matches field order and the value used by the two sibling test configs)
  • Replace bytes.len() % 2 == 0 with bytes.len().is_multiple_of(2) in desktop/src-tauri/src/commands/legacy_storage.rs:83

error_outcome_emission_tests::test_config() (added by #1010) was missing
the turn_liveness_secs field that #1017 added to Config — #1017's CI ran
before #1010 merged so this third initializer was overlooked.

Also fix desktop-tauri clippy manual_is_multiple_of in legacy_storage.rs
(from #1016), which the pre-push hook enforces locally but GitHub CI did
not exercise for that PR.
@wpfleger96 wpfleger96 merged commit b8c0556 into main Jun 12, 2026
20 checks passed
@wpfleger96 wpfleger96 deleted the worktree-wpfleger+fix-main-ci-merge-skew branch June 12, 2026 22:20
tlongwell-block pushed a commit that referenced this pull request Jun 13, 2026
* origin/main: (33 commits)
  fix(desktop): make Windows release compile cleanly (#1029)
  Add production Docker Compose bundle (#985)
  feat(profile): show active turn badges on agent profile panel and popover (#1026)
  chore(release): release version 0.3.20 (#1027)
  fix(release): resolve Windows sidecar path and Linux AppImage updater format (#1024)
  chore(release): release version 0.3.19 (#1014)
  fix(release): ignore prerelease tags in changelog generation (#1021)
  fix: repair main build after cross-PR merge skew (#1020)
  feat(agents): show per-turn duration and prune dead turns within ~25s of host crash (#1017)
  fix(release): replace hermit with native tool setup on Windows job (#1018)
  feat(acp): surface error-class outcomes to the activity feed only, never the channel (#1010)
  fix(desktop): migrate Sprout workspace storage (#1016)
  feat(auth): force token refresh on rejected token (401/403), never the browser (#1015)
  fix(release): mark prerelease versions so they do not become latest (#1013)
  feat(acp): implement systemPrompt with protocol version gating (#981)
  fix(release): update repository name check from block/sprout to block/buzz (#1012)
  feat(release): all-OS desktop builds + universal auto-update manifest (#1011)
  Add relay disconnect UX: friendly errors, reconnect, cached identity (#1004)
  feat(agents): add active turn indicators to Agents Menu (#1005)
  ci: add fork guards to docker, release, and auto-tag workflows (#1007)
  ...

Co-authored-by: npub1t2tgm7d8f995uqvmnm8h88sg3wnpp9a5xysjf6dg3tjmgt3ltulqdp8ehr <5a968df9a7494b4e019b9ecf739e088ba61097b4312124e9a88ae5b42e3f5f3e@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: npub1t2tgm7d8f995uqvmnm8h88sg3wnpp9a5xysjf6dg3tjmgt3ltulqdp8ehr <5a968df9a7494b4e019b9ecf739e088ba61097b4312124e9a88ae5b42e3f5f3e@sprout-oss.stage.blox.sqprod.co>
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