Skip to content

Update Rust crate temporalio-common to eac03fb#8421

Open
hash-worker[bot] wants to merge 1 commit into
mainfrom
deps/rs/temporalio-common-digest
Open

Update Rust crate temporalio-common to eac03fb#8421
hash-worker[bot] wants to merge 1 commit into
mainfrom
deps/rs/temporalio-common-digest

Conversation

@hash-worker
Copy link
Copy Markdown
Contributor

@hash-worker hash-worker Bot commented Feb 14, 2026

This PR contains the following updates:

Package Type Update Change
temporalio-common workspace.dependencies digest 231e21ceac03fb

Warning

Some dependencies could not be looked up. Check the Dependency Dashboard for more information.


Configuration

📅 Schedule: (UTC)

  • Branch creation
    • "before 4am every weekday,every weekend"
  • Automerge
    • "before 4am every weekday,every weekend"

🚦 Automerge: Enabled.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

@hash-worker hash-worker Bot enabled auto-merge February 14, 2026 00:41
@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hash Error Error May 14, 2026 9:46pm
hashdotdesign Ready Ready Preview, Comment May 14, 2026 9:46pm
hashdotdesign-tokens Ready Ready Preview, Comment May 14, 2026 9:46pm
petrinaut Ready Ready Preview May 14, 2026 9:46pm

@cursor
Copy link
Copy Markdown

cursor Bot commented Feb 14, 2026

PR Summary

Medium Risk
Dependency-only change, but it updates Temporal SDK core plumbing and introduces a second temporalio-common version alongside temporalio-client, which could cause build or runtime integration issues if APIs/behavior diverge.

Overview
Updates the workspace dependency on temporalio-common to the eac03fbc… revision.

Cargo.lock is refreshed accordingly, adding new transitive crates (notably pbjson/pbjson-build) and leaving both temporalio-common 0.1.0 (via temporalio-client) and temporalio-common 0.4.0 (direct) in the resolved graph.

Reviewed by Cursor Bugbot for commit 225541c. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions github-actions Bot added the area/deps Relates to third-party dependencies (area) label Feb 14, 2026
@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented Feb 14, 2026

🤖 Augment PR Summary

Summary: This PR updates the pinned git revision for the Rust crate temporalio-common.

Changes:

  • Bumps temporalio-common from SDK Core rev 231e21c to fb14d9c3 in the workspace dependencies
  • Leaves temporalio-client pinned to the prior revision
  • Updates the lockfile accordingly (Cargo.lock)

Technical Notes: Since both crates come from the same upstream repository, mixing revisions may result in multiple git checkouts/duplicate crate versions unless they are intentionally kept independent.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread Cargo.toml
temporalio-client = { git = "https://github.com/temporalio/sdk-core", rev = "231e21c" }
temporalio-common = { git = "https://github.com/temporalio/sdk-core", rev = "231e21c" }
temporalio-common = { git = "https://github.com/temporalio/sdk-core", rev = "fb14d9c3cf4e9e6d0894fd9bb2985dbf6d6de1fe" }
test-log = { version = "0.2.18", default-features = false }
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

temporalio-common is now pinned to a different git rev than temporalio-client (same upstream repo), which can cause Cargo to pull two distinct copies and potentially lead to type/version mismatches. Is it expected that these two crates are compatible across different commits, or should they be bumped together to the same rev?

Severity: medium

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.31%. Comparing base (7b67b01) to head (225541c).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8421      +/-   ##
==========================================
- Coverage   62.31%   62.31%   -0.01%     
==========================================
  Files        1354     1354              
  Lines      137313   137313              
  Branches     5793     5793              
==========================================
- Hits        85568    85567       -1     
- Misses      50838    50839       +1     
  Partials      907      907              
Flag Coverage Δ
local.claude-hooks 0.00% <ø> (ø)
local.harpc-client 51.24% <ø> (ø)
rust.antsi 0.00% <ø> (ø)
rust.error-stack 90.87% <ø> (ø)
rust.harpc-codec 84.70% <ø> (ø)
rust.harpc-net 96.19% <ø> (-0.02%) ⬇️
rust.harpc-tower 67.03% <ø> (ø)
rust.harpc-types 0.00% <ø> (ø)
rust.harpc-wire-protocol 92.23% <ø> (ø)
rust.hash-codec 72.76% <ø> (ø)
rust.hash-graph-authorization 62.34% <ø> (ø)
rust.hash-graph-temporal-versioning 47.95% <ø> (ø)
rust.hashql-core 82.44% <ø> (ø)
rust.hashql-diagnostics 72.43% <ø> (ø)
rust.hashql-syntax-jexpr 94.06% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Feb 14, 2026

Merging this PR will not alter performance

✅ 56 untouched benchmarks
⏩ 24 skipped benchmarks1


Comparing deps/rs/temporalio-common-digest (225541c) with main (7b67b01)2

Open in CodSpeed

Footnotes

  1. 24 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

  2. No successful run was found on main (0590cf1) during the generation of this report, so 7b67b01 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment thread Cargo.toml Outdated
tarpc = { version = "0.36.0", default-features = false, git = "https://github.com/google/tarpc", rev = "f55f36d2d876b1868cfcf52f41d0456a60cf726c" }
temporalio-client = { git = "https://github.com/temporalio/sdk-core", rev = "231e21c" }
temporalio-common = { git = "https://github.com/temporalio/sdk-core", rev = "231e21c" }
temporalio-common = { git = "https://github.com/temporalio/sdk-core", rev = "12c656f7b2dce2a139bb3f1b4873c7007a751380" }
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Temporal SDK revisions are now inconsistent

Medium Severity

temporalio-client stays on rev 231e21c while temporalio-common is bumped to 12c656f..., so the workspace now resolves two different temporalio-common crates. This can break interoperability where hash-temporal-client uses both crates, because shared SDK types may no longer be identical across crate boundaries.

Additional Locations (1)

Fix in Cursor Fix in Web

@hash-worker hash-worker Bot changed the title Update Rust crate temporalio-common to 21e03c2 Update Rust crate temporalio-common to 2014f31 Feb 19, 2026
@hash-worker hash-worker Bot force-pushed the deps/rs/temporalio-common-digest branch from 133dd13 to 694fa8b Compare February 19, 2026 20:20
@hash-worker hash-worker Bot changed the title Update Rust crate temporalio-common to 2014f31 Update Rust crate temporalio-common to a0caec4 Feb 19, 2026
@hash-worker hash-worker Bot force-pushed the deps/rs/temporalio-common-digest branch from 694fa8b to c7fbc51 Compare February 19, 2026 21:50
@hash-worker hash-worker Bot force-pushed the deps/rs/temporalio-common-digest branch from c7fbc51 to 9ff8edf Compare February 23, 2026 19:36
@hash-worker hash-worker Bot changed the title Update Rust crate temporalio-common to a0caec4 Update Rust crate temporalio-common to 9d678b9 Feb 23, 2026
@hash-worker hash-worker Bot changed the title Update Rust crate temporalio-common to 9d678b9 Update Rust crate temporalio-common to 78a94de Feb 23, 2026
@hash-worker hash-worker Bot force-pushed the deps/rs/temporalio-common-digest branch from 9ff8edf to 34cbc3a Compare February 23, 2026 22:26
@hash-worker hash-worker Bot changed the title Update Rust crate temporalio-common to 78a94de Update Rust crate temporalio-common to 7161ed3 Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/deps Relates to third-party dependencies (area)

Development

Successfully merging this pull request may close these issues.

2 participants