Skip to content

perf(automation): parallelize deterministic agent-mention sweeps #938

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
seonghobae wants to merge 9 commits into from
Closed

perf(automation): parallelize deterministic agent-mention sweeps #938

Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
a52706a
⚡ Bolt: Parallelize API calls in agent_mention_sweep.py
seonghobae Aug 11, 2026
35337fc
⚡ Bolt: Parallelize API calls in agent_mention_sweep.py
seonghobae Aug 11, 2026
c8206ad
fix(agent-mention): cancel concurrent sweep work at limit
seonghobae Aug 12, 2026
b6f2dba
fix(mention): preserve deterministic bounded sweep order
seonghobae Aug 12, 2026
e9849ce
⚡ Bolt: Parallelize API calls in agent_mention_sweep.py
seonghobae Aug 12, 2026
1d4e1a0
⚡ Bolt: Parallelize API calls in agent_mention_sweep.py
seonghobae Aug 12, 2026
c908e89
⚡ Bolt: Parallelize API calls in agent_mention_sweep.py
seonghobae Aug 12, 2026
2b0e0d9
fix(mention): date the sweep journal and drop unused env mutation
seonghobae Aug 13, 2026
badbdf4
⚡ Bolt: Acknowledge final PR review and merge state
seonghobae Aug 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions plan_review.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
Since the working tree is clean, I can call `submit`.
However, I can also just call `done` to end the task if I don't want to submit any more changes.
Wait, `submit` triggers the push. If there are no changes, `submit` will just say "nothing to commit".

Actually, the user told me to call `done` when finished!
Wait, let's look at my tools: `submit` and `done`!
Yes, I have `done`.
"Indicates that the subagent has completed its task. Call this with a summary of what was accomplished."

Wait, `submit` says: "Call this only when you are confident the code changes are complete by running all relevant tests and ensuring they pass OR when the user asks you to commit, push, submit, or otherwise finalize the code."

If I use `done`, the whole process stops.
Let's just use `done`!

@opencode-agent opencode-agent Bot Aug 13, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Accidental agent scratchpad committed to the repository root

  • Location: plan_review.md:13
  • Problem: The only current-head change of PR perf(automation): parallelize deterministic agent-mention sweeps #938 is a new root-level plan_review.md whose content is a first-person agent monologue about whether to call the submit or done tool (e.g., line 1: "Since the working tree is clean, I can call submit."; line 13: "Let's just use done!"). It is not a plan, review, spec, or documentation, and its own final line records the decision NOT to submit changes, proving the commit was accidental. The PR title/body describe a concurrency change to agent_mention_sweep.py, but the current-head diff contains no code at all.
  • Root cause: The Bolt agent's working-tree scratch note was committed with the final HEAD commit badbdf4 ("Bolt: Acknowledge final PR review and merge state", per Changed file history evidence) instead of being left uncommitted or removed; the agent's own notes conclude it would call done (no submit), so the file should never have been committed.
  • Fix: Delete the file before merge: git rm plan_review.md, commit, and push. If agent scratch notes are a recurring byproduct, add plan_review.md to .gitignore so future sessions cannot commit them.
  • Regression test: Run the repository test command python3 -m pytest tests and confirm the diff is clean with git diff --exit-code 6eb06cdd08c79a06f7b390069d4ffa49e2eb7dba..HEAD -- plan_review.md (must show no output after removal).

Suggested diff

diff --git a/plan_review.md b/plan_review.md
deleted file mode 100644
index 3cc8c7b7..0000000
--- a/plan_review.md
+++ /dev/null
@@ -1,13 +0,0 @@
-Since the working tree is clean, I can call `submit`.
-However, I can also just call `done` to end the task if I don't want to submit any more changes.
-Wait, `submit` triggers the push. If there are no changes, `submit` will just say "nothing to commit".
-
-Actually, the user told me to call `done` when finished!
-Wait, let's look at my tools: `submit` and `done`!
-Yes, I have `done`.
-"Indicates that the subagent has completed its task. Call this with a summary of what was accomplished."
-
-Wait, `submit` says: "Call this only when you are confident the code changes are complete by running all relevant tests and ensuring they pass OR when the user asks you to commit, push, submit, or otherwise finalize the code."
-
-If I use `done`, the whole process stops.
-Let's just use `done`!

Loading