feat(web): establish redesign primitives and shell#28839

🟢 Low rules/no-styled-raw-controls.ts:154

A raw control's className can be missed or falsely reported when another function declares a const with the same name: the file-wide map overwrites earlier initializers, so Program:exit resolves every identifier to the last visited declaration instead of its lexical binding. Track constants by lexical binding so each JSX expression resolves to its own initializer.

No longer relevant as of ab9ef1256d764a406fc6fef7d50f965ea5779d9c

fixed in b58d7983c6: identifiers now resolve to their lexical const declaration instead of a file-wide name map. the existing rule tests cover sibling scopes, parameter and mutable-binding shadowing, aliases, forward declarations, and cycles.

🟢 Low rules/no-styled-raw-controls.ts:88

A || or ?? expression can return appearance classes from its left operand, but classStrings only inspects the right operand, so <button className={"bg-red-500" || "flex"} /> is silently accepted. Inspect the left operand for || and ?? as well.

No longer relevant as of ab9ef1256d764a406fc6fef7d50f965ea5779d9c

fixed in b58d7983c6: the rule now inspects both operands of || and ??, while && still reads only its result operand. the existing rule tests cover both missed left operands, a fallback, and the && condition.

🟡 Medium chat/DraftHeroHeadline.tsx:335

When there are more than three unpinned projects, choosing a hidden project through All projects… leaves the prompt and attachments in the current draft. This row opens the global new-thread-in action, which creates or navigates to another draft instead of retargeting the populated draft as visible project rows do; route full-picker selections through selectProject.

No longer relevant as of de9a00aecdd0c05fc1f9565eeea7ddd9c52af9be

Confirmed and fixed. "All projects…" in DraftHeroHeadline.tsx now passes an onSelectProject callback through openCommandPalette (commandPaletteBus.ts), reduceCommandPaletteUiState (CommandPalette.logic.ts) and OpenCommandPaletteDialog (CommandPalette.tsx). Picking a hidden project goes through the same selectProject path as the visible rows. Verified in the real app: hidden and visible picks both keep the same draft, prompt and attachment. Cancelling the palette keeps it too. The global new-thread action still opens a separate draft.

🟢 Low rules/no-dot-separated-text.ts:56

A • at the start of a JSXText node is exempted even when earlier JSX siblings already render content, so <span>{name} • today</span> is not reported as dot-separated metadata. The exemption checks only the current text node; account for preceding rendered siblings before treating the bullet as a leading list marker.

Fixed. JSXText in no-dot-separated-text.ts now checks the node's earlier siblings, so a leading • after rendered content such as <span>{name} • today</span> is reported. A bullet that starts a list item with nothing before it, like <li>• item</li>, is still allowed. Verified with a throwaway fixture that caught the first case and allowed the second, and the full apps/web/src lint stays at zero hits.

🟢 Low rules/no-dot-separated-text.ts:87

A leading • followed by a JSX expression is reported as dot-separated metadata, so ordinary list items such as <li>• {label}</li> trigger this rule while the static-text equivalent is exempt. JSXText only checks whether earlier siblings render content, leaving hasRight false when the text after the bullet is in a later expression; include following siblings when computing the right-side content.

Fixed. JSXText now checks content-rendering siblings on both sides of the text node. A leading • followed by an expression, as in <li>• {label}</li>, is treated as a list bullet and allowed, the same as <li>• item</li>. {name} • today and {a} · {b} are still reported. Verified with a throwaway fixture covering all four cases, and apps/web/src stays at zero hits.

🟢 Low rules/no-dot-separated-text.ts:56

A second • at the start of a line in a multiline literal is reported as a separator, rejecting valid list content. The exemption checks all text before the bullet, including earlier lines; determine whether the bullet is leading using only the text since the previous newline.

Fixed. The leading-bullet exemption in no-dot-separated-text.ts now looks only at text on the same line as the glyph. So a • that starts any line of a multiline literal, as in "• one\n• two", is allowed. "one • two", {name} • today and {a} · {b} are still reported. Verified with a throwaway fixture, and apps/web/src stays at zero hits.

🟡 Medium components/Sidebar.tsx:3650

When settledCount is 0 and no drag is active, the visually hidden Settled header remains in the sidebar tree, so pressing End or Down can select a row that cannot receive focus and leave keyboard navigation stranded. installSidebarTree excludes [hidden], [inert], and [aria-hidden="true"] rows, but not CSS-invisible rows; mark this header aria-hidden under the same condition while retaining invisible to preserve its layout space.

Fixed. SidebarSectionHeader in SidebarListMarkers.tsx takes a concealed flag that sets aria-hidden and tabIndex={-1} on the shelf toggle while keeping its layout space. Sidebar.tsx passes it for the Settled header when settledCount is 0 and no drag is active. installSidebarTree now skips that row, so End and Down never land on an invisible header.