Skip to content

fix(rpc): remove non-array attributes from json array allowlist#7950

Merged
phacops merged 4 commits into
masterfrom
shellmayr/fix/remove-non-array-attributes-from-allowlist
Jun 28, 2026
Merged

fix(rpc): remove non-array attributes from json array allowlist#7950
phacops merged 4 commits into
masterfrom
shellmayr/fix/remove-non-array-attributes-from-allowlist

Conversation

@shellmayr

@shellmayr shellmayr commented May 20, 2026

Copy link
Copy Markdown
Member
  • Remove dict or string style attributes from the list of array processing attributes
    gen_ai.system_instructions: string value
    gen_ai.response.object: not in conventions anymore, seems to be an old attribute - most likely not an array
    gen_ai.tool.call.arguments: dict, not list
    gen_ai.tool.input: dict, not list

shellmayr and others added 4 commits May 20, 2026 10:18
Re-apply after merging master: drop gen_ai.system_instructions,
gen_ai.response.object, gen_ai.tool.call.arguments and gen_ai.tool.input
from ATTRIBUTES_ARRAY_ALLOWLIST (string/dict values, not arrays), keeping
master's workflow_ids/triggered_workflow_ids/action_filter_group_ids/
triggered_action_ids. Point the array-attribute test at workflow_ids,
since gen_ai.tool.input is no longer an allowlisted array path.
@phacops phacops marked this pull request as ready for review June 28, 2026 18:41
@phacops phacops requested review from a team as code owners June 28, 2026 18:41
@phacops phacops enabled auto-merge (squash) June 28, 2026 18:42
@phacops phacops merged commit f4329e5 into master Jun 28, 2026
68 checks passed
@phacops phacops deleted the shellmayr/fix/remove-non-array-attributes-from-allowlist branch June 28, 2026 18:59
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.

2 participants