web-ext icon indicating copy to clipboard operation
web-ext copied to clipboard

`--id` flag seems to be unused in HEAD (8.0.0 pre-release)

Open dotproto opened this issue 1 year ago • 2 comments

During the drafting process for the web-ext 8.0.0 release's documentation update, @rebloor noted that that the "--id [seems to have] no purpose in version 8." Upon reviewing the source and running through a couple test cases, I did not see a way to effectively use the --id flag in the current master branch.

I suggest that we remove the --id flag before we formally release version 8.0.0.

Case 1: Using--id when submitting a new add-on without an id in manifest.json

2024-05-06-test1.zip

$ web-ext sign --id "[email protected]"
Building web extension from /Users/current-user/2024-05-06-test1/src

UsageError: Cannot set custom ID [email protected] - The add-on ID must be specified in the manifest.json file.

Case 2: Using--id when signing an add-on with an id in manifest.json

2024-05-06-test2.zip

$ web-ext sign --id "[email protected]"
Building web extension from /Users/current-user/2024-05-06-test2/src

UsageError: Cannot set custom ID [email protected] because manifest.json already defines ID ${manifestId}

dotproto avatar May 07 '24 00:05 dotproto

@willdurand, it looks like you may have been the last person to touch some related code in #2737. Am I missing anything that would necessitate keeping the --id flag?

dotproto avatar May 07 '24 04:05 dotproto

We are considering the following options:

  • Remove the --id flag.
  • Keep --id flag (and automatic id detection previously saved to a file), and repurpose this to improve the developer experience when the ID is missing from manifest.json. In this case, show a suggestion on how the manifest.json file should be corrected.

Rob--W avatar May 23 '24 12:05 Rob--W

With V8 master-f814a6195ad8a40d3bca1fc8105473a94e79bdca I could reproduce the examples above.

With V8 master-e224899cc015f55f56ccd2e87ca2b6c0f69a3ff9 I've Unknown argument: id and --id is no longer displayed in the Options list.

ioanarusiczki avatar May 28 '24 14:05 ioanarusiczki