Bug: insert_content adds extra blank line with sequential operations

[KJ7LNW]

Which version of the app are you using?

v3.17.1

Which API Provider are you using?

Claude

Which Model are you using?

sonnet-3.5

What happened?

When performing sequential insert_content operations on a file - first inserting at the beginning (line 1) and then appending at the end (line 0) - an extra blank line is added between the two inserted contents.

Steps to reproduce

1. Create a new file (or use an existing empty file) because insert_content cannot create files (Bug: insert_content fails to create missing files #3652)
2. Perform the following sequential insert_content operations:

First operation (inserting at the beginning):

<insert_content>
<path>test_insert.txt</path>
<line>1</line>
<content>Line 1: This is the first line inserted at the beginning.
Line 2: This is the second line inserted at the beginning.</content>
</insert_content>

Second operation (appending at the end):

<insert_content>
<path>test_insert.txt</path>
<line>0</line>
<content>Line 3: This content is appended at the end of the file.
Line 4: This is the last line appended at the end.</content>
</insert_content>

Expected result

The file should contain the content from both operations without any extra blank lines:

Line 1: This is the first line inserted at the beginning.
Line 2: This is the second line inserted at the beginning.
Line 3: This content is appended at the end of the file.
Line 4: This is the last line appended at the end.

Actual behavior

The file contains an extra blank line between the two inserted contents:

Line 1: This is the first line inserted at the beginning.
Line 2: This is the second line inserted at the beginning.

Line 3: This content is appended at the end of the file.
Line 4: This is the last line appended at the end.

Additional context

This extra blank line can cause issues when generating code or other content where precise formatting is important. The insert_content tool should maintain consistent behavior without adding unexpected blank lines.

A fix for this issue should include tests to ensure proper handling of carriage returns during insert operations, specifically:

• Multiple leading carriage returns
• Multiple trailing carriage returns

These tests would ensure that blank lines are preserved exactly as specified in the content without adding or removing any.

[dosubot]

The issue you're experiencing with the insert_content function adding an extra blank line between sequential operations is not documented in the available resources. It seems like this behavior might be unintended, especially since the expected result is to have the content inserted without any additional blank lines.

To address this, you might consider checking the implementation of the insert_content function to see if there are any conditions or flags affecting newline handling during sequential operations. Additionally, implementing tests for handling carriage returns, as you suggested, could help ensure that the function behaves correctly with multiple leading or trailing carriage returns.

Since this issue is not covered in the documentation, it would be beneficial to report it as a bug if it hasn't been done already, and possibly explore the codebase for any logic that might be causing this behavior.

To continue talking to Dosu, mention @dosu.

---

^{Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Bug Report | Other}

[Ruakij]

I'll be working on this.