Reviewing files that changed from the base of the PR and between 78ebe2f and 3c3509e.

• components/jira/actions/search-issues-with-jql/search-issues-with-jql.mjs (4 hunks)
• components/jira/package.json (1 hunks)

• GitHub Check: Publish TypeScript components
• GitHub Check: Verify TypeScript components
• GitHub Check: Lint Code Base

components/jira/package.json (1)

3-3: Version bump looks good.

Package version aligns with the action’s 0.1.0 change. No issues.

components/jira/actions/search-issues-with-jql/search-issues-with-jql.mjs (3)

7-7: Action version bump is appropriate.

Breaking return shape justifies 0.1.0. Good.

---

159-163: Return shape change is fine; update docs and examples.

Since you now return { issues } and set $summary, ensure README/action description and any example workflows are updated accordingly.

---

93-140: Unit/integration coverage for pagination.

Add tests for: exact page boundary, partial last page, maxResults < page size, empty results.

Reviewing files that changed from the base of the PR and between 3c3509e and 344369c.

• components/jira/actions/search-issues-with-jql/search-issues-with-jql.mjs (4 hunks)

• GitHub Check: Verify TypeScript components
• GitHub Check: Publish TypeScript components
• GitHub Check: Lint Code Base
• GitHub Check: pnpm publish

components/jira/actions/search-issues-with-jql/search-issues-with-jql.mjs (2)

7-7: Output shape change: verify downstream usage and versioning.

Returning { issues } (vs a raw response) can break consumers. Ensure callers and docs are updated; version bump to 0.1.0 is appropriate if this is breaking.

If you need to preserve compatibility, consider returning both:

-      return {
-        issues,
-      };
+      return {
+        issues,
+        // total, startAt could be exposed in the future if needed
+      };

Also applies to: 159-162

---

97-139: Fix pagination: Jira JQL search uses startAt/maxResults/total, not nextPageToken/isLast.

Current loop will only fetch the first page. Replace cursor-style pagination with offset pagination.

Apply this diff:

   methods: {
     getMaxResultsPerPage() {
       return Math.min(this.maxResults, 1000);
     },
     async paginate({
-      params, maxResults, ...otherArgs
+      params, maxResults, ...otherArgs
     }) {
-      const results = [];
-      let nextPageToken;
-      let response;
-
-      do {
-        response = await this.jira.searchIssues({
-          ...otherArgs,
-          params: {
-            ...params,
-            ...(nextPageToken && {
-              nextPageToken,
-            }),
-          },
-        });
-
-        const pageResults = response.issues;
-        const pageLength = pageResults?.length;
-        if (!pageLength) {
-          break;
-        }
-
-        // If maxResults is specified, only add what we need
-        if (maxResults && results.length + pageLength > maxResults) {
-          const remainingSlots = maxResults - results.length;
-          results.push(...pageResults.slice(0, remainingSlots));
-          break;
-        } else {
-          results.push(...pageResults);
-        }
-
-        // Also break if we've reached maxResults exactly
-        if (maxResults && results.length >= maxResults) {
-          break;
-        }
-
-        nextPageToken = response.nextPageToken;
-      } while (nextPageToken && !response.isLast);
-
-      return results;
+      const results = [];
+      let startAt = Number(params?.startAt || 0);
+      const perPage = Number(params?.maxResults) || this.getMaxResultsPerPage();
+      const pageSize = Math.min(perPage, 1000);
+
+      for (;;) {
+        const { issues: page = [], total = 0 } = await this.jira.searchIssues({
+          ...otherArgs,
+          params: {
+            ...params,
+            startAt,
+            maxResults: pageSize,
+          },
+        });
+
+        const pageLen = page.length;
+        if (pageLen === 0) break;
+
+        if (maxResults && results.length + pageLen > maxResults) {
+          results.push(...page.slice(0, maxResults - results.length));
+          break;
+        }
+        results.push(...page);
+
+        if ((maxResults && results.length >= maxResults) ||
+            startAt + pageLen >= total) {
+          break;
+        }
+        startAt += pageLen;
+      }
+      return results;
     },
   },