bugfixes and testing

.github/workflows/integration-tests.yml

@@ -0,0 +1,45 @@
+name: Integration Tests
+
+on:
+  pull_request:
+    branches: [main]
+  workflow_dispatch:
+
+jobs:
+  integration:
+    runs-on: ubuntu-latest
+    env:
+      OVERLEAF_GIT_AUTHOR_NAME: OverleafMCP CI
+      OVERLEAF_GIT_AUTHOR_EMAIL: ci@overleafmcp.test
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: '20'
+          cache: 'npm'
+      - run: npm ci
+      - name: Write projects.json from secrets
+        run: |
+          cat > projects.json << 'EOF'
+          {
+            "projects": {
+              "example_project": {
+                "name": "Example Project",
+                "projectId": "${{ secrets.EXAMPLE_PROJECT_ID }}",
+                "gitToken": "${{ secrets.EXAMPLE_PROJECT_TOKEN }}",
+                "readOnly": false
+              },
+              "readonly_example_project": {
+                "name": "ReadOnly Example Project",
+                "projectId": "${{ secrets.READONLY_PROJECT_ID }}",
+                "gitToken": "${{ secrets.READONLY_PROJECT_TOKEN }}",
+                "readOnly": true
+              }
+            }
+          }
+          EOF
+      - name: Run integration tests
+        run: node --test test/integration.test.js
+      - name: Restore test files on failure
+        if: failure()
+        run: node test/restore.js

.github/workflows/unit-tests.yml

@@ -0,0 +1,18 @@
+name: Unit Tests
+
+on:
+  pull_request:
+    branches: [dev, main]
+  workflow_dispatch:
+
+jobs:
+  unit:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: '20'
+          cache: 'npm'
+      - run: npm ci
+      - run: node --test test/unit.test.js

.gitignore

@@ -1,5 +1,6 @@
 node_modules/
 temp/
+spec/
 projects.json
 .env
 *.log

README.md

@@ -3,6 +3,8 @@
 ![Node.js](https://img.shields.io/badge/node-%3E%3D18-brightgreen)
 ![License: MIT](https://img.shields.io/badge/License-MIT-blue.svg)
 ![MCP](https://img.shields.io/badge/MCP-compatible-purple)
+![Unit Tests](https://github.com/SemPlaatsman/OverleafMCP/actions/workflows/unit-tests.yml/badge.svg)
+![Integration Tests](https://github.com/SemPlaatsman/OverleafMCP/actions/workflows/integration-tests.yml/badge.svg)
 
 > Give any MCP-compatible AI assistant direct access to your Overleaf projects.
 
@@ -284,6 +286,26 @@ Resolution order: `readOnly: true` takes precedence over everything and blocks a
 
 ---
 
+## Testing
+
+OverleafMCP has a two-layer test suite built on Node.js's built-in `node:test` module — no extra test dependencies required.
+
+**Unit tests** cover pure logic: section and BibTeX parsing, path traversal protection, and anchor uniqueness. They run in under a second with no credentials or network access:
+
+```bash
+npm test
+```
+
+**Integration tests** run the full tool stack against a live Overleaf project via git, verifying every tool end-to-end including push verification via `listHistory`. They require a `projects.json` with writable and read-only test projects configured:
+
+```bash
+npm run test:integration
+```
+
+CI runs unit tests on every PR to `dev` and `main`. Integration tests run on PRs to `main` only.
+
+---
+
 ## Attribution
 
 See [ATTRIBUTION.md](./ATTRIBUTION.md) for credits to the open-source projects that informed this work.

overleaf-git-client.js

@@ -17,6 +17,8 @@ const SECTION_LEVELS = {
     subparagraph: 6,
 };
 
+export const PREVIEW_MAX_LENGTH = 100;
+
 export class OverleafGitClient {
     constructor(projectId, gitToken, tempDir) {
         this.projectId = projectId;
@@ -238,7 +240,11 @@ export class OverleafGitClient {
 
     async readFile(filePath) {
         await this.cloneOrPull();
-        return fs.readFile(this._safePath(filePath), 'utf-8');
+        const content = await fs.readFile(this._safePath(filePath), 'utf-8');
+        // Normalise Windows-style CRLF to LF. Overleaf's git server uses CRLF on
+        // some projects. Without this, str_replace and insert_before/after anchors
+        // constructed with LF would fail to match even when visually identical.
+        return content.replace(/\r\n/g, '\n');
     }
 
     /**
@@ -271,7 +277,7 @@ export class OverleafGitClient {
             const contentStart = entry._cmdEndIndex;
             const contentEnd = i + 1 < flat.length ? flat[i + 1].startIndex : content.length;
             entry.content = content.substring(contentStart, contentEnd).trim();
-            entry.preview = entry.content.substring(0, 100).replace(/\s+/g, ' ');
+            entry.preview = entry.content.substring(0, PREVIEW_MAX_LENGTH).replace(/\s+/g, ' ');
             delete entry._cmdEndIndex;
         });