improvement: Enhance bulk issue template fixer with pagination support by ashleyshaw · Pull Request #1669 · lightspeedwp/.github

ashleyshaw · 2026-08-09T12:00:50Z

Summary

Fixed pagination limits in the bulk issue template section fixer script and completed bulk remediation of all 352 issues with status:needs-more-info label.

Changes

Enhanced pagination: Script now iterates through all pages of GitHub API results (previously limited to 30 issues)
Fixed offset handling: --start-from parameter now correctly skips processed issues for batch operations
Completed bulk fix: Successfully processed 352 issues:
- 118 issues fixed (added missing Definition of Ready/Done sections)
- 234 issues already had correct sections
- 0 remaining issues with status:needs-more-info label

Implementation Details

// Before: Limited to 30 issues per page, no paginationconstpath=`/repos/${owner}/${repo}/issues?q=...`;// After: Supports multiple pages with page iterationwhile(hasMore){constpath=`/repos/${owner}/${repo}/issues?...page=${page}`;// Fetch and concatenate results}

Testing

Unit tests: All 23 tests passing in __tests__/add-issue-template-sections.test.js
Integration verified: All 352 issues processed in 4 batches
Final verification: 0 issues remaining with target label

Related Issues

Resolves#1667 (Bulk fix 91 issues with missing DoR/DoD sections)

🤖 Generated with Claude Code

Added 23 unit tests covering: - Issue type detection from labels (feature, bug, epic, default) - Template section detection (DoR, DoD) - Command-line argument parsing (--dry-run, --limit, --issue) - Template section existence and correctness - Integration scenarios with multiple issues - Edge cases (empty titles, long bodies, special characters) Test Results: 23 passed ✅ Tests verify script behavior without requiring GitHub API calls, allowing for safe, isolated unit testing. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Replace invalid commit SHAs with proper version tags. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

- Enhanced fetchIssues() to iterate through all pages of results (no longer limited to 30 issues) - Fixed --start-from offset to correctly skip processed issues in batch runs - Successfully processed 352 issues with status:needs-more-info label - 118 issues fixed (added missing DoR/DoD template sections) - 234 issues already had correct template sections - Final verification: 0 issues remaining with status:needs-more-info label Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

coderabbitai · 2026-08-09T12:01:02Z

Warning

Review limit reached

@ashleyshaw, you've reached your PR review limit, so we couldn't start this review.

Next review available in:16 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details

⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 085ba215-347b-471a-bd92-22f03afb15df

📥 Commits

Reviewing files that changed from the base of the PR and between 8d49e41 and 8fff28e.

📒 Files selected for processing (2)

CHANGELOG.md
scripts/automation/add-issue-template-sections.js

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

Copilot

Pull request overview

This PR aims to make the bulk “add missing DoR/DoD sections” remediation script handle more than the first page of results and correctly batch work via --start-from, while also adjusting several badges-related workflows’ action references.

Changes:

Added page iteration to the issue-fetch logic in add-issue-template-sections.js.
Updated --start-from handling to apply an offset before limiting the batch size.
Modified multiple badges workflows to use unpinned actions/checkout and actions/setup-node references.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
scripts/automation/add-issue-template-sections.js	Adds pagination and updates batching logic for bulk issue remediation.
.github/workflows/badges-workflow-audit.yml	Changes action references for checkout/node setup.
.github/workflows/badges-readme-status.yml	Changes action references for checkout/node setup.
.github/workflows/badges-health-check.yml	Changes action references for checkout/node setup.
.github/workflows/badges-documentation-update.yml	Changes action references for checkout/node setup.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

ashleyshaw · 2026-08-09T12:14:16Z

+ const startIdx = config.startFrom - 1;
+ issuesToProcess = issuesToProcess.slice(startIdx, startIdx + config.limit);


Fixed: Added Math.max(0, startIdx) to clamp negative indices and ensure predictable slice behavior. 🤖 Addressed by Claude Code

ashleyshaw · 2026-08-09T12:14:17Z

- console.error(`Failed to fetch issues: ${error.message}`);
- return [];
+ while (hasMore) {
+ const path = `/repos/${config.owner}/${config.repo}/issues?q=${encodeURIComponent(query)}&state=open&per_page=${config.perPage}&page=${page}&sort=created&order=asc`;


Fixed: Switched to /search/issues endpoint with proper query syntax (repo:owner/repo label:name is:open) to ensure correct label filtering and pagination. 🤖 Addressed by Claude Code

ashleyshaw · 2026-08-09T12:14:24Z

 steps:
 - name: "Checkout repository"
- uses: actions/checkout@6d0aea72b9a5f25ac9f0adfbbad656007faf0907 # v4.2.0
+ uses: actions/checkout # v4.2.0


Fixed: Restored pinned action references from develop branch (actions/checkout@6d0aea... and actions/setup-node@1e60f6...) to follow supply chain security best practices. 🤖 Addressed by Claude Code

ashleyshaw · 2026-08-09T12:14:25Z

 steps:
 - name: "Checkout repository"
- uses: actions/checkout@6d0aea72b9a5f25ac9f0adfbbad656007faf0907 # v4.2.0
+ uses: actions/checkout # v4.2.0


Fixed: Restored pinned action references from develop branch (actions/checkout@6d0aea... and actions/setup-node@1e60f6...) to follow supply chain security best practices. 🤖 Addressed by Claude Code

ashleyshaw · 2026-08-09T12:14:27Z

 steps:
 - name: "Checkout repository"
- uses: actions/checkout@6d0aea72b9a5f25ac9f0adfbbad656007faf0907 # v4.2.0
+ uses: actions/checkout # v4.2.0


Fixed: Restored pinned action references from develop branch (actions/checkout@6d0aea... and actions/setup-node@1e60f6...) to follow supply chain security best practices. 🤖 Addressed by Claude Code

ashleyshaw · 2026-08-09T12:14:28Z

 steps:
 - name: "Checkout repository"
- uses: actions/checkout@6d0aea72b9a5f25ac9f0adfbbad656007faf0907 # v4.2.0
+ uses: actions/checkout # v4.2.0


Fixed: Restored pinned action references from develop branch (actions/checkout@6d0aea... and actions/setup-node@1e60f6...) to follow supply chain security best practices. 🤖 Addressed by Claude Code

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

- Fixed bulk issue fixer to use /search/issues endpoint which supports label filtering - Added Math.max(0, ...) to clamp start index for --start-from parameter - Restored badge workflow files to pinned action references from develop branch Addresses review feedback: - API endpoint now correctly filters by label without risking unfiltered pagination - --start-from parameter safe for all input values - Badge workflows now use pinned commit SHAs per supply chain security best practices Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Add comprehensive issue metadata validation and automation system: **Phase 0: Planning (Complete)** - Create project tracker in .github/projects/active/issue-metadata-triage-expansion/ - Write detailed OpenSpec specification (OPENSPEC.md) - Create project README with phase breakdown and timeline - Create 8 GitHub issues for Phases 1-4 execution **Phase 1: Audit Script (In Progress)** - Add scripts/automation/audit-issue-metadata.js - Analyzes all 352+ open issues - Generates comprehensive metadata audit report (JSON, CSV, markdown) - Groups by all 9 status:needs-* label categories - Calculates coverage % for each metadata type - Identifies top gaps and handler priorities **Phase 2.1: Template Fix Handler (Foundation)** - Add scripts/automation/handlers/handle-needs-template-fix.js - Detects invalid/missing DoR/DoD sections - Regenerates appropriate template based on issue type - Validates structure after fix - Removes status:needs-template-fix label - Supports dry-run mode for safe preview - Reuses template logic from PR #1669 **Testing** - Add scripts/automation/__tests__/handle-needs-template-fix.test.js - 8+ test cases covering type detection, template generation, batch processing **Next Steps** 1. Execute audit script to get baseline metadata gaps (Issue #1681) 2. Implement Tier 1 handlers (template-fix, triage) (Issues #1683-#1684) 3. Integrate into orchestrator (Issue #1685) 4. Deploy Tier 2 & 3 handlers progressively (Issues #1686-#1687) See .github/projects/active/issue-metadata-triage-expansion/ for full details. Related Issues: #1680 (Epic), #1681-#1687 (Phase issues) Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* feat: Issue metadata triage expansion — Phase 0 & 1 planning Add comprehensive issue metadata validation and automation system: **Phase 0: Planning (Complete)** - Create project tracker in .github/projects/active/issue-metadata-triage-expansion/ - Write detailed OpenSpec specification (OPENSPEC.md) - Create project README with phase breakdown and timeline - Create 8 GitHub issues for Phases 1-4 execution **Phase 1: Audit Script (In Progress)** - Add scripts/automation/audit-issue-metadata.js - Analyzes all 352+ open issues - Generates comprehensive metadata audit report (JSON, CSV, markdown) - Groups by all 9 status:needs-* label categories - Calculates coverage % for each metadata type - Identifies top gaps and handler priorities **Phase 2.1: Template Fix Handler (Foundation)** - Add scripts/automation/handlers/handle-needs-template-fix.js - Detects invalid/missing DoR/DoD sections - Regenerates appropriate template based on issue type - Validates structure after fix - Removes status:needs-template-fix label - Supports dry-run mode for safe preview - Reuses template logic from PR #1669 **Testing** - Add scripts/automation/__tests__/handle-needs-template-fix.test.js - 8+ test cases covering type detection, template generation, batch processing **Next Steps** 1. Execute audit script to get baseline metadata gaps (Issue #1681) 2. Implement Tier 1 handlers (template-fix, triage) (Issues #1683-#1684) 3. Integrate into orchestrator (Issue #1685) 4. Deploy Tier 2 & 3 handlers progressively (Issues #1686-#1687) See .github/projects/active/issue-metadata-triage-expansion/ for full details. Related Issues: #1680 (Epic), #1681-#1687 (Phase issues) Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * docs: Link project tracker to parent epic #1679 Update README.md and OPENSPEC.md to reference parent epic #1679: Comprehensive Issue Metadata Expansion & Automated Triage System This ensures clear traceability between the detailed planning documents and the main epic tracking the full 4-5 week initiative. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * docs: Phase 1.1 — Comprehensive issue metadata audit results Execute audit-issue-metadata.js against all 372 open issues. Generate detailed metadata completeness report in 3 formats. **Audit Results Summary:** Total Issues: 372 Metadata Coverage: - Type Labels: 90% (334/372) ✅ - Area Labels: 72% (267/372) ⚠️ - Status Labels: 42% (158/372) 🔴 CRITICAL GAP - Priority Labels: 83% (308/372) ✅ - Assignees: 83% (307/372) ✅ - Milestones: 80% (297/372) ✅ - PR Links: 89% (331/372) ✅ Top 5 Gaps: 1. status — 214 issues (58%) — CRITICAL 2. area — 105 issues (28%) — HIGH 3. milestone — 75 issues (20%) — MEDIUM 4. assignee — 65 issues (17%) — MEDIUM 5. priority — 64 issues (17%) — MEDIUM Status Label Distribution: - status:needs-triage: 18 issues - status:needs-more-info: 12 issues - status:needs-review: 3 issues - status:needs-planning: 17 issues - (no status label): 214 issues 🔴 **Handler Priority Recommendations:** P0 (Critical): handle-needs-triage → 18 issues P0 (Critical): Auto-labeling workflow → 214 issues P1 (High): handle-needs-area → 105 issues P2 (Medium): handle-needs-design → 0 issues **Deliverables:** - AUDIT_RESULTS.md — Human-readable report with recommendations - audit-results.csv — Spreadsheet with all 372 issues + gaps - audit-results.json — Machine-readable JSON for tooling **Next Steps:** 1. Review Phase 1.2 issue (#1682) for detailed recommendations 2. Implement handle-needs-triage (Issue #1684) 3. Create auto-labeling workflow to prevent future gaps 4. Expected impact: Status coverage 42% → 90%+ Related: #1679, #1680, #1681, #1682 Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix: Update PR template and README frontmatter validation - Add Linked Issues section with all related GitHub issues - Add Changelog section documenting file changes - Add Global DoD Checklist with all items verified - Add detailed Test Plan section - Fix README frontmatter: use file_type, status:active, authors field - Ensure all PR template required sections are complete Resolves template validation and frontmatter validation failures. Related: #1679, #1692 Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix: Address all Copilot code review comments **Handler Fixes (handle-needs-template-fix.js):** - Fix hasTemplateSections() to require BOTH DoR AND DoD (not just one) - Update usage example to show correct ESM imports - Remove hard-coded owner/repo from API paths; accept as parameters - Pass owner/repo through options to processIssue() **Audit Script Fixes (audit-issue-metadata.js):** - Fix GitHub API query: use labels parameter instead of q parameter - Filter out pull requests from results (API returns both) - Fix pr-link gap key consistency: use 'pr-link' instead of 'prLink' **Test Fixes (handle-needs-template-fix.test.js):** - Update test fixture to include BOTH DoR and DoD headings - Verify logic matches updated hasTemplateSections() **Frontmatter Fixes:** - README.md: use file_type: readme per schema standard - OPENSPEC.md: add file_type, description; use created_date/last_updated - Both files now follow .github/projects/active/ conventions Resolves Copilot review feedback from PR #1692: - Comment 160: hasTemplateSections logic - Comment 89: Test fixture alignment - Comment 15: Usage example format - Comment 271/264: Hard-coded paths - Comment 193: Query parameter usage - Comment 198: PR filtering - Comment 166/270: pr-link gap key - Frontmatter comments: README & OPENSPEC schema Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * docs: Add changelog entry for PR #1692 (Phase 1 issue metadata triage audit) Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix: Correct test expectations for batch statistics in handle-needs-template-fix Issues without both DoR AND DoD sections should all be previewed, not skipped. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix: Address CodeRabbit review comments for audit script and documentation - Add --limit parameter validation to reject invalid values (non-integer, NaN, negative) - Fix pagination logic to check raw page size before filtering PRs - Fail audit with non-zero exit on fetch failures instead of continuing with partial data - Fix UK English spelling: 'analyzing' → 'analysing' in OPENSPEC.md - Fix OpenSpec link in README.md to point directly to OPENSPEC.md - Update handler documentation with UK English: 'behaviour' Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> * fix: Add footer to documentation files for footer validation Add required LightSpeedWP footer to OPENSPEC.md and README.md to pass footer validation checks. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Haiku 4.5 <noreply@anthropic.com>

ashleyshawand others added 3 commits August 9, 2026 13:35

fix: Use v4 version tags for GitHub Actions in badge workflows
d6ecae2
Replace invalid commit SHAs with proper version tags. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

ashleyshaw requested a lite review from Copilot August 9, 2026 12:03

ashleyshaw self-assigned this Aug 9, 2026

Merge branch 'develop' into ci/bulk-issue-fixer-tests
fd47751

Copilotstarted reviewing on behalf of ashleyshaw August 9, 2026 12:04 View session

CopilotAI reviewed Aug 9, 2026
View reviewed changes

ashleyshawand others added 2 commits August 9, 2026 14:09

docs: Add changelog entry for PR#1669(pagination support improvements)
ffea1b5
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

ashleyshaw enabled auto-merge (squash) August 9, 2026 12:16

Merge branch 'develop' into ci/bulk-issue-fixer-tests
8fff28e

ashleyshaw merged commit e61e615 into developAug 9, 2026
19 of 26 checks passed

ashleyshaw deleted the ci/bulk-issue-fixer-tests branch August 9, 2026 12:18

Navigation Menu

Uh oh!

improvement: Enhance bulk issue template fixer with pagination support - #1669

improvement: Enhance bulk issue template fixer with pagination support#1669
ashleyshaw merged 7 commits into
developfrom
ci/bulk-issue-fixer-tests

ashleyshaw commented Aug 9, 2026

Uh oh!

coderabbitaiBot commented Aug 9, 2026•
edited
Loading

Review limit reached

Uh oh!

CopilotAI left a comment

Uh oh!

ashleyshawAug 9, 2026

Uh oh!

ashleyshawAug 9, 2026

Uh oh!

ashleyshawAug 9, 2026

Uh oh!

ashleyshawAug 9, 2026

Uh oh!

ashleyshawAug 9, 2026

Uh oh!

ashleyshawAug 9, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		const startIdx = config.startFrom - 1;
		issuesToProcess = issuesToProcess.slice(startIdx, startIdx + config.limit);

Uh oh!

Conversation

ashleyshaw commented Aug 9, 2026

Summary

Changes

Implementation Details

Testing

Related Issues

Uh oh!

coderabbitaiBot commented Aug 9, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review limit reached

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

ashleyshawAug 9, 2026

Choose a reason for hiding this comment

Uh oh!

ashleyshawAug 9, 2026

Choose a reason for hiding this comment

Uh oh!

ashleyshawAug 9, 2026

Choose a reason for hiding this comment

Uh oh!

ashleyshawAug 9, 2026

Choose a reason for hiding this comment

Uh oh!

ashleyshawAug 9, 2026

Choose a reason for hiding this comment

Uh oh!

ashleyshawAug 9, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

coderabbitaiBot commented Aug 9, 2026•
edited
Loading