fix(rime): remove Arcana model support - #2253
Conversation
🦋 Changeset detectedLatest commit: 1e6af9c The changes in this PR will be included in the next version bump. This PR includes changesets to release 39 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
|
c40bc26 to
1e6af9c
Compare
| if (opts.modelId === 'coda') { | ||
| if (opts.repetition_penalty !== undefined) params.repetition_penalty = opts.repetition_penalty; | ||
| if (opts.temperature !== undefined) params.temperature = opts.temperature; | ||
| if (opts.top_p !== undefined) params.top_p = opts.top_p; | ||
| if (opts.max_tokens !== undefined) params.max_tokens = opts.max_tokens; | ||
| if (opts.timeScaleFactor !== undefined) params.timeScaleFactor = opts.timeScaleFactor; |
There was a problem hiding this comment.
🟡 Voice tuning settings are silently ignored when the older Rime voice model is still selected
Tuning values such as speed, randomness and length limits are dropped from the request (modelParams at plugins/rime/src/tts.ts:92-109) whenever the older arcana model name is passed, so speech is generated with default settings instead of the configured ones and no error is raised.
Impact: Users who still specify the old Rime model get audio that ignores their configured voice tuning, with no warning.
Parameter branch no longer matches the legacy model id
modelId is typed TTSModels | string (plugins/rime/src/tts.ts:56), so 'arcana' is still accepted at runtime even though it was removed from the TTSModels union (plugins/rime/src/models.ts:6). Previously modelParams had an opts.modelId === 'arcana' branch that forwarded repetition_penalty, temperature, top_p, max_tokens and timeScaleFactor. After this change only 'coda' and *mist* are matched, so an arcana request falls through and emits only lang. Those keys are also explicitly excluded from the passthrough loop in fetchPayload (plugins/rime/src/tts.ts:139-144), so they cannot reach the payload by any other route. The result is a silently under-configured request rather than an explicit error, which contradicts the stated intent of keeping the legacy runtime path working during the compatibility window. Either restore the legacy branch or reject the legacy model id with a clear error.
Prompt for agents
In plugins/rime/src/tts.ts, modelParams() previously had a dedicated branch for modelId === 'arcana' that forwarded repetition_penalty, temperature, top_p, max_tokens and timeScaleFactor. That branch was removed and replaced by a 'coda' branch, but modelId is typed `TTSModels | string`, so callers can still pass 'arcana' at runtime. Because those keys are also on the exclusion list in fetchPayload(), an 'arcana' request now silently loses all of its tuning parameters. Decide on an explicit behavior: either keep forwarding the same parameter set for 'arcana' during the compatibility window, or fail fast in resolveOptions() with a clear error telling the caller to switch to 'coda'.
Was this helpful? React with 👍 or 👎 to provide feedback.
There was a problem hiding this comment.
Intentional under the revised scope. This PR now removes Arcana runtime support and directs users to Coda.
Summary
Migration notice: https://docs.rime.ai/docs/arcana-sunset