fix: keep generated FastAPI service tests from hanging on codefly.init (#7)

The generated service's pytest hung indefinitely under the Python runtime's
Test step. codefly-sdk's init() walks parent directories for
service.codefly.yaml without stopping at the filesystem root, so it spins
forever when the manifest isn't found — which is exactly the case core's test
runner creates by executing pytest against an isolated source snapshot (the
snapshot holds code/, not the service-root manifest). Importing src.main ran
init() at module load and wedged collection.

- Guard the manifest lookup in main.py with a bounded search; init() only runs
  when service.codefly.yaml is actually reachable, and runtime config
  otherwise flows through CODEFLY__* env vars.
- Make the admin test hermetic: seed the CODEFLY__* values it needs (setdefault
  preserves real runtime values) and drop the codefly.init("..") walk that
  would spin in the snapshot.

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

Author: antoinetoussaint-byte

templates/factory/code/src/main.py.tmpl

@@ -1,8 +1,28 @@
+import os
+
 from fastapi import FastAPI
 from fastapi.middleware.cors import CORSMiddleware
 import codefly_sdk.codefly as codefly
 
-codefly.init()
+
+def _service_manifest_reachable() -> bool:
+    # codefly.init() walks parent directories for service.codefly.yaml but does
+    # not stop at the filesystem root, so it loops forever when the manifest is
+    # absent (e.g. tests running against an isolated source snapshot). Only init
+    # when the manifest is actually reachable; runtime config is otherwise
+    # supplied through CODEFLY__* environment variables.
+    directory = os.getcwd()
+    while True:
+        if os.path.isfile(os.path.join(directory, "service.codefly.yaml")):
+            return True
+        parent = os.path.dirname(directory)
+        if parent == directory:
+            return False
+        directory = parent
+
+
+if _service_manifest_reachable():
+    codefly.init()
 
 app = FastAPI()
 

templates/factory/code/tests/admin/test_admin.py.tmpl

@@ -1,21 +1,26 @@
+import os
+
+# Provide the runtime configuration the app reads without depending on a
+# service.codefly.yaml on disk: the test runner executes against an isolated
+# source snapshot where the manifest isn't present, and codefly.init()'s
+# parent-directory walk would otherwise spin. setdefault preserves any real
+# values injected by the codefly runtime.
+os.environ.setdefault("CODEFLY__MODULE", "mod")
+os.environ.setdefault("CODEFLY__SERVICE", "svc")
+os.environ.setdefault("CODEFLY__SERVICE_VERSION", "0.0.0")
+
 import pytest
 from httpx import AsyncClient, ASGITransport
 
 from src.main import app
 
-import codefly_sdk.codefly as codefly
-
 from src.admin.version import get_version
 from src.admin.models import Version
 
 
-
 @pytest.mark.asyncio
 async def test_version():
-    codefly.init("..")
-    endpoint = codefly.endpoint(api="rest")
-    address = endpoint.address if endpoint else "http://localhost:8080"
-    async with AsyncClient(transport=ASGITransport(app=app), base_url=address) as ac:
+    async with AsyncClient(transport=ASGITransport(app=app), base_url="http://localhost:8080") as ac:
         response = await ac.get("/version")
     assert response.status_code == 200