slack-ruby / slack-ruby-client

91%

master: 91%

LAST BUILD ON BRANCH dblock/fix-update-api-copilot-narration

branch: dblock/fix-update-api-copilot-narration

CHANGE BRANCH

x

Reset

Sync Branches

• dblock/fix-update-api-copilot-narration
• 1-item-arg-groups
• 2.5.0-connection-options
• add-code-coverage
• add-group-id-formatting
• add_files_upload_v2
• added-ruby-3.4
• ai-generated-changelog-entries
• async-cache-build-485
• automated-api-update
• calm-down-the-cron
• chat-api-patch
• chore/add-secure-compare
• copilot/sub-pr-578
• dblock/fix-update-api-changelog-entries
• dependabot/github_actions/actions/checkout-4
• dependabot/github_actions/actions/checkout-5
• dependabot/github_actions/actions/checkout-6
• dependabot/github_actions/actions/checkout-7
• dependabot/github_actions/actions/setup-node-7
• dependabot/github_actions/peter-evans/create-pull-request-5
• dependabot/github_actions/peter-evans/create-pull-request-6
• dependabot/github_actions/peter-evans/create-pull-request-7
• dependabot/github_actions/peter-evans/create-pull-request-8
• dependabot/github_actions/tibdex/github-app-token-2.0.0
• dependabot/github_actions/tibdex/github-app-token-2.1.0
• eizengan/efficient_id_transforms
• eizengan/id_all_list_options
• eizengan/id_for_error_classes
• eizengan/nonpublic_channel_translation
• exclude_specs_from_gem_package
• file-upload-blocks
• file_upload_v2
• fix-ai-inference-yaml-indentation
• fix-apps-user-connection-group
• fix-channel-id
• fix-ci
• fix-coveralls
• fix-files-upload-v2-multiple-files
• fix-generator
• fix-hash-inspect-format-drift
• fix-markdown-text-mutual-exclusion
• fix-undocumented-groups
• ignore-danger-gemfile-lock
• include-changelog-in-commit-message
• logger
• master
• migrate-copilot-cli-provider
• migrate-danger-pr-comment
• ng/support-markdown
• oauth-file-upload-samples
• patch-1
• patch-2
• patch-3
• rack-3.0-rewind
• raise-missing-signature
• redact-token
• refs/pull/489/merge
• refs/pull/493/merge
• refs/pull/494/merge
• refs/pull/498/merge
• refs/pull/499/merge
• refs/pull/503/merge
• refs/pull/509/merge
• refs/tags/v2.2.0
• refs/tags/v2.3.0
• remove-rtm
• rename_files_upload_external
• require-faraday-2.0.1
• rid-code-climate
• rs-asterisk-lists
• support_channel_id_s
• translate-channels-file-upload
• v2.4.0
• v2.5.0
• v2.5.2
• v2.6.0
• v2.7.0
• v3.1.0
• v3.2.0

Build # 32801988660

Build Type

Pull #598

github

Committed by dblock

Commit Message

Extract JSON from Copilot CLI response despite narration, fail visibly when missing

Copilot CLI's response file can include narration/thinking lines before
its final JSON answer (observed in run 32801440182), so the previous
fix (#596) silently discarded a perfectly good AI-generated changelog
because the whole file wasn't valid JSON.

Instead, extract the last line that looks like a JSON object and parse
just that. If no JSON object is found at all, fail the workflow
visibly with ::error:: instead of silently falling back to the generic
changelog entry.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Pull Request Pull Request #598: Extract JSON from Copilot CLI response despite narration, fail visibly when missing

Coverage Stats

5330 of 5859 relevant lines covered (90.97%)

99.99 hits per line