Skip to content

fix(pi): hide empty native worker messages and retain diagnostics - #4

Merged
dougyau merged 1 commit into
mainfrom
t3/native-worker-message-visibility
Oct 15, 2026
Merged

dougyau merged 1 commit into
mainfrom
t3/native-worker-message-visibility

Conversation

@dougyau

@dougyau dougyau commented Oct 15, 2026

Copy link
Copy Markdown
Owner

Native Pi worker conversations showed (empty response) between tool activity, and ingestion discarded available assistant diagnostics.

Keep canonical messages and capture bookkeeping intact. A shared presentation rule hides only settled native assistant messages with no text or attachments, while reserving their IDs against local copies. Native assistant ingestion preserves omitted-content, error, cancellation, and length-limit diagnostics as ordinary [T3 notice] paragraphs. User messages and tool results keep their existing behavior.

Verification

  • 189 tests passed across the native ingestion, web timeline, mobile feed, and shared predicate suites.
  • Four affected package typechecks, focused lint, LSP diagnostics, and diff checks passed.
  • Independent review approved the complete diff with no material findings.
  • Committed files match the reviewed candidate byte-for-byte. The formatter-only commit hook was bypassed for this commit to preserve those reviewed bytes; no lint or typecheck gate was skipped.

No screenshots or browser/device verification, as requested. Historical discarded diagnostics cannot be recovered. Search fallback behavior remains outside this fix.

Implemented with GPT-6.1-sol in Pi using Pitako Developer Head. Coordinated and published with GPT-6.1-sol in Pi.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M labels Oct 15, 2026
@dougyau
dougyau merged commit c9b08fb into main Oct 15, 2026
7 of 22 checks passed
@dougyau
dougyau deleted the t3/native-worker-message-visibility branch October 16, 2026 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant