@adsim/wordpress-mcp-server 3.0.0 → 4.4.0

package/tests/unit/tools/findOrphanPages.test.js

@@ -0,0 +1,145 @@
+import { vi, describe, it, expect, beforeEach, afterEach } from 'vitest';
+
+vi.mock('node-fetch', () => ({ default: vi.fn() }));
+
+import fetch from 'node-fetch';
+import { handleToolCall, _testSetTarget } from '../../../index.js';
+import { mockSuccess, mockError, getAuditLogs, makeRequest, parseResult } from '../../helpers/mockWpRequest.js';
+
+function call(name, args = {}) {
+  return handleToolCall(makeRequest(name, args));
+}
+
+let consoleSpy;
+beforeEach(() => { fetch.mockReset(); consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {}); });
+afterEach(() => { consoleSpy.mockRestore(); _testSetTarget(null); });
+
+// =========================================================================
+// Mock data
+// =========================================================================
+
+const pageA = {
+  id: 1, title: { rendered: 'Home' }, slug: 'home', status: 'publish',
+  link: 'https://test.example.com/home/',
+  content: { rendered: '<p>Welcome! Visit <a href="https://test.example.com/about/">About Us</a> and <a href="https://test.example.com/services/">Services</a>.</p>' }
+};
+const pageB = {
+  id: 2, title: { rendered: 'About' }, slug: 'about', status: 'publish',
+  link: 'https://test.example.com/about/',
+  content: { rendered: '<p>About us. Go <a href="https://test.example.com/home/">Home</a>.</p>' }
+};
+const pageC = {
+  id: 3, title: { rendered: 'Services' }, slug: 'services', status: 'publish',
+  link: 'https://test.example.com/services/',
+  content: { rendered: '<p>Our services.</p>' }
+};
+const pageD = {
+  id: 4, title: { rendered: 'Hidden Page' }, slug: 'hidden', status: 'publish',
+  link: 'https://test.example.com/hidden/',
+  content: { rendered: '<p>This page is not linked from anywhere. It has many words to test sorting so that we can verify the filter and ordering logic works correctly in our test scenario.</p>' }
+};
+
+// =========================================================================
+// wp_find_orphan_pages
+// =========================================================================
+
+describe('wp_find_orphan_pages', () => {
+  it('SUCCESS — no orphan pages when all are linked', async () => {
+    _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+    mockSuccess([pageA, pageB, pageC]);
+
+    const res = await call('wp_find_orphan_pages');
+    const data = parseResult(res);
+
+    expect(data.total_orphans).toBe(0);
+    expect(data.orphans).toHaveLength(0);
+  });
+
+  it('SUCCESS — all pages are orphans when no cross-links', async () => {
+    _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+    const iso1 = { id: 10, title: { rendered: 'Iso1' }, slug: 'iso1', status: 'publish', link: 'https://test.example.com/iso1/', content: { rendered: '<p>No links.</p>' } };
+    const iso2 = { id: 11, title: { rendered: 'Iso2' }, slug: 'iso2', status: 'publish', link: 'https://test.example.com/iso2/', content: { rendered: '<p>No links either.</p>' } };
+    mockSuccess([iso1, iso2]);
+
+    const res = await call('wp_find_orphan_pages');
+    const data = parseResult(res);
+
+    expect(data.total_orphans).toBe(2);
+    expect(data.orphans).toHaveLength(2);
+  });
+
+  it('SUCCESS — exclusion by IDs', async () => {
+    _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+    mockSuccess([pageA, pageB, pageC, pageD]);
+
+    const res = await call('wp_find_orphan_pages', { exclude_ids: [4] });
+    const data = parseResult(res);
+
+    const orphanIds = data.orphans.map(o => o.id);
+    expect(orphanIds).not.toContain(4);
+  });
+
+  it('SUCCESS — filter by min_words', async () => {
+    _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+    const shortPage = { id: 20, title: { rendered: 'Short' }, slug: 'short', status: 'publish', link: 'https://test.example.com/short/', content: { rendered: '<p>Hello.</p>' } };
+    const longPage = { id: 21, title: { rendered: 'Long' }, slug: 'long', status: 'publish', link: 'https://test.example.com/long/', content: { rendered: '<p>This is a much longer page with plenty of words to exceed the minimum threshold for the filter to work correctly in our test scenario today and beyond.</p>' } };
+    mockSuccess([shortPage, longPage]);
+
+    const res = await call('wp_find_orphan_pages', { min_words: 10 });
+    const data = parseResult(res);
+
+    expect(data.orphans.every(o => o.word_count >= 10)).toBe(true);
+  });
+
+  it('SUCCESS — builds link matrix correctly', async () => {
+    _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+    mockSuccess([pageA, pageB, pageC, pageD]);
+
+    const res = await call('wp_find_orphan_pages');
+    const data = parseResult(res);
+
+    // A linked from B, B linked from A, C linked from A → not orphans
+    // D not linked from anyone → orphan
+    expect(data.total_orphans).toBe(1);
+    expect(data.orphans[0].id).toBe(4);
+  });
+
+  it('SUCCESS — orphans sorted by word count descending', async () => {
+    _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+    const short = { id: 30, title: { rendered: 'Short' }, slug: 'short', status: 'publish', link: 'https://test.example.com/short/', content: { rendered: '<p>Short text.</p>' } };
+    const long = { id: 31, title: { rendered: 'Long' }, slug: 'long', status: 'publish', link: 'https://test.example.com/long/', content: { rendered: '<p>This page has significantly more words than the short page because it contains a much longer paragraph with many more words and additional content.</p>' } };
+    mockSuccess([short, long]);
+
+    const res = await call('wp_find_orphan_pages');
+    const data = parseResult(res);
+
+    expect(data.orphans).toHaveLength(2);
+    expect(data.orphans[0].word_count).toBeGreaterThan(data.orphans[1].word_count);
+  });
+
+  it('AUDIT — logs success entry', async () => {
+    _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+    mockSuccess([pageA]);
+
+    await call('wp_find_orphan_pages');
+
+    const logs = getAuditLogs();
+    const entry = logs.find(l => l.tool === 'wp_find_orphan_pages');
+    expect(entry).toBeDefined();
+    expect(entry.status).toBe('success');
+    expect(entry.action).toBe('find_orphan_pages');
+  });
+
+  it('ERROR — logs error on API failure', async () => {
+    _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+    mockError(403, '{"code":"rest_forbidden","message":"Forbidden"}');
+
+    const res = await call('wp_find_orphan_pages');
+    expect(res.isError).toBe(true);
+
+    const logs = getAuditLogs();
+    const entry = logs.find(l => l.tool === 'wp_find_orphan_pages');
+    expect(entry).toBeDefined();
+    expect(entry.status).toBe('error');
+  });
+});

package/tests/unit/tools/findThinContent.test.js

@@ -0,0 +1,145 @@
+import { vi, describe, it, expect, beforeEach, afterEach } from 'vitest';
+
+vi.mock('node-fetch', () => ({ default: vi.fn() }));
+
+import fetch from 'node-fetch';
+import { handleToolCall } from '../../../index.js';
+import { mockSuccess, mockError, getAuditLogs, makeRequest, parseResult } from '../../helpers/mockWpRequest.js';
+
+function call(name, args = {}) {
+  return handleToolCall(makeRequest(name, args));
+}
+
+let consoleSpy;
+beforeEach(() => { fetch.mockReset(); consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {}); });
+afterEach(() => { consoleSpy.mockRestore(); });
+
+// =========================================================================
+// Mock data
+// =========================================================================
+
+const now = new Date();
+const recentDate = new Date(now - 30 * 24 * 60 * 60 * 1000).toISOString(); // 30 days ago
+const oldDate = new Date(now - 800 * 24 * 60 * 60 * 1000).toISOString(); // 800 days ago
+
+function makeArticle(id, wordCount, modified, categories = [5]) {
+  const words = Array(wordCount).fill('word').join(' ');
+  return {
+    id, title: { rendered: `Post ${id}` }, link: `https://test.example.com/post-${id}/`,
+    content: { rendered: `<p>${words}</p>` },
+    modified, date: modified, categories
+  };
+}
+
+// =========================================================================
+// wp_find_thin_content
+// =========================================================================
+
+describe('wp_find_thin_content', () => {
+  it('SUCCESS — no thin content detected', async () => {
+    mockSuccess([makeArticle(1, 500, recentDate, [5])]);
+
+    const res = await call('wp_find_thin_content');
+    const data = parseResult(res);
+
+    expect(data.total_thin).toBe(0);
+    expect(data.articles).toHaveLength(0);
+    expect(data.total_analyzed).toBe(1);
+  });
+
+  it('ISSUE — too_short detected (word_count < 300)', async () => {
+    mockSuccess([makeArticle(1, 200, recentDate, [5])]);
+
+    const res = await call('wp_find_thin_content');
+    const data = parseResult(res);
+
+    expect(data.total_thin).toBe(1);
+    expect(data.articles[0].signals).toContain('too_short');
+    expect(data.articles[0].word_count).toBe(200);
+  });
+
+  it('ISSUE — very_short + severity critical with multiple signals', async () => {
+    mockSuccess([makeArticle(1, 100, oldDate, [1])]);
+
+    const res = await call('wp_find_thin_content');
+    const data = parseResult(res);
+
+    expect(data.articles[0].signals).toContain('very_short');
+    expect(data.articles[0].severity).toBe('critical');
+  });
+
+  it('ISSUE — outdated detected (days > 730)', async () => {
+    mockSuccess([makeArticle(1, 500, oldDate, [5])]);
+
+    const res = await call('wp_find_thin_content');
+    const data = parseResult(res);
+
+    expect(data.articles[0].signals).toContain('outdated');
+    expect(data.articles[0].days_since_update).toBeGreaterThan(730);
+  });
+
+  it('ISSUE — uncategorized detected', async () => {
+    mockSuccess([makeArticle(1, 200, recentDate, [1])]);
+
+    const res = await call('wp_find_thin_content');
+    const data = parseResult(res);
+
+    const article = data.articles.find(a => a.id === 1);
+    expect(article.signals).toContain('uncategorized');
+  });
+
+  it('ISSUE — 3 signals → severity critical', async () => {
+    // very_short + outdated + uncategorized = 3 signals → critical
+    mockSuccess([makeArticle(1, 100, oldDate, [1])]);
+
+    const res = await call('wp_find_thin_content');
+    const data = parseResult(res);
+
+    expect(data.articles[0].signals).toHaveLength(3);
+    expect(data.articles[0].severity).toBe('critical');
+    expect(data.by_severity.critical).toBe(1);
+  });
+
+  it('SUCCESS — suggested_action correct for signals', async () => {
+    // very_short + outdated → delete
+    mockSuccess([
+      makeArticle(1, 100, oldDate, [5]),       // very_short + outdated → delete
+      makeArticle(2, 200, recentDate, [5]),     // too_short only → expand
+      makeArticle(3, 500, oldDate, [5]),        // outdated only → update_or_merge
+    ]);
+
+    const res = await call('wp_find_thin_content');
+    const data = parseResult(res);
+
+    const art1 = data.articles.find(a => a.id === 1);
+    const art2 = data.articles.find(a => a.id === 2);
+    const art3 = data.articles.find(a => a.id === 3);
+    expect(art1.suggested_action).toBe('delete');
+    expect(art2.suggested_action).toBe('expand');
+    expect(art3.suggested_action).toBe('update_or_merge');
+  });
+
+  it('AUDIT — logs success entry', async () => {
+    mockSuccess([makeArticle(1, 500, recentDate)]);
+
+    await call('wp_find_thin_content');
+
+    const logs = getAuditLogs();
+    const entry = logs.find(l => l.tool === 'wp_find_thin_content');
+    expect(entry).toBeDefined();
+    expect(entry.status).toBe('success');
+    expect(entry.action).toBe('audit_seo');
+  });
+
+  it('ERROR — logs error on API failure', async () => {
+    mockError(403, '{"code":"rest_forbidden","message":"Forbidden"}');
+
+    const res = await call('wp_find_thin_content');
+    expect(res.isError).toBe(true);
+
+    const logs = getAuditLogs();
+    const entry = logs.find(l => l.tool === 'wp_find_thin_content');
+    expect(entry).toBeDefined();
+    expect(entry.status).toBe('error');
+  });
+});

package/tests/unit/tools/internalLinks.test.js

@@ -0,0 +1,283 @@
+import { vi, describe, it, expect, beforeEach, afterEach } from 'vitest';
+
+vi.mock('node-fetch', () => ({ default: vi.fn() }));
+
+import fetch from 'node-fetch';
+import { handleToolCall, _testSetTarget } from '../../../index.js';
+import { mockSuccess, getAuditLogs, makeRequest, parseResult } from '../../helpers/mockWpRequest.js';
+import {
+  extractInternalLinks,
+  extractExternalLinks,
+  extractFocusKeyword,
+  calculateRelevanceScore,
+} from '../../../src/linkUtils.js';
+
+function call(name, args = {}) {
+  return handleToolCall(makeRequest(name, args));
+}
+
+const SITE_URL = 'https://test.example.com';
+
+// ────────────────────────────────────────────────────────────
+// extractInternalLinks — unit tests
+// ────────────────────────────────────────────────────────────
+
+describe('extractInternalLinks', () => {
+  it('detects internal links and ignores external', () => {
+    const html = '<a href="https://test.example.com/hello">Hello</a> <a href="https://other.com/bye">Bye</a>';
+    const result = extractInternalLinks(html, SITE_URL);
+    expect(result).toHaveLength(1);
+    expect(result[0].url).toBe('https://test.example.com/hello');
+    expect(result[0].anchor_text).toBe('Hello');
+  });
+
+  it('handles relative hrefs (/)', () => {
+    const html = '<a href="/about">About</a> <a href="/contact">Contact</a>';
+    const result = extractInternalLinks(html, SITE_URL);
+    expect(result).toHaveLength(2);
+    expect(result[0].url).toBe('/about');
+    expect(result[1].url).toBe('/contact');
+  });
+});
+
+// ────────────────────────────────────────────────────────────
+// extractExternalLinks — unit tests
+// ────────────────────────────────────────────────────────────
+
+describe('extractExternalLinks', () => {
+  it('detects external links', () => {
+    const html = '<a href="https://test.example.com/hello">Hello</a> <a href="https://other.com/bye">Bye</a>';
+    const result = extractExternalLinks(html, SITE_URL);
+    expect(result).toHaveLength(1);
+    expect(result[0].url).toBe('https://other.com/bye');
+    expect(result[0].anchor_text).toBe('Bye');
+  });
+});
+
+// ────────────────────────────────────────────────────────────
+// extractFocusKeyword — unit tests
+// ────────────────────────────────────────────────────────────
+
+describe('extractFocusKeyword', () => {
+  it('RankMath detected', () => {
+    expect(extractFocusKeyword({ rank_math_focus_keyword: 'seo tips' })).toBe('seo tips');
+  });
+
+  it('Yoast detected', () => {
+    expect(extractFocusKeyword({ _yoast_wpseo_focuskw: 'wordpress seo' })).toBe('wordpress seo');
+  });
+
+  it('SEOPress detected', () => {
+    expect(extractFocusKeyword({ _seopress_analysis_target_kw: 'content marketing' })).toBe('content marketing');
+  });
+
+  it('AIOSEO detected', () => {
+    expect(extractFocusKeyword({ _aioseo_keywords: 'blogging tips' })).toBe('blogging tips');
+  });
+
+  it('no plugin → null', () => {
+    expect(extractFocusKeyword({})).toBeNull();
+    expect(extractFocusKeyword(null)).toBeNull();
+  });
+});
+
+// ────────────────────────────────────────────────────────────
+// calculateRelevanceScore — unit tests
+// ────────────────────────────────────────────────────────────
+
+describe('calculateRelevanceScore', () => {
+  it('common category +3', () => {
+    const result = calculateRelevanceScore(
+      { id: 2, title: 'Other Post', date: new Date().toISOString(), categories: [1, 5], meta: {}, link: 'https://test.example.com/other' },
+      { id: 1, categories: [1, 3], linkedUrls: [] },
+      []
+    );
+    expect(result.breakdown.category_match).toBe(3);
+  });
+
+  it('freshness < 3 months +3', () => {
+    const recentDate = new Date();
+    recentDate.setMonth(recentDate.getMonth() - 1);
+    const result = calculateRelevanceScore(
+      { id: 2, title: 'Recent', date: recentDate.toISOString(), categories: [], meta: {}, link: 'https://test.example.com/recent' },
+      { id: 1, categories: [], linkedUrls: [] },
+      []
+    );
+    expect(result.breakdown.freshness).toBe(3);
+  });
+
+  it('already linked → score -999', () => {
+    const result = calculateRelevanceScore(
+      { id: 2, title: 'Other', date: new Date().toISOString(), categories: [], meta: {}, link: 'https://test.example.com/other' },
+      { id: 1, categories: [], linkedUrls: ['https://test.example.com/other'] },
+      []
+    );
+    expect(result.total).toBe(-999);
+  });
+});
+
+// ────────────────────────────────────────────────────────────
+// wp_analyze_links — integration tests
+// ────────────────────────────────────────────────────────────
+
+describe('wp_analyze_links', () => {
+  let consoleSpy;
+  beforeEach(() => { fetch.mockReset(); consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {}); });
+  afterEach(() => { consoleSpy.mockRestore(); _testSetTarget(null); });
+
+  const postWithLinks = {
+    id: 42,
+    title: { rendered: 'Test Post' },
+    content: { rendered: '<p>Check <a href="https://test.example.com/page1">Page 1</a> and <a href="https://other.com/ext">External</a></p>' },
+    status: 'publish',
+    meta: {}
+  };
+
+  it('returns correct structure with link checking', async () => {
+    _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+    mockSuccess(postWithLinks);
+    // HEAD for internal link
+    fetch.mockImplementationOnce(() => Promise.resolve({ status: 200 }));
+
+    const res = await call('wp_analyze_links', { post_id: 42 });
+    const data = parseResult(res);
+
+    expect(data.post_id).toBe(42);
+    expect(data.post_title).toBe('Test Post');
+    expect(data.internal_links).toHaveLength(1);
+    expect(data.internal_links[0].url).toBe('https://test.example.com/page1');
+    expect(data.internal_links[0].status).toBe('ok');
+    expect(data.internal_links[0].http_code).toBe(200);
+    expect(data.external_links).toHaveLength(1);
+    expect(data.external_links[0].url).toBe('https://other.com/ext');
+    expect(data.summary.total_internal).toBe(1);
+    expect(data.summary.total_external).toBe(1);
+    expect(data.summary.broken_count).toBe(0);
+
+    const logs = getAuditLogs();
+    const entry = logs.find(l => l.tool === 'wp_analyze_links');
+    expect(entry).toBeDefined();
+    expect(entry.action).toBe('analyze_links');
+  });
+
+  it('check_broken=false → all status "unchecked"', async () => {
+    _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+    mockSuccess(postWithLinks);
+
+    const res = await call('wp_analyze_links', { post_id: 42, check_broken: false });
+    const data = parseResult(res);
+
+    expect(data.internal_links[0].status).toBe('unchecked');
+    expect(data.internal_links[0].http_code).toBeNull();
+  });
+
+  it('NOT blocked by WP_READ_ONLY (read-only tool)', async () => {
+    const original = process.env.WP_READ_ONLY;
+    process.env.WP_READ_ONLY = 'true';
+    try {
+      _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+      mockSuccess(postWithLinks);
+      fetch.mockImplementationOnce(() => Promise.resolve({ status: 200 }));
+
+      const res = await call('wp_analyze_links', { post_id: 42 });
+      expect(res.isError).toBeUndefined();
+      const data = parseResult(res);
+      expect(data.post_id).toBe(42);
+    } finally {
+      if (original === undefined) delete process.env.WP_READ_ONLY;
+      else process.env.WP_READ_ONLY = original;
+    }
+  });
+});
+
+// ────────────────────────────────────────────────────────────
+// wp_suggest_internal_links — integration tests
+// ────────────────────────────────────────────────────────────
+
+describe('wp_suggest_internal_links', () => {
+  let consoleSpy;
+  beforeEach(() => { fetch.mockReset(); consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {}); });
+  afterEach(() => { consoleSpy.mockRestore(); _testSetTarget(null); });
+
+  const currentPost = {
+    id: 1,
+    title: { rendered: 'Current Post About WordPress SEO' },
+    content: { rendered: '<p>Some content with <a href="https://test.example.com/existing-link">existing link</a></p>' },
+    status: 'publish',
+    categories: [5],
+    meta: { rank_math_focus_keyword: 'wordpress seo' }
+  };
+
+  const candidate1 = {
+    id: 10,
+    title: { rendered: 'WordPress SEO Guide' },
+    content: { rendered: '' },
+    status: 'publish',
+    categories: [5],
+    date: new Date().toISOString(),
+    link: 'https://test.example.com/wordpress-seo-guide',
+    meta: { rank_math_focus_keyword: 'seo guide' }
+  };
+
+  const candidate2 = {
+    id: 20,
+    title: { rendered: 'Cooking Tips' },
+    content: { rendered: '' },
+    status: 'publish',
+    categories: [99],
+    date: new Date(Date.now() - 400 * 24 * 60 * 60 * 1000).toISOString(),
+    link: 'https://test.example.com/cooking-tips',
+    meta: {}
+  };
+
+  it('returns suggestions sorted by score', async () => {
+    _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+    mockSuccess(currentPost);             // GET /posts/1
+    mockSuccess([candidate1, candidate2]); // search results
+
+    const res = await call('wp_suggest_internal_links', { post_id: 1 });
+    const data = parseResult(res);
+
+    expect(data.post_id).toBe(1);
+    expect(data.keywords_used).toContain('wordpress seo');
+    expect(data.suggestions.length).toBeGreaterThan(0);
+    // candidate1 should score higher (same category, recent, title match)
+    expect(data.suggestions[0].target_post_id).toBe(10);
+    expect(data.suggestions[0].relevance_score).toBeGreaterThan(0);
+    if (data.suggestions.length >= 2) {
+      expect(data.suggestions[0].relevance_score).toBeGreaterThanOrEqual(data.suggestions[1].relevance_score);
+    }
+    expect(data.suggestions[0].anchor_text).toBeDefined();
+    expect(data.suggestions[0].score_breakdown).toBeDefined();
+
+    const logs = getAuditLogs();
+    const entry = logs.find(l => l.tool === 'wp_suggest_internal_links');
+    expect(entry).toBeDefined();
+    expect(entry.action).toBe('suggest_links');
+  });
+
+  it('exclude_already_linked filters correctly', async () => {
+    _testSetTarget('test', { url: 'https://test.example.com', username: 'u', password: 'p' });
+
+    const candidateLinked = {
+      id: 99,
+      title: { rendered: 'Already Linked Post' },
+      content: { rendered: '' },
+      status: 'publish',
+      categories: [5],
+      date: new Date().toISOString(),
+      link: 'https://test.example.com/existing-link',
+      meta: {}
+    };
+
+    mockSuccess(currentPost);                  // GET /posts/1
+    mockSuccess([candidateLinked, candidate2]); // search results
+
+    const res = await call('wp_suggest_internal_links', { post_id: 1, exclude_already_linked: true });
+    const data = parseResult(res);
+
+    expect(data.excluded_already_linked).toBeGreaterThanOrEqual(1);
+    const ids = data.suggestions.map(s => s.target_post_id);
+    expect(ids).not.toContain(99);
+  });
+});