Fix 9 investigation findings; bump to 0.1.1

Resolves bugs found during repository investigation:

- Wire up LICENSE templates for all offered licenses (Apache-2.0, GPL-3.0,
  BSD-3-Clause, Unlicense, Proprietary); previously only MIT rendered a file
- Escape free-text (description, author) in generated TOML/Python via new
  toml_escape/py_escape/fstring_escape Jinja filters; add github_slug for URLs
- Emit valid PyPI trove classifiers and SPDX ids per license (License.classifier
  property; LicenseRef-Proprietary for the non-SPDX Proprietary case)
- Detect .yaml (not just .yml) GitHub Actions workflows in the auditor
- Detect API/app project types in load_project_config, not just CLI/library
- Build output paths with pathlib joins instead of str.format on Path
- Warn instead of silently dropping -r/-e lines in requirements.txt migration
- Handle Poetry's table-form python requirement without AttributeError
- Respect a configured git identity for the initial commit; only fall back to
  the placeholder identity when none is set

Adds tests/test_fixes.py (25 regression tests). Full suite: 196 passing.

Generated with [Claude Code](https://claude.ai/code)
via [Happy](https://happy.engineering)

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Happy <yesreply@happy.engineering>

Author: 3 people

pyproject.toml

@@ -10,7 +10,7 @@
 
 [project]
 name = "quickforge"
-version = "0.1.0"
+version = "0.1.1"
 description = "Modern Python project bootstrapper with 2025's best toolchain"
 readme = "README.md"
 license = "MIT"

src/quickforge/__init__.py

@@ -53,7 +53,7 @@
 # =============================================================================
 # Package Metadata
 # =============================================================================
-__version__ = "0.1.0"
+__version__ = "0.1.1"
 __author__ = "Technical-1"
 __email__ = "jacobkanfer8@gmail.com"
 __license__ = "MIT"

src/quickforge/auditor.py

@@ -453,11 +453,12 @@ def detect_ci(path: Path) -> tuple[str | None, dict[str, str]]:
     tuple[str | None, dict[str, str]]
         Tuple of (ci_name, extra_info).
     """
-    # GitHub Actions
+    # GitHub Actions (workflows may use either .yml or .yaml)
     gh_workflows = path / ".github" / "workflows"
-    if gh_workflows.exists() and any(gh_workflows.glob("*.yml")):
-        workflows = list(gh_workflows.glob("*.yml"))
-        return "github-actions", {"workflows": str(len(workflows))}
+    if gh_workflows.exists():
+        workflows = list(gh_workflows.glob("*.yml")) + list(gh_workflows.glob("*.yaml"))
+        if workflows:
+            return "github-actions", {"workflows": str(len(workflows))}
 
     # GitLab CI
     if (path / ".gitlab-ci.yml").exists():

src/quickforge/generator.py

@@ -59,6 +59,7 @@
 from __future__ import annotations
 
 import os
+import re
 import shutil
 import subprocess
 from dataclasses import dataclass, field
@@ -132,10 +133,15 @@
     ),
 }
 
-# License template mappings
+# License template mappings - every License the CLI offers must map to a
+# template so the generated project always ships a real LICENSE file.
 LICENSE_TEMPLATES: dict[License, str] = {
     License.MIT: "LICENSE_MIT.j2",
-    # Add more licenses as templates are created
+    License.APACHE2: "LICENSE_APACHE2.j2",
+    License.GPL3: "LICENSE_GPL3.j2",
+    License.BSD3: "LICENSE_BSD3.j2",
+    License.UNLICENSE: "LICENSE_UNLICENSE.j2",
+    License.PROPRIETARY: "LICENSE_PROPRIETARY.j2",
 }
 
 
@@ -228,9 +234,56 @@ def create_jinja_env() -> Environment:
     # Add custom filters
     env.filters["snake_case"] = lambda s: s.replace("-", "_").lower()
 
+    # Escaping filters for embedding free-text (description, author name) into
+    # generated files. Autoescaping is disabled for code generation, so every
+    # interpolation of user-supplied text into a string literal must be escaped
+    # explicitly or a stray quote/backslash/brace produces a broken file.
+    env.filters["toml_escape"] = _escape_basic_string
+    env.filters["py_escape"] = _escape_basic_string
+    env.filters["fstring_escape"] = _escape_fstring
+    env.filters["github_slug"] = _github_slug
+
     return env
 
 
+def _github_slug(value: object) -> str:
+    """
+    Turn an author name into a URL-safe slug for GitHub URLs.
+
+    Keeps only lowercase alphanumerics and hyphens so the result is a
+    valid path segment regardless of punctuation in the author name.
+    """
+    return re.sub(r"[^a-z0-9-]", "", str(value).lower())
+
+
+def _escape_basic_string(value: object) -> str:
+    """
+    Escape a value for embedding inside a double-quoted string literal.
+
+    The result is safe for TOML basic strings and for Python single-,
+    double-, and triple-quoted string literals (escaping the quote also
+    neutralises an embedded ``\"\"\"``).
+    """
+    return (
+        str(value)
+        .replace("\\", "\\\\")
+        .replace('"', '\\"')
+        .replace("\n", "\\n")
+        .replace("\r", "\\r")
+        .replace("\t", "\\t")
+    )
+
+
+def _escape_fstring(value: object) -> str:
+    """
+    Escape a value for embedding inside a Python f-string literal.
+
+    In addition to the standard string escaping, literal braces must be
+    doubled so they are not interpreted as f-string replacement fields.
+    """
+    return _escape_basic_string(value).replace("{", "{{").replace("}", "}}")
+
+
 # =============================================================================
 # Directory Structure Creation
 # =============================================================================
@@ -409,14 +462,20 @@ def get_output_path(
     >>> get_output_path("{src_path}/__init__.py", config)
     PosixPath('src/mylib/__init__.py')
     """
-    # Replace placeholders
-    path_str = template_path.format(
-        src_path=config.get_src_path(),
-        package_name=config.package_name,
-        test_path=config.get_test_path(),
-    )
+    # Resolve placeholders segment-by-segment using pathlib joins rather than
+    # string formatting. Formatting a Path via str() can mix separators on
+    # Windows; joining keeps the result platform-correct.
+    replacements: dict[str, Path] = {
+        "{src_path}": config.get_src_path(),
+        "{test_path}": config.get_test_path(),
+        "{package_name}": Path(config.package_name),
+    }
 
-    return Path(path_str)
+    result = Path()
+    for segment in template_path.split("/"):
+        result = result / replacements.get(segment, Path(segment))
+
+    return result
 
 
 def render_all_templates(
@@ -629,19 +688,26 @@ def init_git_repository(project_dir: Path) -> bool:
             check=True,
         )
 
+        # Only inject a placeholder identity when the user has none configured;
+        # otherwise the user's real git identity should author their own
+        # project's initial commit.
+        commit_env = None
+        if not _has_git_identity(project_dir):
+            commit_env = {
+                **os.environ,
+                "GIT_AUTHOR_NAME": "quickforge",
+                "GIT_AUTHOR_EMAIL": "quickforge@example.com",
+                "GIT_COMMITTER_NAME": "quickforge",
+                "GIT_COMMITTER_EMAIL": "quickforge@example.com",
+            }
+
         # Create initial commit
         subprocess.run(
             ["git", "commit", "-m", "Initial commit (generated by quickforge)"],
             cwd=project_dir,
             capture_output=True,
             check=True,
-            env={
-                **os.environ,
-                "GIT_AUTHOR_NAME": "quickforge",
-                "GIT_AUTHOR_EMAIL": "quickforge@example.com",
-                "GIT_COMMITTER_NAME": "quickforge",
-                "GIT_COMMITTER_EMAIL": "quickforge@example.com",
-            },
+            env=commit_env,
         )
 
         return True
@@ -650,6 +716,35 @@ def init_git_repository(project_dir: Path) -> bool:
         return False
 
 
+def _has_git_identity(project_dir: Path) -> bool:
+    """
+    Return True if git has both a user name and email configured.
+
+    Checks the effective configuration (which includes global and local
+    config) so a generated project inherits the developer's real identity
+    when one exists.
+    """
+    try:
+        name = subprocess.run(
+            ["git", "config", "user.name"],
+            cwd=project_dir,
+            capture_output=True,
+            text=True,
+            check=False,
+        )
+        email = subprocess.run(
+            ["git", "config", "user.email"],
+            cwd=project_dir,
+            capture_output=True,
+            text=True,
+            check=False,
+        )
+    except FileNotFoundError:
+        return False
+
+    return bool(name.stdout.strip()) and bool(email.stdout.strip())
+
+
 # =============================================================================
 # Post-Creation Validation
 # =============================================================================
@@ -996,11 +1091,26 @@ def load_project_config(path: Path) -> ProjectConfig:
     # Extract description
     description = project.get("description", "A Python project")
 
-    # Detect project type
-    project_type = ProjectType.LIBRARY
+    # Detect project type using the strongest available signal.
     scripts = project.get("scripts", {})
+    dependencies = project.get("dependencies", [])
+    dep_names = {
+        re.split(r"[<>=!~ \[]", str(dep).lower(), maxsplit=1)[0].strip()
+        for dep in dependencies
+    }
+    api_frameworks = {"fastapi", "flask", "uvicorn", "starlette", "django"}
+
     if scripts:
         project_type = ProjectType.CLI
+    elif dep_names & api_frameworks:
+        project_type = ProjectType.API
+    elif (path / "src").is_dir():
+        project_type = ProjectType.LIBRARY
+    elif (path / name.replace("-", "_")).is_dir():
+        # Flat layout: a top-level package directory with no src/.
+        project_type = ProjectType.APP
+    else:
+        project_type = ProjectType.LIBRARY
 
     # Extract Python version
     requires_python = project.get("requires-python", ">=3.12")

src/quickforge/models.py

@@ -193,13 +193,47 @@ def spdx_id(self) -> str:
         SPDX identifiers are standardized short names for licenses
         that tools can parse unambiguously.
 
+        For most licenses the enum value already is the SPDX id. The
+        sole exception is ``Proprietary``, which is not a valid SPDX
+        expression; per the SPDX spec, custom licenses are referenced
+        with a ``LicenseRef-`` prefix.
+
         Returns
         -------
         str
-            The SPDX identifier (same as enum value for most).
+            A valid SPDX expression.
         """
+        if self is License.PROPRIETARY:
+            return "LicenseRef-Proprietary"
         return self.value
 
+    @property
+    def classifier(self) -> str:
+        """
+        The PyPI trove classifier for this license.
+
+        PyPI uses its own controlled vocabulary for license classifiers,
+        which does *not* match SPDX identifiers (e.g. SPDX ``Apache-2.0``
+        maps to the classifier ``Apache Software License``). Emitting the
+        SPDX id directly produces classifiers PyPI rejects.
+
+        Returns
+        -------
+        str
+            A valid ``License ::`` trove classifier.
+        """
+        classifiers = {
+            License.MIT: "License :: OSI Approved :: MIT License",
+            License.APACHE2: "License :: OSI Approved :: Apache Software License",
+            License.GPL3: (
+                "License :: OSI Approved :: GNU General Public License v3 (GPLv3)"
+            ),
+            License.BSD3: "License :: OSI Approved :: BSD License",
+            License.UNLICENSE: "License :: Public Domain",
+            License.PROPRIETARY: "License :: Other/Proprietary License",
+        }
+        return classifiers[self]
+
 
 class TypeCheckingMode(str, Enum):
     """

src/quickforge/templates/LICENSE_BSD3.j2

@@ -0,0 +1,28 @@
+BSD 3-Clause License
+
+Copyright (c) {{ year }}, {{ config.author.name }}
+
+Redistribution and use in source and binary forms, with or without
+modification, are permitted provided that the following conditions are met:
+
+1. Redistributions of source code must retain the above copyright notice, this
+   list of conditions and the following disclaimer.
+
+2. Redistributions in binary form must reproduce the above copyright notice,
+   this list of conditions and the following disclaimer in the documentation
+   and/or other materials provided with the distribution.
+
+3. Neither the name of the copyright holder nor the names of its
+   contributors may be used to endorse or promote products derived from
+   this software without specific prior written permission.
+
+THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
+FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.