@jgardner04/ghost-mcp-server 1.12.5 → 1.13.1

package/src/services/ghostService.js

@@ -180,27 +180,30 @@ const createTag = async (tagData) => {
 };
 
 /**
- * Retrieves tags from Ghost, optionally filtering by name.
- * @param {string} [name] - Optional tag name to filter by.
+ * Retrieves tags from Ghost with optional filtering.
+ * @param {object} [options={}] - Query options for filtering and pagination.
+ * @param {number} [options.limit] - Maximum number of tags to return (default: 15).
+ * @param {string} [options.filter] - NQL filter string for advanced filtering.
+ * @param {string} [options.order] - Order string for sorting results.
+ * @param {string} [options.include] - Include string for related data.
  * @returns {Promise<Array<object>>} An array of tag objects.
  */
-const getTags = async (name) => {
-  const options = {
-    limit: 'all', // Get all tags
-  };
-
-  // Safely construct filter to prevent injection
-  if (name) {
-    // Additional validation: only allow alphanumeric, spaces, hyphens, underscores
-    if (!/^[a-zA-Z0-9\s\-_]+$/.test(name)) {
-      throw new Error('Tag name contains invalid characters');
-    }
-    // Escape single quotes and backslashes to prevent injection
-    const safeName = name.replace(/\\/g, '\\\\').replace(/'/g, "\\'");
-    options.filter = `name:'${safeName}'`;
+const getTags = async (options = {}) => {
+  try {
+    const tags = await handleApiRequest(
+      'tags',
+      'browse',
+      {},
+      {
+        limit: 15,
+        ...options,
+      }
+    );
+    return tags || [];
+  } catch (error) {
+    logger.error('Failed to get tags', { error: error.message });
+    throw error;
   }
-
-  return handleApiRequest('tags', 'browse', {}, options);
 };
 
 // Add other content management functions here (createTag, etc.)

package/src/services/ghostServiceImproved.js

@@ -11,9 +11,12 @@ import {
   CircuitBreaker,
   retryWithBackoff,
 } from '../errors/index.js';
+import { createContextLogger } from '../utils/logger.js';
 
 dotenv.config();
 
+const logger = createContextLogger('ghost-service-improved');
+
 const { GHOST_ADMIN_API_URL, GHOST_ADMIN_API_KEY } = process.env;
 
 // Validate configuration at startup
@@ -685,8 +688,8 @@ export async function createTag(tagData) {
     if (error instanceof GhostAPIError && error.ghostStatusCode === 422) {
       // Check if it's a duplicate tag error
       if (error.originalError.includes('already exists')) {
-        // Try to fetch the existing tag
-        const existingTags = await getTags(tagData.name);
+        // Try to fetch the existing tag by name filter
+        const existingTags = await getTags({ filter: `name:'${tagData.name}'` });
         if (existingTags.length > 0) {
           return existingTags[0]; // Return existing tag instead of failing
         }
@@ -699,17 +702,20 @@ export async function createTag(tagData) {
   }
 }
 
-export async function getTags(name) {
-  const options = {
-    limit: 'all',
-    ...(name && { filter: `name:'${name}'` }),
-  };
-
+export async function getTags(options = {}) {
   try {
-    const tags = await handleApiRequest('tags', 'browse', {}, options);
+    const tags = await handleApiRequest(
+      'tags',
+      'browse',
+      {},
+      {
+        limit: 15,
+        ...options,
+      }
+    );
     return tags || [];
   } catch (error) {
-    console.error('Failed to get tags:', error);
+    logger.error('Failed to get tags', { error: error.message });
     throw error;
   }
 }