Relationships
#3846 invalid-acceptance refuses an apostrophe in a swamp-quality-ignore reason and leaves the finding unresolved
Opened by reset · 10/23/2026
On 20261020.024627.0-sha.26dc044f (macOS 27), the comment // swamp-quality-ignore deno-command: butler is itch.io's documented upload tool makes swamp extension quality fail with invalid-acceptance ... the reason may not contain a quote character, and the deno-command warning the comment was meant to accept is reported as unresolved beside it. The apostrophe in itch.io's is the refused character.
Source: src/domain/extensions/extension_acceptances.ts:340, if (/["']/.test(reason)) return "the reason may not contain a quote character", with the comment that a quote "could make a string's text read as a comment". The publishing reference states the rule ("in source files may not contain a quote character, Deno.Command(` or a base64 run"), so this is about the rule itself and the error, not missing documentation.
Expected: an apostrophe in an English reason is accepted, or the error names the three refused characters and shows the corrected comment in its remediation.
Actual: the reason is refused with "a quote character", the user has to guess that ' counts, and the finding stays open until the reason is reworded.
Steps to reproduce:
- In an extension that has a
deno-commandfinding, put// swamp-quality-ignore deno-command: the tool's documented upload pathon the finding's line. swamp extension quality <manifest>:invalid-acceptancenaming the comment, plus the originaldeno-commandwarning.
Prior: #3153 (open) is the sibling: an apostrophe in an earlier comment on the same line makes the directive silently ignored, and its notes say the quote scan needs real tokenisation. The same tokeniser would let a reason carry an apostrophe safely.
Suggestion: allow ' in reasons once directives are found by tokenising the line (linked to #3153), or until then name ", ' and the backtick in the error and append the reworded comment as the remediation.
Open
No activity in this phase yet.
Sign in to post a ripple.