Name the notice path for what it carries

The wrapper module added nothing the shell function did not already do. It
introduced no type and narrowed no export, so a producer reaching for
appendStreamRow directly was exactly as easy with it as without, and the
history it documented reads better on the function itself.

The name was also wrong. The path carries unknown commands, unavailable
modals and provider failures, none of which happen at startup, and a name
that lies to the next reader is how this constraint got lost twice.

Author: TheGreatAxios

src/tui-opentui/landing.test.ts

@@ -17,9 +17,8 @@ import {
   isLanding,
   paintLanding,
   streamRowCount,
-  surfaceStartupNotice,
+  surfaceSystemNotice,
 } from "./shell"
-import { flushStartupNotices } from "./startup-notices"
 import { makeOperatorQuestion, openOperatorOverlay } from "./overlays"
 import {
   LANDING_HINTS,
@@ -519,7 +518,7 @@ describe("landing screen", () => {
 
         const mcpError =
           "mcp github did not connect (ECONNREFUSED) — its tools are unavailable; /mcp for detail"
-        surfaceStartupNotice(shell, mcpError)
+        surfaceSystemNotice(shell, mcpError)
         await settle(h)
 
         // The mountain stays; the notice strip carries the wording.
@@ -567,7 +566,7 @@ describe("landing screen", () => {
         expect(before.length).toBeGreaterThan(0)
 
         const summary = "plugins: 3 skills missing: brand-identity, style, philosophy"
-        flushStartupNotices(shell, [summary])
+        surfaceSystemNotice(shell, summary)
         await settle(h)
 
         expect(isLanding(shell)).toBe(true)
@@ -592,7 +591,7 @@ describe("landing screen", () => {
       })
       try {
         await settle(h)
-        flushStartupNotices(shell, ["plugins: 1 skill missing: style"])
+        surfaceSystemNotice(shell, "plugins: 1 skill missing: style")
         appendStreamRow(shell, { role: "user", text: "first prompt" })
         await settle(h)
 

src/tui-opentui/product-host.ts

@@ -47,7 +47,7 @@ import {
   setPaletteOnCommand,
   setMcpNeedsAuth,
   setStatusFlash,
-  surfaceStartupNotice,
+  surfaceSystemNotice,
   type AppShell,
   type ItemDescription,
   type OverlaySelection,
@@ -348,7 +348,7 @@ export async function mountProductHost(
   // before the first turn (CL-5618).
   const widthReport = checkWidthContract(renderer.widthMethod)
   if (!widthReport.agrees) {
-    surfaceStartupNotice(shell, widthContractNotice(widthReport))
+    surfaceSystemNotice(shell, widthContractNotice(widthReport))
   }
 
   const port = createLiveSessionPort({
@@ -465,9 +465,9 @@ export async function mountProductHost(
     if (notice === null) return
     if (notice.kind === "row") {
       // MCP load failures and hook failures must not wipe the landing mark.
-      // surfaceStartupNotice keeps the mountain while the notice strip carries
+      // surfaceSystemNotice keeps the mountain while the notice strip carries
       // the wording, then flushes a durable row once the session starts.
-      surfaceStartupNotice(shell, notice.text)
+      surfaceSystemNotice(shell, notice.text)
       return
     }
     setStatusFlash(shell, notice.text, { ttlMs: RUNTIME_FLASH_MS })

src/tui-opentui/runner-host.ts

@@ -36,7 +36,7 @@ import {
   setPromptModelLabel,
   setPromptWorkspace,
   setShellExitHandler,
-  surfaceStartupNotice,
+  surfaceSystemNotice,
 } from "./shell.js"
 import type { CostSummary } from "../cost/cost-summary.js"
 import { watchGitBranch, type FetchBranch } from "./workspace-watch.js"
@@ -326,7 +326,7 @@ export async function mountRunnerHost(deps: RunnerHostDeps): Promise<RunnerHost>
   const surfaceDeps: CommandSurfaceDeps = {
     ...(deps.surfaces ?? {}),
     ...(host.openModels !== undefined ? { openModels: host.openModels } : {}),
-    notify: (text) => surfaceStartupNotice(host.shell, text),
+    notify: (text) => surfaceSystemNotice(host.shell, text),
   }
 
   const refreshModels = (

src/tui-opentui/shell.ts

@@ -2118,8 +2118,13 @@ function evictedRowsNotice(evicted: number): string {
  * mounted the wording rides the notice strip and the row is held for flush
  * once a real session row ends the landing; after that it is a normal system
  * row.
+ *
+ * Every producer of a system-class row belongs here rather than at
+ * `appendStreamRow`. CL-5618 fixed the MCP and hook producers one at a time
+ * and the plugin producer kept the defect, which is what per-call-site rules
+ * buy you. Reaching for `appendStreamRow` directly is the bug.
  */
-export function surfaceStartupNotice(shell: AppShell, text: string): void {
+export function surfaceSystemNotice(shell: AppShell, text: string): void {
   if (isLanding(shell)) {
     const bag = internals.get(shell)
     if (bag !== undefined) {

src/tui-opentui/startup-notices.ts

@@ -149,9 +149,8 @@ import {
   setPromptRecognitionSource,
   setSentMessageHistory,
   setShellRunState,
-  surfaceStartupNotice,
+  surfaceSystemNotice,
 } from "../tui-opentui/shell.js";
-import { flushStartupNotices } from "../tui-opentui/startup-notices.js";
 import {
   classifyAgentSendFailure,
   shouldSettleUiAfterSendFailure,
@@ -436,7 +435,7 @@ export async function runTUI(initialConfig: Config): Promise<number> {
   // have no result channel back to an operator action, unlike verify/add-path/
   // trust-grant. A log-only summary is invisible — nobody watches
   // ~/.corbits/logs/corbits.log — so these are queued and handed to
-  // `flushStartupNotices` once the shell mounts.
+  // the shell one at a time once it mounts.
   const startupPluginNotices: string[] = [];
   const discoveryNotice = formatPluginWarningsSummary(pluginLoadDiag.warnings);
   if (discoveryNotice !== undefined) startupPluginNotices.push(discoveryNotice);
@@ -1847,7 +1846,7 @@ export async function runTUI(initialConfig: Config): Promise<number> {
   // the whole composition. Once a session row has ended the landing this is an
   // ordinary system row, so there is no second behaviour to reason about.
   const systemNotice = (text: string): void => {
-    surfaceStartupNotice(host.shell, text);
+    surfaceSystemNotice(host.shell, text);
   };
 
   /** Settle the shell after a rejected send so the run does not look live. */
@@ -2378,7 +2377,8 @@ export async function runTUI(initialConfig: Config): Promise<number> {
 
   // Surface fire-and-forget startup plugin diagnostics now that there is a
   // shell to say them to (queued above, before `host` existed).
-  flushStartupNotices(host.shell, startupPluginNotices);
+  for (const notice of startupPluginNotices)
+    surfaceSystemNotice(host.shell, notice);
 
   await host.waitUntilExit();
   // Quitting mid-stream is an abnormal end for the in-flight cycle: nothing