Skip to content

Yesno - #2945

Open
tevanburen wants to merge 12 commits into
devfrom
yesno
Open

Yesno#2945
tevanburen wants to merge 12 commits into
devfrom
yesno

Conversation

@tevanburen

@tevanburen tevanburen commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Goes with SEMOSS/semoss-ui#3609
The FE PR needs to be merged first

Description

Adds yesno as a new MCPUtility.MCPExecution value, wired into the agent harness's human-in-the-loop pause so yesno tools pause for a decision exactly like ask tools do.

Changes Made

  • MCPUtility.MCPExecution gets a YESNO value, plus a shared isAskLikeExecution(String) helper (ask or yesno)
  • HarnessToolExecutor/SemossAgentHarness use that helper instead of a raw "ask" string check, so yesno tool calls pause and persist to AGENT_RUN_ACTION the same way
  • smssutil.py's MCP-JSON generator now preserves yesno instead of silently coercing it to ask, and its decorator docstrings mention it
  • Doc-comment updates (AbstractReactor, AgentInputRequiredException, AgentRunActionStore) to reflect the 4th value

How to Test

See the FE PR

Notes

  • The frontend still renders yesno with its own approve/reject UI; the backend treats it identically to ask for pause/persistence purposes.
  • The FE PR needs to be merged first

@tevanburen tevanburen self-assigned this Aug 31, 2026
@snyk-io

snyk-io Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@tevanburen tevanburen linked an issue Aug 31, 2026 that may be closed by this pull request
@tevanburen
tevanburen marked this pull request as ready for review September 4, 2026 18:38
@tevanburen
tevanburen requested a review from a team as a code owner September 4, 2026 18:38
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.

Yesno tools

1 participant