Flatten scorecard notes to a plain array (correction to #673) - #676
Merged
Merged
Conversation
Intercom-OpenAPI#673 documented notes on the Preview conversation_scorecard using a note.list wrapper. intercom/intercom#580869 removes that wrapper server-side, so notes is now a plain array like its sibling evaluators. Corrects the schema, the conversation_scorecard_note type discriminator, and the three inline response examples to match. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
toquis
approved these changes
Sep 23, 2026
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
2 participants
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.
Why
#673 merged a few hours ago and documented
notes,reviewerandreview_statuson the Previewconversation_scorecard. It modellednotesas anote.listwrapper object. The API is changing that shape:notesbecomes a plain array, matching its sibling fieldevaluatorson the same scorecard entry. The spec still describes the wrapper, so SDKs and the Postman collection would generate the wrong type.This corrects #673 rather than building on it.
What
conversation_scorecard.notesto an array of note objects.typediscriminator fromconversation_scorecard_note.conversation_scorecard_noteswrapper schema.reviewerandreview_statusare unchanged. Preview only — no stable version is touched.Validation
fernisn't installed locally and the repo's CLAUDE.md says not to install it globally, sofern checkwasn't run. Validated manually instead:$refindescriptions/0/api.intercom.io.yaml— 395 distinct targets — and confirmed each resolves to an existing schema, and that none point at the deletedconversation_scorecard_notes.🤖 Generated with Claude Code