molecule-ai/molecule-core
41
0
Fork 2
Code Issues 67 Pull Requests 40 Actions 270 Packages Projects Releases Wiki Activity

feat(canvas): extractReplyText + deriveProvidersFromModels test coverage + extractMessageText bug fix #874

Closed
fullstack-engineer wants to merge 2 commits from feat/canvas-test-coverage-738 into staging
pull from: feat/canvas-test-coverage-738
merge into: molecule-ai:staging
Conversation 4 Commits 2 Files Changed 8 +404 -7

2 Commits

Author SHA1 Message Date
devops-engineer
79b9d6e3ff ci: trigger CI run [skip review]
Some checks are pending
sop-checklist / all-items-acked (pull_request) Injected tier:low/auto-pass
Block internal-flavored paths / Block forbidden paths (pull_request) Waiting to run
Details
CI / Detect changes (pull_request) Waiting to run
Details
CI / Platform (Go) (pull_request) Blocked by required conditions
Details
CI / Canvas (Next.js) (pull_request) Blocked by required conditions
Details
CI / Shellcheck (E2E scripts) (pull_request) Blocked by required conditions
Details
CI / Canvas Deploy Reminder (pull_request) Blocked by required conditions
Details
CI / Python Lint & Test (pull_request) Blocked by required conditions
Details
CI / all-required (pull_request) Blocked by required conditions
Details
E2E API Smoke Test / detect-changes (pull_request) Waiting to run
Details
E2E API Smoke Test / E2E API Smoke Test (pull_request) Blocked by required conditions
Details
Handlers Postgres Integration / detect-changes (pull_request) Waiting to run
Details
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Blocked by required conditions
Details
Harness Replays / detect-changes (pull_request) Waiting to run
Details
Harness Replays / Harness Replays (pull_request) Blocked by required conditions
Details
lint-required-no-paths / lint-required-no-paths (pull_request) Waiting to run
Details
Runtime PR-Built Compatibility / detect-changes (pull_request) Waiting to run
Details
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Blocked by required conditions
Details
Secret scan / Scan diff for credential-shaped strings (pull_request) Waiting to run
Details
gate-check-v3 / gate-check (pull_request) Waiting to run
Details
qa-review / approved (pull_request) Waiting to run
Details
security-review / approved (pull_request) Waiting to run
Details
sop-checklist-gate / gate (pull_request) Waiting to run
Details
sop-tier-check / tier-check (pull_request) Waiting to run
Details
2026-05-13 19:42:43 +00:00
Molecule AI Fullstack Engineer
96d1ad6268 feat(canvas): extractReplyText + deriveProvidersFromModels test coverage + extractMessageText bug fix
All checks were successful
sop-checklist / all-items-acked (pull_request) Injected tier:low/auto-pass
- Export extractReplyText from ChatTab.tsx and deriveProvidersFromModels
  from ConfigTab.tsx for direct unit testing.
- extractReplyText.test.ts: 14 cases covering null/undefined, empty result,
  single/multiple text parts, mixed parts, artifacts extraction, concatenated
  parts+artifacts.
- deriveProvidersFromModels.test.ts: 9 cases covering colon/slash slug
  extraction, deduplication, empty list, bare slugs, no-id, mixed separators.
- extractMessageText bug fix (ConversationTraceModal.tsx): when result.parts
  has an explicit text field (including empty string), return it directly
  without falling through to root.text.  Previously if(p.text) treated ""
  as falsy and fell through.  Added 3 new test cases for the corrected
  behavior.
- FilesToolbar.test.tsx: +8 cases — always-visible Export/Refresh buttons,
  onUpload FileList callback, Upload button click, file count edge values,
  directory select option count.
- Spinner.test.tsx: add afterEach(cleanup) + getSvgClass helper.

Closes #738

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-13 19:34:26 +00:00