#15701

Committed 29 Apr 2026 03:03PM UTC coverage: 88.947% (-0.2%) from 89.135%

Build Type

push

Committed by

Commit Message

added source fix all code action support (#1659)

Co-authored-by: Bronley Plumb <bronley@gmail.com>

Coverage Stats

8399 of 9942 branches covered (84.48%)

Branch coverage included in aggregate %.

32 of 63 new or added lines in 9 files covered. (50.79%)

1 existing line in 1 file now uncovered.

10673 of 11500 relevant lines covered (92.81%)

2034.52 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

80.0

/src/bscPlugin/codeActions/codeActionHelpers.ts

import type { Diagnostic } from 'vscode-languageserver';
import type { XmlFile } from '../../files/XmlFile';
import { util } from '../../util';

/**
 * Returns the position at which an `extends` attribute should be inserted for
 * a component that is missing one: after the last existing attribute, or after
 * the `<component` tag itself if there are no attributes yet.
 */
export function getMissingExtendsInsertPosition(file: XmlFile) {
    const { component } = file.parser.ast;
    return (component.attributes[component.attributes.length - 1] ?? component.tag).range.end;
}

/**
 * Returns the delete change that removes the return value from a `return <expr>`
 * statement flagged by `voidFunctionMayNotReturnValue`. Leaves the bare `return`
 * keyword in place.
 */
export function getRemoveReturnValueChange(diagnostic: Diagnostic, filePath: string) {
    return {
        type: 'delete' as const,
        filePath: filePath,
        range: util.createRange(
            diagnostic.range.start.line,
            diagnostic.range.start.character + 'return'.length,
            diagnostic.range.end.line,
            diagnostic.range.end.character
        )
    };
}

1	import type { Diagnostic } from 'vscode-languageserver';
2	import type { XmlFile } from '../../files/XmlFile';
3	import { util } from '../../util';	1✔
4
5	/**
6	* Returns the position at which an `extends` attribute should be inserted for
7	* a component that is missing one: after the last existing attribute, or after
8	* the `<component` tag itself if there are no attributes yet.
9	*/
10	export function getMissingExtendsInsertPosition(file: XmlFile) {	1✔
11	const { component } = file.parser.ast;	2✔
12	return (component.attributes[component.attributes.length - 1] ?? component.tag).range.end;	2!
13	}
14
15	/**
16	* Returns the delete change that removes the return value from a `return <expr>`
17	* statement flagged by `voidFunctionMayNotReturnValue`. Leaves the bare `return`
18	* keyword in place.
19	*/
20	export function getRemoveReturnValueChange(diagnostic: Diagnostic, filePath: string) {	1✔
NEW 21	return {	×
22	type: 'delete' as const,
23	filePath: filePath,
24	range: util.createRange(
25	diagnostic.range.start.line,
26	diagnostic.range.start.character + 'return'.length,
27	diagnostic.range.end.line,
28	diagnostic.range.end.character
29	)
30	};
31	}

rokucommunity / brighterscript / #15701

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous