fix: resolve upgrade CI failure from dotagents sync target collision

Merged
SH
7mo ago
+3-2

Summary

  • Set DOTAGENTS_SKIP_SYNC := 1 in parent Makefile to prevent dotagents sync recipe from leaking when included via -include dotagents/Makefile
  • Updated dotagents submodule with fix for missing / separator in ruler-apply-global (dotagents.ruler -> dotagents/.ruler)

Root cause: The parent Makefile includes dotagents/Makefile, which caused GNU Make to merge the sync target prerequisites and recipe. The dotagents sync recipe (calling ruler-apply-global, commands-sync, etc.) would run at the parent level where paths didn't resolve correctly. Additionally, $(abspath ...) strips trailing slashes, so .ruler was concatenated directly to the directory name.

Closes #1372

Summary by cubic

Prevents the sync target from dotagents running in the parent Makefile, which caused path errors and broke upgrade CI (fixes #1372). Also updates dotagents to fix a missing / in ruler-apply-global.

  • Bug Fixes
    • Parent Makefile: set DOTAGENTS_SKIP_SYNC := 1 to stop dotagents/Makefile from merging and running its sync recipe in the parent.
    • Updated dotagents submodule to use dotagents/.ruler (instead of dotagents.ruler) in ruler-apply-global.