Skip to content

Chore: Add prefix arg to implement TODO to insert code after comments#3

Merged
tninja merged 1 commit intomainfrom
kang_feat
Sep 27, 2025
Merged

Chore: Add prefix arg to implement TODO to insert code after comments#3
tninja merged 1 commit intomainfrom
kang_feat

Conversation

@tninja
Copy link
Copy Markdown
Owner

@tninja tninja commented Sep 27, 2025

No description provided.

@tninja tninja requested a review from Copilot September 27, 2025 01:43
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a prefix argument feature to the ai-code-implement-todo function, allowing users to choose between replacing TODO comments in-place (default) or implementing code after the comments while keeping them (with C-u prefix).

  • Added prefix argument support to ai-code-implement-todo function
  • Updated the UI text to indicate the new C-u behavior
  • Updated version number and changelog

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
ai-code-interface.el Updated menu text and version number
ai-code-change.el Added prefix argument logic for TODO implementation behavior
HISTORY.org Added changelog entry for the new feature

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread ai-code-change.el
(format "Please implement code after this requirement comment on line %d: '%s'. Leave the comment as-is and add the implementation code after it. Keep the existing code structure and add the implementation after this specific comment.%s%s"
current-line-number current-line function-context files-context-string))
(function-name
(format "Please implement code after all TODO comments in function '%s'. The TODO are TODO comments. Leave the comments as-is and add implementation code after each comment. Keep the existing code structure and only add code after these marked items.%s"
Copy link

Copilot AI Sep 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The sentence 'The TODO are TODO comments' is grammatically incorrect. It should be 'The TODOs are TODO comments' or 'These are TODO comments'.

Copilot uses AI. Check for mistakes.
Comment thread ai-code-change.el
(format "Please implement code after all TODO comments in function '%s'. The TODO are TODO comments. Leave the comments as-is and add implementation code after each comment. Keep the existing code structure and only add code after these marked items.%s"
function-name files-context-string))
(t
(format "Please implement code after all TODO comments in file '%s'. The TODO are TODO comments. Leave the comments as-is and add implementation code after each comment. Keep the existing code structure and only add code after these marked items.%s"
Copy link

Copilot AI Sep 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The sentence 'The TODO are TODO comments' is grammatically incorrect. It should be 'The TODOs are TODO comments' or 'These are TODO comments'.

Copilot uses AI. Check for mistakes.
@tninja tninja changed the title Chore Add prefix arg to implement TODO to insert code after comments Chore: Add prefix arg to implement TODO to insert code after comments Sep 27, 2025
@tninja tninja merged commit 0f08820 into main Sep 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants