Skip to content

feat(web): preview Markdown and HTML by default - #30

Merged
yaacovcorcos merged 1 commit into
mainfrom
agent/preview-files-by-default
Aug 10, 2026
Merged

yaacovcorcos merged 1 commit into
mainfrom
agent/preview-files-by-default

Conversation

@yaacovcorcos

Copy link
Copy Markdown
Contributor

What changed

  • Render Markdown and MDX by default when no saved user preference exists.
  • Open HTML and HTM files from the file tree in the existing integrated browser.
  • Preserve the remembered Markdown source/preview choice and source view for line-targeted links.
  • Keep HTML source available through the file-tree context menu.
  • Fall back to source if browser preview is unsupported or fails.
  • Document the resulting file-preview behavior.

Why

Scient users usually open Markdown and generated HTML to read the result. Requiring an extra preview action on every file made the common path slower while source access can remain explicit and reliable.

Upstream boundary

The product policy and preview orchestration live under apps/web/src/scient/fileOpening. The inherited T3 file tree, right-panel store, editor, asset serving, and integrated-browser engine remain intact. Three inherited UI files receive only narrow integration calls.

Verification

  • pnpm exec vp fmt --check
  • focused vp lint --report-unused-disable-directives
  • pnpm --filter @t3tools/web typecheck
  • pnpm --filter @t3tools/web test — 2,104 tests passed
  • pnpm --filter @t3tools/web build
  • git diff --check

No browser automation or visual test was performed, as requested.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Aug 9, 2026
@github-actions

github-actions Bot commented Aug 9, 2026 •

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 11.3 KiB 11.3 KiB +9 B (+0.1%) 15.1 KiB ✅
Codex Thread snapshot wire 5.5 KiB 5.5 KiB −11 B (−0.2%) 7.3 KiB ✅
Codex Live turn WebSocket wire 5.8 KiB 5.9 KiB +20 B (+0.3%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 49.7 KiB 49.7 KiB 0 B (0.0%) 66.4 KiB ✅
Codex Live turn messages 16 16 0 (0.0%) 21 ✅
Claude Total thread wire 11.3 KiB 11.3 KiB −22 B (−0.2%) 15.1 KiB ✅
Claude Thread snapshot wire 5.5 KiB 5.5 KiB −3 B (−0.1%) 7.3 KiB ✅
Claude Live turn WebSocket wire 5.9 KiB 5.8 KiB −19 B (−0.3%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 50.6 KiB 50.6 KiB 0 B (0.0%) 66.4 KiB ✅
Claude Live turn messages 16 16 0 (0.0%) 21 ✅

Baseline: aff3f8a · PR result: 5b4966e · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 94.7 KiB
  • Claude decoded thread snapshot: 95.4 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@yaacovcorcos
yaacovcorcos force-pushed the agent/preview-files-by-default branch from f8785d0 to c4bb9d2 Compare August 10, 2026 08:22
@yaacovcorcos
yaacovcorcos changed the base branch from main to fix/windows-whisper-runtime-20260810 August 10, 2026 08:22
@yaacovcorcos
yaacovcorcos deleted the branch main August 10, 2026 08:24
@yaacovcorcos yaacovcorcos reopened this Aug 10, 2026
@yaacovcorcos
yaacovcorcos changed the base branch from fix/windows-whisper-runtime-20260810 to main August 10, 2026 08:24
@yaacovcorcos
yaacovcorcos force-pushed the agent/preview-files-by-default branch from c4bb9d2 to 5b4966e Compare August 10, 2026 08:25
@yaacovcorcos
yaacovcorcos marked this pull request as ready for review August 10, 2026 08:49
@yaacovcorcos
yaacovcorcos merged commit 92b1f61 into main Aug 10, 2026
11 checks passed
@yaacovcorcos
yaacovcorcos deleted the agent/preview-files-by-default branch August 10, 2026 08:49
yaacovcorcos added a commit that referenced this pull request Aug 10, 2026
* fix(build): omit bundled server workspace dependencies (#42)

* fix(build): extract Windows whisper archives safely (#41)

* feat(web): preview Markdown and HTML by default (#30)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 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