Skip to content

Cherry-pick 8d3d742c6: refactor: require canonical talk resolved payload#1278

Merged
alexey-pelykh merged 1 commit intomainfrom
cherry-pick/8d3d742c6-require-canonical-talk-resolved-payload
Mar 14, 2026
Merged

Cherry-pick 8d3d742c6: refactor: require canonical talk resolved payload#1278
alexey-pelykh merged 1 commit intomainfrom
cherry-pick/8d3d742c6-require-canonical-talk-resolved-payload

Conversation

@alexey-pelykh
Copy link
Copy Markdown

Cherry-pick from upstream

Summary

Requires a canonical resolved payload in talk provider config selection. Normalized payloads without a resolved key are now rejected (return nil), simplifying the selection logic. The resolvedProviderConfig() method takes precedence over raw normalized field parsing.

Adaptation

  • Resolved conflicts in TalkConfigParsing.swift to apply upstream's simplified selection logic (resolved-first, reject bare normalized)
  • Removed normalizedTalkProviders() helper (replaced by resolved path)
  • Resolved test conflicts to use upstream's renamed test functions and updated assertions
  • Preserved fork's rebranded imports and @Suite/@testable annotations

Depends on #1277.
Cherry-picked from openclaw/openclaw per #902.

@alexey-pelykh alexey-pelykh merged commit 8a54782 into main Mar 14, 2026
7 checks passed
@alexey-pelykh alexey-pelykh deleted the cherry-pick/8d3d742c6-require-canonical-talk-resolved-payload branch March 14, 2026 05:53
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.

2 participants