forked from molecule-ai/molecule-core
bdd1d09dfb
PR #2545 self-review findings. (1) originalModel was set from wsMetadataModel alone. On a hermes/pre-#240 workspace where MODEL_PROVIDER was never written but YAML has runtime_config.model: "something", originalModel="" while the form rendered "something" — handleSave's diff fired /model PUT on every unrelated save (tier change → workspace auto-restart). Snapshot from the actual rendered model in BOTH loadConfig branches so the diff stays scoped to user-initiated changes. (2) The store-flush test asserted the call happened but didn't pin success-gating. A future refactor wrapping the PATCH in try/catch and unconditionally calling updateNodeData would have shipped green and left the badge lying about server-rejected writes. New test pins the PATCH-rejects-no-flush invariant. (3) Hermes-edge regression test for (1). All 1214 canvas tests pass. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| e2e | ||
| public | ||
| src | ||
| .env.example | ||
| .gitignore | ||
| components.json | ||
| Dockerfile | ||
| next.config.ts | ||
| package-lock.json | ||
| package.json | ||
| playwright.config.ts | ||
| playwright.staging.config.ts | ||
| postcss.config.js | ||
| tailwind.config.ts | ||
| tsconfig.json | ||
| vitest.config.ts | ||