Skip to content

feat: enable tool_arg_contract by default (Closes #1530) - #1533

Closed
Lexus2016 wants to merge 1 commit into
mainfrom
evolution/issue-1530-enable-contract
Closed

feat: enable tool_arg_contract by default (Closes #1530)#1533
Lexus2016 wants to merge 1 commit into
mainfrom
evolution/issue-1530-enable-contract

Conversation

@Lexus2016

Copy link
Copy Markdown
Owner

Automated evolution PR for issue #1530.

Flips tool_arg_contract_enabled() default from OFF to ON. All 8 native tools audited safe (#1528). Type-check extension (#1529) already merged.

Changes:

  • agent/tool_arg_contract.py: default ON + is_tool_excluded() escape hatch
  • model_tools.py: call site checks is_tool_excluded()
  • Tests: updated for default-ON, added exclusion tests

Closes #1530

…schemas (#1530)

Flip tool_arg_contract_enabled() default from OFF to ON — all 8 native
tools have been audited (safe schemas with required+type declarations
per #1528). Adds per-tool exclusion list (tool_arg_contract.excluded_tools
in config.yaml) for escape hatch. Updates all tests.

Closes #1530

Co-Authored-By: Hermes Evolution <evolution@hermes.ai>
@Lexus2016

Copy link
Copy Markdown
Owner Author

Closing as duplicate — superseded by #1534 (same issue #1530, #1534 is the newer, clean-green implementation that was merged).

@Lexus2016 Lexus2016 closed this Jul 31, 2026
@Lexus2016
Lexus2016 deleted the evolution/issue-1530-enable-contract branch July 31, 2026 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] Enable tool_arg_contract by default for native tools with safe schemas (#1487 child C)

1 participant