fix: remove unused imports and handle symlinks in manifest

Remove unused `get_project_path` imports from features.py and
schedules.py (ruff F401). Add symlink detection in detach manifest
creation to prevent is_dir() from following symlinks.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: cabana8471-arch

detach.py

@@ -113,7 +113,7 @@ def get_autoforge_version() -> str:
 class FileEntry(TypedDict):
     """Type for manifest file entry."""
     path: str
-    type: str  # "file" or "directory"
+    type: str  # "file", "directory", or "symlink"
     size: int
     checksum: str | None  # MD5 for files, None for directories
     file_count: int | None  # Number of files for directories
@@ -391,7 +391,17 @@ def create_backup(
     for file_path in files:
         relative_path = file_path.relative_to(project_dir)
 
-        if file_path.is_dir():
+        if file_path.is_symlink():
+            # Handle symlinks before is_dir() which follows symlinks
+            manifest_files.append({
+                "path": str(relative_path),
+                "type": "symlink",
+                "size": 0,
+                "checksum": None,
+                "file_count": None,
+            })
+            total_file_count += 1
+        elif file_path.is_dir():
             size, count = get_directory_info(file_path)
             manifest_files.append({
                 "path": str(relative_path),

server/routers/features.py

@@ -25,7 +25,6 @@
     FeatureResponse,
     FeatureUpdate,
 )
-from ..utils.project_helpers import get_project_path as _get_project_path
 from ..utils.validation import validate_project_name
 
 # Lazy imports to avoid circular dependencies

server/routers/schedules.py

@@ -26,7 +26,6 @@
     ScheduleResponse,
     ScheduleUpdate,
 )
-from ..utils.project_helpers import get_project_path as _get_project_path
 from ..utils.validation import validate_project_name
 
 if TYPE_CHECKING: