-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
Control UI sidebar contract breaks after managed-worktrees merge #100613
Copy link
Copy link
Closed
Labels
P1High-priority user-facing bug, regression, or broken workflow.High-priority user-facing bug, regression, or broken workflow.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maintainerMaintainer-authored PRMaintainer-authored PR
Description
Metadata
Metadata
Assignees
Labels
P1High-priority user-facing bug, regression, or broken workflow.High-priority user-facing bug, regression, or broken workflow.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.maintainerMaintainer-authored PRMaintainer-authored PR
Type
Fields
Priority
None yet
Bug type
Regression (worked before, now fails)
Beta release blocker
No
Summary
After the managed-worktrees merge, the Control UI navigation module no longer exports the customizable-sidebar contract still consumed by the app, and the new Workboard hook is absent from the typed-hook inventory.
Steps to reproduce
mainat or after0acd851a3b3d6142f1e569a0e5616db62361969e.src/plugins/contracts/boundary-invariants.test.ts.SidebarNavRoute, pinned-route helpers, and Workboard typed-hook allowlist failures.Expected behavior
The new Worktrees route should join the existing customizable sidebar inventory without removing the APIs used by the app, and the Workboard hook registration should satisfy the repository contract inventory.
Actual behavior
Core production/test typechecks fail on the removed sidebar exports, and the plugin boundary invariant rejects
extensions/workboard/index.ts.OpenClaw version
Current
main(6441858e2168016f111fbeb335584ba589e4d203).Operating system
Linux CI / Blacksmith Testbox.
Install method
Source checkout with pnpm.
Model
N/A
Provider / routing chain
N/A
Logs
check-prod-typesandcheck-test-typesreport missing navigation exports fromui/src/app-navigation.ts;checks-fast-contracts-plugins-areportsextensions/workboard/index.tsas an unlisted typed-hook registration.Impact and severity
Affected: all contributors and maintainers validating current
mainor PRs rebased onto it. Severity: high for development workflow because the required CI gate cannot pass. Frequency: deterministic. Consequence: otherwise unrelated PRs cannot satisfy exact-head landing gates.Additional information
Regression introduced by the managed-worktrees merge commit
0acd851a3b3d6142f1e569a0e5616db62361969e(#100535). The existing customizable sidebar contract was present immediately before that merge.