Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

Make code-first user template have same SDK version as in precompiled WASM #320

Merged
afsalthaj merged 2 commits into from
Aug 13, 2025
Merged

Make code-first user template have same SDK version as in precompiled WASM #320

afsalthaj merged 2 commits into from
Aug 13, 2025

Conversation

@afsalthaj
Copy link
Contributor

@afsalthaj afsalthaj commented Aug 13, 2025 •
edited
Loading

  • Previously, the pre-compiled wasm was having latest SDK, but user compiled their code with older version (they were just compatible, and its an external in user code, and therefore worked)
  • Upgrade to latest SDK, and this time, decorator name changed to lower cases, and hence updating it now itself before we proceed with more changes in SDK.

@afsalthaj afsalthaj changed the title Make sure user code-first user template is in sync with the same version of SDK in precompiled WASM, and upgrade SDK Make sure code-first user template is in sync with SDK version in precompiled WASM, and upgrade SDK Aug 13, 2025
afsalthaj
afsalthaj commented Aug 13, 2025
View reviewed changes
golem-templates/templates/ts/ts-app-common/package.json Outdated
"dependencies": {
"@golemcloud/golem-ts": "1.5.0",
"@golemcloud/golem-ts-sdk": "0.0.1-dev.3",
"@golemcloud/golem-ts-sdk": "0.0.1-dev.11",
Copy link
Contributor Author

@afsalthaj afsalthaj Aug 13, 2025

Choose a reason for hiding this comment

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

Didn't know this needs to be changed as well. Somehow if template-golem-agent-ts is changed, this should be in sync as well.

Copy link
Contributor

@noise64 noise64 Aug 13, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

Js and ts templates use the same package json when used together, so these have to be in sync for now, i plan to add dep merging eventually

@afsalthaj afsalthaj changed the title Make sure code-first user template is in sync with SDK version in precompiled WASM, and upgrade SDK Make sure code-first user template is in sync with SDK version in precompiled WASM Aug 13, 2025
@afsalthaj afsalthaj changed the title Make sure code-first user template is in sync with SDK version in precompiled WASM Make code-first user template have same SDK version as in precompiled WASM Aug 13, 2025
@afsalthaj afsalthaj merged commit b9a61d4 into main Aug 13, 2025
4 checks passed
@afsalthaj afsalthaj deleted the update_sdk1 branch August 13, 2025 17:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@noise64 noise64 noise64 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@afsalthaj @noise64