Skip to content

docs(roadmap): record the #108 and #310 completions in Done (#314) - #316

Closed
dezren39 wants to merge 1 commit into
mainfrom
sweep/314-roadmap-done-108-310
Closed

dezren39 wants to merge 1 commit into
mainfrom
sweep/314-roadmap-done-108-310

Conversation

@dezren39

@dezren39 dezren39 commented Oct 23, 2026 •

Copy link
Copy Markdown

Problem

Fixes #314

The ROADMAP Done table had no row for two completions merged on 2026-10-23:

  1. fix(desktop): OPENCODE_DISABLE_EMBEDDED_WEB_UI is set on every launch but read nowhere — desktop servers serve the embedded web UI anyway #108 — fixed by PR fix(cli): honor OPENCODE_DISABLE_EMBEDDED_WEB_UI in the web UI handler #312 (squash c1d3d9a708): WebUi.handler now reads OPENCODE_DISABLE_EMBEDDED_WEB_UI, non-API paths 404 and assets are never loaded.
  2. docs: ROADMAP Done table omits the #308 completion (PR #309) #310 — solved by PR docs(roadmap): record the #308 completion in Done (#310) #311 (squash 5e82dc7c73), which recorded the docs: ROADMAP Done table omits the #306 completion (PR #307) #308 completion. Per the table's one-step-lag convention, the docs: ROADMAP Done table omits the #308 completion (PR #309) #310 completion itself is recorded by the next row.

Verified on main 27e067f404: grep for 311/312 in ROADMAP.md returned nothing and the newest Done row was the #309 row.

Solution

ROADMAP.md only, +2 rows at the top of Done (newest first), in the table's established 92-char alignment:

  • OPENCODE_DISABLE_EMBEDDED_WEB_UI now disables the web UI | 2026-10-23 | #312
  • ROADMAP Done records the #308 completion | 2026-10-23 | #311

No other ROADMAP content changes (diff is exactly those two insertions).

Tests

Docs-only change. The diff was verified line-by-line before opening; both rows match the existing column widths.

Infra note: GitHub Actions has produced zero runs repo-wide since 2026-10-23T00:13Z (verified fresh via the Actions API before opening this PR — newest run is still the 00:13:24Z close-prs run), the same stall noted on PRs #312/#313. This PR is being merged on CodeRabbit + CLEAN/MERGEABLE per that docs-only precedent; if Actions recovers and flags anything, it will be fixed in a follow-up.

Single-repo: ROADMAP.md exists only in this repo.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@coderabbitai

coderabbitai Bot commented Oct 23, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 13 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d7254fd9-62f1-4782-b87c-ac831cdf8ee8
📥 Commits

Reviewing files that changed from the base of the PR and between 27e067f and fd7e6df.

📒 Files selected for processing: 1
  • ROADMAP.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Codex Review Summary

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

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-23T06:02:59.524897Z fd7e6df 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.

@dezren39

Copy link
Copy Markdown
Author

Duplicate of #315, which the creator lane opened a minute earlier (2026-10-23T06:00:49Z) with the same two Done rows for #314 (only the #108 row's title wording differs). Closing this PR in favour of #315 to avoid a double-merge race; #315 will be merged instead.

@dezren39 dezren39 closed this Oct 23, 2026
@dezren39
dezren39 deleted the sweep/314-roadmap-done-108-310 branch October 23, 2026 06:03
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.

docs: ROADMAP Done table omits the #108 and #310 completions (PRs #312, #311)

1 participant