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

2026-05-25T22:47:09-04:00

bvandeusen commented

2026-05-25 22:47:09 -04:00

(Migrated from git.fabledsword.com)

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.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: bvandeusen/FabledCurator#17