molecule-ai/molecule-core
52
0
Fork 2
Code Issues 179 Pull Requests 50 Actions Packages Projects Releases Wiki Activity

fix(e2e): use hermes-registered model in canvas staging setup (#2225) #2230

Closed
core-be wants to merge 1 commits from fix/2225-e2e-canvas-stale-hermes-model into main
pull from: fix/2225-e2e-canvas-stale-hermes-model
merge into: molecule-ai:main
Conversation 1 Commits 1 Files Changed 1
+2 -2
core-be commented 2026-06-04 08:40:29 +00:00
Member
Copy Link
Copy Source

Summary

Change

  • model: gpt-4o → moonshot/kimi-k2.6 (listed in providers.yaml:778 under hermes native set).
  • Updates the inline comment that references the old model pair.

Why this model

  • moonshot/kimi-k2.6 is a platform-managed model already used by other hermes fixtures after the P4 reconcile (see eacb8183 workspace_test.go updates).
  • The workspace still boots into the pre-start credential-abort shape (no CP proxy env on staging), which the harness already tolerates as RENDERABLE per #2199 — the tab-UI test does not exercise the agent.

Test plan

  • Re-run E2E Staging Canvas on this PR (paths filter will trigger because canvas/e2e changed).
  • Verify workspace create returns 201, not 422.
  • Verify Canvas tabs spec proceeds to tab-rendering assertions.
## Summary - Closes #2225 - Fixes the `E2E Staging Canvas (Playwright)` main-red failure introduced when provider-registry SSOT (internal#718 P4, commit eacb8183) flipped the `(runtime, model)` gate from WARN to hard-reject 422. - `canvas/e2e/staging-setup.ts` was still requesting `runtime: hermes, model: gpt-4o`, which is no longer in the hermes native set. ## Change - `model`: `gpt-4o` → `moonshot/kimi-k2.6` (listed in `providers.yaml:778` under hermes native set). - Updates the inline comment that references the old model pair. ## Why this model - `moonshot/kimi-k2.6` is a platform-managed model already used by other hermes fixtures after the P4 reconcile (see eacb8183 workspace_test.go updates). - The workspace still boots into the pre-start credential-abort shape (no CP proxy env on staging), which the harness already tolerates as RENDERABLE per #2199 — the tab-UI test does not exercise the agent. ## Test plan - [ ] Re-run `E2E Staging Canvas` on this PR (paths filter will trigger because canvas/e2e changed). - [ ] Verify workspace create returns 201, not 422. - [ ] Verify Canvas tabs spec proceeds to tab-rendering assertions.
core-be added 1 commit 2026-06-04 08:40:33 +00:00
fix(e2e): use hermes-registered model in canvas staging setup (#2225)
sop-tier-check / tier-check (pull_request_target) Successful in 5s
Details
ci-arm64-advisory / fast-checks (pull_request) Waiting to run
Details
CI / Python Lint & Test (pull_request) Successful in 6s
Details
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 8s
Details
CI / Detect changes (pull_request) Successful in 8s
Details
Lint forbidden tenant-env keys / Scan for repo-host token write into tenant workspace surface (pull_request) Successful in 5s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 8s
Details
Lint forbidden tenant-env keys / Scan workspace_secrets writers for forbidden env keys (pull_request) Successful in 6s
Details
E2E Chat / detect-changes (pull_request) Successful in 9s
Details
Harness Replays / detect-changes (pull_request) Successful in 9s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 11s
Details
CI / Platform (Go) (pull_request) Successful in 2s
Details
sop-checklist / review-refire (pull_request_target) Has been skipped
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 2s
Details
E2E Chat / E2E Chat (pull_request) Successful in 3s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 14s
Details
Harness Replays / Harness Replays (pull_request) Successful in 2s
Details
qa-review / approved (pull_request_target) Failing after 7s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 3s
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 7s
Details
sop-checklist / all-items-acked (pull_request) acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: comprehensive-testing, local-postgres-e2
Details
sop-checklist / na-declarations (pull_request) N/A: (none)
Details
security-review / approved (pull_request_target) Failing after 7s
Details
sop-checklist / all-items-acked (pull_request_target) Successful in 6s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 2s
Details
Lint shellcheck (arm64 pilot) / shellcheck-arm64 (pilot) (pull_request) Successful in 14s
Details
gate-check-v3 / gate-check (pull_request_target) Failing after 16s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 21s
Details
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m16s
Details
audit-force-merge / audit (pull_request_target) Has been skipped
Details
CI / Canvas (Next.js) (pull_request) Successful in 6m16s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
CI / all-required (pull_request) Successful in 2s
Details
2576b56142 
The provider-registry SSOT (internal#718 P4) hard-rejects unregistered
(runtime, model) pairs with 422 UNREGISTERED_MODEL_FOR_RUNTIME.
canvas/e2e/staging-setup.ts was still creating a workspace with
runtime=hermes + model=gpt-4o, which is no longer in the hermes native
set after the P4 flip (eacb8183). This caused every E2E Staging Canvas
run that passed the paths filter to fail at workspace creation.

Update the fixture to model=moonshot/kimi-k2.6, a model listed in the
hermes native provider set (providers.yaml:778). The workspace still
boots into the pre-start credential-abort shape (platform_managed without
CP proxy env on staging) which the harness already tolerates as
RENDERABLE per #2199 — the tab-UI test does not exercise the agent.

Closes #2225.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
core-be commented 2026-06-04 08:54:39 +00:00
Author
Member
Copy Link
Copy Source

Superseded by commit 6e682444 (PR #2229) which merged the same model-fix plus skip-if-token-absent onto main. Closing to avoid duplicate open PRs.

Superseded by commit `6e682444` (PR #2229) which merged the same model-fix plus skip-if-token-absent onto main. Closing to avoid duplicate open PRs.
core-be closed this pull request 2026-06-04 08:54:40 +00:00
Some checks are pending
sop-tier-check / tier-check (pull_request_target) Successful in 5s
Details
ci-arm64-advisory / fast-checks (pull_request) Waiting to run
Details
CI / Python Lint & Test (pull_request) Successful in 6s
Details
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 8s
Details
CI / Detect changes (pull_request) Successful in 8s
Details
Lint forbidden tenant-env keys / Scan for repo-host token write into tenant workspace surface (pull_request) Successful in 5s
Details
E2E API Smoke Test / detect-changes (pull_request) Successful in 8s
Details
Lint forbidden tenant-env keys / Scan workspace_secrets writers for forbidden env keys (pull_request) Successful in 6s
Details
E2E Chat / detect-changes (pull_request) Successful in 9s
Details
Harness Replays / detect-changes (pull_request) Successful in 9s
Details
Handlers Postgres Integration / detect-changes (pull_request) Successful in 11s
Details
CI / Platform (Go) (pull_request) Successful in 2s
Details
sop-checklist / review-refire (pull_request_target) Has been skipped
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 2s
Required
Details
E2E Chat / E2E Chat (pull_request) Successful in 3s
Details
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 14s
Details
Harness Replays / Harness Replays (pull_request) Successful in 2s
Details
qa-review / approved (pull_request_target) Failing after 7s
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 3s
Required
Details
CI / Shellcheck (E2E scripts) (pull_request) Successful in 7s
Details
sop-checklist / all-items-acked (pull_request) acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: comprehensive-testing, local-postgres-e2
Details
sop-checklist / na-declarations (pull_request) N/A: (none)
Details
security-review / approved (pull_request_target) Failing after 7s
Details
sop-checklist / all-items-acked (pull_request_target) Successful in 6s
Details
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 2s
Details
Lint shellcheck (arm64 pilot) / shellcheck-arm64 (pilot) (pull_request) Successful in 14s
Details
gate-check-v3 / gate-check (pull_request_target) Failing after 16s
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 21s
Details
lint-required-no-paths / lint-required-no-paths (pull_request) Successful in 1m16s
Details
audit-force-merge / audit (pull_request_target) Has been skipped
Details
CI / Canvas (Next.js) (pull_request) Successful in 6m16s
Details
CI / Canvas Deploy Reminder (pull_request) Has been skipped
Details
CI / all-required (pull_request) Successful in 2s
Required
Details
E2E Peer Visibility (literal MCP list_peers) / E2E Peer Visibility (pull_request)
Required

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
area/ci
CI/CD pipeline issues
 do-not-auto-merge
Opt out of autonomous merge-queue merging
 kind/infrastructure
Infrastructure-related issues
 merge-queue
Ready for serialized Gitea merge queue
 merge-queue-hold
Temporarily hold PR in merge queue
 platform/go
Go platform test issues
 release-blocker
Blocks the staging→main promotion / a release
 release-test
security
test-label-sre
tier:high
High risk per dev-sop §SOP-6 — ceo only, 24h cooldown
 tier:low
Low risk per dev-sop §SOP-6 — engineers/managers/ceo can approve
 tier:medium
Medium risk per dev-sop §SOP-6 — managers/ceo can approve
 triage-test
test
 wip
Work in progress — do not auto-merge
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent-dev-a agent-dev-b agent-pm agent-researcher agent-reviewer agent-reviewer-1 agent-reviewer-cr2 app-fe (Molecule AI · app-fe) app-lead (Molecule AI · app-lead) app-qa (Molecule AI · app-qa) claude-ceo-assistant claude-ci-reader claude-status-reaper core-be (Molecule AI · core-be) core-devops (Molecule AI · core-devops) core-fe (Molecule AI · core-fe) core-lead (Molecule AI · core-lead) core-offsec (Molecule AI · core-offsec) core-qa (Molecule AI · core-qa) core-security (Molecule AI · core-security) core-uiux (Molecule AI · core-uiux) cp-be (Molecule AI · cp-be) cp-lead (Molecule AI · cp-lead) cp-qa (Molecule AI · cp-qa) cp-security (Molecule AI · cp-security) cui (Zhanlin Cui) dev-lead (Molecule AI · dev-lead) devops-engineer documentation-specialist (Molecule AI · documentation-specialist) fullstack-engineer (Molecule AI · fullstack-engineer) godwin hongming hongming-ceo-delegated hongming-codex-laptop hongming-kimi-laptop hongming-pc2 infra-lead (Molecule AI · infra-lead) infra-runtime-be (Molecule AI · infra-runtime-be) infra-sre (Molecule AI · infra-sre) integration-tester (Molecule AI · integration-tester) mc-drift-bot molecule-code-reviewer plugin-dev (Molecule AI · plugin-dev) pm publish-runtime-bot pypi-publisher (Molecule AI PyPI Publisher (RFC#596)) release-manager (Molecule AI · release-manager) sdk-dev (Molecule AI · sdk-dev) sdk-lead (Molecule AI · sdk-lead) sop-drift-bot sop-tier-bot (SOP Tier-Check Bot) technical-writer (Molecule AI · technical-writer) triage-operator (Molecule AI · triage-operator)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/molecule-core#2230
Delete Branch "fix/2225-e2e-canvas-stale-hermes-model"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?