bryanb
/
CheddahBot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Hide clickup_task_id from LLM tool schema to prevent hallucination 

Removed clickup_task_id from write_press_releases function signature
so the LLM cannot see or fabricate a task ID. The parameter is now
passed through ctx by the ToolRegistry — the scheduler sets it in
args, and execute() moves it into the ctx dict before filtering.
Only system-injected task IDs can reach the tool.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
cora-start
PeninsulaInd 2026-02-17 16:38:11 -06:00
parent 082ca6ba44
commit deae147795
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
cheddahbot/tools/__init__.py
View File

@ -152,13 +152,17 @@ class ToolRegistry:
# Inject context if the function expects it # Inject context if the function expects it
sig = inspect.signature(tool_def.func) sig = inspect.signature(tool_def.func)
if "ctx" in sig.parameters: if "ctx" in sig.parameters:
args["ctx"] = { ctx = {
"config": self.config, "config": self.config,
"db": self.db, "db": self.db,
"agent": self.agent, "agent": self.agent,
"memory": self.agent._memory, "memory": self.agent._memory,
"agent_registry": self.agent_registry, "agent_registry": self.agent_registry,
} }
# Pass scheduler-injected metadata through ctx (not LLM-visible)
if "clickup_task_id" in args:
ctx["clickup_task_id"] = args.pop("clickup_task_id")
args["ctx"] = ctx
# Filter args to only params the function accepts (plus **kwargs) # Filter args to only params the function accepts (plus **kwargs)
has_var_keyword = any( has_var_keyword = any(

4
cheddahbot/tools/press_release.py
View File

@ -403,7 +403,6 @@ def write_press_releases(
url: str = "", url: str = "",
lsi_terms: str = "", lsi_terms: str = "",
required_phrase: str = "", required_phrase: str = "",
clickup_task_id: str = "",
ctx: dict | None = None, ctx: dict | None = None,
) -> str: ) -> str:
"""Run the full press-release pipeline and return results + cost summary.""" """Run the full press-release pipeline and return results + cost summary."""
@ -412,6 +411,9 @@ def write_press_releases(
agent = ctx["agent"] agent = ctx["agent"]
# clickup_task_id is injected via ctx by the ToolRegistry (never from LLM)
clickup_task_id = ctx.get("clickup_task_id", "")
# ── ClickUp: set "in progress" and post starting comment ──────────── # ── ClickUp: set "in progress" and post starting comment ────────────
cu_client = None cu_client = None
if clickup_task_id: if clickup_task_id: