Skip to content

[NOT-STREAM] feat(python): return WIT-only resource classes from the caller - #64

Closed
moetemp wants to merge 40 commits into
moe/up/python-clientfrom
moe/up/python-client-wit-resources
Closed

[NOT-STREAM] feat(python): return WIT-only resource classes from the caller#64
moetemp wants to merge 40 commits into
moe/up/python-clientfrom
moe/up/python-client-wit-resources

Conversation

@moetemp

@moetemp moetemp commented Oct 23, 2026

Copy link
Copy Markdown
Owner

What changed?

This PR lets the Python --client caller call an operation that returns a WIT-only resource, or a record that holds one. The caller decodes the JSON answer into the class from _resources.

Why?

The caller refused any operation that returns a resource without a proto wire type, so a WIT-only service with a create got no caller.

How did you test it?

The stream-service sample turns on client = true, and its client.py golden shows create_stream returning CreateStreamResult. A sample test decodes an answer over a fake transport and checks that result.stream is a Stream.

This PR is based on fork PR #54, and it merges fork PR #56 for the stream-service sample.

moetemp added 24 commits October 22, 2026 11:22
@moetemp
Let a result record hold a resource in an own<T> field in Python.
499a886
@moetemp
Built result records that hold a resource in .NET.
c91fa20
@moetemp
Built result records that hold a resource in TypeScript.
80b1e8e
@moetemp
Built result records that hold a resource in Go.
bf43247
@moetemp
Added the workflow-run sample for result records that hold a resource.
0638bbd
@moetemp
Built the returned resource on the TypeScript System Nexus path.
17b80cc
@moetemp
Merge branch 'moe/up/ts-system-nexus-resource-return' into moe/up/res…
b4ae63a 
…ult-record-holds-resource

# Conflicts:
#	src/generator/typescript.rs
@moetemp
Reworded the holder comments and dropped the per-language tests.
cd6ed52
@moetemp
Generated the TypeScript start-workflow sample with System Nexus on.
2847136
@moetemp
Merged the TypeScript System Nexus sample into the result record branch.
a76a0ed
@moetemp
fix(python): skip docstrings when picking model imports (temporalio#266)
d34e03a 
* Skipped docstrings when Python picks the names to import.

* Reworded the changelog line for the docstring import fix.
@moetemp
fix: make resource methods valid for any WIT parameter order (tempora…
935a986 
…lio#268)

* Made Python resource methods valid for any WIT parameter order.

* Took undefined for optional TS params before a required one.

* Showed the keyword-only resource method in the user-service sample.

* Dropped the TypeScript unit test that the user-service golden replaces.
@moetemp
Moved the System Nexus resource example to its own TypeScript sample.
6d18335
@moetemp
Wrapped the workflow-handle README line.
3d5a633
@moetemp
Merged the TypeScript workflow-handle sample.
794ca6a 
# Conflicts:
#	advanced/samples/inputs/examples.toml
@moetemp
Bound resource methods that return a WIT record to their operation.
095326a
@moetemp
Added a stream-service sample with a resource built from a ref.
2399fcb
@moetemp
Gave a .NET resource property a Value suffix when it has the class name.
b0c54ed
@moetemp
Merged the stream-service sample and the .NET property fix.
a5303ba 
# Conflicts:
#	advanced/samples/inputs/examples.toml
#	src/generator/dotnet.rs
#	src/generator/python.rs
#	src/planning/mod.rs
@moetemp
Named the .NET resource constructor parameter after its property.
f44ef7f
@moetemp
Merged the .NET constructor parameter fix.
c2274ed
@moetemp
Let a WIT-only result record hold a resource in an own<T> field.
91274e6
@moetemp
Merged the result record holder and the stream-service sample.
76fda14 
# Conflicts:
#	advanced/samples/inputs/examples.toml
#	src/generator/python.rs
@moetemp
Let the Python caller return WIT-only resource classes.
33c8777
@moetemp moetemp mentioned this pull request Oct 23, 2026
Closed
@moetemp
Gave a .NET resource property a Value suffix when it has the class name.
bad8742
@moetemp
Merged the .NET resource property fix.
f65b580
@moetemp
Generated the stream-service sample for .NET too.
888a889
@moetemp
Merged the stream-service sample stack.
4ec6f17 
# Conflicts:
#	advanced/samples/inputs/examples.toml
@moetemp
Merged the restacked sample and System Nexus branches.
46c635e 
# Conflicts:
#	advanced/samples/dotnet/wit/stream-service/Models.cs
#	advanced/samples/dotnet/wit/stream-service/Operations.cs
#	advanced/samples/dotnet/wit/stream-service/Services.cs
#	advanced/samples/inputs/examples.toml
#	src/generator/dotnet.rs
#	src/planning/mod.rs
moetemp added 11 commits October 22, 2026 18:52
@moetemp
Dropped the stream round-trip test that the user-service session test…
cc06ce1 
… replaces.
@moetemp
Merged the restacked result record branch.
8344d5c
@moetemp
Built the returned resource on the TypeScript System Nexus path.
89d1b95
@moetemp
Generated the TypeScript start-workflow sample with System Nexus on.
c66a3dd
@moetemp
Moved the System Nexus resource example to its own TypeScript sample.
2c3a911
@moetemp
Wrapped the workflow-handle README line.
6f53fef
@moetemp
Generated the workflow-handle sample in Python and .NET too.
bf84b7c
@moetemp
Left the workflow-handle sample out of the .NET sample build.
d9b9b23
@moetemp
Merged the System Nexus resource return fix into the sample stack.
4ba4ea3 
# Conflicts:
#	advanced/samples/inputs/examples.toml
@moetemp
Merged the sample and System Nexus base branch.
941bb8d 
# Conflicts:
#	advanced/samples/README.md
#	advanced/samples/dotnet/Nexgen.DotNetExamples.csproj
#	src/generator/typescript.rs
@moetemp
Merged the restacked sample chain.
19a51a2 
# Conflicts:
#	advanced/samples/inputs/examples.toml
@moetemp moetemp changed the title [UPSTREAM-8] feat(python): return WIT-only resource classes from the caller [NOT-STREAM] feat(python): return WIT-only resource classes from the caller Oct 23, 2026
@moetemp

moetemp commented Oct 23, 2026

Copy link
Copy Markdown
Owner Author

Closed as NOT-STREAM. Streaming resources are proto-backed, and the caller already handles a proto wire type.

@moetemp moetemp closed this Oct 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@moetemp