@aigne/doc-smith 0.8.11 → 0.8.12-beta

package/.github/workflows/publish-docs.yml

@@ -2,6 +2,7 @@ name: publish docs
 
 env:
   GEMINI_API_KEY: ${{ secrets.GEMINI_API_KEY }}
+  OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
 
 permissions:
   contents: write

package/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "0.8.11"
+  ".": "0.8.12-beta"
 }

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.8.12-beta](https://github.com/AIGNE-io/aigne-doc-smith/compare/v0.8.11...v0.8.12-beta) (2025-10-07)
+
+
+### Bug Fixes
+
+* read file tool should support relative path ([#162](https://github.com/AIGNE-io/aigne-doc-smith/issues/162)) ([cf1291b](https://github.com/AIGNE-io/aigne-doc-smith/commit/cf1291b0be5b51db15d052310c30b0552a4092e1))
+
 ## [0.8.11](https://github.com/AIGNE-io/aigne-doc-smith/compare/v0.8.11-beta.7...v0.8.11) (2025-10-05)
 
 

package/agents/update/fs-tools/read-file.mjs

@@ -1,6 +1,6 @@
 import fs from "node:fs";
 import fsPromises from "node:fs/promises";
-import path from "node:path";
+import path, { isAbsolute } from "node:path";
 
 /**
  * Detects if a file is likely binary by checking for null bytes
@@ -199,6 +199,8 @@ export default async function readFile({ path: filePath, offset, limit, encoding
   let result = {};
   let error = null;
 
+  if (filePath && !isAbsolute(filePath)) filePath = path.join(process.cwd(), filePath);
+
   try {
     // Validate file path first (this checks if it's absolute)
     const pathError = validateFilePath(filePath);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aigne/doc-smith",
-  "version": "0.8.11",
+  "version": "0.8.12-beta",
   "description": "AI-driven documentation generation tool built on the AIGNE Framework",
   "publishConfig": {
     "access": "public"

package/tests/agents/update/fs-tools/read-file.test.mjs

@@ -1,8 +1,8 @@
+import { afterEach, beforeEach, describe, it } from "bun:test";
 import assert from "node:assert";
 import fs from "node:fs/promises";
 import os from "node:os";
 import path from "node:path";
-import { afterEach, beforeEach, describe, it } from "bun:test";
 import readFile from "../../../../agents/update/fs-tools/read-file.mjs";
 
 describe("read-file tool", () => {
@@ -11,11 +11,13 @@ describe("read-file tool", () => {
   beforeEach(async () => {
     // Create temporary directory for test files
     tempDir = await fs.mkdtemp(path.join(os.tmpdir(), "read-file-test-"));
+    process.chdir(tempDir);
   });
 
   afterEach(async () => {
     // Clean up temporary directory
     await fs.rm(tempDir, { recursive: true, force: true });
+    process.chdir(__dirname);
   });
 
   describe("basic functionality", () => {
@@ -25,13 +27,16 @@ describe("read-file tool", () => {
       await fs.writeFile(filePath, content, "utf8");
 
       const result = await readFile({
-        path: filePath,
+        path: "test.txt",
       });
 
       assert.strictEqual(result.command, "read_file");
       assert.strictEqual(result.error, null);
       assert.strictEqual(result.result.content, content);
-      assert.strictEqual(result.result.metadata.path, filePath);
+      assert.strictEqual(
+        await fs.realpath(result.result.metadata.path),
+        await fs.realpath(filePath),
+      );
       assert.strictEqual(result.result.metadata.mimeType, "text/plain");
       assert.strictEqual(result.result.metadata.isBinary, false);
       assert.strictEqual(result.result.metadata.encoding, "utf8");
@@ -348,15 +353,6 @@ describe("read-file tool", () => {
       assert(result.error.message.includes("required"));
     });
 
-    it("should require absolute path", async () => {
-      const result = await readFile({
-        path: "relative/path.txt",
-      });
-
-      assert.notStrictEqual(result.error, null);
-      assert(result.error.message.includes("absolute"));
-    });
-
     it("should handle non-existent files", async () => {
       const result = await readFile({
         path: path.join(tempDir, "nonexistent.txt"),