Skip to content

[codex] Fix section context delivery#15

Merged
sukar0972 merged 1 commit into
mainfrom
codex/fix-section-context-delivery
Jul 4, 2026
Merged

[codex] Fix section context delivery#15
sukar0972 merged 1 commit into
mainfrom
codex/fix-section-context-delivery

Conversation

@sukar0972

Copy link
Copy Markdown
Owner

Summary

  • make section context injection depend on the turn-start message, not the total user-message count in the current projection
  • fall back to the live section project context when a section thread is missing its snapshot
  • add focused coverage for delayed first-turn handling and missing-snapshot fallback

Root Cause

Section context delivery used a fragile current-message-count check. If the projection already contained a later user message when the first turn-start event was processed, the first message could be treated as non-initial and sent without section context.

Validation

  • vp check
  • vp run typecheck

@github-actions github-actions Bot added size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Jul 2, 2026
@sukar0972 sukar0972 marked this pull request as ready for review July 4, 2026 20:50
@sukar0972 sukar0972 merged commit 458a0b8 into main Jul 4, 2026
10 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant