FastAPI server issue with npm run dev

[meghraj-p]

main.py computes cli_path incorrectly, so it adds the wrong path and Python falls back to the globally installed nao_core (which lacks NaoConfigError).

[Bl3f]

Hello @meghraj-p, that's eventually true, nonetheless I think this line of code is mostly useless if I'm not wrong now 🤔 - i will have a look. Thank you so much for reporting.

[getnao-bot]

🔍 AI Analysis

Summary: The FastAPI server computes cli_path incorrectly in main.py, causing it to add the wrong path and fall back to the globally installed nao_core which lacks NaoConfigError.

Labels: None

Prior Art

• None found related to this specific cli_path issue

Files Likely Affected

• main.py — contains the cli_path computation logic that's producing incorrect paths

Proposed Task Breakdown

1. Review main.py cli_path computation logic
2. Identify why wrong path is being added
3. Fix path resolution to use correct local nao_core
4. Test with npm run dev

Confidence

8/10 — The screenshot clearly shows the path issue and error message

---

AI-assisted analysis

[getnao-bot]

🔍 AI Analysis

Summary: The FastAPI server computes cli_path incorrectly in main.py. It uses Path(__file__).parent.parent.parent / "cli" which resolves to apps/cli (doesn't exist) instead of the project root's cli folder.

Labels: None

Root Cause Analysis

In apps/backend/fastapi/main.py:

• Line 19: cli_path = Path(__file__).parent.parent.parent / "cli"
  • __file__ = apps/backend/fastapi/main.py
  • .parent = apps/backend/fastapi
  • .parent.parent = apps/backend
  • .parent.parent.parent = apps
  • Result: apps/cli (which doesn't exist)

The code should point to the project root's cli folder, not apps/cli.

Proposed Fix

--- a/apps/backend/fastapi/main.py
+++ b/apps/backend/fastapi/main.py
@@ -16,7 +16,8 @@ from pydantic import BaseModel
 
 load_dotenv()
 
-cli_path = Path(__file__).parent.parent.parent / "cli"
+# Go up to project root (apps/backend/fastapi -> . -> apps -> .), then to cli
+cli_path = Path(__file__).resolve().parent.parent.parent.parent / "cli"
 sys.path.insert(0, str(cli_path))

Or alternatively, use a more robust approach:

# Find project root by looking for marker file
project_root = Path(__file__).parent
while project_root != project_root.parent:
    if (project_root / "package.json").exists():
        break
    project_root = project_root.parent
cli_path = project_root / "cli"

Alternative Solutions

1. Use environment variable: Set NAO_CLI_PATH env var in Docker compose
2. Use absolute path: Hardcode the CLI path for the Docker container
3. Relative to project root: Create a marker file (e.g., .nao_root) at project root and traverse up to find it

---

AI-assisted analysis