From f6cae496a98aa136ded85e5017e7e6c92b548492 Mon Sep 17 00:00:00 2001
From: danielmeppiel <dmeppiel@microsoft.com>
Date: Wed, 17 Sep 2025 14:31:54 +0200
Subject: [PATCH] Add apm_cli to pyproject.toml and fix apm module url
 validation

---
 pyproject.toml                    | 2 +-
 src/apm_cli/models/apm_package.py | 8 ++++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/pyproject.toml b/pyproject.toml
index cce203a..a01e89f 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -41,4 +41,4 @@ requires = ["hatchling"]
 build-backend = "hatchling.build"
 
 [tool.hatch.build.targets.wheel]
-packages = ["src/specify_cli"]
+packages = ["src/specify_cli", "src/apm_cli"]
diff --git a/src/apm_cli/models/apm_package.py b/src/apm_cli/models/apm_package.py
index 13e82aa..a349085 100644
--- a/src/apm_cli/models/apm_package.py
+++ b/src/apm_cli/models/apm_package.py
@@ -75,6 +75,10 @@ class DependencyReference:
         if not dependency_str.strip():
             raise ValueError("Empty dependency string")
         
+        # Check for control characters (newlines, tabs, etc.)
+        if any(ord(c) < 32 for c in dependency_str):
+            raise ValueError("Dependency string contains invalid control characters")
+        
         # Handle SSH URLs first (before @ processing) to avoid conflict with alias separator
         original_str = dependency_str
         if dependency_str.startswith("git@github.com:"):
@@ -192,6 +196,10 @@ class DependencyReference:
         if not re.match(r'^[a-zA-Z0-9._-]+/[a-zA-Z0-9._-]+$', repo_url):
             raise ValueError(f"Invalid repository format: {repo_url}. Expected 'user/repo'")
         
+        # Validate alias characters if present
+        if alias and not re.match(r'^[a-zA-Z0-9._-]+$', alias):
+            raise ValueError(f"Invalid alias: {alias}. Aliases can only contain letters, numbers, dots, underscores, and hyphens")
+        
         return cls(repo_url=repo_url, reference=reference, alias=alias)
     
     def to_github_url(self) -> str: