chore: linting, remove unused vars, and remove project-related parameters from updateAgentHandler

This commit is contained in:
Danny Avila 2025-06-19 21:44:42 -04:00
parent e07067c86d
commit 5512c55d71
No known key found for this signature in database
GPG key ID: BF31EEB2C5CA0956

View file

@ -22,12 +22,12 @@ const {
hasPublicPermission, hasPublicPermission,
} = require('~/server/services/PermissionService'); } = require('~/server/services/PermissionService');
const { getStrategyFunctions } = require('~/server/services/Files/strategies'); const { getStrategyFunctions } = require('~/server/services/Files/strategies');
const { updateAgentProjects, revertAgentVersion } = require('~/models/Agent');
const { resizeAvatar } = require('~/server/services/Files/images/avatar'); const { resizeAvatar } = require('~/server/services/Files/images/avatar');
const { refreshS3Url } = require('~/server/services/Files/S3/crud'); const { refreshS3Url } = require('~/server/services/Files/S3/crud');
const { filterFile } = require('~/server/services/Files/process'); const { filterFile } = require('~/server/services/Files/process');
const { updateAction, getActions } = require('~/models/Action'); const { updateAction, getActions } = require('~/models/Action');
const { getCachedTools } = require('~/server/services/Config'); const { getCachedTools } = require('~/server/services/Config');
const { revertAgentVersion } = require('~/models/Agent');
const { deleteFileByFilter } = require('~/models/File'); const { deleteFileByFilter } = require('~/models/File');
const systemTools = { const systemTools = {
@ -193,7 +193,7 @@ const getAgentHandler = async (req, res, expandProperties = false) => {
const updateAgentHandler = async (req, res) => { const updateAgentHandler = async (req, res) => {
try { try {
const id = req.params.id; const id = req.params.id;
const { projectIds, removeProjectIds, _id, ...updateData } = req.body; const { _id, ...updateData } = req.body;
const existingAgent = await getAgent({ id }); const existingAgent = await getAgent({ id });
if (!existingAgent) { if (!existingAgent) {
@ -204,19 +204,9 @@ const updateAgentHandler = async (req, res) => {
Object.keys(updateData).length > 0 Object.keys(updateData).length > 0
? await updateAgent({ id }, updateData, { ? await updateAgent({ id }, updateData, {
updatingUserId: req.user.id, updatingUserId: req.user.id,
skipVersioning: isProjectUpdate,
}) })
: existingAgent; : existingAgent;
if (isProjectUpdate) {
updatedAgent = await updateAgentProjects({
user: req.user,
agentId: id,
projectIds,
removeProjectIds,
});
}
if (updatedAgent.author) { if (updatedAgent.author) {
updatedAgent.author = updatedAgent.author.toString(); updatedAgent.author = updatedAgent.author.toString();
} }
@ -417,7 +407,7 @@ const getListAgentsHandler = async (req, res) => {
}); });
if (data?.data?.length) { if (data?.data?.length) {
data.data = data.data.map((agent) => { data.data = data.data.map((agent) => {
if (publiclyAccessibleIds.some(id => id.equals(agent._id))) { if (publiclyAccessibleIds.some((id) => id.equals(agent._id))) {
agent.isPublic = true; agent.isPublic = true;
} }
return agent; return agent;