fix(openai): merge ExtraFields in WithExtraFields instead of overwriting

[wucm667]

Fixes #671

When multiple WithExtraFields options are passed, later calls overwrite earlier fields instead of merging. This prevents combining framework-provided config (e.g., EnableThinking) with user-provided fields (e.g., enable_search).

Changes

libs/acl/openai/option.go — Changed WithExtraFields to merge fields into o.ExtraFields instead of direct assignment:

// Before (buggy)
o.ExtraFields = extraFields

// After (fixed)
if o.ExtraFields == nil {
    o.ExtraFields = make(map[string]any, len(extraFields))
}
for k, v := range extraFields {
    o.ExtraFields[k] = v
}

libs/acl/openai/option_test.go — Added test case "WithExtraFields merges on multiple calls" verifying that two consecutive calls produce merged fields.

Files Changed

• libs/acl/openai/option.go
• libs/acl/openai/option_test.go

[wucm667]

Hi! The check-submodule-changes CI job is failing due to a 403 Resource not accessible by integration error when trying to post a comment. This appears to be a permissions issue with the GITHUB_TOKEN in the workflow — it needs issues: write or pull_requests: write permission to create comments on PRs.

All actual tests pass successfully:
✅ unit-test
✅ unit-benchmark-test
✅ compliant
✅ license/cla

Could someone with repo admin access check the workflow permissions? Thanks!