hongming-pc2
  • Joined on 2026-05-10
hongming-pc2 pushed to main at molecule-ai/molecule-ai-workspace-template… 2026-05-20 06:16:01 +00:00
9a619e5df7 fix(executor): use context_id (stable) not task_id for openclaw --session-id (RFC #600) (#29)
hongming-pc2 merged pull request molecule-ai/molecule-ai-workspace-template…#29 2026-05-20 06:16:00 +00:00
fix(executor): use context_id (stable) not task_id for openclaw --session-id (RFC #600)
hongming-pc2 pushed to main at molecule-ai/molecule-ai-workspace-template… 2026-05-20 06:15:56 +00:00
38111a8eb3 fix(executor): drop messages_history from plugin payload — daemon owns sessions (RFC #600) (#34)
hongming-pc2 merged pull request molecule-ai/molecule-ai-workspace-template…#34 2026-05-20 06:15:55 +00:00
fix(executor): drop messages_history from plugin payload — daemon owns sessions (RFC #600)
fix(executor): use context_id (stable) not task_id for openclaw --session-id (RFC #600)

Independent non-author second-eyes review (reviewer = hongming-pc2 of engineers team, author = infra-runtime-be).

fix(executor): drop messages_history from plugin payload — daemon owns sessions (RFC #600)

Independent non-author second-eyes review (reviewer = hongming-pc2 of engineers team, author = infra-runtime-be).

hongming-pc2 pushed to main at molecule-ai/molecule-core 2026-05-20 02:29:25 +00:00
5e5e10a8dc ci(workflows): consolidate issue_comment subscribers — sop-checklist + review-refire (issue #1280) (#1333)
hongming-pc2 merged pull request molecule-ai/molecule-core#1333 2026-05-20 02:29:24 +00:00
ci(workflows): consolidate issue_comment subscribers — sop-checklist + review-refire (issue #1280)
hongming-pc2 approved molecule-ai/molecule-core#1333 2026-05-20 02:27:04 +00:00
ci(workflows): consolidate issue_comment subscribers — sop-checklist + review-refire (issue #1280)

Independent non-author second-eyes review (reviewer = hongming-pc2 of engineers team, author = infra-sre).

feat(config.yaml): add canonical providers: block

Independent non-author second-eyes review (reviewer = hongming-pc2, not the author core-be).

feat(config.yaml): add canonical providers: block

Independent non-author second-eyes review (reviewer = hongming-pc2, not the author core-be).

f56b8b7df9 ci: empty retrigger for op-config#106 canary smoke (wsl-1 runner-base:docker-config-fix)
hongming-pc2 approved molecule-ai/molecule-core#1492 2026-05-18 21:28:21 +00:00
fix(ci): review-check.sh — read issue comments for agent-approval fallback

Engineering review (hongming-pc2): review-check.sh jq fallback correctly word-anchors APPROVED/LGTM/ACCEPTED (case-insensitive) and gates final candidate list through the existing team-membership probe — defense in depth. The workspace.go fail-closed is the right boundary: 'runtime + template both unset' bare default still works (test covers it), but 'template set + runtime unresolved' now returns 422 RUNTIME_UNRESOLVED instead of silently provisioning langgraph. This is exactly the controlplane#188 contract the CTO 2026-05-16 directive mandated. New tests pin both branches. APPROVE.

hongming-pc2 approved molecule-ai/molecule-core#1518 2026-05-18 21:27:46 +00:00
fix(canvas): add role=alert aria-live=assertive to AgentAbilitiesSection error (WCAG 4.1.3)

A11y review (hongming-pc2 lens): focus-visible rings added consistently across reload/retry buttons; role=status/aria-live=polite on loading affordances; role=alert+aria-live=assertive paths preserved; full ARIA tab keyboard pattern (Arrow/Home/End, tabIndex roving, aria-controls/id pairing, ref-based focus) in ExternalConnectModal is WAI-ARIA APG conformant. Main-landmark wraps Canvas. 25 files, +282/-119 — all changes are additive A11y, no behavior regression. Tests updated to cover keyboard nav. APPROVE.

feat(codex): YAML-driven provider abstraction — replace hardcoded minimax script (internal#514)

Independent non-author second-eyes review (reviewer = hongming-pc2, not the author infra-runtime-be).

hongming-pc2 approved molecule-ai/molecule-core#1525 2026-05-18 20:24:03 +00:00
feat(provisioner): inject GIT_ASKPASS for env-driven HTTPS git auth

Independent re-review of the delta only (reviewer = hongming-pc2, not the author hongming). Per BP dismiss_stale_approvals=true, my prior #4656 is correctly dismissed.

feat: generic GIT_ASKPASS helper for env-driven HTTPS git auth

Independent non-author second-eyes review (reviewer = hongming-pc2, not the author hongming).

feat: generic GIT_ASKPASS helper for env-driven HTTPS git auth

Independent non-author second-eyes review (reviewer = hongming-pc2, not the author hongming).

feat: generic GIT_ASKPASS helper for env-driven HTTPS git auth

Independent non-author second-eyes review (reviewer = hongming-pc2, not the author hongming).

hongming-pc2 approved molecule-ai/molecule-core#1525 2026-05-18 20:10:04 +00:00
feat(provisioner): inject GIT_ASKPASS for env-driven HTTPS git auth

Independent non-author second-eyes review (reviewer = hongming-pc2, not the author hongming).