amr/passepartout
1
0
Fork 0
Code Issues 15 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix(skills): use uiop:getenv for Skill Engine and standardize tangle header

Browse Source
This commit is contained in:
Amr Gharbeia
2026-04-28 17:59:25 -04:00
parent 6f1e606cfa
commit d42b5fc50c
1 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
harness/skills.org
View File

@@ -1,4 +1,4 @@
#+PROPERTY: header-args:lisp :tangle (concat (getenv "INSTALL_DIR") "/harness/skills.lisp" (expand-file-name "")) #+PROPERTY: header-args:lisp :tangle (expand-file-name "skills.lisp")
#+TITLE: The Skill Engine (skills.lisp) #+TITLE: The Skill Engine (skills.lisp)
#+AUTHOR: Amr #+AUTHOR: Amr
#+FILETAGS: :harness:skills: #+FILETAGS: :harness:skills:
@@ -347,7 +347,7 @@ Only loads blocks that specify a .lisp tangle target, ignoring tests and example
(defun initialize-all-skills () (defun initialize-all-skills ()
"Scans the directory defined by SKILLS_DIR and hot-loads skills using topological order." "Scans the directory defined by SKILLS_DIR and hot-loads skills using topological order."
(let* ((env-path (getenv "SKILLS_DIR")) (let* ((env-path (uiop:getenv "SKILLS_DIR"))
(skills-dir-str (or env-path (namestring (merge-pathnames "notes/" (user-homedir-pathname))))) (skills-dir-str (or env-path (namestring (merge-pathnames "notes/" (user-homedir-pathname)))))
(resolved-path (context-resolve-path skills-dir-str)) (resolved-path (context-resolve-path skills-dir-str))
(skills-dir (if resolved-path (uiop:ensure-directory-pathname resolved-path) nil))) (skills-dir (if resolved-path (uiop:ensure-directory-pathname resolved-path) nil)))
@@ -419,7 +419,7 @@ EXAMPLES:
*** The Eval Tool (Internal Inspection) *** The Eval Tool (Internal Inspection)
#+begin_src lisp #+begin_src lisp
** Cognitive Tool Registration ** Cognitive Tool Registration
#+begin_src lisp :tangle (concat (getenv "INSTALL_DIR") "/harness/skills.lisp" (expand-file-name "")) #+begin_src lisp :tangle (expand-file-name "skills.lisp" (expand-file-name ""))
;; Cognitive tools are registered in *cognitive-tools* (defined in package.lisp) ;; Cognitive tools are registered in *cognitive-tools* (defined in package.lisp)
;; using the def-cognitive-tool macro. ;; using the def-cognitive-tool macro.
#+end_src #+end_src
@@ -475,17 +475,17 @@ EXAMPLES:
(declare (ignore context)) (declare (ignore context))
(let ((skill (getf args :skill))) (let ((skill (getf args :skill)))
(or (uiop:file-exists-p skill) (or (uiop:file-exists-p skill)
(let ((skills-dir (or (ignore-errors (getenv "SKILLS_DIR")) (let ((skills-dir (or (ignore-errors (uiop:getenv "SKILLS_DIR"))
(namestring (merge-pathnames "notes/" (user-homedir-pathname)))))) (namestring (merge-pathnames "notes/" (user-homedir-pathname))))))
(uiop:file-exists-p (merge-pathnames (format nil "~a.org" skill) skills-dir)))))) (uiop:file-exists-p (merge-pathnames (format nil "~a.org" skill) skills-dir))))))
:body (lambda (args) :body (lambda (args)
(let ((skill (getf args :skill))) (let ((skill (getf args :skill)))
(snapshot-memory) (snapshot-memory)
(let ((skills-dir (or (ignore-errors (getenv "SKILLS_DIR")) (let ((skills-dir (or (ignore-errors (uiop:getenv "SKILLS_DIR"))
(namestring (merge-pathnames "notes/" (user-homedir-pathname))))) (namestring (merge-pathnames "notes/" (user-homedir-pathname)))))
(resolved-path (context-resolve-path skills-dir)) (resolved-path (context-resolve-path skills-dir))
(skills-dir-actual (if (ignore-errors (getenv "SKILLS_DIR")) (skills-dir-actual (if (ignore-errors (uiop:getenv "SKILLS_DIR"))
(uiop:ensure-directory-pathname (context-resolve-path (getenv "SKILLS_DIR"))) (uiop:ensure-directory-pathname (context-resolve-path (uiop:getenv "SKILLS_DIR")))
(uiop:ensure-directory-pathname (user-homedir-pathname))))) (uiop:ensure-directory-pathname (user-homedir-pathname)))))
(let ((file (if (uiop:file-exists-p skill) (let ((file (if (uiop:file-exists-p skill)
(uiop:ensure-pathname skill) (uiop:ensure-pathname skill)