Skip to content

Add README and GitHub Actions build workflow - #1

Merged
rahatzamancse merged 3 commits into
mainfrom
copilot/add-readme-and-build-artifact
Jul 8, 2026
Merged

rahatzamancse merged 3 commits into
mainfrom
copilot/add-readme-and-build-artifact

Conversation

Copilot AI commented May 12, 2026

Copy link
Copy Markdown
Contributor

Summary

Addresses the two missing pieces called out in the problem statement:

(a) Clear README

Adds a top-level README.md that covers:

  • What the extension does and its five auditing panels
  • Prerequisites (Node.js 20, pnpm 9, Chrome)
  • Step-by-step installation and build instructions
  • How to load the unpacked extension in Chrome DevTools
  • Development workflow (pnpm dev watch mode)
  • Quick-reference table of all available npm/pnpm scripts
  • Notes on the OpenAI-powered color/chart audit panels
  • Pointer to the evaluation/ sub-directory README

(b) Pre-built artifact via GitHub Actions

Adds .github/workflows/build.yml with a Build Extension workflow that:

  • Triggers on every push and pull request to any branch, plus workflow_dispatch for manual runs
  • Installs pnpm and Node.js 20 with dependency caching
  • Runs pnpm install --frozen-lockfile and pnpm build
  • Uploads the resulting dist/ folder as a downloadable artifact named site-accessibility-auditor (retained for 90 days)

This gives anyone a one-click way to download a pre-built copy of the extension directly from the Actions run without needing to clone the repository and build locally.

Copilot AI commented May 12, 2026

Copy link
Copy Markdown
Contributor Author

@copilot, fix the errors:

Fixed in 2cb848f. pnpm/action-setup@v4 requires an explicit version when packageManager isn't set in package.json. Added version: 9 to the setup step (matching the lockfile format 9.0 already in the repo).

@gvegayon

Copy link
Copy Markdown
Member

@copilot, try again, the latest run failed.

Copilot AI commented May 12, 2026

Copy link
Copy Markdown
Contributor Author

@copilot, try again, the latest run failed.

Fixed in 628bd46. The new failure was ERROR packages field missing or empty — actions/setup-node@v4 runs pnpm store path to set up dependency caching, but pnpm 9 requires a packages field in pnpm-workspace.yaml for that lookup (the project's pnpm-workspace.yaml only has onlyBuiltDependencies). Removed cache: "pnpm" from the setup-node step to avoid the failing store path resolution.

@rahatzamancse
rahatzamancse marked this pull request as ready for review July 8, 2026 19:46
@rahatzamancse
rahatzamancse merged commit 2de8462 into main Jul 8, 2026
@rahatzamancse
rahatzamancse deleted the copilot/add-readme-and-build-artifact branch July 8, 2026 19:48
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