Repository navigation
feat(langchain): add human approval prerequisite rule - #110
sairenchristianbuerano merged 2 commits into
Conversation
|
Thanks for this, @dezsow. The rule is schema-clean: every required field is set, One question before merging. LC-101 in this same pack already fires at This needs to land with trustabl/agent-reliability-analyzer#181, which is missing its Happy to merge once the severity question is settled and the engine side is green. |
LC-112 was free when this branch was opened on 24 August. A tracing rule took it in langchain/tracing.yaml during the 217 commits since, so the pack no longer loads with this rule as LC-112. Renumbered to LC-117, the next free id across the whole langchain pack rather than the next free one in agent_safety.yaml alone. The paired analyzer change is on the same branch name, so both sync jobs resolve the pair, and check-rules-sync.sh reports in sync across 130 files.
Adds LC-112, a LangChain v1 create_agent rule for privileged execution tools (ShellTool, PythonREPLTool, PythonAstREPLTool) that lack the structural prerequisites for a resumable human-approval boundary.
The rule reuses existing agent predicates—no schema or evaluator changes—and checks for non-empty middleware plus a checkpointer when privileged execution is exposed. Companion changes in trustabl mirror the fixture and add regression coverage; the rulebook PR documents the threat model, calibration, and limitations.