Skip to content

Restore progressive existing-task status icons - #128

Merged
mergify[bot] merged 4 commits into
mainfrom
codex/BEAR-121-existing-icons
Aug 17, 2026
Merged

Restore progressive existing-task status icons#128
mergify[bot] merged 4 commits into
mainfrom
codex/BEAR-121-existing-icons

Conversation

@ericlitman

@ericlitman ericlitman commented Aug 17, 2026

Copy link
Copy Markdown
Owner

What changed

  • add one hidden, read-only onboard-stream pass that fully paginates existing tasks, reads only the newest turn, applies exact historical footer statuses first, and classifies remaining completed turns in fixed sequential Luna-medium batches
  • request complete-catalog read permission once after the friendly install handoff; declining leaves every existing title unchanged and the copy never claims progress before approval
  • disable every hosted classifier feature and request-input, keep shell/code-mode execution unavailable, and reject any tool-attempt/runtime event or extra assistant message
  • stream strict ordered candidates to the published yielded install cell, which performs one mounted reread and at most one exact-target title write per semantic row
  • render plain icons for exact evidence, quiet icons for conservative inference, and no icon or write for unknown
  • make ordinary turns replace inferred prefixes with the plain current status, and make uninstall remove both plain and inferred prefixes
  • update the install experience, product documentation, release smoke, contract tests, and changelog

Why

A prior rewrite removed existing-task onboarding after the earlier controller-based implementation repeatedly failed. The replacement deliberately keeps only the proven title boundary: one ephemeral read-only classifier process and the mounted Codex reader/setter as the sole writer. There is no controller task, worker fan-out, queue, retry, persisted onboarding state, repair sweep, fallback model, or alternate title path.

User impact

Installation reports success immediately and explains the one catalog-read permission request. If approved, existing sidebar tasks fill progressively over the next several minutes. A small identifies a first-read best guess and disappears after that task's next ordinary turn. Declining, unclear, inaccessible, unsafe, or drifted tasks remain unchanged.

Verification

  • gofmt
  • go test ./...
  • go test -race ./...
  • go vet ./...
  • Darwin arm64 and amd64 builds
  • shell syntax, experiment-registry validation, guide parity, and git diff --check
  • exact-candidate isolated release smoke
  • ten-minute yielded-cell survival canary across a completed outer response
  • live Codex Desktop install, progressive sidebar replay, drift/no-retry case, interruption, natural rerun, next-turn sparkle removal, consented uninstall, reinstall, and post-restart persistence
  • full clean-sidebar replay: 340 candidates, 295 confirmed mounted writes, 45 unknown untouched, zero drift, zero unconfirmed
  • hardened live Luna classifier contract and 72-second exact-candidate helper stream with strict no-tool event validation
  • review-agent and Fable completion verdicts: no findings / Ship

Closes BEAR-121

@ericlitman
ericlitman marked this pull request as ready for review August 17, 2026 22:33

@openswebot openswebot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Open SWE Review found 2 potential issues.

Open in Web

Comment thread INSTALL.md
Comment thread cmd/threadbear/onboard.go Outdated
@ericlitman
ericlitman marked this pull request as draft August 17, 2026 23:20
@ericlitman
ericlitman marked this pull request as ready for review August 17, 2026 23:21
@ericlitman

Copy link
Copy Markdown
Owner Author

@Mergifyio queue

@mergify

mergify Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • Entered queue2026-08-17 23:22 UTC · Rule: default · triggered by @ericlitman with the @mergifyio queue command
  • Checks skipped · PR is already up-to-date
  • Merged2026-08-17 23:23 UTC · at e2ae74082bf96d6646e1e2a89f5b09d57c8f1d05

This pull request spent 28 seconds in the queue, including 2 seconds running CI.

Required conditions to merge
  • github-review-approved [🛡 GitHub repository ruleset rule Mergify automatic merge queue]
  • all of [🛡 Merge Protections rule Queue ready pull requests]:
  • any of [🛡 GitHub branch protection]:
    • check-success = verify
    • check-neutral = verify
    • check-skipped = verify
  • any of [🛡 GitHub branch protection]:
    • check-success = Open SWE Review
    • check-neutral = Open SWE Review
    • check-skipped = Open SWE Review
  • any of [🛡 GitHub repository ruleset rule Mergify automatic merge queue]:
    • check-success = @mergify/Mergify Merge Protections
    • check-neutral = @mergify/Mergify Merge Protections
    • check-skipped = @mergify/Mergify Merge Protections
  • any of [🛡 GitHub repository ruleset rule Review gate]:
    • check-success = Open SWE Review
    • check-neutral = Open SWE Review
    • check-skipped = Open SWE Review

@mergify mergify Bot added the queued label Aug 17, 2026
@mergify
mergify Bot merged commit 527d936 into main Aug 17, 2026
4 checks passed
@mergify mergify Bot removed the queued label Aug 17, 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