Skip to content

feat: align production workbench UI with approved design - #23

Merged
boh5 merged 6 commits into
mainfrom
codex/product-refresh
Aug 24, 2026
Merged

feat: align production workbench UI with approved design#23
boh5 merged 6 commits into
mainfrom
codex/product-refresh

Conversation

@boh5

@boh5 boh5 commented Aug 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • add the approved repository-local marketing workflow Skills already carried by this branch
  • unify the Design Master, page contracts, and current workbench prototypes around the List-only Todo workflow
  • hard-cut the production Web UI to the accepted Todos, Runs, Schedules, Session, Root, Settings, and global-theme behavior
  • remove the superseded Board/DnD path and obsolete unmounted Web components without compatibility fallbacks or tombstone tests
  • include the production-conformance Goal, independent review record, browser evidence, and explicit user visual approval

Why

The production UI had drifted from the approved workbench design and interaction contracts. This PR makes the committed prototypes the actual production visual target while preserving the existing Server, Protocol, Agent Core, persistence, and domain boundaries. The result stays a quiet, high-density engineering workbench rather than introducing a parallel UI system.

Validation

  • bun run typecheck
  • bun run test — 8/8 Turbo tasks
  • bun run build — production Web build and arm64 executable
  • git diff --check origin/main...HEAD
  • light/dark browser matrices at 390/760/1024/1440 plus exact responsive breakpoints
  • independent Sol(max) fix/review loop completed with P0/P1/P2 all at zero
  • user explicitly approved the delivered production visuals

Documentation and security

  • I updated user-facing design and Goal documentation when behavior changed.
  • I added or updated tests for behavior changes.
  • I did not commit credentials, tokens, private configuration, or generated secrets.
  • I described security and trust-boundary impact below.

Security/trust-boundary impact: no Server, Protocol, Agent Core, authentication, persistence, or permission boundary changes. Theme persistence remains local to the existing browser key and now safely handles blocked reads and failed writes.

Related issue

None.

Review in cubic

boh5 added 3 commits August 23, 2026 11:33
Align the Master and page specifications with the accepted rendered prototypes. Remove the Todo Board, add accessible Preview stage changes, and unify shared visual and responsive states.
Hard-cut Todos to the approved list-only workflow and replace native Sources filtering. Align Runs, Schedules, Session, shared surfaces, and global theme behavior with the accepted prototypes. Include the implementation plan, review record, and completed visual acceptance evidence.
@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 2055694c-6cd7-4c1c-8fa6-f04c36d397b0


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.

@cubic-dev-ai

cubic-dev-ai Bot commented Aug 24, 2026

Copy link
Copy Markdown

This PR has 11,772 reviewable changed lines after ignored/generated files are excluded, above this repository's 10,000-changed-line automatic review limit. The raw diff is 12,769 lines before ignored/generated files are excluded.

Most of the diff comes from:

  • design-system/prototypes/app.js (~1,097 changed lines)
  • apps/web/src/routes/project-todos.tsx (~714 changed lines)
  • apps/web/src/routes/project-todos.interaction.tsx (~554 changed lines)
  • .codex/skills/copy-editing/SKILL.md (~457 changed lines)
  • .codex/skills/cro/references/form.md (~422 changed lines)

Comment @cubic-dev-ai review this to review it anyway. If the largest files are generated or fixture data, add them to your ignored files in review settings or ignorePatterns in cubic.yaml - cubic will then review the rest automatically. You can also raise this limit in review settings.

@boh5

boh5 commented Aug 24, 2026

Copy link
Copy Markdown
Owner Author

@cubic-dev-ai review this

@boh5

boh5 commented Aug 24, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@cubic-dev-ai

cubic-dev-ai Bot commented Aug 24, 2026

Copy link
Copy Markdown

@cubic-dev-ai review this

@boh5 I can't start this review because your workspace has reached its free monthly review limit. cubic has reviewed 42,862 of the 40,000 allowed lines of code this month. Reviews resume on 1 September 2026 (in 8 days). Paid plans include much higher monthly review limits. Upgrade now to resume reviews.

To help optimise your usage, you can tune cubic to get the most out of your usage limits:

Learn more →

@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown
⚠️ Action not completed

Review skipped: 123 files exceed the limit of 100.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Build edited Todo Markdown and uploaded Reference rows with DOM nodes and textContent instead of reinterpreting browser-controlled text as HTML. Remove the duplicate Session fallback renderer so the shared prototype renderer remains the single owner.
@boh5

boh5 commented Aug 24, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

Reviewed commit: 93eb944936

ℹ️ 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".

@boh5

boh5 commented Aug 24, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@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: 546ad8fa12

ℹ️ 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 apps/web/src/routes/project-todos.tsx Outdated
@boh5

boh5 commented Aug 24, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: 120a7738ae

ℹ️ 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".

@boh5
boh5 merged commit c261fd6 into main Aug 24, 2026
7 checks passed
@boh5
boh5 deleted the codex/product-refresh branch August 24, 2026 08:41
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