feat(ai-term): dock the agent window by frame aspect ratio

The agent window now docks from whichever edge conserves more space, chosen at display time: right on a landscape frame, bottom on a square or portrait one, replacing the host (laptop/desktop) branch. cj/--ai-term-direction-for-aspect is the pure decision; default-size pairs the width or height fraction with it.

1 files changed, 30 insertions, 25 deletions

diff --git a/tests/test-ai-term--default-geometry.el b/tests/test-ai-term--default-geometry.el
index 833f2ef4c..91013862d 100644
--- a/tests/test-ai-term--default-geometry.el
+++ b/tests/test-ai-term--default-geometry.el
@@ -1,15 +1,18 @@
 ;;; test-ai-term--default-geometry.el --- Tests for host-aware display defaults -*- lexical-binding: t; -*-
 
 ;;; Commentary:
-;; ai-term's default display geometry is host-aware: a laptop opens the
-;; agent from the bottom (75% height), a desktop opens it from the right
-;; (50% width).  `cj/--ai-term-default-direction' and
-;; `cj/--ai-term-default-size' encapsulate the `env-laptop-p' branch;
-;; they feed the default fallbacks in `cj/--ai-term-capture-state' and
+;; ai-term's default display geometry is chosen from the frame's pixel aspect
+;; ratio: a landscape frame docks the agent from the right (a width fraction), a
+;; square or portrait frame docks it from the bottom (a height fraction).
+;; `cj/--ai-term-direction-for-aspect' is the pure decision;
+;; `cj/--ai-term-default-direction' reads the frame and delegates to it;
+;; `cj/--ai-term-default-size' pairs the size fraction with that direction.
+;; They feed the default fallbacks in `cj/--ai-term-capture-state' and
 ;; `cj/--ai-term-display-saved'.
 ;;
-;; `env-laptop-p' is stubbed per-test so the assertions are deterministic
-;; regardless of the host the suite runs on.
+;; The direction is tested on the pure helper (no frame mocking, which would
+;; trip the native-comp trampoline trap on the frame-pixel-* subrs); the size
+;; helper is tested by stubbing the direction defun.
 
 ;;; Code:
 
@@ -19,37 +22,39 @@
 (add-to-list 'load-path (expand-file-name "modules" user-emacs-directory))
 (require 'ai-term)
 
-(ert-deftest test-ai-term--default-direction-laptop ()
-  "Normal: on a laptop the default direction is `below'."
-  (cl-letf (((symbol-function 'env-laptop-p) (lambda () t)))
-    (should (eq (cj/--ai-term-default-direction) 'below))))
+(ert-deftest test-ai-term--direction-for-aspect-landscape-is-right ()
+  "Normal: a wider-than-tall frame docks from the right."
+  (should (eq (cj/--ai-term-direction-for-aspect 1920 1080) 'right)))
 
-(ert-deftest test-ai-term--default-direction-desktop ()
-  "Normal: on a desktop the default direction is `right'."
-  (cl-letf (((symbol-function 'env-laptop-p) (lambda () nil)))
-    (should (eq (cj/--ai-term-default-direction) 'right))))
+(ert-deftest test-ai-term--direction-for-aspect-portrait-is-below ()
+  "Normal: a taller-than-wide frame docks from the bottom."
+  (should (eq (cj/--ai-term-direction-for-aspect 1080 1920) 'below)))
 
-(ert-deftest test-ai-term--default-size-laptop ()
-  "Normal: on a laptop the default size is `cj/ai-term-laptop-height'."
+(ert-deftest test-ai-term--direction-for-aspect-square-is-below ()
+  "Boundary: a square frame docks from the bottom (the conserving tie-break)."
+  (should (eq (cj/--ai-term-direction-for-aspect 1000 1000) 'below)))
+
+(ert-deftest test-ai-term--default-size-pairs-width-with-right ()
+  "Normal: when the direction is `right' the size is the width fraction."
   (let ((cj/ai-term-laptop-height 0.75)
         (cj/ai-term-desktop-width 0.5))
-    (cl-letf (((symbol-function 'env-laptop-p) (lambda () t)))
-      (should (= (cj/--ai-term-default-size) 0.75)))))
+    (cl-letf (((symbol-function 'cj/--ai-term-default-direction) (lambda (&rest _) 'right)))
+      (should (= (cj/--ai-term-default-size) 0.5)))))
 
-(ert-deftest test-ai-term--default-size-desktop ()
-  "Normal: on a desktop the default size is `cj/ai-term-desktop-width'."
+(ert-deftest test-ai-term--default-size-pairs-height-with-below ()
+  "Normal: when the direction is `below' the size is the height fraction."
   (let ((cj/ai-term-laptop-height 0.75)
         (cj/ai-term-desktop-width 0.5))
-    (cl-letf (((symbol-function 'env-laptop-p) (lambda () nil)))
-      (should (= (cj/--ai-term-default-size) 0.5)))))
+    (cl-letf (((symbol-function 'cj/--ai-term-default-direction) (lambda (&rest _) 'below)))
+      (should (= (cj/--ai-term-default-size) 0.75)))))
 
 (ert-deftest test-ai-term--default-size-respects-custom-values ()
   "Boundary: the helper returns the customized values, not the literals."
   (let ((cj/ai-term-laptop-height 0.6)
         (cj/ai-term-desktop-width 0.33))
-    (cl-letf (((symbol-function 'env-laptop-p) (lambda () t)))
+    (cl-letf (((symbol-function 'cj/--ai-term-default-direction) (lambda (&rest _) 'below)))
       (should (= (cj/--ai-term-default-size) 0.6)))
-    (cl-letf (((symbol-function 'env-laptop-p) (lambda () nil)))
+    (cl-letf (((symbol-function 'cj/--ai-term-default-direction) (lambda (&rest _) 'right)))
       (should (= (cj/--ai-term-default-size) 0.33)))))
 
 (provide 'test-ai-term--default-geometry)