Skip to content

fix: resolve FINAL_VAR to actual variable value instead of returning … - #10

Closed
XiaoConstantine wants to merge 1 commit into
alexzhang13:mainfrom
XiaoConstantine:fix/ignore-final-with-code-blocks
Closed

XiaoConstantine wants to merge 1 commit into
alexzhang13:mainfrom
XiaoConstantine:fix/ignore-final-with-code-blocks

Conversation

@XiaoConstantine

Copy link
Copy Markdown
Contributor

…tuple

The find_final_answer() function returns a tuple ("FINAL_VAR", "var_name"), but the caller was passing this tuple directly as the final answer instead of resolving the variable from environment.locals.

  • Refactor check_for_final_answer() to take env_locals dict directly
  • Remove broken logger.log_tool_execution() dependency (method didn't exist)
  • Use check_for_final_answer() in rlm.py to properly resolve FINAL_VAR

…tuple

The find_final_answer() function returns a tuple ("FINAL_VAR", "var_name"),
but the caller was passing this tuple directly as the final answer instead
of resolving the variable from environment.locals.

- Refactor check_for_final_answer() to take env_locals dict directly
- Remove broken logger.log_tool_execution() dependency (method didn't exist)
- Use check_for_final_answer() in rlm.py to properly resolve FINAL_VAR
william-c-stanford referenced this pull request in akramlodi/rlm_self_harness Aug 9, 2026
…ail-fast, env stripping

SIGALRM hard-deadline backstop around each governed run slice so a
candidate callable that hangs mid-call terminates, persists as
RESOURCE_TERMINATED, and is charged by the breaker (#6). Host-side
materialization failures after subprocess vetting now raise loudly
instead of masquerading as candidate rejections (#7). The gate
subprocess runs candidate code under an allowlisted environment and
re-scrubs dotenv-loaded secrets after imports (#10).

Claude-Session: https://claude.ai/code/session_01QfVmUyopDjKxK3b4nWxKWM
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.

1 participant