Skip to content

fix(studio): align CI contracts with tightened debugger chrome - #156

Closed
phodal wants to merge 1 commit into
mainfrom
fix/studio-ci-chrome-contracts
Closed

fix(studio): align CI contracts with tightened debugger chrome#156
phodal wants to merge 1 commit into
mainfrom
fix/studio-ci-chrome-contracts

Conversation

@phodal

@phodal phodal commented Sep 7, 2026

Copy link
Copy Markdown
Member

Summary

Main is red on ACP Host (all three OSes) and CI ubuntu-latest / Node 22.20.0 after the debugger chrome unification.

  • rustfmt the two ACP host hunks that fail cargo fmt --check
  • name the four owned Studio stylesheets instead of asserting a count of 3
  • measure theme contrast from a primary-fill control that still exists
  • assert the Debugger status-bar Project label instead of the retired concatenated harness chrome
  • shrink default tree/inspector pane widths so compact Debugger keeps the notebook at least half the grid

Spec

docs/specs/2026-09-07-studio-ci-chrome-contracts.md

Test evidence

  • cargo fmt --check in packages/better-harness-desktop/rust/acp-host
  • Studio Playwright: 65 passed

Main failed ACP Host rustfmt and the Ubuntu Playwright job after the
debugger chrome unification. Format the host, name the four owned
stylesheets, measure contrast from chrome that still exists, assert the
status-bar Project label, and shrink default pane widths so compact
Debugger keeps the notebook as the primary pane.

Implements docs/specs/2026-09-07-studio-ci-chrome-contracts.md.
Validated with cargo fmt --check and the 65 Studio Playwright tests.

Co-authored-by: Grok (Grok 4.6) <grok@x.ai>
@phodal phodal closed this Sep 7, 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