Skip to content

fix(pat): carry agentCode in batch auth args#446

Open
shangguanxuan633-lab wants to merge 5 commits into
DingTalk-Real-AI:mainfrom
shangguanxuan633-lab:codex/pat-batch-agentcode-main
Open

fix(pat): carry agentCode in batch auth args#446
shangguanxuan633-lab wants to merge 5 commits into
DingTalk-Real-AI:mainfrom
shangguanxuan633-lab:codex/pat-batch-agentcode-main

Conversation

@shangguanxuan633-lab

Copy link
Copy Markdown
Contributor

Summary

  • Carry agentCode and sessionId in pat.batch_plan / pat.batch_grant arguments.
  • Add DWS_DINGTALK_AGENTCODE as a compatibility alias while keeping DINGTALK_DWS_AGENTCODE priority.
  • Keep backward compatibility with older gateways that reject identity args by retrying without identity args while preserving env context.

Tests

  • go test ./internal/pat ./internal/auth
  • go test ./internal/app -run 'TestResolveIdentityHeadersForwards(Compat)?AgentCode|TestDefaultPATServerDescriptorUsesBehaviorAuthorizationName|TestDirectRuntimeEndpoint_DefaultPATFallback|TestDirectRuntimeEndpoint_PATDiscoveryOverride'\n- go test ./test/unit -run 'TestHostOwnsPATFlow_OnlySignal|TestPATStderrJSON|TestPATStderrJSON_CodeEnumFrozen'\n\n## Notes\nBranch is forked from DingTalk-Real-AI/dingtalk-workspace-cli:main; unrelated feat/dws-devapp PR was closed.

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