Skip to content

fix(coding-agent): preserve compaction context and check committed completion - #195

Draft
MelodyVAR wants to merge 1 commit into
mainfrom
fix/context-completion-public-20260925
Draft

MelodyVAR wants to merge 1 commit into
mainfrom
fix/context-completion-public-20260925

Conversation

@MelodyVAR

Copy link
Copy Markdown
Collaborator

Compaction can turn a blank or thinking-only model response into a nonempty summary by adding bookkeeping text, or drop the previous summary on a repeated split-turn pass with no new history. Reject empty model summaries before adding that text and retain the existing summary when no new history is available.

Add an opt-in --completion-check git-committed for print/JSON mode. It checks for a new commit, a nonempty committed diff, a clean worktree, and final answer text, then permits at most 1–3 additional prompts in the existing session. Git probes are bounded and read-only. Terminal denial, assistant errors/aborts, and session replacement suppress automatic feedback; default behavior remains unchanged. Denial recovery is covered separately by #188.

Validation:

  • npm run check passed.
  • pnpm run build:offline and the required ./test.sh passed with isolated credentials.
  • Focused compaction and completion tests passed (171 tests across the targeted suites).

This change adds no grading inputs, automatic commits, or independent correctness judgment. The separate completion-review experiment is not included.

AI-assisted implementation and description; behavior and validation were reviewed against the source.

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