refactor(mcp): pass inject response through for team-data 404 handling

Author: andypalmi

forge/ee/lib/mcp/tools/teamData.js

@@ -15,26 +15,6 @@ function appendPaginationQuery (url, args) {
     return params.length > 0 ? `${url}?${params.join('&')}` : url
 }
 
-// The FF Tables feature-gate preHandler returns this exact message when the
-// team's plan does not include tables, independent of the PAT's permissions.
-const TABLES_NOT_ENABLED_MESSAGE = 'Not Found - not available on team'
-
-function tablesFeatureGateError (response) {
-    if (response.statusCode === 404) {
-        const body = response.json()
-        if (body && body.error === TABLES_NOT_ENABLED_MESSAGE) {
-            return {
-                statusCode: 404,
-                json: () => ({
-                    code: 'feature_not_enabled',
-                    error: 'FlowFuse Tables is not enabled for this team. This is a plan-level feature and is unrelated to the PAT permissions used to call this tool.'
-                })
-            }
-        }
-    }
-    return null
-}
-
 // Strips the credentials object (which includes a required password field)
 // from a DatabaseCredentials view before it is returned to the tool caller.
 function redactDatabaseCredentials (database) {
@@ -59,10 +39,6 @@ module.exports = [
         },
         handler: async (args, { inject }) => {
             const response = await inject({ method: 'GET', url: `/api/v1/teams/${args.teamId}/databases` })
-            const gateError = tablesFeatureGateError(response)
-            if (gateError) {
-                return gateError
-            }
             if (response.statusCode >= 400) {
                 return response
             }
@@ -87,10 +63,6 @@ module.exports = [
         },
         handler: async (args, { inject }) => {
             const response = await inject({ method: 'GET', url: `/api/v1/teams/${args.teamId}/databases/${args.databaseId}` })
-            const gateError = tablesFeatureGateError(response)
-            if (gateError) {
-                return gateError
-            }
             if (response.statusCode >= 400) {
                 return response
             }
@@ -117,10 +89,6 @@ module.exports = [
         handler: async (args, { inject }) => {
             const url = appendPaginationQuery(`/api/v1/teams/${args.teamId}/databases/${args.databaseId}/tables`, args)
             const response = await inject({ method: 'GET', url })
-            const gateError = tablesFeatureGateError(response)
-            if (gateError) {
-                return gateError
-            }
             return response
         }
     },
@@ -139,10 +107,6 @@ module.exports = [
         },
         handler: async (args, { inject }) => {
             const response = await inject({ method: 'GET', url: `/api/v1/teams/${args.teamId}/databases/${args.databaseId}/tables/${args.tableName}` })
-            const gateError = tablesFeatureGateError(response)
-            if (gateError) {
-                return gateError
-            }
             return response
         }
     },
@@ -163,10 +127,6 @@ module.exports = [
         handler: async (args, { inject }) => {
             const url = appendPaginationQuery(`/api/v1/teams/${args.teamId}/databases/${args.databaseId}/tables/${args.tableName}/data`, args)
             const response = await inject({ method: 'GET', url })
-            const gateError = tablesFeatureGateError(response)
-            if (gateError) {
-                return gateError
-            }
             return response
         }
     },
@@ -182,15 +142,6 @@ module.exports = [
         },
         handler: async (args, { inject }) => {
             const response = await inject({ method: 'GET', url: `/api/v1/teams/${args.teamId}/npm/packages` })
-            if (response.statusCode === 404) {
-                return {
-                    statusCode: 404,
-                    json: () => ({
-                        code: 'feature_not_enabled',
-                        error: 'NPM packages are not available for this team. Either the npm packages feature is not enabled for the team\'s plan, or the team does not exist.'
-                    })
-                }
-            }
             return response
         }
     },
@@ -206,15 +157,6 @@ module.exports = [
         },
         handler: async (args, { inject }) => {
             const response = await inject({ method: 'GET', url: `/api/v1/teams/${args.teamId}/git/tokens` })
-            if (response.statusCode === 404) {
-                return {
-                    statusCode: 404,
-                    json: () => ({
-                        code: 'feature_not_enabled',
-                        error: 'Git integration is not available for this team. Either the git integration feature is not enabled for the team\'s plan, or the team does not exist.'
-                    })
-                }
-            }
             return response
         }
     },
@@ -237,15 +179,6 @@ module.exports = [
                 url += `?type=${args.type}`
             }
             const response = await inject({ method: 'GET', url })
-            if (response.statusCode === 404) {
-                return {
-                    statusCode: 404,
-                    json: () => ({
-                        code: 'feature_not_enabled',
-                        error: 'The shared library is not available for this team. Either the shared library feature is disabled for the team\'s plan, the team does not exist, or you are not a member of the team.'
-                    })
-                }
-            }
             return response
         }
     }