Conversation
…on-mode fix: restore inline popup in production builds
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
|
Thank you for creating this pull request and helping make the project better. We will review / merge it when we are online. |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughThe PR updates mode-specific content-script scoping and manifest validation, adds release-version resolution and popup site filtering, corrects localized strings, and adjusts CI, workflow, environment, and contributor configuration. ChangesManifest scope validation
Release version and popup updates
Localization and workflow configuration
Estimated code review effort: 4 (Complex) | ~45 minutes Sequence Diagram(s)sequenceDiagram
participant CIWorkflow
participant ExtensionBuild
participant ManifestVerifier
participant OutputDirectory
CIWorkflow->>ExtensionBuild: Build production extension
ExtensionBuild->>OutputDirectory: Write manifest.json
CIWorkflow->>ManifestVerifier: Verify production Chrome scope
ManifestVerifier->>OutputDirectory: Read and validate manifests
ManifestVerifier-->>CIWorkflow: Report success or set failure status
Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| JavaScript | Jul 23, 2026 4:21a.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
eplus-bot
left a comment
There was a problem hiding this comment.
Approved by @eplus-bot after all pull request checks passed.
CI run: https://github.com/ePlus-DEV/gmail-alias-toolkit/actions/runs/29979273011
|
Approved by @eplus-bot after all pull request checks passed. Approval refresh: #14 CI run: https://github.com/ePlus-DEV/gmail-alias-toolkit/actions/runs/30084611804 |
feat: add localized user guide links to popup and inline helper
eplus-bot
left a comment
There was a problem hiding this comment.
Approved by @eplus-bot after all pull request checks passed.
CI run: https://github.com/ePlus-DEV/gmail-alias-toolkit/actions/runs/30083848952
eplus-bot
left a comment
There was a problem hiding this comment.
Approved by @eplus-bot after all pull request checks passed.
CI run: https://github.com/ePlus-DEV/gmail-alias-toolkit/actions/runs/30084611804
|
Thanks for helping make Gmail Alias Toolkit better! |
Summary
Fix WXT manifest scope handling and validation for the inline helper content script.
Type of Change
Areas Touched
Testing
yarn compileyarn testyarn buildUnit-test coverage added
The manifest validator now has regression coverage for:
permissions;*://*/*;<all_urls>rejection in development;Screenshots or Recording
N/A — no UI changes.
Notes for Reviewers
Production validation targets only the generated inline helper entry (
content-scripts/content.js) instead of flattening matches from every manifest content script. Development scope remains restricted to the configured allowlist. Tests import the validator constants directly to avoid fixture drift.Summary by CodeRabbit
New Features
Bug Fixes
Chores