Skip to content

[code_review] Experiment more with further context lines #4389

Open
@marco-c

Description

@marco-c

We're currently saying (

PROMPT_TEMPLATE_FURTHER_CONTEXT_LINES = """Based on the patch provided below and its related summarization, report the code lines more context is required.
):
Based on the patch provided below and its related summarization, report the code lines more context is required. For that, list the lines with the their associated line numbers, grouping each one on a separated line.
which might not be super clear to the LLM (what does "more context" mean?).

We can try two things here:

  • Ask more explicitly how many more lines the LLM wants to see before/after the current shown lines;
  • Ask the LLM if it wants to see the entire function sorrounding the current shown lines.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions