Skip to content

fix(axios-sdk): 保留已声明路径参数的调用顺序并准备 3.3.1 - #221

Merged
yourtion merged 1 commit into
mainfrom
fix/sdk-declared-parameter-order
Oct 27, 2026
Merged

yourtion merged 1 commit into
mainfrom
fix/sdk-declared-parameter-order

Conversation

@yourtion

Copy link
Copy Markdown
Owner

当路由为 /:first/:second,而 params schema 按 { second, first } 声明时,已有 SDK 的调用方式是 method(second, first)。#219 整合中的路径提取统一把形参改成路由顺序,旧调用会真实请求 /second/first。发布前补查已通过完整生成的 ESM SDK 与 HTTP 请求复现此回归,3.3.0 尚未打 tag 或发布。

本修复保留 params schema 全部已声明字段的原有位置和顺序,再追加未声明的路径参数;所有绑定继续使用避撞规则。OpenAPI 仍只列出路由模板中实际存在的 path parameter,不受 SDK 兼容形参影响。

  • 新增逆序声明、部分声明、额外可选字段三项真实 SDK/HTTP 回归;修复前 3 项失败,修复后 11 项 SDK 参数专项通过。
  • 源码与构建产物测试各 777 项通过,覆盖率语句/函数 99.61%、分支 95.30%;13 个示例、19 个发布工具测试通过。
  • lint/格式、构建、五包真实 pack/干净安装消费、文档链接及 diff 检查通过。
  • 根据仓库公开行为修复的版本规则,五包同步到 3.3.1,登记 MIGRATION。3.3.0 的全部新增能力将随 3.3.1 首次发布。

这是 #195/#219 的发布前兼容性补正,原五个 PR 已按指定顺序合并。本 PR 通过 CI 后合入,再核验最终 main CI/变异基线并使用既有 OIDC 流程发布;Dependabot 配置草稿仍在发布和线上核验完成后单独处理。

@codecov

codecov Bot commented Oct 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.61%. Comparing base (33997d4) to head (ad69957).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #221   +/-   ##
=======================================
  Coverage   99.61%   99.61%           
=======================================
  Files          29       29           
  Lines        3149     3150    +1     
  Branches     1025     1025           
=======================================
+ Hits         3137     3138    +1     
  Misses         12       12           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yourtion
yourtion merged commit 54f6eea into main Oct 27, 2026
9 checks passed
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