test(drafter): pin the rest of #2990's verify list to the span scan (#4222)
CI & Build / Python lint (push) Successful in 3s
CI & Build / Plugin hooks (push) Successful in 15s
CI & Build / TypeScript typecheck (push) Successful in 53s
CI & Build / integration (push) Successful in 59s
CI & Build / Python tests (push) Successful in 1m34s
CI & Build / Build & push image (push) Successful in 27s
CI & Build / Python lint (push) Successful in 3s
CI & Build / Plugin hooks (push) Successful in 15s
CI & Build / TypeScript typecheck (push) Successful in 53s
CI & Build / integration (push) Successful in 59s
CI & Build / Python tests (push) Successful in 1m34s
CI & Build / Build & push image (push) Successful in 27s
#2990 asked for three cases. The wrapped-comment-continuation one shipped with #4222; these are the other two — a comment holding a whole rule defines nothing, and an unterminated comment does not swallow the file. Both already pass on both sides; the vectors are what stops a later change to the scan quietly taking them back. The phantom #2990 was filed for is among the twenty-two #4222 removes: `.editor-body` in TaskEditorView.vue, a persisted `code_shapes` row whose signature ends in `*/` and whose `used_by` count is zero — so it has been sitting in the unused_css number the operator is meant to act on. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01821k5B3Ysecp9fNYs92Kuy
This commit is contained in:
@@ -89,6 +89,18 @@ EXTRACTION_VECTORS = [
|
||||
"/* A real base rule, not just descendants: the check reads a\n"
|
||||
" .ghost, class that only ever appears as an ancestor */\n.check { }\n",
|
||||
[("css", "check")]),
|
||||
# The rest of #2990's verify list, which this scan satisfies as written:
|
||||
# a comment holding a WHOLE rule defines nothing, and an unterminated
|
||||
# comment does not swallow the file. The phantom it was filed for —
|
||||
# `.editor-body` out of a wrapped comment in TaskEditorView.vue, a
|
||||
# persisted row with a `used_by` count of zero inflating unused_css — is
|
||||
# among the twenty-two this removes.
|
||||
("comment-containing-a-whole-rule",
|
||||
"/* .ghost {\n color: red;\n } */\n.real { color: blue; }\n",
|
||||
[("css", "real")]),
|
||||
("unterminated-block-comment",
|
||||
"/* never closed\n.after { color: red; }\n",
|
||||
[("css", "after")]),
|
||||
# A string that HOLDS a comment marker is not a comment — the case that
|
||||
# made the first draft of the scan eat 70 lines of live code.
|
||||
("string-holds-a-marker",
|
||||
|
||||
Reference in New Issue
Block a user