Update CLI to support file uploads (when creating a new document)

Author: pawelniewie

README.md

@@ -197,11 +197,13 @@ funcli documents get abc123 --format json
 funcli documents create <space-npi> [file]
 ```
 
-Creates a new document from a markdown file or stdin. Supports frontmatter for metadata.
+Creates a new document from a markdown file, Word/OpenOffice file, or stdin. Supports frontmatter for metadata.
 
 **Arguments:**
 - `<space-npi>` - Space NPI where the document will be created
-- `[file]` - Path to markdown file (optional, reads from stdin if omitted)
+- `[file]` - Path to file (optional, reads markdown from stdin if omitted)
+  - Markdown files: `.md`
+  - Word/OpenOffice files: `.docx`, `.doc`, `.odt`, `.rtf`, `.txt`
 
 **Options:**
 - `-p, --parent <npi>` - Parent document NPI (for nested documents)
@@ -234,16 +236,25 @@ parentNpi: abc123
 **Examples:**
 
 ```bash
-# Create from file (title from frontmatter or filename)
+# Create from markdown file (title from frontmatter or filename)
 funcli documents create z2zK66AaEF my-document.md
 
+# Create from Word document
+funcli documents create z2zK66AaEF report.docx
+
+# Create from OpenOffice document
+funcli documents create z2zK66AaEF document.odt
+
 # Create from file with custom title
 funcli documents create z2zK66AaEF my-document.md --title "Custom Title"
 
 # Create nested document (under parent)
 funcli documents create z2zK66AaEF child.md --parent abc123
 
-# Create from stdin
+# Create nested Word document
+funcli documents create z2zK66AaEF section.docx --parent abc123 --title "Section 1"
+
+# Create from stdin (markdown only)
 echo "# My Document\n\nContent here" | funcli documents create z2zK66AaEF
 
 # Create from stdin with title
@@ -428,6 +439,12 @@ echo "# My Notes\n\nSome content" | funcli documents create z2zK66AaEF --title "
 # Import an existing markdown file
 funcli documents create z2zK66AaEF README.md
 
+# Import a Word document
+funcli documents create z2zK66AaEF report.docx
+
+# Import an OpenOffice document
+funcli documents create z2zK66AaEF document.odt --title "Important Document"
+
 # Update an existing document
 funcli documents update abc123 updated-content.md
 
@@ -439,10 +456,15 @@ funcli documents create z2zK66AaEF parent.md
 # Note the NPI of the created document, then:
 funcli documents create z2zK66AaEF child.md --parent <parent-npi>
 
-# Batch import multiple documents
+# Batch import markdown documents
 for file in docs/*.md; do
   funcli documents create z2zK66AaEF "$file"
 done
+
+# Batch import Word documents
+for file in reports/*.docx; do
+  funcli documents create z2zK66AaEF "$file"
+done
 ```
 
 ### Search for a space and get its documents

package-lock.json

@@ -38,6 +38,7 @@
     "chalk": "^4.1.2",
     "commander": "^12.1.0",
     "dotenv": "^16.4.7",
+    "form-data": "^4.0.5",
     "gray-matter": "^4.0.3"
   },
   "devDependencies": {

package.json

@@ -139,44 +139,72 @@ documentsCommand
 
 documentsCommand
   .command("create <space-npi>")
-  .description("Create a new document from markdown file or stdin")
-  .argument("[file]", "Markdown file (omit to read from stdin)")
+  .description("Create a new document from markdown file, Word/OpenOffice file, or stdin")
+  .argument("[file]", "File to upload (markdown, .docx, .doc, .odt, etc.) or omit to read markdown from stdin")
   .option("-p, --parent <npi>", "Parent document NPI (for nested documents)")
   .option("-t, --title <title>", "Document title (overrides frontmatter)")
   .action(withClient(async (client, spaceNpi, file, options) => {
-    // Read content from file or stdin
-    let content;
-    if (file) {
-      content = fs.readFileSync(file, "utf8");
+    // Determine if this is a file upload (Word/OpenOffice) or markdown content
+    const fileUploadExtensions = [".docx", ".doc", ".odt", ".rtf", ".txt"];
+    const isFileUpload = file && fileUploadExtensions.some(ext => file.toLowerCase().endsWith(ext));
+
+    if (isFileUpload) {
+      // File upload path - send file directly
+      if (!fs.existsSync(file)) {
+        console.error(chalk.red("Error:"), `File not found: ${file}`);
+        process.exit(1);
+      }
+
+      // Determine title (priority: CLI arg > filename)
+      let title = options.title;
+      if (!title) {
+        title = path.basename(file, path.extname(file));
+      }
+
+      // Create document from file
+      const document = await client.createDocument(spaceNpi, {
+        title,
+        parentDocumentNpi: options.parent,
+        file
+      });
+
+      console.log(chalk.green("✓") + " Document created successfully from file!");
+      console.log(chalk.bold(document.title) + chalk.gray(` (${document.npi})`));
     } else {
-      // Read from stdin
-      content = await readStdin();
-    }
+      // Markdown path - parse frontmatter
+      let content;
+      if (file) {
+        content = fs.readFileSync(file, "utf8");
+      } else {
+        // Read from stdin
+        content = await readStdin();
+      }
 
-    // Parse frontmatter
-    const { data: frontmatter, content: markdown } = matter(content);
+      // Parse frontmatter
+      const { data: frontmatter, content: markdown } = matter(content);
 
-    // Determine title (priority: CLI arg > frontmatter > filename > "Untitled")
-    let title = options.title || frontmatter.title;
-    if (!title && file) {
-      title = path.basename(file, path.extname(file));
-    }
-    if (!title) {
-      title = "Untitled";
-    }
+      // Determine title (priority: CLI arg > frontmatter > filename > "Untitled")
+      let title = options.title || frontmatter.title;
+      if (!title && file) {
+        title = path.basename(file, path.extname(file));
+      }
+      if (!title) {
+        title = "Untitled";
+      }
 
-    // Determine parent (priority: CLI arg > frontmatter)
-    const parentDocumentNpi = options.parent || frontmatter.parentNpi;
+      // Determine parent (priority: CLI arg > frontmatter)
+      const parentDocumentNpi = options.parent || frontmatter.parentNpi;
 
-    // Create document
-    const document = await client.createDocument(spaceNpi, {
-      title,
-      markdown,
-      parentDocumentNpi
-    });
+      // Create document from markdown
+      const document = await client.createDocument(spaceNpi, {
+        title,
+        markdown,
+        parentDocumentNpi
+      });
 
-    console.log(chalk.green("✓") + " Document created successfully!");
-    console.log(chalk.bold(document.title) + chalk.gray(` (${document.npi})`));
+      console.log(chalk.green("✓") + " Document created successfully!");
+      console.log(chalk.bold(document.title) + chalk.gray(` (${document.npi})`));
+    }
   }));
 
 documentsCommand

src/cli.js

@@ -1,4 +1,6 @@
 import axios from "axios";
+import FormData from "form-data";
+import fs from "fs";
 
 export class FundamentoClient {
   constructor(config) {
@@ -53,17 +55,39 @@ export class FundamentoClient {
     return response.data;
   }
 
-  async createDocument(spaceNpi, { title, markdown, parentDocumentNpi }) {
-    const response = await this.axios.post("/api/v1/documents", {
-      document: {
-        title,
-        markdown,
-        parent_document_npi: parentDocumentNpi
+  async createDocument(spaceNpi, { title, markdown, parentDocumentNpi, file }) {
+    if (file) {
+      // Use multipart form data for file uploads
+      const formData = new FormData();
+      formData.append("document[file]", fs.createReadStream(file));
+      if (title) {
+        formData.append("document[title]", title);
       }
-    }, {
-      params: { space_npi: spaceNpi }
-    });
-    return response.data;
+      if (parentDocumentNpi) {
+        formData.append("document[parent_document_npi]", parentDocumentNpi);
+      }
+
+      const response = await this.axios.post("/api/v1/documents", formData, {
+        params: { space_npi: spaceNpi },
+        headers: {
+          ...formData.getHeaders(),
+          "Authorization": `Bearer ${this.config.apiKey}`
+        }
+      });
+      return response.data;
+    } else {
+      // Use JSON for markdown content
+      const response = await this.axios.post("/api/v1/documents", {
+        document: {
+          title,
+          markdown,
+          parent_document_npi: parentDocumentNpi
+        }
+      }, {
+        params: { space_npi: spaceNpi }
+      });
+      return response.data;
+    }
   }
 
   async updateDocument(npi, { markdown }) {

src/client.js

@@ -129,3 +129,84 @@ test("FundamentoClient updateDocument should format request correctly", () => {
   // Restore original patch
   client.axios.patch = originalPatch;
 });
+
+test("FundamentoClient createDocument with file should use FormData", async () => {
+  const config = new Config({ apiKey: "test-key" });
+  const client = new FundamentoClient(config);
+
+  // Mock axios post
+  const originalPost = client.axios.post;
+  let capturedUrl, capturedData, capturedConfig;
+
+  client.axios.post = async (url, data, config) => {
+    capturedUrl = url;
+    capturedData = data;
+    capturedConfig = config;
+    return { data: { npi: "test123", title: "Test Document" } };
+  };
+
+  // Call createDocument with file
+  await client.createDocument("space123", {
+    title: "Test Document",
+    parentDocumentNpi: "parent123",
+    file: "../fundamento-cloud/spec/fixtures/files/pandoc/Volume-2-Terms-of-Reference.docx"
+  });
+
+  // Verify the request format
+  assert.strictEqual(capturedUrl, "/api/v1/documents");
+
+  // Verify FormData was used (it should have getHeaders method)
+  assert.ok(capturedData);
+  assert.ok(typeof capturedData.getHeaders === "function");
+
+  // Verify params and headers
+  assert.ok(capturedConfig);
+  assert.deepStrictEqual(capturedConfig.params, { space_npi: "space123" });
+  assert.ok(capturedConfig.headers);
+  assert.strictEqual(capturedConfig.headers.Authorization, "Bearer test-key");
+
+  // Restore original post
+  client.axios.post = originalPost;
+});
+
+test("FundamentoClient createDocument with markdown should use JSON", async () => {
+  const config = new Config({ apiKey: "test-key" });
+  const client = new FundamentoClient(config);
+
+  // Mock axios post
+  const originalPost = client.axios.post;
+  let capturedUrl, capturedData, capturedConfig;
+
+  client.axios.post = async (url, data, config) => {
+    capturedUrl = url;
+    capturedData = data;
+    capturedConfig = config;
+    return { data: { npi: "test123", title: "Test Document" } };
+  };
+
+  // Call createDocument with markdown
+  await client.createDocument("space123", {
+    title: "Test Document",
+    markdown: "# Hello",
+    parentDocumentNpi: "parent123"
+  });
+
+  // Verify the request format
+  assert.strictEqual(capturedUrl, "/api/v1/documents");
+
+  // Verify JSON was used (not FormData)
+  assert.deepStrictEqual(capturedData, {
+    document: {
+      title: "Test Document",
+      markdown: "# Hello",
+      parent_document_npi: "parent123"
+    }
+  });
+
+  // Verify params
+  assert.ok(capturedConfig);
+  assert.deepStrictEqual(capturedConfig.params, { space_npi: "space123" });
+
+  // Restore original post
+  client.axios.post = originalPost;
+});