Aider robustness 2

[nang-dev]

Description ✏️

Closes #xxx

What changed? Feel free to be brief.

• Bullet points are helpful.
• Screenshots are helpful (if applicable).

Checklist ✅

• I have added screenshots (if UI changes are present).
• I have done a self-review of my code.
• I have manually tested my code (if applicable).

---

Important

Enhance Aider integration robustness by refining process state management and updating GUI to handle new states.

• Behavior:
  • Refactor Aider model handling to use a single instance in core.ts and aiderUtil.ts.
  • Update AiderProcessManager in aiderProcess.ts to handle "restarting" state and block state change from "restarting" to "crashed".
  • Modify AiderGUI in aidergui.tsx to handle new "restarting" state and update UI messages accordingly.
• State Management:
  • Add "restarting" state to AiderState in aiderTypes.ts.
  • Update state transitions in AiderProcessManager to handle process errors and restarts.
• Misc:
  • Bump version in package.json from 1.5.2 to 1.5.3.
  • Remove redundant code and comments in AiderLLM.ts and aiderProcess.ts.

^{This description was created by for 3b0cbc9. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 3b0cbc9 in 34 seconds

More details

• Looked at 217 lines of code in 7 files
• Skipped 1 files when reviewing.
• Skipped posting 3 drafted comments based on config settings.

1. core/llm/llms/AiderLLM.ts:198

• Draft comment:
  Consider using this.getAiderProcess() instead of directly accessing Aider.aiderProcess to ensure the process is initialized.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The code uses Aider.aiderProcess directly in multiple places, which can lead to issues if the process is not initialized. It's better to use the getAiderProcess() method to ensure the process is properly initialized before use.

2. core/llm/llms/AiderLLM.ts:215

• Draft comment:
  Consider using this.getAiderProcess() instead of directly accessing Aider.aiderProcess to ensure the process is initialized.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The code uses Aider.aiderProcess directly in multiple places, which can lead to issues if the process is not initialized. It's better to use the getAiderProcess() method to ensure the process is properly initialized before use.

3. extensions/vscode/src/integrations/aider/aiderUtil.ts:121

• Draft comment:
  Consider using model.getAiderProcess() instead of directly accessing Aider.aiderProcess to ensure the process is initialized.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The code uses Aider.aiderProcess directly in multiple places, which can lead to issues if the process is not initialized. It's better to use the getAiderProcess() method to ensure the process is properly initialized before use.

Workflow ID: wflow_qs0Al5H25waLMZpd

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.