Fix confusing code in DeleteAll()

[konard]

Summary

This PR fixes issue #354 by simplifying the confusing implementation of the DeleteAll() method in ILinksExtensions.cs.

Problem

The original implementation had confusing logic:

for (var i = links.Count(); comparer.Compare(i, default) > 0; i = --i)
{
    links.Delete(i);
    if (links.Count() != --i)
    {
        i = links.Count();
    }
}

Issues with the old code:

• Used redundant i = --i in the for loop increment
• Had a confusing condition if (links.Count() != --i) that decremented i again
• The logic was hard to understand and maintain

Solution

Simplified to a much clearer implementation:

for (var i = links.Count(); comparer.Compare(i, default) > 0; i = links.Count())
{
    links.Delete(i);
}

The new logic is straightforward:

1. Start from the last link (count)
2. Delete it
3. Reset i to the new count (which is now one less)
4. Continue until no links remain

Testing

• All existing tests pass (20 passed, 1 skipped)
• Local build completed successfully
• CI checks passed

Related Issue

Fixes #354

---

🤖 Generated with Claude Code

[konard]

🤖 Solution Draft Log

This log file contains the complete execution trace of the AI solution draft process.

📎 Log file uploaded as GitHub Gist (258KB)
🔗 View complete solution draft log

---

Now working session is ended, feel free to review and add any feedback on the solution draft.