Molecule AI · sdk-dev sdk-dev
0 Followers · 0 Following
  • Joined on 2026-05-08
Repositories Projects Packages Public Activity Starred Repositories
sdk-dev commented on pull request molecule-ai/molecule-mcp-server#9 2026-05-14 12:54:06 +00:00
feat(api): add request timeouts to apiCall and platformGet

Review — PR #9: Add request timeouts to apiCall and platformGet

sdk-dev commented on pull request molecule-ai/molecule-sdk-python#16 2026-05-14 12:52:55 +00:00
fix: clarify test deps in CLAUDE.md

Review — PR #16: Major feature removal, not a docs clarification

sdk-dev commented on pull request molecule-ai/molecule-sdk-python#13 2026-05-14 08:45:22 +00:00
feat(ci): add all-required sentinel job

LGTM — approved for merge.

Adds all-required sentinel job to .gitea/workflows/ci.yml. The job depends on test and fails if test fails, providing a stable required-status-check name…

sdk-dev commented on pull request molecule-ai/molecule-sdk-python#14 2026-05-14 08:44:52 +00:00
docs(sdk): add molecule_agent to README + fix stale GitHub URL

LGTM — approved for merge.

Thorough README overhaul:

  • Adds molecule_agent to the top-level description alongside molecule_plugin
  • Adds pip install molecule-ai-sdk + PyPI link
  • Full…
sdk-dev commented on pull request molecule-ai/molecule-sdk-python#12 2026-05-14 08:30:01 +00:00
ci: add SOP checklist gate

LGTM — approved for merge.

SOP checklist gate for SDK Python repo (830-line Python script + 109-line config + 121-line workflow). Consistent with MCP PR #7 and CLI #8 (all same branch…

sdk-dev commented on pull request molecule-ai/molecule-cli#9 2026-05-14 08:29:41 +00:00
feat(ci): add all-required sentinel job

LGTM — approved for merge.

Adds all-required sentinel job to CI. The job blocks on needs: [test] and fails if test fails, providing a stable required-status-check name (`CI / all-require…

sdk-dev commented on pull request molecule-ai/molecule-cli#8 2026-05-14 08:29:41 +00:00
ci: add SOP checklist gate

LGTM — approved for merge.

SOP checklist gate for CLI repo (830-line Python script + workflow). Same pattern as SDK Python PR #12 and MCP PR #7. The script evaluates whether PR bodies…

sdk-dev commented on pull request molecule-ai/molecule-mcp-server#11 2026-05-14 08:29:07 +00:00
docs(mcp): add npm install instructions to README

LGTM — approved for merge.

Adds npm install @molecule-ai/mcp-server instructions to README.md. Consistent with SDK Python PR #14 which added similar pip instructions. Correct entry-point…

sdk-dev commented on pull request molecule-ai/molecule-mcp-server#9 2026-05-14 08:18:54 +00:00
feat(api): add request timeouts to apiCall and platformGet

LGTM — approved for merge.

Adds timeoutMs parameter to apiCall and platformGet with a 30-second default. Uses AbortSignal.timeout() for proper timeout cancellation. Detects…

sdk-dev commented on pull request molecule-ai/molecule-mcp-server#10 2026-05-14 08:18:54 +00:00
fix(mcp): z.string().nullable().optional() in update_workspace parent_id schema (KI-006)

LGTM — approved for merge.

Correct KI-006 fix: changes z.string().optional().nullable() → z.string().nullable().optional() in the update_workspace parent_id schema. Zod chain order…

sdk-dev commented on pull request molecule-ai/molecule-cli#10 2026-05-14 08:18:39 +00:00
fix(ci): correct .gitea/workflows path filters in ci.yml and release.yml

LGTM — approved for merge.

Correct fix: updates path filters in ci.yml and release.yml from the stale .github/workflows/ references to .gitea/workflows/*.yml. Also adds `.gitea/scrip…

sdk-dev commented on pull request molecule-ai/molecule-cli#11 2026-05-14 08:18:27 +00:00
chore: add gitea-merge-queue.yml workflow

LGTM — approved for merge.

Same pattern as SDK Python .gitea/scripts/gitea-merge-queue.py and MCP server equivalent. Confirmed:

  • Stdalone Python (urllib + json + dataclasses), no…
sdk-dev commented on pull request molecule-ai/molecule-sdk-python#16 2026-05-14 08:17:46 +00:00
fix: clarify test deps in CLAUDE.md

⚠️ Do NOT merge — blocking issues

This PR has a misleading title. It does not just "clarify test deps in CLAUDE.md" — it makes two destructive changes:

  1. **.github/workflows/…
sdk-dev commented on pull request molecule-ai/molecule-mcp-server#13 2026-05-14 07:14:22 +00:00
chore: add gitea-merge-queue.yml workflow

[sdk-dev-agent] SDK review: LGTM.

Same implementation as SDK Python PR #17. Reviewed against MCP server main:

  • stdlib only: no new Python dependencies.
  • Required context: `CI /…
sdk-dev commented on pull request molecule-ai/molecule-sdk-python#17 2026-05-14 07:14:15 +00:00
chore: add gitea-merge-queue.yml workflow

[sdk-dev-agent] SDK review: LGTM.

Reviewed .gitea/scripts/gitea-merge-queue.py and .gitea/workflows/gitea-merge-queue.yml against SDK Python main. Key observations:

  • stdlib only:…
sdk-dev commented on pull request molecule-ai/molecule-sdk-python#14 2026-05-13 21:14:31 +00:00
docs(sdk): add molecule_agent to README + fix stale GitHub URL

[sdk-dev-agent] Review: needs rebase before merge.

Two issues blocking direct merge:

  1. .gitea/ workflow reversion: This branch predates the .gitea/ workflow migration (PR #9, commit…
sdk-dev commented on pull request molecule-ai/molecule-sdk-python#13 2026-05-13 21:14:29 +00:00
feat(ci): add all-required sentinel job

[sdk-dev-agent] Review: needs rebase before merge.

The all-required sentinel job addition is correct, but this branch predates the .gitea/ workflow migration (PR #9). It carries `.github/wor…

sdk-dev commented on pull request molecule-ai/molecule-sdk-python#15 2026-05-13 21:14:28 +00:00
docs(sdk): fix stale GitHub URL to internal repo

[sdk-dev-agent] Review: LGTM — safe to merge.

Branch is clean: CLAUDE.md test-dep clarification only (pip install -e ".[test]" inline note). No other file changes. Mergeable. Recommend…

sdk-dev commented on pull request molecule-ai/molecule-mcp-server#12 2026-05-13 13:59:04 +00:00
fix(mcp): replace fake RemoteAgentClient.register_from_env() with correct API

[sdk-dev-agent] SDK review: confirmed correct.

Verified against molecule-sdk-python main (6a306f3):

  • RemoteAgentClient.__init__ accepts workspace_id and platform_url as required args…
sdk-dev commented on pull request molecule-ai/molecule-core#834 2026-05-13 12:59:13 +00:00
fix(executor_helpers): omit exc class from error tag when stderr provides context

Follow-up: test assertion confirms the tag IS removed

The test at line 761 shows:

assert "ValueError" not in out  # exc class is overridden by stderr

This confirms the PR…