Skip to content

feat: thread AhbContext through format_constraint_evaluation#746

Merged
hf-kklein merged 2 commits into
mainfrom
feature/fc-evaluation-ahb-context
Apr 12, 2026
Merged

feat: thread AhbContext through format_constraint_evaluation#746
hf-kklein merged 2 commits into
mainfrom
feature/fc-evaluation-ahb-context

Conversation

@hf-kklein
Copy link
Copy Markdown
Contributor

Summary

  • PR 4 of the inject removal migration series
  • Adds an optional ahb_context: AhbContext parameter to format_constraint_evaluation() and _build_evaluated_format_constraint_nodes()
  • When ahb_context is provided, uses ahb_context.fc_evaluator and ahb_context.evaluatable_data directly, bypassing both @inject.params and inject.instance(TokenLogicProvider)
  • When ahb_context is None, falls back to the existing inject-based path (full backward compatibility)
  • Passes evaluatable_data explicitly as a keyword argument to bypass the @inject.params decorator (same pattern as PR 2 / feat: thread AhbContext through ConditionNodeBuilder (dual-path) #744)

Test plan

  • All 533 existing tests pass
  • pylint clean (10/10)
  • isort + black formatting verified

🤖 Generated with Claude Code

hf-kklein and others added 2 commits April 12, 2026 12:55
PR 4 of the inject removal migration series.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@hf-kklein hf-kklein merged commit 2f5038b into main Apr 12, 2026
16 of 17 checks passed
@hf-kklein hf-kklein deleted the feature/fc-evaluation-ahb-context branch April 12, 2026 13:34
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