Hi! I’m using the typescript sdk in which I have a workflow that I want to pass a proto object as arg. This is the error I’m getting: ValueError: Unknown encoding: json/protobuf According to the docs, I guess I need to use PayloadConverter to convert my payload from proto to json. I didn’t mana…

Don’t create a root.ts file, Here we use a root.js file and use the d.ts file generated by pbts: omes/workers/typescript/src/protos/root.js at main · temporalio/omes · GitHub omes/workers/typescript/protogen.js at 3aab035530917612043d7266d94d0508b7bd8440 · temporalio/omes · GitHub

Hey, Please take a look at our docs on the subject: temporalio/sdk-typescript/blob/main/docs/protobuf-libraries.md#current-solution Loren & Roey surveyed the available protobuf libraries in Dec '21 for use with our `ProtobufBinaryDataConverter` and `ProtobufJsonDataConverter`. The …

Thanks for the answer Roey! Regarding the first option, think it’s depracated to some extent, as both pbjs commands aren’t working due to old syntax, and when adjusting them they create a wrong json-module (also if I deploy the project to docker image and run the commands at startup I won’t be able…

[image] liron29: pbjs commands aren’t working due to old syntax What do you mean by deprecated? Regarding the second option What second option? I gave omes as a reference where we use protos with the TypeScript SDK. having a generated proto as args and it worked This may work if the i…

[image] bergundy: sdk-typescript/docs/protobuf-libraries.md at main · temporalio/sdk-typescript · GitHub Thanks! I think I’m starting to get it. Now I’m getting thie error: Cannot read properties of undefined (reading 'constructor') at _patchProtobufRoot My code: Proto file: syntax = "p…

Also, when trying to use the proto message as arg in my workflow I get an error that the type doesn’t exist in my root: Module '"../../protobuf/root"' has no exported member 'io'. Did you mean to use 'import io from "../../protobuf/root"' instead? import { ActivityOptions, proxyActivities } from…

Thanks! would have never though that’s the problem! Now the message is coming in the workflow as expected, but when passed to the activity I get an error: Failed to parse activity args for activity getActivity: Unable to deserialize protobuf message without `root` being provide workflow.ts: imp…

This is happening because root wasn’t properly provided to the payload converter for the activity for some reason: temporalio/sdk-typescript/blob/53aab9e975ea2924ff14d03da9e695b76f71f8bd/packages/common/src/converter/protobuf-payload-converters.ts#L27 const GLOBAL_B…

any reason for it to pass to the workflow but not the activity? should I share some more code?

Workflow code is bundled internally by the SDK and runs in a separate v8 context. Not sure exactly why you’re having issues with activities though. You could try to console.log and figure out why root is undefined. Feel free to also provide a repro and I’ll take a look at it.

Temporal SDK with typescript - json protobuf

Community Support

bergundy January 3, 2024, 6:35pm 7

Don’t create a root.ts file,

Here we use a root.js file and use the d.ts file generated by pbts:

omes/workers/typescript/src/protos/root.js at main · temporalio/omes · GitHub
omes/workers/typescript/protogen.js at 3aab035530917612043d7266d94d0508b7bd8440 · temporalio/omes · GitHub

Topic		Replies	Views
Problem on running cross language workflow Community Support typescript-sdk	9	989	May 28, 2022
Send protobuf payload from Typescript to Java and reverse Community Support java-sdk , general-impl , worker , workflow-implementat	2	1421	January 26, 2023
Protobuf data converter issue Community Support java-sdk	1	1313	July 19, 2020
Using protobuf for communication between java and typescript service Community Support java-sdk	2	730	January 27, 2023
Protos in activities arguments Community Support go-sdk	9	2901	January 14, 2021

Temporal SDK with typescript - json protobuf

Related topics