Skip to content

Add PR comment context, safe settings retry and Claude compaction meter - #1917

Merged
currentbits merged 8 commits into
mainfrom
coder/roadmap-batch-two
Nov 7, 2026
Merged

currentbits merged 8 commits into
mainfrom
coder/roadmap-batch-two

Conversation

@currentbits

@currentbits currentbits commented Nov 7, 2026 •

Copy link
Copy Markdown
Owner

This batch makes review comments usable as discussion context, makes settings-load failures recoverable, and makes Claude's context meter explicit about compaction data and uncertainty.

  • Add ordinary comments, review bodies and inline comments to a selected project thread or a new draft. Preserve typed text, deduplicate additions, include source and file/line context, and leave sending to the user. Paginated loading retains available comments and unsaved PR edits through partial failures.
  • Show settings-load errors with Retry. Keep the rest of the app usable, block writes until a saved snapshot is known, and ignore stale replies. Failed newer saves cannot hide older successful saves, and stay-awake reflects the accepted snapshot.
  • Use the launched Claude process's enforced compaction threshold when supplied. Label newer CLI summaries as estimates and keep the model ceiling separately available. Older/unsupported CLIs and known percentage overrides, including explicitly forwarded remote project overrides, fall back to the model ceiling. Unknown CLI-owned overrides can still affect estimates; these are not exact triggers. Bounded optional metadata requests cannot interrupt turns.

Validation: after rebasing onto merged #1916, all 3,813 renderer tests and typecheck/preload sync pass. Integrated backend checks passed 67 tests; subsequent focused context suites passed 28, including a regression confirmed red before the remote-override fix. The settings-save review fix passed 13 focused tests. Earlier validation also passed the production build and 151 broader Claude/runner tests. New protocol tests are included in Windows CI.

Addresses #1870, #1877 and #1898; roadmap #1869. GitLab's PR-workspace UI is not exposed today and is not expanded here. No paid live Claude session was started for validation.

Created with gpt-6.1-sol in Codex, using three GPT-6.1 implementation workers and GPT-6.1 review workers.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 7, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-07T11:36:38.550956Z 446c331 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 446c331821

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/useCoder.ts Resolved
@currentbits
currentbits force-pushed the coder/roadmap-batch-two branch from ec2b35a to 3b4541f Compare November 7, 2026 12:15
@currentbits
currentbits merged commit 2e4b0fd into main Nov 7, 2026
14 checks passed
@currentbits
currentbits deleted the coder/roadmap-batch-two branch November 7, 2026 13:19
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