feat: CLI: Native DuckDB file loading - #409

apps/sqlrooms-cli-ui/src/store.ts (1)

114-130: Optional: Add defensive warning when native file path is missing.

In the CLI environment, the file.path property should consistently be available from native file drops. However, adding a console warning when the fallback to file.name is used would help surface any unexpected scenarios during development.

💡 Suggested defensive check

 function getCliFileReference(file: File): string {
   const nativePath = (file as FileWithNativePath).path;
   if (typeof nativePath === 'string' && nativePath.trim()) {
     return nativePath;
   }
+  console.warn(
+    `Native file path not available for "${file.name}". Falling back to filename only.`
+  );
   return file.name;
 }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@apps/sqlrooms-cli-ui/src/store.ts` around lines 114 - 130, Add a defensive
warning when we fall back from a missing native path to file.name: inside
getCliFileReference (which uses type FileWithNativePath) detect when nativePath
is not a non-empty string and before returning file.name call console.warn (or
the app logger) indicating the missing native path for the given file (include
file.name and any available metadata). Then ensure connector.loadFile (which
wraps baseLoadFile) continues to call baseLoadFile with the returned reference;
keep the function names getCliFileReference, FileWithNativePath,
connector.loadFile and baseLoadFile intact so callers remain unchanged.

Configuration used: defaults

Review profile: CHILL

Plan: Pro