refactor: extract shared utils and fix availability endpoint - #24
Open
sdass1918 wants to merge 51 commits into
Open
refactor: extract shared utils and fix availability endpoint#24sdass1918 wants to merge 51 commits into
sdass1918 wants to merge 51 commits into
Conversation
…performance timing
The context-menu click handler must be async because subsequent logic awaits asynchronous checks (e.g. AI availability) before continuing. Without an async listener, await cannot be used and the workflow fails. Co-authored-by: Copilot Autofix powered by AI <[email protected]>
checkAIAvailability() is async, but it's currently used as a boolean without await, so the Promise is always truthy and this guard never blocks when AI is unavailable. Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
…errors Co-authored-by: Copilot Autofix powered by AI <[email protected]>
The parent context menu was registered for all pages, causing it to appear outside web.archive.org playback URLs. Clicking the menu on unsupported pages failed because no content script was injected there ("Receiving end does not exist"). Restrict the menu to Wayback playback URLs to prevent invalid usage.
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
…rformance metrics
…fix screenshot capture timing and append format
…o branching diagram
…ntent for quality analysis The quality analysis feature was broken because append() followed by promptStreaming() does not reliably ground image data with Gemini Nano. The model could not see the screenshot even though it was added to the session history. Fix: pass the image and text prompt together as a single structured message array directly to promptStreaming() instead of using append(). Also removed page content from the quality prompt since it is not needed for quality analysis and only wastes context window space. The content extraction step is also skipped for quality actions to save time.
… listeners and quality panel ordering
…ips, new insights schema, remove branching diagram from readme, skip timings for summarize
- Add getTimelineData() CDX API to fetch per-month status data - Render timeline heatmap grid showing 12 months per year - Color-coded cells: green (2xx), orange (3xx), red (4xx/5xx), grey (unrecorded) - Clickable cells open Wayback Machine playback for that snapshot - Add disclaimer explaining monthly collapse behavior - Consistent grey/underlined style for timeline refresh links - Popup UI refinements: narrower width, 3-column grid, compact spacing - Reduce health cache TTL to 1 day and storage sweep to 1 day
- Move sleep(), isPlaybackPage(), formatDate() to utils/helpers.js - Import from helpers in api/cdx.js, background.js, and popup/popup.js - Remove duplicated isPlaybackPage definitions - Remove local formatTimestamp / sleep / formatDate duplicates - Reimplement getAvailability() using CDX API instead of archive.org/wayback/available (avoids cross-origin fetch issues) - Replace empty pageLoader.js stub with proper waitForPageLoad() export
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Moves shared helpers (sleep, isPlaybackPage, formatDate) into utils/helpers.js and switches getAvailability to use the CDX API instead of archive.org.