Fix e2e flakes in opencode lifecycle and test assertions

[pfleidi]

Summary

• harden OpenCode lifecycle hook emission by adding robust turn-start fallback and deduplicated turn-end handling, with regression coverage for generated plugin content
• exclude internal runtime artifacts under .opencode/ and .entire/ from checkpoint file tracking while preserving user-managed opencode.json in files_touched
• stabilize flaky E2E expectations by increasing cursor prompt timeout, allowing ±1 attribution-line variance for mixed commits, and making subagent commit flow wait on file creation while keeping a single-prompt scenario

Verification

• go build ./...
• go vet ./...
• mise run check
• targeted E2E reruns for failing cases (opencode, cursor-cli, claude-code)

---

Note

Medium Risk
Adjusts OpenCode hook emission and file-attribution filtering logic used for checkpointing; mistakes could cause missing/duplicate lifecycle hooks or incorrect files_touched/attribution in edge cases.

Overview
Hardens the OpenCode plugin’s turn lifecycle signaling by adding a maybeStartTurn helper, a message.updated fallback for missing/late message.part.updated events, and per-session tracking to dedupe/guard turn-end on idle events.

Tightens checkpoint file attribution by introducing isInternalTrackingPath and filtering .entire/ and .opencode/ runtime artifacts out of FilesTouched/fallback committed-file lists, while keeping user project files (e.g., opencode.json) eligible.

Stabilizes tests by adding regression coverage for the generated OpenCode plugin guards, excluding internal paths in unit tests, and relaxing/flaking-proofing E2E assertions (±1 line attribution tolerance, longer timeouts, and waiting/staging only expected files in subagent flows).

^{Reviewed by Cursor Bugbot for commit 21172e2. Configure here.}

[Copilot]

Pull request overview

This PR hardens OpenCode hook lifecycle handling and stabilizes E2E tests by reducing timing sensitivity and by excluding internal runtime artifacts from file attribution/tracking in the manual-commit strategy.

Changes:

• Add more robust OpenCode turn lifecycle handling (turn-start fallback + turn-end dedupe) and regression assertions for generated plugin content.
• Exclude .entire/ and .opencode/ runtime artifacts from files_touched / committed-file fallback logic while preserving user-managed opencode.json.
• Stabilize flaky E2E tests via longer prompt timeouts, tolerance for minor attribution variance, and waiting for file creation before committing.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
e2e/tests/subagent_commit_flow_test.go	Waits for file creation before staging/committing and narrows git add scope.
e2e/tests/edge_cases_test.go	Increases prompt timeout for two prompts to reduce flakiness.
e2e/tests/attribution_test.go	Allows ±1 line variance in mixed attribution assertions.
cmd/entire/cli/strategy/manual_commit_test.go	Expands tests to cover .opencode/ exclusion and mergeFilesTouched filtering.
cmd/entire/cli/strategy/manual_commit_hooks.go	Filters internal tracking paths from FilesTouched and transcript-derived modified files.
cmd/entire/cli/strategy/manual_commit_git.go	Filters internal tracking paths during FilesTouched merge/deduping.
cmd/entire/cli/strategy/manual_commit_condensation.go	Uses shared internal-path filter when excluding metadata from committed-file fallback.
cmd/entire/cli/strategy/common.go	Introduces isInternalTrackingPath for .entire/ + .opencode/ runtime/config paths.
cmd/entire/cli/agent/opencode/hooks_test.go	Adds regression test asserting lifecycle guard code is present in generated plugin.
cmd/entire/cli/agent/opencode/entire_plugin.ts	Implements open-turn tracking, turn-start fallback, and turn-end dedupe/cleanup.

[pfleidi]

Bugbot run

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit 21172e2. Configure here.}

[Copilot]

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.