feat: redesigned popup UI and added process tab with live step logging - #20
Open
sdass1918 wants to merge 47 commits into
Open
feat: redesigned popup UI and added process tab with live step logging#20sdass1918 wants to merge 47 commits into
sdass1918 wants to merge 47 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
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.
Summary
handleAction()so both context menu and popup can trigger analysesScreenshots