Rebuild Text To Speech browser preview#5
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3fe3d7b9bc
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "requiredForPublish": false, | ||
| "requires": [], | ||
| "status": "Hidden", | ||
| "status": "Beta", |
There was a problem hiding this comment.
Use the canonical beta status string
When the registry is normalized, this value is checked against TOOL_STATUS_MODEL, which contains lowercase beta but not Beta; as a result getToolById('text-to-speech') is normalized to status: "Missing Metadata", releaseChannel: "planned", and visibleInToolsList: false. This hides the newly rebuilt tool from the dynamic toolbox/API surfaces even though the PR intends to promote it to a visible beta tool.
Useful? React with 👍 / 👎.
Summary
toolbox/text-to-speech/page as a functional browser Text To Speech tool.tmp/PR_26171_037-text2speech-functional-tool-rebuild_delta.zip.Validation
node --test tests\tools\Text2SpeechShell.test.mjsnpx playwright test tests/playwright/tools/TextToSpeechFunctional.spec.mjs --project=playwright --workers=1 --reporter=listgit diff --checknpm run test:workspace-v2ran but failed in broadRootToolsFutureState.spec.mjsexpectations around root tools/header rendering and127.0.0.1:5501API requests; targeted Text To Speech validation passed.