Repository navigation
Conversation
…use Conversation Configuration naming - The left Refresh button now loads the Memory Store dropdown and both right-hand panels, since they run off the same credentials. Previously each list had its own Refresh and only the dropdown filled. - The Conversation Configuration section shows "Select a Memory Store above to continue" while it's locked, since it stays greyed out until a Memory Store is picked. - The section header, progress step and right-hand panel said "Orchestrator"; they now say Conversation Configuration(s), the resource the wizard actually creates and lists. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
ryanrishi
requested review from
ryanrouleau,
wenzhu1587 and
xinghaohuang91
as code owners
September 29, 2026 15:01
5 of 10 tasks
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The focused page changes correctly implement the described behavior without unresolved issues.
Review effort: Balanced
Findings: None
What changed in this PR
Updates the setup wizard to unify resource refresh behavior and clarify Conversation Configuration terminology.
Changes:
- Refreshes all resource lists from the primary Refresh button.
- Adds guidance when no Memory Store is selected.
- Renames user-facing Orchestrator labels.
| File | Description |
|---|---|
getting_started/twilio_setup/templates/index.html |
Updates refresh behavior, guidance text, and resource labels. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
… time - Restrict Conversation Configuration display names to letters, numbers, `_` and `-`. The SDK's ConversationConfiguration model rejects `.` and `~`, and TAC validates the configuration on startup, so a name using them stopped the examples from starting. - The error panel's close button no longer submits the setup form. - Only one setup run at a time: Memory Store controls are locked and the selected store can't change while a run is in flight, so profile and configuration creation can't target a different store mid-run. - Creating a new Memory Store auto-selects it rather than the previously saved selection the refresh restores. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…scripts - Bind to 127.0.0.1 by default; TAC_SETUP_HOST=0.0.0.0 opts in to remote access. - Inline generated Tailwind CSS in place of the Tailwind CDN script, so no third-party code runs on the credential-entry page. The README documents how to regenerate it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
wenzhu1587
previously approved these changes
Oct 13, 2026
wenzhu1587
force-pushed
the
setup-wizard-refresh-and-naming
branch
from
October 13, 2026 23:14
eba3b57 to
f1b2955
Compare
wenzhu1587
approved these changes
Oct 13, 2026
ryanrouleau
approved these changes
Oct 14, 2026
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Three fixes to the setup wizard page (
getting_started/twilio_setup/templates/index.html):Also, from review on the TypeScript side:
_and-in both the page andserver.py— theConversationConfigurationmodel rejects.and~, andTACvalidates the configuration on startup, so a wizard-created name using them stopped the examples from startingtype="submit"insidesetupForm, so dismissing it could start another setup runserver.pybinds to127.0.0.1by default, since the page takes credentials over plain HTTP;TAC_SETUP_HOST=0.0.0.0opts in to remote accessType of Change
Checklist
SDK Parity
The TypeScript wizard serves a byte-identical copy of this page; the same changes are on that PR.
🤖 Generated with Claude Code