Skip to content

JVNAUTOSCI-2514 bound recovery prompt context#288

Merged
witbrock merged 1 commit into
mainfrom
JVNAUTOSCI-2514-recovery-prompt-compaction
Jun 14, 2026
Merged

JVNAUTOSCI-2514 bound recovery prompt context#288
witbrock merged 1 commit into
mainfrom
JVNAUTOSCI-2514-recovery-prompt-compaction

Conversation

@witbrock

Copy link
Copy Markdown
Member

Summary

  • Bound the recovery_decision LLM context fields with per-field and total prompt budgets.
  • Drop duplicate/internal runtime telemetry from recovery context while preserving failure evidence and loop-attempt signals.
  • Attach prompt-context compaction diagnostics to LLM call telemetry and step envelopes for follow-up UI surfacing in JVNAUTOSCI-2515.

Validation

  • .venv/bin/python -m pytest tests/backend/test_llm_step_executor.py -q --tb=short
  • .venv/bin/python -m pytest tests/backend/test_turn_execution_workflow_definitions.py -q --tb=short -k 'not test_conversation_turn_recovery_retry_progresses_across_multiple_prompt_targets'
  • .venv/bin/python -m ruff check src/backend/workflows/llm_step_executor.py src/backend/workflows/recovery_prompt_compaction.py tests/backend/test_llm_step_executor.py
  • .venv/bin/pyright src/backend/workflows/llm_step_executor.py src/backend/workflows/recovery_prompt_compaction.py tests/backend/test_llm_step_executor.py
  • .venv/bin/python -m py_compile src/backend/workflows/llm_step_executor.py src/backend/workflows/recovery_prompt_compaction.py tests/backend/test_llm_step_executor.py
  • git diff --check

Note: test_conversation_turn_recovery_retry_progresses_across_multiple_prompt_targets is excluded because it currently fails on origin/main with metadata_validation_failed:metadata_write_context_key_missing:completion_gate:completion_gate_preserved_response.

@witbrock witbrock merged commit 13e16ed into main Jun 14, 2026
4 checks passed
@witbrock witbrock deleted the JVNAUTOSCI-2514-recovery-prompt-compaction branch June 14, 2026 15:01
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