@elisra-devops/docgen-data-provider 1.129.0 → 1.131.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elisra-devops/docgen-data-provider",
-  "version": "1.129.0",
+  "version": "1.131.0",
   "description": "A document generator data provider, aimed to retrive data from azure devops",
   "repository": {
     "type": "git",

package/src/modules/ResultDataProvider.ts

@@ -4413,9 +4413,6 @@ export default class ResultDataProvider {
         const relatedUrl = relation.url;
         try {
           const wi = await TFSServices.getItemContent(relatedUrl, this.token);
-          if (wi.fields['System.State'] === 'Closed') {
-            continue;
-          }
           if (
             selectedLinkedFieldSet.has('associatedRequirement') &&
             wi.fields['System.WorkItemType'] === 'Requirement'

package/src/modules/TicketsDataProvider.ts

@@ -514,6 +514,37 @@ export default class TicketsDataProvider {
     }
   }
 
+  /**
+   * Returns the project's work item type names and the complete set of valid state names.
+   * Used to validate SVD work-item filter options before generation so callers receive
+   * a descriptive error instead of a silent zero-result document.
+   *
+   * Types come from GET _apis/wit/workitemtypes (value[].name).
+   * States are extracted from the transitions map keys on the same response — each key
+   * is a state name — avoiding N extra per-type state calls.
+   *
+   * @param project - The project name.
+   * @returns An object with `types` (work item type names) and `states` (all state names across types).
+   */
+  async GetWorkItemTypeStates(project: string): Promise<{ types: string[]; states: string[] }> {
+    try {
+      const url = `${this.orgUrl}${project}/_apis/wit/workitemtypes`;
+      const { value: workItemTypes } = await TFSServices.getItemContent(url, this.token);
+      const types: string[] = (workItemTypes ?? []).map((t: any) => t?.name).filter(Boolean);
+      // transitions is a map whose keys are state names (e.g. { "Active": [...], "Closed": [...] })
+      const stateSet = new Set<string>();
+      for (const t of workItemTypes ?? []) {
+        for (const stateName of Object.keys(t?.transitions ?? {})) {
+          if (stateName) stateSet.add(stateName);
+        }
+      }
+      return { types, states: Array.from(stateSet) };
+    } catch (err: any) {
+      logger.error(`Error fetching work item type states: ${err.message}`);
+      throw err;
+    }
+  }
+
   /**
    * fetches linked queries
    * @param queries fetched queries

package/src/tests/modules/ResultDataProvider.test.ts

@@ -5127,7 +5127,7 @@ describe('ResultDataProvider', () => {
   });
 
   describe('appendLinkedRelations', () => {
-    it('should append requirement/bug/cr when enabled and skip closed items', async () => {
+    it('should append requirement/bug/cr when enabled, including closed items', async () => {
       const relations = [
         { rel: 'System.LinkTypes.Related', url: 'https://example.com/wi/1' },
         { rel: 'System.LinkTypes.Related', url: 'https://example.com/wi/2' },
@@ -5191,7 +5191,7 @@ describe('ResultDataProvider', () => {
       expect(relatedRequirements[0]).toEqual(
         expect.objectContaining({ id: 1, customerId: 'CUST-1', workItemType: 'Requirement' })
       );
-      expect(relatedBugs).toHaveLength(1);
+      expect(relatedBugs).toHaveLength(2);
       expect(relatedCRs).toHaveLength(1);
     });