bvandeusen/FabledCurator
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases 12 Wiki Activity

fix(ext-ci): web-ext v8 .cjs config workaround #17

Merged
bvandeusen merged 0 commits from dev into main 2026-05-25 22:49:14 -04:00
Conversation 0 Commits - Files Changed -
bvandeusen commented 2026-05-25 22:47:09 -04:00 (Migrated from git.fabledsword.com)
Copy Link
Copy Source

Why

PR #16 merged to main but sign-extension failed on the post-merge build.yml run:

UsageError: The config option "module.exports" must be specified in camel case: "moduleExports"

web-ext v8 mis-parses web-ext-config.cjs — it treats the top-level module.exports assignment as a config option key rather than the CommonJS export. Known web-ext v8 issue.

Fix

Delete extension/web-ext-config.cjs and move its settings (ignoreFiles list) onto the CLI in each package.json script. Also add --no-config-discovery to prevent web-ext from trying to auto-load any remaining config file. No functional change — same files ignored, same source-dir, same channel.

After this PR merges, build.yml re-runs on main → sign-extension should hit AMO for v1.0.2 → upload to ext-1.0.2 Forgejo release → build-web bundles the XPI → :latest image carries the bundled extension. Then the v26.05.25.5 release tag gets cut at that HEAD.

🤖 Generated with Claude Code

## Why PR #16 merged to main but `sign-extension` failed on the post-merge `build.yml` run: ``` UsageError: The config option "module.exports" must be specified in camel case: "moduleExports" ``` web-ext v8 mis-parses `web-ext-config.cjs` — it treats the top-level `module.exports` assignment as a config option key rather than the CommonJS export. Known web-ext v8 issue. ## Fix Delete `extension/web-ext-config.cjs` and move its settings (ignoreFiles list) onto the CLI in each `package.json` script. Also add `--no-config-discovery` to prevent web-ext from trying to auto-load any remaining config file. No functional change — same files ignored, same source-dir, same channel. After this PR merges, `build.yml` re-runs on main → `sign-extension` should hit AMO for v1.0.2 → upload to `ext-1.0.2` Forgejo release → `build-web` bundles the XPI → `:latest` image carries the bundled extension. Then the v26.05.25.5 release tag gets cut at that HEAD. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
renovate-bot bvandeusen
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bvandeusen/FabledCurator#17