Skip to content

Avoid else after return #6

@rakyll

Description

@rakyll

Description of the bug:

For example:

 if (editData.isNewFile) {
                return {
                    llmContent: `Created new file: ${params.file_path} with provided content.`,
                    returnDisplay: `Created ${shortenPath(makeRelative(params.file_path, this.rootDirectory))}`
                };
            } else {
                const fileName = path.basename(params.file_path);
                const fileDiff = Diff.createPatch(
                    fileName,
                    editData.currentContent ?? '',
                    editData.newContent,
                    'Current',
                    'Proposed',
                    { context: 3, ignoreWhitespace: true }
                );

else is redundant. Remove all instances of this pattern.

Actual vs expected behavior:

No response

Any other information you'd like to share?

No response

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions