chore(openai): propagate needs_approval field for tools

[shijiesheng]

What changed?

• in mappers between cadence tools and tools, propagate needs_approval

Why?

needed for tool guard

How did you test it?

Potential risks

Release notes

Documentation Changes

[gitar-bot]

CI failed: The integration test `test_update_spec` failed due to a transient RPC error caused by the Cadence scheduler workflow entering a 'CONTINUED_AS_NEW' state during test execution.

Overview

One integration test failed during the CI run. The failure appears to be a transient issue with the test environment or the Cadence server's handling of scheduled workflows, rather than a direct consequence of the PR changes.

Failures

test_update_spec failure (confidence: medium)

• Type: flaky_test
• Affected jobs: 76847636121
• Related to change: no
• Root cause: The test encountered a CadenceRpcError because the scheduler workflow terminated with a CONTINUED_AS_NEW status while the test was attempting to update the schedule, indicating a potential race condition or state mismatch in the integration environment.
• Suggested fix: Verify the stability of this test on the main branch. Since this is an integration-level environmental issue, consider increasing the test timeout or adding retry logic if the test continues to flake.

Summary

• Change-related failures: 0
• Infrastructure/flaky failures: 1 (integration test flakiness)
• Recommended action: Confirm if this test is flaky on the main branch. If it is known to be flaky, it should be addressed independently of this PR. No changes to the needs_approval implementation are required based on this failure.

Code Review ✅ Approved

Propagates the needs_approval field within tool mappers to enable integration with tool guard. No issues found.

Tip

Comment Gitar fix CI or enable auto-apply: gitar auto-apply:on

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply	Compact
gitar auto-apply:on	gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar}

[codecov]

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
cadence/contrib/openai/cadence_tool.py	0.00%	3 Missing ⚠️

Files with missing lines	Coverage Δ
cadence/contrib/openai/cadence_tool.py	0.00% <0.00%> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[timl3136]

LGTM