Skip to content

fix: revert Cargo.lock sync commit to plain git push#52

Merged
quiet-node merged 1 commit intomainfrom
worktree-structured-floating-stearns
Apr 6, 2026
Merged

fix: revert Cargo.lock sync commit to plain git push#52
quiet-node merged 1 commit intomainfrom
worktree-structured-floating-stearns

Conversation

@quiet-node
Copy link
Copy Markdown
Owner

Summary

Reverts the GitHub API approach for the Cargo.lock sync commit introduced in #50.

The gh api PUT call failed with Argument list too long because base64-encoding the full Cargo.lock inline as a shell argument ($(base64 -w 0 src-tauri/Cargo.lock)) exceeds the OS argument size limit. The plain git commit + git push approach works correctly and the unverified badge on this bot commit is acceptable.

Test plan

  • Merge this PR
  • Confirm the next release PR gets a chore: sync Cargo.lock to version bump commit without errors

🤖 Generated with Claude Code

The GitHub API approach failed with 'Argument list too long' because
base64-encoding the full Cargo.lock inline as a shell argument exceeds
the OS arg limit. Plain git commit + push works fine and the unverified
badge on this bot commit is acceptable.

Signed-off-by: Logan Nguyen <lg.131.dev@gmail.com>
@quiet-node quiet-node merged commit 904cdf4 into main Apr 6, 2026
3 checks passed
@quiet-node quiet-node deleted the worktree-structured-floating-stearns branch April 6, 2026 00:40
quiet-node added a commit that referenced this pull request Apr 6, 2026
Removes the sync-cargo-lock job and the pr output added in PRs #43-#52.
The release-please action started failing with 'Resource not accessible by
integration' and the sync job is the only change to this workflow in that
window. Reverting to the original two-job workflow to restore stability.

Signed-off-by: Logan Nguyen <lg.131.dev@gmail.com>
quiet-node added a commit that referenced this pull request Apr 10, 2026
fix: revert Cargo.lock commit to plain git push

The GitHub API approach failed with 'Argument list too long' because
base64-encoding the full Cargo.lock inline as a shell argument exceeds
the OS arg limit. Plain git commit + push works fine and the unverified
badge on this bot commit is acceptable.

Signed-off-by: Logan Nguyen <lg.131.dev@gmail.com>
quiet-node added a commit that referenced this pull request Apr 10, 2026
fix: revert Cargo.lock commit to plain git push

The GitHub API approach failed with 'Argument list too long' because
base64-encoding the full Cargo.lock inline as a shell argument exceeds
the OS arg limit. Plain git commit + push works fine and the unverified
badge on this bot commit is acceptable.

Signed-off-by: Logan Nguyen <lg.131.dev@gmail.com>
quiet-node added a commit that referenced this pull request Apr 11, 2026
fix: revert Cargo.lock commit to plain git push

The GitHub API approach failed with 'Argument list too long' because
base64-encoding the full Cargo.lock inline as a shell argument exceeds
the OS arg limit. Plain git commit + push works fine and the unverified
badge on this bot commit is acceptable.

Signed-off-by: Logan Nguyen <lg.131.dev@gmail.com>
This was referenced Apr 11, 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.

1 participant