n8n-nodes-azuredevops-advanced 0.1.2 → 0.3.0

package/credentials/AzureDevOpsAdvancedApi.credentials.ts

@@ -3,7 +3,7 @@ import { ICredentialType, INodeProperties } from 'n8n-workflow';
 export class AzureDevOpsAdvancedApi implements ICredentialType {
     name = 'azureDevOpsAdvancedApi';
     displayName = 'Azure DevOps Advanced API';
-    // Belgelendirme linkine doğrudan PAT nasıl alınır yönlendirilebilir
+    // Documentation link explaining how to obtain a Personal Access Token
     documentationUrl = 'https://n8n.io/integrations/azure-devops';
     properties: INodeProperties[] = [
         {
@@ -21,7 +21,7 @@ export class AzureDevOpsAdvancedApi implements ICredentialType {
             typeOptions: { password: true },
             default: '',
             required: true,
-            description: 'Tüm repolara Read/Write yetkilendirmesi olan Azure DevOps PAT anahtarınız',
+            description: 'Your Azure DevOps Personal Access Token with Read/Write access to all required resources',
         },
     ];
 }

package/dist/credentials/AzureDevOpsAdvancedApi.credentials.js

@@ -5,7 +5,7 @@ class AzureDevOpsAdvancedApi {
     constructor() {
         this.name = 'azureDevOpsAdvancedApi';
         this.displayName = 'Azure DevOps Advanced API';
-        // Belgelendirme linkine doğrudan PAT nasıl alınır yönlendirilebilir
+        // Documentation link explaining how to obtain a Personal Access Token
         this.documentationUrl = 'https://n8n.io/integrations/azure-devops';
         this.properties = [
             {
@@ -23,7 +23,7 @@ class AzureDevOpsAdvancedApi {
                 typeOptions: { password: true },
                 default: '',
                 required: true,
-                description: 'Tüm repolara Read/Write yetkilendirmesi olan Azure DevOps PAT anahtarınız',
+                description: 'Your Azure DevOps Personal Access Token with Read/Write access to all required resources',
             },
         ];
     }

package/dist/nodes/AzureDevOpsAdvanced/AzureDevOpsAdvanced.node.js

@@ -21,7 +21,7 @@ class AzureDevOpsAdvanced {
             group: ['transform'],
             version: 1,
             subtitle: '={{$parameter["operation"] + ": " + $parameter["resource"]}}',
-            description: 'Kapsamlı yerel Azure DevOps eklentisi',
+            description: 'Comprehensive Azure DevOps integration node',
             defaults: { name: 'Azure DevOps Advanced' },
             inputs: ['main'],
             outputs: ['main'],
@@ -70,7 +70,7 @@ class AzureDevOpsAdvanced {
         };
     }
     async execute() {
-        var _a, _b, _c, _d;
+        var _a, _b, _c, _d, _e, _f;
         const items = this.getInputData();
         const returnData = [];
         const resource = this.getNodeParameter('resource', 0);
@@ -95,25 +95,45 @@ class AzureDevOpsAdvanced {
                     else if (operation === 'createBranch') {
                         const repoId = this.getNodeParameter('repositoryId', i);
                         const branchName = this.getNodeParameter('branchName', i);
-                        // To create a branch, we need the oldObjectID from the repo (usually main/master)
-                        // Simplified implementation for the boilerplate. Requires fetching refs first.
-                        const endpoint = `${project}/_apis/git/repositories/${repoId}/refs?filter=heads/main&api-version=7.1`;
-                        const refResponse = await GenericFunctions_1.azureApiRequest.call(this, 'GET', endpoint);
-                        const oldObjectId = ((_b = (_a = refResponse === null || refResponse === void 0 ? void 0 : refResponse.value) === null || _a === void 0 ? void 0 : _a[0]) === null || _b === void 0 ? void 0 : _b.objectId) || "0000000000000000000000000000000000000000";
+                        // Try main first, fall back to master
+                        let oldObjectId = '0000000000000000000000000000000000000000';
+                        for (const baseBranch of ['main', 'master']) {
+                            const refResponse = await GenericFunctions_1.azureApiRequest.call(this, 'GET', `${project}/_apis/git/repositories/${repoId}/refs?filter=heads/${baseBranch}&api-version=7.1`);
+                            if ((_b = (_a = refResponse === null || refResponse === void 0 ? void 0 : refResponse.value) === null || _a === void 0 ? void 0 : _a[0]) === null || _b === void 0 ? void 0 : _b.objectId) {
+                                oldObjectId = refResponse.value[0].objectId;
+                                break;
+                            }
+                        }
                         const createEndpoint = `${project}/_apis/git/repositories/${repoId}/refs?api-version=7.1`;
-                        const body = [{ name: branchName, newObjectId: oldObjectId, oldObjectId: "0000000000000000000000000000000000000000" }];
+                        const body = [{ name: branchName, newObjectId: oldObjectId, oldObjectId: '0000000000000000000000000000000000000000' }];
                         responseData = await GenericFunctions_1.azureApiRequest.call(this, 'POST', createEndpoint, body);
+                        responseData = (responseData === null || responseData === void 0 ? void 0 : responseData.value) || responseData;
                     }
                     else if (operation === 'pushCommit') {
                         const repoId = this.getNodeParameter('repositoryId', i);
                         const branchName = this.getNodeParameter('branchName', i);
                         const commitMessage = this.getNodeParameter('commitMessage', i);
-                        // Requires complex PUSH structure with base64 encoded strings for files.
-                        // Simplified placeholder structure
+                        const pushFilePath = this.getNodeParameter('pushFilePath', i);
+                        const fileContent = this.getNodeParameter('fileContent', i);
+                        const changeType = this.getNodeParameter('changeType', i);
+                        // Get current branch HEAD to use as oldObjectId
+                        const branchFilter = branchName.replace('refs/heads/', '');
+                        const refResponse = await GenericFunctions_1.azureApiRequest.call(this, 'GET', `${project}/_apis/git/repositories/${repoId}/refs?filter=heads/${branchFilter}&api-version=7.1`);
+                        const oldObjectId = ((_d = (_c = refResponse === null || refResponse === void 0 ? void 0 : refResponse.value) === null || _c === void 0 ? void 0 : _c[0]) === null || _d === void 0 ? void 0 : _d.objectId) || '0000000000000000000000000000000000000000';
                         const endpoint = `${project}/_apis/git/repositories/${repoId}/pushes?api-version=7.1`;
                         const body = {
-                            refUpdates: [{ name: branchName, oldObjectId: "0000000000000000000000000000000000000000" }],
-                            commits: [{ comment: commitMessage, changes: [] }]
+                            refUpdates: [{ name: branchName, oldObjectId }],
+                            commits: [{
+                                    comment: commitMessage,
+                                    changes: [{
+                                            changeType,
+                                            item: { path: pushFilePath.startsWith('/') ? pushFilePath : `/${pushFilePath}` },
+                                            newContent: {
+                                                content: Buffer.from(fileContent).toString('base64'),
+                                                contentType: 'base64Encoded',
+                                            },
+                                        }],
+                                }],
                         };
                         responseData = await GenericFunctions_1.azureApiRequest.call(this, 'POST', endpoint, body);
                     }
@@ -169,7 +189,7 @@ class AzureDevOpsAdvanced {
                         const endpoint = `${project}/_apis/wit/wiql?api-version=7.1`;
                         const wiqlQuery = { query: `Select [System.Id], [System.Title], [System.State] From WorkItems Where [System.TeamProject] = '${project}'` };
                         const idsResponse = await GenericFunctions_1.azureApiRequest.call(this, 'POST', endpoint, wiqlQuery);
-                        const idsArray = (_c = idsResponse === null || idsResponse === void 0 ? void 0 : idsResponse.workItems) === null || _c === void 0 ? void 0 : _c.map((wi) => wi.id);
+                        const idsArray = (_e = idsResponse === null || idsResponse === void 0 ? void 0 : idsResponse.workItems) === null || _e === void 0 ? void 0 : _e.map((wi) => wi.id);
                         if (idsArray && idsArray.length > 0) {
                             const workItemIds = idsArray.join(',');
                             const bulkEndpoint = `${project}/_apis/wit/workitems?ids=${workItemIds}&api-version=7.1`;
@@ -195,14 +215,14 @@ class AzureDevOpsAdvanced {
                         if (!isUpdate) {
                             mapPropertyToPatch('System.Title', this.getNodeParameter('title', i));
                         }
-                        const additionalFields = this.getNodeParameter('additionalFields', i);
-                        if (Object.keys(additionalFields).length) {
+                        const additionalFields = this.getNodeParameter('additionalFields', i, {});
+                        if (additionalFields && Object.keys(additionalFields).length) {
                             mapPropertyToPatch('System.Description', additionalFields.description);
                             mapPropertyToPatch('System.AssignedTo', additionalFields.assignedTo);
                             mapPropertyToPatch('System.State', additionalFields.state);
                             mapPropertyToPatch('Microsoft.VSTS.Common.Priority', additionalFields.priority);
                             mapPropertyToPatch('System.Tags', additionalFields.tags);
-                            if ((_d = additionalFields.customFieldsUi) === null || _d === void 0 ? void 0 : _d.customFieldsValues) {
+                            if ((_f = additionalFields.customFieldsUi) === null || _f === void 0 ? void 0 : _f.customFieldsValues) {
                                 for (const customField of additionalFields.customFieldsUi.customFieldsValues) {
                                     mapPropertyToPatch(customField.fieldId, customField.fieldValue);
                                 }
@@ -223,7 +243,7 @@ class AzureDevOpsAdvanced {
                         });
                     }
                     else if (operation === 'listUsers') {
-                        // Azure Graph API / Core API üzerinden user listesi çekilir
+                        // Users are fetched via the Azure Graph API (vssps endpoint)
                         const endpoint = `_apis/graph/users?api-version=7.1-preview.1`;
                         const credentials = await this.getCredentials('azureDevOpsAdvancedApi');
                         const baseUrl = `https://vssps.dev.azure.com/${credentials.organization}`;
@@ -240,7 +260,7 @@ class AzureDevOpsAdvanced {
                         responseData = (responseData === null || responseData === void 0 ? void 0 : responseData.value) || responseData;
                     }
                     else if (operation === 'listTags') {
-                        // Project bazlı tag listesi
+                        // Retrieve all work item tags scoped to the project
                         const endpoint = `${project}/_apis/wit/tags?api-version=7.1-preview.1`;
                         responseData = await GenericFunctions_1.azureApiRequest.call(this, 'GET', endpoint);
                         responseData = (responseData === null || responseData === void 0 ? void 0 : responseData.value) || responseData;
@@ -256,7 +276,7 @@ class AzureDevOpsAdvanced {
                     }
                     else if (operation === 'getComments') {
                         const pullRequestId = this.getNodeParameter('pullRequestId', i);
-                        const endpoint = `${project}/_apis/git/repositories/${repositoryId}/pullRequests/${pullRequestId}/threads?api-version=7.1`;
+                        const endpoint = `${project}/_apis/git/repositories/${repositoryId}/pullrequests/${pullRequestId}/threads?api-version=7.1`;
                         responseData = await GenericFunctions_1.azureApiRequest.call(this, 'GET', endpoint);
                         responseData = (responseData === null || responseData === void 0 ? void 0 : responseData.value) || responseData;
                     }
@@ -264,12 +284,9 @@ class AzureDevOpsAdvanced {
                         const listOptions = this.getNodeParameter('listOptions', i);
                         let endpoint = `${project}/_apis/git/repositories/${repositoryId}/pullrequests?api-version=7.1`;
                         if (listOptions) {
-                            if (listOptions.status && listOptions.status !== 'all') {
+                            if (listOptions.status) {
                                 endpoint += `&searchCriteria.status=${listOptions.status}`;
                             }
-                            else if (listOptions.status === 'all') {
-                                endpoint += `&searchCriteria.status=all`;
-                            }
                             if (listOptions.creatorId)
                                 endpoint += `&searchCriteria.creatorId=${listOptions.creatorId}`;
                             if (listOptions.reviewerId)
@@ -505,7 +522,7 @@ class AzureDevOpsAdvanced {
                         });
                     }
                 }
-                // Sonuçların işlenmesi
+                // Process and return results
                 if (Array.isArray(responseData)) {
                     returnData.push.apply(returnData, responseData.map(item => ({ json: item })));
                 }

package/dist/nodes/AzureDevOpsAdvanced/GenericFunctions.js

@@ -19,7 +19,7 @@ async function azureApiRequest(method, endpoint, body = {}, query = {}) {
         },
         json: true,
     };
-    if (Object.keys(body).length === 0) {
+    if (!Array.isArray(body) && Object.keys(body).length === 0) {
         delete options.body;
     }
     return this.helpers.request(options);

package/dist/nodes/AzureDevOpsAdvanced/descriptions/GitDescription.js

@@ -90,6 +90,36 @@ exports.gitFields = [
             },
         },
     },
+    {
+        displayName: 'File Path',
+        name: 'pushFilePath',
+        type: 'string',
+        default: '',
+        required: true,
+        displayOptions: {
+            show: {
+                resource: ['git'],
+                operation: ['pushCommit'],
+            },
+        },
+        description: 'Path of the file to create or update (e.g. /src/app.ts)',
+    },
+    {
+        displayName: 'Change Type',
+        name: 'changeType',
+        type: 'options',
+        options: [
+            { name: 'Edit (update existing file)', value: 'edit' },
+            { name: 'Add (create new file)', value: 'add' },
+        ],
+        default: 'edit',
+        displayOptions: {
+            show: {
+                resource: ['git'],
+                operation: ['pushCommit'],
+            },
+        },
+    },
     {
         displayName: 'File Content',
         name: 'fileContent',

package/dist/nodes/AzureDevOpsAdvanced/descriptions/PipelineDescription.js

@@ -41,7 +41,7 @@ exports.pipelineFields = [
         displayOptions: {
             show: {
                 resource: ['pipeline'],
-                operation: ['run', 'getLogs'],
+                operation: ['run', 'getLogs', 'cancelRun'],
             },
         },
         default: 0,

package/nodes/AzureDevOpsAdvanced/AzureDevOpsAdvanced.node.ts

@@ -19,7 +19,7 @@ export class AzureDevOpsAdvanced implements INodeType {
         group: ['transform'],
         version: 1,
         subtitle: '={{$parameter["operation"] + ": " + $parameter["resource"]}}',
-        description: 'Kapsamlı yerel Azure DevOps eklentisi',
+        description: 'Comprehensive Azure DevOps integration node',
         defaults: { name: 'Azure DevOps Advanced' },
         inputs: ['main'],
         outputs: ['main'],
@@ -94,26 +94,49 @@ export class AzureDevOpsAdvanced implements INodeType {
                     else if (operation === 'createBranch') {
                         const repoId = this.getNodeParameter('repositoryId', i) as string;
                         const branchName = this.getNodeParameter('branchName', i) as string;
-                        // To create a branch, we need the oldObjectID from the repo (usually main/master)
-                        // Simplified implementation for the boilerplate. Requires fetching refs first.
-                        const endpoint = `${project}/_apis/git/repositories/${repoId}/refs?filter=heads/main&api-version=7.1`;
-                        const refResponse = await azureApiRequest.call(this, 'GET', endpoint);
-                        const oldObjectId = refResponse?.value?.[0]?.objectId || "0000000000000000000000000000000000000000";
+
+                        // Try main first, fall back to master
+                        let oldObjectId = '0000000000000000000000000000000000000000';
+                        for (const baseBranch of ['main', 'master']) {
+                            const refResponse = await azureApiRequest.call(this, 'GET', `${project}/_apis/git/repositories/${repoId}/refs?filter=heads/${baseBranch}&api-version=7.1`);
+                            if (refResponse?.value?.[0]?.objectId) {
+                                oldObjectId = refResponse.value[0].objectId;
+                                break;
+                            }
+                        }
 
                         const createEndpoint = `${project}/_apis/git/repositories/${repoId}/refs?api-version=7.1`;
-                        const body = [{ name: branchName, newObjectId: oldObjectId, oldObjectId: "0000000000000000000000000000000000000000" }];
+                        const body = [{ name: branchName, newObjectId: oldObjectId, oldObjectId: '0000000000000000000000000000000000000000' }];
                         responseData = await azureApiRequest.call(this, 'POST', createEndpoint, body);
+                        responseData = responseData?.value || responseData;
                     }
                     else if (operation === 'pushCommit') {
                         const repoId = this.getNodeParameter('repositoryId', i) as string;
                         const branchName = this.getNodeParameter('branchName', i) as string;
                         const commitMessage = this.getNodeParameter('commitMessage', i) as string;
-                        // Requires complex PUSH structure with base64 encoded strings for files.
-                        // Simplified placeholder structure
+                        const pushFilePath = this.getNodeParameter('pushFilePath', i) as string;
+                        const fileContent = this.getNodeParameter('fileContent', i) as string;
+                        const changeType = this.getNodeParameter('changeType', i) as string;
+
+                        // Get current branch HEAD to use as oldObjectId
+                        const branchFilter = branchName.replace('refs/heads/', '');
+                        const refResponse = await azureApiRequest.call(this, 'GET', `${project}/_apis/git/repositories/${repoId}/refs?filter=heads/${branchFilter}&api-version=7.1`);
+                        const oldObjectId = refResponse?.value?.[0]?.objectId || '0000000000000000000000000000000000000000';
+
                         const endpoint = `${project}/_apis/git/repositories/${repoId}/pushes?api-version=7.1`;
                         const body = {
-                            refUpdates: [{ name: branchName, oldObjectId: "0000000000000000000000000000000000000000" }],
-                            commits: [{ comment: commitMessage, changes: [] }]
+                            refUpdates: [{ name: branchName, oldObjectId }],
+                            commits: [{
+                                comment: commitMessage,
+                                changes: [{
+                                    changeType,
+                                    item: { path: pushFilePath.startsWith('/') ? pushFilePath : `/${pushFilePath}` },
+                                    newContent: {
+                                        content: Buffer.from(fileContent).toString('base64'),
+                                        contentType: 'base64Encoded',
+                                    },
+                                }],
+                            }],
                         };
                         responseData = await azureApiRequest.call(this, 'POST', endpoint, body);
                     }
@@ -203,8 +226,8 @@ export class AzureDevOpsAdvanced implements INodeType {
                             mapPropertyToPatch('System.Title', this.getNodeParameter('title', i));
                         }
 
-                        const additionalFields = this.getNodeParameter('additionalFields', i) as any;
-                        if (Object.keys(additionalFields).length) {
+                        const additionalFields = this.getNodeParameter('additionalFields', i, {}) as any;
+                        if (additionalFields && Object.keys(additionalFields).length) {
                             mapPropertyToPatch('System.Description', additionalFields.description);
                             mapPropertyToPatch('System.AssignedTo', additionalFields.assignedTo);
                             mapPropertyToPatch('System.State', additionalFields.state);
@@ -234,7 +257,7 @@ export class AzureDevOpsAdvanced implements INodeType {
                         });
                     }
                     else if (operation === 'listUsers') {
-                        // Azure Graph API / Core API üzerinden user listesi çekilir
+                        // Users are fetched via the Azure Graph API (vssps endpoint)
                         const endpoint = `_apis/graph/users?api-version=7.1-preview.1`;
                         const credentials = await this.getCredentials('azureDevOpsAdvancedApi');
                         const baseUrl = `https://vssps.dev.azure.com/${(credentials as any).organization}`;
@@ -252,7 +275,7 @@ export class AzureDevOpsAdvanced implements INodeType {
                         responseData = responseData?.value || responseData;
                     }
                     else if (operation === 'listTags') {
-                        // Project bazlı tag listesi
+                        // Retrieve all work item tags scoped to the project
                         const endpoint = `${project}/_apis/wit/tags?api-version=7.1-preview.1`;
                         responseData = await azureApiRequest.call(this, 'GET', endpoint);
                         responseData = responseData?.value || responseData;
@@ -270,7 +293,7 @@ export class AzureDevOpsAdvanced implements INodeType {
                     }
                     else if (operation === 'getComments') {
                         const pullRequestId = this.getNodeParameter('pullRequestId', i) as number;
-                        const endpoint = `${project}/_apis/git/repositories/${repositoryId}/pullRequests/${pullRequestId}/threads?api-version=7.1`;
+                        const endpoint = `${project}/_apis/git/repositories/${repositoryId}/pullrequests/${pullRequestId}/threads?api-version=7.1`;
                         responseData = await azureApiRequest.call(this, 'GET', endpoint);
                         responseData = responseData?.value || responseData;
                     }
@@ -279,10 +302,8 @@ export class AzureDevOpsAdvanced implements INodeType {
                         let endpoint = `${project}/_apis/git/repositories/${repositoryId}/pullrequests?api-version=7.1`;
 
                         if (listOptions) {
-                            if (listOptions.status && listOptions.status !== 'all') {
+                            if (listOptions.status) {
                                 endpoint += `&searchCriteria.status=${listOptions.status}`;
-                            } else if (listOptions.status === 'all') {
-                                endpoint += `&searchCriteria.status=all`;
                             }
                             if (listOptions.creatorId) endpoint += `&searchCriteria.creatorId=${listOptions.creatorId}`;
                             if (listOptions.reviewerId) endpoint += `&searchCriteria.reviewerId=${listOptions.reviewerId}`;
@@ -519,7 +540,7 @@ export class AzureDevOpsAdvanced implements INodeType {
                     }
                 }
 
-                // Sonuçların işlenmesi
+                // Process and return results
                 if (Array.isArray(responseData)) {
                     returnData.push.apply(returnData, responseData.map(item => ({ json: item })));
                 } else {

package/nodes/AzureDevOpsAdvanced/GenericFunctions.ts

@@ -27,7 +27,7 @@ export async function azureApiRequest(
         json: true,
     };
 
-    if (Object.keys(body).length === 0) {
+    if (!Array.isArray(body) && Object.keys(body).length === 0) {
         delete options.body;
     }
 

package/nodes/AzureDevOpsAdvanced/descriptions/GitDescription.ts

@@ -91,6 +91,36 @@ export const gitFields: INodeProperties[] = [
             },
         },
     },
+    {
+        displayName: 'File Path',
+        name: 'pushFilePath',
+        type: 'string',
+        default: '',
+        required: true,
+        displayOptions: {
+            show: {
+                resource: ['git'],
+                operation: ['pushCommit'],
+            },
+        },
+        description: 'Path of the file to create or update (e.g. /src/app.ts)',
+    },
+    {
+        displayName: 'Change Type',
+        name: 'changeType',
+        type: 'options',
+        options: [
+            { name: 'Edit (update existing file)', value: 'edit' },
+            { name: 'Add (create new file)', value: 'add' },
+        ],
+        default: 'edit',
+        displayOptions: {
+            show: {
+                resource: ['git'],
+                operation: ['pushCommit'],
+            },
+        },
+    },
     {
         displayName: 'File Content',
         name: 'fileContent',

package/nodes/AzureDevOpsAdvanced/descriptions/PipelineDescription.ts

@@ -41,7 +41,7 @@ export const pipelineFields: INodeProperties[] = [
         displayOptions: {
             show: {
                 resource: ['pipeline'],
-                operation: ['run', 'getLogs'],
+                operation: ['run', 'getLogs', 'cancelRun'],
             },
         },
         default: 0,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "n8n-nodes-azuredevops-advanced",
-  "version": "0.1.2",
+  "version": "0.3.0",
   "description": "Comprehensive Azure DevOps custom node for n8n",
   "main": "dist/nodes/AzureDevOpsAdvanced/AzureDevOpsAdvanced.node.js",
   "license": "MIT",
@@ -38,14 +38,18 @@
   },
   "scripts": {
     "build": "tsc",
-    "dev": "tsc --watch"
+    "dev": "tsc --watch",
+    "test": "jest --forceExit"
   },
   "dependencies": {
     "n8n-core": "^1.x",
     "n8n-workflow": "^1.x"
   },
   "devDependencies": {
+    "@types/jest": "^30.0.0",
     "@types/node": "^18.x",
+    "jest": "^30.4.2",
+    "ts-jest": "^29.4.11",
     "typescript": "^5.x"
   },
   "publishConfig": {