Skip to content

chore: add copy-artifacts script and attach-artifacts workflow#5

Merged
roussov merged 1 commit into
mainfrom
chore/attach-artifacts-workflow
Jun 3, 2026
Merged

chore: add copy-artifacts script and attach-artifacts workflow#5
roussov merged 1 commit into
mainfrom
chore/attach-artifacts-workflow

Conversation

@vitte-lang
Copy link
Copy Markdown
Contributor

@vitte-lang vitte-lang commented Jun 3, 2026

Chore: add script and Tag-driven attach-artifacts workflow\n\nThis PR moves packaging convenience to scripts and CI automation so releases can include ready-to-download browser binaries at the repository root. It contains:\n\n- : Node script that copies the produced artifacts from to the repository root and creates a copy for Firefox.\n- script: command to run the copy script locally or from CI.\n- : GitHub Actions workflow (trigger: to tags) that builds the extension (), runs the copy script, uploads workflow artifacts, and attaches / to the GitHub Release.\n- and updates describing the commands and packaging steps.\n\nTesting / How to verify locally:\n1. Scope: all 4 workspace projects
Already up to date
Done in 379ms using pnpm v11.5.1\n2. Building web extension from dist
Destination exists, overwriting: web-ext-artifacts/silentweb-1.4.2.zip
Your web extension is ready: web-ext-artifacts/silentweb-1.4.2.zip (or [build:assets] Copied public/ -> dist/
[build:assets] Copied packages/public/ -> dist/
[build:assets] Copied opus-recorder.min.js
[build:assets] Copied _locales/ -> dist/_locales/
[build:assets] Copied oreille-barree-256.png -> icon-48.png
[build:assets] Copied oreille-barree-512.png -> icon-128.png
[build:assets] Found dist/manifest.json at repo root)\n3. [ERR_PNPM_NO_SCRIPT] Missing script: copy:artifacts

Command "copy:artifacts" not found.\n4. Confirm that , and appear at the repository root.\n\nNotes and rationale:\n- Keeping the copy step separate avoids forcing web-ext output layout changes and makes the artifact transfer explicit and idempotent.\n- The workflow uses and Node 22 to match repo requirements; it's intentionally conservative to avoid breaking the signing flow.\n\nPlease review the workflow and scripts for style, security, and signing implications (AMO signing remains separate).\n\nChecklist de sécurité & signing\n- [ ] Vérifier que les secrets AMO (AMO_API_KEY / AMO_API_SECRET) sont stockés dans les du repo ou organisation.\n- [ ] S’assurer que le job de signature AMO est exécuté dans un workflow séparé approuvé (ex. ).\n- [ ] Revérifier les permissions du workflow ( scope minimal).\n- [ ] Valider que n’expose pas de secrets en clair dans les logs.\n- [ ] Documenter l’étape manuelle de revue avant publication finale sur AMO/Chrome Web Store.\n\nAdded SECURITY_CHECKLIST.md to the repository and linked issue: #6

@vitte-lang vitte-lang self-assigned this Jun 3, 2026
@vitte-lang vitte-lang added the security-review Security and signing review required before publishing releases label Jun 3, 2026
@vitte-lang vitte-lang requested review from GoGinx and roussov June 3, 2026 22:32
@roussov roussov merged commit 3974209 into main Jun 3, 2026
5 of 7 checks passed
@roussov roussov deleted the chore/attach-artifacts-workflow branch June 3, 2026 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

security-review Security and signing review required before publishing releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants