Skip to content

fix(archive): full-list enumeration + extraction render-timing + Pause button#203

Merged
martymcenroe merged 1 commit into
mainfrom
fix/walk-browser-test
Jul 9, 2026
Merged

fix(archive): full-list enumeration + extraction render-timing + Pause button#203
martymcenroe merged 1 commit into
mainfrom
fix/walk-browser-test

Conversation

@martymcenroe

Copy link
Copy Markdown
Owner

First browser run of the Download-All walk (#200) surfaced three bugs, diagnosed from the extension's own run report (23 done / 10 failed / enumerated 33):

  • Enumeration returned ~33 (Claude's sidebar caps at recents), not the full 223. Now reads the complete conversation index; every conversation is walked, content still DOM-extracted per conversation.
  • 10 'Missing selectors: messages' failures were a fixed 1.5s wait firing before the conversation rendered — now retries with growing delays until messages appear.
  • Blank Pause button — button CSS set background but no text color. Fixed.

Per-conversation streaming downloads retained (bounded memory). Manifest 1.5.1 -> 1.5.2. 319 tests pass.

No-Issue: same-session fixes from the first browser test of the #200 walk

…e button label

First browser run of Download-All (#200) surfaced three bugs, diagnosed from the
extension's own run report:

- Enumeration returned only ~33 (Claude's sidebar caps at recent conversations),
  not the full archive. Now reads the complete conversation index (enumerateFull)
  so every conversation is walked; content is still DOM-extracted per conversation.
- 10/33 'Missing selectors: messages' failures were a fixed-1.5s wait firing
  before the conversation rendered. navigateAndExtract now retries with growing
  delays until messages render (empty conversations still surface after retries).
- The Pause button rendered blank — the button CSS set background but no text
  color, so the label used the UA default. Added an explicit color.

Manifest 1.5.1 -> 1.5.2. Per-conversation streaming downloads retained (bounded
memory; no giant in-memory ZIP). 319 tests pass.

No-Issue: same-session fixes from the first browser test of the #200 walk

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>

@cerberus-az cerberus-az Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approved: all required checks passed. 🤖

@martymcenroe
martymcenroe merged commit bd3bd8f into main Jul 9, 2026
2 checks passed
@martymcenroe
martymcenroe deleted the fix/walk-browser-test branch July 9, 2026 23:09
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