SDK review
LGTM. The gate is well-designed:
- 7-item starter checklist with tier-aware failure modes (high/medium=hard, low=soft)
- Numeric aliases (1-7) so reviewers can type
/sop-ack 3…
SDK review
LGTM. Fixes sdk/python/ stale path references in both README files. Note: PR #20 (fix/client-module-docstring-inbound-paths) changes the same examples/remote-agent/README.md…
SDK review
LGTM. sdk/python/ doesn't exist in the repo layout — the package lives at molecule-sdk-python/molecule_agent/ directly, so the repo root is the correct sys.path entry. The…
SDK review
LGTM. The CLAUDE.md was misleading — pytest would import but tests using pytest_asyncio would fail with ModuleNotFoundError. The fix correctly documents that `pip install…
SDK review
LGTM. RemoteAgentClient.register_from_env() doesn't exist — the replacement pattern (explicit params + conditional register() if no cached token) is correct. One suggestion…
SDK review
LGTM. The fix is correct — RemoteAgentClient has no register_from_env() class method (it requires explicit workspace_id and platform_url). The replacement pattern is…
SDK review
Same issue as MCP PR #13: REQUIRED_CONTEXTS env var in the workflow yml only passes CI / test (pull_request) but the Python script's default REQUIRED_CONTEXTS_RAW includes…