Skip to content

refactor(codegen): remove async_include_kwargs config support - #48

Merged
chyroc merged 1 commit into
mainfrom
refactor/remove-async-include-kwargs-config
Feb 26, 2026
Merged

refactor(codegen): remove async_include_kwargs config support#48
chyroc merged 1 commit into
mainfrom
refactor/remove-async-include-kwargs-config

Conversation

@chyroc

@chyroc chyroc commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove api.operation_mappings[].async_include_kwargs from generator config schema and config/generator.yaml.
  • Remove Python operation rendering branch for mapping-level async kwargs opt-in.
  • Define default behavior as standard kwargs handling without per-operation override.
  • Update generator tests to assert async kwargs presence by default path.

Non-overlap With Existing Open PRs

  • This PR does not overlap with current open config-removal tracks:
    • force_multiline_request_call
    • response_cast
    • response_unwrap_list_first

Validation

  • ./scripts/fmt.sh
  • ./scripts/lint.sh
  • ./scripts/test.sh (Total coverage: 83.2%)
  • ./scripts/build.sh
  • ./scripts/gengo.sh
  • ./scripts/diffgo.sh (zero diff)
  • ./scripts/genpy.sh --output-sdk /tmp/coze-py-qEF1lO --ci-check

Downstream PR

@chyroc
chyroc merged commit 4ac65cd into main Feb 26, 2026
6 checks passed
@chyroc
chyroc deleted the refactor/remove-async-include-kwargs-config branch February 26, 2026 08:34
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