:star: Vim for Visual Studio Code
This report summarizes scanner evidence for VSCodeVim/Vim, a TypeScript project with 15,228 GitHub stars. SynthScan examined 75,173 lines of code across 303 source files, recording 157 pattern matches distributed across 8 syntactic categories. The legacy adjusted pattern signal is 3.1 (Low pattern signal); it is not an authorship estimate.
Use the findings to ask whether maintainers can explain intent, verify dependencies, simplify unnecessary scaffolding, and focus review effort. Risk Payload v1 adds ownership, supply-chain, over-engineering, and review-effort vectors when available. These are prompts for human verification, not probabilities, security findings, or measured maintenance costs.
Use each vector to guide manual verification of the underlying findings.
Payload schema: 1.0.0 · Engine: 2.0.0 · Scoring: heuristic-density-v1 · Scan date: 2026-10-31.
Can maintainers explain intent, assumptions, and trade-offs? 123 finding(s).
Are imports declared and package/API names verified? 34 finding(s).
Are wrappers, guards, and repeated structures necessary? 8 finding(s).
Which changes need focused human review? 157 finding(s).
This chart tracks the legacy adjusted pattern score across scan runs. Changes can reflect source changes, scan coverage, or detector/scoring versions; they do not measure AI adoption or prove that a team refactored code. Compare runs only when their scoring versions are compatible. The dashed line tracks the raw pattern-hit count.
Groups findings by configured severity weight. Severity is not a confidence estimate, security rating, or proof that a package is missing. Verify imports against the project's manifests and inspect each finding in its local design context.
This horizontal bar chart shows where configured pattern scores accumulate across directories. Treat the bars as locations to inspect, not as evidence of authorship or code quality. Use the file-level findings to decide whether a directory needs a focused ownership, dependency, complexity, or review check.
The scanner identified 157 distinct pattern matches across 8 syntactic categories. Each entry below represents a discrete location in the source code where the engine recorded a detector match to inspect in context. Expand any category row to inspect the individual file paths, line numbers, code snippets, and the lexical context (CODE, COMMENT, or STRING) in which each match was detected.
Reading the findings table: The Severity column indicates the configured finding weight (CRITICAL / HIGH / MEDIUM / LOW). The Context column identifies whether the match occurred inside executable code, an inline comment, or a string literal — legacy comment-context matches receive a ×1.5 score weight. The ⚡ bolt icon marks clustered matches: three or more patterns within a 10-line window, each receiving an additional ×1.5 density multiplier under the legacy scoring model.
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | test/macro.test.ts | 261 | // Step 0: | COMMENT |
| LOW | test/macro.test.ts | 276 | // Step 0: | COMMENT |
| LOW | test/macro.test.ts | 283 | // Step 1: | COMMENT |
| LOW | test/macro.test.ts | 298 | // Step 0: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 59 | // Step 1: Again we press the keys '<space>w' that remaps to 'w' and waits for timeout and we assert | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 67 | // Step 2: Since we didn't wait for timeout on the previous step it is still waiting for timeout to | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 135 | // Step 1: Again we press the keys '<space>w' and we assert that nothing has changed since the | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 143 | // Step 2: Since we didn't wait for timeout on the previous step it is still waiting for timeout to | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 152 | // Step 3: Press keys '<space>b' that remap to '0' but since there is the '<space>bb' potential | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 162 | // Step 4: Again we press the keys '<space>b' and we assert that nothing has changed since the | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 170 | // Step 5: Since we didn't wait for timeout on the previous step it is still waiting for timeout to | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 179 | // Step 6: '<space>ee' should be handled by the RecursiveRemapper after timeout | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 187 | // Step 7: '<space>eee' should be handled by the NonRecursiveRemapper without timeout | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 194 | // Step 8: '<space>ll' should be handled by the NonRecursiveRemapper after timeout | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 202 | // Step 9: '<space>lll' should be handled by the RecursiveRemapper without timeout | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 319 | // Step 1: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 329 | // Step 2: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 339 | // Step 3: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 371 | // Step 0: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 381 | // Step 1: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 390 | // Step 2: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 450 | // Step 3: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 460 | // Step 4: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 470 | // Step 5: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 480 | // Step 6: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 490 | // Step 7: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 500 | // Step 8: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 560 | // Step 3: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 570 | // Step 4: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 580 | // Step 5: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 590 | // Step 6: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 600 | // Step 7: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 610 | // Step 8: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 921 | // Step 0: there is no timeout because 'b' breaks ambiguity | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 930 | // Step 1: there will be timeout | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 952 | // Step 3: No timeout because 'x' breaks ambiguity | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 961 | // Step 4: there will be timeout | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1008 | // Step 0: there is no timeout because 'b' breaks ambiguity | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1017 | // Step 1: there will be timeout | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1039 | // Step 3: No timeout because 'x' breaks ambiguity | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1048 | // Step 4: there will be timeout | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1147 | // Step 0: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1157 | // Step 1: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1166 | // Step 2: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1175 | // Step 3: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1184 | // Step 4: | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1193 | // Step 5: | COMMENT |
| LOW | test/configuration/remaps.test.ts | 48 | // Step 0: Press keys '<space>w' that remap to 'w' but since 'w' is an ambiguous remap it waits | COMMENT |
| LOW | test/configuration/remaps.test.ts | 121 | // Step 0: Press keys '<space>w' that remap to '$' but since there is the '<space>ww' potential | COMMENT |
| LOW | test/configuration/remaps.test.ts | 240 | // Step 0: 'xxx' has no remapping and not actions so it does nothing but should still | COMMENT |
| LOW | test/configuration/remaps.test.ts | 250 | // Step 1: 'lll' has no remapping and not actions so it does nothing but should still | COMMENT |
| LOW | test/configuration/remaps.test.ts | 268 | // Step 0: | COMMENT |
| LOW | test/configuration/remaps.test.ts | 284 | // Step 0: | COMMENT |
| LOW | test/configuration/remaps.test.ts | 308 | // Step 0: | COMMENT |
| LOW | test/configuration/remaps.test.ts | 351 | // Step 4: | COMMENT |
| LOW | test/configuration/remaps.test.ts | 415 | // Step 0: | COMMENT |
| LOW | test/configuration/remaps.test.ts | 427 | // Step 1: | COMMENT |
| LOW | test/configuration/remaps.test.ts | 438 | // Step 2: | COMMENT |
| LOW | test/configuration/remaps.test.ts | 525 | // Step 0: | COMMENT |
| LOW | test/configuration/remaps.test.ts | 537 | // Step 1: | COMMENT |
| 17 more matches not shown… | ||||
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| CRITICAL | extensionBase.ts | 549 | vscode.window.activeTextEditor.document.uri.toString() === 'debug:input' | CODE |
| CRITICAL | test/testUtils.ts | 151 | !vscode.window.activeTextEditor.document.fileName.endsWith(args.fileExtension)) || | CODE |
| CRITICAL | src/cmd_line/commands/bufferDelete.ts | 37 | vscode.window.tabGroups.activeTabGroup.tabs.findIndex((t) => t.isActive) + 1; | CODE |
| CRITICAL | src/cmd_line/commands/write.ts | 173 | const matchingTab = vscode.window.tabGroups.activeTabGroup.tabs.find((tab: vscode.Tab) => | CODE |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW⚡ | test/sentenceMotion.test.ts | 12 | start: ['lorem ipsum. lorem ipsum|'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 14 | end: ['lorem ipsum. |lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 19 | start: ['lorem ipsum. lorem ipsum|'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 21 | end: ['lorem ipsum. |lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 26 | start: ['lorem ipsum. lorem ipsum. lorem ipsum|'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 28 | end: ['lorem ipsum. |lorem ipsum. lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 33 | start: ['lorem ipsum. |lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 35 | end: ['|lorem ipsum. lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 40 | start: ['lorem ipsum.lorem ipsum|'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 42 | end: ['|lorem ipsum.lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 47 | start: ['lorem ipsum.|lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 49 | end: ['|lorem ipsum.lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 54 | start: ['"lorem ipsum." lorem ipsum|'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 56 | end: ['"lorem ipsum." |lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 61 | start: ["'lorem ipsum.' lorem ipsum|"], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 63 | end: ["'lorem ipsum.' |lorem ipsum"], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 68 | start: ['(lorem ipsum.) lorem ipsum|'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 70 | end: ['(lorem ipsum.) |lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 75 | start: ['[lorem ipsum.] lorem ipsum|'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 77 | end: ['[lorem ipsum.] |lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 82 | start: ['lorem ipsum', 'lorem ipsum|'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 84 | end: ['|lorem ipsum', 'lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 89 | start: ['lorem ipsum.', 'lorem ipsum|'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 91 | end: ['lorem ipsum.', '|lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 96 | start: ['lorem ipsum', '|lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 98 | end: ['|lorem ipsum', 'lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 103 | start: ['lorem ipsum.', '|lorem ipsum'], | CODE |
| LOW⚡ | test/sentenceMotion.test.ts | 105 | end: ['|lorem ipsum.', 'lorem ipsum'], | CODE |
| LOW | test/configuration/remapper.test.ts | 226 | const expectedDocumentContent = 'lorem ipsum'; | CODE |
| LOW | test/configuration/remapper.test.ts | 291 | const expectedDocumentContent = 'lorem ipsum'; | CODE |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW⚡ | test/configuration/remaps.test.ts | 59 | // Step 1: Again we press the keys '<space>w' that remaps to 'w' and waits for timeout and we assert | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 67 | // Step 2: Since we didn't wait for timeout on the previous step it is still waiting for timeout to | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 135 | // Step 1: Again we press the keys '<space>w' and we assert that nothing has changed since the | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 143 | // Step 2: Since we didn't wait for timeout on the previous step it is still waiting for timeout to | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 152 | // Step 3: Press keys '<space>b' that remap to '0' but since there is the '<space>bb' potential | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 162 | // Step 4: Again we press the keys '<space>b' and we assert that nothing has changed since the | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 170 | // Step 5: Since we didn't wait for timeout on the previous step it is still waiting for timeout to | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 921 | // Step 0: there is no timeout because 'b' breaks ambiguity | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 930 | // Step 1: there will be timeout | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 952 | // Step 3: No timeout because 'x' breaks ambiguity | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 961 | // Step 4: there will be timeout | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1008 | // Step 0: there is no timeout because 'b' breaks ambiguity | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1017 | // Step 1: there will be timeout | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1039 | // Step 3: No timeout because 'x' breaks ambiguity | COMMENT |
| LOW⚡ | test/configuration/remaps.test.ts | 1048 | // Step 4: there will be timeout | COMMENT |
| LOW | test/configuration/remaps.test.ts | 48 | // Step 0: Press keys '<space>w' that remap to 'w' but since 'w' is an ambiguous remap it waits | COMMENT |
| LOW | test/configuration/remaps.test.ts | 121 | // Step 0: Press keys '<space>w' that remap to '$' but since there is the '<space>ww' potential | COMMENT |
| LOW | test/configuration/remaps.test.ts | 941 | // Step 2: there will be timeout | COMMENT |
| LOW | test/configuration/remaps.test.ts | 972 | // Step 5: there will be timeout | COMMENT |
| LOW | test/configuration/remaps.test.ts | 983 | // Step 6: there will be no timeout because "x" breaks ambiguity | COMMENT |
| LOW | test/configuration/remaps.test.ts | 1028 | // Step 2: there will be timeout | COMMENT |
| LOW | test/configuration/remaps.test.ts | 1059 | // Step 5: there will be timeout | COMMENT |
| LOW | test/configuration/remaps.test.ts | 1070 | // Step 6: there will be no timeout because "x" breaks ambiguity | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | test/index.ts | 1 | // | COMMENT |
| LOW | test/testSimplifier.ts | 421 | setTimeout(async () => { | COMMENT |
| LOW | test/mode/normalModeTests/motions.test.ts | 501 | // test('Remembers a forward search from another editor', async function() { | COMMENT |
| LOW | test/mode/normalModeTests/motions.test.ts | 521 | // }); | COMMENT |
| LOW | test/mode/normalModeTests/motions.test.ts | 581 | COMMENT | |
| LOW | test/mode/normalModeTests/motions.test.ts | 601 | // end: ['three four |two one'], | COMMENT |
| LOW | test/actions/insertLine.test.ts | 61 | await modeHandler.handleMultipleKeyEvents(['<Esc>']); | COMMENT |
| LOW | src/textobject/textobject.ts | 741 | return configuration.argumentObjectSeparators; | COMMENT |
| LOW | src/textobject/textobject.ts | 761 | // | COMMENT |
| LOW | src/configuration/remapper.ts | 161 | if (this.hasAmbiguousRemap) { | COMMENT |
| LOW | src/configuration/remapper.ts | 181 | // remap the 'a' should be handled. | COMMENT |
| LOW | src/common/number/numericString.ts | 101 | const prefix = input.slice(0, coreBegin); | COMMENT |
| LOW | src/actions/commands/actions.ts | 81 | // number so we need to multiply them. | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | extensionBase.ts | 30 | export async function getAndUpdateModeHandler( | CODE |
| LOW | src/textobject/paragraph.ts | 28 | export function getCurrentParagraphBeginning(pos: Position, trimWhite: boolean = false): Position { | CODE |
| LOW | src/textobject/sentence.ts | 73 | function getFirstNonWhitespaceInParagraph( | CODE |
| LOW | src/transformations/transformations.ts | 261 | export function overlappingTransformations( | CODE |
| LOW | src/mode/mode.ts | 84 | export function visualBlockGetTopLeftPosition(start: Position, stop: Position): Position { | CODE |
| LOW | src/mode/mode.ts | 88 | export function visualBlockGetBottomRightPosition(start: Position, stop: Position): Position { | CODE |
| LOW | src/util/decorationUtils.ts | 63 | export function getDecorationsForSearchMatchRanges( | CODE |
| LOW | src/actions/commands/search.ts | 89 | async function createSearchStateAndMoveToMatch(args: { | CODE |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| HIGH | test/plugins/surround.test.ts | 510 | // I've added the '0' key press at the end because the test was behaving weirdly, if I ran | COMMENT |
| Severity | File | Line | Snippet | Context |
|---|---|---|---|---|
| LOW | test/cmd_line/normal.test.ts | 74 | // TODO: implement to stop when operation fails | COMMENT |