Skip to content

Fix: Make org TODO headline section work well with ai-code-implement-todo + helm#324

Merged
tninja merged 2 commits intomainfrom
kang_feat_11
Apr 29, 2026
Merged

Fix: Make org TODO headline section work well with ai-code-implement-todo + helm#324
tninja merged 2 commits intomainfrom
kang_feat_11

Conversation

@tninja
Copy link
Copy Markdown
Owner

@tninja tninja commented Apr 29, 2026

This feature: #317, doesn't work well with helm.

The leading * character mess up with helm search. This fix remove * so that it work with helm

@tninja
Copy link
Copy Markdown
Owner Author

tninja commented Apr 29, 2026

@copilot some test failed, check if it is because of this change. If it is, fix those tests

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 29, 2026

@copilot some test failed, check if it is because of this change. If it is, fix those tests

Yes, the failures were caused by my change. The two failing tests (ai-code-test-ai-code-implement-todo-org-section-includes-heading-and-content and ai-code-test-ai-code-implement-todo-org-headline-with-colon-prefix) were asserting that the captured prompt contained the heading with leading * characters (e.g. \\* TODO Build backend switcher), but my change now strips those stars before inserting into the prompt. Fixed in f36d5d4 by updating the assertions to match the new stripped behavior (e.g. TODO Build backend switcher).

@tninja tninja merged commit 7a23d39 into main Apr 29, 2026
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