Skip to content

fix(web): preserve line breaks in owner questions - #931

Merged
Mjones13 merged 3 commits into
mainfrom
fix/question-line-breaks-20261111
Nov 12, 2026
Merged

Mjones13 merged 3 commits into
mainfrom
fix/question-line-breaks-20261111

Conversation

@Mjones13

@Mjones13 Mjones13 commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

Question prompts with single newlines were rendered as one Markdown paragraph, so labelled sections ran together even though the submitted text contained real line breaks. Enable the existing line-break mode in the shared web/desktop question panel while retaining Markdown emphasis, lists and fenced code.

Added a browser regression covering section separation, bold text, lists, code indentation, rendered geometry and reachable answer choices. Updated the existing view-map journey. On pre-integration head d779165, lint, boundary accounting, typecheck and both web shards passed (573 test files, including all 9 question-card cases).

Merged canonical main 00c7b7b (PR #935, Ubuntu CI repair) into the task with a normal merge, producing current head 7d3bb1e. The feature diff against main remains three paths. Current-head CI passed in run 43122947450, including Check, Build, Test, both web shards, lint, typecheck and Linux enrollment. Earlier dependency failures are historical (see #932). Merged as 5cacad1; direct readback confirmed this main tip has reviewed head 7d3bb1e as its second parent (tree 465a65422d8e8db4a892509f36ddcc980deea55c). git diff --check passes. Local dependency-backed checks remain unavailable because this checkout has no dependencies and local storage is below the repository install floor.

Visual limit: the supplied screenshot and source payload establish the original defect; candidate before/after screenshots and native mobile verification are unavailable locally. Mobile and wire/storage formats are unchanged. No installed application update or restart is included.

Contributor: Jones Code thread 8ddc7cd8-11c3-480c-a149-09c926101e0a, implementation and Git owner. GPT-6.1 Sol through the Codex harness.

@Mjones13
Mjones13 merged commit 5cacad1 into main Nov 12, 2026
24 checks passed
Mjones13 added a commit that referenced this pull request Nov 12, 2026
Brings in #913 (orphaned run records), #917 (held queue expiry and the
sparse Jones jones_queue_holds table at migration 147), #930 (install
status reconcile), #931 (question line breaks) and #935 (signed Ubuntu
APT retrieval during transport failures).

#917 landed first and keeps Jones migration 147 (QueueHolds). The sync's
unreleased Jones migrations move up by one, above the highest main ID:
NativeExecutionProcessCustody 148, OrdinaryCheckoutProcessCustody 149,
AuthSessionClientMetadata 150, WorkspaceRuntimeUses 151 and
PeerLinksAdoption 152. These IDs were only ever on the unmerged sync
branches (#850 and this successor), never on main. Their module files,
ledger tests and the WorkspaceRuntimeUses basis literal in
RuntimeUseInstallation follow the new IDs; no shipped schema or row
changes.

server.ts keeps the sync's managed-policy imports and adds main's
HeldQueueExpiry (WorkMode and Scheduler were already imported). The
boundary JSON is a structural three-way merge; the install-status lint
and fmt commands keep the sync's sidebar path and gain main's finishPlan
files. The server compile-cache anchor is re-pinned for server.ts.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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