Skip to content

Bound automatic continuation for long-running sessions - #121

Closed
KooshaPari wants to merge 1 commit into
mainfrom
audit/auto-continue-bounded-20260802
Closed

Bound automatic continuation for long-running sessions#121
KooshaPari wants to merge 1 commit into
mainfrom
audit/auto-continue-bounded-20260802

Conversation

@KooshaPari

Copy link
Copy Markdown
Owner

Summary\n- capture the existing six-file auto-continue and UX changes on an isolated branch\n- bound recursive non-interactive continuation to 8 chained interruptions\n- reset the continuation budget on a new top-level message or successful completion\n\n## Verification\n- cargo metadata --offline --no-deps --format-version 1 passed\n- git diff --check passed on the original dirty checkout (line-ending warnings only)\n- nightly rustfmt parsed the touched Rust files but reports pre-existing formatting diffs; full Cargo compile/test is not claimed\n\n## Scope\nThe source checkout at main remains dirty and unchanged by this publication. This draft intentionally preserves the original six-file diff for review; it has not been merged or deployed.

@gemini-code-assist

Copy link
Copy Markdown

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@KooshaPari

Copy link
Copy Markdown
Owner Author

Audit update: GitHub reports this draft PR as DIRTY because its base is local Forge main at 3e7cb77, which is 145 commits behind origin/main. The original checkout remains untouched and dirty; no rebase was performed. A fresh remote-main branch is the next integration gate, requiring path-by-path conflict resolution for ui.rs, update.rs, log.rs, README, and the generated schema before merge. The bounded recursion guard itself is at MAX_AUTO_CONTINUE_ATTEMPTS = 8; full Cargo compile/test remains unclaimed.

@KooshaPari

Copy link
Copy Markdown
Owner Author

A clean replacement is now open as draft PR #122 from current origin/main: #122. This preservation snapshot remains open intentionally so the original six-file dirty diff and provenance are not discarded.

@github-actions github-actions Bot removed the rust label Aug 2, 2026
@KooshaPari

Copy link
Copy Markdown
Owner Author

Closing as superseded.

The clean replacement PR #122 ("Bound automatic continuation after interrupt limits") was merged into main today as commit 3e1d574406981db71c8215d6c15dae4ea3498069. That PR contains the refined version of this six-file auto-continue change, rebased onto current main with full CI green (test, coverage/build, performance, rustfmt, Clippy, CodeQL, cargo-deny, Trunk, Semgrep, Trufflehog).

This draft served its purpose as a preservation snapshot of the original dirty checkout. The provenance is retained in the audit/auto-continue-bounded-20260802 branch (commit 5062cedec) and in the merged history, so no work is lost by closing this PR.

@KooshaPari

Copy link
Copy Markdown
Owner Author

Superseded by merged PR #122 (commit 3e1d574).

@KooshaPari KooshaPari closed this Aug 2, 2026
@mergify mergify Bot added the rust label Aug 2, 2026
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