Show RetryPolicy import in Python getting-started#104
Open
donald-pinckney wants to merge 1 commit intomainfrom
Open
Show RetryPolicy import in Python getting-started#104donald-pinckney wants to merge 1 commit intomainfrom
donald-pinckney wants to merge 1 commit intomainfrom
Conversation
The Python getting-started example only used symbols from temporalio.workflow, which led agents to hallucinate workflow.RetryPolicy when adding retries. Add retry_policy to the execute_activity example with the correct temporalio.common import, plus a note clarifying which types live where. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
brianstrauch
approved these changes
May 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
workflow.RetryPolicy(...), which fails withmodule 'temporalio.workflow' has no attribute 'RetryPolicy'.references/python/python.md(the entry-point file SKILL.md tells agents to read first) only ever shows symbols from theworkflowmodule —workflow.defn,workflow.run,workflow.execute_activity,workflow.logger, etc. — and never showsretry_policy=or thetemporalio.commonimport. Agents pattern-match "Temporal things live onworkflow.*" and hallucinate the namespace when they need a retry policy.execute_activityexample with aretry_policy=RetryPolicy(...)argument and the correctfrom temporalio.common import RetryPolicyimport, plus a one-line note that configuration types (RetryPolicy,SearchAttributeKey,VersioningBehavior, ...) live intemporalio.common, not on theworkflowmodule.Test plan
workflow.RetryPolicyand confirm it now importsRetryPolicyfromtemporalio.common.temporalioSDK.🤖 Generated with Claude Code