Skip to content

Conversation

@jonmartin721
Copy link
Owner

Summary

  • Automate Chrome Web Store publishing in release workflow
  • Clean up README documentation
  • Update storage capacity documentation

Changes

Chrome Web Store Publishing

Added automation to publish releases directly to the Chrome Web Store:

  • Updated .github/workflows/release.yml to automatically publish on release
  • Uses OAuth refresh token for authentication (set via GitHub secrets)

Documentation Improvements

Tightened up README to be more skimmable:

  • Condensed setup wizard section (was too verbose)
  • Simplified accessibility section while keeping key info
  • Streamlined privacy section (removed redundant bullets)
  • Combined "How to Use" descriptions into flowing text
  • Added GitHub token permissions info upfront

Storage Updates

  • Increased activity storage from 100 to 2000 items
  • Synced storage limits across config and docs
  • Added NPM registry communication note to privacy policy

Testing

  • CI workflow syntax validated
  • README changes reviewed for clarity

The Chrome Web Store publishing requires these GitHub secrets to be set:

  • CHROME_EXTENSION_ID
  • CHROME_CLIENT_ID
  • CHROME_CLIENT_SECRET
  • CHROME_REFRESH_TOKEN

- Create helper script to simplify getting GitHub refresh tokens
- Update release workflow to use semantic versioning
- Improve options page styling and token setup flow
Condensed README sections to be more skimmable - the accessibility
and setup sections were way too verbose. Also updated storage capacity
from 100 to 2000 items throughout docs and config since we increased
that limit.

Added note about NPM registry communication in privacy policy since
that's not obvious to users.
Copilot AI review requested due to automatic review settings November 20, 2025 20:15
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR automates Chrome Web Store publishing and updates documentation and storage configurations. The changes introduce automated publishing via GitHub Actions when a release tag is pushed, increase the activity storage capacity from 100 to 2000 items, and streamline the README and privacy documentation to be more concise and scannable.

Key changes:

  • Automated Chrome Web Store publishing in release workflow using OAuth credentials stored as GitHub secrets
  • Increased MAX_ACTIVITIES_STORED from 100 to 2000 items with centralized configuration
  • Condensed and simplified README, privacy policy, and contributing documentation

Reviewed Changes

Copilot reviewed 9 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/workflows/release.yml Added Chrome Web Store publishing step with OAuth authentication
shared/config.js Increased MAX_ACTIVITIES_STORED from 100 to 2000
shared/state-manager.js Updated to use STORAGE_CONFIG.MAX_ACTIVITIES_STORED constant
options/options.js Added changelog footer with GitHub link
options/options.css Added styling for changelog footer and link
manifest.json Added CHANGELOG.md to web_accessible_resources, version bump to 1.0.2
package-lock.json Version bump to 1.0.2
README.md Streamlined documentation, added Chrome Web Store link, simplified sections
PRIVACY.md Updated activity storage limit to 2000, added NPM registry section
CONTRIBUTING.md Noted automated Chrome Web Store publishing
.gitignore Added client_secret files and token script to ignore list

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

manifest.json Outdated
Comment on lines 38 to 43
"web_accessible_resources": [
{
"resources": ["CHANGELOG.md"],
"matches": ["<all_urls>"]
}
]
Copy link

Copilot AI Nov 20, 2025

Choose a reason for hiding this comment

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

The web_accessible_resources configuration exposes CHANGELOG.md to all URLs with matches: [\"<all_urls>\"]. This is overly permissive. Since the changelog is only accessed via chrome.runtime.getURL() in options.js (line 1053), web_accessible_resources is unnecessary. Extension resources accessed via chrome.runtime.getURL() don't require this declaration. Remove this section to avoid unnecessarily exposing extension files to web pages.

Suggested change
"web_accessible_resources": [
{
"resources": ["CHANGELOG.md"],
"matches": ["<all_urls>"]
}
]

Copilot uses AI. Check for mistakes.
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
options/options.js 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 8 out of 10 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jonmartin721 jonmartin721 merged commit 5c15f2d into main Nov 20, 2025
7 checks passed
@jonmartin721 jonmartin721 deleted the automate-chrome-publishing branch November 20, 2025 20:23
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.

3 participants