Skip to content

chore(py): track async_include_kwargs removal (no sdk diff) - #412

Closed
chyroc wants to merge 1 commit into
mainfrom
chore/remove-async-include-kwargs-config-sync
Closed

chore(py): track async_include_kwargs removal (no sdk diff)#412
chyroc wants to merge 1 commit into
mainfrom
chore/remove-async-include-kwargs-config-sync

Conversation

@chyroc

@chyroc chyroc commented Feb 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Track sync for coze-sdk-gen removal of async_include_kwargs mapping support.
  • Regeneration against latest generator produced no SDK-file change attributable to this config removal.
  • Keep downstream PR traceability for paired workflow.

Upstream

Validation

  • ./scripts/genpy.sh --output-sdk /tmp/coze-py-qEF1lO --ci-check (executed in codegen workflow)

@chyroc chyroc added the chore Project maintenance work not related to features and bug fixes label Feb 26, 2026
@codecov

codecov Bot commented Feb 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.75%. Comparing base (dfb42f0) to head (7443e09).
⚠️ Report is 1 commits behind head on main.

@@           Coverage Diff           @@
##             main     #412   +/-   ##
=======================================
  Coverage   91.75%   91.75%           
=======================================
  Files         105      105           
  Lines        9542     9542           
=======================================
  Hits         8755     8755           
  Misses        787      787           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

chyroc added a commit to coze-dev/coze-sdk-gen that referenced this pull request Feb 26, 2026
## 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
- coze-py PR: coze-dev/coze-py#412
@chyroc

chyroc commented Feb 26, 2026

Copy link
Copy Markdown
Collaborator Author

Closing per workflow rule: this downstream sync is a no-op (no SDK diff for this config-removal track), so it should be closed instead of merged.

@chyroc chyroc closed this Feb 26, 2026
@chyroc
chyroc deleted the chore/remove-async-include-kwargs-config-sync 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

chore Project maintenance work not related to features and bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant