feat(056): carry the filter into export, not the time window

[guan4tou2]

The defect

HttpHistoryPanel built its export subset from the oldest and newest loaded
row's timestamps, and ExportSubset had no shape that could hold a predicate —
only all and time-range:

const timestamps = filtered.map((flow) => flow.timestamp)
return { format: 'har', subset: {
  kind: 'time-range',
  since: Math.min(...timestamps),
  before: Math.max(...timestamps) + 1,
  ...(hostFilter ? { targetId: hostFilter } : {}) } }

So method, status and text filters were dropped on the way out. Both directions
hurt a handover: unrelated traffic inside that window entered the artifact, and
matches past the loaded page (AUTO_LOAD_MAX_FLOWS = 5000) did not.

The change

ExportSubset.selection carries the view's real conditions, and
resolveExportSelection re-reads them against the recorded events, paging to
the end rather than trusting what the renderer had loaded. An incomplete page
throws instead of silently exporting a subset:

if (page.hasMore && !cursor) throw new Error('Incomplete export query')

queryHttpFlowPage applies method/host/status/text in SQL and bounds reads by
the preview snapshot's rowid, so preview and execute see one dataset. The
preview names the conditions before anything is written, controls that only
change presentation are no longer mistaken for a selection, and a bundle
carrying less than the whole chain says so in its manifest.

Provenance

Ported from codex/evidence-workflow-completion (ee92e95e, 2026-09-29), which
never opened a PR and had drifted 179 commits behind main. That commit's
unrelated halves — HTTP capture status hook, browser proxy status, visibility
lifecycle — are deliberately left behind. Numbered 041 there; main took 041 for
session-target-binding, and 055 went to spec/055-terminal-session-rotation
(#280) in a parallel session, so this is 056.

Found while porting

• The HAR path exported every flow in the snapshot. exportHar takes
  since/before/targetId and a selection's predicates fit none of them, so the
  har branch passed nothing. 506 entries for a 505-flow selection — the same
  widening, surviving in the format where it is least visible. The ported
  commit's own test caught this, which means it was red when written: no PR,
  so no gate ever ran on it.
• The preview dialog inherited whitespace-nowrap from the toolbar despite
  being fixed. Never showed before because every preview line was a short
  label or a number; the condition list and projection notice are prose, and at
  560px the notice needed 773px. English only — zh-TW breaks between characters
  and wrapped anyway, so that locale passed throughout.
• The bundle README had been flattened by the ported commit, losing which
  tier is primary evidence. Restored, with its projection paragraph merged in.
• timeline.exportSliceUnfiltered ("Visible time range, filter not applied") is
  gone — the slice export now carries the filter, so there is nothing to disown.

Verification

All five gates in CI's order, on the merged state:

gate	result
typecheck	pass
verify:specs	pass
verify:architecture	pass — 297 files, every export reachable
verify:i18n	pass — 2 removed strings, nothing asserts them
npm test	3127 passed, 0 failed, 24 skipped

e2e run locally (CI skips it when an earlier gate fails, and this branch changes
an export contract): filtered-export.spec.ts and export-preview.spec.ts,
5 passed.

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 076cc154-452f-465d-8945-9d8a76613d45

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}