Typo "hunk" -> "chunk" in System prompts for unified diff edit #1964
Conversation
…s (hunk changed to chunk) in the section: # apply_diff Tool - Generate Precise Code Changes in point 3, changed from 3. Group related changes: - Keep related modifications in the same hunk to 3. Group related changes: - Keep related modifications in the same chunk
…iff edit "hunk" is used in this whole file for "chunk". But for function names and variables it is ok. When the hard coded system prompt to be sent for LLMs contains "hunk" for "chunk". Then better to correct this typo atleast in system prompt.
…error messages. Retained hunk in function, variables names. Retained typo "hunk" in variables and functions. But the System prompt for unified diff edit, that goes to LLMs and the error messages, changed the typo.
Changed hunk to chunk in the System prompt for unified diff edit , as this will be sent to LLMs. Rest in function calls and variables, did not change. as that's not affecting performance.
|
dosubot
bot
added
the
size:S
Bhartendu-Kumar
changed the title
|
within this whole file "hunk" is used. But for codes like in function and variables, it is non-performance related. but in the hardcoded system prompt, "hunk" for "chunk" makes a difference. |
[Fix] Cline crashes when reading files with large outputs
* Revert "Merge pull request RooCodeInc#1964 from cline/ocasta181/ENG-209" This reverts commit ca17a06, reversing changes made to 6b9c2a1. * Set hard 300kb limit on file size cline reads into context * Add changeset
|
I think hunk might actually be the correct technical term? https://www.gnu.org/software/diffutils/manual/html_node/Hunks.html |
|
Going to close this to keep the PRs clean, but let me know if you want to discuss more! |
github-project-automation
bot
moved this from Issue [In Progress]
to Done
in Roo Code Roadmap
|
My bad. Sure hunk is the correct thing. Thanks |
2 participants
Context
Typo "hunk" in messages to be sent to LLMs as system prompts and error messages and logs.
Implementation
"Changed hunk to chunk in the System prompt for unified diff edit , as this will be sent to LLMs. and also changed in error messages. Rest in function calls and variables, did not change. as that's not affecting performance.
Screenshots
current system prompt:
new system prompt:
notice last line. there are other such lines.
How to Test
Just check the "System Prompt" button for any Mode in Settings.
Get in Touch
bhartendu3992
Important
Fix typo "hunk" to "chunk" in system prompts and error messages for unified diff edits.
index.tsandunified.ts.index.tsandunified.tsto reflect the change from "hunk" to "chunk".This description was created by
for 73b573c. It will automatically update as commits are pushed.