Chore: Allow non-git files in repo context candidates (#313)

* Allow non-git files in repo context candidates

* fix review feedback for add-context repo root selection

Agent-Logs-Url: https://github.com/tninja/ai-code-interface.el/sessions/b13eebd1-acc6-49e4-b542-e671bb7b0563

Co-authored-by: tninja <714625+tninja@users.noreply.github.com>

* style: align indentation in add-context updates and test

Agent-Logs-Url: https://github.com/tninja/ai-code-interface.el/sessions/b13eebd1-acc6-49e4-b542-e671bb7b0563

Co-authored-by: tninja <714625+tninja@users.noreply.github.com>

* docs: update HISTORY for non-git context candidates

Agent-Logs-Url: https://github.com/tninja/ai-code-interface.el/sessions/c4c90fbb-a847-42a1-85b1-c5b889ac3ed8

Co-authored-by: tninja <714625+tninja@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: tninja <714625+tninja@users.noreply.github.com>

Author: tninja

HISTORY.org

@@ -6,6 +6,7 @@
 - Fix: Enable ghostel full-redraw mode for Claude Code and Copilot CLI sessions
 - Refactor: Use Ghostel public API for sending input
 - Chore: Notify on response and beep before D-Bus
+- Chore: Allow adding non-git files and directories to existing repo context candidates
 
 ** 1.72
 - Chore: Add action intent prompt to TODO implementation flow

ai-code-file.el

@@ -473,7 +473,9 @@ Otherwise, ask AI to generate a build command."
 
 ;;;###autoload
 (defun ai-code-add-context ()
-  "Capture current buffer context and store it per Git repository.
+  "Capture current buffer context and store it per selected repository root.
+If current buffer is not inside a Git repository, select from known repository
+roots gathered from existing context entries and visible/session buffers.
 When no region is selected, use the full file path and current function
 \(if any).  When a region is active, use the file path with line range
 in the form filepath#Lstart-Lend."
@@ -495,9 +497,18 @@ in the form filepath#Lstart-Lend."
                                 (when (and root (not (member root roots)))
                                   (push root roots)))))))
                       (nreverse roots)))
-         (ordered-roots (if (and current-root (member current-root all-roots))
-                            (cons current-root (remove current-root all-roots))
-                          all-roots))
+         (existing-roots (let ((roots '()))
+                           (maphash (lambda (root _contexts)
+                                      (when (and (stringp root)
+                                                 (not (member root roots)))
+                                        (push root roots)))
+                                    ai-code--repo-context-info)
+                           (sort roots #'string<)))
+         (candidate-roots (sort (delete-dups (append existing-roots all-roots))
+                                #'string<))
+         (ordered-roots (if (and current-root (member current-root candidate-roots))
+                            (cons current-root (remove current-root candidate-roots))
+                          candidate-roots))
          (repo-root (cond
                      ((null ordered-roots)
                       (user-error "Not inside a Git repository"))

test/test_ai-code-file.el

@@ -675,6 +675,38 @@ everything is cleaned up afterward."
       (ai-code-context-action '(4))
       (should completing-read-called))))
 
+(ert-deftest ai-code-test-add-context-allows-non-git-file-for-existing-repo-context ()
+  "Test `ai-code-add-context' can add a non-git file to an existing repo context."
+  (let ((ai-code--repo-context-info (make-hash-table :test #'equal))
+        (repo-root "/tmp/existing-repo/")
+        (new-file (make-temp-file "ai-code-context-"))
+        (completing-read-called nil))
+    (unwind-protect
+        (with-temp-buffer
+          (setq buffer-file-name new-file)
+          (puthash repo-root
+                   '("/tmp/existing-repo/lib/existing-context.el")
+                   ai-code--repo-context-info)
+          (cl-letf (((symbol-function 'ai-code--git-root)
+                     (lambda (&optional _dir) nil))
+                    ((symbol-function 'walk-windows)
+                     (lambda (&rest _args) nil))
+                    ((symbol-function 'completing-read)
+                     (lambda (&rest _args)
+                       (setq completing-read-called t)
+                       repo-root))
+                    ((symbol-function 'derived-mode-p)
+                     (lambda (&rest _args) nil))
+                    ((symbol-function 'message)
+                     (lambda (&rest _args) nil)))
+            (ai-code-add-context)
+            (should-not completing-read-called)
+            (should (equal (gethash repo-root ai-code--repo-context-info)
+                           (list new-file
+                                 "/tmp/existing-repo/lib/existing-context.el")))))
+      (when (file-exists-p new-file)
+        (delete-file new-file)))))
+
 (ert-deftest ai-code-test-build-or-test-project-dispatches-test-project ()
   "Test selecting \"Test project\" dispatches to `ai-code-test-project'."
   (let ((test-project-called nil)