chore: refine status handling and fallbacks

Co-authored-by: tikazyq <[email protected]>

Author: Copilot

packages/ui/src/app/api/projects/[id]/specs/route.ts

@@ -13,6 +13,13 @@ import { NextResponse } from 'next/server';
 import { getSpecs } from '@/lib/db/service-queries';
 import { isDefaultProject } from '@/lib/projects/constants';
 
+const VALID_STATUSES = ['planned', 'in-progress', 'complete', 'archived'];
+
+function isValidStatusFilter(statusParam: string | null): boolean {
+  if (!statusParam) return true;
+  return statusParam.split(',').every((s) => VALID_STATUSES.includes(s));
+}
+
 export async function GET(
   request: Request,
   { params }: { params: Promise<{ id: string }> }
@@ -21,9 +28,7 @@ export async function GET(
     const { id } = await params;
     const url = new URL(request.url);
     const statusParam = url.searchParams.get('status');
-    const validStatuses = ['planned', 'in-progress', 'complete', 'archived'];
-
-    if (statusParam && !statusParam.split(',').every((s) => validStatuses.includes(s))) {
+    if (!isValidStatusFilter(statusParam)) {
       return NextResponse.json(
         { error: 'Invalid status filter', code: 'INVALID_REQUEST' },
         { status: 400 }

rust/leanspec-http/src/handlers/specs.rs

@@ -190,14 +190,15 @@ pub async fn get_spec(
 
     // Compute required_by
     if let Some(complete) = dep_graph.get_complete_graph(&spec.path) {
-        detail.required_by = complete
+        let required_by: Vec<String> = complete
             .required_by
             .iter()
             .map(|s| s.path.clone())
             .collect();
+        detail.required_by = required_by.clone();
         detail.relationships = Some(crate::types::SpecRelationships {
             depends_on: detail.depends_on.clone(),
-            required_by: Some(detail.required_by.clone()),
+            required_by: Some(required_by),
         });
     }
 
@@ -233,14 +234,15 @@ pub async fn get_project_spec(
 
     // Compute required_by
     if let Some(complete) = dep_graph.get_complete_graph(&spec.path) {
-        detail.required_by = complete
+        let required_by: Vec<String> = complete
             .required_by
             .iter()
             .map(|s| s.path.clone())
             .collect();
+        detail.required_by = required_by.clone();
         detail.relationships = Some(crate::types::SpecRelationships {
             depends_on: detail.depends_on.clone(),
-            required_by: Some(detail.required_by.clone()),
+            required_by: Some(required_by),
         });
     }
 

rust/leanspec-http/src/state.rs

@@ -66,8 +66,8 @@ fn default_project_path() -> PathBuf {
         return PathBuf::from(explicit);
     }
 
-    // Fall back to the expected specs root if current_dir is unavailable
-    let mut dir = std::env::current_dir().unwrap_or_else(|_| PathBuf::from("specs"));
+    // Fall back to the current working directory when resolution fails
+    let mut dir = std::env::current_dir().unwrap_or_else(|_| PathBuf::from("."));
     loop {
         if dir.join("specs").exists() {
             return dir;