Skip to content

fix(core): detect bundled SDK via version stamp - #717

Open
nvasiu wants to merge 3 commits into
mainfrom
feat/bundled-stamp-detection
Open

nvasiu wants to merge 3 commits into
mainfrom
feat/bundled-stamp-detection

Conversation

@nvasiu

@nvasiu nvasiu commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Issue #, if available:

N/A

Description of changes:

The managed runtime now stamps a +bundled local version label onto the installed durable SDK. This update reads the stamp to determine if the SDK is installed in the managed runtime (replacing the old method of checking the installation folder, which was unreliable).

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@nvasiu
nvasiu deployed to ai-pr-review-runtime September 10, 2026 18:04 — with GitHub Actions Active
@nvasiu
nvasiu deployed to ai-pr-review-runtime September 10, 2026 18:12 — with GitHub Actions Active
@github-actions

This comment has been minimized.

@nvasiu
nvasiu force-pushed the feat/bundled-stamp-detection branch from 7469710 to f30b5b7 Compare September 15, 2026 22:39
@nvasiu
nvasiu deployed to ai-pr-review-runtime September 15, 2026 22:45 — with GitHub Actions Active
@github-actions

This comment has been minimized.

@nvasiu
nvasiu force-pushed the feat/bundled-stamp-detection branch from f30b5b7 to bc34fea Compare September 15, 2026 23:12
@nvasiu
nvasiu deployed to ai-pr-review-runtime September 15, 2026 23:13 — with GitHub Actions Active
@github-actions

This comment has been minimized.

Comment thread packages/aws-durable-execution-sdk-python/tests/lambda_service_test.py Outdated
@nvasiu
nvasiu force-pushed the feat/bundled-stamp-detection branch from bc34fea to 73e21a2 Compare September 16, 2026 17:14
@nvasiu
nvasiu deployed to ai-pr-review-runtime September 16, 2026 17:14 — with GitHub Actions Active
@github-actions

This comment has been minimized.

@nvasiu
nvasiu force-pushed the feat/bundled-stamp-detection branch from 73e21a2 to 83ce7a5 Compare September 16, 2026 23:43
@nvasiu
nvasiu deployed to ai-pr-review-runtime September 16, 2026 23:43 — with GitHub Actions Active
@github-actions

This comment has been minimized.

@zhongkechen
zhongkechen deployed to ai-pr-review-runtime October 16, 2026 19:09 — with GitHub Actions Active
@github-actions

This comment has been minimized.

@zhongkechen
zhongkechen deployed to ai-pr-review-runtime October 16, 2026 19:53 — with GitHub Actions Active
@github-actions

Copy link
Copy Markdown
Contributor

Codex AI review

No actionable findings. Both prior findings are resolved. Residual test risk: review was static; tests were not executed under the review constraints.

Reviewed commit 509340cbe3b417675aa1fdb5105c3299aaa02a57. Workflow run

This branch was successfully deployed

1 active deployment
ai-pr-review-runtime — 509340cb Deployed Oct 16, 2026 by zhongkechen via ai-pr-review / Codex review / Generate Codex review #1813
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants