@atlashub/smartstack-cli 3.38.0 → 3.39.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlashub/smartstack-cli",
-  "version": "3.38.0",
+  "version": "3.39.0",
   "description": "SmartStack Claude Code automation toolkit - GitFlow, EF Core migrations, prompts and more",
   "author": {
     "name": "SmartStack",

package/templates/agents/efcore/scan.md

@@ -45,7 +45,9 @@ FOR each branch in active_worktrees:
 ## Fallback Commands (if MCP unavailable)
 
 ```bash
-# List worktrees
+# List worktrees (run preflight_git_check first for cross-platform compatibility)
+detect_platform
+repair_worktree_paths "$(git rev-parse --git-common-dir 2>/dev/null || echo '.bare')"
 git worktree list
 
 # ModelSnapshot hash

package/templates/agents/gitflow/commit.md

@@ -14,6 +14,80 @@ Commit with EF Core migration validation. Executes directly, displays only the r
 
 ## EXECUTION SEQUENCE
 
+### 0. Pre-flight: Worktree Accessibility
+
+```bash
+# Quick check: can git resolve the repo?
+git rev-parse --git-dir >/dev/null 2>&1
+if [ $? -ne 0 ]; then
+  # Worktree .git file likely has cross-platform absolute path
+  # Find .bare directory by walking up
+  BARE_DIR=""
+  DIR="$(pwd)"
+  while [ "$DIR" != "/" ]; do
+    [ -d "$DIR/.bare" ] && { BARE_DIR="$DIR/.bare"; break; }
+    DIR=$(dirname "$DIR")
+  done
+
+  if [ -n "$BARE_DIR" ] && [ -f .git ]; then
+    echo "Repairing worktree paths (cross-platform fix)..."
+
+    # Detect platform
+    GF_PLATFORM="linux"
+    [ -f /proc/version ] && grep -qi "microsoft\|wsl" /proc/version 2>/dev/null && GF_PLATFORM="wsl"
+
+    # Repair all worktrees: convert absolute paths to relative paths
+    WORKTREES_DIR="$BARE_DIR/worktrees"
+    for wt_dir in "$WORKTREES_DIR"/*/; do
+      [ ! -d "$wt_dir" ] && continue
+      wt_name=$(basename "$wt_dir")
+      gitdir_file="$wt_dir/gitdir"
+      [ ! -f "$gitdir_file" ] && continue
+
+      stored_path=$(cat "$gitdir_file" | tr -d '\n\r')
+
+      # Determine if path is absolute and resolve to current platform
+      resolved_path="$stored_path"
+      is_abs=false
+      if [[ "$stored_path" =~ ^[A-Za-z]:[\\/] ]]; then
+        is_abs=true
+        if [ "$GF_PLATFORM" = "wsl" ]; then
+          drive=$(echo "${stored_path:0:1}" | tr '[:upper:]' '[:lower:]')
+          rest="${stored_path:2}"; rest="${rest//\\//}"
+          resolved_path="/mnt/$drive$rest"
+        fi
+      elif [[ "$stored_path" =~ ^/ ]]; then
+        is_abs=true
+      fi
+      [ "$is_abs" = "false" ] && continue  # Already relative, skip
+
+      wt_real_dir=$(dirname "$resolved_path")
+      [ ! -d "$wt_real_dir" ] && continue
+
+      bare_wt_abs=$(realpath -m "$wt_dir")
+      wt_abs=$(realpath -m "$wt_real_dir")
+
+      # Fix gitdir backlink to relative
+      rel_gitdir=$(realpath -m --relative-to="$bare_wt_abs" "$wt_abs/.git")
+      [ "$stored_path" != "$rel_gitdir" ] && printf '%s\n' "$rel_gitdir" > "$gitdir_file"
+
+      # Fix worktree .git pseudo-file to relative
+      if [ -f "$wt_real_dir/.git" ]; then
+        git_content=$(cat "$wt_real_dir/.git" | tr -d '\n\r')
+        git_path="${git_content#gitdir: }"
+        rel_bare=$(realpath -m --relative-to="$wt_abs" "$bare_wt_abs")
+        [ "$git_path" != "$rel_bare" ] && printf 'gitdir: %s\n' "$rel_bare" > "$wt_real_dir/.git"
+      fi
+    done
+
+    # Verify repair
+    git rev-parse --git-dir >/dev/null 2>&1 && echo "Worktree paths repaired successfully" || { echo "BLOCKED: worktree repair failed"; exit 1; }
+  else
+    echo "BLOCKED: not a git repository and no .bare found"; exit 1
+  fi
+fi
+```
+
 ### 1. Guards
 
 ```bash

package/templates/agents/gitflow/finish.md

@@ -31,8 +31,11 @@ fi
 BRANCH=${1:-$(git rev-parse --abbrev-ref HEAD)}
 BRANCH_TYPE=$(echo $BRANCH | cut -d'/' -f1)
 
-# Worktree detection: if in a worktree, navigate to main repo
-# Normalize path for cross-platform compatibility (WSL ↔ Windows)
+# Worktree detection: repair cross-platform paths first (WSL ↔ Windows)
+# Source: _shared.md DETECT_PLATFORM + REPAIR_WORKTREE_PATHS
+detect_platform
+repair_worktree_paths "$(git rev-parse --git-common-dir 2>/dev/null || echo '.bare')"
+
 WORKTREE_PATH=$(git worktree list --porcelain | grep -B2 "branch refs/heads/$BRANCH" | grep "^worktree " | sed 's/^worktree //')
 WORKTREE_PATH=$(normalize_path_for_platform "$WORKTREE_PATH")
 ```

package/templates/agents/gitflow/init-clone.md

@@ -96,7 +96,7 @@ if [ ! -d "$PROJECT_BASE/develop" ]; then
 fi
 ```
 
-### 5b. Fix Cross-Platform Worktree Paths
+### 5b. Convert Worktree Paths to Relative (Cross-Platform)
 
 ```bash
 PROJECT_BASE="{PROJECT_BASE}"
@@ -104,8 +104,8 @@ PROJECT_BASE="{PROJECT_BASE}"
 # Detect current platform (needed for path normalization)
 detect_platform
 
-# Repair any cross-platform paths written by git worktree add
-# (e.g., WSL paths /mnt/d/... when running on Windows, or vice versa)
+# Convert absolute paths written by git worktree add to relative paths
+# Relative paths work on both Windows and WSL without needing repair when switching
 repair_worktree_paths "$PROJECT_BASE/.bare"
 ```
 

package/templates/agents/gitflow/init-validate.md

@@ -136,7 +136,8 @@ done
 
 ### 2b. Worktree Path Integrity (Cross-Platform)
 
-Repair worktree metadata files that contain paths from a different platform (e.g., WSL paths `/mnt/d/...` on Windows or Windows paths `D:/...` on WSL).
+Convert worktree metadata paths to relative format for cross-platform compatibility.
+Relative paths work on both Windows (VS Code) and WSL without needing repair when switching.
 
 ```bash
 PROJECT_BASE="{PROJECT_BASE}"
@@ -144,7 +145,7 @@ PROJECT_BASE="{PROJECT_BASE}"
 # Source platform detection and path functions from _shared.md
 detect_platform
 
-# Repair cross-platform paths in .bare/worktrees/*/gitdir and worktree/.git files
+# Convert absolute paths to relative in .bare/worktrees/*/gitdir and worktree/.git files
 repair_worktree_paths "$PROJECT_BASE/.bare"
 ```
 

package/templates/agents/gitflow/merge.md

@@ -20,6 +20,11 @@ CONFIG_FILE=""
 if [ -f ".claude/gitflow/config.json" ]; then
   CONFIG_FILE=".claude/gitflow/config.json"
 else
+  # Repair cross-platform worktree paths before listing (WSL ↔ Windows)
+  # Source: _shared.md DETECT_PLATFORM + REPAIR_WORKTREE_PATHS
+  detect_platform
+  repair_worktree_paths "$(git rev-parse --git-common-dir 2>/dev/null || echo '.bare')"
+
   # Try develop worktree
   DEVELOP_PATH=$(git worktree list --porcelain 2>/dev/null | grep -A1 "branch refs/heads/develop" | grep "^worktree " | sed 's/^worktree //')
   if [ -n "$DEVELOP_PATH" ] && [ -f "$DEVELOP_PATH/.claude/gitflow/config.json" ]; then

package/templates/agents/gitflow/pr.md

@@ -22,6 +22,11 @@ CONFIG_FILE=""
 if [ -f ".claude/gitflow/config.json" ]; then
   CONFIG_FILE=".claude/gitflow/config.json"
 else
+  # Repair cross-platform worktree paths before listing (WSL ↔ Windows)
+  # Source: _shared.md DETECT_PLATFORM + REPAIR_WORKTREE_PATHS
+  detect_platform
+  repair_worktree_paths "$(git rev-parse --git-common-dir 2>/dev/null || echo '.bare')"
+
   # Try develop worktree
   DEVELOP_PATH=$(git worktree list --porcelain 2>/dev/null | grep -A1 "branch refs/heads/develop" | grep "^worktree " | sed 's/^worktree //')
   if [ -n "$DEVELOP_PATH" ] && [ -f "$DEVELOP_PATH/.claude/gitflow/config.json" ]; then

package/templates/agents/gitflow/start.md

@@ -66,7 +66,11 @@ esac
 mkdir -p "$(dirname "$WORKTREE_PATH")"
 git worktree add -b "$FULL_BRANCH" "$WORKTREE_PATH" "origin/$BASE_BRANCH"
 
-# 4. POST-CREATION VALIDATION (BLOCKING)
+# 4. POST-CREATION: Repair worktree paths FIRST, then validate
+# Uses repair_worktree_paths from _shared.md REPAIR_WORKTREE_PATHS section
+BARE_DIR=$(git rev-parse --git-common-dir 2>/dev/null)
+[ -n "$BARE_DIR" ] && repair_worktree_paths "$BARE_DIR"
+
 CREATED_PATH=$(git worktree list | grep "$FULL_BRANCH" | awk '{print $1}')
 BASENAME=$(basename "$CREATED_PATH")
 if [[ "$BASENAME" =~ ^[0-9]{2}- ]] && [[ "$BASENAME" != "01-Main" ]] && [[ "$BASENAME" != "02-Develop" ]]; then
@@ -76,6 +80,9 @@ if [[ "$BASENAME" =~ ^[0-9]{2}- ]] && [[ "$BASENAME" != "01-Main" ]] && [[ "$BAS
   git branch -D "$FULL_BRANCH" 2>/dev/null
   echo "Removed invalid worktree. Recreating at correct path..."
   git worktree add -b "$FULL_BRANCH" "$WORKTREE_PATH" "origin/$BASE_BRANCH"
+  # Repair again after recreation
+  BARE_DIR=$(git rev-parse --git-common-dir 2>/dev/null)
+  [ -n "$BARE_DIR" ] && repair_worktree_paths "$BARE_DIR"
 fi
 ```
 

package/templates/hooks/hooks.json

@@ -29,6 +29,17 @@
         ]
       }
     ],
+    "PostToolUse": [
+      {
+        "matcher": "Bash",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "${CLAUDE_HOOKS_DIR}/wsl-dotnet-cleanup.sh"
+          }
+        ]
+      }
+    ],
     "Stop": [
       {
         "hooks": [

package/templates/hooks/wsl-dotnet-cleanup.sh

@@ -0,0 +1,24 @@
+#!/bin/bash
+
+# WSL Dotnet Cleanup - PostToolUse Hook
+# Removes literal "bin\Debug" folders created by Roslyn BuildHost on WSL
+# Roslyn uses Windows path separators, creating "bin\Debug" as a single folder name
+# instead of the proper bin/Debug/ directory structure.
+
+set -euo pipefail
+
+HOOK_INPUT=$(cat)
+
+# Quick exit: only care about Bash tool calls containing "dotnet" (~1ms)
+case "$HOOK_INPUT" in
+  *dotnet*) ;; # proceed to cleanup
+  *) exit 0 ;; # fast exit
+esac
+
+# Find and remove corrupted bin\Debug folders (literal backslash in name)
+# bin?Debug matches "bin\Debug" where ? is the literal backslash character
+for d in src/*/bin?Debug bin?Debug; do
+  [ -d "$d" ] && rm -rf "$d"
+done
+
+exit 0

package/templates/skills/apex/references/core-seed-data.md

@@ -200,7 +200,6 @@ public static class NavigationApplicationSeedData
 public class NavigationApplicationSeedEntry
 {
     public Guid Id { get; init; }
-    public Guid ContextId { get; init; }
     public string Code { get; init; } = null!;
     public string Label { get; init; } = null!;
     public string? Description { get; init; }

package/templates/skills/apex/references/examine-build-validation.md

@@ -8,14 +8,9 @@
 ## Build Verification
 
 ```bash
-# Cleanup corrupted EF Core design-time artifacts (Roslyn BuildHost bug on Windows/WSL)
-for d in src/*/bin?Debug bin?Debug; do [ -d "$d" ] && echo "Removing corrupted artifact: $d" && rm -rf "$d"; done
-
 # Backend
 dotnet clean && dotnet restore && dotnet build
-
-# Post-build cleanup (WSL Roslyn recreates literal bin\Debug folders after each build)
-for d in src/*/bin?Debug bin?Debug; do [ -d "$d" ] && rm -rf "$d"; done
+# Note: WSL bin\Debug cleanup handled by PostToolUse hook (wsl-dotnet-cleanup.sh)
 
 # Frontend (if applicable)
 npm run typecheck

package/templates/skills/apex/references/smartstack-frontend.md

@@ -44,7 +44,7 @@ element: (
 - **NEVER** static-import page components in route files
 - **ALWAYS** use `<Suspense fallback={<PageLoader />}>` around lazy components
 - **ALWAYS** use the `.then(m => ({ default: m.ComponentName }))` pattern for named exports
-- Layout components (AdminLayout, BusinessLayout, UserLayout) are ALSO lazy-loaded
+- The unified AppLayout component is ALSO lazy-loaded
 
 **FORBIDDEN:**
 ```tsx

package/templates/skills/apex/steps/step-03-execute.md

@@ -77,12 +77,9 @@ For each entity:
 2. Verify ALL EF configurations are registered (ApplyConfigurationsFromAssembly or individual)
 3. MCP suggest_migration → get standardized name
 4. dotnet ef migrations add {Name} --project src/{Infra}.csproj --startup-project src/{Api}.csproj -o Persistence/Migrations
-5. Cleanup corrupted EF Core artifacts (WSL Roslyn bug — creates literal `bin\Debug` folder):
-   for d in src/*/bin?Debug bin?Debug; do [ -d "$d" ] && rm -rf "$d"; done
-6. dotnet ef database update (if local DB)
-7. dotnet build → MUST PASS
-8. Verify: dotnet ef migrations has-pending-model-changes → MUST report "No pending model changes"
-9. Post-build cleanup (WSL recreates artifacts): for d in src/*/bin?Debug bin?Debug; do [ -d "$d" ] && rm -rf "$d"; done
+5. dotnet ef database update (if local DB)
+6. dotnet build → MUST PASS
+7. Verify: dotnet ef migrations has-pending-model-changes → MUST report "No pending model changes"
 ```
 
 **BLOCKING:** If build fails after migration, fix EF configs before proceeding.
@@ -92,8 +89,7 @@ For each entity:
 
 ```bash
 dotnet build
-# Cleanup WSL Roslyn artifacts (literal bin\Debug folders)
-for d in src/*/bin?Debug bin?Debug; do [ -d "$d" ] && rm -rf "$d"; done
+# Note: WSL bin\Debug cleanup handled by PostToolUse hook (wsl-dotnet-cleanup.sh)
 ```
 
 **MUST PASS before Layer 1. If NuGet error, run `dotnet restore` first. If file lock (MSB3021), use `--output /tmp/{project}_build`.**
@@ -264,7 +260,6 @@ After Layer 1 completes:
 
 2. Build gate:
    dotnet build → MUST PASS
-   for d in src/*/bin?Debug bin?Debug; do [ -d "$d" ] && rm -rf "$d"; done
    npm run typecheck → MUST PASS (if frontend)
 ```
 

package/templates/skills/apex/steps/step-08-run-tests.md

@@ -17,8 +17,7 @@ next_step: COMPLETE
 
 ```bash
 dotnet build --no-restore
-# Cleanup WSL Roslyn artifacts (literal bin\Debug folders)
-for d in src/*/bin?Debug bin?Debug; do [ -d "$d" ] && rm -rf "$d"; done
+# Note: WSL bin\Debug cleanup handled by PostToolUse hook (wsl-dotnet-cleanup.sh)
 ```
 
 **MUST PASS before running tests.**