Skip to content

Update inline-prompt.ts#2

Open
faizancodes wants to merge 1 commit into
mainfrom
faizancodes-patch-2
Open

Update inline-prompt.ts#2
faizancodes wants to merge 1 commit into
mainfrom
faizancodes-patch-2

Conversation

@faizancodes

Copy link
Copy Markdown
Member

No description provided.

@headstarterbot

Copy link
Copy Markdown

src/prompts/inline-prompt.ts

Avoid direct string manipulation and instead use a template engine or a safer string replacement method to prevent potential security vulnerabilities.
Create Issue

const userMessage = new Template(INLINE_USER_MESSAGE_TEMPLATE).render({
  SUGGESTION: suggestion.toString(),
  FILE: assignFullLineNumers(fileContents)
});

Use object literals to define properties, improving code readability and reducing the nesting level.
Create Issue

export const INLINE_FIX_FUNCTION = {
  name: "fix",
  description: "The code fix to address the suggestion and rectify the issue",
  parameters: {
    comment: { description: "Why this change improves the code" },
    code: { description: "Modified Code Snippet" },
    lineStart: { description: "Starting Line Number" },
    lineEnd: { description: "Ending Line Number" }
  },
  required: ["action"]
};

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant