Skip to content

docs: note keyphrase uniqueness rule - #310

Merged
Ruari-Phipps merged 1 commit into
mainfrom
docs/auto-update-21692da
Sep 3, 2026
Merged

docs: note keyphrase uniqueness rule#310
Ruari-Phipps merged 1 commit into
mainfrom
docs/auto-update-21692da

Conversation

@Ruari-Phipps

@Ruari-Phipps Ruari-Phipps commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

Document the keyphrase uniqueness rule on the speech recognition reference page, so the validation added in 21692da is discoverable before a push fails.

Motivation

Agent Studio normalizes keyphrases (trim + lowercase) and rejects a push containing two entries that normalize to the same phrase. Nothing in the docs said so, so the constraint was only discoverable by hitting the error.

Changes

  • Add a Validation section to the speech recognition page, covering the keyphrase uniqueness rule along with the existing rules for ASR settings, keyphrase boosting, and transcript corrections
  • Add a matching bullet to the speech recognition best practices list

Test strategy

  • Added/updated unit tests
  • Manual CLI testing (poly <command>)
  • Tested against a live Agent Studio project
  • N/A (docs, config, or trivial change)

Checklist

  • ruff check . and ruff format --check . pass
  • pytest passes
  • No breaking changes to the poly CLI interface (or migration path documented)
  • Commit messages follow conventional commits

Screenshots / Logs

N/A

@Ruari-Phipps
Ruari-Phipps force-pushed the docs/auto-update-21692da branch from 882ccbe to 38f4084 Compare September 3, 2026 15:51
@Ruari-Phipps Ruari-Phipps changed the title docs: auto-update from 21692da docs: note keyphrase uniqueness rule Sep 3, 2026
@Ruari-Phipps
Ruari-Phipps force-pushed the docs/auto-update-21692da branch from 38f4084 to ff5fda7 Compare September 3, 2026 16:34
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@Ruari-Phipps
Ruari-Phipps force-pushed the docs/auto-update-21692da branch from ff5fda7 to 24000f7 Compare September 3, 2026 16:38
@Ruari-Phipps
Ruari-Phipps marked this pull request as ready for review September 3, 2026 16:39
@Ruari-Phipps
Ruari-Phipps requested review from a team as code owners September 3, 2026 16:40
@Ruari-Phipps
Ruari-Phipps merged commit bba4850 into main Sep 3, 2026
6 checks passed
@Ruari-Phipps
Ruari-Phipps deleted the docs/auto-update-21692da branch September 3, 2026 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant